From 254a7df768167685b835b87a54d04596b570e38a Mon Sep 17 00:00:00 2001 From: Rodrigo Lima Date: Thu, 7 May 2020 15:03:58 -0400 Subject: Remove powermock from NicsImplTest - Remove powermock from NicsImplTest and add new constructor to NicsImpl Issue-ID: SDC-3043 Signed-off-by: Rodrigo Lima Change-Id: Iec9343a403278f7db41d1654960c1060c3e75a99 --- .../sdcrests/vsp/rest/services/NicsImpl.java | 15 +++++++-- .../sdcrests/vsp/rest/services/NicsImplTest.java | 36 +++------------------- 2 files changed, 16 insertions(+), 35 deletions(-) (limited to 'openecomp-be') diff --git a/openecomp-be/api/openecomp-sdc-rest-webapp/vendor-software-products-rest/vendor-software-products-rest-services/src/main/java/org/openecomp/sdcrests/vsp/rest/services/NicsImpl.java b/openecomp-be/api/openecomp-sdc-rest-webapp/vendor-software-products-rest/vendor-software-products-rest-services/src/main/java/org/openecomp/sdcrests/vsp/rest/services/NicsImpl.java index c2d53805d2..11ed508286 100644 --- a/openecomp-be/api/openecomp-sdc-rest-webapp/vendor-software-products-rest/vendor-software-products-rest-services/src/main/java/org/openecomp/sdcrests/vsp/rest/services/NicsImpl.java +++ b/openecomp-be/api/openecomp-sdc-rest-webapp/vendor-software-products-rest/vendor-software-products-rest-services/src/main/java/org/openecomp/sdcrests/vsp/rest/services/NicsImpl.java @@ -46,9 +46,18 @@ import java.util.Collection; @Service("nics") @Scope(value = "prototype") public class NicsImpl implements Nics { - private NicManager nicManager = NicManagerFactory.getInstance().createInterface(); - private ComponentManager componentManager = - ComponentManagerFactory.getInstance().createInterface(); + private final NicManager nicManager; + private final ComponentManager componentManager; + + public NicsImpl() { + this.nicManager = NicManagerFactory.getInstance().createInterface(); + this.componentManager = ComponentManagerFactory.getInstance().createInterface(); + } + + public NicsImpl(NicManager nicManager, ComponentManager componentManager) { + this.nicManager = nicManager; + this.componentManager = componentManager; + } @Override public Response list(String vspId, String versionId, String componentId, String user) { diff --git a/openecomp-be/api/openecomp-sdc-rest-webapp/vendor-software-products-rest/vendor-software-products-rest-services/src/test/java/org/openecomp/sdcrests/vsp/rest/services/NicsImplTest.java b/openecomp-be/api/openecomp-sdc-rest-webapp/vendor-software-products-rest/vendor-software-products-rest-services/src/test/java/org/openecomp/sdcrests/vsp/rest/services/NicsImplTest.java index 7c16fde904..a6a4e98f82 100644 --- a/openecomp-be/api/openecomp-sdc-rest-webapp/vendor-software-products-rest/vendor-software-products-rest-services/src/test/java/org/openecomp/sdcrests/vsp/rest/services/NicsImplTest.java +++ b/openecomp-be/api/openecomp-sdc-rest-webapp/vendor-software-products-rest/vendor-software-products-rest-services/src/test/java/org/openecomp/sdcrests/vsp/rest/services/NicsImplTest.java @@ -24,15 +24,12 @@ import org.apache.http.HttpStatus; import org.junit.Assert; import org.junit.Before; import org.junit.Test; -import org.junit.runner.RunWith; import org.mockito.ArgumentMatchers; import org.mockito.Mock; import org.openecomp.sdc.logging.api.Logger; import org.openecomp.sdc.logging.api.LoggerFactory; import org.openecomp.sdc.vendorsoftwareproduct.ComponentManager; -import org.openecomp.sdc.vendorsoftwareproduct.ComponentManagerFactory; import org.openecomp.sdc.vendorsoftwareproduct.NicManager; -import org.openecomp.sdc.vendorsoftwareproduct.NicManagerFactory; import org.openecomp.sdc.vendorsoftwareproduct.dao.type.NicEntity; import org.openecomp.sdc.vendorsoftwareproduct.types.CompositionEntityResponse; import org.openecomp.sdc.vendorsoftwareproduct.types.QuestionnaireResponse; @@ -44,8 +41,6 @@ import org.openecomp.sdcrests.vendorsoftwareproducts.types.NicDto; import org.openecomp.sdcrests.vendorsoftwareproducts.types.NicRequestDto; import org.openecomp.sdcrests.vendorsoftwareproducts.types.QuestionnaireResponseDto; import org.openecomp.sdcrests.wrappers.GenericCollectionWrapper; -import org.powermock.core.classloader.annotations.PrepareForTest; -import org.powermock.modules.junit4.PowerMockRunner; import javax.ws.rs.core.Response; import java.util.Collection; @@ -53,24 +48,15 @@ import java.util.Collections; import java.util.UUID; import static org.mockito.MockitoAnnotations.initMocks; -import static org.powermock.api.mockito.PowerMockito.mockStatic; -import static org.powermock.api.mockito.PowerMockito.when; +import static org.mockito.Mockito.when; -@RunWith(PowerMockRunner.class) -@PrepareForTest({NicsImpl.class, ComponentManagerFactory.class, NicManagerFactory.class}) public class NicsImplTest { private Logger logger = LoggerFactory.getLogger(NicsImplTest.class); - @Mock - private NicManagerFactory nicManagerFactory; - @Mock private NicManager nicManager; - @Mock - private ComponentManagerFactory componentManagerFactory; - @Mock private ComponentManager componentManager; @@ -80,21 +66,13 @@ public class NicsImplTest { private final String nicId = "" + System.currentTimeMillis(); private final String user = "cs0008"; + private NicsImpl bean; + @Before public void setUp() { try { initMocks(this); - mockStatic(ComponentManagerFactory.class); - when(ComponentManagerFactory.getInstance()).thenReturn(componentManagerFactory); - when(componentManagerFactory.createInterface()).thenReturn(componentManager); - - mockStatic(NicManagerFactory.class); - when(NicManagerFactory.getInstance()).thenReturn(nicManagerFactory); - when(nicManagerFactory.createInterface()).thenReturn(nicManager); - - - NicEntity e = new NicEntity(); e.setComponentId(componentId); e.setId(nicId); @@ -132,6 +110,7 @@ public class NicsImplTest { ArgumentMatchers.eq(componentId), ArgumentMatchers.eq(nicId))).thenReturn(qr); + bean = new NicsImpl(nicManager, componentManager); } catch (Exception e) { logger.error(e.getMessage(), e); @@ -140,7 +119,6 @@ public class NicsImplTest { @Test public void testList() { - NicsImpl bean = new NicsImpl(); Response rsp = bean.list(vspId, versionId, componentId, user); Assert.assertEquals("Response should be 200", HttpStatus.SC_OK, rsp.getStatus()); @@ -162,7 +140,6 @@ public class NicsImplTest { dto.setNetworkId(nicId); dto.setNetworkType("External"); - NicsImpl bean = new NicsImpl(); Response rsp = bean.create(dto, vspId, versionId, componentId, user); Assert.assertEquals("Response should be 200", HttpStatus.SC_OK, rsp.getStatus()); Object e = rsp.getEntity(); @@ -178,7 +155,6 @@ public class NicsImplTest { @Test public void testDelete() { - NicsImpl bean = new NicsImpl(); Response rsp = bean.delete(vspId, versionId, componentId, nicId, user); Assert.assertEquals("Response should be 200", HttpStatus.SC_OK, rsp.getStatus()); Assert.assertNull(rsp.getEntity()); @@ -187,7 +163,6 @@ public class NicsImplTest { @Test public void testGet() { - NicsImpl bean = new NicsImpl(); Response rsp = bean.get(vspId, versionId, componentId, nicId, user); Assert.assertEquals("Response should be 200", HttpStatus.SC_OK, rsp.getStatus()); Assert.assertNotNull(rsp.getEntity()); @@ -195,7 +170,6 @@ public class NicsImplTest { @Test public void testUpdate() { - NicsImpl bean = new NicsImpl(); NicRequestDto dto = new NicRequestDto(); dto.setDescription("hello"); dto.setName("name"); @@ -210,7 +184,6 @@ public class NicsImplTest { @Test public void testGetQuestionaire() { - NicsImpl bean = new NicsImpl(); Response rsp = bean.getQuestionnaire(vspId, versionId, componentId, nicId, user); Assert.assertEquals("Response should be 200", HttpStatus.SC_OK, rsp.getStatus()); try { @@ -226,7 +199,6 @@ public class NicsImplTest { @Test public void testUpdateQuestionaire() { - NicsImpl bean = new NicsImpl(); Response rsp = bean.updateQuestionnaire("helloworld", vspId, versionId, componentId, nicId, user); Assert.assertEquals("Response should be 200", HttpStatus.SC_OK, rsp.getStatus()); Assert.assertNull(rsp.getEntity()); -- cgit 1.2.3-korg