Enhance naming service support

correct sourcing for service-model-name in NamingMS
remove zone ID from NamingMS input for service name
follow up removal of autowired on NamingRequestObject

Change-Id: Icba2e32c07ee0c7d7739ce21bd547e3030c1e271
Issue-ID: SO-1976
Signed-off-by: Benjamin, Max (mb388a) <mb388a@us.att.com>

3 files changed, 13 insertions, 37 deletions

diff --git a/bpmn/so-bpmn-tasks/src/test/java/org/onap/so/bpmn/infrastructure/namingservice/tasks/NamingServiceCreateTasksTest.java b/bpmn/so-bpmn-tasks/src/test/java/org/onap/so/bpmn/infrastructure/namingservice/tasks/NamingServiceCreateTasksTest.java
index 3f62a61518..44e5e0e707 100644
--- a/bpmn/so-bpmn-tasks/src/test/java/org/onap/so/bpmn/infrastructure/namingservice/tasks/NamingServiceCreateTasksTest.java
+++ b/bpmn/so-bpmn-tasks/src/test/java/org/onap/so/bpmn/infrastructure/namingservice/tasks/NamingServiceCreateTasksTest.java
@@ -106,7 +106,7 @@ public class NamingServiceCreateTasksTest extends BaseTaskTest {
         String generatedName = "generatedWanTransportServiceName";
         doReturn(generatedName).when(namingServiceResources).generateServiceInstanceName(any());
         NamingServiceCreateTasks spy = Mockito.spy(namingServiceCreateTasks);
-        doReturn("testZoneId").when(spy).getRelatedZoneId(any());
+
         spy.createWanTransportServiceName(execution);
         verify(namingServiceResources, times(1)).generateServiceInstanceName(any());
         assertEquals(generatedName, serviceInstance.getServiceInstanceName());
@@ -124,37 +124,4 @@ public class NamingServiceCreateTasksTest extends BaseTaskTest {
         verify(namingServiceResources, times(1)).generateServiceInstanceName(any());
         assertEquals(generatedName, serviceInstance.getServiceInstanceName());
     }
-
-    @Test
-    public void getRelatedZoneIdTest() throws Exception {
-
-        String zoneId = "zoneId";
-        Zone zone = new Zone();
-        zone.setZoneId(zoneId);
-
-        CloudRegion cloudRegion = new CloudRegion();
-        cloudRegion.setCloudOwner("testCloudOwner");
-        cloudRegion.setLcpCloudRegionId("testLcpCloudRegionId");
-        GeneralBuildingBlock gBBInput = execution.getGeneralBuildingBlock();
-        gBBInput.setCloudRegion(cloudRegion);
-
-        AAIResultWrapper cloudRegionWrapper =
-                new AAIResultWrapper(new AAICommonObjectMapperProvider().getMapper().writeValueAsString(cloudRegion));
-
-        Relationships relationships = Mockito.mock(Relationships.class);
-        relationships.getAll().add(cloudRegionWrapper);
-        Optional<Relationships> relationshipsOp = Optional.of(relationships);
-
-        doReturn(cloudRegionWrapper).when(MOCK_bbInputSetupUtils).getAAIResourceDepthOne(any());
-
-        NamingServiceCreateTasks spy = Mockito.spy(namingServiceCreateTasks);
-
-        doReturn(relationshipsOp).when(spy).getRelationshipsFromWrapper(isA(AAIResultWrapper.class));
-        doReturn(zone).when(spy).getRelatedZone(any());
-        spy.setBbInputSetupUtils(MOCK_bbInputSetupUtils);
-
-        String returnedZoneId = spy.getRelatedZoneId(execution);
-
-        assertEquals(zoneId, returnedZoneId);
-    }
 }
diff --git a/bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/namingservice/NamingRequestObjectTest.java b/bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/namingservice/NamingRequestObjectTest.java
index 49f858d2ce..7c50b2031d 100644
--- a/bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/namingservice/NamingRequestObjectTest.java
+++ b/bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/namingservice/NamingRequestObjectTest.java
@@ -35,7 +35,6 @@ public class NamingRequestObjectTest {
     private String resourceName = "resourceName";
     private String serviceModelName = "serviceModelName";
     private String modelVersion = "modelVersion";
-    private String zoneId = "zoneId";
 
     @Test
     public void namingRequestObjectTest() {
@@ -68,7 +67,6 @@ public class NamingRequestObjectTest {
         expected.put(NamingServiceConstants.NS_RESOURCE_NAME, resourceName);
         expected.put(NamingServiceConstants.NS_SERVICE_MODEL_NAME, serviceModelName);
         expected.put(NamingServiceConstants.NS_MODEL_VERSION, modelVersion);
-        expected.put(NamingServiceConstants.NS_ZONE_ID, zoneId);
 
         NamingRequestObject namingRequestObject = new NamingRequestObject();
         namingRequestObject.setExternalKeyValue(externalKey);
@@ -77,7 +75,6 @@ public class NamingRequestObjectTest {
         namingRequestObject.setResourceNameValue(resourceName);
         namingRequestObject.setModelVersionValue(modelVersion);
         namingRequestObject.setServiceModelNameValue(serviceModelName);
-        namingRequestObject.setZoneIdValue(zoneId);
 
         HashMap<String, String> actual = namingRequestObject.getNamingRequestObjectMap();
         assertThat(actual, sameBeanAs(expected));
diff --git a/bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/orchestration/NamingServiceResourcesTest.java b/bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/orchestration/NamingServiceResourcesTest.java
index 1f5b42b94e..c60e749a29 100644
--- a/bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/orchestration/NamingServiceResourcesTest.java
+++ b/bpmn/so-bpmn-tasks/src/test/java/org/onap/so/client/orchestration/NamingServiceResourcesTest.java
@@ -123,4 +123,16 @@ public class NamingServiceResourcesTest extends TestDataSetup {
         verify(MOCK_namingClient, times(1)).postNameGenRequest(any(NameGenRequest.class));
         assertEquals("generatedServiceInstanceName", generatedName);
     }
+
+    @Test
+    public void deleteServiceInstanceNameTest() throws Exception {
+        NameGenDeleteRequest req = new NameGenDeleteRequest();
+        doReturn(req).when(MOCK_namingRequestObjectBuilder).nameGenDeleteRequestMapper(isA(List.class));
+
+        NamingRequestObject nrObject = new NamingRequestObject();
+        nrObject.setExternalKeyValue("testExternalKey");
+        nrObject.setNamingTypeValue(NamingServiceConstants.NAMING_TYPE_SERVICE);
+        namingServiceResources.deleteServiceInstanceName(nrObject);
+        verify(MOCK_namingClient, times(1)).deleteNameGenRequest(any(NameGenDeleteRequest.class));
+    }
 }