From 619d5f7dd8a03c4d53ff9adacfd5d208f1371735 Mon Sep 17 00:00:00 2001 From: Michal Kabaj Date: Mon, 18 Feb 2019 12:29:31 +0100 Subject: PombaReqest and ServiceInstance improvements - fix encapsulation issues - make object immutable Change-Id: I7f66aa7ec5e0ba63a9178ad27c8a9466f04e0869 Issue-ID: VID-422 Signed-off-by: Michal Kabaj --- .../java/org/onap/vid/aai/PombaClientImplTest.java | 19 ++++++++---------- .../org/onap/vid/controller/VidControllerTest.java | 3 +-- .../onap/vid/services/PombaServiceImplTest.java | 23 +++++++++++++++++++++- 3 files changed, 31 insertions(+), 14 deletions(-) (limited to 'vid-app-common/src/test/java/org') 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); -- cgit 1.2.3-korg