From 0cc593ea8c181cfbcde343be45b8cf1cdd6ddbff Mon Sep 17 00:00:00 2001 From: eleonorali Date: Thu, 14 Jun 2018 14:00:40 +0300 Subject: TOSCA Consolidation Data - Refactoring Part 3 - Add Missing UTs for refactored code - Refactor APIs: - addVolumeToConsolidationData - addGroupIdToConsolidationData Change-Id: I83dedcbdf9715c6f72faa5fc242e76ea10efe386 Issue-ID: SDC-1330 Signed-off-by: eleonorali --- .../ComputeConsolidationDataHandler.java | 35 +++- .../consolidation/EntityConsolidationData.java | 172 ++++++++-------- .../heattotosca/ConsolidationDataUtil.java | 31 +-- .../ResourceTranslationNovaServerImpl.java | 14 +- .../ComputeConsolidationDataHandlerTest.java | 227 +++++++++++++++++++++ .../ComputeConsolidationDataTest.java | 153 ++++++++++---- .../ComputeTemplateConsolidationDataTest.java | 140 +++++++++++++ .../consolidation/EntityConsolidationDataTest.java | 42 ++-- .../FileComputeConsolidationDataTest.java | 144 ++++++++++--- .../FilePortConsolidationDataTest.java | 102 +++++++++ .../consolidation/PortConsolidationDataTest.java | 225 ++++++++++++++++++++ .../consolidation/ServiceTemplateFileNameEnum.java | 7 + .../TypeComputeConsolidationDataTest.java | 59 +++--- 13 files changed, 1112 insertions(+), 239 deletions(-) create mode 100644 openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeConsolidationDataHandlerTest.java create mode 100644 openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeTemplateConsolidationDataTest.java create mode 100644 openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/FilePortConsolidationDataTest.java create mode 100644 openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/PortConsolidationDataTest.java create mode 100644 openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ServiceTemplateFileNameEnum.java (limited to 'openecomp-be/lib') diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeConsolidationDataHandler.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeConsolidationDataHandler.java index dc2fc96169..8e319e96a0 100644 --- a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeConsolidationDataHandler.java +++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeConsolidationDataHandler.java @@ -94,26 +94,49 @@ public class ComputeConsolidationDataHandler implements ConsolidationDataHandler } /** - * Add port to compute consolidation data entity base on given keys.s + * Add port to compute consolidation data entity base on given keys. * */ public void addPortToConsolidationData(TranslateTo translateTo, String computeNodeType, String computeNodeTemplateId, String portType, String portNodeTemplateId) { - ComputeTemplateConsolidationData entityConsolidationData = + ComputeTemplateConsolidationData consolidationData = getComputeTemplateConsolidationData(translateTo, computeNodeType, computeNodeTemplateId); - entityConsolidationData.addPort(portType, portNodeTemplateId); + consolidationData.addPort(portType, portNodeTemplateId); + } + + /** + * Add volume to consolidation data. + * + */ + public void addVolumeToConsolidationData(TranslateTo translateTo, String computeNodeType, + String computeNodeTemplateId, String requirementId, RequirementAssignment requirementAssignment) { + ComputeTemplateConsolidationData consolidationData = + getComputeTemplateConsolidationData(translateTo, computeNodeType, + computeNodeTemplateId); + consolidationData.addVolume(requirementId, requirementAssignment); + } + + /** + * Add group id information to consolidation data. + * + * @param translatedGroupId Group id of which compute node is a part + */ + + public void addGroupIdToConsolidationData(TranslateTo translateTo, String computeNodeType, + String computeNodeTemplateId, String translatedGroupId) { + ComputeTemplateConsolidationData consolidationData = + getComputeTemplateConsolidationData(translateTo, computeNodeType, + computeNodeTemplateId); + consolidationData.addGroupId(translatedGroupId); } private ComputeTemplateConsolidationData getComputeTemplateConsolidationData( TranslateTo translateTo, String computeNodeType, String computeNodeTemplateId) { - ServiceTemplate serviceTemplate = translateTo.getServiceTemplate(); String serviceTemplateFileName = ToscaUtil.getServiceTemplateFileName(serviceTemplate); return getComputeTemplateConsolidationData(serviceTemplateFileName, computeNodeType, computeNodeTemplateId); - } - private ComputeTemplateConsolidationData getComputeTemplateConsolidationData( String serviceTemplateFileName, String computeNodeType, String computeNodeTemplateId) { diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/EntityConsolidationData.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/EntityConsolidationData.java index 7101037562..3987c54054 100644 --- a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/EntityConsolidationData.java +++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/EntityConsolidationData.java @@ -67,42 +67,42 @@ public class EntityConsolidationData { } /** - * Gets group ids point to me. - * - * @return the group ids point to me - */ + * Gets group ids point to me. + * + * @return the group ids point to me + */ public List getGroupIds() { return groupIds; } /** - * Sets group ids point to me. - * - * @param groupIds the group ids point to me - */ + * Sets group ids point to me. + * + * @param groupIds the group ids point to me + */ public void setGroupIds(List groupIds) { this.groupIds = groupIds; } /** - * Sets node connected to me. - * - * @param nodesConnectedIn the node connected to me - */ + * Sets node connected to me. + * + * @param nodesConnectedIn the node connected to me + */ public void setNodesConnectedIn(Map> nodesConnectedIn) { this.nodesConnectedIn = nodesConnectedIn; } /** - * Add nodeConnectedIn. - * - * @param nodeTemplateId the node template id which has connection to me - * @param requirementId the requirement id - * @param requirementAssignment the requirement assignment - */ + * Add node connected to me. + * + * @param nodeTemplateId the node template id which has connection to me + * @param requirementId the requirement id + * @param requirementAssignment the requirement assignment + */ public void addNodesConnectedIn(String nodeTemplateId, String requirementId, - RequirementAssignment requirementAssignment) { + RequirementAssignment requirementAssignment) { if (this.nodesConnectedIn == null) { this.nodesConnectedIn = new HashMap<>(); @@ -114,42 +114,42 @@ public class EntityConsolidationData { } /** - * Gets node connected to me. - * - * @return the node connected to me - */ + * Gets node connected to me. + * + * @return the node connected to me + */ public Map> getNodesConnectedIn() { return nodesConnectedIn; } /** - * Gets node connected from me. - * - * @return the node connected from me - */ + * Gets node connected from me. + * + * @return the node connected from me + */ public Map> getNodesConnectedOut() { return nodesConnectedOut; } /** - * Sets node connected from me. - * - * @param nodesConnectedOut the node connected from me - */ + * Sets node connected from me. + * + * @param nodesConnectedOut the node connected from me + */ public void setNodesConnectedOut(Map> nodesConnectedOut) { this.nodesConnectedOut = nodesConnectedOut; } /** - * Add nodeConnectedOut. - * - * @param nodeTemplateId the node template id which is connected from me - * @param requirementId the requirement id - * @param requirementAssignment the requirement assignment - */ + * Add nodeConnectedOut. + * + * @param nodeTemplateId the node template id which is connected from me + * @param requirementId the requirement id + * @param requirementAssignment the requirement assignment + */ public void addNodesConnectedOut(String nodeTemplateId, String requirementId, - RequirementAssignment requirementAssignment) { + RequirementAssignment requirementAssignment) { if (this.nodesConnectedOut == null) { this.nodesConnectedOut = new HashMap<>(); @@ -161,29 +161,29 @@ public class EntityConsolidationData { } /** - * Gets nodes get attr in. - * - * @return the get attr in - */ + * Gets nodes get attr in. + * + * @return the get attr in + */ public Map> getNodesGetAttrIn() { return nodesGetAttrIn; } /** - * Sets nodes get attr in. - * - * @param nodesGetAttrIn the get attr in - */ + * Sets nodes get attr in. + * + * @param nodesGetAttrIn the get attr in + */ public void setNodesGetAttrIn(Map> nodesGetAttrIn) { this.nodesGetAttrIn = nodesGetAttrIn; } /** - * Add nodes get attr in data. - * - * @param nodeTemplateId the node template id - * @param getAttrFuncData get attr data - */ + * Add nodes get attr in data. + * + * @param nodeTemplateId the node template id + * @param getAttrFuncData get attr data + */ public void addNodesGetAttrIn(String nodeTemplateId, GetAttrFuncData getAttrFuncData) { if (nodesGetAttrIn == null) { @@ -195,28 +195,28 @@ public class EntityConsolidationData { } /** - * Gets output parameters get attr from me. - * - * @return the get attr from me - */ + * Gets output parameters get attr from me. + * + * @return the get attr from me + */ public List getOutputParametersGetAttrIn() { return outputParametersGetAttrIn; } /** - * Sets output parameters get attr from me. - * - * @param outputParametersGetAttrIn the output parameters get attr from me - */ + * Sets output parameters get attr from me. + * + * @param outputParametersGetAttrIn the output parameters get attr from me + */ public void setOutputParametersGetAttrIn(List outputParametersGetAttrIn) { this.outputParametersGetAttrIn = outputParametersGetAttrIn; } /** - * Add output parameters get attr data. - * - * @param getAttrFuncData get attr data - */ + * Add output parameters get attr data. + * + * @param getAttrFuncData get attr data + */ public void addOutputParamGetAttrIn(GetAttrFuncData getAttrFuncData) { if (outputParametersGetAttrIn == null) { @@ -227,29 +227,29 @@ public class EntityConsolidationData { } /** - * Gets nodes get attr out. - * - * @return the get attr out - */ + * Gets nodes get attr out. + * + * @return the get attr out + */ public Map> getNodesGetAttrOut() { return nodesGetAttrOut; } /** - * Sets nodes get attr out. - * - * @param nodesGetAttrOut the get attr out - */ + * Sets nodes get attr out. + * + * @param nodesGetAttrOut the get attr out + */ public void setNodesGetAttrOut(Map> nodesGetAttrOut) { this.nodesGetAttrOut = nodesGetAttrOut; } /** - * Add nodes get attr out data. - * - * @param nodeTemplateId the node template id - * @param getAttrFuncData get attr data - */ + * Add nodes get attr out data. + * + * @param nodeTemplateId the node template id + * @param getAttrFuncData get attr data + */ public void addNodesGetAttrOut(String nodeTemplateId, GetAttrFuncData getAttrFuncData) { if (nodesGetAttrOut == null) { @@ -277,10 +277,8 @@ public class EntityConsolidationData { * @return true in case get attr list same for all port types. * otherwise return false */ - public boolean isGetAttrOutFromEntityLegal( - Collection entityConsolidationDataList, - Map> portTypeToIds) { - + public boolean isGetAttrOutFromEntityLegal(Collection + entityConsolidationDataList, Map> portTypeToIds) { for (String portType : portTypeToIds.keySet()) { Set startingGetAttrFunc = getEntityGetAttrFuncAsSet(portType); @@ -293,7 +291,6 @@ public class EntityConsolidationData { } } } - return true; } @@ -307,4 +304,17 @@ public class EntityConsolidationData { .flatMap(entry -> entry.getValue().stream()) .collect(Collectors.toSet()); } + + /** + * Add group id information to consolidation data. + * + * @param groupId Group id of which compute node is a part + */ + + void addGroupId(String groupId) { + if (groupIds == null) { + groupIds = new ArrayList<>(); + } + groupIds.add(groupId); + } } diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/ConsolidationDataUtil.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/ConsolidationDataUtil.java index 0d8a07a41f..b766f30388 100644 --- a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/ConsolidationDataUtil.java +++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/ConsolidationDataUtil.java @@ -16,7 +16,6 @@ package org.openecomp.sdc.translator.services.heattotosca; -import java.util.ArrayList; import java.util.List; import java.util.Map; import java.util.Objects; @@ -36,6 +35,7 @@ import org.openecomp.sdc.tosca.services.ToscaUtil; import org.openecomp.sdc.translator.datatypes.heattotosca.TranslationContext; import org.openecomp.sdc.translator.datatypes.heattotosca.to.TranslateTo; import org.openecomp.sdc.translator.datatypes.heattotosca.unifiedmodel.consolidation.ComputeConsolidationData; +import org.openecomp.sdc.translator.datatypes.heattotosca.unifiedmodel.consolidation.ComputeConsolidationDataHandler; import org.openecomp.sdc.translator.datatypes.heattotosca.unifiedmodel.consolidation.ComputeTemplateConsolidationData; import org.openecomp.sdc.translator.datatypes.heattotosca.unifiedmodel.consolidation.ConsolidationData; import org.openecomp.sdc.translator.datatypes.heattotosca.unifiedmodel.consolidation.ConsolidationDataHandler; @@ -282,21 +282,6 @@ public class ConsolidationDataUtil { .contains(nestedNodeTemplateId); } - /** - * Update group id information in consolidation data. - * - * @param entityConsolidationData Entity consolidation data (Port/Compute) - * @param translatedGroupId Group id of which compute node is a part - */ - public static void updateGroupIdInConsolidationData(EntityConsolidationData - entityConsolidationData, - String translatedGroupId) { - if (entityConsolidationData.getGroupIds() == null) { - entityConsolidationData.setGroupIds(new ArrayList<>()); - } - entityConsolidationData.getGroupIds().add(translatedGroupId); - } - /** * Update volume information in consolidation data. * @@ -309,17 +294,13 @@ public class ConsolidationDataUtil { String computeType, String computeNodeTemplateId, String requirementId, - RequirementAssignment - requirementAssignment) { - TranslationContext translationContext = translateTo.getContext(); - ServiceTemplate serviceTemplate = translateTo.getServiceTemplate(); - ComputeTemplateConsolidationData computeTemplateConsolidationData = - getComputeTemplateConsolidationData(translationContext, serviceTemplate, computeType, - computeNodeTemplateId); - computeTemplateConsolidationData.addVolume(requirementId, requirementAssignment); + RequirementAssignment requirementAssignment) { + ComputeConsolidationDataHandler handler = + translateTo.getContext().getComputeConsolidationDataHandler(); + handler.addVolumeToConsolidationData( + translateTo, computeType, computeNodeTemplateId, requirementId, requirementAssignment); } - /** * Update port in consolidation data. * diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/impl/resourcetranslation/ResourceTranslationNovaServerImpl.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/impl/resourcetranslation/ResourceTranslationNovaServerImpl.java index d6f0d7a2b8..e70ca5f2f9 100644 --- a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/impl/resourcetranslation/ResourceTranslationNovaServerImpl.java +++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/main/java/org/openecomp/sdc/translator/services/heattotosca/impl/resourcetranslation/ResourceTranslationNovaServerImpl.java @@ -55,7 +55,7 @@ import org.openecomp.sdc.translator.datatypes.heattotosca.PropertyRegexMatcher; import org.openecomp.sdc.translator.datatypes.heattotosca.TranslationContext; import org.openecomp.sdc.translator.datatypes.heattotosca.to.TranslateTo; import org.openecomp.sdc.translator.datatypes.heattotosca.to.TranslatedHeatResource; -import org.openecomp.sdc.translator.datatypes.heattotosca.unifiedmodel.consolidation.ComputeTemplateConsolidationData; +import org.openecomp.sdc.translator.datatypes.heattotosca.unifiedmodel.consolidation.ComputeConsolidationDataHandler; import org.openecomp.sdc.translator.services.heattotosca.ConsolidationDataUtil; import org.openecomp.sdc.translator.services.heattotosca.Constants; import org.openecomp.sdc.translator.services.heattotosca.HeatToToscaUtil; @@ -167,14 +167,10 @@ public class ResourceTranslationNovaServerImpl extends ResourceTranslationBase { } private void updateComputeConsolidationDataGroup(TranslateTo translateTo, - NodeTemplate novaNodeTemplate, - String groupId) { - TranslationContext translationContext = translateTo.getContext(); - ServiceTemplate serviceTemplate = translateTo.getServiceTemplate(); - ComputeTemplateConsolidationData computeTemplateConsolidationData = ConsolidationDataUtil - .getComputeTemplateConsolidationData(translationContext, serviceTemplate, - novaNodeTemplate.getType(), translateTo.getTranslatedId()); - ConsolidationDataUtil.updateGroupIdInConsolidationData(computeTemplateConsolidationData, groupId); + NodeTemplate novaNodeTemplate, String groupId) { + ComputeConsolidationDataHandler handler = translateTo.getContext().getComputeConsolidationDataHandler(); + handler.addGroupIdToConsolidationData(translateTo,novaNodeTemplate.getType(), + translateTo.getTranslatedId(), groupId); } private boolean isHintOfTypeNovaServerGroup(HeatOrchestrationTemplate heatOrchestrationTemplate, diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeConsolidationDataHandlerTest.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeConsolidationDataHandlerTest.java new file mode 100644 index 0000000000..6df781e375 --- /dev/null +++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeConsolidationDataHandlerTest.java @@ -0,0 +1,227 @@ +/* + * Copyright © 2016-2018 European Support Limited + * + * 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. + */ + +package org.openecomp.sdc.translator.datatypes.heattotosca.unifiedmodel.consolidation; + +import java.util.HashMap; +import java.util.Map; + +import org.junit.Before; +import org.junit.Test; +import org.mockito.Mock; +import org.mockito.Mockito; +import org.mockito.MockitoAnnotations; + +import org.onap.sdc.tosca.datatypes.model.NodeTemplate; +import org.onap.sdc.tosca.datatypes.model.RequirementAssignment; +import org.onap.sdc.tosca.datatypes.model.ServiceTemplate; +import org.onap.sdc.tosca.datatypes.model.TopologyTemplate; +import org.openecomp.sdc.heat.datatypes.model.HeatOrchestrationTemplate; +import org.openecomp.sdc.tosca.services.ToscaConstants; +import org.openecomp.sdc.translator.datatypes.heattotosca.to.TranslateTo; + + +public class ComputeConsolidationDataHandlerTest { + + private static final String COMPUTE_NODE_TEMPLATE_ID_1 = "computeNodeTemplateId"; + private static final String COMPUTE_NODE_TEMPLATE_ID_2 = "computeNodeTemplateId2"; + private static final String COMPUTE_NODE_TYPE_1 = "computeNodeType1"; + + private static final String PORT_NODE_TYPE_1 = "portType1"; + private static final String PORT_NODE_TEMPLATE_ID_1 = "portNodeTemplateId1"; + + private static final String GROUP_ID = "groupId"; + private static final String REQUIREMENT_ID = "requirementId"; + private static final String SERVICE_FILE_NAME_PREFIX = "Main"; + private static final String SERVICE_FILE_NAME = SERVICE_FILE_NAME_PREFIX + "ServiceTemplate.yaml"; + + @Before + public void setUp() { + MockitoAnnotations.initMocks(this); + consolidationDataHandler = new ComputeConsolidationDataHandler(computeConsolidationData); + } + + @Mock + private static ComputeConsolidationData computeConsolidationData; + @Mock + private static TranslateTo translateTo; + @Mock + private static ServiceTemplate serviceTemplate; + @Mock + private static ComputeTemplateConsolidationData consolidationData; + @Mock + private static HeatOrchestrationTemplate heatOrchestrationTemplate; + + private static ComputeConsolidationDataHandler consolidationDataHandler; + private static final RequirementAssignment requirementAssignment = new RequirementAssignment(); + + @Test + public void testAddNodesConnectedOut() { + mockEntities(COMPUTE_NODE_TEMPLATE_ID_1); + consolidationDataHandler.addNodesConnectedOut( + translateTo, COMPUTE_NODE_TEMPLATE_ID_1, REQUIREMENT_ID, requirementAssignment); + Mockito.verify(computeConsolidationData).addComputeTemplateConsolidationData( + SERVICE_FILE_NAME, COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); + Mockito.verify(consolidationData).addNodesConnectedOut( + COMPUTE_NODE_TEMPLATE_ID_1, REQUIREMENT_ID, requirementAssignment); + } + + @Test + public void testAddNodesConnectedOut_consolidationDataNotExist() { + mockEntities_NullConsolidationData(COMPUTE_NODE_TEMPLATE_ID_1); + consolidationDataHandler.addNodesConnectedOut( + translateTo, COMPUTE_NODE_TEMPLATE_ID_1, REQUIREMENT_ID, requirementAssignment); + Mockito.verify(computeConsolidationData).addComputeTemplateConsolidationData( + SERVICE_FILE_NAME, COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); + Mockito.verify(consolidationData, Mockito.times(0)).addNodesConnectedOut( + Mockito.any(), Mockito.any(), Mockito.any()); + } + + @Test + public void testAddNodesConnectedIn() { + String sourceNodeTemplateId = COMPUTE_NODE_TEMPLATE_ID_1; + String dependentNodeTemplateId = COMPUTE_NODE_TEMPLATE_ID_2; + mockEntities(dependentNodeTemplateId); + consolidationDataHandler.addNodesConnectedIn(translateTo, sourceNodeTemplateId, + dependentNodeTemplateId, "targetResourceId", REQUIREMENT_ID, requirementAssignment); + Mockito.verify(computeConsolidationData).addComputeTemplateConsolidationData( + SERVICE_FILE_NAME, COMPUTE_NODE_TYPE_1, dependentNodeTemplateId); + Mockito.verify(consolidationData).addNodesConnectedIn( + sourceNodeTemplateId, REQUIREMENT_ID, requirementAssignment); + } + + @Test + public void testAddNodesConnectedIn_consolidationDataNotExist() { + String dependentNodeTemplateId = COMPUTE_NODE_TEMPLATE_ID_2; + mockEntities_NullConsolidationData(dependentNodeTemplateId); + consolidationDataHandler.addNodesConnectedIn(translateTo, COMPUTE_NODE_TEMPLATE_ID_1, + dependentNodeTemplateId, "targetResourceId", REQUIREMENT_ID, requirementAssignment); + Mockito.verify(computeConsolidationData).addComputeTemplateConsolidationData( + SERVICE_FILE_NAME, COMPUTE_NODE_TYPE_1, dependentNodeTemplateId); + Mockito.verify(consolidationData, Mockito.times(0)).addNodesConnectedIn( + Mockito.any(), Mockito.any(), Mockito.any()); + } + + @Test + public void testRemoveParamNameFromAttrFuncList() { + mockEntities(COMPUTE_NODE_TEMPLATE_ID_2); + consolidationDataHandler.removeParamNameFromAttrFuncList(serviceTemplate, heatOrchestrationTemplate, + "paramName", COMPUTE_NODE_TEMPLATE_ID_1, COMPUTE_NODE_TEMPLATE_ID_2); + Mockito.verify(computeConsolidationData).addComputeTemplateConsolidationData( + SERVICE_FILE_NAME, COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_2); + Mockito.verify(consolidationData).removeParamNameFromAttrFuncList("paramName"); + } + + @Test + public void testRemoveParamNameFromAttrFuncList_consolidationDataNotExist() { + mockEntities_NullConsolidationData(COMPUTE_NODE_TEMPLATE_ID_2); + consolidationDataHandler.removeParamNameFromAttrFuncList(serviceTemplate, heatOrchestrationTemplate, + "paramName", COMPUTE_NODE_TEMPLATE_ID_1, COMPUTE_NODE_TEMPLATE_ID_2); + Mockito.verify(computeConsolidationData).addComputeTemplateConsolidationData( + SERVICE_FILE_NAME, COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_2); + Mockito.verify(consolidationData, Mockito.times(0)) + .removeParamNameFromAttrFuncList(Mockito.any()); + } + + @Test + public void testAddConsolidationData() { + consolidationDataHandler.addConsolidationData(SERVICE_FILE_NAME, + COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); + Mockito.verify(computeConsolidationData).addComputeTemplateConsolidationData( + SERVICE_FILE_NAME, COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); + } + + @Test + public void testAddPortToConsolidationData() { + mockEntities(COMPUTE_NODE_TEMPLATE_ID_1); + consolidationDataHandler.addPortToConsolidationData(translateTo, COMPUTE_NODE_TYPE_1, + COMPUTE_NODE_TEMPLATE_ID_1, PORT_NODE_TYPE_1, PORT_NODE_TEMPLATE_ID_1); + Mockito.verify(computeConsolidationData).addComputeTemplateConsolidationData( + SERVICE_FILE_NAME, COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); + Mockito.verify(consolidationData).addPort(PORT_NODE_TYPE_1, PORT_NODE_TEMPLATE_ID_1); + } + + @Test + public void testAddVolumeToConsolidationData() { + mockEntities(COMPUTE_NODE_TEMPLATE_ID_1); + consolidationDataHandler.addVolumeToConsolidationData( + translateTo, COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1, + REQUIREMENT_ID, requirementAssignment); + Mockito.verify(computeConsolidationData).addComputeTemplateConsolidationData( + SERVICE_FILE_NAME, COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); + Mockito.verify(consolidationData).addVolume(REQUIREMENT_ID, requirementAssignment); + } + + + @Test + public void testAddGroupIdToConsolidationData() { + mockEntities(COMPUTE_NODE_TEMPLATE_ID_1); + consolidationDataHandler.addGroupIdToConsolidationData( + translateTo, COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1, GROUP_ID); + Mockito.verify(computeConsolidationData).addComputeTemplateConsolidationData( + SERVICE_FILE_NAME, COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); + Mockito.verify(consolidationData).addGroupId(GROUP_ID); + } + + private void mockEntities(String nodeTemplateId) { + mockServiceTemplate(nodeTemplateId); + mockTranslateTo(nodeTemplateId); + mockComputeConsolidationData(); + } + + private void mockEntities_NullConsolidationData(String nodeTemplateId) { + mockServiceTemplate(nodeTemplateId); + mockTranslateTo(nodeTemplateId); + mockNullConsolidationData(); + } + + private void mockServiceTemplate(String nodeTemplateId) { + TopologyTemplate topologyTemplate = createTopologyTemplate(nodeTemplateId); + Mockito.when(serviceTemplate.getTopology_template()).thenReturn(topologyTemplate); + + Map metadata = new HashMap<>(); + metadata.put(ToscaConstants.ST_METADATA_TEMPLATE_NAME, SERVICE_FILE_NAME_PREFIX); + Mockito.when(serviceTemplate.getMetadata()).thenReturn(metadata); + } + + private void mockTranslateTo(String nodeTemplateId) { + Mockito.when(translateTo.getTranslatedId()).thenReturn(nodeTemplateId); + Mockito.when(translateTo.getServiceTemplate()).thenReturn(serviceTemplate); + } + + private void mockComputeConsolidationData() { + Mockito.when(computeConsolidationData + .addComputeTemplateConsolidationData(Mockito.anyString(), Mockito.anyString(), + Mockito.anyString())).thenReturn(consolidationData); + } + + private void mockNullConsolidationData() { + Mockito.when(computeConsolidationData + .addComputeTemplateConsolidationData(Mockito.anyString(), Mockito.anyString(), + Mockito.anyString())).thenReturn(null); + } + + private TopologyTemplate createTopologyTemplate(String nodeTemplateId) { + TopologyTemplate topologyTemplate = new TopologyTemplate(); + Map nodeTemplates = new HashMap<>(); + NodeTemplate nodeTemplate = new NodeTemplate(); + nodeTemplate.setType(COMPUTE_NODE_TYPE_1); + nodeTemplates.put(nodeTemplateId, nodeTemplate); + topologyTemplate.setNode_templates(nodeTemplates); + return topologyTemplate; + } + +} diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeConsolidationDataTest.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeConsolidationDataTest.java index a92e298c0a..32e9f0d30d 100644 --- a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeConsolidationDataTest.java +++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeConsolidationDataTest.java @@ -16,83 +16,125 @@ package org.openecomp.sdc.translator.datatypes.heattotosca.unifiedmodel.consolidation; +import java.util.EnumMap; import java.util.HashSet; import java.util.Set; import org.junit.Assert; +import org.junit.Before; import org.junit.Test; +import org.mockito.Mock; +import org.mockito.Mockito; +import org.mockito.MockitoAnnotations; public class ComputeConsolidationDataTest { - private static final String SERVICE_TEMPLATE_FILE_NAME_1 = "serviceTemplateFileName1"; - private static final String SERVICE_TEMPLATE_FILE_NAME_2 = "serviceTemplateFileName2"; private static final String COMPUTE_NODE_TEMPLATE_ID_1 = "computeNodeTemplateId1"; private static final String COMPUTE_NODE_TEMPLATE_ID_2 = "computeNodeTemplateId2"; private static final String COMPUTE_NODE_TYPE_1 = "computeNodeType1"; private static final String COMPUTE_NODE_TYPE_2 = "computeNodeType2"; + private final EnumMap mockMap = + new EnumMap<>(ServiceTemplateFileNameEnum.class); + + @Mock + private FileComputeConsolidationData mockFileComputeConsolidationData1; + @Mock + private FileComputeConsolidationData mockFileComputeConsolidationData2; + + private final ComputeConsolidationData consolidationData = new ComputeConsolidationData(); + + @Before + public void setUp() { + MockitoAnnotations.initMocks(this); + addMocksToMap(); + } + @Test public void testAddComputeTemplateConsolidationData() { - ComputeConsolidationData consolidationData = new ComputeConsolidationData(); - consolidationData.addComputeTemplateConsolidationData(SERVICE_TEMPLATE_FILE_NAME_1, - COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); + String serviceTemplate = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(); + consolidationData.addComputeTemplateConsolidationData( + serviceTemplate, COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); Set expectedServiceTemplateNames = new HashSet<>(); - expectedServiceTemplateNames.add(SERVICE_TEMPLATE_FILE_NAME_1); - checkComputeConsolidationData(consolidationData, SERVICE_TEMPLATE_FILE_NAME_1, expectedServiceTemplateNames); + expectedServiceTemplateNames.add(serviceTemplate); + checkComputeConsolidationData(consolidationData, serviceTemplate, + expectedServiceTemplateNames); } @Test public void testAddSameConsolidationDataTwice_noNewCreated() { - ComputeConsolidationData consolidationData = new ComputeConsolidationData(); - consolidationData.addComputeTemplateConsolidationData(SERVICE_TEMPLATE_FILE_NAME_1, + String serviceTemplate = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(); + consolidationData.addComputeTemplateConsolidationData(serviceTemplate, COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); - consolidationData.getFileComputeConsolidationData(SERVICE_TEMPLATE_FILE_NAME_1); - Set expectedServiceTemplateNames = new HashSet<>(); - expectedServiceTemplateNames.add(SERVICE_TEMPLATE_FILE_NAME_1); - checkComputeConsolidationData(consolidationData, SERVICE_TEMPLATE_FILE_NAME_1, expectedServiceTemplateNames); + expectedServiceTemplateNames.add(serviceTemplate); + checkComputeConsolidationData(consolidationData, serviceTemplate, expectedServiceTemplateNames); - consolidationData.addComputeTemplateConsolidationData(SERVICE_TEMPLATE_FILE_NAME_1, + consolidationData.addComputeTemplateConsolidationData(serviceTemplate, COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); - consolidationData.getFileComputeConsolidationData(SERVICE_TEMPLATE_FILE_NAME_1); - checkComputeConsolidationData(consolidationData, SERVICE_TEMPLATE_FILE_NAME_1, expectedServiceTemplateNames); + checkComputeConsolidationData(consolidationData, serviceTemplate, expectedServiceTemplateNames); } @Test - public void testAddDiffConsolidationData_SameServiceTemplate_DiffNodeTypes() { - ComputeConsolidationData consolidationData = new ComputeConsolidationData(); - - consolidationData.addComputeTemplateConsolidationData(SERVICE_TEMPLATE_FILE_NAME_1, + public void testAddDiffConsolidationData_diffNodeTypes() { + String serviceTemplate = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(); + consolidationData.addComputeTemplateConsolidationData(serviceTemplate, COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); Set expectedServiceTemplateNames = new HashSet<>(); - expectedServiceTemplateNames.add(SERVICE_TEMPLATE_FILE_NAME_1); - checkComputeConsolidationData(consolidationData, SERVICE_TEMPLATE_FILE_NAME_1, expectedServiceTemplateNames); + expectedServiceTemplateNames.add(serviceTemplate); + checkComputeConsolidationData(consolidationData, serviceTemplate, expectedServiceTemplateNames); - consolidationData.addComputeTemplateConsolidationData(SERVICE_TEMPLATE_FILE_NAME_1, + consolidationData.addComputeTemplateConsolidationData(serviceTemplate, COMPUTE_NODE_TYPE_2, COMPUTE_NODE_TEMPLATE_ID_2); - checkComputeConsolidationData(consolidationData, SERVICE_TEMPLATE_FILE_NAME_1, expectedServiceTemplateNames); + checkComputeConsolidationData(consolidationData, serviceTemplate, expectedServiceTemplateNames); } @Test - public void testAddDiffConsolidationData_DiffServiceTemplate() { - ComputeConsolidationData consolidationData = new ComputeConsolidationData(); - - consolidationData.addComputeTemplateConsolidationData(SERVICE_TEMPLATE_FILE_NAME_1, - COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); + public void testAddDiffConsolidationData_diffServiceTemplate() { + String serviceTemplate1 = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(); + consolidationData.addComputeTemplateConsolidationData(serviceTemplate1, COMPUTE_NODE_TYPE_1, + COMPUTE_NODE_TEMPLATE_ID_1); Set expectedServiceTemplateNames = new HashSet<>(); - expectedServiceTemplateNames.add(SERVICE_TEMPLATE_FILE_NAME_1); - checkComputeConsolidationData(consolidationData, SERVICE_TEMPLATE_FILE_NAME_1, expectedServiceTemplateNames); + expectedServiceTemplateNames.add(serviceTemplate1); + checkComputeConsolidationData(consolidationData, serviceTemplate1, expectedServiceTemplateNames); + + String serviceTemplate2 = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_2.name(); + consolidationData.addComputeTemplateConsolidationData(serviceTemplate2, COMPUTE_NODE_TYPE_2, + COMPUTE_NODE_TEMPLATE_ID_2); + expectedServiceTemplateNames.add(serviceTemplate2); + checkComputeConsolidationData(consolidationData, serviceTemplate2, expectedServiceTemplateNames); + } - consolidationData.addComputeTemplateConsolidationData(SERVICE_TEMPLATE_FILE_NAME_2, - COMPUTE_NODE_TYPE_2, COMPUTE_NODE_TEMPLATE_ID_2); - expectedServiceTemplateNames.add(SERVICE_TEMPLATE_FILE_NAME_2); - checkComputeConsolidationData(consolidationData, SERVICE_TEMPLATE_FILE_NAME_2, expectedServiceTemplateNames); + @Test + public void testAddSameConsolidationDataTwice_testWithMock() throws Exception { + String serviceTemplate = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(); + addAndCheckComputeTemplateConsolidationData(serviceTemplate, COMPUTE_NODE_TYPE_1, + COMPUTE_NODE_TEMPLATE_ID_1); + addAndCheckComputeTemplateConsolidationData(serviceTemplate, COMPUTE_NODE_TYPE_1, + COMPUTE_NODE_TEMPLATE_ID_1, 2); + } + @Test + public void testAddDiffConsolidationData_diffNodeTypes_testWithMock() throws Exception { + String serviceTemplate = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(); + addAndCheckComputeTemplateConsolidationData(serviceTemplate, COMPUTE_NODE_TYPE_1, + COMPUTE_NODE_TEMPLATE_ID_1); + addAndCheckComputeTemplateConsolidationData(serviceTemplate, COMPUTE_NODE_TYPE_2, + COMPUTE_NODE_TEMPLATE_ID_2); + } + + @Test + public void testAddDiffConsolidationData_diffServiceTemplate_testWithMock() throws Exception { + String serviceTemplate1 = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(); + addAndCheckComputeTemplateConsolidationData(serviceTemplate1, COMPUTE_NODE_TYPE_1, + COMPUTE_NODE_TEMPLATE_ID_1); + String serviceTemplate2 = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(); + addAndCheckComputeTemplateConsolidationData(serviceTemplate2, COMPUTE_NODE_TYPE_2, + COMPUTE_NODE_TEMPLATE_ID_2); } private void checkComputeConsolidationData(ComputeConsolidationData consolidationData, String serviceTemplateFileName, Set expectedServiceTemplateNames) { - FileComputeConsolidationData fileComputeConsolidationData = consolidationData .getFileComputeConsolidationData(serviceTemplateFileName); Assert.assertNotNull(fileComputeConsolidationData); @@ -107,5 +149,44 @@ public class ComputeConsolidationDataTest { Assert.assertTrue(allServiceTemplateFileNames.containsAll(expectedServiceTemplateNames)); } + private void addAndCheckComputeTemplateConsolidationData(String serviceTemplateFileName, + String computeNodeType, String computeNodeTemplateId) throws Exception { + addAndCheckComputeTemplateConsolidationData(serviceTemplateFileName, computeNodeType, + computeNodeTemplateId, 1); + } + + private void addAndCheckComputeTemplateConsolidationData(String serviceTemplateFileName, + String computeNodeType, String computeNodeTemplateId, int expectedTime) throws Exception { + FileComputeConsolidationData fileComputeConsolidationDataMock = + setFileComputeConsolidationDataMock(serviceTemplateFileName); + consolidationData.addComputeTemplateConsolidationData( + serviceTemplateFileName, computeNodeType, computeNodeTemplateId); + + Mockito.verify(fileComputeConsolidationDataMock, Mockito.times(expectedTime)) + .addComputeTemplateConsolidationData(computeNodeType, computeNodeTemplateId); + } + + private FileComputeConsolidationData setFileComputeConsolidationDataMock( + String serviceTemplateName) throws Exception { + FileComputeConsolidationData mock = getFileComputeConsolidationDataMock(serviceTemplateName); + consolidationData.setFileComputeConsolidationData(serviceTemplateName, mock); + return mock; + } + + private void addMocksToMap() { + mockMap.put(ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1, mockFileComputeConsolidationData1); + mockMap.put(ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_2, mockFileComputeConsolidationData2); + } + + private FileComputeConsolidationData getFileComputeConsolidationDataMock(String serviceTemplateName) + throws Exception { + ServiceTemplateFileNameEnum enumValue = ServiceTemplateFileNameEnum.valueOf(serviceTemplateName); + FileComputeConsolidationData mock = mockMap.get(enumValue); + if (mock == null) { + throw new Exception("This service Template File Name doesn't supported. " + + "Please add it to ServiceTemplateFileName enum"); + } + return mock; + } } diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeTemplateConsolidationDataTest.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeTemplateConsolidationDataTest.java new file mode 100644 index 0000000000..9f32aec8aa --- /dev/null +++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ComputeTemplateConsolidationDataTest.java @@ -0,0 +1,140 @@ +/* + * Copyright © 2016-2018 European Support Limited + * + * 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. + */ + +package org.openecomp.sdc.translator.datatypes.heattotosca.unifiedmodel.consolidation; + +import java.util.Arrays; +import java.util.HashMap; +import java.util.List; +import java.util.Map; +import org.junit.Assert; +import org.junit.Test; + +import org.onap.sdc.tosca.datatypes.model.RequirementAssignment; + +public class ComputeTemplateConsolidationDataTest { + + private static final String COMPUTE_NODE_TEMPLATE_ID_1 = "computeNodeTemplateId1"; + private static final String COMPUTE_NODE_TEMPLATE_ID_2 = "computeNodeTemplateId2"; + private static final String REQUIREMENT_ID_1 = "requirementId1"; + private static final String REQUIREMENT_ID_2 = "requirementId2"; + + private static final String PORT_TYPE_1 = "portType1"; + private static final String PORT_TYPE_2 = "portType2"; + private static final String PORT_NODE_TEMPLATE_ID_1 = "portNodeTemplateId1"; + private static final String PORT_NODE_TEMPLATE_ID_2 = "portNodeTemplateId2"; + + + private final ComputeTemplateConsolidationData consolidationData + = new ComputeTemplateConsolidationData(); + + @Test + public void testAddPort_SamePortType() { + Map expectedPorts = new HashMap<>(); + expectedPorts.put(PORT_TYPE_1, new String[]{PORT_NODE_TEMPLATE_ID_1}); + addAndCheckPorts(PORT_TYPE_1, PORT_NODE_TEMPLATE_ID_1, expectedPorts); + + expectedPorts.put(PORT_TYPE_1, new String[]{PORT_NODE_TEMPLATE_ID_1, PORT_NODE_TEMPLATE_ID_2}); + addAndCheckPorts(PORT_TYPE_1, PORT_NODE_TEMPLATE_ID_2, expectedPorts); + } + + @Test + public void testAddPort_DiffPortType() { + Map expectedPorts = new HashMap<>(); + expectedPorts.put(PORT_TYPE_1, new String[]{PORT_NODE_TEMPLATE_ID_1}); + addAndCheckPorts(PORT_TYPE_1, PORT_NODE_TEMPLATE_ID_1, expectedPorts); + + expectedPorts.put(PORT_TYPE_2, new String[]{PORT_NODE_TEMPLATE_ID_2}); + addAndCheckPorts(PORT_TYPE_2, PORT_NODE_TEMPLATE_ID_2, expectedPorts); + } + + @Test + public void testAddVolume_SameComputeNode() { + Map expectedVolumes = new HashMap<>(); + expectedVolumes.put(COMPUTE_NODE_TEMPLATE_ID_1, new String[]{REQUIREMENT_ID_1}); + addAndCheckVolume(REQUIREMENT_ID_1, COMPUTE_NODE_TEMPLATE_ID_1, expectedVolumes); + + expectedVolumes.put(COMPUTE_NODE_TEMPLATE_ID_1, new String[]{REQUIREMENT_ID_1, REQUIREMENT_ID_2}); + addAndCheckVolume(REQUIREMENT_ID_2, COMPUTE_NODE_TEMPLATE_ID_1, expectedVolumes); + } + + @Test + public void testAddVolume_DiffComputeNode() { + Map expectedVolumes = new HashMap<>(); + expectedVolumes.put(COMPUTE_NODE_TEMPLATE_ID_1, new String[]{REQUIREMENT_ID_1}); + addAndCheckVolume(REQUIREMENT_ID_1, COMPUTE_NODE_TEMPLATE_ID_1, expectedVolumes); + + expectedVolumes.put(COMPUTE_NODE_TEMPLATE_ID_2, new String[]{REQUIREMENT_ID_2}); + addAndCheckVolume(REQUIREMENT_ID_2, COMPUTE_NODE_TEMPLATE_ID_2, expectedVolumes); + } + + private void addAndCheckPorts(String portType, String portNodeTemplateId, + Map expectedPorts) { + consolidationData.addPort(portType, portNodeTemplateId); + checkPorts(consolidationData.getPorts(), expectedPorts); + } + + private void checkPorts(Map> actualAllPorts, + Map expectedAllPorts) { + Assert.assertNotNull(actualAllPorts); + expectedAllPorts.keySet().forEach(expectedPortType -> { + Assert.assertTrue(actualAllPorts.containsKey(expectedPortType)); + Assert.assertEquals(expectedAllPorts.size(), actualAllPorts.size()); + checkPortsPerType(actualAllPorts, expectedAllPorts, expectedPortType); + }); + } + + private void checkPortsPerType(Map> actualAllPorts, Map expectedAllPorts, + String expectedPortType) { + List actualPorts = actualAllPorts.get(expectedPortType); + List expectedPortList = Arrays.asList(expectedAllPorts.get(expectedPortType)); + Assert.assertEquals(expectedPortList.size(), actualPorts.size()); + actualPorts.forEach(actualPort -> + Assert.assertTrue(expectedPortList.contains(actualPort))); + } + + private void addAndCheckVolume(String requirementId, String computeNodeTemplateId, + Map expectedVolumes) { + RequirementAssignment requirementAssignment1 = createRequirement(computeNodeTemplateId); + consolidationData.addVolume(requirementId, requirementAssignment1); + checkVolumes(consolidationData.getVolumes(), expectedVolumes); + } + + private void checkVolumes(Map> actualVolumes, + Map expectedVolumes) { + Assert.assertNotNull(actualVolumes); + expectedVolumes.keySet().forEach(nodeTemplateId -> { + Assert.assertTrue(actualVolumes.containsKey(nodeTemplateId)); + Assert.assertEquals(expectedVolumes.size(), actualVolumes.size()); + checkVolumesPerType(actualVolumes, expectedVolumes, nodeTemplateId); + }); + } + + private void checkVolumesPerType(Map> + actualVolumes, Map expectedVolumes, String nodeTemplateId) { + List actualRequirementAssignmentData = actualVolumes.get(nodeTemplateId); + List requirementIds = Arrays.asList(expectedVolumes.get(nodeTemplateId)); + Assert.assertEquals(requirementIds.size(), actualRequirementAssignmentData.size()); + actualRequirementAssignmentData.forEach(actualRequirementAssignment -> + Assert.assertTrue(requirementIds.contains(actualRequirementAssignment.getRequirementId()))); + } + + private RequirementAssignment createRequirement(String nodeTemplateId) { + RequirementAssignment requirementAssignment = new RequirementAssignment(); + requirementAssignment.setNode(nodeTemplateId); + return requirementAssignment; + } +} diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/EntityConsolidationDataTest.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/EntityConsolidationDataTest.java index fd96f93bb5..447314c25f 100644 --- a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/EntityConsolidationDataTest.java +++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/EntityConsolidationDataTest.java @@ -34,6 +34,13 @@ public class EntityConsolidationDataTest { private static final String NODE_TEMPLATE_ID_2 = "nodeTemplateId2"; private static final String REQUIREMENT_ID_1 = "requirementId1"; private static final String REQUIREMENT_ID_2 = "requirementId2"; + private static final String GROUP_ID_1 = "groupId1"; + private static final String GROUP_ID_2 = "groupId2"; + private static final String FIELD_1 = "field1"; + private static final String FIELD_2 = "field2"; + + + private final EntityConsolidationData consolidationData = new EntityConsolidationData(); @Test public void testIsGetAttrOutFromEntityLegal() { @@ -74,7 +81,6 @@ public class EntityConsolidationDataTest { getAttOutMap.put("server_0_networkrole_1_port", Collections.singletonList(getAttrFuncData)); GetAttrFuncData getAttrFuncData1 = new GetAttrFuncData("vmac_address", "accessIPv4"); - Map> getAttOutMap1 = new HashMap<>(); getAttOutMap.put("server_0_networkrole_2_port", Collections.singletonList(getAttrFuncData1)); @@ -82,7 +88,6 @@ public class EntityConsolidationDataTest { entityConsolidationData.setNodesGetAttrOut(getAttOutMap); EntityConsolidationData entityConsolidationData1 = new EntityConsolidationData(); - //entityConsolidationData1.setNodesGetAttrOut(getAttOutMap1); List entityConsolidationDataList = Arrays.asList(entityConsolidationData, entityConsolidationData1); @@ -125,9 +130,7 @@ public class EntityConsolidationDataTest { @Test public void testAddNodesConnectedIn_SameNodeTemplateIds() { - EntityConsolidationData consolidationData = new EntityConsolidationData(); Map expectedNodesConnectedData = new HashMap<>(); - addNodesConnectedIn(consolidationData,NODE_TEMPLATE_ID_1, REQUIREMENT_ID_1); expectedNodesConnectedData.put(NODE_TEMPLATE_ID_1, new String[]{REQUIREMENT_ID_1}); checkNodesConnected(consolidationData.getNodesConnectedIn(), expectedNodesConnectedData); @@ -139,9 +142,7 @@ public class EntityConsolidationDataTest { @Test public void testAddNodesConnectedIn_DiffNodeTemplateIds() { - EntityConsolidationData consolidationData = new EntityConsolidationData(); Map expectedNodesConnectedData = new HashMap<>(); - addNodesConnectedIn(consolidationData, NODE_TEMPLATE_ID_1, REQUIREMENT_ID_1); expectedNodesConnectedData.put(NODE_TEMPLATE_ID_1, new String[]{REQUIREMENT_ID_1}); checkNodesConnected(consolidationData.getNodesConnectedIn(), expectedNodesConnectedData); @@ -153,9 +154,7 @@ public class EntityConsolidationDataTest { @Test public void testAddNodesConnectedOut_SameNodeTemplateIds() { - EntityConsolidationData consolidationData = new EntityConsolidationData(); Map expectedNodesConnectedData = new HashMap<>(); - addNodesConnectedOut(consolidationData, NODE_TEMPLATE_ID_1, REQUIREMENT_ID_1); expectedNodesConnectedData.put(NODE_TEMPLATE_ID_1, new String[]{REQUIREMENT_ID_1}); checkNodesConnected(consolidationData.getNodesConnectedOut(), expectedNodesConnectedData); @@ -167,9 +166,7 @@ public class EntityConsolidationDataTest { @Test public void testAddNodesConnectedOut_DiffNodeTemplateIds() { - EntityConsolidationData consolidationData = new EntityConsolidationData(); Map expectedNodesConnectedData = new HashMap<>(); - addNodesConnectedOut(consolidationData, NODE_TEMPLATE_ID_1, REQUIREMENT_ID_1); expectedNodesConnectedData.put(NODE_TEMPLATE_ID_1, new String[]{REQUIREMENT_ID_1}); checkNodesConnected(consolidationData.getNodesConnectedOut(), expectedNodesConnectedData); @@ -181,15 +178,13 @@ public class EntityConsolidationDataTest { @Test public void testAddOutputParamGetAttrIn() { - EntityConsolidationData consolidationData = new EntityConsolidationData(); - GetAttrFuncData getAttrFuncData1 = createGetAttrFuncData("field1"); - + GetAttrFuncData getAttrFuncData1 = createGetAttrFuncData(FIELD_1); consolidationData.addOutputParamGetAttrIn(getAttrFuncData1); List outputParametersGetAttrIn = consolidationData.getOutputParametersGetAttrIn(); Assert.assertEquals(1, outputParametersGetAttrIn.size()); Assert.assertTrue(outputParametersGetAttrIn.contains(getAttrFuncData1)); - GetAttrFuncData getAttrFuncData2 = createGetAttrFuncData("field2"); + GetAttrFuncData getAttrFuncData2 = createGetAttrFuncData(FIELD_2); consolidationData.addOutputParamGetAttrIn(getAttrFuncData2); Assert.assertEquals(2,outputParametersGetAttrIn.size()); Assert.assertTrue(outputParametersGetAttrIn.contains(getAttrFuncData1)); @@ -198,15 +193,13 @@ public class EntityConsolidationDataTest { @Test public void testRemoveParamNameFromAttrFuncList() { - EntityConsolidationData consolidationData = new EntityConsolidationData(); - GetAttrFuncData getAttrFuncData1 = createGetAttrFuncData("field1"); - + GetAttrFuncData getAttrFuncData1 = createGetAttrFuncData(FIELD_1); consolidationData.addOutputParamGetAttrIn(getAttrFuncData1); // verify that getAttrFuncData was added List outputParametersGetAttrIn = consolidationData.getOutputParametersGetAttrIn(); Assert.assertEquals(1, outputParametersGetAttrIn.size()); - consolidationData.removeParamNameFromAttrFuncList("field2"); + consolidationData.removeParamNameFromAttrFuncList(FIELD_2); //verify that not existing getAttrFuncData parameter wasn't removed and no Exception outputParametersGetAttrIn = consolidationData.getOutputParametersGetAttrIn(); Assert.assertEquals(1, outputParametersGetAttrIn.size()); @@ -217,6 +210,19 @@ public class EntityConsolidationDataTest { Assert.assertEquals(0, outputParametersGetAttrIn.size()); } + @Test + public void testAddGroupId() { + consolidationData.addGroupId(GROUP_ID_1); + List groupIds = consolidationData.getGroupIds(); + Assert.assertNotNull(groupIds); + Assert.assertTrue(groupIds.contains(GROUP_ID_1)); + Assert.assertEquals(1, consolidationData.getGroupIds().size()); + + consolidationData.addGroupId(GROUP_ID_2); + Assert.assertEquals(2, consolidationData.getGroupIds().size()); + Assert.assertTrue(groupIds.contains(GROUP_ID_2)); + } + private GetAttrFuncData createGetAttrFuncData(String field) { GetAttrFuncData getAttrFuncData = new GetAttrFuncData(); getAttrFuncData.setFieldName(field); diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/FileComputeConsolidationDataTest.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/FileComputeConsolidationDataTest.java index eab3bd25b1..5d3d58e74c 100644 --- a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/FileComputeConsolidationDataTest.java +++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/FileComputeConsolidationDataTest.java @@ -17,67 +17,109 @@ package org.openecomp.sdc.translator.datatypes.heattotosca.unifiedmodel.consolidation; import java.util.Collection; +import java.util.EnumMap; import java.util.HashSet; import java.util.Set; import org.junit.Assert; +import org.junit.Before; import org.junit.Test; +import org.mockito.Mock; +import org.mockito.Mockito; +import org.mockito.MockitoAnnotations; public class FileComputeConsolidationDataTest { private static final String COMPUTE_NODE_TEMPLATE_ID_1 = "computeNodeTemplateId1"; private static final String COMPUTE_NODE_TEMPLATE_ID_2 = "computeNodeTemplateId2"; - private static final String COMPUTE_NODE_TYPE_1 = "computeNodeType1"; - private static final String COMPUTE_NODE_TYPE_2 = "computeNodeType2"; + private final FileComputeConsolidationData consolidationData = new FileComputeConsolidationData(); + + private enum ComputeNodeTypeEnum { + COMPUTE_NODE_TYPE_1, + COMPUTE_NODE_TYPE_2 + } + + private final EnumMap mockMap = + new EnumMap<>(ComputeNodeTypeEnum.class); + + @Mock + private TypeComputeConsolidationData mockTypeComputeConsolidationData1; + @Mock + private TypeComputeConsolidationData mockTypeComputeConsolidationData2; + + @Before + public void setUp() { + MockitoAnnotations.initMocks(this); + addMocksToMap(); + } @Test public void testAddComputeTemplateConsolidationData() { - FileComputeConsolidationData consolidationData = new FileComputeConsolidationData(); - consolidationData.addComputeTemplateConsolidationData(COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); - + String computeNodeType = ComputeNodeTypeEnum.COMPUTE_NODE_TYPE_1.name(); + consolidationData.addComputeTemplateConsolidationData(computeNodeType, COMPUTE_NODE_TEMPLATE_ID_1); Set expectedComputeNodeTypes = new HashSet<>(); - expectedComputeNodeTypes.add(COMPUTE_NODE_TYPE_1); - checkComputeConsolidationData(consolidationData, COMPUTE_NODE_TYPE_1, expectedComputeNodeTypes); + expectedComputeNodeTypes.add(computeNodeType); + checkComputeConsolidationData(consolidationData, computeNodeType, expectedComputeNodeTypes); } @Test public void testAddSameConsolidationDataTwice_noNewCreated() { - FileComputeConsolidationData consolidationData = new FileComputeConsolidationData(); - consolidationData.addComputeTemplateConsolidationData(COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); + String computeNodeType = ComputeNodeTypeEnum.COMPUTE_NODE_TYPE_1.name(); + consolidationData.addComputeTemplateConsolidationData(computeNodeType, COMPUTE_NODE_TEMPLATE_ID_1); TypeComputeConsolidationData firstTypeComputeConsolidationData = - consolidationData.getTypeComputeConsolidationData(COMPUTE_NODE_TYPE_1); + consolidationData.getTypeComputeConsolidationData(computeNodeType); - consolidationData.addComputeTemplateConsolidationData(COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); + consolidationData.addComputeTemplateConsolidationData(computeNodeType, COMPUTE_NODE_TEMPLATE_ID_1); TypeComputeConsolidationData secondTypeComputeConsolidationData = - consolidationData.getTypeComputeConsolidationData(COMPUTE_NODE_TYPE_1); + consolidationData.getTypeComputeConsolidationData(computeNodeType); Assert.assertEquals(firstTypeComputeConsolidationData, secondTypeComputeConsolidationData); } @Test - public void testAddDiffConsolidationData_DiffNodeType() { - FileComputeConsolidationData consolidationData = new FileComputeConsolidationData(); + public void testAddConsolidationData_diffNodeType() { + String computeNodeType1 = ComputeNodeTypeEnum.COMPUTE_NODE_TYPE_1.name(); + consolidationData.addComputeTemplateConsolidationData(computeNodeType1, COMPUTE_NODE_TEMPLATE_ID_1); + Set expectedComputeNodeTypes = new HashSet<>(); + expectedComputeNodeTypes.add(computeNodeType1); + checkComputeConsolidationData(consolidationData, computeNodeType1, expectedComputeNodeTypes); - consolidationData.addComputeTemplateConsolidationData(COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); - Set expectedComputeNodeTypes = new HashSet<>(); - expectedComputeNodeTypes.add(COMPUTE_NODE_TYPE_1); - checkComputeConsolidationData(consolidationData, COMPUTE_NODE_TYPE_1, expectedComputeNodeTypes); + String computeNodeType2 = ComputeNodeTypeEnum.COMPUTE_NODE_TYPE_2.name(); + consolidationData.addComputeTemplateConsolidationData(computeNodeType2, COMPUTE_NODE_TEMPLATE_ID_2); + expectedComputeNodeTypes.add(computeNodeType2); + checkComputeConsolidationData(consolidationData, computeNodeType2, expectedComputeNodeTypes); + } + + @Test + public void testAddConsolidationData_diffNodeTemplateId() { + String computeNodeType = ComputeNodeTypeEnum.COMPUTE_NODE_TYPE_1.name(); + consolidationData.addComputeTemplateConsolidationData(computeNodeType, COMPUTE_NODE_TEMPLATE_ID_1); + Set expectedComputeNodeTypes = new HashSet<>(); + expectedComputeNodeTypes.add(computeNodeType); + checkComputeConsolidationData(consolidationData, computeNodeType, expectedComputeNodeTypes); - consolidationData.addComputeTemplateConsolidationData(COMPUTE_NODE_TYPE_2, COMPUTE_NODE_TEMPLATE_ID_2); - expectedComputeNodeTypes.add(COMPUTE_NODE_TYPE_2); - checkComputeConsolidationData(consolidationData, COMPUTE_NODE_TYPE_2, expectedComputeNodeTypes); + consolidationData.addComputeTemplateConsolidationData(computeNodeType, COMPUTE_NODE_TEMPLATE_ID_2); + checkComputeConsolidationData(consolidationData, computeNodeType, expectedComputeNodeTypes); } @Test - public void testAddDiffConsolidationData_SameNodeType_MultiNodeTemplateId() { - FileComputeConsolidationData consolidationData = new FileComputeConsolidationData(); + public void testAddSameConsolidationDataTwice_testWithMock() throws Exception { + String computeNodeType = ComputeNodeTypeEnum.COMPUTE_NODE_TYPE_1.name(); + addAndCheckComputeTemplateConsolidationData(computeNodeType, COMPUTE_NODE_TEMPLATE_ID_1); + addAndCheckComputeTemplateConsolidationData(computeNodeType, COMPUTE_NODE_TEMPLATE_ID_1, 2); + } - consolidationData.addComputeTemplateConsolidationData(COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_1); - Set expectedComputeNodeTypes = new HashSet<>(); - expectedComputeNodeTypes.add(COMPUTE_NODE_TYPE_1); - checkComputeConsolidationData(consolidationData, COMPUTE_NODE_TYPE_1, expectedComputeNodeTypes); + @Test + public void testAddConsolidationData_diffNodeType_testWithMock() throws Exception { + String computeNodeType = ComputeNodeTypeEnum.COMPUTE_NODE_TYPE_1.name(); + addAndCheckComputeTemplateConsolidationData(computeNodeType, COMPUTE_NODE_TEMPLATE_ID_1); + addAndCheckComputeTemplateConsolidationData(computeNodeType, COMPUTE_NODE_TEMPLATE_ID_2); + } - consolidationData.addComputeTemplateConsolidationData(COMPUTE_NODE_TYPE_1, COMPUTE_NODE_TEMPLATE_ID_2); - checkComputeConsolidationData(consolidationData, COMPUTE_NODE_TYPE_1, expectedComputeNodeTypes); + @Test + public void testAddConsolidationData_diffNodeTemplateId_testWithMock() throws Exception { + String computeNodeType = ComputeNodeTypeEnum.COMPUTE_NODE_TYPE_1.name(); + addAndCheckComputeTemplateConsolidationData(computeNodeType, COMPUTE_NODE_TEMPLATE_ID_1); + addAndCheckComputeTemplateConsolidationData(computeNodeType, COMPUTE_NODE_TEMPLATE_ID_2); } private void checkComputeConsolidationData(FileComputeConsolidationData fileComputeConsolidationData, @@ -89,10 +131,52 @@ public class FileComputeConsolidationDataTest { } private void checkGetAllComputeTypes(FileComputeConsolidationData fileComputeConsolidationData, - Set expectedComputeNodeTypes) { + Set expectedComputeNodeTypes) { Collection allComputeTypes = fileComputeConsolidationData.getAllComputeTypes(); Assert.assertNotNull(allComputeTypes); Assert.assertEquals(allComputeTypes.size(), expectedComputeNodeTypes.size()); Assert.assertTrue(allComputeTypes.containsAll(expectedComputeNodeTypes)); } + + private void addAndCheckComputeTemplateConsolidationData( + String computeNodeType, String computeNodeTemplateId) throws Exception { + addAndCheckComputeTemplateConsolidationData(computeNodeType, + computeNodeTemplateId, 1); + } + + private void addAndCheckComputeTemplateConsolidationData(String computeNodeType, + String computeNodeTemplateId, int expectedTime) throws Exception { + TypeComputeConsolidationData fileComputeConsolidationDataMock = + setTypeComputeConsolidationDataMock(computeNodeType); + + consolidationData.addComputeTemplateConsolidationData( + computeNodeType, computeNodeTemplateId); + + Mockito.verify(fileComputeConsolidationDataMock, Mockito.times(expectedTime)) + .addComputeTemplateConsolidationData(computeNodeTemplateId); + } + + private void addMocksToMap() { + mockMap.put(ComputeNodeTypeEnum.COMPUTE_NODE_TYPE_1, mockTypeComputeConsolidationData1); + mockMap.put(ComputeNodeTypeEnum.COMPUTE_NODE_TYPE_2, mockTypeComputeConsolidationData2); + } + + private TypeComputeConsolidationData setTypeComputeConsolidationDataMock( + String computeType) throws Exception { + TypeComputeConsolidationData typeComputeConsolidationDataMock = + getFileComputeConsolidationDataMock(computeType); + consolidationData.setTypeComputeConsolidationData(computeType, typeComputeConsolidationDataMock); + return typeComputeConsolidationDataMock; + } + + private TypeComputeConsolidationData getFileComputeConsolidationDataMock(String computeType) throws Exception { + ComputeNodeTypeEnum enumValue = ComputeNodeTypeEnum.valueOf(computeType); + TypeComputeConsolidationData mock = mockMap.get(enumValue); + if (mock == null) { + throw new Exception("This compute Type doesn't support. " + + "Please add it to ComputeNodeTypeEnum enum"); + } + return mock; + } + } diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/FilePortConsolidationDataTest.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/FilePortConsolidationDataTest.java new file mode 100644 index 0000000000..8dd0307a43 --- /dev/null +++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/FilePortConsolidationDataTest.java @@ -0,0 +1,102 @@ +/* + * Copyright © 2016-2018 European Support Limited + * + * 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. + */ + +package org.openecomp.sdc.translator.datatypes.heattotosca.unifiedmodel.consolidation; + +import java.util.Collection; +import java.util.HashSet; +import java.util.Set; +import org.junit.Assert; +import org.junit.Test; + +import org.openecomp.sdc.heat.datatypes.model.HeatResourcesTypes; + +public class FilePortConsolidationDataTest { + + private static final String PORT_NODE_TEMPLATE_ID_1 = "portNodeTemplateId1"; + private static final String PORT_NODE_TEMPLATE_ID_2 = "portNodeTemplateId2"; + private static final String PORT_RESOURCE_ID_1 = "pcm_port_01"; + private static final String PORT_RESOURCE_ID_2 = "pcm_port_02"; + private static final String PORT_RESOURCE_TYPE = HeatResourcesTypes.NEUTRON_PORT_RESOURCE_TYPE.getHeatResource(); + + private final FilePortConsolidationData consolidationData = new FilePortConsolidationData(); + + @Test + public void testAddPortConsolidationData() { + consolidationData.addPortTemplateConsolidationData(PORT_NODE_TEMPLATE_ID_1, PORT_RESOURCE_ID_1, + PORT_RESOURCE_TYPE); + Set expectedComputeNodeTypes = new HashSet<>(); + expectedComputeNodeTypes.add(PORT_NODE_TEMPLATE_ID_1); + checkComputeConsolidationData(consolidationData, PORT_NODE_TEMPLATE_ID_1, expectedComputeNodeTypes); + } + + @Test + public void testAddSameConsolidationDataTwice_noNewCreated() { + consolidationData.addPortTemplateConsolidationData(PORT_NODE_TEMPLATE_ID_1, PORT_RESOURCE_ID_1, + PORT_RESOURCE_TYPE); + PortTemplateConsolidationData firstPortTemplateConsolidationData = + consolidationData.getPortTemplateConsolidationData(PORT_NODE_TEMPLATE_ID_1); + + consolidationData.addPortTemplateConsolidationData(PORT_NODE_TEMPLATE_ID_1, PORT_RESOURCE_ID_1, + PORT_RESOURCE_TYPE); + PortTemplateConsolidationData secondPortTemplateConsolidationData = + consolidationData.getPortTemplateConsolidationData(PORT_NODE_TEMPLATE_ID_1); + Assert.assertEquals(firstPortTemplateConsolidationData, secondPortTemplateConsolidationData); + } + + @Test + public void testAddDiffConsolidationData_diffNodeTemplateIds() { + consolidationData.addPortTemplateConsolidationData(PORT_NODE_TEMPLATE_ID_1, PORT_RESOURCE_ID_1, + PORT_RESOURCE_TYPE); + Set expectedComputeNodeTypes = new HashSet<>(); + expectedComputeNodeTypes.add(PORT_NODE_TEMPLATE_ID_1); + checkComputeConsolidationData(consolidationData, PORT_NODE_TEMPLATE_ID_1, expectedComputeNodeTypes); + + consolidationData.addPortTemplateConsolidationData(PORT_NODE_TEMPLATE_ID_2, PORT_RESOURCE_ID_2, + PORT_RESOURCE_TYPE); + expectedComputeNodeTypes.add(PORT_NODE_TEMPLATE_ID_2); + checkComputeConsolidationData(consolidationData, PORT_NODE_TEMPLATE_ID_2, expectedComputeNodeTypes); + } + + @Test + public void testAddDiffConsolidationData_diffResourceIds() { + consolidationData.addPortTemplateConsolidationData(PORT_NODE_TEMPLATE_ID_1, PORT_RESOURCE_ID_1, + PORT_RESOURCE_TYPE); + Set expectedComputeNodeTypes = new HashSet<>(); + expectedComputeNodeTypes.add(PORT_NODE_TEMPLATE_ID_1); + checkComputeConsolidationData(consolidationData, PORT_NODE_TEMPLATE_ID_1, expectedComputeNodeTypes); + + consolidationData.addPortTemplateConsolidationData(PORT_NODE_TEMPLATE_ID_1, PORT_RESOURCE_ID_2, + PORT_RESOURCE_TYPE); + checkComputeConsolidationData(consolidationData, PORT_NODE_TEMPLATE_ID_1, expectedComputeNodeTypes); + } + + private void checkComputeConsolidationData(FilePortConsolidationData filePortConsolidationData, + String computeNodeTemplateId, Set expectedPortNodeTemplateIds) { + PortTemplateConsolidationData consolidationData = + filePortConsolidationData.getPortTemplateConsolidationData(computeNodeTemplateId); + Assert.assertNotNull(consolidationData); + checkGetTemplateIds(filePortConsolidationData, expectedPortNodeTemplateIds); + } + + private void checkGetTemplateIds(FilePortConsolidationData filePortConsolidationData, + Set expectedPortNodeTemplateIds) { + Collection allPortNodeTemplateIds = filePortConsolidationData.getAllPortNodeTemplateIds(); + Assert.assertNotNull(allPortNodeTemplateIds); + Assert.assertEquals(allPortNodeTemplateIds.size(), expectedPortNodeTemplateIds.size()); + Assert.assertTrue(allPortNodeTemplateIds.containsAll(expectedPortNodeTemplateIds)); + } +} diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/PortConsolidationDataTest.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/PortConsolidationDataTest.java new file mode 100644 index 0000000000..05382e12ce --- /dev/null +++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/PortConsolidationDataTest.java @@ -0,0 +1,225 @@ +/* + * Copyright © 2016-2018 European Support Limited + * + * 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. + */ + +package org.openecomp.sdc.translator.datatypes.heattotosca.unifiedmodel.consolidation; + +import java.util.EnumMap; +import java.util.HashSet; +import java.util.Set; +import org.junit.Assert; +import org.junit.Before; +import org.junit.Test; +import org.mockito.Mock; +import org.mockito.Mockito; +import org.mockito.MockitoAnnotations; +import org.openecomp.sdc.heat.datatypes.model.Resource; + +public class PortConsolidationDataTest { + + private static final String PORT_NODE_TYPE_1 = "portNodeType1"; + private static final String PORT_NODE_TYPE_2 = "portNodeType2"; + private static final String PORT_NODE_TEMPLATE_ID_1 = "portNodeTemplateId1"; + private static final String PORT_NODE_TEMPLATE_ID_2 = "portNodeTemplateId2"; + private static final String SUB_INTERFACE_NODE_TEMPLATE_ID_1 = "subInterfaceNodeTemplateId1"; + private static final String SUB_INTERFACE_NODE_TEMPLATE_ID_2 = "subInterfaceNodeTemplateId2"; + private static final String PORT_RESOURCE_ID = "portResourceId"; + private static final String PORT_RESOURCE_TYPE = "portResourceType"; + + private final EnumMap mockMap = + new EnumMap<>(ServiceTemplateFileNameEnum.class); + + @Mock + private FilePortConsolidationData mockFilePortConsolidationData1; + @Mock + private FilePortConsolidationData mockFilePortConsolidationData2; + + private final Resource resource = new Resource(); + private final PortConsolidationData consolidationData = new PortConsolidationData(); + + @Before + public void setUp() { + MockitoAnnotations.initMocks(this); + addMocksToMap(); + } + + @Test + public void testAddConsolidationData_noNewCreated() { + String serviceTemplateName = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(); + consolidationData.addPortTemplateConsolidationData(serviceTemplateName, PORT_NODE_TYPE_1, + PORT_RESOURCE_ID, PORT_RESOURCE_TYPE); + consolidationData.getFilePortConsolidationData(serviceTemplateName); + Set expectedServiceTemplateNames = new HashSet<>(); + expectedServiceTemplateNames.add(serviceTemplateName); + checkPortConsolidationData(consolidationData, serviceTemplateName, expectedServiceTemplateNames); + + consolidationData.addPortTemplateConsolidationData(serviceTemplateName, PORT_NODE_TYPE_1, + PORT_RESOURCE_ID, PORT_RESOURCE_TYPE); + consolidationData.getFilePortConsolidationData(serviceTemplateName); + checkPortConsolidationData(consolidationData, serviceTemplateName, expectedServiceTemplateNames); + } + + @Test + public void testAddConsolidationData_DiffNodeTypes() { + String serviceTemplateName = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(); + consolidationData.addPortTemplateConsolidationData(serviceTemplateName, PORT_NODE_TYPE_1, + PORT_RESOURCE_ID, PORT_RESOURCE_TYPE); + Set expectedServiceTemplateNames = new HashSet<>(); + expectedServiceTemplateNames.add(serviceTemplateName); + checkPortConsolidationData(consolidationData, serviceTemplateName, expectedServiceTemplateNames); + consolidationData.addPortTemplateConsolidationData(serviceTemplateName, PORT_NODE_TYPE_2, + PORT_RESOURCE_ID, PORT_RESOURCE_TYPE); + checkPortConsolidationData(consolidationData, serviceTemplateName, expectedServiceTemplateNames); + } + + @Test + public void testAddConsolidationData_DiffServiceTemplate() { + String serviceTemplateName1 = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(); + consolidationData.addPortTemplateConsolidationData( + serviceTemplateName1, PORT_NODE_TYPE_1, PORT_RESOURCE_ID, PORT_RESOURCE_TYPE); + Set expectedServiceTemplateNames = new HashSet<>(); + expectedServiceTemplateNames.add(serviceTemplateName1); + checkPortConsolidationData(consolidationData, serviceTemplateName1, expectedServiceTemplateNames); + + String serviceTemplateName2 = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_2.name(); + consolidationData.addPortTemplateConsolidationData( + serviceTemplateName2, PORT_NODE_TYPE_2, PORT_RESOURCE_ID, PORT_RESOURCE_TYPE); + expectedServiceTemplateNames.add(serviceTemplateName2); + checkPortConsolidationData(consolidationData, serviceTemplateName2, expectedServiceTemplateNames); + } + + @Test + public void testAddConsolidationData_DiffNodeTypes_testWithMock() throws Exception { + String serviceTemplateName = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(); + addAndCheckPortTemplateConsolidationData(serviceTemplateName, + PORT_NODE_TYPE_1); + addAndCheckPortTemplateConsolidationData(serviceTemplateName, + PORT_NODE_TYPE_2); + } + + @Test + public void testAddConsolidationData_DiffServiceTemplate_testWithMock() throws Exception { + addAndCheckPortTemplateConsolidationData(ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(), + PORT_NODE_TYPE_1); + addAndCheckPortTemplateConsolidationData(ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_2.name(), + PORT_NODE_TYPE_2); + } + + @Test + public void testAddSubInterfaceConsolidationData_Same() throws Exception { + String serviceTemplateName = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(); + addAndCheckSubInterfaceTemplateConsolidationData(serviceTemplateName, + SUB_INTERFACE_NODE_TEMPLATE_ID_1, PORT_NODE_TEMPLATE_ID_1); + addAndCheckSubInterfaceTemplateConsolidationData(serviceTemplateName, + SUB_INTERFACE_NODE_TEMPLATE_ID_1, PORT_NODE_TEMPLATE_ID_1, 2); + } + + @Test + public void testAddSubInterfaceConsolidationData_diffNodeTempId() throws Exception { + String serviceTemplateName = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(); + addAndCheckSubInterfaceTemplateConsolidationData(serviceTemplateName, + SUB_INTERFACE_NODE_TEMPLATE_ID_1, PORT_NODE_TEMPLATE_ID_1); + addAndCheckSubInterfaceTemplateConsolidationData(serviceTemplateName, + SUB_INTERFACE_NODE_TEMPLATE_ID_2, PORT_NODE_TEMPLATE_ID_1); + } + + @Test + public void testAddSubInterfaceConsolidationData_diffNodeTempId_diffParentPort() throws Exception { + String serviceTemplateName = ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(); + addAndCheckSubInterfaceTemplateConsolidationData(serviceTemplateName, + SUB_INTERFACE_NODE_TEMPLATE_ID_1, PORT_NODE_TEMPLATE_ID_1); + addAndCheckSubInterfaceTemplateConsolidationData(serviceTemplateName, + SUB_INTERFACE_NODE_TEMPLATE_ID_2, PORT_NODE_TEMPLATE_ID_2); + } + + @Test + public void testAddSubInterfaceConsolidationData_diffServiceTemp() throws Exception { + addAndCheckSubInterfaceTemplateConsolidationData( + ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1.name(), + SUB_INTERFACE_NODE_TEMPLATE_ID_1, SUB_INTERFACE_NODE_TEMPLATE_ID_1); + addAndCheckSubInterfaceTemplateConsolidationData( + ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_2.name(), + SUB_INTERFACE_NODE_TEMPLATE_ID_2, SUB_INTERFACE_NODE_TEMPLATE_ID_2); + } + + private void addAndCheckPortTemplateConsolidationData(String serviceTemplateFileName, + String portNodeType) throws Exception { + FilePortConsolidationData filePortConsolidationDataMock = + setFilePortConsolidationDataMock(serviceTemplateFileName); + + consolidationData.addPortTemplateConsolidationData( + serviceTemplateFileName, portNodeType, PORT_RESOURCE_ID, PORT_RESOURCE_TYPE); + + Mockito.verify(filePortConsolidationDataMock) + .addPortTemplateConsolidationData(portNodeType, PORT_RESOURCE_ID, PORT_RESOURCE_TYPE); + } + + private void addAndCheckSubInterfaceTemplateConsolidationData(String serviceTemplateFileName, + String subInterfaceNodeTemplateId, String subInterfaceNodeTemplateType) throws Exception { + addAndCheckSubInterfaceTemplateConsolidationData(serviceTemplateFileName, subInterfaceNodeTemplateId, + subInterfaceNodeTemplateType, 1); + } + + private void addAndCheckSubInterfaceTemplateConsolidationData(String serviceTemplateFileName, + String subInterfaceNodeTemplateId, String parentPortNodeTemplateId, int expectedTime) throws Exception { + + FilePortConsolidationData filePortConsolidationDataMock = + setFilePortConsolidationDataMock(serviceTemplateFileName); + consolidationData.addSubInterfaceTemplateConsolidationData(serviceTemplateFileName, resource, + subInterfaceNodeTemplateId, parentPortNodeTemplateId); + + Mockito.verify(filePortConsolidationDataMock, Mockito.times(expectedTime)) + .addSubInterfaceTemplateConsolidationData(resource, subInterfaceNodeTemplateId, + parentPortNodeTemplateId); + } + + private void addMocksToMap() { + mockMap.put(ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_1, mockFilePortConsolidationData1); + mockMap.put(ServiceTemplateFileNameEnum.SERVICE_TEMPLATE_FILE_NAME_2, mockFilePortConsolidationData2); + } + + private FilePortConsolidationData setFilePortConsolidationDataMock(String serviceTemplateName) throws Exception { + FilePortConsolidationData filePortConsolidationDataMock = + getFileComputeConsolidationDataMock(serviceTemplateName); + consolidationData.setFilePortConsolidationData(serviceTemplateName, filePortConsolidationDataMock); + return filePortConsolidationDataMock; + } + + private FilePortConsolidationData getFileComputeConsolidationDataMock(String serviceTemplateName) throws Exception { + ServiceTemplateFileNameEnum enumValue = ServiceTemplateFileNameEnum.valueOf(serviceTemplateName); + FilePortConsolidationData mock = mockMap.get(enumValue); + if (mock == null) { + throw new Exception("This service Template File Name doesn't supported. " + + "Please add it to ServiceTemplateFileName enum"); + } + return mock; + } + + private void checkPortConsolidationData(PortConsolidationData consolidationData, + String serviceTemplateFileName, Set expectedServiceTemplateNames) { + FilePortConsolidationData filePortConsolidationData = consolidationData + .getFilePortConsolidationData(serviceTemplateFileName); + Assert.assertNotNull(filePortConsolidationData); + checkGetAllServiceTemplateFileNames(consolidationData, expectedServiceTemplateNames); + } + + private void checkGetAllServiceTemplateFileNames(PortConsolidationData consolidationData, + Set expectedServiceTemplateNames) { + Set allServiceTemplateFileNames = consolidationData.getAllServiceTemplateFileNames(); + Assert.assertNotNull(allServiceTemplateFileNames); + Assert.assertEquals(expectedServiceTemplateNames.size(), allServiceTemplateFileNames.size()); + Assert.assertTrue(allServiceTemplateFileNames.containsAll(expectedServiceTemplateNames)); + } +} diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ServiceTemplateFileNameEnum.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ServiceTemplateFileNameEnum.java new file mode 100644 index 0000000000..63da373c3a --- /dev/null +++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/ServiceTemplateFileNameEnum.java @@ -0,0 +1,7 @@ +package org.openecomp.sdc.translator.datatypes.heattotosca.unifiedmodel.consolidation; + +public enum ServiceTemplateFileNameEnum { + SERVICE_TEMPLATE_FILE_NAME_1, + SERVICE_TEMPLATE_FILE_NAME_2 +} + diff --git a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/TypeComputeConsolidationDataTest.java b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/TypeComputeConsolidationDataTest.java index 6388620e7c..727dd5c43c 100644 --- a/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/TypeComputeConsolidationDataTest.java +++ b/openecomp-be/lib/openecomp-sdc-translator-lib/openecomp-sdc-translator-core/src/test/java/org/openecomp/sdc/translator/datatypes/heattotosca/unifiedmodel/consolidation/TypeComputeConsolidationDataTest.java @@ -33,51 +33,47 @@ public class TypeComputeConsolidationDataTest { private static final String COMPUTE_NODE_TEMPLATE_ID_1 = "computeNodeTemplateId1"; private static final String COMPUTE_NODE_TEMPLATE_ID_2 = "computeNodeTemplateId2"; - private static final String SERVER_NETWORKROLE_1_PORT = "server_networkrole_1_port"; - private static final String SERVER_NETWORKROLE_2_PORT = "server_networkrole_2_port"; + private static final String SERVER_NETWORK_ROLE_1_PORT = "server_network_role_1_port"; + private static final String SERVER_NETWORK_ROLE_2_PORT = "server_network_role_2_port"; + + private final TypeComputeConsolidationData consolidationData = new TypeComputeConsolidationData(); @Test public void testCollectAllPortsOfEachTypeFromComputesNoPorts() { - TypeComputeConsolidationData typeComputeConsolidationData = new TypeComputeConsolidationData(); - ComputeTemplateConsolidationData computeTemplateConsolidationData = new ComputeTemplateConsolidationData(); - - typeComputeConsolidationData.setComputeTemplateConsolidationData("server_type", + consolidationData.setComputeTemplateConsolidationData("server_type", computeTemplateConsolidationData); - Map> stringListMap = typeComputeConsolidationData.collectAllPortsOfEachTypeFromComputes(); + Map> stringListMap = consolidationData.collectAllPortsOfEachTypeFromComputes(); Assert.assertTrue(Objects.nonNull(stringListMap) && MapUtils.isEmpty(stringListMap)); - } @Test public void testCollectAllPortsOfEachTypeFromComputes() { - TypeComputeConsolidationData typeComputeConsolidationData = new TypeComputeConsolidationData(); Map> ports = new HashMap<>(); - ports.put(SERVER_NETWORKROLE_1_PORT, - Arrays.asList("server_0_networkrole_1_port", "server_1_networkrole_1_port")); + ports.put(SERVER_NETWORK_ROLE_1_PORT, + Arrays.asList("server_0_network_role_1_port", "server_1_network_role_1_port")); ComputeTemplateConsolidationData computeTemplateConsolidationData = new ComputeTemplateConsolidationData(); computeTemplateConsolidationData.setPorts(ports); - typeComputeConsolidationData.setComputeTemplateConsolidationData("server_type", + consolidationData.setComputeTemplateConsolidationData("server_type", computeTemplateConsolidationData); - Map> stringListMap = typeComputeConsolidationData.collectAllPortsOfEachTypeFromComputes(); - Assert.assertTrue(stringListMap.containsKey(SERVER_NETWORKROLE_1_PORT) - && stringListMap.get(SERVER_NETWORKROLE_1_PORT).size() == 2); - + Map> stringListMap = consolidationData.collectAllPortsOfEachTypeFromComputes(); + Assert.assertTrue(stringListMap.containsKey(SERVER_NETWORK_ROLE_1_PORT) + && stringListMap.get(SERVER_NETWORK_ROLE_1_PORT).size() == 2); } @Test public void testCollectAllPortsOfEachTypeFromComputesWithMultipleCompute() { Map> ports = new HashMap<>(); - ports.put(SERVER_NETWORKROLE_1_PORT, - Arrays.asList("server_0_networkrole_1_port_1", "server_1_networkrole_1_port_2")); + ports.put(SERVER_NETWORK_ROLE_1_PORT, + Arrays.asList("server_0_network_role_1_port_1", "server_1_network_role_1_port_2")); Map> ports1 = new HashMap<>(); - ports1.put(SERVER_NETWORKROLE_2_PORT, - Arrays.asList("server_0_networkrole_2_port_1", "server_1_networkrole_2_port_2")); + ports1.put(SERVER_NETWORK_ROLE_2_PORT, + Arrays.asList("server_0_network_role_2_port_1", "server_1_network_role_2_port_2")); ComputeTemplateConsolidationData computeTemplateConsolidationData = new ComputeTemplateConsolidationData(); computeTemplateConsolidationData.setPorts(ports); @@ -85,27 +81,25 @@ public class TypeComputeConsolidationDataTest { ComputeTemplateConsolidationData computeTemplateConsolidationData1 = new ComputeTemplateConsolidationData(); computeTemplateConsolidationData1.setPorts(ports1); - TypeComputeConsolidationData typeComputeConsolidationData = new TypeComputeConsolidationData(); - typeComputeConsolidationData.setComputeTemplateConsolidationData("server_type", + consolidationData.setComputeTemplateConsolidationData("server_type", computeTemplateConsolidationData); - typeComputeConsolidationData.setComputeTemplateConsolidationData("server_type1", + consolidationData.setComputeTemplateConsolidationData("server_type1", computeTemplateConsolidationData1); - Map> stringListMap = typeComputeConsolidationData.collectAllPortsOfEachTypeFromComputes(); + Map> stringListMap = consolidationData.collectAllPortsOfEachTypeFromComputes(); Assert.assertTrue(stringListMap.size() == 2); - Assert.assertTrue(stringListMap.get(SERVER_NETWORKROLE_1_PORT).size() == 2); - Assert.assertTrue(stringListMap.get(SERVER_NETWORKROLE_1_PORT).contains("server_0_networkrole_1_port_1") - && stringListMap.get(SERVER_NETWORKROLE_1_PORT).contains("server_1_networkrole_1_port_2")); + Assert.assertTrue(stringListMap.get(SERVER_NETWORK_ROLE_1_PORT).size() == 2); + Assert.assertTrue(stringListMap.get(SERVER_NETWORK_ROLE_1_PORT).contains("server_0_network_role_1_port_1") + && stringListMap.get(SERVER_NETWORK_ROLE_1_PORT).contains("server_1_network_role_1_port_2")); - Assert.assertTrue(stringListMap.get(SERVER_NETWORKROLE_2_PORT).size() == 2); - Assert.assertTrue(stringListMap.get(SERVER_NETWORKROLE_2_PORT).contains("server_0_networkrole_2_port_1") - && stringListMap.get(SERVER_NETWORKROLE_2_PORT).contains("server_1_networkrole_2_port_2")); + Assert.assertTrue(stringListMap.get(SERVER_NETWORK_ROLE_2_PORT).size() == 2); + Assert.assertTrue(stringListMap.get(SERVER_NETWORK_ROLE_2_PORT).contains("server_0_network_role_2_port_1") + && stringListMap.get(SERVER_NETWORK_ROLE_2_PORT).contains("server_1_network_role_2_port_2")); } @Test public void testAddComputeTemplateConsolidationData() { - TypeComputeConsolidationData consolidationData = new TypeComputeConsolidationData(); consolidationData.addComputeTemplateConsolidationData(COMPUTE_NODE_TEMPLATE_ID_1); Set expectedNodeTemplateIds = new HashSet<>(); @@ -115,7 +109,6 @@ public class TypeComputeConsolidationDataTest { @Test public void testAddSameConsolidationDataTwice_noNewCreated() { - TypeComputeConsolidationData consolidationData = new TypeComputeConsolidationData(); consolidationData.addComputeTemplateConsolidationData(COMPUTE_NODE_TEMPLATE_ID_1); ComputeTemplateConsolidationData firstComputeTemplateConsolidationData = consolidationData.getComputeTemplateConsolidationData(COMPUTE_NODE_TEMPLATE_ID_1); @@ -128,8 +121,6 @@ public class TypeComputeConsolidationDataTest { @Test public void testAddDiffConsolidationData_DiffNodeTemplateId() { - TypeComputeConsolidationData consolidationData = new TypeComputeConsolidationData(); - final ComputeTemplateConsolidationData firstComputeTemplateConsolidationData = consolidationData .addComputeTemplateConsolidationData(COMPUTE_NODE_TEMPLATE_ID_1); Set expectedNodeTemplateIds1 = new HashSet<>(); -- cgit 1.2.3-korg