From 5769b4d5d7d8a7bf9788b2bbda781a9384df7626 Mon Sep 17 00:00:00 2001 From: ramverma Date: Wed, 3 Apr 2019 17:53:57 +0000 Subject: Fix classes in pap to align with change in models Change-Id: I0ecef8f004967e2fffa33eff2ee7e6f035d3d94c Issue-ID: POLICY-1443 Signed-off-by: ramverma --- .../pap/main/comm/PdpModifyRequestMapTest.java | 18 +- .../pap/main/comm/msgdata/UpdateDataTest.java | 26 +- .../onap/policy/pap/main/mapper/MapperTest.java | 418 --------------------- .../main/rest/TestPdpGroupDeployControllerV1.java | 4 +- .../main/rest/TestPdpGroupQueryControllerV1.java | 2 +- 5 files changed, 42 insertions(+), 426 deletions(-) delete mode 100644 main/src/test/java/org/onap/policy/pap/main/mapper/MapperTest.java (limited to 'main/src/test/java') diff --git a/main/src/test/java/org/onap/policy/pap/main/comm/PdpModifyRequestMapTest.java b/main/src/test/java/org/onap/policy/pap/main/comm/PdpModifyRequestMapTest.java index c2993528..91d7d435 100644 --- a/main/src/test/java/org/onap/policy/pap/main/comm/PdpModifyRequestMapTest.java +++ b/main/src/test/java/org/onap/policy/pap/main/comm/PdpModifyRequestMapTest.java @@ -42,6 +42,7 @@ import java.util.List; import java.util.Map; import java.util.Queue; import java.util.function.Consumer; + import org.junit.Before; import org.junit.Test; import org.mockito.ArgumentCaptor; @@ -54,6 +55,7 @@ import org.onap.policy.models.pdp.concepts.PdpMessage; import org.onap.policy.models.pdp.concepts.PdpStateChange; import org.onap.policy.models.pdp.concepts.PdpStatus; import org.onap.policy.models.pdp.concepts.PdpUpdate; +import org.onap.policy.models.pdp.concepts.ToscaPolicyIdentifier; import org.onap.policy.models.pdp.enums.PdpState; import org.onap.policy.models.tosca.authorative.concepts.ToscaPolicy; import org.onap.policy.pap.main.PapConstants; @@ -132,7 +134,7 @@ public class PdpModifyRequestMapTest { response.setState(PdpState.SAFE); response.setPdpGroup(update.getPdpGroup()); response.setPdpSubgroup(update.getPdpSubgroup()); - response.setPolicies(update.getPolicies()); + response.setPolicies(convertToscaPolicyToToscaPolicyIndentifier()); map = new PdpModifyRequestMap(makeParameters()) { @@ -583,4 +585,18 @@ public class PdpModifyRequestMapTest { return cng; } + + /** + * Converts a ToscaPolicy list to ToscaPolicyIdentifier list. + * + * @return the ToscaPolicyIdentifier list + */ + private List convertToscaPolicyToToscaPolicyIndentifier() { + final List toscaPolicies = update.getPolicies(); + final List toscaPolicyIdentifiers = new ArrayList<>(); + for (final ToscaPolicy toscaPolicy : toscaPolicies) { + toscaPolicyIdentifiers.add(new ToscaPolicyIdentifier(toscaPolicy.getName(), toscaPolicy.getVersion())); + } + return toscaPolicyIdentifiers; + } } diff --git a/main/src/test/java/org/onap/policy/pap/main/comm/msgdata/UpdateDataTest.java b/main/src/test/java/org/onap/policy/pap/main/comm/msgdata/UpdateDataTest.java index 06a3bf01..127843a0 100644 --- a/main/src/test/java/org/onap/policy/pap/main/comm/msgdata/UpdateDataTest.java +++ b/main/src/test/java/org/onap/policy/pap/main/comm/msgdata/UpdateDataTest.java @@ -28,10 +28,13 @@ import static org.mockito.Mockito.when; import java.util.ArrayList; import java.util.Arrays; +import java.util.List; + import org.junit.Before; import org.junit.Test; import org.onap.policy.models.pdp.concepts.PdpStatus; import org.onap.policy.models.pdp.concepts.PdpUpdate; +import org.onap.policy.models.pdp.concepts.ToscaPolicyIdentifier; import org.onap.policy.models.tosca.authorative.concepts.ToscaPolicy; import org.onap.policy.pap.main.comm.TimerManager; import org.onap.policy.pap.main.parameters.PdpModifyRequestMapParams; @@ -69,7 +72,7 @@ public class UpdateDataTest { response.setName(MY_NAME); response.setPdpGroup(update.getPdpGroup()); response.setPdpSubgroup(update.getPdpSubgroup()); - response.setPolicies(update.getPolicies()); + response.setPolicies(convertToscaPolicyToToscaPolicyIndentifier(update.getPolicies())); data = new MyData(update); } @@ -112,15 +115,16 @@ public class UpdateDataTest { @Test public void testUpdateDataCheckResponse_MismatchedPoliciesLength() { - response.setPolicies(Arrays.asList(update.getPolicies().get(0))); + response.setPolicies(convertToscaPolicyToToscaPolicyIndentifier(Arrays.asList(update.getPolicies().get(0)))); assertEquals("policies do not match", data.checkResponse(response)); } @Test public void testUpdateDataCheckResponse_MismatchedPolicies() { - ArrayList policies = new ArrayList<>(update.getPolicies()); - policies.set(0, makePolicy(DIFFERENT, "10.0.0")); + ArrayList policies = + new ArrayList<>(convertToscaPolicyToToscaPolicyIndentifier(update.getPolicies())); + policies.set(0, new ToscaPolicyIdentifier(DIFFERENT, "10.0.0")); response.setPolicies(policies); @@ -180,4 +184,18 @@ public class UpdateDataTest { // do nothing } } + + /** + * Converts a ToscaPolicy list to ToscaPolicyIdentifier list. + * + * @param toscaPolicies the list of ToscaPolicy + * @return the ToscaPolicyIdentifier list + */ + private List convertToscaPolicyToToscaPolicyIndentifier(List toscaPolicies) { + final List toscaPolicyIdentifiers = new ArrayList<>(); + for (final ToscaPolicy toscaPolicy : toscaPolicies) { + toscaPolicyIdentifiers.add(new ToscaPolicyIdentifier(toscaPolicy.getName(), toscaPolicy.getVersion())); + } + return toscaPolicyIdentifiers; + } } diff --git a/main/src/test/java/org/onap/policy/pap/main/mapper/MapperTest.java b/main/src/test/java/org/onap/policy/pap/main/mapper/MapperTest.java deleted file mode 100644 index 350bd509..00000000 --- a/main/src/test/java/org/onap/policy/pap/main/mapper/MapperTest.java +++ /dev/null @@ -1,418 +0,0 @@ -/* - * ============LICENSE_START======================================================= - * ONAP PAP - * ================================================================================ - * Copyright (C) 2019 AT&T Intellectual Property. 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. - * ============LICENSE_END========================================================= - */ - -package org.onap.policy.pap.main.mapper; - -import static org.junit.Assert.assertEquals; - -import java.util.Arrays; -import java.util.Iterator; -import java.util.List; -import java.util.Map; -import java.util.TreeMap; -import java.util.function.BiConsumer; -import org.junit.Test; -import org.onap.policy.models.pdp.concepts.Pdp; -import org.onap.policy.models.pdp.concepts.PdpGroup; -import org.onap.policy.models.pdp.concepts.PdpGroups; -import org.onap.policy.models.pdp.concepts.PdpSubGroup; -import org.onap.policy.models.pdp.concepts.PolicyIdent; -import org.onap.policy.models.pdp.concepts.PolicyIdentOptVersion; -import org.onap.policy.models.pdp.concepts.PolicyTypeIdent; -import org.onap.policy.models.pdp.enums.PdpHealthStatus; -import org.onap.policy.models.pdp.enums.PdpState; -import org.onap.policy.models.tosca.authorative.concepts.ToscaPolicy; -import org.onap.policy.pap.main.internal.PdpDeployPolicies; -import org.powermock.reflect.Whitebox; - -public class MapperTest { - private static final String MY_NAME = "my-name"; - private static final String MY_VERSION = "10.11.12"; - private static final PdpHealthStatus MY_HEALTH = PdpHealthStatus.TEST_IN_PROGRESS; - private static final String MY_INSTANCE = "my_1"; - private static final String MY_MESSAGE = "my message"; - private static final PdpState MY_STATE = PdpState.SAFE; - private static final int CURRENT_COUNT = 1; - private static final int DESIRED_COUNT = 2; - private static final String PDP_TYPE1 = "drools"; - private static final String PROPKEY1 = "prop-a"; - private static final String PROPVAL1 = "value-a"; - private static final String PROPKEY2 = "prop-b"; - private static final String PROPVAL2 = "value-b"; - private static final String MY_DESCRIPTION = "my description"; - - @Test - public void testConstructor() throws Exception { - Whitebox.invokeConstructor(Mapper.class); - } - - @Test - public void testPdpDeployPolicies() { - - // TO-INTERNAL - - // test populated object - org.onap.policy.models.pap.concepts.PdpDeployPolicies external = - new org.onap.policy.models.pap.concepts.PdpDeployPolicies(); - - external.setPolicies(Arrays.asList(makePolicyIdentOptVersion(1), makePolicyIdentOptVersion(2))); - compareWithExternal(external, Mapper.toInternal(external)); - } - - private org.onap.policy.models.pap.concepts.PolicyIdentOptVersion makePolicyIdentOptVersion(int count) { - - org.onap.policy.models.pap.concepts.PolicyIdentOptVersion type = - new org.onap.policy.models.pap.concepts.PolicyIdentOptVersion(); - - - type.setName(MY_NAME + "-id-" + count); - type.setVersion(count + ".1.2"); - - return type; - } - - @Test - public void testPdpGroups() { - - // TO-EXTERNAL - - // test populated object - PdpGroups internal = new PdpGroups(); - internal.setGroups(Arrays.asList(makeGroup(1), makeGroup(2))); - - compareWithInternal(internal, Mapper.toExternal(internal)); - } - - private PdpGroup makeGroup(int count) { - PdpGroup group = new PdpGroup(); - - group.setDescription(MY_DESCRIPTION + "-" + count); - group.setName(MY_NAME); - group.setVersion(MY_VERSION); - group.setPdpGroupState(MY_STATE); - - int inc = count + 1; - group.setPdpSubgroups(Arrays.asList(makeSubGroup(count * 10), makeSubGroup(inc * 10))); - group.setProperties(makeProperties()); - - return group; - } - - private PdpSubGroup makeSubGroup(int count) { - PdpSubGroup subgroup = new PdpSubGroup(); - - subgroup.setCurrentInstanceCount(CURRENT_COUNT); - subgroup.setDesiredInstanceCount(DESIRED_COUNT); - - int inc = count + 1; - subgroup.setPdpInstances(Arrays.asList(makePdp(count), makePdp(inc))); - subgroup.setPdpType(PDP_TYPE1); - subgroup.setPolicies(Arrays.asList(makePolicy(count, count), makePolicy(inc, inc))); - subgroup.setProperties(makeProperties()); - subgroup.setSupportedPolicyTypes(Arrays.asList(makePolicyType(count), makePolicyType(inc))); - - return subgroup; - } - - private Pdp makePdp(int count) { - Pdp object = new Pdp(); - - object.setHealthy(MY_HEALTH); - object.setInstanceId(MY_INSTANCE + count); - object.setMessage(MY_MESSAGE); - object.setPdpState(MY_STATE); - - return object; - } - - private PolicyTypeIdent makePolicyType(int count) { - PolicyTypeIdent type = new PolicyTypeIdent(); - - type.setName(MY_NAME + "-type-" + count); - type.setVersion(count + ".1.2"); - - return type; - } - - private Map makeProperties() { - Map map = new TreeMap<>(); - - map.put(PROPKEY1, PROPVAL1); - map.put(PROPKEY2, PROPVAL2); - - return map; - } - - @Test - public void testPdpGroup() { - - // TO-EXTERNAL - - // test populated object - PdpGroup internal = makeGroup(1); - compareWithInternal(internal, Mapper.toExternal(internal)); - } - - @Test - public void testPdpSubGroup() { - - // TO-EXTERNAL - - // test populated object - PdpSubGroup internal = makeSubGroup(1); - compareWithInternal(internal, Mapper.toExternal(internal)); - } - - @Test - public void testPolicy() { - - // TO-EXTERNAL - - // test populated object - ToscaPolicy internal = makePolicy(1, 1); - compareWithInternal(internal, Mapper.toExternal(internal)); - } - - private ToscaPolicy makePolicy(int count, int typeCount) { - ToscaPolicy policy = new ToscaPolicy(); - - policy.setName(MY_NAME + "-policy-" + count); - policy.setVersion("1.2." + count); - policy.setType(MY_NAME + "-type-" + typeCount); - policy.setTypeVersion("1.2." + typeCount); - - return policy; - } - - @Test - public void testPdpInstanceDetails() { - - // TO-EXTERNAL - - // test populated object - Pdp internal = makePdp(1); - compareWithInternal(internal, Mapper.toExternal(internal)); - - - // TO-INTERNAL - - // test populated object - org.onap.policy.models.pap.concepts.PdpInstanceDetails external = - new org.onap.policy.models.pap.concepts.PdpInstanceDetails(); - external.setHealthy(MY_HEALTH); - external.setInstanceId(MY_INSTANCE); - external.setMessage(MY_MESSAGE); - external.setPdpState(MY_STATE); - compareWithExternal(external, Mapper.toInternal(external)); - } - - @Test - public void testPolicyIdent() { - - // TO-EXTERNAL - - // first test default object - PolicyIdent internal = new PolicyIdent(); - org.onap.policy.models.pap.concepts.PolicyIdent external = Mapper.toExternal(internal); - compareWithInternal(internal, external); - - // now test populated object - internal.setName(MY_NAME); - internal.setVersion(MY_VERSION); - external = Mapper.toExternal(internal); - compareWithInternal(internal, external); - - - // TO-INTERNAL - - external = new org.onap.policy.models.pap.concepts.PolicyIdent(); - external.setName(MY_NAME); - external.setVersion(MY_VERSION); - compareWithExternal(external, Mapper.toInternal(external)); - } - - @Test - public void testPolicyTypeIdent() { - - // TO-EXTERNAL - - // first test default object - PolicyTypeIdent internal = new PolicyTypeIdent(); - org.onap.policy.models.pap.concepts.PolicyTypeIdent external = Mapper.toExternal(internal); - compareWithInternal(internal, external); - - // now test populated object - internal.setName(MY_NAME); - internal.setVersion(MY_VERSION); - external = Mapper.toExternal(internal); - compareWithInternal(internal, external); - - - // TO-INTERNAL - - external = new org.onap.policy.models.pap.concepts.PolicyTypeIdent(); - external.setName(MY_NAME); - external.setVersion(MY_VERSION); - compareWithExternal(external, Mapper.toInternal(external)); - } - - @Test - public void testPolicyIdentOptVersion() { - - // TO-EXTERNAL - - PolicyIdentOptVersion internal = new PolicyIdentOptVersion(); - internal.setName(MY_NAME); - internal.setVersion(MY_VERSION); - org.onap.policy.models.pap.concepts.PolicyIdentOptVersion external = Mapper.toExternal(internal); - compareWithInternal(internal, external); - - - // TO-INTERNAL - - external = new org.onap.policy.models.pap.concepts.PolicyIdentOptVersion(); - external.setName(MY_NAME); - external.setVersion(MY_VERSION); - compareWithExternal(external, Mapper.toInternal(external)); - } - - - /** - * Compares the items in two lists. - * - * @param expected the expected items - * @param actual the actual items - * @param compareItem function to compare an expected item with an actual item - */ - private static void compareList(List expected, List actual, BiConsumer compareItem) { - - assertEquals(expected.size(), actual.size()); - - Iterator expIterator = expected.iterator(); - for (R actualItem : actual) { - L expectedItem = expIterator.next(); - compareItem.accept(expectedItem, actualItem); - } - } - - - // compares actual internal objects with expected external objects - - private static void compareWithExternal(org.onap.policy.models.pap.concepts.PolicyIdent expected, - PolicyIdent actual) { - - assertEquals(expected.getName(), actual.getName()); - assertEquals(expected.getVersion(), actual.getVersion()); - } - - private static void compareWithExternal(org.onap.policy.models.pap.concepts.PolicyTypeIdent expected, - PolicyTypeIdent actual) { - - assertEquals(expected.getName(), actual.getName()); - assertEquals(expected.getVersion(), actual.getVersion()); - } - - private static void compareWithExternal(org.onap.policy.models.pap.concepts.PolicyIdentOptVersion expected, - PolicyIdentOptVersion actual) { - - assertEquals(expected.getName(), actual.getName()); - assertEquals(expected.getVersion(), actual.getVersion()); - } - - private static void compareWithExternal(org.onap.policy.models.pap.concepts.PdpDeployPolicies expected, - PdpDeployPolicies actual) { - - compareList(expected.getPolicies(), actual.getPolicies(), MapperTest::compareWithExternal); - } - - private static void compareWithExternal(org.onap.policy.models.pap.concepts.PdpInstanceDetails expected, - Pdp actual) { - - assertEquals(expected.getHealthy(), actual.getHealthy()); - assertEquals(expected.getInstanceId(), actual.getInstanceId()); - assertEquals(expected.getMessage(), actual.getMessage()); - assertEquals(expected.getPdpState(), actual.getPdpState()); - } - - // compares actual external objects with expected internal objects - - private static void compareWithInternal(PolicyIdent expected, - org.onap.policy.models.pap.concepts.PolicyIdent actual) { - - assertEquals(expected.getName(), actual.getName()); - assertEquals(expected.getVersion(), actual.getVersion()); - } - - private static void compareWithInternal(PolicyTypeIdent expected, - org.onap.policy.models.pap.concepts.PolicyTypeIdent actual) { - - assertEquals(expected.getName(), actual.getName()); - assertEquals(expected.getVersion(), actual.getVersion()); - } - - private static void compareWithInternal(PolicyIdentOptVersion expected, - org.onap.policy.models.pap.concepts.PolicyIdentOptVersion actual) { - - assertEquals(expected.getName(), actual.getName()); - assertEquals(expected.getVersion(), actual.getVersion()); - } - - private static void compareWithInternal(PdpGroups expected, org.onap.policy.models.pap.concepts.PdpGroups actual) { - compareList(expected.getGroups(), actual.getGroups(), MapperTest::compareWithInternal); - } - - private static void compareWithInternal(PdpGroup expected, org.onap.policy.models.pap.concepts.PdpGroup actual) { - assertEquals(expected.getDescription(), actual.getDescription()); - assertEquals(expected.getName(), actual.getName()); - assertEquals(expected.getVersion(), actual.getVersion()); - assertEquals(expected.getPdpGroupState(), actual.getPdpGroupState()); - compareList(expected.getPdpSubgroups(), actual.getPdpSubgroups(), MapperTest::compareWithInternal); - assertEquals(expected.getProperties(), actual.getProperties()); - } - - private static void compareWithInternal(PdpSubGroup expected, - org.onap.policy.models.pap.concepts.PdpSubGroup actual) { - - assertEquals(expected.getCurrentInstanceCount(), actual.getCurrentInstanceCount()); - assertEquals(expected.getDesiredInstanceCount(), actual.getDesiredInstanceCount()); - compareList(expected.getPdpInstances(), actual.getPdpInstances(), MapperTest::compareWithInternal); - assertEquals(expected.getPdpType(), actual.getPdpType()); - compareList(expected.getPolicies(), actual.getPolicies(), MapperTest::compareWithInternal); - assertEquals(expected.getProperties(), actual.getProperties()); - compareList(expected.getSupportedPolicyTypes(), actual.getSupportedPolicyTypes(), - MapperTest::compareWithInternal); - } - - private static void compareWithInternal(ToscaPolicy expected, org.onap.policy.models.pap.concepts.Policy actual) { - assertEquals(expected.getName(), actual.getName()); - assertEquals(expected.getVersion(), actual.getPolicyVersion()); - assertEquals(expected.getType(), actual.getPolicyType()); - assertEquals(expected.getTypeVersion(), actual.getPolicyTypeVersion()); - } - - private static void compareWithInternal(Pdp expected, - org.onap.policy.models.pap.concepts.PdpInstanceDetails actual) { - - assertEquals(expected.getHealthy(), actual.getHealthy()); - assertEquals(expected.getInstanceId(), actual.getInstanceId()); - assertEquals(expected.getMessage(), actual.getMessage()); - assertEquals(expected.getPdpState(), actual.getPdpState()); - } -} diff --git a/main/src/test/java/org/onap/policy/pap/main/rest/TestPdpGroupDeployControllerV1.java b/main/src/test/java/org/onap/policy/pap/main/rest/TestPdpGroupDeployControllerV1.java index b6d0f1d7..e02e85c1 100644 --- a/main/src/test/java/org/onap/policy/pap/main/rest/TestPdpGroupDeployControllerV1.java +++ b/main/src/test/java/org/onap/policy/pap/main/rest/TestPdpGroupDeployControllerV1.java @@ -32,11 +32,11 @@ import javax.ws.rs.core.MediaType; import javax.ws.rs.core.Response; import org.junit.Test; -import org.onap.policy.models.pap.concepts.PdpGroup; import org.onap.policy.models.pap.concepts.PdpGroupDeployResponse; import org.onap.policy.models.pap.concepts.PdpPolicies; -import org.onap.policy.models.pap.concepts.PdpSubGroup; import org.onap.policy.models.pap.concepts.Policy; +import org.onap.policy.models.pdp.concepts.PdpGroup; +import org.onap.policy.models.pdp.concepts.PdpSubGroup; public class TestPdpGroupDeployControllerV1 extends CommonPapRestServer { diff --git a/main/src/test/java/org/onap/policy/pap/main/rest/TestPdpGroupQueryControllerV1.java b/main/src/test/java/org/onap/policy/pap/main/rest/TestPdpGroupQueryControllerV1.java index f0bc12bd..869bc198 100644 --- a/main/src/test/java/org/onap/policy/pap/main/rest/TestPdpGroupQueryControllerV1.java +++ b/main/src/test/java/org/onap/policy/pap/main/rest/TestPdpGroupQueryControllerV1.java @@ -27,7 +27,7 @@ import javax.ws.rs.client.Invocation; import javax.ws.rs.core.Response; import org.junit.Test; -import org.onap.policy.models.pap.concepts.PdpGroups; +import org.onap.policy.models.pdp.concepts.PdpGroups; /** * Class to perform unit test of {@link PdpGroupQueryControllerV1}. -- cgit 1.2.3-korg