From 339d29c714f60de6e4e50b883259fc1a9a16ee84 Mon Sep 17 00:00:00 2001 From: Rodrigo Lima Date: Fri, 1 May 2020 15:25:06 -0400 Subject: Remove powermockito from OrchestrationTemplateCandidateImplTest - Remove powermockito from OrchestrationTemplateCandidateImplTest. Add new constructor to OrchestrationTemplateCandidateImpl Issue-ID: SDC-3007 Signed-off-by: Rodrigo Lima Change-Id: I5f0a2061796e8410a5b225c43baa70d7e0269684 --- .../OrchestrationTemplateCandidateImpl.java | 27 ++++++++++--- .../OrchestrationTemplateCandidateImplTest.java | 47 ++++------------------ 2 files changed, 29 insertions(+), 45 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/OrchestrationTemplateCandidateImpl.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/OrchestrationTemplateCandidateImpl.java index e4262df105..c331deb134 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/OrchestrationTemplateCandidateImpl.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/OrchestrationTemplateCandidateImpl.java @@ -72,12 +72,27 @@ import org.springframework.stereotype.Service; public class OrchestrationTemplateCandidateImpl implements OrchestrationTemplateCandidate { private static final Logger LOGGER = LoggerFactory.getLogger(OrchestrationTemplateCandidateImpl.class); - private OrchestrationTemplateCandidateManager candidateManager = - OrchestrationTemplateCandidateManagerFactory.getInstance().createInterface(); - private VendorSoftwareProductManager vendorSoftwareProductManager = VspManagerFactory - .getInstance().createInterface(); - private ActivityLogManager activityLogManager = - ActivityLogManagerFactory.getInstance().createInterface(); + private final OrchestrationTemplateCandidateManager candidateManager; + + private final VendorSoftwareProductManager vendorSoftwareProductManager; + private final ActivityLogManager activityLogManager; + + + public OrchestrationTemplateCandidateImpl() { + this.candidateManager = OrchestrationTemplateCandidateManagerFactory.getInstance().createInterface(); + this.vendorSoftwareProductManager = VspManagerFactory.getInstance().createInterface(); + this.activityLogManager = ActivityLogManagerFactory.getInstance().createInterface(); + } + + // Constructor used in test to avoid mock static + public OrchestrationTemplateCandidateImpl( + OrchestrationTemplateCandidateManager candidateManager, + VendorSoftwareProductManager vendorSoftwareProductManager, + ActivityLogManager activityLogManager) { + this.candidateManager = candidateManager; + this.vendorSoftwareProductManager = vendorSoftwareProductManager; + this.activityLogManager = activityLogManager; + } @Override public Response upload(final String vspId, final String versionId, 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/OrchestrationTemplateCandidateImplTest.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/OrchestrationTemplateCandidateImplTest.java index 589be207c1..dec6342cb8 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/OrchestrationTemplateCandidateImplTest.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/OrchestrationTemplateCandidateImplTest.java @@ -24,9 +24,7 @@ import static junit.framework.TestCase.assertEquals; import static junit.framework.TestCase.assertFalse; import static org.mockito.ArgumentMatchers.any; import static org.mockito.MockitoAnnotations.initMocks; -import static org.powermock.api.mockito.PowerMockito.mock; -import static org.powermock.api.mockito.PowerMockito.mockStatic; -import static org.powermock.api.mockito.PowerMockito.when; +import static org.mockito.Mockito.when; import java.io.IOException; import java.util.Arrays; @@ -40,18 +38,15 @@ import org.apache.cxf.jaxrs.ext.multipart.ContentDisposition; 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.mockito.Mockito; import org.openecomp.core.utilities.orchestration.OnboardingTypesEnum; import org.openecomp.sdc.activitylog.ActivityLogManager; -import org.openecomp.sdc.activitylog.ActivityLogManagerFactory; import org.openecomp.sdc.logging.api.Logger; import org.openecomp.sdc.logging.api.LoggerFactory; import org.openecomp.sdc.vendorsoftwareproduct.OrchestrationTemplateCandidateManager; -import org.openecomp.sdc.vendorsoftwareproduct.OrchestrationTemplateCandidateManagerFactory; import org.openecomp.sdc.vendorsoftwareproduct.VendorSoftwareProductManager; -import org.openecomp.sdc.vendorsoftwareproduct.VspManagerFactory; import org.openecomp.sdc.vendorsoftwareproduct.types.OrchestrationTemplateActionResponse; import org.openecomp.sdc.vendorsoftwareproduct.types.UploadFileResponse; import org.openecomp.sdc.vendorsoftwareproduct.types.UploadFileStatus; @@ -61,12 +56,7 @@ import org.openecomp.sdc.vendorsoftwareproduct.types.candidateheat.Module; import org.openecomp.sdcrests.vendorsoftwareproducts.types.FileDataStructureDto; import org.openecomp.sdcrests.vendorsoftwareproducts.types.OrchestrationTemplateActionResponseDto; import org.openecomp.sdcrests.vendorsoftwareproducts.types.UploadFileResponseDto; -import org.powermock.core.classloader.annotations.PrepareForTest; -import org.powermock.modules.junit4.PowerMockRunner; -@RunWith(PowerMockRunner.class) -@PrepareForTest({VspManagerFactory.class, ActivityLogManagerFactory.class, - OrchestrationTemplateCandidateManagerFactory.class, OrchestrationTemplateCandidateImpl.class}) public class OrchestrationTemplateCandidateImplTest { private Logger logger = LoggerFactory.getLogger(OrchestrationTemplateCandidateImplTest.class); @@ -75,14 +65,9 @@ public class OrchestrationTemplateCandidateImplTest { private OrchestrationTemplateCandidateManager candidateManager; @Mock private VendorSoftwareProductManager vendorSoftwareProductManager; - @Mock - private VspManagerFactory vspManagerFactory; + @Mock private ActivityLogManager activityLogManager; - @Mock - private ActivityLogManagerFactory activityLogManagerFactory; - @Mock - OrchestrationTemplateCandidateManagerFactory orchestrationTemplateCandidateManagerFactory; private OrchestrationTemplateCandidateImpl orchestrationTemplateCandidate; @@ -96,15 +81,6 @@ public class OrchestrationTemplateCandidateImplTest { public void setUp(){ try { initMocks(this); - mockStatic(VspManagerFactory.class); - when(VspManagerFactory.getInstance()).thenReturn(vspManagerFactory); - when(vspManagerFactory.createInterface()).thenReturn(vendorSoftwareProductManager); - mockStatic(ActivityLogManagerFactory.class); - when(ActivityLogManagerFactory.getInstance()).thenReturn(activityLogManagerFactory); - when(activityLogManagerFactory.createInterface()).thenReturn(activityLogManager); - mockStatic(OrchestrationTemplateCandidateManagerFactory.class); - when(OrchestrationTemplateCandidateManagerFactory.getInstance()).thenReturn(orchestrationTemplateCandidateManagerFactory); - when(orchestrationTemplateCandidateManagerFactory.createInterface()).thenReturn(candidateManager); UploadFileResponse uploadFileResponse = new UploadFileResponse(); uploadFileResponse.setOnboardingType(OnboardingTypesEnum.ZIP); uploadFileResponse.setNetworkPackageName("test"); @@ -148,6 +124,9 @@ public class OrchestrationTemplateCandidateImplTest { ArgumentMatchers.eq(candidateId), ArgumentMatchers.any())).thenReturn(Optional.of(fds)); + orchestrationTemplateCandidate = + new OrchestrationTemplateCandidateImpl(candidateManager, vendorSoftwareProductManager, activityLogManager); + }catch (Exception e){ logger.error(e.getMessage(), e); @@ -156,22 +135,20 @@ public class OrchestrationTemplateCandidateImplTest { @Test public void uploadSignedTest() { - orchestrationTemplateCandidate = new OrchestrationTemplateCandidateImpl(); Response response = orchestrationTemplateCandidate.upload("1", "1", mockAttachment("filename.zip"), "1"); assertEquals(Response.Status.OK.getStatusCode(), response.getStatus()); } @Test public void uploadNotSignedTest(){ - orchestrationTemplateCandidate = new OrchestrationTemplateCandidateImpl(); Response response = orchestrationTemplateCandidate.upload("1", "1", mockAttachment("filename.csar"), "1"); assertEquals(Response.Status.OK.getStatusCode(), response.getStatus()); } private Attachment mockAttachment(final String fileName) { - final Attachment attachment = mock(Attachment.class); + final Attachment attachment = Mockito.mock(Attachment.class); when(attachment.getContentDisposition()).thenReturn(new ContentDisposition("test")); - final DataHandler dataHandler = mock(DataHandler.class); + final DataHandler dataHandler = Mockito.mock(DataHandler.class); when(dataHandler.getName()).thenReturn(fileName); when(attachment.getDataHandler()).thenReturn(dataHandler); final byte[] bytes = "upload package Test".getBytes(); @@ -181,7 +158,6 @@ public class OrchestrationTemplateCandidateImplTest { @Test public void uploadSignNotValidTest() { - orchestrationTemplateCandidate = new OrchestrationTemplateCandidateImpl(); Response response = orchestrationTemplateCandidate.upload("1", "1", mockAttachment("filename.zip"), "1"); assertEquals(Response.Status.OK.getStatusCode(), response.getStatus()); assertFalse(((UploadFileResponseDto)response.getEntity()).getErrors().isEmpty()); @@ -189,7 +165,6 @@ public class OrchestrationTemplateCandidateImplTest { @Test public void testCandidateGet() throws IOException { - orchestrationTemplateCandidate = new OrchestrationTemplateCandidateImpl(); Response rsp = orchestrationTemplateCandidate.get(candidateId, versionId, user); Assert.assertEquals("Response status equals", Response.Status.OK.getStatusCode(), rsp.getStatus()); Assert.assertNotEquals(rsp.getHeaderString("Content-Disposition").indexOf("Candidate"),-1); @@ -199,7 +174,6 @@ public class OrchestrationTemplateCandidateImplTest { @Test public void testVendorSoftwareProductGet() throws IOException { - orchestrationTemplateCandidate = new OrchestrationTemplateCandidateImpl(); Response rsp = orchestrationTemplateCandidate.get(softwareProductId, versionId, user); Assert.assertEquals("Response status equals", Response.Status.OK.getStatusCode(), rsp.getStatus()); Assert.assertNotEquals(rsp.getHeaderString("Content-Disposition").indexOf("Processed"),-1); @@ -209,7 +183,6 @@ public class OrchestrationTemplateCandidateImplTest { @Test public void testMissingGet() throws IOException { - orchestrationTemplateCandidate = new OrchestrationTemplateCandidateImpl(); Response rsp = orchestrationTemplateCandidate.get(UUID.randomUUID().toString(), versionId, user); Assert.assertEquals("Response status equals", Response.Status.NOT_FOUND.getStatusCode(), rsp.getStatus()); } @@ -217,7 +190,6 @@ public class OrchestrationTemplateCandidateImplTest { @Test public void testAbort() { try { - orchestrationTemplateCandidate = new OrchestrationTemplateCandidateImpl(); Response rsp = orchestrationTemplateCandidate.abort(candidateId, versionId); Assert.assertEquals("Response status equals", Response.Status.OK.getStatusCode(), rsp.getStatus()); Assert.assertNull(rsp.getEntity()); @@ -231,7 +203,6 @@ public class OrchestrationTemplateCandidateImplTest { @Test public void testProcess() { try { - orchestrationTemplateCandidate = new OrchestrationTemplateCandidateImpl(); Response rsp = orchestrationTemplateCandidate.process(candidateId, versionId, user); Assert.assertEquals("Response status equals", Response.Status.OK.getStatusCode(), rsp.getStatus()); Assert.assertNotNull(rsp.getEntity()); @@ -249,7 +220,6 @@ public class OrchestrationTemplateCandidateImplTest { try { FileDataStructureDto dto = new FileDataStructureDto(); dto.setArtifacts(Arrays.asList("a", "b", "c")); - orchestrationTemplateCandidate = new OrchestrationTemplateCandidateImpl(); Response rsp = orchestrationTemplateCandidate.updateFilesDataStructure(candidateId, versionId, dto, user); Assert.assertEquals("Response status equals", Response.Status.OK.getStatusCode(), rsp.getStatus()); } @@ -264,7 +234,6 @@ public class OrchestrationTemplateCandidateImplTest { try { FileDataStructureDto dto = new FileDataStructureDto(); dto.setArtifacts(Arrays.asList("a", "b", "c")); - orchestrationTemplateCandidate = new OrchestrationTemplateCandidateImpl(); Response rsp = orchestrationTemplateCandidate.getFilesDataStructure(candidateId, versionId, user); Assert.assertEquals("Response status equals", Response.Status.OK.getStatusCode(), rsp.getStatus()); } -- cgit 1.2.3-korg