From ff317808f308af03321de0f0b0849cd9f6f8ccad Mon Sep 17 00:00:00 2001 From: vasraz Date: Tue, 31 May 2022 18:31:59 +0100 Subject: Maintain VFC UI added interface operations after an upgrade Signed-off-by: Vasyl Razinkov Change-Id: I908a4480a8929f9aeeabf4f5c14049cade6ae22b Issue-ID: SDC-4018 --- .../be/components/ResourceImportManagerTest.java | 80 +++++++++++++++++++++- 1 file changed, 78 insertions(+), 2 deletions(-) (limited to 'catalog-be/src/test') diff --git a/catalog-be/src/test/java/org/openecomp/sdc/be/components/ResourceImportManagerTest.java b/catalog-be/src/test/java/org/openecomp/sdc/be/components/ResourceImportManagerTest.java index df679f1402..32d174f194 100644 --- a/catalog-be/src/test/java/org/openecomp/sdc/be/components/ResourceImportManagerTest.java +++ b/catalog-be/src/test/java/org/openecomp/sdc/be/components/ResourceImportManagerTest.java @@ -47,6 +47,8 @@ import java.util.Collections; import java.util.HashMap; import java.util.List; import java.util.Map; +import org.apache.commons.collections4.CollectionUtils; +import org.apache.commons.collections4.MapUtils; import org.apache.commons.lang3.tuple.ImmutablePair; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.BeforeEach; @@ -357,6 +359,43 @@ class ResourceImportManagerTest { assertSetInterfaceImplementation(createResource.left); } + @Test + void testReimportVfcWithInterfaceImplementation() throws IOException { + user.setUserId(resourceMD.getContactId()); + when(userAdmin.getUser(anyString(), anyBoolean())).thenReturn(user); + + setResourceBusinessLogicMock(); + + String jsonContent = ImportUtilsTest.loadCustomTypeFileNameToJsonString("custom-types-node-type-with-interface-impl.yml"); + + Map interfaceTypes = new HashMap<>(); + final InterfaceDefinition interfaceDefinition = new InterfaceDefinition(); + interfaceDefinition.setType("tosca.interfaces.node.lifecycle.Standard"); + Map operations = new HashMap<>(); + operations.put("configure", new OperationDataDefinition()); + interfaceDefinition.setOperations(operations); + interfaceTypes.put("tosca.interfaces.node.lifecycle.standard", interfaceDefinition); + when(interfaceOperationBusinessLogic.getAllInterfaceLifecycleTypes(any())).thenReturn(Either.left(interfaceTypes)); + + ImmutablePair createResource = + importManager.importNormativeResource(jsonContent, resourceMD, user, true, true, false); + assertNotNull(createResource); + Resource resource = createResource.getLeft(); + assertNotNull(resource); + assertSetInterfaceImplementation(resource); + + final GraphVertex graphVertex = new GraphVertex(); + graphVertex.setUniqueId("1-2-3-4-5-6-7"); + when(janusGraphDao.getByCriteria(eq(VertexTypeEnum.NODE_TYPE), anyMap(), anyMap(), eq(JsonParseFlagEnum.ParseAll), any(), anyBoolean())) + .thenReturn(Either.left(Arrays.asList(graphVertex))); + when(toscaOperationFacade.getToscaElement(anyString())).thenReturn(Either.left(resource)); + createResource = reimportVfc(resource, jsonContent); + assertNotNull(createResource); + resource = createResource.getLeft(); + assertNotNull(resource); + assertSetInterfaceImplementationAfterReimport(resource); + } + @Test void testResourceCreationWithInterfaceImplementation_UnknownInterface() throws IOException { user.setUserId(resourceMD.getContactId()); @@ -508,14 +547,33 @@ class ResourceImportManagerTest { } - private ImmutablePair reimportVfc(Resource resource, String jsonContent) { + private ImmutablePair reimportVfc(final Resource resource, final String jsonContent) { List propertiesList = resource.getProperties(); - PropertyDefinition propertyDefinition = new PropertyDefinition(); + if (CollectionUtils.isEmpty(propertiesList)) { + propertiesList = new ArrayList<>(); + } + final PropertyDefinition propertyDefinition = new PropertyDefinition(); propertyDefinition.setName("oneMore"); propertyDefinition.setUserCreated(true); propertyDefinition.setType("boolean"); propertiesList.add(propertyDefinition); resource.setProperties(propertiesList); + + Map interfaces = resource.getInterfaces(); + if (MapUtils.isEmpty(interfaces)) { + interfaces = new HashMap<>(); + } + final InterfaceDefinition interfaceDefinition = new InterfaceDefinition(); + interfaceDefinition.setType("tosca.interfaces.relationship.Configure"); + interfaceDefinition.setUserCreated(true); + final OperationDataDefinition operationDataDefinition = new OperationDataDefinition(); + operationDataDefinition.setName("add_source"); + final Map operationDataDefinitionMap = new HashMap<>(); + operationDataDefinitionMap.put(operationDataDefinition.getName(), operationDataDefinition); + interfaceDefinition.setOperations(operationDataDefinitionMap); + interfaces.put(interfaceDefinition.getType(), interfaceDefinition); + resource.setInterfaces(interfaces); + return importManager.importNormativeResource(jsonContent, resourceMD, user, true, true, false); } @@ -614,6 +672,24 @@ class ResourceImportManagerTest { assertTrue(operations.containsKey(operationDataDefinition.getName()))); } + private void assertSetInterfaceImplementationAfterReimport(final Resource resource) { + final Map interfaces = resource.getInterfaces(); + assertNotNull(interfaces); + assertEquals(2, interfaces.size()); + + InterfaceDefinition interfaceDefinition = interfaces.get("tosca.interfaces.node.lifecycle.Standard"); + assertTrue(interfaces.containsKey(interfaceDefinition.getType())); + assertFalse(interfaceDefinition.isUserCreated()); + final Map operations_1 = interfaceDefinition.getOperations(); + operations_1.values().forEach(operationDataDefinition -> assertTrue(operations_1.containsKey(operationDataDefinition.getName()))); + + interfaceDefinition = interfaces.get("tosca.interfaces.relationship.Configure"); + assertTrue(interfaces.containsKey(interfaceDefinition.getType())); + assertTrue(interfaceDefinition.isUserCreated()); + final Map operations_2 = interfaceDefinition.getOperations(); + operations_2.values().forEach(operationDataDefinition -> assertTrue(operations_2.containsKey(operationDataDefinition.getName()))); + } + private void testSetDerivedFrom(Resource resource) { assertEquals(1, resource.getDerivedFrom().size()); assertEquals("tosca.nodes.Root", resource.getDerivedFrom().get(0)); -- cgit 1.2.3-korg