From 88f2318a04c7769a4dabec9b2adb5291117166ed Mon Sep 17 00:00:00 2001 From: Parshad Patel Date: Mon, 11 Mar 2019 16:55:06 +0900 Subject: Fix sonar code smell issues in apex-pdp Fix method has more than 7 parameters issues Issue-ID: POLICY-1535 Change-Id: I12d995e323bbdbe3762c903fcb116dfe81bf7b54 Signed-off-by: Parshad Patel --- .../auth/clicodegen/CodeGeneratorCliEditor.java | 49 +++---- .../auth/clicodegen/PolicyStateDefBuilder.java | 149 +++++++++++++++++++++ .../auth/clicodegen/PolicyStateTaskBuilder.java | 106 +++++++++++++++ 3 files changed, 277 insertions(+), 27 deletions(-) create mode 100644 auth/cli-codegen/src/main/java/org/onap/policy/apex/auth/clicodegen/PolicyStateDefBuilder.java create mode 100644 auth/cli-codegen/src/main/java/org/onap/policy/apex/auth/clicodegen/PolicyStateTaskBuilder.java (limited to 'auth/cli-codegen/src/main') diff --git a/auth/cli-codegen/src/main/java/org/onap/policy/apex/auth/clicodegen/CodeGeneratorCliEditor.java b/auth/cli-codegen/src/main/java/org/onap/policy/apex/auth/clicodegen/CodeGeneratorCliEditor.java index 0636bbef6..4c3583e46 100644 --- a/auth/cli-codegen/src/main/java/org/onap/policy/apex/auth/clicodegen/CodeGeneratorCliEditor.java +++ b/auth/cli-codegen/src/main/java/org/onap/policy/apex/auth/clicodegen/CodeGeneratorCliEditor.java @@ -410,18 +410,16 @@ public class CodeGeneratorCliEditor { * @param outputName the output name * @return a CLI command for a policy state task definition */ - public ST createPolicyStateTask(final String policyName, final String version, final String stateName, - final String taskLocalName, final String taskName, final String taskVersion, - final String outputType, final String outputName) { + public ST createPolicyStateTask(PolicyStateTaskBuilder policyStateTaskBuilder) { final ST st = stg.getInstanceOf("policyStateTask"); - st.add(POLICY_NAME, policyName); - st.add(VERSION, version); - st.add(STATE_NAME, stateName); - st.add(TASK_LOCAL_NAME, taskLocalName); - st.add(TASK_NAME, taskName); - st.add(TASK_VERSION, taskVersion); - st.add(OUTPUT_TYPE, outputType); - st.add(OUTPUT_NAME, outputName); + st.add(POLICY_NAME, policyStateTaskBuilder.getPolicyName()); + st.add(VERSION, policyStateTaskBuilder.getVersion()); + st.add(STATE_NAME, policyStateTaskBuilder.getStateName()); + st.add(TASK_LOCAL_NAME, policyStateTaskBuilder.getTaskLocalName()); + st.add(TASK_NAME, policyStateTaskBuilder.getTaskName()); + st.add(TASK_VERSION, policyStateTaskBuilder.getTaskVersion()); + st.add(OUTPUT_TYPE, policyStateTaskBuilder.getOutputType()); + st.add(OUTPUT_NAME, policyStateTaskBuilder.getOutputName()); return st; } @@ -468,23 +466,20 @@ public class CodeGeneratorCliEditor { * @param ctxRefs any context reference for the state * @return a CLI command for a policy state definition */ - public ST createPolicyStateDef(final String policyName, final String version, final String stateName, - final String triggerName, final String triggerVersion, final String defaultTask, - final String defaultTaskVersion, final List outputs, final List tasks, - final List tsLogic, final List finalizerLogics, final List ctxRefs) { + public ST createPolicyStateDef(PolicyStateDefBuilder policyStateDefBuilder) { final ST st = stg.getInstanceOf("policyStateDef"); - st.add(POLICY_NAME, policyName); - st.add(VERSION, version); - st.add(STATE_NAME, stateName); - st.add(TRIGGER_NAME, triggerName); - st.add(TRIGGER_VERSION, triggerVersion); - st.add(DEFAULT_TASK, defaultTask); - st.add(DEFAULT_TASK_VERSION, defaultTaskVersion); - st.add(OUTPUTS, outputs); - st.add(TASKS, tasks); - st.add(TS_LOGIC, tsLogic); - st.add(FINALIZER_LOGICS, finalizerLogics); - st.add(CTX_REFS, ctxRefs); + st.add(POLICY_NAME, policyStateDefBuilder.getPolicyName()); + st.add(VERSION, policyStateDefBuilder.getVersion()); + st.add(STATE_NAME, policyStateDefBuilder.getStateName()); + st.add(TRIGGER_NAME, policyStateDefBuilder.getTriggerName()); + st.add(TRIGGER_VERSION, policyStateDefBuilder.getTriggerVersion()); + st.add(DEFAULT_TASK, policyStateDefBuilder.getDefaultTask()); + st.add(DEFAULT_TASK_VERSION, policyStateDefBuilder.getDefaultTaskVersion()); + st.add(OUTPUTS, policyStateDefBuilder.getOutputs()); + st.add(TASKS, policyStateDefBuilder.getTasks()); + st.add(TS_LOGIC, policyStateDefBuilder.getTsLogic()); + st.add(FINALIZER_LOGICS, policyStateDefBuilder.getFinalizerLogics()); + st.add(CTX_REFS, policyStateDefBuilder.getCtxRefs()); return st; } diff --git a/auth/cli-codegen/src/main/java/org/onap/policy/apex/auth/clicodegen/PolicyStateDefBuilder.java b/auth/cli-codegen/src/main/java/org/onap/policy/apex/auth/clicodegen/PolicyStateDefBuilder.java new file mode 100644 index 000000000..ea5cc6610 --- /dev/null +++ b/auth/cli-codegen/src/main/java/org/onap/policy/apex/auth/clicodegen/PolicyStateDefBuilder.java @@ -0,0 +1,149 @@ +/*- + * ============LICENSE_START======================================================= + * Copyright (C) 2019 Samsung Electronics Co., Ltd. All rights reserved. + * ================================================================================ + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + * + * SPDX-License-Identifier: Apache-2.0 + * ============LICENSE_END========================================================= + */ + +package org.onap.policy.apex.auth.clicodegen; + +import java.util.List; +import org.stringtemplate.v4.ST; + +public class PolicyStateDefBuilder { + + private String policyName; + private String version; + private String stateName; + private String triggerName; + private String triggerVersion; + private String defaultTask; + private String defaultTaskVersion; + private List outputs; + private List tasks; + private List tsLogic; + private List finalizerLogics; + private List ctxRefs; + + + public String getPolicyName() { + return policyName; + } + + public String getVersion() { + return version; + } + + public String getStateName() { + return stateName; + } + + public String getTriggerName() { + return triggerName; + } + + public String getTriggerVersion() { + return triggerVersion; + } + + public String getDefaultTask() { + return defaultTask; + } + + public String getDefaultTaskVersion() { + return defaultTaskVersion; + } + + public List getOutputs() { + return outputs; + } + + public List getTasks() { + return tasks; + } + + public List getTsLogic() { + return tsLogic; + } + + public List getFinalizerLogics() { + return finalizerLogics; + } + + public List getCtxRefs() { + return ctxRefs; + } + + public PolicyStateDefBuilder setPolicyName(String policyName) { + this.policyName = policyName; + return this; + } + + public PolicyStateDefBuilder setVersion(String version) { + this.version = version; + return this; + } + + public PolicyStateDefBuilder setStateName(String stateName) { + this.stateName = stateName; + return this; + } + + public PolicyStateDefBuilder setTriggerName(String triggerName) { + this.triggerName = triggerName; + return this; + } + + public PolicyStateDefBuilder setTriggerVersion(String triggerVersion) { + this.triggerVersion = triggerVersion; + return this; + } + + public PolicyStateDefBuilder setDefaultTask(String defaultTask) { + this.defaultTask = defaultTask; + return this; + } + + public PolicyStateDefBuilder setDefaultTaskVersion(String defaultTaskVersion) { + this.defaultTaskVersion = defaultTaskVersion; + return this; + } + + public PolicyStateDefBuilder setOutputs(List outputs) { + this.outputs = outputs; + return this; + } + + public PolicyStateDefBuilder setTasks(List tasks) { + this.tasks = tasks; + return this; + } + + public PolicyStateDefBuilder setTsLogic(List tsLogic) { + this.tsLogic = tsLogic; + return this; + } + + public PolicyStateDefBuilder setFinalizerLogics(List finalizerLogics) { + this.finalizerLogics = finalizerLogics; + return this; + } + + public PolicyStateDefBuilder setCtxRefs(List ctxRefs) { + this.ctxRefs = ctxRefs; + return this; + } +} diff --git a/auth/cli-codegen/src/main/java/org/onap/policy/apex/auth/clicodegen/PolicyStateTaskBuilder.java b/auth/cli-codegen/src/main/java/org/onap/policy/apex/auth/clicodegen/PolicyStateTaskBuilder.java new file mode 100644 index 000000000..0ac7a3e98 --- /dev/null +++ b/auth/cli-codegen/src/main/java/org/onap/policy/apex/auth/clicodegen/PolicyStateTaskBuilder.java @@ -0,0 +1,106 @@ +/*- + * ============LICENSE_START======================================================= + * Copyright (C) 2019 Samsung Electronics Co., Ltd. All rights reserved. + * ================================================================================ + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + * + * SPDX-License-Identifier: Apache-2.0 + * ============LICENSE_END========================================================= + */ + +package org.onap.policy.apex.auth.clicodegen; + +public class PolicyStateTaskBuilder { + + private String policyName; + private String version; + private String stateName; + private String taskLocalName; + private String taskName; + private String taskVersion; + private String outputType; + private String outputName; + + public String getPolicyName() { + return policyName; + } + + public String getVersion() { + return version; + } + + public String getStateName() { + return stateName; + } + + public String getTaskLocalName() { + return taskLocalName; + } + + public String getTaskName() { + return taskName; + } + + public String getTaskVersion() { + return taskVersion; + } + + public String getOutputType() { + return outputType; + } + + public String getOutputName() { + return outputName; + } + + public PolicyStateTaskBuilder setPolicyName(String policyName) { + this.policyName = policyName; + return this; + } + + public PolicyStateTaskBuilder setVersion(String version) { + this.version = version; + return this; + } + + public PolicyStateTaskBuilder setStateName(String stateName) { + this.stateName = stateName; + return this; + } + + public PolicyStateTaskBuilder setTaskLocalName(String taskLocalName) { + this.taskLocalName = taskLocalName; + return this; + } + + public PolicyStateTaskBuilder setTaskName(String taskName) { + this.taskName = taskName; + return this; + } + + public PolicyStateTaskBuilder setTaskVersion(String taskVersion) { + this.taskVersion = taskVersion; + return this; + } + + public PolicyStateTaskBuilder setOutputType(String outputType) { + this.outputType = outputType; + return this; + } + + public PolicyStateTaskBuilder setOutputName(String outputName) { + this.outputName = outputName; + return this; + } + +} -- cgit 1.2.3-korg