aboutsummaryrefslogtreecommitdiffstats
path: root/vid-app-common/src/test/java
diff options
context:
space:
mode:
authorMichal Kabaj <michal.kabaj@nokia.com>2019-02-18 12:29:31 +0100
committerMichal Kabaj <michal.kabaj@nokia.com>2019-02-18 12:29:31 +0100
commit619d5f7dd8a03c4d53ff9adacfd5d208f1371735 (patch)
tree0530da744a122ca1ff316e5c69f98d89a696f9d7 /vid-app-common/src/test/java
parente9381e08b6a48468ff6f3ccaa03d0b19dd47d535 (diff)
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>
Diffstat (limited to 'vid-app-common/src/test/java')
-rw-r--r--vid-app-common/src/test/java/org/onap/vid/aai/PombaClientImplTest.java19
-rw-r--r--vid-app-common/src/test/java/org/onap/vid/controller/VidControllerTest.java3
-rw-r--r--vid-app-common/src/test/java/org/onap/vid/services/PombaServiceImplTest.java23
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);