From c1e4dcf4a8030bbcf04f487fd7a6a910bb5eced1 Mon Sep 17 00:00:00 2001 From: shrikantawachar Date: Wed, 14 Feb 2018 19:37:52 +0530 Subject: Enforcing HEAT validation after HEAT upload Refactored method name , removed unused variable, Moved variable to near its usage. Change-Id: Ief21e88bd6a4ec1b1683b89089cfdb1a948a5402 Issue-ID: SDC-1027 Signed-off-by: shrikantawachar --- .../impl/VendorSoftwareProductManagerImpl.java | 8 ++++---- .../impl/OrchestrationTemplateCandidateManagerImplTest.java | 2 -- .../impl/VendorSoftwareProductManagerImplTest.java | 11 ++--------- 3 files changed, 6 insertions(+), 15 deletions(-) (limited to 'openecomp-be/backend') diff --git a/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/VendorSoftwareProductManagerImpl.java b/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/VendorSoftwareProductManagerImpl.java index bf5f19fb2b..8961df70ea 100644 --- a/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/VendorSoftwareProductManagerImpl.java +++ b/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/VendorSoftwareProductManagerImpl.java @@ -214,8 +214,6 @@ public class VendorSoftwareProductManagerImpl implements VendorSoftwareProductMa } validationResponse.setUploadDataErrors(validateOrchestrationTemplate(orchestrationTemplate)); } - Collection componentDependencies = - componentDependencyModelDao.list(new ComponentDependencyModelEntity(vspId, version, null)); QuestionnaireValidationResult questionnaireValidationResult = validateQuestionnaire (vspDetails.getId(), vspDetails.getVersion(), vspDetails.getOnboardingMethod()); @@ -230,6 +228,9 @@ public class VendorSoftwareProductManagerImpl implements VendorSoftwareProductMa } } + Collection componentDependencies = + componentDependencyModelDao.list(new ComponentDependencyModelEntity(vspId, version, null)); + if (validateComponentDependencies(componentDependencies)) { vspErrors .add(ComponentDependencyModelErrorBuilder.getcyclicDependencyComponentErrorBuilder()); @@ -251,11 +252,10 @@ public class VendorSoftwareProductManagerImpl implements VendorSoftwareProductMa if (validationData.isEmpty()) { vspErrors.add(VendorSoftwareProductInvalidErrorBuilder .candidateDataNotProcessedOrAbortedErrorBuilder(fileName)); - validationResponse.setVspErrors(vspErrors); } else { vspErrors.add(VendorSoftwareProductInvalidErrorBuilder.invalidProcessedCandidate(fileName)); - validationResponse.setVspErrors(vspErrors); } + validationResponse.setVspErrors(vspErrors); } } diff --git a/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/test/java/org/openecomp/sdc/vendorsoftwareproduct/impl/OrchestrationTemplateCandidateManagerImplTest.java b/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/test/java/org/openecomp/sdc/vendorsoftwareproduct/impl/OrchestrationTemplateCandidateManagerImplTest.java index e1d58c6a18..a86b1ef0c5 100644 --- a/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/test/java/org/openecomp/sdc/vendorsoftwareproduct/impl/OrchestrationTemplateCandidateManagerImplTest.java +++ b/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/test/java/org/openecomp/sdc/vendorsoftwareproduct/impl/OrchestrationTemplateCandidateManagerImplTest.java @@ -77,8 +77,6 @@ public class OrchestrationTemplateCandidateManagerImplTest { private static final String COMPONENT_NAME_3 = "oam_server"; private static final String COMPONENT_NAME_4 = "ps_server"; - private static final String USER1 = "vspTestUser1"; - @Mock private VendorSoftwareProductInfoDao vspInfoDaoMock; @Mock diff --git a/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/test/java/org/openecomp/sdc/vendorsoftwareproduct/impl/VendorSoftwareProductManagerImplTest.java b/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/test/java/org/openecomp/sdc/vendorsoftwareproduct/impl/VendorSoftwareProductManagerImplTest.java index 2e10f1351e..a9223db285 100644 --- a/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/test/java/org/openecomp/sdc/vendorsoftwareproduct/impl/VendorSoftwareProductManagerImplTest.java +++ b/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/test/java/org/openecomp/sdc/vendorsoftwareproduct/impl/VendorSoftwareProductManagerImplTest.java @@ -586,7 +586,7 @@ public class VendorSoftwareProductManagerImplTest { } */ @Test - public void testSubmitWithCandidateDataNotProcessed() throws IOException { + public void testValidateWithCandidateDataNotProcessed() throws IOException { VspDetails vsp = createVspDetails(VSP_ID, VERSION01, "Vsp1", "Test-vsp", "vendorName", "vlm1Id", "icon", "category", "subCategory", "licenseAgreementId", @@ -608,13 +608,10 @@ public class VendorSoftwareProductManagerImplTest { Assert.assertNotNull(validationResponse.getVspErrors()); Assert.assertEquals(validationResponse.getVspErrors().size(), 1); - verify(versioningManagerMock, never()) - .submit(VendorSoftwareProductConstants.VENDOR_SOFTWARE_PRODUCT_VERSIONABLE_TYPE, VSP_ID, - USER1, null); } @Test - public void testSubmitWithCandidateProcessedIsInvalid() throws IOException { + public void testValidateWithCandidateProcessedIsInvalid() throws IOException { VspDetails vsp = createVspDetails(VSP_ID, VERSION01, "Vsp1", "Test-VSP", "vendorName", "vl1Id", "icond", "category", "subcategory", "licenseAgreementId", Collections .singletonList("featureGroupId")); @@ -633,10 +630,6 @@ public class VendorSoftwareProductManagerImplTest { Assert.assertFalse(validationResponse.isValid()); Assert.assertNotNull(validationResponse.getVspErrors()); Assert.assertEquals(validationResponse.getVspErrors().size(), 1); - - verify(versioningManagerMock, never()) - .submit(VendorSoftwareProductConstants.VENDOR_SOFTWARE_PRODUCT_VERSIONABLE_TYPE, VSP_ID, - USER1, null); } private void testLegalUpload(String vspId, Version version, InputStream upload, String user) { -- cgit 1.2.3-korg