PombaReqest and ServiceInstance improvements

- fix encapsulation issues
- make object immutable

Change-Id: I7f66aa7ec5e0ba63a9178ad27c8a9466f04e0869
Issue-ID: VID-422
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>

3 files changed, 31 insertions, 14 deletions

diff --git a/vid-app-common/src/test/java/org/onap/vid/aai/PombaClientImplTest.java b/vid-app-common/src/test/java/org/onap/vid/aai/PombaClientImplTest.java
index b1c4139b7..4e832d5f7 100644
--- a/vid-app-common/src/test/java/org/onap/vid/aai/PombaClientImplTest.java
+++ b/vid-app-common/src/test/java/org/onap/vid/aai/PombaClientImplTest.java
@@ -2,9 +2,7 @@
  * ============LICENSE_START=======================================================
  * VID
  * ================================================================================
- * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved.
- * ================================================================================
- * Modifications Copyright 2019 Nokia
+ * Copyright (C) 2019 Nokia
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -74,19 +72,18 @@ public class PombaClientImplTest {
         ServiceInstance serviceInstance2 = createServiceInstance("serviceType2", "serviceInstanceId2", "customerId2",
             "modelVersion2", "modelInvariantId2");
 
-        PombaRequest pombaRequest = new PombaRequest();
-        pombaRequest.serviceInstanceList = Lists.newArrayList(serviceInstance1, serviceInstance2);
+        PombaRequest pombaRequest = new PombaRequest(Lists.newArrayList(serviceInstance1, serviceInstance2));
         return pombaRequest;
     }
 
     private ServiceInstance createServiceInstance(String serviceType, String serviceInstanceId, String customerId,
         String modelVersionId, String modelInvariantId) {
-        ServiceInstance serviceInstance = new ServiceInstance();
-        serviceInstance.serviceType = serviceType;
-        serviceInstance.serviceInstanceId = serviceInstanceId;
-        serviceInstance.customerId = customerId;
-        serviceInstance.modelInvariantId = modelInvariantId;
-        serviceInstance.modelVersionId = modelVersionId;
+        ServiceInstance serviceInstance = new ServiceInstance(
+            serviceInstanceId,
+            modelVersionId,
+            modelInvariantId,
+            customerId,
+            serviceType);
         return serviceInstance;
     }
 
diff --git a/vid-app-common/src/test/java/org/onap/vid/controller/VidControllerTest.java b/vid-app-common/src/test/java/org/onap/vid/controller/VidControllerTest.java
index aeb03b60c..8f0eb087d 100644
--- a/vid-app-common/src/test/java/org/onap/vid/controller/VidControllerTest.java
+++ b/vid-app-common/src/test/java/org/onap/vid/controller/VidControllerTest.java
@@ -151,8 +151,7 @@ public class VidControllerTest {
 
     @Test
     public void verifyServiceInstance_shouldReturnOk() throws Exception {
-        PombaRequest pombaRequest = new PombaRequest();
-        pombaRequest.serviceInstanceList = ImmutableList.of(new ServiceInstance());
+        PombaRequest pombaRequest = new PombaRequest(ImmutableList.of(new ServiceInstance()));
 
         mockMvc.perform(post(REST_MODELS_SERVICES_VERIFY_SERVICE)
             .contentType(MediaType.APPLICATION_JSON)
diff --git a/vid-app-common/src/test/java/org/onap/vid/services/PombaServiceImplTest.java b/vid-app-common/src/test/java/org/onap/vid/services/PombaServiceImplTest.java
index d0eec2619..aaa64484f 100644
--- a/vid-app-common/src/test/java/org/onap/vid/services/PombaServiceImplTest.java
+++ b/vid-app-common/src/test/java/org/onap/vid/services/PombaServiceImplTest.java
@@ -1,8 +1,29 @@
+/*-
+ * ============LICENSE_START=======================================================
+ * VID
+ * ================================================================================
+ * Copyright (C) 2018 Nokia
+ * ================================================================================
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * ============LICENSE_END=========================================================
+ */
 package org.onap.vid.services;
 
 import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
 import static org.mockito.MockitoAnnotations.initMocks;
+
+import com.google.common.collect.ImmutableList;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
 import org.mockito.Mockito;
@@ -32,7 +53,7 @@ public class PombaServiceImplTest {
 
   @Test
   public void testVerify() {
-    PombaRequest pombaRequest = new PombaRequest();
+    PombaRequest pombaRequest = new PombaRequest(ImmutableList.of());
     testSubject.verify(pombaRequest);
     verify(pombaClientInterface, times(1))
         .verify(pombaRequest);