vnf spin up gr api vnf s base module fails 69/79469/1
authorBenjamin, Max (mb388a) <mb388a@us.att.com>
Fri, 1 Mar 2019 17:48:00 +0000 (12:48 -0500)
committerBenjamin, Max (mb388a) <mb388a@us.att.com>
Fri, 1 Mar 2019 17:48:14 +0000 (12:48 -0500)
map object representations of json back to json strings
marshal all objects to json strings before sending
all input params converted to Map<String, Object>
Updated userParams map from HashMap<String, String> to Map<String,
Object> as HashMap<String, String> userParams was causing VNF spin up
to fail. Input json in Userparams was not accepted as a valid json
format. Updated other files that were affected by this change.

Change-Id: I0c00fc00c4c11b54ace4df7be8d5bfc80d41d130
Issue-ID: SO-1582
Signed-off-by: Benjamin, Max (mb388a) <mb388a@us.att.com>
43 files changed:
adapters/mso-adapter-utils/src/main/java/org/onap/so/cloudify/utils/MsoCloudifyUtils.java
adapters/mso-adapter-utils/src/main/java/org/onap/so/openstack/utils/MsoHeatUtils.java
adapters/mso-adapter-utils/src/test/java/org/onap/so/cloudify/utils/MsoCloudifyUtilsTest.java
adapters/mso-adapter-utils/src/test/java/org/onap/so/openstack/utils/MsoHeatUtilsUnitTest.java [new file with mode: 0644]
adapters/mso-adapter-utils/src/test/resources/__files/MsoHeatUtils/free-form.json [new file with mode: 0644]
adapters/mso-adapters-rest-interface/src/main/java/org/onap/so/adapters/vnfrest/CreateVfModuleRequest.java
adapters/mso-adapters-rest-interface/src/main/java/org/onap/so/adapters/vnfrest/CreateVolumeGroupRequest.java
adapters/mso-adapters-rest-interface/src/main/java/org/onap/so/adapters/vnfrest/UpdateVfModuleRequest.java
adapters/mso-adapters-rest-interface/src/main/java/org/onap/so/adapters/vnfrest/UpdateVolumeGroupRequest.java
adapters/mso-openstack-adapters/src/main/java/org/onap/so/adapters/vnf/MsoVnfAdapter.java
adapters/mso-openstack-adapters/src/main/java/org/onap/so/adapters/vnf/MsoVnfAdapterAsync.java
adapters/mso-openstack-adapters/src/main/java/org/onap/so/adapters/vnf/MsoVnfAdapterAsyncImpl.java
adapters/mso-openstack-adapters/src/main/java/org/onap/so/adapters/vnf/MsoVnfAdapterImpl.java
adapters/mso-openstack-adapters/src/main/java/org/onap/so/adapters/vnf/MsoVnfCloudifyAdapterImpl.java
adapters/mso-openstack-adapters/src/main/java/org/onap/so/adapters/vnf/MsoVnfPluginAdapterImpl.java
adapters/mso-openstack-adapters/src/test/java/org/onap/so/adapters/vnf/MsoVnfAdapterAsyncImplTest.java
adapters/mso-openstack-adapters/src/test/java/org/onap/so/adapters/vnf/MsoVnfAdapterImplTest.java
adapters/mso-openstack-adapters/src/test/java/org/onap/so/adapters/vnf/MsoVnfCloudifyAdapterImplTest.java
adapters/mso-openstack-adapters/src/test/java/org/onap/so/adapters/vnf/MsoVnfMulticloudAdapterImplTest.java
adapters/mso-openstack-adapters/src/test/java/org/onap/so/adapters/vnf/MsoVnfPluginAdapterImplTest.java
adapters/mso-openstack-adapters/src/test/java/org/onap/so/adapters/vnf/VnfAdapterRestTest.java
adapters/mso-openstack-adapters/src/test/java/org/onap/so/adapters/vnf/VolumeGroupAdapterCommon.java
bpmn/MSOCommonBPMN/src/main/java/org/onap/so/bpmn/servicedecomposition/generalobjects/RequestContext.java
bpmn/MSOCommonBPMN/src/main/java/org/onap/so/bpmn/servicedecomposition/tasks/BBInputSetupMapperLayer.java
bpmn/MSOCommonBPMN/src/test/java/org/onap/so/bpmn/servicedecomposition/tasks/BBInputSetupMapperLayerTest.java
bpmn/so-bpmn-tasks/src/main/java/org/onap/so/client/adapter/vnf/mapper/VnfAdapterObjectMapper.java
bpmn/so-bpmn-tasks/src/main/java/org/onap/so/client/adapter/vnf/mapper/VnfAdapterVfModuleObjectMapper.java
bpmn/so-bpmn-tasks/src/main/java/org/onap/so/client/sdnc/mapper/GCTopologyOperationRequestMapper.java
bpmn/so-bpmn-tasks/src/main/java/org/onap/so/client/sdnc/mapper/GeneralTopologyObjectMapper.java
bpmn/so-bpmn-tasks/src/main/java/org/onap/so/client/sdnc/mapper/NetworkTopologyOperationRequestMapper.java
bpmn/so-bpmn-tasks/src/main/java/org/onap/so/client/sdnc/mapper/ServiceTopologyOperationMapper.java
bpmn/so-bpmn-tasks/src/main/java/org/onap/so/client/sdnc/mapper/VfModuleTopologyOperationRequestMapper.java
bpmn/so-bpmn-tasks/src/main/java/org/onap/so/client/sdnc/mapper/VnfTopologyOperationRequestMapper.java
bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/adapter/vnf/mapper/VnfAdapterObjectMapperTest.java
bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/adapter/vnf/mapper/VnfAdapterVfModuleObjectMapperPayloadTest.java
bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/adapter/vnf/mapper/VnfAdapterVfModuleObjectMapperTest.java
bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/orchestration/SDNCConfigurationResourcesTest.java
bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/sdnc/mapper/GCTopologyOperationRequestMapperTest.java
bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/sdnc/mapper/GeneralTopologyObjectMapperTest.java
bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/sdnc/mapper/NetworkTopologyOperationRequestMapperTest.java
bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/sdnc/mapper/ServiceTopologyOperationMapperTest.java
bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/sdnc/mapper/VfModuleTopologyOperationRequestMapperTest.java
bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/sdnc/mapper/VnfTopologyOperationRequestMapperTest.java

