From c04dd4a6984734869ef6888f95d304d3d53c1923 Mon Sep 17 00:00:00 2001 From: ramverma Date: Tue, 6 Aug 2019 14:14:01 +0000 Subject: Changing parameters to use ParameterGroupImpl 1. Changing all the decoders & handlers parameters to use ParameterGroupImpl. 2. Fixed all the test cases. Issue-ID: POLICY-1890 Change-Id: I4a124c0acc650020977ac8e2cee143c72003973a Signed-off-by: ramverma --- ...PolicyForwarderConfigurationParameterGroup.java | 11 - .../main/parameters/CommonTestData.java | 33 +- .../DummyPolicyDecoderParameterGroup.java | 39 +-- .../DummyPolicyForwarderParameterGroup.java | 2 + .../DummyReceptionHandlerParameterGroup.java | 77 +---- .../parameters/DummyPolicyForwarderParameters.json | 8 - ...coderCsarPdpxConfigurationParameterBuilder.java | 94 ------ ...DecoderCsarPdpxConfigurationParameterGroup.java | 85 +---- ...icyDecoderFileInCsarToPolicyParameterGroup.java | 40 +-- .../handling/file/FileSystemReceptionHandler.java | 15 +- ...eptionHandlerConfigurationParameterBuilder.java | 71 ---- ...eceptionHandlerConfigurationParameterGroup.java | 35 +- ...eptionHandlerConfigurationParameterBuilder.java | 357 --------------------- ...eceptionHandlerConfigurationParameterGroup.java | 168 +--------- .../reception/decoding/pdpx/CommonTestData.java | 53 +++ .../decoding/pdpx/TestPolicyDecoderCsarPdpx.java | 25 +- ...DecoderCsarPdpxConfigurationParameterGroup.java | 31 +- ...ecoderFileInCsarToPolicyParameterGroupTest.java | 32 +- .../file/PolicyDecoderFileInCsarToPolicyTest.java | 21 +- ...eceptionHandlerConfigurationParameterGroup.java | 22 +- ...eceptionHandlerConfigurationParameterGroup.java | 77 +---- .../test/resources/parameters/EmptyParameters.json | 2 + .../FileInCsarPolicyDecoderParameters.json | 4 + .../FileInCsarPolicyDecoderParametersInvalid.json | 4 + .../parameters/PdpxPolicyDecoderParameters.json | 8 + .../PdpxPolicyDecoderParametersInvalid.json | 8 + .../PolicyDecoderConfigurationParameterGroup.java | 17 +- ...eceptionHandlerConfigurationParameterGroup.java | 20 +- 28 files changed, 252 insertions(+), 1107 deletions(-) delete mode 100644 main/src/test/resources/parameters/DummyPolicyForwarderParameters.json delete mode 100644 plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/decoding/pdpx/PolicyDecoderCsarPdpxConfigurationParameterBuilder.java delete mode 100644 plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/file/FileSystemReceptionHandlerConfigurationParameterBuilder.java delete mode 100644 plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/sdc/SdcReceptionHandlerConfigurationParameterBuilder.java create mode 100644 plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/pdpx/CommonTestData.java create mode 100644 plugins/reception-plugins/src/test/resources/parameters/EmptyParameters.json create mode 100644 plugins/reception-plugins/src/test/resources/parameters/FileInCsarPolicyDecoderParameters.json create mode 100644 plugins/reception-plugins/src/test/resources/parameters/FileInCsarPolicyDecoderParametersInvalid.json create mode 100644 plugins/reception-plugins/src/test/resources/parameters/PdpxPolicyDecoderParameters.json create mode 100644 plugins/reception-plugins/src/test/resources/parameters/PdpxPolicyDecoderParametersInvalid.json diff --git a/main/src/main/java/org/onap/policy/distribution/main/parameters/PolicyForwarderConfigurationParameterGroup.java b/main/src/main/java/org/onap/policy/distribution/main/parameters/PolicyForwarderConfigurationParameterGroup.java index f8ba764b..41e97616 100644 --- a/main/src/main/java/org/onap/policy/distribution/main/parameters/PolicyForwarderConfigurationParameterGroup.java +++ b/main/src/main/java/org/onap/policy/distribution/main/parameters/PolicyForwarderConfigurationParameterGroup.java @@ -33,15 +33,4 @@ public abstract class PolicyForwarderConfigurationParameterGroup extends Paramet public PolicyForwarderConfigurationParameterGroup(final String name) { super(name); } - - @Override - public String getName() { - return name; - } - - @Override - public void setName(final String name) { - this.name = name; - } - } diff --git a/main/src/test/java/org/onap/policy/distribution/main/parameters/CommonTestData.java b/main/src/test/java/org/onap/policy/distribution/main/parameters/CommonTestData.java index 1e53ed24..9c6d1c10 100644 --- a/main/src/test/java/org/onap/policy/distribution/main/parameters/CommonTestData.java +++ b/main/src/test/java/org/onap/policy/distribution/main/parameters/CommonTestData.java @@ -23,9 +23,7 @@ package org.onap.policy.distribution.main.parameters; import java.io.File; -import java.util.ArrayList; import java.util.HashMap; -import java.util.List; import java.util.Map; import org.onap.policy.common.endpoints.parameters.RestServerParameters; @@ -36,7 +34,6 @@ import org.onap.policy.distribution.forwarding.parameters.PolicyForwarderParamet import org.onap.policy.distribution.main.testclasses.DummyPolicyDecoderParameterGroup; import org.onap.policy.distribution.main.testclasses.DummyPolicyForwarderParameterGroup; import org.onap.policy.distribution.main.testclasses.DummyReceptionHandlerParameterGroup; -import org.onap.policy.distribution.main.testclasses.DummyReceptionHandlerParameterGroup.DummyReceptionHandlerParameterGroupBuilder; import org.onap.policy.distribution.reception.parameters.PluginHandlerParameters; import org.onap.policy.distribution.reception.parameters.PolicyDecoderConfigurationParameterGroup; import org.onap.policy.distribution.reception.parameters.PolicyDecoderParameters; @@ -123,14 +120,11 @@ public class CommonTestData { final Map receptionHandlerConfigurationParameters = new HashMap(); if (!isEmpty) { - final List messageBusAddress = new ArrayList<>(); - messageBusAddress.add("localhost"); - final List artifactTypes = new ArrayList<>(); - artifactTypes.add("TOSCA_CSAR"); final DummyReceptionHandlerParameterGroup dummyReceptionHandlerParameterGroup = - new DummyReceptionHandlerParameterGroupBuilder().setMyStringParameter(MY_STRING_PARAMETER_VALUE) - .setMyIntegerParameter(MY_INTEGER_PARAMETER_VALUE) - .setMyBooleanParameter(MY_BOOLEAN_PARAMETER_VALUE).build(); + new DummyReceptionHandlerParameterGroup(); + dummyReceptionHandlerParameterGroup.setMyStringParameter(MY_STRING_PARAMETER_VALUE); + dummyReceptionHandlerParameterGroup.setMyIntegerParameter(MY_INTEGER_PARAMETER_VALUE); + dummyReceptionHandlerParameterGroup.setMyBooleanParameter(MY_BOOLEAN_PARAMETER_VALUE); receptionHandlerConfigurationParameters.put(RECEPTION_CONFIGURATION_PARAMETERS, dummyReceptionHandlerParameterGroup); } @@ -195,13 +189,14 @@ public class CommonTestData { final Map policyForwarderConfigurationParameters = new HashMap(); if (!isEmpty) { - final String fileName = "src/test/resources/parameters/DummyPolicyForwarderParameters.json"; - try { - policyForwarderConfigurationParameters.put(FORWARDER_CONFIGURATION_PARAMETERS, - coder.decode(new File(fileName), DummyPolicyForwarderParameterGroup.class)); - } catch (final CoderException exp) { - throw new RuntimeException("cannot read/decode " + fileName, exp); - } + final DummyPolicyForwarderParameterGroup parameters = new DummyPolicyForwarderParameterGroup(); + parameters.setHostname(FORWARDER_HOST); + parameters.setManaged(true); + parameters.setUserName("myUser"); + parameters.setPassword("myPassword"); + parameters.setPort(1234); + parameters.setUseHttps(true); + policyForwarderConfigurationParameters.put(FORWARDER_CONFIGURATION_PARAMETERS, parameters); } return policyForwarderConfigurationParameters; } @@ -218,7 +213,9 @@ public class CommonTestData { new HashMap(); if (!isEmpty) { final DummyPolicyDecoderParameterGroup dummyPolicyForwarderParameterGroup = - new DummyPolicyDecoderParameterGroup(POLICY_NAME, POLICY_TYPE); + new DummyPolicyDecoderParameterGroup(); + dummyPolicyForwarderParameterGroup.setPolicyName(POLICY_NAME); + dummyPolicyForwarderParameterGroup.setPolicyType(POLICY_TYPE); policyDecoderConfigurationParameters.put(DECODER_CONFIGURATION_PARAMETERS, dummyPolicyForwarderParameterGroup); } diff --git a/main/src/test/java/org/onap/policy/distribution/main/testclasses/DummyPolicyDecoderParameterGroup.java b/main/src/test/java/org/onap/policy/distribution/main/testclasses/DummyPolicyDecoderParameterGroup.java index 5d1738c5..555169e9 100644 --- a/main/src/test/java/org/onap/policy/distribution/main/testclasses/DummyPolicyDecoderParameterGroup.java +++ b/main/src/test/java/org/onap/policy/distribution/main/testclasses/DummyPolicyDecoderParameterGroup.java @@ -20,7 +20,11 @@ package org.onap.policy.distribution.main.testclasses; -import org.onap.policy.common.parameters.GroupValidationResult; +import lombok.Getter; +import lombok.Setter; + +import org.onap.policy.common.parameters.annotations.NotBlank; +import org.onap.policy.common.parameters.annotations.NotNull; import org.onap.policy.distribution.reception.parameters.PolicyDecoderConfigurationParameterGroup; /** @@ -28,37 +32,16 @@ import org.onap.policy.distribution.reception.parameters.PolicyDecoderConfigurat * * @author Ram Krishna Verma (ram.krishna.verma@ericsson.com) */ +@Getter +@Setter +@NotNull +@NotBlank public class DummyPolicyDecoderParameterGroup extends PolicyDecoderConfigurationParameterGroup { private String policyName; private String policyType; - /** - * Constructor for instantiating the {@link DummyPolicyDecoderParameterGroup} class. - * - * @param policyName the policy name - * @param policyType the policy type - */ - public DummyPolicyDecoderParameterGroup(final String policyName, final String policyType) { - super(); - this.policyName = policyName; - this.policyType = policyType; - } - - - public String getPolicyName() { - return policyName; - } - - - public String getPolicyType() { - return policyType; + public DummyPolicyDecoderParameterGroup() { + super(DummyPolicyDecoderParameterGroup.class.getSimpleName()); } - - @Override - public GroupValidationResult validate() { - final GroupValidationResult validationResult = new GroupValidationResult(this); - return validationResult; - } - } diff --git a/main/src/test/java/org/onap/policy/distribution/main/testclasses/DummyPolicyForwarderParameterGroup.java b/main/src/test/java/org/onap/policy/distribution/main/testclasses/DummyPolicyForwarderParameterGroup.java index 5c7dfcf0..3e44cee3 100644 --- a/main/src/test/java/org/onap/policy/distribution/main/testclasses/DummyPolicyForwarderParameterGroup.java +++ b/main/src/test/java/org/onap/policy/distribution/main/testclasses/DummyPolicyForwarderParameterGroup.java @@ -22,6 +22,7 @@ package org.onap.policy.distribution.main.testclasses; import lombok.Getter; +import lombok.Setter; import org.onap.policy.common.parameters.annotations.Min; import org.onap.policy.common.parameters.annotations.NotBlank; @@ -32,6 +33,7 @@ import org.onap.policy.distribution.main.parameters.PolicyForwarderConfiguration * Dummy policy forwarder parameter group. */ @Getter +@Setter @NotNull @NotBlank public class DummyPolicyForwarderParameterGroup extends PolicyForwarderConfigurationParameterGroup { diff --git a/main/src/test/java/org/onap/policy/distribution/main/testclasses/DummyReceptionHandlerParameterGroup.java b/main/src/test/java/org/onap/policy/distribution/main/testclasses/DummyReceptionHandlerParameterGroup.java index 20f433d0..649d7f4a 100644 --- a/main/src/test/java/org/onap/policy/distribution/main/testclasses/DummyReceptionHandlerParameterGroup.java +++ b/main/src/test/java/org/onap/policy/distribution/main/testclasses/DummyReceptionHandlerParameterGroup.java @@ -1,6 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2018 Ericsson. All rights reserved. + * Modifications Copyright (C) 2019 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -20,79 +21,25 @@ package org.onap.policy.distribution.main.testclasses; -import org.onap.policy.common.parameters.GroupValidationResult; +import lombok.Getter; +import lombok.Setter; + +import org.onap.policy.common.parameters.annotations.NotBlank; +import org.onap.policy.common.parameters.annotations.NotNull; import org.onap.policy.distribution.reception.parameters.ReceptionHandlerConfigurationParameterGroup; +@Getter +@Setter +@NotNull +@NotBlank public class DummyReceptionHandlerParameterGroup extends ReceptionHandlerConfigurationParameterGroup { private String myStringParameter; private int myIntegerParameter; private boolean myBooleanParameter; - /** - * Inner static class is to used as a Builder. - * - */ - public static class DummyReceptionHandlerParameterGroupBuilder { - private String myStringParameter; - private int myIntegerParameter; - private boolean myBooleanParameter; - - public DummyReceptionHandlerParameterGroupBuilder setMyStringParameter(final String val) { - myStringParameter = val; - return this; - } - - public DummyReceptionHandlerParameterGroupBuilder setMyIntegerParameter(final int val) { - myIntegerParameter = val; - return this; - } - - public DummyReceptionHandlerParameterGroupBuilder setMyBooleanParameter(final boolean val) { - myBooleanParameter = val; - return this; - } - - /** - * Creates a new DummyReceptionHandlerConfigurationParameterGroup instance. - */ - public DummyReceptionHandlerParameterGroup build() { - return new DummyReceptionHandlerParameterGroup(this); - } - } - - /** - * The constructor for instantiating PssdConfigurationParameterGroup. It is kept private so that - * it could only be called by PssdConfigurationBuilder. - * - * @param builder stores all the values used by PssdConfigurationParametersGroup - */ - private DummyReceptionHandlerParameterGroup(final DummyReceptionHandlerParameterGroupBuilder builder) { - myStringParameter = builder.myStringParameter; - myIntegerParameter = builder.myIntegerParameter; - myBooleanParameter = builder.myBooleanParameter; - } - - public String getMyStringParameter() { - return myStringParameter; + public DummyReceptionHandlerParameterGroup() { + super(DummyReceptionHandlerParameterGroup.class.getSimpleName()); } - - public int getMyIntegerParameter() { - return myIntegerParameter; - } - - public boolean isMyBooleanParameter() { - return myBooleanParameter; - } - - - /** - * {@inheritDoc}. - */ - @Override - public GroupValidationResult validate() { - return new GroupValidationResult(this); - } - } diff --git a/main/src/test/resources/parameters/DummyPolicyForwarderParameters.json b/main/src/test/resources/parameters/DummyPolicyForwarderParameters.json deleted file mode 100644 index 60b0d4af..00000000 --- a/main/src/test/resources/parameters/DummyPolicyForwarderParameters.json +++ /dev/null @@ -1,8 +0,0 @@ -{ - "useHttps": true, - "hostname": "192.168.99.100", - "port": 1234, - "userName": "myUser", - "password": "myPassword", - "isManaged": true -} \ No newline at end of file diff --git a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/decoding/pdpx/PolicyDecoderCsarPdpxConfigurationParameterBuilder.java b/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/decoding/pdpx/PolicyDecoderCsarPdpxConfigurationParameterBuilder.java deleted file mode 100644 index 4c781030..00000000 --- a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/decoding/pdpx/PolicyDecoderCsarPdpxConfigurationParameterBuilder.java +++ /dev/null @@ -1,94 +0,0 @@ -/*- - * ============LICENSE_START======================================================= - * Copyright (C) 2018 Ericsson. 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.distribution.reception.decoding.pdpx; - -/** - * This class builds an instance of {@link PolicyDecodeCsarPdpxConfigurationParameterGroup} class. - */ -public class PolicyDecoderCsarPdpxConfigurationParameterBuilder { - - private String policyNamePrefix; - private String onapName; - private String version; - private String priority; - private String riskType; - private String riskLevel; - - public String getPolicyNamePrefix() { - return policyNamePrefix; - } - - public PolicyDecoderCsarPdpxConfigurationParameterBuilder setPolicyNamePrefix(final String policyNamePrefix) { - this.policyNamePrefix = policyNamePrefix; - return this; - } - - public String getOnapName() { - return onapName; - } - - public PolicyDecoderCsarPdpxConfigurationParameterBuilder setOnapName(final String onapName) { - this.onapName = onapName; - return this; - } - - public String getVersion() { - return version; - } - - public PolicyDecoderCsarPdpxConfigurationParameterBuilder setVersion(final String version) { - this.version = version; - return this; - } - - public String getPriority() { - return priority; - } - - public PolicyDecoderCsarPdpxConfigurationParameterBuilder setPriority(final String priority) { - this.priority = priority; - return this; - } - - public String getRiskType() { - return riskType; - } - - public PolicyDecoderCsarPdpxConfigurationParameterBuilder setRiskType(final String riskType) { - this.riskType = riskType; - return this; - } - - public String getRiskLevel() { - return riskLevel; - } - - public PolicyDecoderCsarPdpxConfigurationParameterBuilder setRiskLevel(final String riskLevel) { - this.riskLevel = riskLevel; - return this; - } - - public PolicyDecoderCsarPdpxConfigurationParameterGroup build() { - return new PolicyDecoderCsarPdpxConfigurationParameterGroup(this); - } -} - - diff --git a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/decoding/pdpx/PolicyDecoderCsarPdpxConfigurationParameterGroup.java b/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/decoding/pdpx/PolicyDecoderCsarPdpxConfigurationParameterGroup.java index 26fd2a5a..f376c391 100644 --- a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/decoding/pdpx/PolicyDecoderCsarPdpxConfigurationParameterGroup.java +++ b/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/decoding/pdpx/PolicyDecoderCsarPdpxConfigurationParameterGroup.java @@ -1,6 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2018 Intel. All rights reserved. + * Modifications Copyright (C) 2019 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -20,16 +21,18 @@ package org.onap.policy.distribution.reception.decoding.pdpx; -import org.onap.policy.common.parameters.GroupValidationResult; -import org.onap.policy.common.parameters.ValidationStatus; -import org.onap.policy.common.utils.validation.ParameterValidationUtils; -import org.onap.policy.distribution.reception.handling.sdc.SdcReceptionHandlerConfigurationParameterGroup; +import lombok.Getter; + +import org.onap.policy.common.parameters.annotations.NotBlank; +import org.onap.policy.common.parameters.annotations.NotNull; import org.onap.policy.distribution.reception.parameters.PolicyDecoderConfigurationParameterGroup; /** - * This class handles reading, parsing and validating of the paramaters for the - * {@link PolicyDecoderCsarPdpx}. + * This class handles reading, parsing and validating of the paramaters for the {@link PolicyDecoderCsarPdpx}. */ +@Getter +@NotNull +@NotBlank public class PolicyDecoderCsarPdpxConfigurationParameterGroup extends PolicyDecoderConfigurationParameterGroup { private String policyNamePrefix; @@ -39,74 +42,8 @@ public class PolicyDecoderCsarPdpxConfigurationParameterGroup extends PolicyDeco private String riskType; private String riskLevel; - /** - * The constructor for instantiating {@link SdcReceptionHandlerConfigurationParameterGroup} - * class. - * - * @param builder the SDC configuration builder - */ - public PolicyDecoderCsarPdpxConfigurationParameterGroup( - final PolicyDecoderCsarPdpxConfigurationParameterBuilder builder) { - policyNamePrefix = builder.getPolicyNamePrefix(); - onapName = builder.getOnapName(); - version = builder.getVersion(); - priority = builder.getPriority(); - riskType = builder.getRiskType(); - riskLevel = builder.getRiskLevel(); - } - - public String getPolicyNamePrefix() { - return policyNamePrefix; - } - - public String getOnapName() { - return onapName; - } - - public String getVersion() { - return version; - } - - public String getPriority() { - return priority; - } - - public String getRiskType() { - return riskType; - } - - public String getRiskLevel() { - return riskLevel; - } - - /** - * {@inheritDoc}. - */ - @Override - public GroupValidationResult validate() { - final GroupValidationResult validationResult = new GroupValidationResult(this); - validateStringElement(validationResult, policyNamePrefix, "policyNamePrefix"); - validateStringElement(validationResult, onapName, "onapName"); - validateStringElement(validationResult, version, "version"); - validateStringElement(validationResult, priority, "priority"); - validateStringElement(validationResult, riskType, "riskType"); - validateStringElement(validationResult, riskLevel, "riskLevel"); - return validationResult; - } - - /** - * Validate the string element. - * - * @param validationResult the result object - * @param element the element to validate - * @param elementName the element name for error message - */ - private void validateStringElement(final GroupValidationResult validationResult, final String element, - final String elementName) { - if (!ParameterValidationUtils.validateStringParameter(element)) { - validationResult.setResult(elementName, ValidationStatus.INVALID, - elementName + " must be a non-blank string"); - } + public PolicyDecoderCsarPdpxConfigurationParameterGroup() { + super(PolicyDecoderCsarPdpxConfigurationParameterGroup.class.getSimpleName()); } } diff --git a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/decoding/policy/file/PolicyDecoderFileInCsarToPolicyParameterGroup.java b/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/decoding/policy/file/PolicyDecoderFileInCsarToPolicyParameterGroup.java index 1df563c3..7aff5b4c 100644 --- a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/decoding/policy/file/PolicyDecoderFileInCsarToPolicyParameterGroup.java +++ b/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/decoding/policy/file/PolicyDecoderFileInCsarToPolicyParameterGroup.java @@ -1,6 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2018 Ericsson. All rights reserved. + * Modifications Copyright (C) 2019 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -20,9 +21,10 @@ package org.onap.policy.distribution.reception.decoding.policy.file; -import org.onap.policy.common.parameters.GroupValidationResult; -import org.onap.policy.common.parameters.ValidationStatus; -import org.onap.policy.common.utils.validation.ParameterValidationUtils; +import lombok.Getter; + +import org.onap.policy.common.parameters.annotations.NotBlank; +import org.onap.policy.common.parameters.annotations.NotNull; import org.onap.policy.distribution.reception.parameters.PolicyDecoderConfigurationParameterGroup; /** @@ -30,37 +32,15 @@ import org.onap.policy.distribution.reception.parameters.PolicyDecoderConfigurat * * @author Ram Krishna Verma (ram.krishna.verma@ericsson.com) */ +@Getter +@NotNull +@NotBlank public class PolicyDecoderFileInCsarToPolicyParameterGroup extends PolicyDecoderConfigurationParameterGroup { private String policyFileName; private String policyTypeFileName; - /** - * Constructor for instantiating {@link PolicyDecoderFileInCsarToPolicyParameterGroup} class. - * - * @param policyFileName the policy file name - * @param policyTypeFileName the policy type file name - */ - public PolicyDecoderFileInCsarToPolicyParameterGroup(final String policyFileName, final String policyTypeFileName) { - this.policyFileName = policyFileName; - this.policyTypeFileName = policyTypeFileName; - } - - public String getPolicyFileName() { - return policyFileName; - } - - public String getPolicyTypeFileName() { - return policyTypeFileName; - } - - @Override - public GroupValidationResult validate() { - final GroupValidationResult validationResult = new GroupValidationResult(this); - if (!ParameterValidationUtils.validateStringParameter(policyFileName)) { - validationResult.setResult("policyFileName", ValidationStatus.INVALID, - "must be a non-blank string containing the policy file name"); - } - return validationResult; + public PolicyDecoderFileInCsarToPolicyParameterGroup() { + super(PolicyDecoderFileInCsarToPolicyParameterGroup.class.getSimpleName()); } } diff --git a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/file/FileSystemReceptionHandler.java b/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/file/FileSystemReceptionHandler.java index 77269bb0..47312e62 100644 --- a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/file/FileSystemReceptionHandler.java +++ b/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/file/FileSystemReceptionHandler.java @@ -62,9 +62,8 @@ public class FileSystemReceptionHandler extends AbstractReceptionHandler { try { final FileSystemReceptionHandlerConfigurationParameterGroup handlerParameters = ParameterService.get(parameterGroupName); - final FileClientHandler fileClientHandler = new FileClientHandler(this, - handlerParameters.getWatchPath(), - handlerParameters.getMaxThread()); + final FileClientHandler fileClientHandler = + new FileClientHandler(this, handlerParameters.getWatchPath(), handlerParameters.getMaxThread()); final Thread fileWatcherThread = new Thread(fileClientHandler); fileWatcherThread.start(); } catch (final Exception ex) { @@ -114,10 +113,10 @@ public class FileSystemReceptionHandler extends AbstractReceptionHandler { * @param maxThread the max thread number * @throws InterruptedException if it occurs */ - protected void startWatchService(final WatchService watcher, - final Path dir, int maxThread) throws InterruptedException { + protected void startWatchService(final WatchService watcher, final Path dir, final int maxThread) + throws InterruptedException { WatchKey key; - ExecutorService pool = Executors.newFixedThreadPool(maxThread); + final ExecutorService pool = Executors.newFixedThreadPool(maxThread); try { running = true; @@ -135,7 +134,7 @@ public class FileSystemReceptionHandler extends AbstractReceptionHandler { } } - private void processFileEvents(Path dir, WatchKey key, ExecutorService pool) { + private void processFileEvents(final Path dir, final WatchKey key, final ExecutorService pool) { for (final WatchEvent event : key.pollEvents()) { @SuppressWarnings("unchecked") final WatchEvent ev = (WatchEvent) event; @@ -148,7 +147,7 @@ public class FileSystemReceptionHandler extends AbstractReceptionHandler { waitForFileToBeReady(fullFilePath); createPolicyInputAndCallHandler(fullFilePath); LOGGER.debug("CSAR complete: {}", fileName); - } catch (InterruptedException e) { + } catch (final InterruptedException e) { LOGGER.error("waitForFileToBeReady interrupted", e); Thread.currentThread().interrupt(); } diff --git a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/file/FileSystemReceptionHandlerConfigurationParameterBuilder.java b/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/file/FileSystemReceptionHandlerConfigurationParameterBuilder.java deleted file mode 100644 index 415e0d94..00000000 --- a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/file/FileSystemReceptionHandlerConfigurationParameterBuilder.java +++ /dev/null @@ -1,71 +0,0 @@ -/*- - * ============LICENSE_START======================================================= - * Copyright (C) 2018 Intel Corp. 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.distribution.reception.handling.file; - -/** - * This class builds an instance of {@link FileSystemReceptionHandlerConfigurationParameterGroup} class. - * - */ -public class FileSystemReceptionHandlerConfigurationParameterBuilder { - - private String watchPath; - private int maxThread = 1; - - /** - * Set watchPath to this {@link FileSystemReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param watchPath the watchPath - */ - public FileSystemReceptionHandlerConfigurationParameterBuilder setWatchPath(final String watchPath) { - this.watchPath = watchPath; - return this; - } - - /** - * Set maxThread to this {@link FileSystemReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param maxThread the max thread number in the thread pool - */ - public FileSystemReceptionHandlerConfigurationParameterBuilder setMaxThread(final int maxThread) { - this.maxThread = maxThread; - return this; - } - - /** - * Returns the watchPath of this {@link FileSystemReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the watchPath - */ - public String getWatchPath() { - return watchPath; - } - - /** - * Returns the maxThread of this {@link FileSystemReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the maxThread - */ - public int getMaxThread() { - return maxThread; - } -} - - diff --git a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/file/FileSystemReceptionHandlerConfigurationParameterGroup.java b/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/file/FileSystemReceptionHandlerConfigurationParameterGroup.java index f9045816..a12130ea 100644 --- a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/file/FileSystemReceptionHandlerConfigurationParameterGroup.java +++ b/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/file/FileSystemReceptionHandlerConfigurationParameterGroup.java @@ -1,6 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2018 Intel. All rights reserved. + * Modifications Copyright (C) 2019 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -22,37 +23,30 @@ package org.onap.policy.distribution.reception.handling.file; import java.io.File; +import lombok.Getter; +import lombok.Setter; + import org.onap.policy.common.parameters.GroupValidationResult; import org.onap.policy.common.parameters.ValidationStatus; -import org.onap.policy.common.utils.validation.ParameterValidationUtils; +import org.onap.policy.common.parameters.annotations.NotBlank; +import org.onap.policy.common.parameters.annotations.NotNull; import org.onap.policy.distribution.reception.parameters.ReceptionHandlerConfigurationParameterGroup; /** * This class handles reading, parsing and validating of the Policy SDC Service Distribution parameters from Json * format, which strictly adheres to the interface:IConfiguration, defined by SDC SDK. */ +@Getter +@Setter +@NotNull +@NotBlank public class FileSystemReceptionHandlerConfigurationParameterGroup extends ReceptionHandlerConfigurationParameterGroup { private String watchPath; private int maxThread; - /** - * The constructor for instantiating {@link FileSystemReceptionHandlerConfigurationParameterGroup} class. - * - * @param builder the SDC configuration builder - */ - public FileSystemReceptionHandlerConfigurationParameterGroup( - final FileSystemReceptionHandlerConfigurationParameterBuilder builder) { - watchPath = builder.getWatchPath(); - maxThread = builder.getMaxThread(); - } - - public String getWatchPath() { - return watchPath; - } - - public int getMaxThread() { - return maxThread; + public FileSystemReceptionHandlerConfigurationParameterGroup() { + super(FileSystemReceptionHandlerConfigurationParameterGroup.class.getSimpleName()); } /** @@ -62,9 +56,6 @@ public class FileSystemReceptionHandlerConfigurationParameterGroup extends Recep public GroupValidationResult validate() { final GroupValidationResult validationResult = new GroupValidationResult(this); validatePathElement(validationResult, watchPath, "watchPath"); - if (!ParameterValidationUtils.validateIntParameter(maxThread)) { - validationResult.setResult("maxThread", ValidationStatus.INVALID, "must be a positive integer"); - } return validationResult; } @@ -80,7 +71,7 @@ public class FileSystemReceptionHandlerConfigurationParameterGroup extends Recep final String elementName) { boolean valid = false; if (element != null) { - File file = new File(element); + final File file = new File(element); if (file.exists() && file.isDirectory()) { valid = true; } diff --git a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/sdc/SdcReceptionHandlerConfigurationParameterBuilder.java b/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/sdc/SdcReceptionHandlerConfigurationParameterBuilder.java deleted file mode 100644 index 65305c1c..00000000 --- a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/sdc/SdcReceptionHandlerConfigurationParameterBuilder.java +++ /dev/null @@ -1,357 +0,0 @@ -/*- - * ============LICENSE_START======================================================= - * Copyright (C) 2018 Ericsson. 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.distribution.reception.handling.sdc; - -import java.util.List; - -/** - * This class builds an instance of {@link SdcReceptionHandlerConfigurationParameterGroup} class. - * - * @author Ram Krishna Verma (ram.krishna.verma@ericsson.com) - */ -public class SdcReceptionHandlerConfigurationParameterBuilder { - - private boolean activeserverTlsAuth; - private boolean filterinEmptyResources; - private boolean useHttpsWithDmaap; - private int pollingTimeout; - private int pollingInterval; - private String user; - private String password; - private String consumerId; - private String consumerGroup; - private String asdcAddress; - private String environmentName; - private String keystorePath; - private String keystorePassword; - private List messageBusAddress; - private List artifactTypes; - private int retryDelay; - - /** - * Set activeserverTlsAuth to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param activeserverTlsAuth the activeserverTlsAuth - */ - public SdcReceptionHandlerConfigurationParameterBuilder setActiveserverTlsAuth(final boolean activeserverTlsAuth) { - this.activeserverTlsAuth = activeserverTlsAuth; - return this; - } - - /** - * Set filterinEmptyResources to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param filterinEmptyResources the filterinEmptyResources - */ - public SdcReceptionHandlerConfigurationParameterBuilder setFilterinEmptyResources( - final boolean filterinEmptyResources) { - this.filterinEmptyResources = filterinEmptyResources; - return this; - } - - /** - * Set useHttpsWithDmaap to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param useHttpsWithDmaap the useHttpsWithDmaap - */ - public SdcReceptionHandlerConfigurationParameterBuilder setUseHttpsWithDmaap(final Boolean useHttpsWithDmaap) { - this.useHttpsWithDmaap = useHttpsWithDmaap; - return this; - } - - /** - * Set pollingInterval to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param pollingInterval the pollingInterval - */ - public SdcReceptionHandlerConfigurationParameterBuilder setPollingInterval(final int pollingInterval) { - this.pollingInterval = pollingInterval; - return this; - } - - /** - * Set pollingTimeout to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param pollingTimeout the pollingTimeout - */ - public SdcReceptionHandlerConfigurationParameterBuilder setPollingTimeout(final int pollingTimeout) { - this.pollingTimeout = pollingTimeout; - return this; - } - - /** - * Set asdcAddress to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param asdcAddress the asdcAddress - */ - public SdcReceptionHandlerConfigurationParameterBuilder setAsdcAddress(final String asdcAddress) { - this.asdcAddress = asdcAddress; - return this; - } - - /** - * Set user to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param user the user - */ - public SdcReceptionHandlerConfigurationParameterBuilder setUser(final String user) { - this.user = user; - return this; - } - - /** - * Set password to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param password the password - */ - public SdcReceptionHandlerConfigurationParameterBuilder setPassword(final String password) { - this.password = password; - return this; - } - - /** - * Set consumerId to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param consumerId the consumerId - */ - public SdcReceptionHandlerConfigurationParameterBuilder setConsumerId(final String consumerId) { - this.consumerId = consumerId; - return this; - } - - /** - * Set consumerGroup to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param consumerGroup the consumerGroup - */ - public SdcReceptionHandlerConfigurationParameterBuilder setConsumerGroup(final String consumerGroup) { - this.consumerGroup = consumerGroup; - return this; - } - - /** - * Set environmentName to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param environmentName the environmentName - */ - public SdcReceptionHandlerConfigurationParameterBuilder setEnvironmentName(final String environmentName) { - this.environmentName = environmentName; - return this; - } - - /** - * Set keystorePath to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param keystorePath the keystorePath - */ - public SdcReceptionHandlerConfigurationParameterBuilder setKeystorePath(final String keystorePath) { - this.keystorePath = keystorePath; - return this; - } - - /** - * Set keystorePassword to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param keystorePassword the keystorePassword - */ - public SdcReceptionHandlerConfigurationParameterBuilder setKeystorePassword(final String keystorePassword) { - this.keystorePassword = keystorePassword; - return this; - } - - /** - * Set messageBusAddress to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param messageBusAddress the messageBusAddress - */ - public SdcReceptionHandlerConfigurationParameterBuilder setMessageBusAddress(final List messageBusAddress) { - this.messageBusAddress = messageBusAddress; - return this; - } - - /** - * Set artifactTypes to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param artifactTypes the artifactTypes - */ - public SdcReceptionHandlerConfigurationParameterBuilder setArtifactTypes(final List artifactTypes) { - this.artifactTypes = artifactTypes; - return this; - } - - /** - * Set retryDelay to this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @param retryDelay the retryDelay - */ - public SdcReceptionHandlerConfigurationParameterBuilder setRetryDelay(final int retryDelay) { - this.retryDelay = retryDelay; - return this; - } - - /** - * Returns the active server TlsAuth of this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the activeserverTlsAuth - */ - public boolean isActiveserverTlsAuth() { - return activeserverTlsAuth; - } - - /** - * Returns the isFilterinEmptyResources flag of this {@link SdcReceptionHandlerConfigurationParameterBuilder} - * instance. - * - * @return the isFilterinEmptyResources - */ - public boolean isFilterinEmptyResources() { - return filterinEmptyResources; - } - - /** - * Returns the isUseHttpsWithDmaap flag of this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the isUseHttpsWithDmaap - */ - public Boolean getIsUseHttpsWithDmaap() { - return useHttpsWithDmaap; - } - - /** - * Returns the polling interval of this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the pollingInterval - */ - public int getPollingInterval() { - return pollingInterval; - } - - /** - * Returns the polling timeout of this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the pollingTimeout - */ - public int getPollingTimeout() { - return pollingTimeout; - } - - /** - * Returns the asdc address of this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the asdcAddress - */ - public String getAsdcAddress() { - return asdcAddress; - } - - /** - * Returns the user of this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the user - */ - public String getUser() { - return user; - } - - /** - * Returns the password of this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the password - */ - public String getPassword() { - return password; - } - - /** - * Returns the consumer id of this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the consumerId - */ - public String getConsumerId() { - return consumerId; - } - - /** - * Returns the consumer group of this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the consumerGroup - */ - public String getConsumerGroup() { - return consumerGroup; - } - - /** - * Returns the environment name of this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the environmentName - */ - public String getEnvironmentName() { - return environmentName; - } - - /** - * Returns the keystore path of this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the keystorePath - */ - public String getKeystorePath() { - return keystorePath; - } - - /** - * Returns the keystore password of this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the keystorePassword - */ - public String getKeystorePassword() { - return keystorePassword; - } - - /** - * Returns the message bus address of this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the messageBusAddress - */ - public List getMessageBusAddress() { - return messageBusAddress; - } - - /** - * Returns the artifact types of this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the artifactTypes - */ - public List getArtifactTypes() { - return artifactTypes; - } - - /** - * Returns the retryDelay of this {@link SdcReceptionHandlerConfigurationParameterBuilder} instance. - * - * @return the retryDelay - */ - public int getRetryDelay() { - return retryDelay; - } - -} - - diff --git a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/sdc/SdcReceptionHandlerConfigurationParameterGroup.java b/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/sdc/SdcReceptionHandlerConfigurationParameterGroup.java index 6746dc35..f9aed90b 100644 --- a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/sdc/SdcReceptionHandlerConfigurationParameterGroup.java +++ b/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/sdc/SdcReceptionHandlerConfigurationParameterGroup.java @@ -1,6 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2018 Intel. All rights reserved. + * Modifications Copyright (C) 2019 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -22,15 +23,19 @@ package org.onap.policy.distribution.reception.handling.sdc; import java.util.List; -import org.onap.policy.common.parameters.GroupValidationResult; -import org.onap.policy.common.parameters.ValidationStatus; -import org.onap.policy.common.utils.validation.ParameterValidationUtils; +import lombok.Getter; + +import org.onap.policy.common.parameters.annotations.NotBlank; +import org.onap.policy.common.parameters.annotations.NotNull; import org.onap.policy.distribution.reception.parameters.ReceptionHandlerConfigurationParameterGroup; /** * This class handles reading, parsing and validating of the Policy SDC Service Distribution parameters from Json * format, which strictly adheres to the interface:IConfiguration, defined by SDC SDK. */ +@Getter +@NotNull +@NotBlank public class SdcReceptionHandlerConfigurationParameterGroup extends ReceptionHandlerConfigurationParameterGroup { private String asdcAddress; @@ -50,161 +55,8 @@ public class SdcReceptionHandlerConfigurationParameterGroup extends ReceptionHan private boolean isFilterInEmptyResources; private boolean isUseHttpsWithDmaap; - /** - * The constructor for instantiating {@link SdcReceptionHandlerConfigurationParameterGroup} class. - * - * @param builder the SDC configuration builder - */ - public SdcReceptionHandlerConfigurationParameterGroup( - final SdcReceptionHandlerConfigurationParameterBuilder builder) { - asdcAddress = builder.getAsdcAddress(); - messageBusAddress = builder.getMessageBusAddress(); - user = builder.getUser(); - password = builder.getPassword(); - pollingInterval = builder.getPollingInterval(); - pollingTimeout = builder.getPollingTimeout(); - retryDelay = builder.getRetryDelay(); - consumerId = builder.getConsumerId(); - artifactTypes = builder.getArtifactTypes(); - consumerGroup = builder.getConsumerGroup(); - environmentName = builder.getEnvironmentName(); - keyStorePath = builder.getKeystorePath(); - keyStorePassword = builder.getKeystorePassword(); - activeServerTlsAuth = builder.isActiveserverTlsAuth(); - isFilterInEmptyResources = builder.isFilterinEmptyResources(); - isUseHttpsWithDmaap = builder.getIsUseHttpsWithDmaap(); - - } - - public String getAsdcAddress() { - return asdcAddress; - } - - public List getMessageBusAddress() { - return messageBusAddress; - } - - public String getUser() { - return user; - } - - public String getPassword() { - return password; - } - - public int getPollingInterval() { - return pollingInterval; - } - - public int getPollingTimeout() { - return pollingTimeout; - } - - public int getRetryDelay() { - return retryDelay; - } - - public String getConsumerId() { - return consumerId; - } - - public List getArtifactTypes() { - return artifactTypes; - } - - public String getConsumerGroup() { - return consumerGroup; - } - - public String getEnvironmentName() { - return environmentName; - } - - public String getKeyStorePassword() { - return keyStorePassword; - } - - public boolean isActiveServerTlsAuth() { - return activeServerTlsAuth; - } - - public String getKeyStorePath() { - return keyStorePath; - } - - public boolean isFilterInEmptyResources() { - return isFilterInEmptyResources; - } - - public boolean isUseHttpsWithDmaap() { - return isUseHttpsWithDmaap; - } - - /** - * {@inheritDoc}. - */ - @Override - public GroupValidationResult validate() { - final GroupValidationResult validationResult = new GroupValidationResult(this); - validateStringElement(validationResult, asdcAddress, "asdcAddress"); - validateStringElement(validationResult, user, "user"); - validateStringElement(validationResult, consumerId, "consumerId"); - validateStringElement(validationResult, consumerGroup, "consumerGroup"); - validateStringElement(validationResult, keyStorePath, "keyStorePath"); - validateStringElement(validationResult, keyStorePassword, "keyStorePassword"); - validateIntElement(validationResult, pollingInterval, "pollingInterval"); - validateIntElement(validationResult, pollingTimeout, "pollingTimeout"); - validateIntElement(validationResult, retryDelay, "retryDelay"); - validateStringListElement(validationResult, messageBusAddress, "messageBusAddress"); - validateStringListElement(validationResult, artifactTypes, "artifactTypes"); - return validationResult; - } - - /** - * Validate the integer Element. - * - * @param validationResult the result object - * @param element the element to validate - * @param elementName the element name for error message - */ - private void validateIntElement(final GroupValidationResult validationResult, final int element, - final String elementName) { - if (!ParameterValidationUtils.validateIntParameter(element)) { - validationResult.setResult(elementName, ValidationStatus.INVALID, - elementName + " must be a positive integer"); - } - } - - /** - * Validate the String List Element. - * - * @param validationResult the result object - * @param element the element to validate - * @param elementName the element name for error message - */ - private void validateStringListElement(final GroupValidationResult validationResult, final List element, - final String elementName) { - for (final String temp : element) { - if (!ParameterValidationUtils.validateStringParameter(temp)) { - validationResult.setResult(elementName, ValidationStatus.INVALID, - "the string of " + elementName + "must be a non-blank string"); - } - } - } - - /** - * Validate the string element. - * - * @param validationResult the result object - * @param element the element to validate - * @param elementName the element name for error message - */ - private void validateStringElement(final GroupValidationResult validationResult, final String element, - final String elementName) { - if (!ParameterValidationUtils.validateStringParameter(element)) { - validationResult.setResult(elementName, ValidationStatus.INVALID, - elementName + " must be a non-blank string"); - } + public SdcReceptionHandlerConfigurationParameterGroup() { + super(SdcReceptionHandlerConfigurationParameterGroup.class.getSimpleName()); } } diff --git a/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/pdpx/CommonTestData.java b/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/pdpx/CommonTestData.java new file mode 100644 index 00000000..5006e078 --- /dev/null +++ b/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/pdpx/CommonTestData.java @@ -0,0 +1,53 @@ +/*- + * ============LICENSE_START======================================================= + * Copyright (C) 2019 Nordix Foundation. + * ================================================================================ + * 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.distribution.reception.decoding.pdpx; + +import java.io.File; + +import org.onap.policy.common.utils.coder.Coder; +import org.onap.policy.common.utils.coder.CoderException; +import org.onap.policy.common.utils.coder.StandardCoder; + +/** + * Class to create parameters for test cases. + * + * @author Ram Krishna Verma (ram.krishna.verma@est.tech) + */ +public class CommonTestData { + + public static final Coder coder = new StandardCoder(); + + /** + * Returns PolicyDecoderParameters for test cases. + * + * @param fileName the file name to load the parameters + * @param clazz the parameter class to be returned + * @return the specific PolicyDecoderParameters object + */ + public static T getPolicyDecoderParameters(final String fileName, final Class clazz) { + final StandardCoder coder = new StandardCoder(); + try { + return coder.decode(new File(fileName), clazz); + } catch (final CoderException exp) { + throw new RuntimeException("cannot read/decode " + fileName, exp); + } + } +} diff --git a/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/pdpx/TestPolicyDecoderCsarPdpx.java b/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/pdpx/TestPolicyDecoderCsarPdpx.java index 0cd8b6f4..9cfbaf80 100644 --- a/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/pdpx/TestPolicyDecoderCsarPdpx.java +++ b/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/pdpx/TestPolicyDecoderCsarPdpx.java @@ -1,6 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2018 Intel. All rights reserved. + * Modifications Copyright (C) 2019 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -41,17 +42,15 @@ import org.onap.policy.models.tosca.authorative.concepts.ToscaServiceTemplate; */ public class TestPolicyDecoderCsarPdpx { - private static final String CSAR_TO_OPTIMIZATION_POLICY_CONFIGURATION = "csarToOptimizationPolicyConfiguration"; - /** * Set up for test cases. */ @BeforeClass public static void setUp() { - final ParameterGroup parameterGroup = new PolicyDecoderCsarPdpxConfigurationParameterBuilder() - .setOnapName("onapName").setPolicyNamePrefix("OOF").setPriority("5").setRiskLevel("2") - .setRiskType("Test").setVersion("1.0").build(); - parameterGroup.setName(CSAR_TO_OPTIMIZATION_POLICY_CONFIGURATION); + final ParameterGroup parameterGroup = CommonTestData.getPolicyDecoderParameters( + "src/test/resources/parameters/PdpxPolicyDecoderParameters.json", + PolicyDecoderCsarPdpxConfigurationParameterGroup.class); + parameterGroup.setName(PolicyDecoderCsarPdpxConfigurationParameterGroup.class.getSimpleName()); ParameterService.register(parameterGroup); } @@ -60,14 +59,14 @@ public class TestPolicyDecoderCsarPdpx { */ @AfterClass public static void tearDown() { - ParameterService.deregister(CSAR_TO_OPTIMIZATION_POLICY_CONFIGURATION); + ParameterService.deregister(PolicyDecoderCsarPdpxConfigurationParameterGroup.class.getSimpleName()); } @Test public void testHpaPolicy2Vnf() throws IOException, PolicyDecodingException { final Csar csar = new Csar("src/test/resources/service-TestNs8-csar.csar"); final PolicyDecoderCsarPdpx policyDecoderCsarPdpx = new PolicyDecoderCsarPdpx(); - policyDecoderCsarPdpx.configure(CSAR_TO_OPTIMIZATION_POLICY_CONFIGURATION); + policyDecoderCsarPdpx.configure(PolicyDecoderCsarPdpxConfigurationParameterGroup.class.getSimpleName()); final Collection ret = policyDecoderCsarPdpx.decode(csar); @@ -98,7 +97,7 @@ public class TestPolicyDecoderCsarPdpx { public void testHpaPolicySriov() throws IOException, PolicyDecodingException { final Csar csar = new Csar("src/test/resources/hpaPolicySriov.csar"); final PolicyDecoderCsarPdpx policyDecoderCsarPdpx = new PolicyDecoderCsarPdpx(); - policyDecoderCsarPdpx.configure(CSAR_TO_OPTIMIZATION_POLICY_CONFIGURATION); + policyDecoderCsarPdpx.configure(PolicyDecoderCsarPdpxConfigurationParameterGroup.class.getSimpleName()); final Collection policies = policyDecoderCsarPdpx.decode(csar); final ToscaServiceTemplate policy = (ToscaServiceTemplate) policies.toArray()[0]; @@ -138,7 +137,7 @@ public class TestPolicyDecoderCsarPdpx { public void testHpaPolicyPciePassthrough() throws IOException, PolicyDecodingException { final Csar csar = new Csar("src/test/resources/hpaPolicyPciePassthrough.csar"); final PolicyDecoderCsarPdpx policyDecoderCsarPdpx = new PolicyDecoderCsarPdpx(); - policyDecoderCsarPdpx.configure(CSAR_TO_OPTIMIZATION_POLICY_CONFIGURATION); + policyDecoderCsarPdpx.configure(PolicyDecoderCsarPdpxConfigurationParameterGroup.class.getSimpleName()); final Collection policies = policyDecoderCsarPdpx.decode(csar); assertEquals(2, policies.size()); @@ -181,7 +180,7 @@ public class TestPolicyDecoderCsarPdpx { public void testHpaPolicyHugePage() throws IOException, PolicyDecodingException { final Csar csar = new Csar("src/test/resources/hpaPolicyHugePage.csar"); final PolicyDecoderCsarPdpx policyDecoderCsarPdpx = new PolicyDecoderCsarPdpx(); - policyDecoderCsarPdpx.configure(CSAR_TO_OPTIMIZATION_POLICY_CONFIGURATION); + policyDecoderCsarPdpx.configure(PolicyDecoderCsarPdpxConfigurationParameterGroup.class.getSimpleName()); final Collection policies = policyDecoderCsarPdpx.decode(csar); assertEquals(2, policies.size()); @@ -214,7 +213,7 @@ public class TestPolicyDecoderCsarPdpx { public void testS3p0PciVendorId() throws IOException, PolicyDecodingException { final Csar csar = new Csar("src/test/resources/s3p_0_pciVendorId.csar"); final PolicyDecoderCsarPdpx policyDecoderCsarPdpx = new PolicyDecoderCsarPdpx(); - policyDecoderCsarPdpx.configure(CSAR_TO_OPTIMIZATION_POLICY_CONFIGURATION); + policyDecoderCsarPdpx.configure(PolicyDecoderCsarPdpxConfigurationParameterGroup.class.getSimpleName()); final Collection policies = policyDecoderCsarPdpx.decode(csar); assertEquals(1, policies.size()); @@ -248,7 +247,7 @@ public class TestPolicyDecoderCsarPdpx { public void testserviceVcpeWithAll() throws IOException, PolicyDecodingException { final Csar csar = new Csar("src/test/resources/service-VcpeWithAll-csar.csar"); final PolicyDecoderCsarPdpx policyDecoderCsarPdpx = new PolicyDecoderCsarPdpx(); - policyDecoderCsarPdpx.configure(CSAR_TO_OPTIMIZATION_POLICY_CONFIGURATION); + policyDecoderCsarPdpx.configure(PolicyDecoderCsarPdpxConfigurationParameterGroup.class.getSimpleName()); final Collection policies = policyDecoderCsarPdpx.decode(csar); assertEquals(5, policies.size()); diff --git a/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/pdpx/TestPolicyDecoderCsarPdpxConfigurationParameterGroup.java b/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/pdpx/TestPolicyDecoderCsarPdpxConfigurationParameterGroup.java index bebb8459..d0de432a 100644 --- a/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/pdpx/TestPolicyDecoderCsarPdpxConfigurationParameterGroup.java +++ b/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/pdpx/TestPolicyDecoderCsarPdpxConfigurationParameterGroup.java @@ -1,6 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2018 Ericsson. All rights reserved. + * Modifications Copyright (C) 2019 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -32,17 +33,13 @@ import org.onap.policy.common.parameters.ValidationStatus; */ public class TestPolicyDecoderCsarPdpxConfigurationParameterGroup { - private static final String OPTIMIZATION_POLICY_CONFIGURATION = "optimizationPolicyConfiguration"; - @Test - public void testBuilderAndGetters() { - final PolicyDecoderCsarPdpxConfigurationParameterGroup parameterGroup = - new PolicyDecoderCsarPdpxConfigurationParameterBuilder().setOnapName("onapName") - .setPolicyNamePrefix("OOF").setPriority("5").setRiskLevel("2").setRiskType("Test") - .setVersion("1.0").build(); - parameterGroup.setName(OPTIMIZATION_POLICY_CONFIGURATION); + public void testValidParameters() { + final PolicyDecoderCsarPdpxConfigurationParameterGroup parameterGroup = CommonTestData + .getPolicyDecoderParameters("src/test/resources/parameters/PdpxPolicyDecoderParameters.json", + PolicyDecoderCsarPdpxConfigurationParameterGroup.class); - assertEquals(OPTIMIZATION_POLICY_CONFIGURATION, parameterGroup.getName()); + assertEquals(PolicyDecoderCsarPdpxConfigurationParameterGroup.class.getSimpleName(), parameterGroup.getName()); assertEquals("onapName", parameterGroup.getOnapName()); assertEquals("OOF", parameterGroup.getPolicyNamePrefix()); assertEquals("5", parameterGroup.getPriority()); @@ -54,11 +51,19 @@ public class TestPolicyDecoderCsarPdpxConfigurationParameterGroup { @Test public void testInvalidParameters() { - final PolicyDecoderCsarPdpxConfigurationParameterGroup parameterGroup = - new PolicyDecoderCsarPdpxConfigurationParameterBuilder().setOnapName("").setPolicyNamePrefix("") - .setPriority("").setRiskLevel("").setRiskType("").setVersion("").build(); - parameterGroup.setName(OPTIMIZATION_POLICY_CONFIGURATION); + final PolicyDecoderCsarPdpxConfigurationParameterGroup parameterGroup = CommonTestData + .getPolicyDecoderParameters("src/test/resources/parameters/PdpxPolicyDecoderParametersInvalid.json", + PolicyDecoderCsarPdpxConfigurationParameterGroup.class); assertEquals(ValidationStatus.INVALID, parameterGroup.validate().getStatus()); } + + @Test + public void testEmptyParameters() { + final PolicyDecoderCsarPdpxConfigurationParameterGroup configurationParameters = + CommonTestData.getPolicyDecoderParameters("src/test/resources/parameters/EmptyParameters.json", + PolicyDecoderCsarPdpxConfigurationParameterGroup.class); + + assertEquals(ValidationStatus.INVALID, configurationParameters.validate().getStatus()); + } } diff --git a/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/policy/file/PolicyDecoderFileInCsarToPolicyParameterGroupTest.java b/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/policy/file/PolicyDecoderFileInCsarToPolicyParameterGroupTest.java index 2dd09985..56ddc406 100644 --- a/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/policy/file/PolicyDecoderFileInCsarToPolicyParameterGroupTest.java +++ b/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/policy/file/PolicyDecoderFileInCsarToPolicyParameterGroupTest.java @@ -1,6 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2018 Ericsson. All rights reserved. + * Modifications Copyright (C) 2019 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -24,6 +25,7 @@ import static org.junit.Assert.assertEquals; import org.junit.Test; import org.onap.policy.common.parameters.ValidationStatus; +import org.onap.policy.distribution.reception.decoding.pdpx.CommonTestData; /** * Class to perform unit test of {@link PolicyDecoderFileInCsarToPolicyParameterGroup}. @@ -33,32 +35,34 @@ import org.onap.policy.common.parameters.ValidationStatus; public class PolicyDecoderFileInCsarToPolicyParameterGroupTest { @Test - public void testConstructorAndGetters() { - final PolicyDecoderFileInCsarToPolicyParameterGroup configurationParameters = - new PolicyDecoderFileInCsarToPolicyParameterGroup("SamplePolicy", "SamplePolicyType"); - configurationParameters.setName("myConfiguration"); + public void testValidParameters() { + final PolicyDecoderFileInCsarToPolicyParameterGroup configurationParameters = CommonTestData + .getPolicyDecoderParameters("src/test/resources/parameters/FileInCsarPolicyDecoderParameters.json", + PolicyDecoderFileInCsarToPolicyParameterGroup.class); - assertEquals("myConfiguration", configurationParameters.getName()); - assertEquals("SamplePolicy", configurationParameters.getPolicyFileName()); - assertEquals("SamplePolicyType", configurationParameters.getPolicyTypeFileName()); + assertEquals(PolicyDecoderFileInCsarToPolicyParameterGroup.class.getSimpleName(), + configurationParameters.getName()); + assertEquals("apex_ddf_policy", configurationParameters.getPolicyFileName()); + assertEquals("apex_ddf_policy_type", configurationParameters.getPolicyTypeFileName()); assertEquals(ValidationStatus.CLEAN, configurationParameters.validate().getStatus()); } @Test - public void testInvalidPolicyFileName() { + public void testInvalidParameters() { final PolicyDecoderFileInCsarToPolicyParameterGroup configurationParameters = - new PolicyDecoderFileInCsarToPolicyParameterGroup("", "SamplePolicyType"); - configurationParameters.setName("myConfiguration"); + CommonTestData.getPolicyDecoderParameters( + "src/test/resources/parameters/FileInCsarPolicyDecoderParametersInvalid.json", + PolicyDecoderFileInCsarToPolicyParameterGroup.class); assertEquals(ValidationStatus.INVALID, configurationParameters.validate().getStatus()); } @Test - public void testEmptyPolicyType() { + public void testEmptyParameters() { final PolicyDecoderFileInCsarToPolicyParameterGroup configurationParameters = - new PolicyDecoderFileInCsarToPolicyParameterGroup("SamplePolicy", ""); - configurationParameters.setName("myConfiguration"); + CommonTestData.getPolicyDecoderParameters("src/test/resources/parameters/EmptyParameters.json", + PolicyDecoderFileInCsarToPolicyParameterGroup.class); - assertEquals(ValidationStatus.CLEAN, configurationParameters.validate().getStatus()); + assertEquals(ValidationStatus.INVALID, configurationParameters.validate().getStatus()); } } diff --git a/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/policy/file/PolicyDecoderFileInCsarToPolicyTest.java b/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/policy/file/PolicyDecoderFileInCsarToPolicyTest.java index 2d0ba3ba..451e4f1b 100644 --- a/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/policy/file/PolicyDecoderFileInCsarToPolicyTest.java +++ b/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/decoding/policy/file/PolicyDecoderFileInCsarToPolicyTest.java @@ -1,6 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2018 Ericsson. All rights reserved. + * Modifications Copyright (C) 2019 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -36,6 +37,7 @@ import org.mockito.runners.MockitoJUnitRunner; import org.onap.policy.common.parameters.ParameterService; import org.onap.policy.common.utils.coder.CoderException; import org.onap.policy.distribution.model.Csar; +import org.onap.policy.distribution.reception.decoding.pdpx.CommonTestData; import org.onap.policy.models.tosca.authorative.concepts.ToscaEntity; /** @@ -46,18 +48,15 @@ import org.onap.policy.models.tosca.authorative.concepts.ToscaEntity; @RunWith(MockitoJUnitRunner.class) public class PolicyDecoderFileInCsarToPolicyTest { - private static final String POLICY_FILE_NAME = "apex_ddf_policy"; - private static final String POLICY_TYPE_FILE_NAME = "apex_ddf_policy_type"; - private static final String GROUP_NAME = "apexPdpDecoderConfiguration"; - /** * Set up. */ @BeforeClass public static void setUp() { - final PolicyDecoderFileInCsarToPolicyParameterGroup configurationParameters = - new PolicyDecoderFileInCsarToPolicyParameterGroup(POLICY_FILE_NAME, POLICY_TYPE_FILE_NAME); - configurationParameters.setName(GROUP_NAME); + final PolicyDecoderFileInCsarToPolicyParameterGroup configurationParameters = CommonTestData + .getPolicyDecoderParameters("src/test/resources/parameters/FileInCsarPolicyDecoderParameters.json", + PolicyDecoderFileInCsarToPolicyParameterGroup.class); + configurationParameters.setName(PolicyDecoderFileInCsarToPolicyParameterGroup.class.getSimpleName()); ParameterService.register(configurationParameters); } @@ -66,14 +65,14 @@ public class PolicyDecoderFileInCsarToPolicyTest { */ @AfterClass public static void tearDown() { - ParameterService.deregister(GROUP_NAME); + ParameterService.deregister(PolicyDecoderFileInCsarToPolicyParameterGroup.class.getSimpleName()); } @Test public void testDecodePolicy() { final PolicyDecoderFileInCsarToPolicy decoder = new PolicyDecoderFileInCsarToPolicy(); - decoder.configure(GROUP_NAME); + decoder.configure(PolicyDecoderFileInCsarToPolicyParameterGroup.class.getSimpleName()); final File file = new File("src/test/resources/service-Sampleservice.csar"); final Csar csar = new Csar(file.getAbsolutePath()); @@ -91,7 +90,7 @@ public class PolicyDecoderFileInCsarToPolicyTest { public void testDecodePolicyZipError() { final PolicyDecoderFileInCsarToPolicy decoder = new PolicyDecoderFileInCsarToPolicy(); - decoder.configure(GROUP_NAME); + decoder.configure(PolicyDecoderFileInCsarToPolicyParameterGroup.class.getSimpleName()); final File file = new File("unknown.csar"); final Csar csar = new Csar(file.getAbsolutePath()); @@ -111,7 +110,7 @@ public class PolicyDecoderFileInCsarToPolicyTest { public void testDecodePolicyCoderError() { final PolicyDecoderFileInCsarToPolicy decoder = new PolicyDecoderFileInCsarToPolicy(); - decoder.configure(GROUP_NAME); + decoder.configure(PolicyDecoderFileInCsarToPolicyParameterGroup.class.getSimpleName()); final File file = new File("src/test/resources/service-Sampleservice-test.csar"); final Csar csar = new Csar(file.getAbsolutePath()); diff --git a/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/handling/file/TestFileSystemReceptionHandlerConfigurationParameterGroup.java b/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/handling/file/TestFileSystemReceptionHandlerConfigurationParameterGroup.java index 1d32b191..dab97ec3 100644 --- a/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/handling/file/TestFileSystemReceptionHandlerConfigurationParameterGroup.java +++ b/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/handling/file/TestFileSystemReceptionHandlerConfigurationParameterGroup.java @@ -52,9 +52,9 @@ public class TestFileSystemReceptionHandlerConfigurationParameterGroup { try { validPath = tempFolder.getRoot().getAbsolutePath(); - final FileSystemReceptionHandlerConfigurationParameterBuilder builder = - new FileSystemReceptionHandlerConfigurationParameterBuilder().setWatchPath(validPath).setMaxThread(2); - configParameters = new FileSystemReceptionHandlerConfigurationParameterGroup(builder); + configParameters = new FileSystemReceptionHandlerConfigurationParameterGroup(); + configParameters.setWatchPath(validPath); + configParameters.setMaxThread(2); } catch (final Exception e) { fail("test should not thrown an exception here: " + e.getMessage()); } @@ -78,25 +78,13 @@ public class TestFileSystemReceptionHandlerConfigurationParameterGroup { assertFalse(validationResult.isValid()); } - @Test - public void testFileSystemReceptionHandlerConfigurationParameterBuilder() { - - final FileSystemReceptionHandlerConfigurationParameterBuilder builder = - new FileSystemReceptionHandlerConfigurationParameterBuilder().setWatchPath("/foo/bar"); - final FileSystemReceptionHandlerConfigurationParameterGroup configParameters = - new FileSystemReceptionHandlerConfigurationParameterGroup(builder); - - assertEquals("/foo/bar", configParameters.getWatchPath()); - } - @Test public void testFileSystemReceptionHandlerConfigurationParameterBuilderWithInvalidPath() throws IOException { final String invalidPath = tempFolder.newFile("foobar").getAbsolutePath(); - final FileSystemReceptionHandlerConfigurationParameterBuilder builder = - new FileSystemReceptionHandlerConfigurationParameterBuilder().setWatchPath(invalidPath); final FileSystemReceptionHandlerConfigurationParameterGroup configParameters = - new FileSystemReceptionHandlerConfigurationParameterGroup(builder); + new FileSystemReceptionHandlerConfigurationParameterGroup(); + configParameters.setWatchPath(invalidPath); final GroupValidationResult validateResult = configParameters.validate(); assertFalse(validateResult.isValid()); diff --git a/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/handling/sdc/TestSdcReceptionHandlerConfigurationParameterGroup.java b/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/handling/sdc/TestSdcReceptionHandlerConfigurationParameterGroup.java index f9a65c54..30bcdf11 100644 --- a/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/handling/sdc/TestSdcReceptionHandlerConfigurationParameterGroup.java +++ b/plugins/reception-plugins/src/test/java/org/onap/policy/distribution/reception/handling/sdc/TestSdcReceptionHandlerConfigurationParameterGroup.java @@ -1,6 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2018 Intel. All rights reserved. + * Modifications Copyright (C) 2019 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -34,6 +35,8 @@ import java.util.Arrays; import org.junit.Test; import org.onap.policy.common.parameters.GroupValidationResult; +import org.onap.policy.common.parameters.ValidationStatus; +import org.onap.policy.distribution.reception.decoding.pdpx.CommonTestData; /** * Class to perform unit test of {@link SdcConfiguration}. @@ -86,75 +89,11 @@ public class TestSdcReceptionHandlerConfigurationParameterGroup { } @Test - public void testSdcConfigurationBuilder() { + public void testEmptyParameters() { + final SdcReceptionHandlerConfigurationParameterGroup configurationParameters = + CommonTestData.getPolicyDecoderParameters("src/test/resources/parameters/EmptyParameters.json", + SdcReceptionHandlerConfigurationParameterGroup.class); - final SdcReceptionHandlerConfigurationParameterBuilder builder = - new SdcReceptionHandlerConfigurationParameterBuilder().setAsdcAddress("localhost") - .setConsumerGroup("policy-group").setConsumerId("policy-id").setEnvironmentName("TEST") - .setKeystorePassword("password").setKeystorePath("dummyPath").setPassword("policy") - .setPollingInterval(10).setPollingTimeout(20).setRetryDelay(30).setUser("policy") - .setUseHttpsWithDmaap(false).setActiveserverTlsAuth(false).setFilterinEmptyResources(true) - .setArtifactTypes(Arrays.asList("TOSCA_CSAR")).setMessageBusAddress(Arrays.asList("localhost")); - final SdcReceptionHandlerConfigurationParameterGroup configParameters = - new SdcReceptionHandlerConfigurationParameterGroup(builder); - configParameters.setName("SDCConfiguration"); - - assertEquals(Arrays.asList("localhost"), configParameters.getMessageBusAddress()); - assertEquals(Arrays.asList("TOSCA_CSAR"), configParameters.getArtifactTypes()); - assertEquals("localhost", configParameters.getAsdcAddress()); - assertEquals("policy", configParameters.getUser()); - assertEquals("policy", configParameters.getPassword()); - assertEquals(10, configParameters.getPollingInterval()); - assertEquals(20, configParameters.getPollingTimeout()); - assertEquals(30, configParameters.getRetryDelay()); - assertEquals("policy-id", configParameters.getConsumerId()); - assertEquals("policy-group", configParameters.getConsumerGroup()); - assertEquals("TEST", configParameters.getEnvironmentName()); - assertEquals("dummyPath", configParameters.getKeyStorePath()); - assertEquals("password", configParameters.getKeyStorePassword()); - assertEquals(false, configParameters.isActiveServerTlsAuth()); - assertEquals(true, configParameters.isFilterInEmptyResources()); - assertEquals(false, configParameters.isUseHttpsWithDmaap()); - } - - @Test - public void testSdcConfigurationWithNullList() { - - final SdcReceptionHandlerConfigurationParameterBuilder builder = - new SdcReceptionHandlerConfigurationParameterBuilder().setAsdcAddress("localhost") - .setConsumerGroup("policy-group").setConsumerId("policy-id").setEnvironmentName("TEST") - .setKeystorePassword("password").setKeystorePath("dummyPath").setPassword("policy") - .setPollingInterval(10).setPollingTimeout(20).setUser("policy").setUseHttpsWithDmaap(false) - .setActiveserverTlsAuth(false).setFilterinEmptyResources(true) - .setArtifactTypes(Arrays.asList("TOSCA_CSAR")).setMessageBusAddress(null); - final SdcReceptionHandlerConfigurationParameterGroup configParameters = - new SdcReceptionHandlerConfigurationParameterGroup(builder); - configParameters.setName("SDCConfiguration"); - - try { - configParameters.validate(); - fail("Test must throw an exception"); - } catch (final Exception exp) { - assertTrue(exp.getMessage().contains("collection parameter \"messageBusAddress\" is null")); - } - } - - @Test - public void testSdcConfigurationWithEmptyStringList() { - - final SdcReceptionHandlerConfigurationParameterBuilder builder = - new SdcReceptionHandlerConfigurationParameterBuilder().setAsdcAddress("localhost") - .setConsumerGroup("policy-group").setConsumerId("policy-id").setEnvironmentName("TEST") - .setKeystorePassword("password").setKeystorePath("dummyPath").setPassword("policy") - .setPollingInterval(10).setPollingTimeout(20).setUser("policy").setUseHttpsWithDmaap(false) - .setActiveserverTlsAuth(false).setFilterinEmptyResources(true) - .setArtifactTypes(Arrays.asList("")).setMessageBusAddress(Arrays.asList("localhost")); - final SdcReceptionHandlerConfigurationParameterGroup configParameters = - new SdcReceptionHandlerConfigurationParameterGroup(builder); - configParameters.setName("SDCConfiguration"); - - final GroupValidationResult validationResult = configParameters.validate(); - assertFalse(validationResult.isValid()); - assertTrue(validationResult.getResult().contains("must be a non-blank string")); + assertEquals(ValidationStatus.INVALID, configurationParameters.validate().getStatus()); } } diff --git a/plugins/reception-plugins/src/test/resources/parameters/EmptyParameters.json b/plugins/reception-plugins/src/test/resources/parameters/EmptyParameters.json new file mode 100644 index 00000000..7a73a41b --- /dev/null +++ b/plugins/reception-plugins/src/test/resources/parameters/EmptyParameters.json @@ -0,0 +1,2 @@ +{ +} \ No newline at end of file diff --git a/plugins/reception-plugins/src/test/resources/parameters/FileInCsarPolicyDecoderParameters.json b/plugins/reception-plugins/src/test/resources/parameters/FileInCsarPolicyDecoderParameters.json new file mode 100644 index 00000000..ce67ef47 --- /dev/null +++ b/plugins/reception-plugins/src/test/resources/parameters/FileInCsarPolicyDecoderParameters.json @@ -0,0 +1,4 @@ +{ + "policyFileName": "apex_ddf_policy", + "policyTypeFileName": "apex_ddf_policy_type" +} \ No newline at end of file diff --git a/plugins/reception-plugins/src/test/resources/parameters/FileInCsarPolicyDecoderParametersInvalid.json b/plugins/reception-plugins/src/test/resources/parameters/FileInCsarPolicyDecoderParametersInvalid.json new file mode 100644 index 00000000..50719d16 --- /dev/null +++ b/plugins/reception-plugins/src/test/resources/parameters/FileInCsarPolicyDecoderParametersInvalid.json @@ -0,0 +1,4 @@ +{ + "policyFileName": "", + "policyTypeFileName": "" +} \ No newline at end of file diff --git a/plugins/reception-plugins/src/test/resources/parameters/PdpxPolicyDecoderParameters.json b/plugins/reception-plugins/src/test/resources/parameters/PdpxPolicyDecoderParameters.json new file mode 100644 index 00000000..9f686632 --- /dev/null +++ b/plugins/reception-plugins/src/test/resources/parameters/PdpxPolicyDecoderParameters.json @@ -0,0 +1,8 @@ +{ + "policyNamePrefix": "OOF", + "onapName": "onapName", + "version": "1.0", + "priority": "5", + "riskType": "Test", + "riskLevel": "2" +} \ No newline at end of file diff --git a/plugins/reception-plugins/src/test/resources/parameters/PdpxPolicyDecoderParametersInvalid.json b/plugins/reception-plugins/src/test/resources/parameters/PdpxPolicyDecoderParametersInvalid.json new file mode 100644 index 00000000..a716452e --- /dev/null +++ b/plugins/reception-plugins/src/test/resources/parameters/PdpxPolicyDecoderParametersInvalid.json @@ -0,0 +1,8 @@ +{ + "policyNamePrefix": "", + "onapName": "", + "version": "", + "priority": "", + "riskType": "", + "riskLevel": "" +} \ No newline at end of file diff --git a/reception/src/main/java/org/onap/policy/distribution/reception/parameters/PolicyDecoderConfigurationParameterGroup.java b/reception/src/main/java/org/onap/policy/distribution/reception/parameters/PolicyDecoderConfigurationParameterGroup.java index 00ae0a3a..1a3551a8 100644 --- a/reception/src/main/java/org/onap/policy/distribution/reception/parameters/PolicyDecoderConfigurationParameterGroup.java +++ b/reception/src/main/java/org/onap/policy/distribution/reception/parameters/PolicyDecoderConfigurationParameterGroup.java @@ -1,6 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2018 Ericsson. All rights reserved. + * Modifications Copyright (C) 2019 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -21,6 +22,7 @@ package org.onap.policy.distribution.reception.parameters; import org.onap.policy.common.parameters.ParameterGroup; +import org.onap.policy.common.parameters.ParameterGroupImpl; import org.onap.policy.distribution.reception.decoding.PolicyDecoder; /** @@ -28,18 +30,9 @@ import org.onap.policy.distribution.reception.decoding.PolicyDecoder; * * @author Ram Krishna Verma (ram.krishna.verma@ericsson.com) */ -public abstract class PolicyDecoderConfigurationParameterGroup implements ParameterGroup { +public abstract class PolicyDecoderConfigurationParameterGroup extends ParameterGroupImpl { - private String name; - - @Override - public String getName() { - return name; - } - - @Override - public void setName(final String name) { - this.name = name; + public PolicyDecoderConfigurationParameterGroup(final String name) { + super(name); } - } diff --git a/reception/src/main/java/org/onap/policy/distribution/reception/parameters/ReceptionHandlerConfigurationParameterGroup.java b/reception/src/main/java/org/onap/policy/distribution/reception/parameters/ReceptionHandlerConfigurationParameterGroup.java index 3fa877db..d43f0856 100644 --- a/reception/src/main/java/org/onap/policy/distribution/reception/parameters/ReceptionHandlerConfigurationParameterGroup.java +++ b/reception/src/main/java/org/onap/policy/distribution/reception/parameters/ReceptionHandlerConfigurationParameterGroup.java @@ -1,6 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2018 Ericsson. All rights reserved. + * Modifications Copyright (C) 2019 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -21,24 +22,15 @@ package org.onap.policy.distribution.reception.parameters; import org.onap.policy.common.parameters.ParameterGroup; +import org.onap.policy.common.parameters.ParameterGroupImpl; import org.onap.policy.distribution.reception.handling.ReceptionHandler; /** - * Base class of all {@link ParameterGroup} classes for configuration parameters for - * {@link ReceptionHandler} classes. + * Base class of all {@link ParameterGroup} classes for configuration parameters for {@link ReceptionHandler} classes. */ -public abstract class ReceptionHandlerConfigurationParameterGroup implements ParameterGroup { +public abstract class ReceptionHandlerConfigurationParameterGroup extends ParameterGroupImpl { - private String name; - - @Override - public String getName() { - return name; - } - - @Override - public void setName(final String name) { - this.name = name; + public ReceptionHandlerConfigurationParameterGroup(final String name) { + super(name); } - } -- cgit 1.2.3-korg