index 9212fbc..f50a6fd 100644 (file)
@@ -1032,14 +1032,14 @@ public class MsoCloudifyUtils extends MsoCommonUtils implements VduPlugin{
         * Normalize an input value to an Object, based on the target parameter type.
         * If the type is not recognized, it will just be returned unchanged (as a string).
         */
-       public Object convertInputValue (String inputValue, HeatTemplateParam templateParam)
+       public Object convertInputValue (Object inputValue, HeatTemplateParam templateParam)
        {
                String type = templateParam.getParamType();
       logger.debug("Parameter: {} is of type {}", templateParam.getParamName(), type);
 
       if (type.equalsIgnoreCase("number")) {
                        try {
-                               return Integer.valueOf(inputValue);
+                               return Integer.valueOf(inputValue.toString());
                        }
                        catch (Exception e) {
           logger.debug("Unable to convert {} to an integer!", inputValue);
@@ -1047,14 +1047,14 @@ public class MsoCloudifyUtils extends MsoCommonUtils implements VduPlugin{
                        }
                } else if (type.equalsIgnoreCase("json")) {
                        try {
-                               return new ObjectMapper().readTree(inputValue);
+                               return JSON_MAPPER.writeValueAsString(inputValue);
                        }
                        catch (Exception e) {
           logger.debug("Unable to convert {} to a JsonNode!", inputValue);
           return null;
                        }
                } else if (type.equalsIgnoreCase("boolean")) {
-                       return new Boolean(inputValue);
+                       return new Boolean(inputValue.toString());
                }
 
                // Nothing else matched.  Return the original string
index efc232e..ee1c79c 100644 (file)
@@ -86,6 +86,24 @@ import org.springframework.context.annotation.Primary;
 import org.springframework.core.env.Environment;
 import org.springframework.stereotype.Component;
 
+import com.fasterxml.jackson.core.JsonProcessingException;
+import com.fasterxml.jackson.core.type.TypeReference;
+import com.fasterxml.jackson.databind.JsonNode;
+import com.fasterxml.jackson.databind.ObjectMapper;
+import com.woorea.openstack.base.client.OpenStackConnectException;
+import com.woorea.openstack.base.client.OpenStackRequest;
+import com.woorea.openstack.base.client.OpenStackResponseException;
+import com.woorea.openstack.heat.Heat;
+import com.woorea.openstack.heat.model.CreateStackParam;
+import com.woorea.openstack.heat.model.Resources;
+import com.woorea.openstack.heat.model.Stack;
+import com.woorea.openstack.heat.model.Stack.Output;
+import com.woorea.openstack.heat.model.Stacks;
+import com.woorea.openstack.keystone.Keystone;
+import com.woorea.openstack.keystone.model.Access;
+import com.woorea.openstack.keystone.model.Authentication;
+import com.woorea.openstack.keystone.utils.KeystoneUtils;
+
 @Primary
 @Component
 public class MsoHeatUtils extends MsoCommonUtils implements VduPlugin{
@@ -1005,7 +1023,7 @@ public class MsoHeatUtils extends MsoCommonUtils implements VduPlugin{
                return heatStack.getOutputs();
        }
 
-       public void copyStringOutputsToInputs(Map<String, String> inputs,
+       public void copyStringOutputsToInputs(Map<String, Object> inputs,
                        Map<String, Object> otherStackOutputs, boolean overWrite) {
                if (inputs == null || otherStackOutputs == null)
                        return;
@@ -1245,7 +1263,7 @@ public class MsoHeatUtils extends MsoCommonUtils implements VduPlugin{
      * (heat variable type) -> java Object type
      * string -> String
      * number -> Integer
-     * json -> JsonNode XXX Removed with MSO-1475 / 1802
+     * json -> marshal object to json
      * comma_delimited_list -> ArrayList
      * boolean -> Boolean
      * if any of the conversions should fail, we will default to adding it to the inputs
@@ -1256,7 +1274,7 @@ public class MsoHeatUtils extends MsoCommonUtils implements VduPlugin{
      * @param template the HeatTemplate object - this is so we can also verify if the param is valid for this template
      * @return HashMap<String, Object> of the inputs, cleaned and converted
      */
-       public Map<String, Object> convertInputMap(Map<String, String> inputs, HeatTemplate template) {
+       public Map<String, Object> convertInputMap(Map<String, Object> inputs, HeatTemplate template) {
                HashMap<String, Object> newInputs = new HashMap<>();
                HashMap<String, HeatTemplateParam> params = new HashMap<>();
                HashMap<String, HeatTemplateParam> paramAliases = new HashMap<>();
@@ -1309,13 +1327,13 @@ public class MsoHeatUtils extends MsoCommonUtils implements VduPlugin{
         logger.debug("Parameter: {} is of type {}", key, type);
         if ("string".equalsIgnoreCase(type)) {
                                // Easiest!
-                               String str = inputs.get(key);
+                               String str = inputs.get(key).toString();
                                if (alias)
                                        newInputs.put(realName, str);
                                else
                                        newInputs.put(key, str);
                        } else if ("number".equalsIgnoreCase(type)) {
-                               String integerString = inputs.get(key);
+                               String integerString = inputs.get(key).toString();
                                Integer anInteger = null;
                                try {
                                        anInteger = Integer.parseInt(integerString);
@@ -1336,16 +1354,21 @@ public class MsoHeatUtils extends MsoCommonUtils implements VduPlugin{
                                                newInputs.put(key, integerString);
                                }
                        } else if ("json".equalsIgnoreCase(type)) {
-                               // MSO-1475 - Leave this as a string now
-                               String jsonString = inputs.get(key);
-            logger.debug("Skipping conversion to jsonNode...");
-            if (alias)
+                               Object jsonObj = inputs.get(key);
+                               String jsonString;
+                               try {
+                                       jsonString = JSON_MAPPER.writeValueAsString(jsonObj);
+                               } catch (JsonProcessingException e) {
+                                       logger.error("failed to map to json, directly converting to string instead", e);
+                                       jsonString = jsonObj.toString();
+                               }
+                       if (alias)
                                newInputs.put(realName, jsonString);
                        else
                                newInputs.put(key, jsonString);
                        //}
                        } else if ("comma_delimited_list".equalsIgnoreCase(type)) {
-                               String commaSeparated = inputs.get(key);
+                               String commaSeparated = inputs.get(key).toString();
                                try {
                                        List<String> anArrayList = this.convertCdlToArrayList(commaSeparated);
                                        if (alias)
@@ -1360,7 +1383,7 @@ public class MsoHeatUtils extends MsoCommonUtils implements VduPlugin{
                                                newInputs.put(key, commaSeparated);
                                }
                        } else if ("boolean".equalsIgnoreCase(type)) {
-                               String booleanString = inputs.get(key);
+                               String booleanString = inputs.get(key).toString();
                                Boolean aBool = Boolean.valueOf(booleanString);
                                if (alias)
                                        newInputs.put(realName, aBool);
@@ -1368,7 +1391,7 @@ public class MsoHeatUtils extends MsoCommonUtils implements VduPlugin{
                                        newInputs.put(key, aBool);
                        } else {
                                // it's null or something undefined - just add it back as a String
-                               String str = inputs.get(key);
+                               String str = inputs.get(key).toString();
                                if (alias)
                                        newInputs.put(realName, str);
                                else
index 7c5111b..f63aab7 100644 (file)
@@ -23,7 +23,7 @@ package org.onap.so.cloudify.utils;
 
 import static com.shazam.shazamcrest.MatcherAssert.assertThat;
 import static com.shazam.shazamcrest.matcher.Matchers.sameBeanAs;
-import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.fail;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.ArgumentMatchers.eq;
@@ -33,7 +33,9 @@ import static org.mockito.Mockito.spy;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 
-import com.fasterxml.jackson.databind.JsonNode;
+import java.io.IOException;
+import java.nio.file.Files;
+import java.nio.file.Paths;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
@@ -52,17 +54,23 @@ import org.onap.so.adapters.vdu.VduModelInfo;
 import org.onap.so.adapters.vdu.VduStateType;
 import org.onap.so.adapters.vdu.VduStatus;
 import org.onap.so.cloud.CloudConfig;
-import org.onap.so.cloudify.beans.DeploymentInfoBuilder;
-import org.onap.so.db.catalog.beans.CloudIdentity;
-import org.onap.so.db.catalog.beans.CloudSite;
 import org.onap.so.cloudify.beans.DeploymentInfo;
+import org.onap.so.cloudify.beans.DeploymentInfoBuilder;
 import org.onap.so.cloudify.beans.DeploymentStatus;
 import org.onap.so.cloudify.v3.client.Cloudify;
 import org.onap.so.cloudify.v3.model.AzureConfig;
+import org.onap.so.db.catalog.beans.CloudIdentity;
+import org.onap.so.db.catalog.beans.CloudSite;
 import org.onap.so.db.catalog.beans.CloudifyManager;
 import org.onap.so.db.catalog.beans.HeatTemplateParam;
 import org.onap.so.openstack.exceptions.MsoAdapterException;
 import org.onap.so.openstack.exceptions.MsoException;
+import org.skyscreamer.jsonassert.JSONAssert;
+
+import com.fasterxml.jackson.core.JsonParseException;
+import com.fasterxml.jackson.core.type.TypeReference;
+import com.fasterxml.jackson.databind.JsonMappingException;
+import com.fasterxml.jackson.databind.ObjectMapper;
 
 public class MsoCloudifyUtilsTest {
 
@@ -282,21 +290,31 @@ public class MsoCloudifyUtilsTest {
        }
 
        @Test
-       public void convertInputValue_successful() {
-               MsoCloudifyUtils testedObject = new MsoCloudifyUtils();
-
-               HeatTemplateParam heatTemplateParam = new HeatTemplateParam();
-               heatTemplateParam.setParamType("number");
-               Object result = testedObject.convertInputValue("5", heatTemplateParam);
-               assertTrue(result instanceof Integer);
-
-               heatTemplateParam.setParamType("json");
-               Object result2 = testedObject.convertInputValue("{\"key\": \"value\"}", heatTemplateParam);
-               assertTrue(result2 instanceof JsonNode);
+       public void convertInputValueTest() throws JsonParseException, JsonMappingException, IOException {
+               MsoCloudifyUtils utils = new MsoCloudifyUtils();
+               ObjectMapper mapper = new ObjectMapper();
+               HeatTemplateParam paramNum = new HeatTemplateParam();
+               paramNum.setParamType("number");
+               paramNum.setParamName("my-number");
+               
+               HeatTemplateParam paramString = new HeatTemplateParam();
+               paramString.setParamType("string");
+               paramString.setParamName("my-string");
+               
+               HeatTemplateParam paramJson = new HeatTemplateParam();
+               paramJson.setParamType("json");
+               paramJson.setParamName("my-json");
+               
+               Map<String, Object> jsonMap = mapper.readValue(getJson("free-form.json"), new TypeReference<Map<String, Object>>(){});
+               
+               assertEquals(3, utils.convertInputValue("3", paramNum));
+               assertEquals("hello", utils.convertInputValue("hello", paramString));
+               JSONAssert.assertEquals(getJson("free-form.json"), utils.convertInputValue(jsonMap, paramJson).toString(), false);
+               
+       }
 
-               heatTemplateParam.setParamType("boolean");
-               Object result3 = testedObject.convertInputValue("true", heatTemplateParam);
-               assertTrue(result3 instanceof Boolean);
+       private String getJson(String filename) throws IOException {
+                return new String(Files.readAllBytes(Paths.get("src/test/resources/__files/MsoHeatUtils/" + filename)));
        }
 
        private void mockCloudConfig(MsoCloudifyUtils testedObjectSpy) {
diff --git a/adapters/mso-adapter-utils/src/test/java/org/onap/so/openstack/utils/MsoHeatUtilsUnitTest.java b/adapters/mso-adapter-utils/src/test/java/org/onap/so/openstack/utils/MsoHeatUtilsUnitTest.java
new file mode 100644 (file)
index 0000000..40bbe90
--- /dev/null
@@ -0,0 +1,67 @@
+package org.onap.so.openstack.utils;
+
+import static org.junit.Assert.assertEquals;
+
+import java.io.IOException;
+import java.nio.file.Files;
+import java.nio.file.Paths;
+import java.util.HashMap;
+import java.util.Map;
+import java.util.Set;
+
+import org.junit.Test;
+import org.onap.so.db.catalog.beans.HeatTemplate;
+import org.onap.so.db.catalog.beans.HeatTemplateParam;
+import org.skyscreamer.jsonassert.JSONAssert;
+
+import com.fasterxml.jackson.core.JsonParseException;
+import com.fasterxml.jackson.core.type.TypeReference;
+import com.fasterxml.jackson.databind.JsonMappingException;
+import com.fasterxml.jackson.databind.ObjectMapper;
+
+public class MsoHeatUtilsUnitTest {
+
+       
+       private ObjectMapper mapper = new ObjectMapper();
+       @Test
+       public void convertInputMapTest() throws JsonParseException, JsonMappingException, IOException {
+               MsoHeatUtils utils = new MsoHeatUtils();
+               
+               Map<String, Object> input = new HashMap<>();
+               HeatTemplate template = new HeatTemplate();
+               template.setArtifactUuid("my-uuid");
+               Set<HeatTemplateParam> parameters = template.getParameters();
+               HeatTemplateParam paramNum = new HeatTemplateParam();
+               paramNum.setParamType("number");
+               paramNum.setParamName("my-number");
+               input.put("my-number", "3");
+               
+               HeatTemplateParam paramString = new HeatTemplateParam();
+               paramString.setParamType("string");
+               paramString.setParamName("my-string");
+               input.put("my-string", "hello");
+               
+               HeatTemplateParam paramJson = new HeatTemplateParam();
+               paramJson.setParamType("json");
+               paramJson.setParamName("my-json");
+               
+               Map<String, Object> jsonMap = mapper.readValue(getJson("free-form.json"), new TypeReference<Map<String, Object>>(){});
+               input.put("my-json", jsonMap);
+               
+               parameters.add(paramNum);
+               parameters.add(paramString);
+               parameters.add(paramJson);
+
+               Map<String, Object> output = utils.convertInputMap(input, template);
+               
+               assertEquals(3, output.get("my-number"));
+               assertEquals("hello", output.get("my-string"));
+               JSONAssert.assertEquals(getJson("free-form.json"), (String)output.get("my-json"), false);
+       }
+       
+       
+       private String getJson(String filename) throws IOException {
+                return new String(Files.readAllBytes(Paths.get("src/test/resources/__files/MsoHeatUtils/" + filename)));
+       }
+       
+}
diff --git a/adapters/mso-adapter-utils/src/test/resources/__files/MsoHeatUtils/free-form.json b/adapters/mso-adapter-utils/src/test/resources/__files/MsoHeatUtils/free-form.json
new file mode 100644 (file)
index 0000000..3efe436
--- /dev/null
@@ -0,0 +1,7 @@
+{
+       "a" : "value",
+       "b" : {
+               "my-key" : "my-value",
+               "array" : [1, 3, 4]
+       }
+}
\ No newline at end of file
index a383aac..c62dc0d 100644 (file)
@@ -58,7 +58,7 @@ public class CreateVfModuleRequest extends VfRequestCommon {
        private Boolean backout = true;
        private Boolean enableBridge;
 
-       private Map<String, String> vfModuleParams = new HashMap<>();
+       private Map<String, Object> vfModuleParams = new HashMap<>();
        private MsoRequest msoRequest = new MsoRequest();
 
        public String getCloudSiteId() {
@@ -189,11 +189,11 @@ public class CreateVfModuleRequest extends VfRequestCommon {
                this.backout = backout;
        }
 
-       public Map<String, String> getVfModuleParams() {
+       public Map<String, Object> getVfModuleParams() {
                return vfModuleParams;
        }
 
-       public void setVfModuleParams(Map<String, String> vfModuleParams) {
+       public void setVfModuleParams(Map<String, Object> vfModuleParams) {
                this.vfModuleParams = vfModuleParams;
        }
 
index 698b2cc..214abd4 100644 (file)
@@ -41,7 +41,7 @@ public class CreateVolumeGroupRequest extends VfRequestCommon {
        private String vnfVersion;
        private String vfModuleType;
        private String modelCustomizationUuid;
-       private Map<String,String> volumeGroupParams = new HashMap<>();
+       private Map<String,Object> volumeGroupParams = new HashMap<>();
        private Boolean failIfExists;
        private Boolean enableBridge;
        private Boolean suppressBackout;
@@ -91,11 +91,11 @@ public class CreateVolumeGroupRequest extends VfRequestCommon {
                this.vfModuleType = vfModuleType;
        }
 
-       public Map<String, String> getVolumeGroupParams() {
+       public Map<String, Object> getVolumeGroupParams() {
                return volumeGroupParams;
        }
 
-       public void setVolumeGroupParams(Map<String, String> volumeGroupParams) {
+       public void setVolumeGroupParams(Map<String, Object> volumeGroupParams) {
                this.volumeGroupParams = volumeGroupParams;
        }
 
index 0fb5c05..1c7696a 100644 (file)
@@ -58,7 +58,7 @@ public class UpdateVfModuleRequest extends VfRequestCommon {
        private Boolean failIfExists;
        private Boolean backout;
 
-       private Map<String,String> vfModuleParams = new HashMap<>();
+       private Map<String,Object> vfModuleParams = new HashMap<>();
        private MsoRequest msoRequest = new MsoRequest();
 
        public String getCloudSiteId() {
@@ -197,11 +197,11 @@ public class UpdateVfModuleRequest extends VfRequestCommon {
                this.backout = backout;
        }
 
-       public Map<String, String> getVfModuleParams() {
+       public Map<String, Object> getVfModuleParams() {
                return vfModuleParams;
        }
 
-       public void setVfModuleParams(Map<String, String> vfModuleParams) {
+       public void setVfModuleParams(Map<String, Object> vfModuleParams) {
                this.vfModuleParams = vfModuleParams;
        }
 
index d9f83aa..8ddef1e 100644 (file)
@@ -41,7 +41,7 @@ public class UpdateVolumeGroupRequest extends VfRequestCommon {
        private String vnfVersion;
        private String vfModuleType;
        private String modelCustomizationUuid;
-       private Map<String,String> volumeGroupParams = new HashMap<>();
+       private Map<String,Object> volumeGroupParams = new HashMap<>();
        private MsoRequest msoRequest = new MsoRequest();
 
        public UpdateVolumeGroupRequest() {
@@ -112,11 +112,11 @@ public class UpdateVolumeGroupRequest extends VfRequestCommon {
                this.modelCustomizationUuid = modelCustomizationUuid;
        }
 
-       public Map<String, String> getVolumeGroupParams() {
+       public Map<String, Object> getVolumeGroupParams() {
                return volumeGroupParams;
        }
 
-       public void setVolumeGroupParams(Map<String, String> volumeGroupParams) {
+       public void setVolumeGroupParams(Map<String, Object> volumeGroupParams) {
                this.volumeGroupParams = volumeGroupParams;
        }
 
index 7c49c9f..c54b067 100644 (file)
@@ -50,7 +50,7 @@ public interface MsoVnfAdapter
                             @WebParam(name="vnfName") @XmlElement(required=true) String vnfName,
                             @WebParam(name="requestType") @XmlElement(required=false) String requestType,
                                                        @WebParam(name="volumeGroupHeatStackId") @XmlElement(required=false) String volumeGroupHeatStackId,
-                                                       @WebParam(name="inputs") Map<String,String> inputs,
+                                                       @WebParam(name="inputs") Map<String,Object> inputs,
                                                        @WebParam(name="failIfExists") Boolean failIfExists,
                                                        @WebParam(name="backout") Boolean backout,
                                                        @WebParam(name="enableBridge") Boolean enableBridge,
@@ -68,7 +68,7 @@ public interface MsoVnfAdapter
                                                        @WebParam(name="vnfName") @XmlElement(required=true) String vnfName,
                                                        @WebParam(name="requestType") @XmlElement(required=false) String requestType,
                                                        @WebParam(name="volumeGroupHeatStackId") @XmlElement(required=false) String volumeGroupHeatStackId,
-                                                       @WebParam(name="inputs") Map<String,String> inputs,
+                                                       @WebParam(name="inputs") Map<String,Object> inputs,
                                                        @WebParam(name="request") MsoRequest msoRequest,
                                                        @WebParam(name="outputs", mode=Mode.OUT) Holder<Map<String,String>> outputs,
                                                        @WebParam(name="rollback", mode=Mode.OUT) Holder<VnfRollback> rollback )
@@ -109,7 +109,7 @@ public interface MsoVnfAdapter
                                                        @WebParam(name="volumeGroupHeatStackId") @XmlElement(required=false) String volumeGroupHeatStackId,
                                                        @WebParam(name="baseVfHeatStackId") @XmlElement(required=false) String baseVfHeatStackId,
                                                        @WebParam(name = "modelCustomizationUuid") @XmlElement(required = false) String modelCustomizationUuid,
-                                                       @WebParam(name="inputs") Map<String,String> inputs,
+                                                       @WebParam(name="inputs") Map<String,Object> inputs,
                                                        @WebParam(name="failIfExists") Boolean failIfExists,
                                                        @WebParam(name="backout") Boolean backout,
                                                        @WebParam(name="enableBridge") Boolean enableBridge,
@@ -138,7 +138,7 @@ public interface MsoVnfAdapter
                                                        @WebParam(name="baseVfHeatStackId") @XmlElement(required=false) String baseVfHeatStackId,
                                                        @WebParam(name="vfModuleStackId") @XmlElement(required=false) String vfModuleStackId,
                                                        @WebParam(name = "modelCustomizationUuid") @XmlElement(required = false) String modelCustomizationUuid,
-                                                       @WebParam(name="inputs") Map<String,String> inputs,
+                                                       @WebParam(name="inputs") Map<String,Object> inputs,
                                                        @WebParam(name="request") MsoRequest msoRequest,
                                                        @WebParam(name="outputs", mode=Mode.OUT) Holder<Map<String,String>> outputs,
                                                        @WebParam(name="rollback", mode=Mode.OUT) Holder<VnfRollback> rollback )
index b67a412..6f61f02 100644 (file)
@@ -53,7 +53,7 @@ public interface MsoVnfAdapterAsync
                                                        @WebParam(name="vnfName") @XmlElement(required=true) String vnfName,
                                                        @WebParam(name="requestType") @XmlElement(required=false) String requestType,
                                                        @WebParam(name="volumeGroupHeatStackId") @XmlElement(required=false) String volumeGroupHeatStackId,
-                                                       @WebParam(name="inputs") Map<String,String> inputs,
+                                                       @WebParam(name="inputs") Map<String,Object> inputs,
                                                        @WebParam(name="failIfExists") Boolean failIfExists,
                                                        @WebParam(name="backout") Boolean backout,
                                                        @WebParam(name="enableBridge") Boolean enableBridge,
@@ -70,7 +70,7 @@ public interface MsoVnfAdapterAsync
                                                        @WebParam(name="vnfName") @XmlElement(required=true) String vnfName,
                                                        @WebParam(name="requestType") @XmlElement(required=false) String requestType,
                                                        @WebParam(name="volumeGroupHeatStackId") @XmlElement(required=false) String volumeGroupHeatStackId,
-                                                       @WebParam(name="inputs") Map<String,String> inputs,
+                                                       @WebParam(name="inputs") Map<String,Object> inputs,
                                                        @WebParam(name="messageId") @XmlElement(required=true) String messageId,
                                                        @WebParam(name="request") MsoRequest msoRequest,
                                                        @WebParam(name="notificationUrl") @XmlElement(required=true) String notificationUrl );
index 6340f76..2c9d377 100644 (file)
@@ -125,7 +125,7 @@ public class MsoVnfAdapterAsyncImpl implements MsoVnfAdapterAsync {
                             String vnfName,
                             String requestType,
                             String volumeGroupHeatStackId,
-                            Map <String, String> inputs,
+                            Map <String, Object> inputs,
                             Boolean failIfExists,
                             Boolean backout,
                             Boolean enableBridge,
@@ -214,7 +214,7 @@ public class MsoVnfAdapterAsyncImpl implements MsoVnfAdapterAsync {
                             String vnfName,
                             String requestType,
                             String volumeGroupHeatStackId,
-                            Map <String, String> inputs,
+                            Map <String, Object> inputs,
                             String messageId,
                             MsoRequest msoRequest,
                             String notificationUrl) {
index 94434cd..9616076 100644 (file)
@@ -180,7 +180,7 @@ public class MsoVnfAdapterImpl implements MsoVnfAdapter {
                            String vnfName,
                            String requestType,
                            String volumeGroupHeatStackId,
-                           Map <String, String> inputs,
+                           Map <String, Object> inputs,
                            Boolean failIfExists,
                            Boolean backout,
                            Boolean enableBridge,
@@ -265,7 +265,7 @@ public class MsoVnfAdapterImpl implements MsoVnfAdapter {
                            String vnfName,
                            String requestType,
                            String volumeGroupHeatStackId,
-                           Map <String, String> inputs,
+                           Map <String, Object> inputs,
                            MsoRequest msoRequest,
                            Holder <Map <String, String>> outputs,
                            Holder <VnfRollback> rollback) throws VnfException {
@@ -474,7 +474,7 @@ public class MsoVnfAdapterImpl implements MsoVnfAdapter {
         return stringOutputs;
     }
 
-    private Map <String, Object> copyStringInputs (Map <String, String> stringInputs) {
+    private Map <String, Object> copyStringInputs (Map <String, Object> stringInputs) {
         return new HashMap <> (stringInputs);
     }
 
@@ -579,7 +579,7 @@ public class MsoVnfAdapterImpl implements MsoVnfAdapter {
             String volumeGroupHeatStackId,
             String baseVfHeatStackId,
             String modelCustomizationUuid,
-            Map <String, String> inputs,
+            Map <String, Object> inputs,
             Boolean failIfExists,
             Boolean backout,
             Boolean enableBridge,
@@ -1343,7 +1343,7 @@ public class MsoVnfAdapterImpl implements MsoVnfAdapter {
                            String baseVfHeatStackId,
                            String vfModuleStackId,
                            String modelCustomizationUuid,
-                           Map <String, String> inputs,
+                           Map <String, Object> inputs,
                            MsoRequest msoRequest,
                            Holder <Map <String, String>> outputs,
                            Holder <VnfRollback> rollback) throws VnfException {
@@ -1794,8 +1794,8 @@ public class MsoVnfAdapterImpl implements MsoVnfAdapter {
                                hasJson = true;
                                String jsonString = null;
                                try {
-                                       jsonString = inputs.get(parm.getParamName());
-                                       jsonNode = new ObjectMapper().readTree(jsonString);
+                                       jsonString = JSON_MAPPER.writeValueAsString(inputs.get(parm.getParamName()));
+                                       jsonNode = JSON_MAPPER.readTree(jsonString);
                                } catch (JsonParseException jpe) {
                                        //TODO - what to do here?
                                        //for now - send the error to debug, but just leave it as a String
@@ -1816,8 +1816,8 @@ public class MsoVnfAdapterImpl implements MsoVnfAdapter {
                                hasJson = true;
                                String jsonString = null;
                                try {
-                                       jsonString = inputs.get(parm.getParamAlias());
-                                       jsonNode = new ObjectMapper().readTree(jsonString);
+                                       jsonString = (String)inputs.get(parm.getParamAlias());
+                                       jsonNode = JSON_MAPPER.readTree(jsonString);
                                } catch (JsonParseException jpe) {
                                        //TODO - what to do here?
                                        //for now - send the error to debug, but just leave it as a String
@@ -1847,7 +1847,7 @@ public class MsoVnfAdapterImpl implements MsoVnfAdapter {
                         // They've submitted using an alias name. Remove that from inputs, and add back using real name.
                         String realParamName = parm.getParamName ();
                         String alias = parm.getParamAlias ();
-                        String value = inputs.get (alias);
+                        Object value = inputs.get (alias);
                         LOGGER.debug ("*Found an Alias: paramName=" + realParamName
                                       + ",alias="
                                       + alias
@@ -1911,9 +1911,9 @@ public class MsoVnfAdapterImpl implements MsoVnfAdapter {
             Map<String, Object> inputsTwo = null;
             if (hasJson && jsonParams.size() > 0) {
                inputsTwo = new HashMap<>();
-               for (Map.Entry<String, String> entry : inputs.entrySet()) {
+               for (Map.Entry<String, Object> entry : inputs.entrySet()) {
                        String keyParamName = entry.getKey();
-                       String value = entry.getValue();
+                       Object value = entry.getValue();
                        if (jsonParams.containsKey(keyParamName)) {
                                inputsTwo.put(keyParamName, jsonParams.get(keyParamName));
                        } else {
@@ -2085,7 +2085,7 @@ public class MsoVnfAdapterImpl implements MsoVnfAdapter {
      */
     private boolean valetCreateRequest(String cloudSiteId, String tenantId, Map<String, Object> heatFilesObjects, Map<String, Object> nestedTemplatesChecked,
                String vfModuleName, boolean backout, HeatTemplate heatTemplate, String newEnvironmentString, Map<String, Object> goldenInputs,
-               MsoRequest msoRequest, Map<String, String> inputs, boolean failRequestOnValetFailure, Holder<Map<String, Object>> valetModifiedParamsHolder) throws VnfException {
+               MsoRequest msoRequest, Map<String, Object> inputs, boolean failRequestOnValetFailure, Holder<Map<String, Object>> valetModifiedParamsHolder) throws VnfException {
                boolean valetSucceeded = false;
                String valetErrorMessage = "more detail not available";
                try {
@@ -2095,8 +2095,8 @@ public class MsoVnfAdapterImpl implements MsoVnfAdapter {
                        HeatRequest heatRequest = new HeatRequest(vfModuleName, backout, heatTemplate.getTimeoutMinutes(),
                                        heatTemplate.getTemplateBody(), newEnvironmentString, files, goldenInputs);
                        GenericValetResponse<ValetCreateResponse> createReq = this.vci.callValetCreateRequest(msoRequest.getRequestId(),
-                                       cloudSiteId, tenantId, msoRequest.getServiceInstanceId(), inputs.get("vnf_id"),
-                                       inputs.get("vnf_name"), inputs.get("vf_module_id"), inputs.get("vf_module_name"), keystoneUrl,
+                                       cloudSiteId, tenantId, msoRequest.getServiceInstanceId(), (String)inputs.get("vnf_id"),
+                                       (String)inputs.get("vnf_name"), (String)inputs.get("vf_module_id"), (String)inputs.get("vf_module_name"), keystoneUrl,
                                        heatRequest);
                        ValetCreateResponse vcr = createReq.getReturnObject();
                        if (vcr != null && createReq.getStatusCode() == 200) {
@@ -2145,7 +2145,7 @@ public class MsoVnfAdapterImpl implements MsoVnfAdapter {
        private boolean valetUpdateRequest(String cloudSiteId, String tenantId,
                        Map<String, Object> heatFilesObjects, Map<String, Object> nestedTemplatesChecked, String vfModuleName,
                        boolean backout, HeatTemplate heatTemplate, String newEnvironmentString,
-                       Map<String, Object> goldenInputs, MsoRequest msoRequest, Map<String, String> inputs,
+                       Map<String, Object> goldenInputs, MsoRequest msoRequest, Map<String, Object> inputs,
                        boolean failRequestOnValetFailure, Holder<Map<String, Object>> valetModifiedParamsHolder) throws VnfException {
 
                boolean valetSucceeded = false;
@@ -2158,8 +2158,8 @@ public class MsoVnfAdapterImpl implements MsoVnfAdapter {
                                        heatTemplate.getTemplateBody(), newEnvironmentString, files, goldenInputs);
                        // vnf name is not sent to MSO on update requests - so we will set it to the vf module name for now
                        GenericValetResponse<ValetUpdateResponse> updateReq = this.vci.callValetUpdateRequest(msoRequest.getRequestId(),
-                                       cloudSiteId, tenantId, msoRequest.getServiceInstanceId(), inputs.get("vnf_id"),
-                                       vfModuleName, inputs.get("vf_module_id"), vfModuleName, keystoneUrl,
+                                       cloudSiteId, tenantId, msoRequest.getServiceInstanceId(), (String)inputs.get("vnf_id"),
+                                       vfModuleName, (String)inputs.get("vf_module_id"), vfModuleName, keystoneUrl,
                                        heatRequest);
                        ValetUpdateResponse vur = updateReq.getReturnObject();
                        if (vur != null && updateReq.getStatusCode() == 200) {
index 00e7237..f34a794 100644 (file)
@@ -132,7 +132,7 @@ public class MsoVnfCloudifyAdapterImpl implements MsoVnfAdapter {
                            String vnfName,
                            String requestType,
                            String volumeGroupHeatStackId,
-                           Map <String, String> inputs,
+                           Map <String, Object> inputs,
                            Boolean failIfExists,
                            Boolean backout,
                            Boolean enableBridge,
@@ -160,7 +160,7 @@ public class MsoVnfCloudifyAdapterImpl implements MsoVnfAdapter {
                            String vnfName,
                            String requestType,
                            String volumeGroupHeatStackId,
-                           Map <String, String> inputs,
+                           Map <String, Object> inputs,
                            MsoRequest msoRequest,
                            Holder <Map <String, String>> outputs,
                            Holder <VnfRollback> rollback)
@@ -428,7 +428,7 @@ public class MsoVnfCloudifyAdapterImpl implements MsoVnfAdapter {
        return;
     }
 
-    private void sendMapToDebug(Map<String, String> inputs) {
+    private void sendMapToDebug(Map<String, Object> inputs) {
        int i = 0;
        StringBuilder sb = new StringBuilder("inputs:");
        if (inputs == null) {
@@ -567,7 +567,7 @@ public class MsoVnfCloudifyAdapterImpl implements MsoVnfAdapter {
             String volumeGroupId,
             String baseVfModuleId,
             String modelCustomizationUuid,
-            Map <String, String> inputs,
+            Map <String, Object> inputs,
             Boolean failIfExists,
             Boolean backout,
             Boolean enableBridge,
@@ -605,7 +605,7 @@ public class MsoVnfCloudifyAdapterImpl implements MsoVnfAdapter {
 
         if (inputs == null) {
                // Create an empty set of inputs
-               inputs = new HashMap<String,String>();
+               inputs = new HashMap<>();
                LOGGER.debug("inputs == null - setting to empty");
         } else {
                this.sendMapToDebug(inputs);
@@ -1212,7 +1212,7 @@ public class MsoVnfCloudifyAdapterImpl implements MsoVnfAdapter {
                            String baseVfHeatStackId,
                            String vfModuleStackId,
                            String modelCustomizationUuid,
-                           Map <String, String> inputs,
+                           Map <String, Object> inputs,
                            MsoRequest msoRequest,
                            Holder <Map <String, String>> outputs,
                            Holder <VnfRollback> rollback) throws VnfException
index 09ee0cd..13212be 100644 (file)
@@ -155,7 +155,7 @@ public class MsoVnfPluginAdapterImpl implements MsoVnfAdapter {
                            String vnfName,
                            String requestType,
                            String volumeGroupHeatStackId,
-                           Map <String, String> inputs,
+                           Map <String, Object> inputs,
                            Boolean failIfExists,
                            Boolean backout,
                            Boolean enableBridge,
@@ -183,7 +183,7 @@ public class MsoVnfPluginAdapterImpl implements MsoVnfAdapter {
                            String vnfName,
                            String requestType,
                            String volumeGroupHeatStackId,
-                           Map <String, String> inputs,
+                           Map <String, Object> inputs,
                            MsoRequest msoRequest,
                            Holder <Map <String, String>> outputs,
                            Holder <VnfRollback> rollback)
@@ -379,14 +379,14 @@ public class MsoVnfPluginAdapterImpl implements MsoVnfAdapter {
         * Normalize an input value to an Object, based on the target parameter type.
         * If the type is not recognized, it will just be returned unchanged (as a string).
         */
-       private Object convertInputValue (String inputValue, HeatTemplateParam templateParam)
+       private Object convertInputValue (Object inputValue, HeatTemplateParam templateParam)
        {
                String type = templateParam.getParamType();
                LOGGER.debug("Parameter: " + templateParam.getParamName() + " is of type " + type);
 
                if (type.equalsIgnoreCase("number")) {
                        try {
-                               return Integer.valueOf(inputValue);
+                               return Integer.valueOf(inputValue.toString());
                        }
                        catch (Exception e) {
                                LOGGER.debug("Unable to convert " + inputValue + " to an integer!" , e);
@@ -394,7 +394,7 @@ public class MsoVnfPluginAdapterImpl implements MsoVnfAdapter {
                        }
                } else if (type.equalsIgnoreCase("json")) {
                        try {
-                               JsonNode jsonNode = new ObjectMapper().readTree(inputValue);
+                               JsonNode jsonNode = JSON_MAPPER.readTree(JSON_MAPPER.writeValueAsString(inputValue));
                                return jsonNode;
                        }
                        catch (Exception e) {
@@ -402,7 +402,7 @@ public class MsoVnfPluginAdapterImpl implements MsoVnfAdapter {
                                return null;
                        }
                } else if (type.equalsIgnoreCase("boolean")) {
-                       return new Boolean(inputValue);
+                       return new Boolean(inputValue.toString());
                }
 
                // Nothing else matched.  Return the original string
@@ -471,7 +471,7 @@ public class MsoVnfPluginAdapterImpl implements MsoVnfAdapter {
        return;
     }
 
-    private void sendMapToDebug(Map<String, String> inputs) {
+    private void sendMapToDebug(Map<String, Object> inputs) {
        int i = 0;
        StringBuilder sb = new StringBuilder("inputs:");
        if (inputs == null) {
@@ -608,7 +608,7 @@ public class MsoVnfPluginAdapterImpl implements MsoVnfAdapter {
             String volumeGroupId,
             String baseVfModuleId,
             String modelCustomizationUuid,
-            Map <String, String> inputs,
+            Map <String, Object> inputs,
             Boolean failIfExists,
             Boolean backout,
             Boolean enableBridge,
@@ -1212,7 +1212,7 @@ public class MsoVnfPluginAdapterImpl implements MsoVnfAdapter {
                            String baseVfHeatStackId,
                            String vfModuleStackId,
                            String modelCustomizationUuid,
-                           Map <String, String> inputs,
+                           Map <String, Object> inputs,
                            MsoRequest msoRequest,
                            Holder <Map <String, String>> outputs,
                            Holder <VnfRollback> rollback) throws VnfException
index 4f1c2c8..3ebaea9 100644 (file)
@@ -76,7 +76,7 @@ public class MsoVnfAdapterAsyncImplTest extends BaseRestTestUtils {
                String vnfName = "DEV-VF-1802-it3-pwt3-v6-vSAMP10a-addon2-Replace-1001/stackId";
                String notificationUrl = "http://localhost:"+wireMockPort+"/notify/adapterNotify/updateVnfNotificationRequest";
                instance.createVnfA("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", vnfName, "VFMOD",
-                               "volumeGroupHeatStackId|1", new HashMap<String, String>(), Boolean.FALSE, Boolean.TRUE, Boolean.FALSE, "messageId",
+                               "volumeGroupHeatStackId|1", new HashMap<String, Object>(), Boolean.FALSE, Boolean.TRUE, Boolean.FALSE, "messageId",
                                msoRequest, notificationUrl);
 
                verify(1,postRequestedFor(urlEqualTo("/notify/adapterNotify/updateVnfNotificationRequest")));
@@ -86,7 +86,7 @@ public class MsoVnfAdapterAsyncImplTest extends BaseRestTestUtils {
        public void createVNFTest_Exception() throws Exception {
                String notificationUrl = "http://localhost:"+wireMockPort+"/notify/adapterNotify/updateVnfNotificationRequest";
                instance.createVnfA("mdt1", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "vSAMP12", "VFMOD",
-                               "volumeGroupHeatStackId|1", new HashMap<String, String>(), Boolean.FALSE, Boolean.TRUE, Boolean.FALSE, "messageId",
+                               "volumeGroupHeatStackId|1", new HashMap<String, Object>(), Boolean.FALSE, Boolean.TRUE, Boolean.FALSE, "messageId",
                                null, notificationUrl);
 
                verify(1,postRequestedFor(urlEqualTo("/notify/adapterNotify/updateVnfNotificationRequest")));
@@ -99,7 +99,7 @@ public class MsoVnfAdapterAsyncImplTest extends BaseRestTestUtils {
                msoRequest.setRequestId("12345");
                msoRequest.setServiceInstanceId("12345");
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                stubFor(post(urlPathEqualTo("/notify/adapterNotify/updateVnfNotificationRequest")).withRequestBody
                                (containing("messageId"))
@@ -116,7 +116,7 @@ public class MsoVnfAdapterAsyncImplTest extends BaseRestTestUtils {
                msoRequest.setRequestId("12345");
                msoRequest.setServiceInstanceId("12345");
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                stubFor(post(urlPathEqualTo("/notify/adapterNotify/updateVnfNotificationRequest")).withRequestBody
                                (containing("messageId"))
index 849c081..be3b54e 100644 (file)
@@ -104,7 +104,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
 
                MsoRequest msoRequest = getMsoRequest();
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.createVfModule("MTN13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                                "volumeGroupHeatStackId|1", "baseVfHeatStackId", "88a6ca3ee0394ade9403f075db23167e", map,
@@ -124,7 +124,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
 
                MsoRequest msoRequest = getMsoRequest();
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                                "volumeGroupHeatStackId|1", "baseVfHeatStackId", "88a6ca3ee0394ade9403f075db23167e", map,
@@ -144,7 +144,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
 
                MsoRequest msoRequest = getMsoRequest();
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                                "volumeGroupHeatStackId|1", "baseVfHeatStackId", "88a6ca3ee0394ade9403f075db23167e", map,
@@ -164,7 +164,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
 
                MsoRequest msoRequest = getMsoRequest();
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                                "volumeGroupHeatStackId|1", "baseVfHeatStackId", "88a6ca3ee0394ade9403f075db23167e", map,
@@ -181,7 +181,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
 
                MsoRequest msoRequest = getMsoRequest();
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                                "volumeGroupHeatStackId|1", "baseVfHeatStackId", "88a6ca3ee0394ade9403f075db23167e", map,
@@ -195,7 +195,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
                expectedException.expect(VnfException.class);
                MsoRequest msoRequest = getMsoRequest();
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                                "volumeGroupHeatStackId|1", "baseVfHeatStackId", "88a6ca3ee0394ade9403f075db23167e", map,
@@ -211,7 +211,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
 
                MsoRequest msoRequest = getMsoRequest();
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                                "volumeGroupHeatStackId", "baseVfHeatStackId", "88a6ca3ee0394ade9403f075db23167e", map,
@@ -230,7 +230,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
 
                MsoRequest msoRequest = getMsoRequest();
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                                "volumeGroupHeatStackId", "baseVfHeatStackId", "88a6ca3ee0394ade9403f075db23167e", map,
@@ -250,7 +250,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
 
                MsoRequest msoRequest = getMsoRequest();
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                                "volumeGroupHeatStackId", "baseVfHeatStackId", "88a6ca3ee0394ade9403f075db23167e", map,
@@ -268,7 +268,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
 
                MsoRequest msoRequest = getMsoRequest();
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                                "volumeGroupHeatStackId", "baseVfHeatStackId", "88a6ca3ee0394ade9403f075db23167e", map,
@@ -300,7 +300,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
 
                MsoRequest msoRequest = getMsoRequest();
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                                "volumeGroupHeatStackId", "baseVfHeatStackId", "88a6ca3ee0394ade9403f075db23167e", map,
@@ -337,7 +337,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
 
                MsoRequest msoRequest = getMsoRequest();
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "XVFMOD",
                                "volumeGroupHeatStackId", "baseVfHeatStackId", null, map,
@@ -355,7 +355,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
 
                MsoRequest msoRequest = getMsoRequest();
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                                "volumeGroupHeatStackId", "baseVfHeatStackId", "88a6ca3ee0394ade9403f075db23167e", map,
@@ -375,7 +375,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
                expectedException.expect(VnfException.class);
                MsoRequest msoRequest = getMsoRequest();
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.updateVfModule("mdt1", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "vSAMP12", "VFMOD",
                                "volumeGroupHeatStackId|1", "baseVfHeatStackId", "vfModuleStackId",
@@ -388,7 +388,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
                expectedException.expect(VnfNotFound.class);
                MsoRequest msoRequest = getMsoRequest();
                mockOpenStackResponseAccess(wireMockPort);
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.updateVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", vnfName, "VFMOD",
                                "volumeGroupHeatStackId", "baseVfHeatStackId", "vfModuleStackId",
@@ -405,7 +405,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
                stubFor(get(urlPathEqualTo("/mockPublicUrl/stacks/volumeGroupHeatStackId"))
                                .willReturn(aResponse().withHeader("Content-Type", "application/json")
                                                .withStatus(HttpStatus.SC_INTERNAL_SERVER_ERROR)));
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.updateVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", vnfName, "VFMOD",
                                "volumeGroupHeatStackId", "baseVfHeatStackId", "vfModuleStackId",
@@ -422,7 +422,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
                stubFor(get(urlPathEqualTo("/mockPublicUrl/stacks/volumeGroupHeatStackId"))
                                .willReturn(aResponse().withHeader("Content-Type", "application/json")
                                                .withStatus(HttpStatus.SC_NOT_FOUND)));
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.updateVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", vnfName, "VFMOD",
                                "volumeGroupHeatStackId", "baseVfHeatStackId", "vfModuleStackId",
@@ -438,7 +438,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
                mockOpenStackGetStackVfModule_200();
                mockOpenstackGetWithResponse("/mockPublicUrl/stacks/volumeGroupHeatStackId",HttpStatus.SC_OK,"OpenstackResponse_Stack_Created_VfModule.json");
                mockOpenstackGetWithResponse("/mockPublicUrl/stacks/baseVfHeatStackId",HttpStatus.SC_INTERNAL_SERVER_ERROR,"OpenstackResponse_Stack_Created_VfModule.json");
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.updateVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", vnfName, "VFMOD",
                                "volumeGroupHeatStackId", "baseVfHeatStackId", "vfModuleStackId",
@@ -454,7 +454,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
                mockOpenStackGetStackVfModule_200();
                mockOpenstackGetWithResponse("/mockPublicUrl/stacks/volumeGroupHeatStackId",HttpStatus.SC_OK,"OpenstackResponse_Stack_Created_VfModule.json");
                mockOpenstackGetWithResponse("/mockPublicUrl/stacks/baseVfHeatStackId",HttpStatus.SC_NOT_FOUND,"OpenstackResponse_Stack_Created_VfModule.json");
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.updateVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", vnfName, "VFMOD",
                                "volumeGroupHeatStackId", "baseVfHeatStackId", "vfModuleStackId",
@@ -470,7 +470,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
                mockOpenStackGetStackVfModule_200();
                mockOpenstackGetWithResponse("/mockPublicUrl/stacks/volumeGroupHeatStackId",HttpStatus.SC_OK,"OpenstackResponse_Stack_Created_VfModule.json");
                mockOpenstackGetWithResponse("/mockPublicUrl/stacks/baseVfHeatStackId",HttpStatus.SC_OK,"OpenstackResponse_Stack_Created_VfModule.json");
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.updateVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", vnfName, "VFMOD",
                                "volumeGroupHeatStackId", "baseVfHeatStackId", "vfModuleStackId",
@@ -489,7 +489,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
 
                VfModuleCustomization vfModuleCustomization = getVfModuleCustomization();
                vfModuleCustomization.getVfModule().getModuleHeatTemplate().setParameters(new HashSet<>());
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.updateVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", vnfName, "VFMOD",
                                "volumeGroupHeatStackId", "baseVfHeatStackId", "vfModuleStackId",
@@ -509,7 +509,7 @@ public class MsoVnfAdapterImplTest extends BaseRestTestUtils {
 
                VfModuleCustomization vfModuleCustomization = getVfModuleCustomization();
                vfModuleCustomization.getVfModule().getModuleHeatTemplate().setParameters(new HashSet<>());
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                instance.updateVfModule("MTN13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", vnfName, "VFMOD",
                                "volumeGroupHeatStackId", "baseVfHeatStackId", "vfModuleStackId",
index addc12a..4484390 100644 (file)
@@ -265,7 +265,7 @@ public class MsoVnfCloudifyAdapterImplTest extends BaseRestTestUtils {
                msoRequest.setRequestId("12345");
                msoRequest.setServiceInstanceId("12345");
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                map.put("key1", "value1");
                try {
                        instance.createVnf("mdt1", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "vSAMP12", "VFMOD",
@@ -282,7 +282,7 @@ public class MsoVnfCloudifyAdapterImplTest extends BaseRestTestUtils {
                msoRequest.setRequestId("12345");
                msoRequest.setServiceInstanceId("12345");
 
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                
                map.put("key1", "value1");
                try {
index 1fd3ffc..75cea9e 100644 (file)
@@ -66,7 +66,7 @@ public class MsoVnfMulticloudAdapterImplTest extends BaseRestTestUtils{
     @Test
     public void createVfModule() throws Exception {
         //expectedException.expect(VnfException.class);
-        Map<String, String> stackInputs = new HashMap<>();
+        Map<String, Object> stackInputs = new HashMap<>();
         stackInputs.put("oof_directives", "{}");
         stackInputs.put("sdnc_directives", "{}");
         stackInputs.put("generic_vnf_id", "genVNFID");
index a422680..0cd3d1a 100644 (file)
@@ -56,7 +56,7 @@ public class MsoVnfPluginAdapterImplTest extends BaseRestTestUtils {
     public void createVfModule_ModelCustUuidIsNull() throws Exception {
         expectedException.expect(VnfException.class);
         MsoRequest msoRequest = getMsoRequest();
-        Map<String, String> map = new HashMap<>();
+        Map<String, Object> map = new HashMap<>();
         map.put("key1", "value1");
         msoVnfPluginAdapter.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                 "volumeGroupHeatStackId|1", "baseVfHeatStackId", null, map,
@@ -68,7 +68,7 @@ public class MsoVnfPluginAdapterImplTest extends BaseRestTestUtils {
     public void createVfModule_ModelCustUuidIsNotFound() throws Exception {
         expectedException.expect(VnfException.class);
         MsoRequest msoRequest = getMsoRequest();
-        Map<String, String> map = new HashMap<>();
+        Map<String, Object> map = new HashMap<>();
         map.put("key1", "value1");
         msoVnfPluginAdapter.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                 "volumeGroupHeatStackId|1", "baseVfHeatStackId", "88a6ca3ee0394ade9403f075db23167e", map,
@@ -80,7 +80,7 @@ public class MsoVnfPluginAdapterImplTest extends BaseRestTestUtils {
     public void createVfModule_VduException() throws Exception {
         expectedException.expect(VnfException.class);
         MsoRequest msoRequest = getMsoRequest();
-        Map<String, String> map = new HashMap<>();
+        Map<String, Object> map = new HashMap<>();
         map.put("key1", "value1");
         msoVnfPluginAdapter.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                 "volumeGroupHeatStackId|1", "baseVfHeatStackId", "9b339a61-69ca-465f-86b8-1c72c582b8e8", map,
@@ -94,7 +94,7 @@ public class MsoVnfPluginAdapterImplTest extends BaseRestTestUtils {
         mockOpenStackGetStackVfModule_200();
 
         MsoRequest msoRequest = getMsoRequest();
-        Map<String, String> map = new HashMap<>();
+        Map<String, Object> map = new HashMap<>();
         map.put("key1", "value1");
         msoVnfPluginAdapter.createVfModule("MTN13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                 null, "baseVfHeatStackId", "9b339a61-69ca-465f-86b8-1c72c582b8e8", map,
@@ -107,7 +107,7 @@ public class MsoVnfPluginAdapterImplTest extends BaseRestTestUtils {
         expectedException.expect(VnfException.class);
         mockOpenStackResponseAccess(wireMockPort);
         MsoRequest msoRequest = getMsoRequest();
-        Map<String, String> map = new HashMap<>();
+        Map<String, Object> map = new HashMap<>();
         map.put("key1", "value1");
         msoVnfPluginAdapter.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                 "volumeGroupHeatStackId|1", "baseVfHeatStackId", "9b339a61-69ca-465f-86b8-1c72c582b8e8", map,
@@ -125,7 +125,7 @@ public class MsoVnfPluginAdapterImplTest extends BaseRestTestUtils {
                         .withBodyFile("OpenstackResponse_Stack_Created_VfModule.json")
                         .withStatus(HttpStatus.SC_OK)));
         MsoRequest msoRequest = getMsoRequest();
-        Map<String, String> map = new HashMap<>();
+        Map<String, Object> map = new HashMap<>();
         map.put("key1", "value1");
         msoVnfPluginAdapter.createVfModule("mtn13", "88a6ca3ee0394ade9403f075db23167e", "vnf", "1", "", vnfName, "", "VFMOD",
                 "volumeGroupHeatStackId", "baseVfHeatStackId", "9b339a61-69ca-465f-86b8-1c72c582b8e8", map,
index c502620..343b624 100644 (file)
@@ -169,7 +169,7 @@ public class VnfAdapterRestTest extends BaseRestTestUtils {
                MsoRequest msoReq = new MsoRequest();
                boolean failIfExists = true;
                Boolean enableBridge = null;
-               Map<String, String> vfModuleParams = new HashMap<String, String>();
+               Map<String, Object> vfModuleParams = new HashMap<>();
 
 
                vfModuleParams.put("vf_module_id", VF_MODULE_ID);
@@ -236,7 +236,7 @@ public class VnfAdapterRestTest extends BaseRestTestUtils {
                MsoRequest msoReq = new MsoRequest();
                boolean failIfExists = true;
                boolean enableBridge = false;
-               Map<String, String> vfModuleParams = new HashMap<String, String>();
+               Map<String, Object> vfModuleParams = new HashMap<>();
 
                vfModuleParams.put("vf_module_id", VF_MODULE_ID);
                vfModuleParams.put("vnf_id", AAI_VNF_ID);
@@ -385,7 +385,7 @@ public class VnfAdapterRestTest extends BaseRestTestUtils {
                msoRequest.setRequestId(MSO_REQUEST_ID);
                msoRequest.setServiceInstanceId(MSO_SERVICE_INSTANCE_ID);
                
-               Map<String, String> vfModuleParams = new HashMap<String, String>();
+               Map<String, Object> vfModuleParams = new HashMap<>();
 
                vfModuleParams.put("exn_direct_net_fqdn", EXN_DIRECT_NET_FQDN);
                vfModuleParams.put("exn_hsl_net_fqdn", EXN_HSL_NET_FQDN);
@@ -511,7 +511,7 @@ public class VnfAdapterRestTest extends BaseRestTestUtils {
                MsoRequest msoReq = new MsoRequest();
                boolean failIfExists = true;
                boolean enableBridge = false;
-               Map<String, String> vfModuleParams = new HashMap<String, String>();
+               Map<String, Object> vfModuleParams = new HashMap<>();
 
                vfModuleParams.put("vf_module_id", VF_MODULE_ID);
                vfModuleParams.put("vnf_id", AAI_VNF_ID);
index 6898279..d0d06ae 100644 (file)
@@ -56,7 +56,7 @@ public class VolumeGroupAdapterCommon extends BaseRestTestUtils {
         request.setModelCustomizationUuid(MODEL_CUSTOMIZATION_UUID);
         request.setSkipAAI(true);
         request.setVolumeGroupStackId(VOUME_GROUP_NAME);
-        Map<String, String> volumeGroupParams = getVolumeGroupParamsMap();
+        Map<String, Object> volumeGroupParams = getVolumeGroupParamsMap();
         request.setVolumeGroupParams(volumeGroupParams);
         MsoRequest msoReq = getMsoRequest();
         request.setMsoRequest(msoReq);
@@ -104,7 +104,7 @@ public class VolumeGroupAdapterCommon extends BaseRestTestUtils {
         request.setFailIfExists(true);
         request.setEnableBridge(false);
         request.setSuppressBackout(true);
-        Map<String, String> volumeGroupParams = getVolumeGroupParamsMap();
+        Map<String, Object> volumeGroupParams = getVolumeGroupParamsMap();
         request.setVolumeGroupParams(volumeGroupParams);
         MsoRequest msoReq = getMsoRequest();
         request.setMsoRequest(msoReq);
@@ -112,8 +112,8 @@ public class VolumeGroupAdapterCommon extends BaseRestTestUtils {
         return request;
     }
 
-    private Map<String, String> getVolumeGroupParamsMap() {
-        Map<String, String> volumeGroupParams =  new HashMap<>();
+    private Map<String, Object> getVolumeGroupParamsMap() {
+        Map<String, Object> volumeGroupParams =  new HashMap<>();
         volumeGroupParams.put("fsb_volume_type_0","volume_type");
         volumeGroupParams.put("fsb_volume_image_name_1","vol_img_1");
         volumeGroupParams.put("fsb_volume_image_name_0","vol_img_0");
index d37c8d1..cd66e51 100644 (file)
@@ -43,7 +43,7 @@ public class RequestContext implements Serializable{
        @JsonProperty("subscription-service-type")
        private String subscriptionServiceType;
        @JsonProperty("user-params")
-       private HashMap<String, String> userParams;
+       private Map<String, Object> userParams = new HashMap<>();
        @JsonProperty("action")
        private String action;
        @JsonProperty("callback-url")
@@ -87,10 +87,10 @@ public class RequestContext implements Serializable{
        public void setSubscriptionServiceType(String subscriptionServiceType) {
                this.subscriptionServiceType = subscriptionServiceType;
        }
-       public HashMap<String, String> getUserParams() {
+       public Map<String, Object> getUserParams() {
                return userParams;
        }
-       public void setUserParams(HashMap<String, String> userParams) {
+       public void setUserParams(Map<String, Object> userParams) {
                this.userParams = userParams;
        }
        public String getAction() {
index d6597ec..81a504d 100644 (file)
@@ -355,8 +355,8 @@ public class BBInputSetupMapperLayer {
                return requestParams;
        }
        
-       protected HashMap<String,String> mapNameValueUserParams(org.onap.so.serviceinstancebeans.RequestParameters requestParameters) {         
-               HashMap<String,String> userParamsResult = new HashMap<String,String>();
+       protected Map<String,Object> mapNameValueUserParams(org.onap.so.serviceinstancebeans.RequestParameters requestParameters) {             
+               Map<String,Object> userParamsResult = new HashMap<String,Object>();
                if (requestParameters.getUserParams() != null) {
                        List<Map<String, Object>> userParams = requestParameters.getUserParams();
                        for (Map<String, Object> userParamsMap : userParams) {
index 7e76e52..753a354 100644 (file)
@@ -643,7 +643,7 @@ public class BBInputSetupMapperLayerTest {
        @Test
        public void testMapNameValueUserParams() throws IOException {           
                RequestDetails requestDetails = mapper.readValue(new File(RESOURCE_PATH + "RequestDetailsInput_mapReqContext.json"), RequestDetails.class);
-               HashMap<String,String> actual = bbInputSetupMapperLayer.mapNameValueUserParams(requestDetails.getRequestParameters());
+               Map<String,Object> actual = bbInputSetupMapperLayer.mapNameValueUserParams(requestDetails.getRequestParameters());
 
                assertTrue(actual.containsKey("name1"));
                assertTrue(actual.containsValue("value1"));
index 701817a..2559087 100644 (file)
@@ -110,8 +110,8 @@ public class VnfAdapterObjectMapper {
                return deleteVolumeGroupRequest;
        }
        
-       public Map<String, String> createVolumeGroupParams(RequestContext requestContext,GenericVnf genericVnf, VolumeGroup volumeGroup, String sdncVfModuleQueryResponse) throws JsonParseException, JsonMappingException, IOException {
-               Map<String, String> volumeGroupParams = new HashMap<>();
+       public Map<String, Object> createVolumeGroupParams(RequestContext requestContext,GenericVnf genericVnf, VolumeGroup volumeGroup, String sdncVfModuleQueryResponse) throws JsonParseException, JsonMappingException, IOException {
+               Map<String, Object> volumeGroupParams = new HashMap<>();
                final String USER_PARAM_NAME_KEY = "name";
         final String USER_PARAM_VALUE_KEY = "value";
                // sdncVfModuleQueryResponse will not be available in aLaCarte case
@@ -147,7 +147,7 @@ public class VnfAdapterObjectMapper {
                return msoRequest;
        }
        
-       private void buildParamsMapFromSdncParams(Map<String,String> volumeGroupParams, GenericResourceApiParam sdncParameters) {               
+       private void buildParamsMapFromSdncParams(Map<String,Object> volumeGroupParams, GenericResourceApiParam sdncParameters) {               
                if (sdncParameters != null) {
                        List<GenericResourceApiParamParam> sdncParametersList = sdncParameters.getParam();
                        if (sdncParametersList != null) {
index 352d4ec..98174d5 100644 (file)
@@ -161,7 +161,7 @@ public class VnfAdapterVfModuleObjectMapper {
                return msoRequest;              
        }
        
-       private Map<String,String> buildVfModuleParamsMap(RequestContext requestContext, ServiceInstance serviceInstance, GenericVnf genericVnf, 
+       private Map<String,Object> buildVfModuleParamsMap(RequestContext requestContext, ServiceInstance serviceInstance, GenericVnf genericVnf, 
                                VfModule vfModule, String sdncVnfQueryResponse, String sdncVfModuleQueryResponse) throws JsonParseException, JsonMappingException, IOException {
                
                
@@ -169,7 +169,7 @@ public class VnfAdapterVfModuleObjectMapper {
                GenericResourceApiVfModuleTopology vfModuleTop = mapper.readValue(sdncVfModuleQueryResponse, GenericResourceApiVfModuleTopology.class);
                GenericResourceApiVnftopologyVnfTopology vnfTopology = vnfTop.getVnfTopology();
                GenericResourceApiVfmoduletopologyVfModuleTopology vfModuleTopology = vfModuleTop.getVfModuleTopology();
-               Map<String,String> paramsMap = new HashMap<>();
+               Map<String,Object> paramsMap = new HashMap<>();
 
                if( vfModuleTopology.getSdncGeneratedCloudResources()) {
                        buildParamsMapFromVfModuleSdncResponse(paramsMap, vfModuleTopology, true);
@@ -184,17 +184,15 @@ public class VnfAdapterVfModuleObjectMapper {
                buildMandatoryParamsMap(paramsMap, serviceInstance, genericVnf, vfModule);
                
                // Parameters received from the request should overwrite any parameters received from SDNC
+               paramsMap.putAll(requestContext.getUserParams());
                
-               if (requestContext.getUserParams() != null) {
-                       paramsMap.putAll(requestContext.getUserParams());
-               }
                if (vfModule.getCloudParams() != null) {
                        paramsMap.putAll(vfModule.getCloudParams());
                }
                return paramsMap;
        }
        
-       private void  buildMandatoryParamsMap(Map<String,String> paramsMap, ServiceInstance serviceInstance, GenericVnf genericVnf, VfModule vfModule) {                
+       private void  buildMandatoryParamsMap(Map<String,Object> paramsMap, ServiceInstance serviceInstance, GenericVnf genericVnf, VfModule vfModule) {                
                paramsMap.put("vnf_id", genericVnf.getVnfId());
                paramsMap.put("vnf_name", genericVnf.getVnfName());
                paramsMap.put("vf_module_id", vfModule.getVfModuleId());
@@ -209,7 +207,7 @@ public class VnfAdapterVfModuleObjectMapper {
                }               
        }
        
-       private void buildParamsMapFromVnfSdncResponse(Map<String,String> paramsMap, GenericResourceApiVnftopologyVnfTopology vnfTopology, Map<String,String> networkRoleMap, boolean skipVnfResourceAssignments) throws JsonParseException, JsonMappingException, IOException {                                
+       private void buildParamsMapFromVnfSdncResponse(Map<String,Object> paramsMap, GenericResourceApiVnftopologyVnfTopology vnfTopology, Map<String,String> networkRoleMap, boolean skipVnfResourceAssignments) throws JsonParseException, JsonMappingException, IOException {                                
                // Get VNF parameters from SDNC response
                GenericResourceApiParam vnfParametersData = vnfTopology.getVnfParametersData();
                buildParamsMapFromSdncParams(paramsMap, vnfParametersData);
@@ -225,7 +223,7 @@ public class VnfAdapterVfModuleObjectMapper {
                }
        }
        
-       private void buildAvailabilityZones (Map<String,String> paramsMap, GenericResourceApiVnfresourceassignmentsVnfResourceAssignments vnfResourceAssignments) {             
+       private void buildAvailabilityZones (Map<String,Object> paramsMap, GenericResourceApiVnfresourceassignmentsVnfResourceAssignments vnfResourceAssignments) {             
                GenericResourceApiVnfresourceassignmentsVnfresourceassignmentsAvailabilityZones availabilityZones = vnfResourceAssignments.getAvailabilityZones();
                if (availabilityZones != null) {
                        List<String> availabilityZonesList = availabilityZones.getAvailabilityZone();
@@ -237,7 +235,7 @@ public class VnfAdapterVfModuleObjectMapper {
                }
        }
        
-       private void buildVnfNetworks (Map<String,String> paramsMap, GenericResourceApiVnfresourceassignmentsVnfResourceAssignments vnfResourceAssignments, Map<String,String> networkRoleMap) {
+       private void buildVnfNetworks (Map<String,Object> paramsMap, GenericResourceApiVnfresourceassignmentsVnfResourceAssignments vnfResourceAssignments, Map<String,String> networkRoleMap) {
                GenericResourceApiVnfresourceassignmentsVnfresourceassignmentsVnfNetworks vnfNetworks = vnfResourceAssignments.getVnfNetworks();
                if (vnfNetworks != null) {
                        List<GenericResourceApiVnfNetworkData> vnfNetworksList = vnfNetworks.getVnfNetwork();
@@ -264,7 +262,7 @@ public class VnfAdapterVfModuleObjectMapper {
                }
        }
        
-       private void buildVnfNetworkSubnets(Map<String,String> paramsMap, GenericResourceApiVnfNetworkData vnfNetwork, String vnfNetworkKey) {
+       private void buildVnfNetworkSubnets(Map<String,Object> paramsMap, GenericResourceApiVnfNetworkData vnfNetwork, String vnfNetworkKey) {
                String vnfNetworkString = convertToString(vnfNetwork);
                Optional<String> ipv4Ips = jsonPath.locateResult(vnfNetworkString, "$.subnets-data.subnet-data[*].[?(@.ip-version == 'ipv4' && @.dhcp-enabled == 'Y')].subnet-id");
                if(ipv4Ips.isPresent())
@@ -275,7 +273,7 @@ public class VnfAdapterVfModuleObjectMapper {
                        addPairToMap(paramsMap, vnfNetworkKey, V6_SUBNET_ID, ipv6Ips.get());
        }       
 
-       private void buildParamsMapFromVfModuleSdncResponse(Map<String,String> paramsMap, GenericResourceApiVfmoduletopologyVfModuleTopology vfModuleTopology, boolean skipVfModuleAssignments) throws JsonParseException, JsonMappingException, IOException {          
+       private void buildParamsMapFromVfModuleSdncResponse(Map<String,Object> paramsMap, GenericResourceApiVfmoduletopologyVfModuleTopology vfModuleTopology, boolean skipVfModuleAssignments) throws JsonParseException, JsonMappingException, IOException {          
                // Get VF Module parameters from SDNC response
                GenericResourceApiParam vfModuleParametersData = vfModuleTopology.getVfModuleParameters();
                buildParamsMapFromSdncParams(paramsMap, vfModuleParametersData);
@@ -322,7 +320,7 @@ public class VnfAdapterVfModuleObjectMapper {
                }
        }
        
-       protected void buildVlanInformation(Map<String, String> paramsMap,
+       protected void buildVlanInformation(Map<String, Object> paramsMap,
                        GenericResourceApiVmNetworkData network, String key, String networkKey) {
                
                String networkString = convertToString(network);
@@ -352,7 +350,7 @@ public class VnfAdapterVfModuleObjectMapper {
                }
        }
 
-       private void buildVfModuleVmNames(Map<String,String> paramsMap, GenericResourceApiVmTopologyData vm, String key) {
+       private void buildVfModuleVmNames(Map<String,Object> paramsMap, GenericResourceApiVmTopologyData vm, String key) {
                String values = "";
                GenericResourceApiVmtopologydataVmNames vmNames = vm.getVmNames();
                if (vmNames != null) {
@@ -373,7 +371,7 @@ public class VnfAdapterVfModuleObjectMapper {
                }
        }
        
-       private void buildVfModuleFloatingIps(Map<String,String> paramsMap, GenericResourceApiVmNetworkData network, String key, String networkKey) {
+       private void buildVfModuleFloatingIps(Map<String,Object> paramsMap, GenericResourceApiVmNetworkData network, String key, String networkKey) {
                GenericResourceApiVmnetworkdataFloatingIps floatingIps = network.getFloatingIps();
                if (floatingIps != null) {
                        List<String> floatingIpV4List = floatingIps.getFloatingIpV4();
@@ -397,7 +395,7 @@ public class VnfAdapterVfModuleObjectMapper {
                }
        }
        
-       private void buildVfModuleInterfaceRoutePrefixes(Map<String,String> paramsMap, GenericResourceApiVmNetworkData network, String key, String networkKey) {
+       private void buildVfModuleInterfaceRoutePrefixes(Map<String,Object> paramsMap, GenericResourceApiVmNetworkData network, String key, String networkKey) {
                GenericResourceApiVmnetworkdataInterfaceRoutePrefixes interfaceRoutePrefixes = network.getInterfaceRoutePrefixes();
                if (interfaceRoutePrefixes != null) {
                        List<String> interfaceRoutePrefixesList = interfaceRoutePrefixes.getInterfaceRoutePrefix();
@@ -421,7 +419,7 @@ public class VnfAdapterVfModuleObjectMapper {
                }
        }
        
-       private void buildVfModuleSriovParameters(Map<String,String> paramsMap, GenericResourceApiVmNetworkData network, String networkKey) {
+       private void buildVfModuleSriovParameters(Map<String,Object> paramsMap, GenericResourceApiVmNetworkData network, String networkKey) {
                // SRIOV Parameters
                GenericResourceApiVmnetworkdataSriovParameters sriovParameters = network.getSriovParameters();
                if (sriovParameters != null) {
@@ -447,7 +445,7 @@ public class VnfAdapterVfModuleObjectMapper {
                }
        }
        
-       private void buildVfModuleNetworkInformation(Map<String,String> paramsMap, GenericResourceApiVmNetworkData network, String key, String networkKey) {
+       private void buildVfModuleNetworkInformation(Map<String,Object> paramsMap, GenericResourceApiVmNetworkData network, String key, String networkKey) {
                
                GenericResourceApiVmnetworkdataNetworkInformationItems networkInformationItems = network.getNetworkInformationItems();
                StringBuilder sbIpv4Ips = new StringBuilder();
@@ -500,7 +498,7 @@ public class VnfAdapterVfModuleObjectMapper {
         * Build Count of SubInterfaces, VLAN Tag, network_name, network_id,
         * ip_address (V4 and V6) and Floating IPs Addresses (V4 and V6) for Heat Template
         */     
-       private void buildParamsMapFromVfModuleForHeatTemplate(Map<String,String> paramsMap, GenericResourceApiVmTopologyData vm) {
+       private void buildParamsMapFromVfModuleForHeatTemplate(Map<String,Object> paramsMap, GenericResourceApiVmTopologyData vm) {
                GenericResourceApiVmtopologydataVmNames vmNames = vm.getVmNames();
        
                if (vmNames != null) {
@@ -520,7 +518,7 @@ public class VnfAdapterVfModuleObjectMapper {
        /*
         * Parse vnfcNames data to build Mapping from GenericResourceApi SDNC for Heat Template.
         */     
-       private void parseVnfcNamesData(Map<String,String> paramsMap, GenericResourceApiVmtopologydataVmnamesVnfcNames vnfcNames) {
+       private void parseVnfcNamesData(Map<String,Object> paramsMap, GenericResourceApiVmtopologydataVmnamesVnfcNames vnfcNames) {
                
                if (vnfcNames != null) {
                        GenericResourceApiVnfcNetworkData vnfcNetworks = vnfcNames.getVnfcNetworks();
@@ -544,7 +542,7 @@ public class VnfAdapterVfModuleObjectMapper {
         * Build Count of SubInterfaces, VLAN Tag, network_name, network_id,
         * ip_address (V4 and V6) and Floating IPs Addresses (V4 and V6) for Heat Template
         */     
-       private void parseVnfcNetworkData(Map<String,String> paramsMap, GenericResourceApiVnfcnetworkdataVnfcNetworkData vnfcNetworkdata, int networkDataIdx) {
+       private void parseVnfcNetworkData(Map<String,Object> paramsMap, GenericResourceApiVnfcnetworkdataVnfcNetworkData vnfcNetworkdata, int networkDataIdx) {
                
                String vmTypeKey = vnfcNetworkdata.getVnfcType();
                GenericResourceApiVnfcnetworkdataVnfcnetworkdataVnfcPorts vnfcPorts = vnfcNetworkdata.getVnfcPorts();
@@ -588,7 +586,7 @@ public class VnfAdapterVfModuleObjectMapper {
         * Example: fw_subint_ctrl_port_0_subintcount
         * 
         */
-       private void buildVfModuleSubInterfacesCount(Map<String,String> paramsMap, String keyPrefix, GenericResourceApiSubInterfaceNetworkData vnicSubInterfaces) {
+       private void buildVfModuleSubInterfacesCount(Map<String,Object> paramsMap, String keyPrefix, GenericResourceApiSubInterfaceNetworkData vnicSubInterfaces) {
 
                List<GenericResourceApiSubinterfacenetworkdataSubInterfaceNetworkData> subInterfaceNetworkDataList = vnicSubInterfaces.getSubInterfaceNetworkData();
                
@@ -614,7 +612,7 @@ public class VnfAdapterVfModuleObjectMapper {
         * Example: fw_0_subint_ctrl_port_0_vlan_ids
         * 
         */
-       protected void buildVfModuleVlanTag(Map<String,String> paramsMap, String keyPrefix, String vnicSubInterfaces) {
+       protected void buildVfModuleVlanTag(Map<String,Object> paramsMap, String keyPrefix, String vnicSubInterfaces) {
                
                List<String> vlanTagIds = jsonPath.locateResultList(vnicSubInterfaces, "$.sub-interface-network-data[*].vlan-tag-id");
 
@@ -627,7 +625,7 @@ public class VnfAdapterVfModuleObjectMapper {
         * Example: fw_0_subint_ctrl_port_0_net_names
         * 
         */
-       protected void buildVfModuleNetworkName(Map<String,String> paramsMap, String keyPrefix, String vnicSubInterfaces) {
+       protected void buildVfModuleNetworkName(Map<String,Object> paramsMap, String keyPrefix, String vnicSubInterfaces) {
                
                List<String> neworkNames = jsonPath.locateResultList(vnicSubInterfaces, "$.sub-interface-network-data[*].network-name");
                
@@ -641,7 +639,7 @@ public class VnfAdapterVfModuleObjectMapper {
         * Example: fw_0_subint_ctrl_port_0_net_ids
         * 
         */
-       protected void buildVfModuleNetworkId(Map<String,String> paramsMap, String keyPrefix, String vnicSubInterfaces) {
+       protected void buildVfModuleNetworkId(Map<String,Object> paramsMap, String keyPrefix, String vnicSubInterfaces) {
                
                List<String> neworkIds = jsonPath.locateResultList(vnicSubInterfaces, "$.sub-interface-network-data[*].network-id");
                
@@ -654,7 +652,7 @@ public class VnfAdapterVfModuleObjectMapper {
         * {vm-type}_{index}_subint_{network-role}_port_{index}_ip_{index}  -- for ipV4
         * key = vm-type, networkRoleKey = NetWork-Role
         */
-       protected void buildVfModuleIpV4AddressHeatTemplate(Map<String,String> paramsMap, String keyPrefix, String vnicSubInterfaces) {
+       protected void buildVfModuleIpV4AddressHeatTemplate(Map<String,Object> paramsMap, String keyPrefix, String vnicSubInterfaces) {
                
                List<String> ipv4Ips = jsonPath.locateResultList(vnicSubInterfaces, "$.sub-interface-network-data[*].network-information-items.network-information-item[?(@.ip-version == 'ipv4')].network-ips.network-ip[*]");
                
@@ -672,7 +670,7 @@ public class VnfAdapterVfModuleObjectMapper {
         * {vm-type}_{index}_subint_{network-role}_port_{index}_v6_ip_{index} -- for ipV6
         * key = vm-type, networkRoleKey = NetWork-Role
         */
-       protected void buildVfModuleIpV6AddressHeatTemplate(Map<String,String> paramsMap, String keyPrefix, String vnicSubInterfaces) {
+       protected void buildVfModuleIpV6AddressHeatTemplate(Map<String,Object> paramsMap, String keyPrefix, String vnicSubInterfaces) {
                
                List<String> ipv6Ips = jsonPath.locateResultList(vnicSubInterfaces, "$.sub-interface-network-data[*].network-information-items.network-information-item[?(@.ip-version == 'ipv6')].network-ips.network-ip[*]");
 
@@ -688,7 +686,7 @@ public class VnfAdapterVfModuleObjectMapper {
         * Building Criteria : 
         * {vm-type}_subint_{network-role}_port_{index}_floating_ip  -- for ipV4
         */
-       protected void buildVfModuleFloatingIpV4HeatTemplate(Map<String,String> paramsMap, String keyPrefix, String vnicSubInterfaces) {
+       protected void buildVfModuleFloatingIpV4HeatTemplate(Map<String,Object> paramsMap, String keyPrefix, String vnicSubInterfaces) {
                
                List<String> floatingV4 = jsonPath.locateResultList(vnicSubInterfaces, "$.sub-interface-network-data[*].floating-ips.floating-ip-v4[*]");
 
@@ -704,7 +702,7 @@ public class VnfAdapterVfModuleObjectMapper {
         * Building Criteria : 
         * {vm-type}_subint_{network-role}_port_{index}_floating_v6_ip -- for ipV6
         */
-       protected void buildVfModuleFloatingIpV6HeatTemplate(Map<String,String> paramsMap, String keyPrefix, String vnicSubInterfaces) {
+       protected void buildVfModuleFloatingIpV6HeatTemplate(Map<String,Object> paramsMap, String keyPrefix, String vnicSubInterfaces) {
                                
                List<String> floatingV6 = jsonPath.locateResultList(vnicSubInterfaces, "$.sub-interface-network-data[*].floating-ips.floating-ip-v6[*]");
                
@@ -714,19 +712,19 @@ public class VnfAdapterVfModuleObjectMapper {
                addPairToMap(paramsMap, keyPrefix, FLOATING_V6_IP, floatingV6);
        }
        
-       protected void addPairToMap(Map<String, String> paramsMap, String keyPrefix, String key, String value) {
+       protected void addPairToMap(Map<String, Object> paramsMap, String keyPrefix, String key, String value) {
                
                addPairToMap(paramsMap, keyPrefix, key, Collections.singletonList(value));
        }
 
-       protected void addPairToMap(Map<String, String> paramsMap, String keyPrefix, String key, List<String> value) {
+       protected void addPairToMap(Map<String, Object> paramsMap, String keyPrefix, String key, List<String> value) {
                
                if (!value.isEmpty()) {
                        paramsMap.put(keyPrefix + key, Joiner.on(",").join(value));
                }
        }
        
-       private void buildParamsMapFromSdncParams(Map<String,String> paramsMap, GenericResourceApiParam parametersData) {               
+       private void buildParamsMapFromSdncParams(Map<String,Object> paramsMap, GenericResourceApiParam parametersData) {               
                if (parametersData != null) {
                        List<GenericResourceApiParamParam> paramsList = parametersData.getParam();
                        if (paramsList != null) {
index 2cd0947..e46c456 100644 (file)
@@ -22,16 +22,27 @@ package org.onap.so.client.sdnc.mapper;
 
 import java.net.URI;
 
-import org.onap.sdnc.northbound.client.model.*;
-import org.onap.so.bpmn.servicedecomposition.bbobjects.*;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiConfigurationinformationConfigurationInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiGcTopologyOperationInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiGcrequestinputGcRequestInput;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestActionEnumeration;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestinformationRequestInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiSdncrequestheaderSdncRequestHeader;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiServiceinformationServiceInformation;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.Configuration;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.Customer;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.GenericVnf;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.ServiceInstance;
 import org.onap.so.bpmn.servicedecomposition.generalobjects.RequestContext;
 import org.onap.so.client.sdnc.beans.SDNCSvcAction;
+import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Component;
 
 @Component(value = "sdncGCTopologyOperationRequestMapper")
 public class GCTopologyOperationRequestMapper {
 
-    private static final GeneralTopologyObjectMapper generalTopologyObjectMapper = new GeneralTopologyObjectMapper();
+       @Autowired
+    private GeneralTopologyObjectMapper generalTopologyObjectMapper;
 
     public GenericResourceApiGcTopologyOperationInformation assignOrActivateVnrReqMapper(SDNCSvcAction svcAction,
                                                                                          GenericResourceApiRequestActionEnumeration reqAction,
@@ -82,5 +93,4 @@ public class GCTopologyOperationRequestMapper {
 
     }
 
-
 }
index 7632831..fedbde2 100644 (file)
 
 package org.onap.so.client.sdnc.mapper;
 
-import org.onap.sdnc.northbound.client.model.*;
-import org.onap.so.bpmn.servicedecomposition.bbobjects.*;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+
+import org.onap.sdnc.northbound.client.model.GenericResourceApiConfigurationinformationConfigurationInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiGcrequestinputGcRequestInput;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiNetworkinformationNetworkInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiOnapmodelinformationOnapModelInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiParam;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestActionEnumeration;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestinformationRequestInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiSdncrequestheaderSdncRequestHeader;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiServiceinformationServiceInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiVfmoduleinformationVfModuleInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiVnfinformationVnfInformation;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.Configuration;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.Customer;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.GenericVnf;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.L3Network;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.ServiceInstance;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.VfModule;
 import org.onap.so.bpmn.servicedecomposition.generalobjects.RequestContext;
+import org.onap.so.client.exception.MapperException;
 import org.onap.so.client.sdnc.beans.SDNCSvcAction;
-import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.core.env.Environment;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.springframework.stereotype.Component;
-import org.onap.so.client.exception.MapperException;
+
+import com.fasterxml.jackson.core.JsonProcessingException;
+import com.fasterxml.jackson.databind.ObjectMapper;
 
 @Component
 public class GeneralTopologyObjectMapper {
        
+       private static final Logger logger = LoggerFactory.getLogger(GeneralTopologyObjectMapper.class);
+       private ObjectMapper mapper = new ObjectMapper();
 
        /*
         * Build GenericResourceApiRequestinformationRequestInformation
@@ -194,4 +218,22 @@ public class GeneralTopologyObjectMapper {
         }
            return gcRequestInput;
        }
+    
+    
+    public String mapUserParamValue(Object value) {
+       if (value == null) {
+               return null;
+       } else {
+               if (value instanceof Map || value instanceof Set || value instanceof List) {
+                       try {
+                                       return mapper.writeValueAsString(value);
+                               } catch (JsonProcessingException e) {
+                                       logger.error("could not map value to string", e);
+                                       throw new IllegalArgumentException(e);
+                               }
+               } else {
+                       return value.toString();
+               }
+       }
+    }
 }
index 2b4834a..188a228 100644 (file)
@@ -23,15 +23,6 @@ package org.onap.so.client.sdnc.mapper;
 import java.util.Map;
 import java.util.UUID;
 
-import org.onap.so.bpmn.servicedecomposition.bbobjects.CloudRegion;
-import org.onap.so.bpmn.servicedecomposition.bbobjects.Customer;
-import org.onap.so.bpmn.servicedecomposition.bbobjects.L3Network;
-import org.onap.so.bpmn.servicedecomposition.bbobjects.ServiceInstance;
-import org.onap.so.bpmn.servicedecomposition.generalobjects.RequestContext;
-import org.onap.so.client.sdnc.beans.SDNCSvcAction;
-import org.onap.so.client.sdnc.beans.SDNCSvcOperation;
-import org.springframework.stereotype.Component;
-
 import org.onap.sdnc.northbound.client.model.GenericResourceApiNetworkOperationInformation;
 import org.onap.sdnc.northbound.client.model.GenericResourceApiNetworkinformationNetworkInformation;
 import org.onap.sdnc.northbound.client.model.GenericResourceApiNetworkrequestinputNetworkRequestInput;
@@ -41,6 +32,15 @@ import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestActionEnum
 import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestinformationRequestInformation;
 import org.onap.sdnc.northbound.client.model.GenericResourceApiSdncrequestheaderSdncRequestHeader;
 import org.onap.sdnc.northbound.client.model.GenericResourceApiServiceinformationServiceInformation;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.CloudRegion;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.Customer;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.L3Network;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.ServiceInstance;
+import org.onap.so.bpmn.servicedecomposition.generalobjects.RequestContext;
+import org.onap.so.client.sdnc.beans.SDNCSvcAction;
+import org.onap.so.client.sdnc.beans.SDNCSvcOperation;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.stereotype.Component;
 
 /**
  * Mapper creating SDNC request
@@ -49,7 +49,8 @@ import org.onap.sdnc.northbound.client.model.GenericResourceApiServiceinformatio
 @Component
 public class NetworkTopologyOperationRequestMapper {
        
-       static GeneralTopologyObjectMapper generalTopologyObjectMapper = new GeneralTopologyObjectMapper();
+       @Autowired
+       private GeneralTopologyObjectMapper generalTopologyObjectMapper;
 
        public GenericResourceApiNetworkOperationInformation reqMapper(SDNCSvcOperation svcOperation,
                        SDNCSvcAction svcAction, GenericResourceApiRequestActionEnumeration reqAction, L3Network network, ServiceInstance serviceInstance,
@@ -68,11 +69,11 @@ public class NetworkTopologyOperationRequestMapper {
                req.setNetworkInformation(networkInformation);
 
                if (requestContext.getUserParams() != null) {
-                       for (Map.Entry<String, String> entry : requestContext.getUserParams().entrySet()) {
+                       for (Map.Entry<String, Object> entry : requestContext.getUserParams().entrySet()) {
                                GenericResourceApiParam networkInputParameters = new GenericResourceApiParam();
                                GenericResourceApiParamParam paramItem = new GenericResourceApiParamParam();
-                               paramItem.setName(entry.getKey()); 
-                               paramItem.setValue(entry.getValue()); 
+                               paramItem.setName(entry.getKey());
+                               paramItem.setValue(generalTopologyObjectMapper.mapUserParamValue(entry.getValue())); 
                                networkInputParameters.addParamItem(paramItem);
                                networkRequestInput.setNetworkInputParameters(networkInputParameters);
                        }
index 5b23707..505466b 100644 (file)
@@ -23,13 +23,6 @@ package org.onap.so.client.sdnc.mapper;
 import java.util.Map;
 import java.util.UUID;
 
-import org.onap.so.bpmn.servicedecomposition.bbobjects.Customer;
-import org.onap.so.bpmn.servicedecomposition.bbobjects.ServiceInstance;
-import org.onap.so.bpmn.servicedecomposition.generalobjects.RequestContext;
-import org.onap.so.client.sdnc.beans.SDNCSvcAction;
-import org.onap.so.client.sdnc.beans.SDNCSvcOperation;
-import org.springframework.stereotype.Component;
-
 import org.onap.sdnc.northbound.client.model.GenericResourceApiParam;
 import org.onap.sdnc.northbound.client.model.GenericResourceApiParamParam;
 import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestActionEnumeration;
@@ -38,11 +31,19 @@ import org.onap.sdnc.northbound.client.model.GenericResourceApiSdncrequestheader
 import org.onap.sdnc.northbound.client.model.GenericResourceApiServiceOperationInformation;
 import org.onap.sdnc.northbound.client.model.GenericResourceApiServiceinformationServiceInformation;
 import org.onap.sdnc.northbound.client.model.GenericResourceApiServicerequestinputServiceRequestInput;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.Customer;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.ServiceInstance;
+import org.onap.so.bpmn.servicedecomposition.generalobjects.RequestContext;
+import org.onap.so.client.sdnc.beans.SDNCSvcAction;
+import org.onap.so.client.sdnc.beans.SDNCSvcOperation;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.stereotype.Component;
 
 @Component
 public class ServiceTopologyOperationMapper{
 
-       static GeneralTopologyObjectMapper generalTopologyObjectMapper = new GeneralTopologyObjectMapper();
+       @Autowired
+       public GeneralTopologyObjectMapper generalTopologyObjectMapper;
        
        public GenericResourceApiServiceOperationInformation reqMapper (SDNCSvcOperation svcOperation,
                        SDNCSvcAction svcAction, GenericResourceApiRequestActionEnumeration resourceAction,ServiceInstance serviceInstance, Customer customer, RequestContext requestContext) {
@@ -62,13 +63,13 @@ public class ServiceTopologyOperationMapper{
                servOpInput.setServiceRequestInput(servReqInfo);
                
                if(requestContext.getUserParams()!=null){
-                       for (Map.Entry<String, String> entry : requestContext.getUserParams().entrySet()) {
+                       for (Map.Entry<String, Object> entry : requestContext.getUserParams().entrySet()) {
                                GenericResourceApiServicerequestinputServiceRequestInput serviceRequestInput = new GenericResourceApiServicerequestinputServiceRequestInput(); 
                                serviceRequestInput.setServiceInstanceName(serviceInstance.getServiceInstanceName()); 
                                GenericResourceApiParam serviceInputParameters = new GenericResourceApiParam(); 
                                GenericResourceApiParamParam paramItem = new GenericResourceApiParamParam(); 
                                paramItem.setName(entry.getKey()); 
-                               paramItem.setValue(entry.getValue()); 
+                               paramItem.setValue(generalTopologyObjectMapper.mapUserParamValue(entry.getValue())); 
                                serviceInputParameters.addParamItem(paramItem ); 
                                serviceRequestInput.serviceInputParameters(serviceInputParameters); 
                                servOpInput.setServiceRequestInput(serviceRequestInput ); 
index 8e0072b..f70ac83 100644 (file)
@@ -25,13 +25,25 @@ package org.onap.so.client.sdnc.mapper;
 import java.util.Map;
 import java.util.UUID;
 
+import org.onap.sdnc.northbound.client.model.GenericResourceApiParam;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiParamParam;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestActionEnumeration;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestinformationRequestInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiSdncrequestheaderSdncRequestHeader;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiServiceinformationServiceInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiSvcActionEnumeration;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiVfModuleOperationInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiVfModuleResponseInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiVfmoduleinformationVfModuleInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiVfmodulerequestinputVfModuleRequestInput;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiVnfinformationVnfInformation;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.CloudRegion;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.Customer;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.GenericVnf;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.ServiceInstance;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.VfModule;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.VolumeGroup;
 import org.onap.so.bpmn.servicedecomposition.generalobjects.RequestContext;
-import org.onap.so.bpmn.servicedecomposition.bbobjects.ServiceInstance;
 import org.onap.so.client.exception.MapperException;
 import org.onap.so.client.sdnc.beans.SDNCSvcAction;
 import org.onap.so.client.sdnc.beans.SDNCSvcOperation;
@@ -39,27 +51,17 @@ import org.onap.so.logger.MessageEnum;
 import org.onap.so.logger.MsoLogger;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
+import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Component;
 
-import org.onap.sdnc.northbound.client.model.GenericResourceApiParam;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiParamParam;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestActionEnumeration;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestinformationRequestInformation;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiSdncrequestheaderSdncRequestHeader;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiServiceinformationServiceInformation;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiSvcActionEnumeration;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiVfModuleOperationInformation;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiVnfinformationVnfInformation;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiVfmoduleinformationVfModuleInformation;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiVfmodulerequestinputVfModuleRequestInput;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiVfModuleResponseInformation;
-
 import com.fasterxml.jackson.databind.ObjectMapper;
 
 @Component
 public class VfModuleTopologyOperationRequestMapper {
        private static final Logger logger = LoggerFactory.getLogger(VfModuleTopologyOperationRequestMapper.class);
-       static GeneralTopologyObjectMapper generalTopologyObjectMapper = new GeneralTopologyObjectMapper();
+       
+       @Autowired
+       private GeneralTopologyObjectMapper generalTopologyObjectMapper;
 
        public GenericResourceApiVfModuleOperationInformation reqMapper(SDNCSvcOperation svcOperation,
                        SDNCSvcAction svcAction,  VfModule vfModule, VolumeGroup volumeGroup, GenericVnf vnf, ServiceInstance serviceInstance,
@@ -126,10 +128,10 @@ public class VfModuleTopologyOperationRequestMapper {
                GenericResourceApiParam vfModuleInputParameters = new GenericResourceApiParam();
                
                if (requestContext != null && requestContext.getUserParams() != null) {
-                       for (Map.Entry<String, String> entry : requestContext.getUserParams().entrySet()) {
+                       for (Map.Entry<String, Object> entry : requestContext.getUserParams().entrySet()) {
                                GenericResourceApiParamParam paramItem = new GenericResourceApiParamParam();
                                paramItem.setName(entry.getKey()); 
-                               paramItem.setValue(entry.getValue()); 
+                               paramItem.setValue(generalTopologyObjectMapper.mapUserParamValue(entry.getValue())); 
                                vfModuleInputParameters.addParamItem(paramItem);
                        }
                }
index e860d3c..7de393b 100644 (file)
@@ -26,6 +26,16 @@ import java.util.Map;
 import java.util.UUID;
 
 import org.apache.commons.lang.StringUtils;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiParam;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiParamParam;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestActionEnumeration;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestinformationRequestInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiSdncrequestheaderSdncRequestHeader;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiServiceinformationServiceInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiVnfOperationInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiVnfinformationVnfInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiVnfrequestinputVnfRequestInput;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiVnfrequestinputVnfrequestinputVnfNetworkInstanceGroupIds;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.CloudRegion;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.Customer;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.GenericVnf;
@@ -36,23 +46,14 @@ import org.onap.so.bpmn.servicedecomposition.generalobjects.RequestContext;
 import org.onap.so.bpmn.servicedecomposition.modelinfo.ModelInfoInstanceGroup;
 import org.onap.so.client.sdnc.beans.SDNCSvcAction;
 import org.onap.so.client.sdnc.beans.SDNCSvcOperation;
+import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Component;
 
-import org.onap.sdnc.northbound.client.model.GenericResourceApiParam;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiParamParam;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestActionEnumeration;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestinformationRequestInformation;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiSdncrequestheaderSdncRequestHeader;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiServiceinformationServiceInformation;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiVnfOperationInformation;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiVnfinformationVnfInformation;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiVnfrequestinputVnfRequestInput;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiVnfrequestinputVnfrequestinputVnfNetworkInstanceGroupIds;
-
 @Component
 public class VnfTopologyOperationRequestMapper {
 
-       static GeneralTopologyObjectMapper generalTopologyObjectMapper = new GeneralTopologyObjectMapper();
+       @Autowired
+       private GeneralTopologyObjectMapper generalTopologyObjectMapper;
 
        public GenericResourceApiVnfOperationInformation reqMapper(SDNCSvcOperation svcOperation, SDNCSvcAction svcAction,
                        GenericResourceApiRequestActionEnumeration requestAction, GenericVnf vnf, ServiceInstance serviceInstance,
@@ -84,10 +85,10 @@ public class VnfTopologyOperationRequestMapper {
 
                GenericResourceApiParam vnfInputParameters = new GenericResourceApiParam();
                if (requestContext.getUserParams() != null) {
-                       for (Map.Entry<String, String> entry : requestContext.getUserParams().entrySet()) {
+                       for (Map.Entry<String, Object> entry : requestContext.getUserParams().entrySet()) {
                                GenericResourceApiParamParam paramItem = new GenericResourceApiParamParam();
                                paramItem.setName(entry.getKey());
-                               paramItem.setValue(entry.getValue());
+                               paramItem.setValue(generalTopologyObjectMapper.mapUserParamValue(entry.getValue()));
                                vnfInputParameters.addParamItem(paramItem);
                                vnfRequestInput.setVnfInputParameters(vnfInputParameters);
                        }
index 339e9cb..5169663 100644 (file)
@@ -72,7 +72,6 @@ public class VnfAdapterObjectMapperTest {
        }
        
        @Test
-       @Ignore
        public void test_createVolumeGroupRequestMapper() throws Exception {
                RequestContext requestContext = new RequestContext();
                requestContext.setMsoRequestId("msoRequestId");
@@ -123,7 +122,7 @@ public class VnfAdapterObjectMapperTest {
                expectedCreateVolumeGroupRequest.setVfModuleType(volumeGroup.getModelInfoVfModule().getModelName());
                expectedCreateVolumeGroupRequest.setModelCustomizationUuid(volumeGroup.getModelInfoVfModule().getModelCustomizationUUID());
                
-               Map<String, String> volumeGroupParams = new HashMap<>();
+               Map<String, Object> volumeGroupParams = new HashMap<>();
                volumeGroupParams.put("vnf_id", genericVnf.getVnfId());
                volumeGroupParams.put("vnf_name", genericVnf.getVnfName());
                volumeGroupParams.put("vf_module_id", volumeGroup.getVolumeGroupId());
@@ -201,7 +200,7 @@ public class VnfAdapterObjectMapperTest {
                expectedCreateVolumeGroupRequest.setVfModuleType(volumeGroup.getModelInfoVfModule().getModelName());
                expectedCreateVolumeGroupRequest.setModelCustomizationUuid(volumeGroup.getModelInfoVfModule().getModelCustomizationUUID());
 
-               Map<String, String> volumeGroupParams = new HashMap<>();
+               Map<String, Object> volumeGroupParams = new HashMap<>();
                volumeGroupParams.put("vnf_id", genericVnf.getVnfId());
                volumeGroupParams.put("vnf_name", genericVnf.getVnfName());
                volumeGroupParams.put("vf_module_id", volumeGroup.getVolumeGroupId());
@@ -285,7 +284,6 @@ public class VnfAdapterObjectMapperTest {
        }
        
        @Test
-       @Ignore
        public void test_createVolumeGroupParams() throws Exception {
                GenericVnf genericVnf = new GenericVnf();
                genericVnf.setVnfId("vnfId");
@@ -308,7 +306,7 @@ public class VnfAdapterObjectMapperTest {
                expectedVolumeGroupParams.put("paramTwo", "paramTwoValue");
                expectedVolumeGroupParams.put("paramThree", "paramThreeValue");
                
-               Map<String, String> actualVolumeGroupParams = vnfAdapterObjectMapper.createVolumeGroupParams(requestContext,genericVnf, volumeGroup, sdncVfModuleQueryResponse);
+               Map<String, Object> actualVolumeGroupParams = vnfAdapterObjectMapper.createVolumeGroupParams(requestContext,genericVnf, volumeGroup, sdncVfModuleQueryResponse);
                
                assertEquals(expectedVolumeGroupParams, actualVolumeGroupParams);
        }
@@ -329,13 +327,12 @@ public class VnfAdapterObjectMapperTest {
                expectedVolumeGroupParams.put("vf_module_id", volumeGroup.getVolumeGroupId());
                expectedVolumeGroupParams.put("vf_module_name", volumeGroup.getVolumeGroupName());
                RequestContext requestContext = new RequestContext();
-               Map<String, String> actualVolumeGroupParams = vnfAdapterObjectMapper.createVolumeGroupParams(requestContext,genericVnf, volumeGroup, null);
+               Map<String, Object> actualVolumeGroupParams = vnfAdapterObjectMapper.createVolumeGroupParams(requestContext,genericVnf, volumeGroup, null);
 
                assertEquals(expectedVolumeGroupParams, actualVolumeGroupParams);
        }
 
     @Test
-    @Ignore
     public void test_createVolumeGroupParams_with_user_params() throws Exception {
         GenericVnf genericVnf = new GenericVnf();
         genericVnf.setVnfId("vnfId");
@@ -366,7 +363,7 @@ public class VnfAdapterObjectMapperTest {
         expectedVolumeGroupParams.put("paramThree", "paramThreeValue");
         expectedVolumeGroupParams.put("userParamKey", "userParamValue");
 
-        Map<String, String> actualVolumeGroupParams = vnfAdapterObjectMapper.createVolumeGroupParams(requestContext,genericVnf, volumeGroup, sdncVfModuleQueryResponse);
+        Map<String, Object> actualVolumeGroupParams = vnfAdapterObjectMapper.createVolumeGroupParams(requestContext,genericVnf, volumeGroup, sdncVfModuleQueryResponse);
 
         assertEquals(expectedVolumeGroupParams, actualVolumeGroupParams);
     }
index a156c38..00e5c10 100644 (file)
@@ -76,7 +76,7 @@ public class VnfAdapterVfModuleObjectMapperPayloadTest {
                serviceInstance.setModelInfoServiceInstance(modelInfoServiceInstance);
 
                RequestContext requestContext = new RequestContext();
-               HashMap<String, String> userParams = new HashMap<String, String>();
+               Map<String, Object> userParams = new HashMap<>();
                userParams.put("key1", "value2");
                requestContext.setMsoRequestId("requestId");
                requestContext.setUserParams(userParams);
@@ -150,7 +150,7 @@ public class VnfAdapterVfModuleObjectMapperPayloadTest {
                serviceInstance.setModelInfoServiceInstance(modelInfoServiceInstance);
 
                RequestContext requestContext = new RequestContext();
-               HashMap<String, String> userParams = new HashMap<String, String>();
+               Map<String, Object> userParams = new HashMap<>();
                userParams.put("key1", "value2");
                requestContext.setMsoRequestId("requestId");
                requestContext.setUserParams(userParams);
@@ -223,7 +223,7 @@ public class VnfAdapterVfModuleObjectMapperPayloadTest {
                serviceInstance.setModelInfoServiceInstance(modelInfoServiceInstance);
                
                RequestContext requestContext = new RequestContext();
-               HashMap<String, String> userParams = new HashMap<String, String>();
+               Map<String, Object> userParams = new HashMap<>();
                userParams.put("key1", "value2");
                requestContext.setMsoRequestId("requestId");
                requestContext.setUserParams(userParams);
@@ -295,7 +295,7 @@ public class VnfAdapterVfModuleObjectMapperPayloadTest {
 
                //
                RequestContext requestContext = new RequestContext();
-               HashMap<String, String> userParams = new HashMap<String, String>();
+               Map<String, Object> userParams = new HashMap<>();
                userParams.put("key1", "value2");
                requestContext.setMsoRequestId("requestId");
                requestContext.setUserParams(userParams);
@@ -375,7 +375,7 @@ public class VnfAdapterVfModuleObjectMapperPayloadTest {
 
                //
                RequestContext requestContext = new RequestContext();
-               HashMap<String, String> userParams = new HashMap<String, String>();
+               Map<String, Object> userParams = new HashMap<>();
                userParams.put("key1", "value2");
                requestContext.setMsoRequestId("requestId");
                requestContext.setUserParams(userParams);
@@ -450,7 +450,7 @@ public class VnfAdapterVfModuleObjectMapperPayloadTest {
                serviceInstance.setModelInfoServiceInstance(modelInfoServiceInstance);
 
                RequestContext requestContext = new RequestContext();
-               HashMap<String, String> userParams = new HashMap<String, String>();
+               Map<String, Object> userParams = new HashMap<>();
                userParams.put("key1", "value2");
                requestContext.setMsoRequestId("requestId");
                requestContext.setUserParams(userParams);
@@ -521,7 +521,7 @@ public class VnfAdapterVfModuleObjectMapperPayloadTest {
                serviceInstance.setModelInfoServiceInstance(modelInfoServiceInstance);
 
                RequestContext requestContext = new RequestContext();
-               HashMap<String, String> userParams = new HashMap<String, String>();
+               Map<String, Object> userParams = new HashMap<>();
                userParams.put("key1", "value1");
                requestContext.setMsoRequestId("requestId");
                requestContext.setUserParams(userParams);
@@ -591,7 +591,7 @@ public class VnfAdapterVfModuleObjectMapperPayloadTest {
                serviceInstance.setModelInfoServiceInstance(modelInfoServiceInstance);
 
                RequestContext requestContext = new RequestContext();
-               HashMap<String, String> userParams = new HashMap<String, String>();
+               Map<String, Object> userParams = new HashMap<>();
                userParams.put("key1", "value1");
                requestContext.setMsoRequestId("requestId");
                requestContext.setUserParams(userParams);
@@ -661,7 +661,7 @@ public class VnfAdapterVfModuleObjectMapperPayloadTest {
                serviceInstance.setModelInfoServiceInstance(modelInfoServiceInstance);
 
                RequestContext requestContext = new RequestContext();
-               HashMap<String, String> userParams = new HashMap<String, String>();
+               Map<String, Object> userParams = new HashMap<>();
                userParams.put("key1", "value1");
                requestContext.setMsoRequestId("requestId");
                requestContext.setUserParams(userParams);
@@ -798,7 +798,7 @@ public class VnfAdapterVfModuleObjectMapperPayloadTest {
                serviceInstance.setModelInfoServiceInstance(modelInfoServiceInstance);
                
                RequestContext requestContext = new RequestContext();
-               HashMap<String, String> userParams = new HashMap<String, String>();
+               Map<String, Object> userParams = new HashMap<>();
                userParams.put("key1", "value1");
                requestContext.setMsoRequestId("requestId");
                requestContext.setUserParams(userParams);
@@ -853,7 +853,7 @@ public class VnfAdapterVfModuleObjectMapperPayloadTest {
                
                String json = new String(Files.readAllBytes(Paths.get(JSON_FILE_LOCATION + "grApiVmNetworkSubSectionWith5GParams.json")));
                GenericResourceApiVmNetworkData network = omapper.readValue(json, GenericResourceApiVmNetworkData.class);
-               Map<String, String> paramsMap = new HashMap<>();
+               Map<String, Object> paramsMap = new HashMap<>();
                vfModuleObjectMapper.buildVlanInformation(paramsMap, network, "testKey", "testType");
                
                assertEquals("1,3", paramsMap.get("testKey_testType_private_vlans"));
index 160feed..81d30e3 100644 (file)
@@ -67,7 +67,7 @@ public class VnfAdapterVfModuleObjectMapperTest {
        
        @Test
        public void addPairToMapTest() {
-               Map<String, String> map = new HashMap<>();
+               Map<String, Object> map = new HashMap<>();
                
                mapper.addPairToMap(map, "test", "_key", Arrays.asList("a", "b"));
                
index 4d90a3f..0023066 100644 (file)
@@ -22,9 +22,7 @@ package org.onap.so.client.orchestration;
 
 import static org.junit.Assert.assertNotNull;
 import static org.mockito.ArgumentMatchers.any;
-import static org.mockito.ArgumentMatchers.isA;
-import static org.mockito.Mockito.doReturn;
-import static org.mockito.Mockito.times;
+import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.Mockito.verify;
 
 import java.net.URI;
@@ -35,10 +33,11 @@ import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
-import org.mockito.Mockito;
-import org.mockito.Spy;
 import org.mockito.junit.MockitoJUnitRunner;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiGcTopologyOperationInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestActionEnumeration;
 import org.onap.so.bpmn.common.data.TestDataSetup;
+import org.onap.so.bpmn.servicedecomposition.bbobjects.Configuration;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.Customer;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.GenericVnf;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.ServiceInstance;
@@ -47,19 +46,17 @@ import org.onap.so.bpmn.servicedecomposition.generalobjects.RequestContext;
 import org.onap.so.client.exception.BadResponseException;
 import org.onap.so.client.exception.MapperException;
 import org.onap.so.client.sdnc.SDNCClient;
-import org.onap.so.client.sdnc.endpoint.SDNCTopology;
+import org.onap.so.client.sdnc.beans.SDNCSvcAction;
 import org.onap.so.client.sdnc.mapper.GCTopologyOperationRequestMapper;
 
-import org.onap.sdnc.northbound.client.model.GenericResourceApiGcTopologyOperationInformation;
-
 @RunWith(MockitoJUnitRunner.Silent.class)
 public class SDNCConfigurationResourcesTest extends TestDataSetup{
        
        @InjectMocks
     private SDNCConfigurationResources sdncConfigurationResources = new SDNCConfigurationResources();
        
-       @Spy
-       GCTopologyOperationRequestMapper MOCK_gcTopologyMapper ;
+       @Mock
+       private GCTopologyOperationRequestMapper MOCK_gcTopologyMapper ;
        
        @Mock
        protected SDNCClient MOCK_sdncClient;
@@ -82,24 +79,34 @@ public class SDNCConfigurationResourcesTest extends TestDataSetup{
     @Test
     public void activateVnrConfigurationTest() throws BadResponseException, MapperException, URISyntaxException {
         GenericResourceApiGcTopologyOperationInformation response = sdncConfigurationResources.activateVnrConfiguration(serviceInstance,requestContext,customer,vpnBondingLink.getVnrConfiguration(),vnf,"uuid",new URI("http://localhost"));       
-        assertNotNull(response);
+        verify(MOCK_gcTopologyMapper).assignOrActivateVnrReqMapper(
+                       eq(SDNCSvcAction.ACTIVATE), eq(GenericResourceApiRequestActionEnumeration.CREATEGENERICCONFIGURATIONINSTANCE),
+                       eq(serviceInstance), eq(requestContext), eq(customer), any(Configuration.class), any(GenericVnf.class), any(String.class), any(URI.class));
+
     }
 
     @Test
     public void assignVnrConfigurationTest() throws BadResponseException, MapperException, URISyntaxException {
         GenericResourceApiGcTopologyOperationInformation response = sdncConfigurationResources.assignVnrConfiguration(serviceInstance,requestContext,customer,vpnBondingLink.getVnrConfiguration(),vnf,"uuid",new URI("http://localhost"));    
-        assertNotNull(response);
+        verify(MOCK_gcTopologyMapper).assignOrActivateVnrReqMapper(
+                       eq(SDNCSvcAction.ASSIGN), eq(GenericResourceApiRequestActionEnumeration.CREATEGENERICCONFIGURATIONINSTANCE),
+                       eq(serviceInstance), eq(requestContext), eq(customer), any(Configuration.class), any(GenericVnf.class), any(String.class), any(URI.class));
+
     }
 
     @Test
     public void unAssignVnrConfigurationTest() throws BadResponseException, MapperException , URISyntaxException{
         GenericResourceApiGcTopologyOperationInformation response = sdncConfigurationResources.unAssignVnrConfiguration(serviceInstance,requestContext,vpnBondingLink.getVnrConfiguration(),"uuid",new URI("http://localhost"));
-        assertNotNull(response);
+        verify(MOCK_gcTopologyMapper).deactivateOrUnassignVnrReqMapper(
+                       eq(SDNCSvcAction.UNASSIGN), eq(serviceInstance), eq(requestContext), any(Configuration.class), any(String.class), any(URI.class));
+
     }
 
     @Test
     public void deactivateVnrConfigurationTest() throws BadResponseException, MapperException , URISyntaxException{
         GenericResourceApiGcTopologyOperationInformation response = sdncConfigurationResources.deactivateVnrConfiguration(serviceInstance,requestContext,vpnBondingLink.getVnrConfiguration(),"uuid",new URI("http://localhost"));      
-        assertNotNull(response);
+        verify(MOCK_gcTopologyMapper).deactivateOrUnassignVnrReqMapper(
+                       eq(SDNCSvcAction.DEACTIVATE), eq(serviceInstance), eq(requestContext), any(Configuration.class), any(String.class), any(URI.class));
+
     }
 }
index f01eeae..f4d442b 100644 (file)
@@ -24,9 +24,15 @@ import java.net.URI;
 import java.net.URISyntaxException;
 import java.util.HashMap;
 import java.util.List;
+import java.util.Map;
 
 import org.junit.Assert;
 import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.InjectMocks;
+import org.mockito.Spy;
+import org.mockito.junit.MockitoJUnitRunner;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiGcTopologyOperationInformation;
 import org.onap.so.bpmn.common.data.TestDataSetup;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.Configuration;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.GenericVnf;
@@ -38,11 +44,14 @@ import org.onap.so.bpmn.servicedecomposition.bbobjects.VpnBondingLink;
 import org.onap.so.bpmn.servicedecomposition.generalobjects.RequestContext;
 import org.onap.so.client.sdnc.beans.SDNCSvcAction;
 
-import org.onap.sdnc.northbound.client.model.GenericResourceApiGcTopologyOperationInformation;
-
-
-public class GCTopologyOperationRequestMapperTest extends TestDataSetup{
+@RunWith(MockitoJUnitRunner.Silent.class)
+public class GCTopologyOperationRequestMapperTest extends TestDataSetup {
 
+       
+       @Spy
+       private GeneralTopologyObjectMapper generalTopologyObjectMapper;
+       
+       @InjectMocks
     private GCTopologyOperationRequestMapper genObjMapper = new GCTopologyOperationRequestMapper();
 
     @Test
@@ -80,13 +89,13 @@ public class GCTopologyOperationRequestMapperTest extends TestDataSetup{
     private RequestContext getRequestContext() {
         RequestContext requestContext = new RequestContext();
         requestContext.setMsoRequestId("MsoRequestId");
-        HashMap<String, String> userParams = getUserParams();
+        Map<String, Object> userParams = getUserParams();
         requestContext.setUserParams(userParams);
         return requestContext;
     }
 
-    private HashMap<String, String> getUserParams() {
-        HashMap<String,String> userParams = new HashMap<>();
+    private Map<String, Object> getUserParams() {
+        Map<String,Object> userParams = new HashMap<>();
         userParams.put("lppCustomerId","lppCustomerId");
         return userParams;
     }
index bf2cd34..3bb5427 100644 (file)
@@ -29,6 +29,7 @@ import static org.junit.Assert.assertThat;
 import static org.junit.Assert.assertTrue;
 
 import java.util.HashMap;
+import java.util.Map;
 
 import org.junit.After;
 import org.junit.Before;
@@ -104,7 +105,7 @@ public class GeneralTopologyObjectMapperTest  extends TestDataSetup {
                customer.getServiceSubscription().getServiceInstances().add(serviceInstance);
                //
                RequestContext requestContext = new RequestContext();
-               HashMap<String, String> userParams = new HashMap<String, String>();
+               Map<String, Object> userParams = new HashMap<>();
                userParams.put("key1", "value1");
                requestContext.setUserParams(userParams);
                requestContext.setProductFamilyId("productFamilyId");
index 307fe28..6a14f8b 100644 (file)
@@ -29,9 +29,16 @@ import java.io.IOException;
 import java.nio.file.Files;
 import java.nio.file.Paths;
 import java.util.HashMap;
+import java.util.Map;
 
 import org.junit.Before;
 import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.InjectMocks;
+import org.mockito.Spy;
+import org.mockito.junit.MockitoJUnitRunner;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiNetworkOperationInformation;
+import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestActionEnumeration;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.CloudRegion;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.Collection;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.Customer;
@@ -45,10 +52,9 @@ import org.onap.so.bpmn.servicedecomposition.modelinfo.ModelInfoServiceInstance;
 import org.onap.so.client.sdnc.beans.SDNCSvcAction;
 import org.onap.so.client.sdnc.beans.SDNCSvcOperation;
 
-import org.onap.sdnc.northbound.client.model.GenericResourceApiNetworkOperationInformation;
-import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestActionEnumeration;
 import com.fasterxml.jackson.databind.ObjectMapper;
 
+@RunWith(MockitoJUnitRunner.class)
 public class NetworkTopologyOperationRequestMapperTest {
 
        private final static String JSON_FILE_LOCATION = "src/test/resources/__files/BuildingBlocks/";
@@ -59,6 +65,12 @@ public class NetworkTopologyOperationRequestMapperTest {
        private RequestContext requestContext;
        private L3Network network;
        private CloudRegion cloudRegion;
+       
+       @Spy
+       private GeneralTopologyObjectMapper generalTopologyObjectMapper;
+       
+       @InjectMocks
+       private NetworkTopologyOperationRequestMapper mapper = new NetworkTopologyOperationRequestMapper();
 
        @Before
        public void before() {
@@ -93,7 +105,7 @@ public class NetworkTopologyOperationRequestMapperTest {
                serviceInstance.setCollection(networkCollection);
                //
                requestContext = new RequestContext();
-               HashMap<String, String> userParams = new HashMap<String, String>();
+               Map<String, Object> userParams = new HashMap<>();
                userParams.put("key1", "value1");
                requestContext.setUserParams(userParams);
                requestContext.setProductFamilyId("productFamilyId");
@@ -115,7 +127,6 @@ public class NetworkTopologyOperationRequestMapperTest {
        @Test
        public void createGenericResourceApiNetworkOperationInformationTest() throws Exception {
 
-               NetworkTopologyOperationRequestMapper mapper = new NetworkTopologyOperationRequestMapper();
                GenericResourceApiNetworkOperationInformation networkSDNCrequest = mapper.reqMapper(
                                SDNCSvcOperation.NETWORK_TOPOLOGY_OPERATION, SDNCSvcAction.ASSIGN, GenericResourceApiRequestActionEnumeration.CREATENETWORKINSTANCE, network, serviceInstance, customer,
                                requestContext, cloudRegion);
@@ -132,7 +143,6 @@ public class NetworkTopologyOperationRequestMapperTest {
        @Test
        public void reqMapperTest() throws Exception {
 
-               NetworkTopologyOperationRequestMapper mapper = new NetworkTopologyOperationRequestMapper();
                GenericResourceApiNetworkOperationInformation networkSDNCrequest = mapper.reqMapper(
                                SDNCSvcOperation.NETWORK_TOPOLOGY_OPERATION, SDNCSvcAction.ASSIGN, GenericResourceApiRequestActionEnumeration.CREATENETWORKINSTANCE, network, serviceInstance, customer,
                                requestContext, cloudRegion);
@@ -143,7 +153,6 @@ public class NetworkTopologyOperationRequestMapperTest {
 
        @Test
        public void reqMapperNoCollectionTest() throws Exception {
-               NetworkTopologyOperationRequestMapper mapper = new NetworkTopologyOperationRequestMapper();
                GenericResourceApiNetworkOperationInformation networkSDNCrequest = mapper.reqMapper(
                                SDNCSvcOperation.NETWORK_TOPOLOGY_OPERATION, SDNCSvcAction.ASSIGN, GenericResourceApiRequestActionEnumeration.CREATENETWORKINSTANCE, network, serviceInstanceNoCollection, customer,
                                requestContext, cloudRegion);
@@ -154,7 +163,7 @@ public class NetworkTopologyOperationRequestMapperTest {
        @Test
        public void createGenericResourceApiNetworkOperationInformation_UnassignTest() throws Exception {
 
-               NetworkTopologyOperationRequestMapper mapperUnassign = new NetworkTopologyOperationRequestMapper();
+               NetworkTopologyOperationRequestMapper mapperUnassign = mapper;
                GenericResourceApiNetworkOperationInformation networkSDNCrequestUnassign = mapperUnassign.reqMapper(
                                SDNCSvcOperation.NETWORK_TOPOLOGY_OPERATION, SDNCSvcAction.UNASSIGN, GenericResourceApiRequestActionEnumeration.DELETENETWORKINSTANCE, network, serviceInstance, customer,
                                requestContext, cloudRegion);
@@ -172,7 +181,6 @@ public class NetworkTopologyOperationRequestMapperTest {
        @Test
        public void createGenericResourceApiNetworkOperationInformationNoNetworkNameTest() throws Exception {
 
-               NetworkTopologyOperationRequestMapper mapper = new NetworkTopologyOperationRequestMapper();
                //set network name NULL
                network.setNetworkName(null);
                GenericResourceApiNetworkOperationInformation networkSDNCrequest = mapper.reqMapper(
index 5d8b31b..0bf06a0 100644 (file)
@@ -26,8 +26,13 @@ import static com.shazam.shazamcrest.matcher.Matchers.sameBeanAs;
 import java.nio.file.Files;
 import java.nio.file.Paths;
 import java.util.HashMap;
+import java.util.Map;
 
 import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.InjectMocks;
+import org.mockito.Spy;
+import org.mockito.junit.MockitoJUnitRunner;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.Customer;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.ServiceInstance;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.ServiceSubscription;
@@ -41,8 +46,15 @@ import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestActionEnum
 import org.onap.sdnc.northbound.client.model.GenericResourceApiServiceOperationInformation;
 import com.fasterxml.jackson.databind.ObjectMapper;
 
+@RunWith(MockitoJUnitRunner.class)
 public class ServiceTopologyOperationMapperTest {
 
+       @Spy
+       private GeneralTopologyObjectMapper generalTopologyObjectMapper;
+       
+       @InjectMocks
+       private ServiceTopologyOperationMapper mapper = new ServiceTopologyOperationMapper();
+
        @Test
        public void reqMapperTest() throws Exception {
                // prepare and set service instance
@@ -64,12 +76,11 @@ public class ServiceTopologyOperationMapperTest {
 
                //prepare RequestContext
                RequestContext requestContext = new RequestContext();
-               HashMap<String, String> userParams = new HashMap<String, String>();
+               Map<String, Object> userParams = new HashMap<>();
                userParams.put("key1", "value1");
                requestContext.setUserParams(userParams);
                requestContext.setProductFamilyId("productFamilyId");
 
-               ServiceTopologyOperationMapper mapper = new ServiceTopologyOperationMapper();
                GenericResourceApiServiceOperationInformation serviceOpInformation = mapper.reqMapper(
                                SDNCSvcOperation.SERVICE_TOPOLOGY_OPERATION, SDNCSvcAction.ASSIGN, GenericResourceApiRequestActionEnumeration.CREATESERVICEINSTANCE, serviceInstance, customer,
                                requestContext);
index 7d5aa9d..282f23c 100644 (file)
@@ -35,6 +35,10 @@ import java.util.Map;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.rules.ExpectedException;
+import org.junit.runner.RunWith;
+import org.mockito.InjectMocks;
+import org.mockito.Spy;
+import org.mockito.junit.MockitoJUnitRunner;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.CloudRegion;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.Customer;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.GenericVnf;
@@ -54,11 +58,18 @@ import org.onap.so.client.sdnc.beans.SDNCSvcOperation;
 import org.onap.sdnc.northbound.client.model.GenericResourceApiVfModuleOperationInformation;
 import com.fasterxml.jackson.databind.ObjectMapper;
 
+@RunWith(MockitoJUnitRunner.class)
 public class VfModuleTopologyOperationRequestMapperTest {
 
        private final static String JSON_FILE_LOCATION = "src/test/resources/__files/BuildingBlocks/";
        private final static String ERRORMESSAGE = "VF Module model info is null for testVfModuleId";
        
+       @Spy
+       private GeneralTopologyObjectMapper generalTopologyObjectMapper;
+       
+       @InjectMocks
+       private VfModuleTopologyOperationRequestMapper mapper = new VfModuleTopologyOperationRequestMapper();
+
        @Rule
        public ExpectedException expectedException = ExpectedException.none();
 
@@ -84,7 +95,7 @@ public class VfModuleTopologyOperationRequestMapperTest {
                customer.getServiceSubscription().getServiceInstances().add(serviceInstance);
                //
                RequestContext requestContext = new RequestContext();
-               HashMap<String,String> userParams = new HashMap<String,String>();
+               Map<String,Object> userParams = new HashMap<>();
                userParams.put("key1", "value1");               
                requestContext.setUserParams(userParams);
                requestContext.setProductFamilyId("productFamilyId");
@@ -124,7 +135,6 @@ public class VfModuleTopologyOperationRequestMapperTest {
 
                CloudRegion cloudRegion = new CloudRegion();
 
-               VfModuleTopologyOperationRequestMapper mapper = new VfModuleTopologyOperationRequestMapper();
                GenericResourceApiVfModuleOperationInformation vfModuleSDNCrequest = mapper.reqMapper(
                                SDNCSvcOperation.VF_MODULE_TOPOLOGY_OPERATION, SDNCSvcAction.ASSIGN, vfModule, volumeGroup, vnf, serviceInstance, customer,
                                cloudRegion, requestContext, null);
@@ -159,7 +169,6 @@ public class VfModuleTopologyOperationRequestMapperTest {
                vfModule.setVfModuleId("testVfModuleId");
                vfModule.setVfModuleName("testVfModuleName");
 
-               VfModuleTopologyOperationRequestMapper mapper = new VfModuleTopologyOperationRequestMapper();
                GenericResourceApiVfModuleOperationInformation vfModuleSDNCrequest = mapper.reqMapper(
                                SDNCSvcOperation.VF_MODULE_TOPOLOGY_OPERATION, SDNCSvcAction.UNASSIGN, vfModule, null, vnf, serviceInstance, null,
                                null, null, null);
@@ -197,9 +206,9 @@ public class VfModuleTopologyOperationRequestMapperTest {
                //
                RequestContext requestContext = new RequestContext();
                RequestParameters requestParameters = new RequestParameters();
-               HashMap<String,Object> userParams1 = new HashMap<String,Object>();
+               HashMap<String,Object> userParams1 = new HashMap<>();
                userParams1.put("key1", "value1");
-               List<Map<String,Object>> userParams = new ArrayList<Map<String,Object>>();
+               List<Map<String,Object>> userParams = new ArrayList<>();
                userParams.add(userParams1);
                
                requestParameters.setUserParams(userParams);
@@ -230,7 +239,6 @@ public class VfModuleTopologyOperationRequestMapperTest {
 
                CloudRegion cloudRegion = new CloudRegion();
 
-               VfModuleTopologyOperationRequestMapper mapper = new VfModuleTopologyOperationRequestMapper();
                GenericResourceApiVfModuleOperationInformation vfModuleSDNCrequest = mapper.reqMapper(
                                SDNCSvcOperation.VF_MODULE_TOPOLOGY_OPERATION, SDNCSvcAction.ASSIGN, vfModule, null, vnf, serviceInstance, customer,
                                cloudRegion, requestContext, null);
@@ -261,7 +269,7 @@ public class VfModuleTopologyOperationRequestMapperTest {
                customer.getServiceSubscription().getServiceInstances().add(serviceInstance);
                //
                RequestContext requestContext = new RequestContext();
-               HashMap<String, String> userParams = new HashMap<String, String>();
+               Map<String, Object> userParams = new HashMap<String, Object>();
                userParams.put("key1", "value1");
                requestContext.setUserParams(userParams);
                requestContext.setProductFamilyId("productFamilyId");
@@ -284,7 +292,6 @@ public class VfModuleTopologyOperationRequestMapperTest {
 
                CloudRegion cloudRegion = new CloudRegion();
                
-               VfModuleTopologyOperationRequestMapper mapper = new VfModuleTopologyOperationRequestMapper();
                expectedException.expect(MapperException.class);
                expectedException.expectMessage(ERRORMESSAGE);
                
index 63c3680..229a8cf 100644 (file)
@@ -25,9 +25,14 @@ import static org.junit.Assert.assertNull;
 
 import java.util.ArrayList;
 import java.util.HashMap;
+import java.util.Map;
 import java.util.List;
 
 import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.InjectMocks;
+import org.mockito.Spy;
+import org.mockito.junit.MockitoJUnitRunner;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.CloudRegion;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.Customer;
 import org.onap.so.bpmn.servicedecomposition.bbobjects.GenericVnf;
@@ -45,8 +50,15 @@ import org.onap.so.client.sdnc.beans.SDNCSvcOperation;
 import org.onap.sdnc.northbound.client.model.GenericResourceApiRequestActionEnumeration;
 import org.onap.sdnc.northbound.client.model.GenericResourceApiVnfOperationInformation;
 
+@RunWith(MockitoJUnitRunner.class)
 public class VnfTopologyOperationRequestMapperTest {
 
+       @Spy
+       private GeneralTopologyObjectMapper generalTopologyObjectMapper;
+       
+       @InjectMocks
+       private VnfTopologyOperationRequestMapper mapper = new VnfTopologyOperationRequestMapper();
+
        @Test
        public void reqMapperTest() throws Exception {
                // prepare and set service instance
@@ -106,14 +118,13 @@ public class VnfTopologyOperationRequestMapperTest {
 
                //prepare RequestContext
                RequestContext requestContext = new RequestContext();
-               HashMap<String, String> userParams = new HashMap<String, String>();
+               Map<String, Object> userParams = new HashMap<>();
                userParams.put("key1", "value1");
                requestContext.setUserParams(userParams);
                requestContext.setProductFamilyId("productFamilyId");
 
                CloudRegion cloudRegion = new CloudRegion();
 
-               VnfTopologyOperationRequestMapper mapper = new VnfTopologyOperationRequestMapper();
                GenericResourceApiVnfOperationInformation vnfOpInformation = mapper.reqMapper(
                                SDNCSvcOperation.VNF_TOPOLOGY_OPERATION, SDNCSvcAction.ASSIGN,GenericResourceApiRequestActionEnumeration.CREATEVNFINSTANCE, vnf, serviceInstance, customer,
                                cloudRegion, requestContext,true);