Remove usage of outdated library JMockit (catalog-model)

Signed-off-by: Vasyl Razinkov <vasyl.razinkov@est.tech>
Change-Id: I3888fb6f9a4c89aefc3dcb124529abd726c400d9
Issue-ID: SDC-3379

3 files changed, 136 insertions, 134 deletions

diff --git a/catalog-model/pom.xml b/catalog-model/pom.xml
index c71ea9e883..e9e8a7292d 100644
--- a/catalog-model/pom.xml
+++ b/catalog-model/pom.xml
@@ -260,12 +260,6 @@
     </dependency>
 
     <dependency>
-      <groupId>org.jmockit</groupId>
-      <artifactId>jmockit</artifactId>
-      <scope>test</scope>
-    </dependency>
-
-    <dependency>
       <groupId>org.assertj</groupId>
       <artifactId>assertj-core</artifactId>
       <scope>test</scope>
diff --git a/catalog-model/src/main/java/org/openecomp/sdc/be/model/GroupInstance.java b/catalog-model/src/main/java/org/openecomp/sdc/be/model/GroupInstance.java
index 386febdf86..331b407328 100644
--- a/catalog-model/src/main/java/org/openecomp/sdc/be/model/GroupInstance.java
+++ b/catalog-model/src/main/java/org/openecomp/sdc/be/model/GroupInstance.java
@@ -40,7 +40,7 @@ public class GroupInstance extends GroupInstanceDataDefinition {
         super();
     }
 
-    public GroupInstance(GroupInstanceDataDefinition r) {
+    public GroupInstance(final GroupInstanceDataDefinition r) {
         super(r);
     }
 
@@ -50,12 +50,11 @@ public class GroupInstance extends GroupInstanceDataDefinition {
      * @return
      */
     public List<GroupInstanceProperty> convertToGroupInstancesProperties() {
-        List<GroupInstanceProperty> groupInstancesProperties = null;
-        List<PropertyDataDefinition> propertiesList = super.getProperties();
-        if (propertiesList != null && !propertiesList.isEmpty()) {
-            groupInstancesProperties = propertiesList.stream().map(GroupInstanceProperty::new).collect(Collectors.toList());
+        final List<PropertyDataDefinition> propertiesList = super.getProperties();
+        if (CollectionUtils.isNotEmpty(propertiesList)) {
+            return propertiesList.stream().map(GroupInstanceProperty::new).collect(Collectors.toList());
         }
-        return groupInstancesProperties;
+        return null;
     }
 
     /**
@@ -64,23 +63,22 @@ public class GroupInstance extends GroupInstanceDataDefinition {
      *
      * @param groupInstancesProperties
      */
-    public void convertFromGroupInstancesProperties(List<GroupInstanceProperty> groupInstancesProperties) {
+    public void convertFromGroupInstancesProperties(final List<GroupInstanceProperty> groupInstancesProperties) {
         if (groupInstancesProperties != null && !groupInstancesProperties.isEmpty()) {
-            List<PropertyDataDefinition> propList = groupInstancesProperties.stream().map(PropertyDataDefinition::new).collect(Collectors.toList());
-            super.setProperties(propList);
+            super.setProperties(groupInstancesProperties.stream().map(PropertyDataDefinition::new).collect(Collectors.toList()));
         }
     }
 
     private void removeArtifactsDuplicates() {
-        List<String> artifacts = getArtifacts();
-        Set<String> artifactsSet = new HashSet<>();
+        final List<String> artifacts = getArtifacts();
+        final Set<String> artifactsSet = new HashSet<>();
         if (artifacts != null && !artifacts.isEmpty()) {
             artifactsSet.addAll(artifacts);
             artifacts.clear();
             artifacts.addAll(artifactsSet);
         }
-        List<String> giArtifacts = getGroupInstanceArtifacts();
-        Set<String> giArtifactsSet = new HashSet<>();
+        final List<String> giArtifacts = getGroupInstanceArtifacts();
+        final Set<String> giArtifactsSet = new HashSet<>();
         if (giArtifacts != null && !giArtifacts.isEmpty()) {
             giArtifactsSet.addAll(giArtifacts);
             giArtifacts.clear();
@@ -89,16 +87,16 @@ public class GroupInstance extends GroupInstanceDataDefinition {
     }
 
     private void clearArtifactsUuid() {
-        List<String> artifactsUuid = getArtifactsUuid();
+        final List<String> artifactsUuid = getArtifactsUuid();
         if (CollectionUtils.isNotEmpty(artifactsUuid)) {
             artifactsUuid.clear();
         } else if (artifactsUuid == null) {
             setArtifactsUuid(new ArrayList<>());
         }
-        List<String> giartifactsUuid = this.getGroupInstanceArtifactsUuid();
-        if (CollectionUtils.isNotEmpty(giartifactsUuid)) {
-            giartifactsUuid.clear();
-        } else if (giartifactsUuid == null) {
+        final List<String> groupInstanceArtifactsUuid = this.getGroupInstanceArtifactsUuid();
+        if (CollectionUtils.isNotEmpty(groupInstanceArtifactsUuid)) {
+            groupInstanceArtifactsUuid.clear();
+        } else if (groupInstanceArtifactsUuid == null) {
             setGroupInstanceArtifactsUuid(new ArrayList<>());
         }
     }
@@ -108,19 +106,19 @@ public class GroupInstance extends GroupInstanceDataDefinition {
      *
      * @param deploymentArtifacts
      */
-    public void alignArtifactsUuid(Map<String, ArtifactDefinition> deploymentArtifacts) {
-        List<String> artifactIds = getArtifacts();
+    public void alignArtifactsUuid(final Map<String, ArtifactDefinition> deploymentArtifacts) {
+        final List<String> artifactIds = getArtifacts();
         log.debug("the artifacts Id's are: {}, and the deployment artifacts Id's are: {}", artifactIds, deploymentArtifacts);
         if (CollectionUtils.isNotEmpty(artifactIds) && deploymentArtifacts != null) {
             removeArtifactsDuplicates();
             clearArtifactsUuid();
-            List<String> artifactUuids = getArtifactsUuid();
-            List<String> giArtifactUuids = getGroupInstanceArtifactsUuid();
-            for (String artifactId : artifactIds) {
-                String label = artifactId.substring(artifactId.lastIndexOf('.') + 1);
-                ArtifactDefinition artifact = deploymentArtifacts.get(label);
+            final List<String> artifactUuids = getArtifactsUuid();
+            final List<String> giArtifactUuids = getGroupInstanceArtifactsUuid();
+            for (final String artifactId : artifactIds) {
+                final var label = artifactId.substring(artifactId.lastIndexOf('.') + 1);
+                final ArtifactDefinition artifact = deploymentArtifacts.get(label);
                 log.debug("current artifact id: {}, current artifact definition: {}", artifactId, artifact);
-                ArtifactTypeEnum artifactType = ArtifactTypeEnum.parse(artifact.getArtifactType());
+                final var artifactType = ArtifactTypeEnum.parse(artifact.getArtifactType());
                 if (artifactType != ArtifactTypeEnum.HEAT_ENV) {
                     addArtifactsIdToCollection(artifactUuids, artifact);
                 } else {
@@ -130,8 +128,8 @@ public class GroupInstance extends GroupInstanceDataDefinition {
         }
     }
 
-    private void addArtifactsIdToCollection(List<String> artifactUuids, ArtifactDefinition artifact) {
-        if (!artifactUuids.contains(artifact.getArtifactUUID()) && StringUtils.isNotEmpty(artifact.getArtifactUUID())) {
+    private void addArtifactsIdToCollection(final List<String> artifactUuids, final ArtifactDefinition artifact) {
+        if (StringUtils.isNotEmpty(artifact.getArtifactUUID()) && !artifactUuids.contains(artifact.getArtifactUUID())) {
             artifactUuids.add(artifact.getArtifactUUID());
         }
     }
diff --git a/catalog-model/src/test/java/org/openecomp/sdc/be/model/GroupInstanceTest.java b/catalog-model/src/test/java/org/openecomp/sdc/be/model/GroupInstanceTest.java
index 8c306883bf..cda3bcfbef 100644
--- a/catalog-model/src/test/java/org/openecomp/sdc/be/model/GroupInstanceTest.java
+++ b/catalog-model/src/test/java/org/openecomp/sdc/be/model/GroupInstanceTest.java
@@ -7,9 +7,9 @@
  * 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.
@@ -20,107 +20,117 @@
 
 package org.openecomp.sdc.be.model;
 
-import mockit.Deencapsulation;
-import org.junit.Test;
-import org.openecomp.sdc.be.datatypes.elements.GroupInstanceDataDefinition;
-import org.openecomp.sdc.be.datatypes.elements.PropertyDataDefinition;
-
+import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.LinkedList;
 import java.util.List;
 import java.util.Map;
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.Test;
+import org.openecomp.sdc.be.datatypes.elements.GroupInstanceDataDefinition;
+import org.openecomp.sdc.be.datatypes.elements.PropertyDataDefinition;
+
+class GroupInstanceTest {
+
+    private GroupInstance createTestSubject() {
+        return new GroupInstance();
+    }
+
+    @Test
+    void testCtor() throws Exception {
+        Assertions.assertNotNull(new GroupInstance(new GroupInstanceDataDefinition()));
+    }
+
+    @Test
+    void testConvertToGroupInstancesProperties() throws Exception {
+        final GroupInstance testSubject = createTestSubject();
+        List<GroupInstanceProperty> result;
+
+        result = testSubject.convertToGroupInstancesProperties();
+        Assertions.assertNull(result);
+
+        final List<PropertyDataDefinition> properties = new LinkedList<>();
+        properties.add(new PropertyDataDefinition());
+        testSubject.setProperties(properties);
+        result = testSubject.convertToGroupInstancesProperties();
+        Assertions.assertNotNull(result);
+        Assertions.assertFalse(result.isEmpty());
+    }
+
+    @Test
+    void testConvertFromGroupInstancesProperties() throws Exception {
+        final GroupInstance testSubject = createTestSubject();
+        List<GroupInstanceProperty> groupInstancesProperties = null;
+
+        groupInstancesProperties = null;
+        testSubject.convertFromGroupInstancesProperties(groupInstancesProperties);
+        List<PropertyDataDefinition> properties = testSubject.getProperties();
+        Assertions.assertNull(properties);
+
+        groupInstancesProperties = new LinkedList<>();
+        groupInstancesProperties.add(new GroupInstanceProperty());
+        testSubject.convertFromGroupInstancesProperties(groupInstancesProperties);
+        properties = testSubject.getProperties();
+        Assertions.assertNotNull(properties);
+        Assertions.assertFalse(properties.isEmpty());
+    }
+
+    @Test
+    void testAlignArtifactsUuid() throws Exception {
+        final GroupInstance testSubject = createTestSubject();
+        Map<String, ArtifactDefinition> deploymentArtifacts = null;
+
+        testSubject.alignArtifactsUuid(deploymentArtifacts);
+        Assertions.assertNull(testSubject.getArtifacts());
+
+        deploymentArtifacts = new HashMap<>();
+
+        final List<String> artifacts = new ArrayList<>();
+        artifacts.add("mock1");
+        testSubject.setArtifacts(artifacts);
+
+        final List<String> artifactUuids = new ArrayList<>();
+        artifactUuids.add("mock1");
+        testSubject.setArtifactsUuid(artifactUuids);
+        final var artifactDefinition = new ArtifactDefinition();
+        artifactDefinition.setArtifactUUID("mock1");
+        deploymentArtifacts.put("mock1", artifactDefinition);
+        testSubject.alignArtifactsUuid(deploymentArtifacts);
+        List<String> groupInstanceArtifactsUuid = testSubject.getGroupInstanceArtifactsUuid();
+        Assertions.assertNotNull(groupInstanceArtifactsUuid);
+        Assertions.assertTrue(groupInstanceArtifactsUuid.isEmpty());
+
+        artifactDefinition.setArtifactUUID("mock2");
+        artifactDefinition.setArtifactType("HEAT_ENV");
+        deploymentArtifacts.put("mock2", artifactDefinition);
+        testSubject.alignArtifactsUuid(deploymentArtifacts);
+        groupInstanceArtifactsUuid = testSubject.getGroupInstanceArtifactsUuid();
+        Assertions.assertNotNull(groupInstanceArtifactsUuid);
+        Assertions.assertFalse(groupInstanceArtifactsUuid.isEmpty());
+        List<String> artifactsUuid = testSubject.getArtifactsUuid();
+        Assertions.assertNotNull(artifactsUuid);
+        Assertions.assertTrue(artifactsUuid.isEmpty());
+
+        testSubject.setArtifactsUuid(null);
+        testSubject.alignArtifactsUuid(deploymentArtifacts);
+        artifactsUuid = testSubject.getArtifactsUuid();
+        Assertions.assertNotNull(artifactsUuid);
+        Assertions.assertTrue(artifactsUuid.isEmpty());
+
+        List<String> groupInstanceArtifacts = new ArrayList<>();
+        groupInstanceArtifacts.add("mock1");
+        testSubject.setGroupInstanceArtifacts(groupInstanceArtifacts);
+        testSubject.setGroupInstanceArtifactsUuid(null);
+        testSubject.alignArtifactsUuid(deploymentArtifacts);
+        groupInstanceArtifactsUuid = testSubject.getGroupInstanceArtifactsUuid();
+        Assertions.assertNotNull(groupInstanceArtifactsUuid);
+        Assertions.assertFalse(groupInstanceArtifactsUuid.isEmpty());
+
+        testSubject.setGroupInstanceArtifactsUuid(groupInstanceArtifacts);
+        testSubject.alignArtifactsUuid(deploymentArtifacts);
+        groupInstanceArtifactsUuid = testSubject.getGroupInstanceArtifactsUuid();
+        Assertions.assertNotNull(groupInstanceArtifactsUuid);
+        Assertions.assertFalse(groupInstanceArtifactsUuid.isEmpty());
+    }
 
-public class GroupInstanceTest {
-
-	private GroupInstance createTestSubject() {
-		return new GroupInstance();
-	}
-
-	@Test
-	public void testCtor() throws Exception {
-		new GroupInstance(new GroupInstanceDataDefinition());
-	}
-	
-	@Test
-	public void testConvertToGroupInstancesProperties() throws Exception {
-		GroupInstance testSubject;
-		List<GroupInstanceProperty> result;
-
-		// default test
-		testSubject = createTestSubject();
-		result = testSubject.convertToGroupInstancesProperties();
-		List<PropertyDataDefinition> properties = new LinkedList<>();
-		properties.add(new PropertyDataDefinition());
-		testSubject.setProperties(properties);
-		result = testSubject.convertToGroupInstancesProperties();
-	}
-
-	@Test
-	public void testConvertFromGroupInstancesProperties() throws Exception {
-		GroupInstance testSubject;
-		List<GroupInstanceProperty> groupInstancesProperties = null;
-
-		// test 1
-		testSubject = createTestSubject();
-		groupInstancesProperties = null;
-		testSubject.convertFromGroupInstancesProperties(groupInstancesProperties);
-		groupInstancesProperties = new LinkedList<>();
-		groupInstancesProperties.add(new GroupInstanceProperty());
-		testSubject.convertFromGroupInstancesProperties(groupInstancesProperties);
-	}
-
-	@Test
-	public void testRemoveArtifactsDuplicates() throws Exception {
-		GroupInstance testSubject;
-
-		// default test
-		testSubject = createTestSubject();
-		Deencapsulation.invoke(testSubject, "removeArtifactsDuplicates");
-		LinkedList<String> artifacts = new LinkedList<>();
-		artifacts.add("mock");
-		testSubject.setArtifacts(artifacts);
-		LinkedList<String> groupInstanceArtifacts = new LinkedList<>();
-		groupInstanceArtifacts.add("mock");
-		testSubject.setGroupInstanceArtifacts(groupInstanceArtifacts);
-		Deencapsulation.invoke(testSubject, "removeArtifactsDuplicates");
-	}
-
-	@Test
-	public void testClearArtifactsUuid() throws Exception {
-		GroupInstance testSubject;
-
-		// default test
-		testSubject = createTestSubject();
-		Deencapsulation.invoke(testSubject, "clearArtifactsUuid");
-	}
-
-	@Test
-	public void testAlignArtifactsUuid() throws Exception {
-		GroupInstance testSubject;
-		Map<String, ArtifactDefinition> deploymentArtifacts = null;
-
-		// default test
-		testSubject = createTestSubject();
-		testSubject.alignArtifactsUuid(deploymentArtifacts);
-		LinkedList<String> artifacts = new LinkedList<>();
-		artifacts.add("mock");
-		testSubject.setArtifacts(artifacts);
-		testSubject.alignArtifactsUuid(deploymentArtifacts);
-		deploymentArtifacts = new HashMap<>();
-		deploymentArtifacts.put("mock", new ArtifactDefinition());
-		testSubject.alignArtifactsUuid(deploymentArtifacts);
-	}
-
-	@Test
-	public void testAddArtifactsIdToCollection() throws Exception {
-		GroupInstance testSubject;
-		List<String> artifactUuids = new LinkedList<>();
-		ArtifactDefinition artifact = new ArtifactDefinition();
-
-		// default test
-		testSubject = createTestSubject();
-		Deencapsulation.invoke(testSubject, "addArtifactsIdToCollection", artifactUuids, artifact);
-		artifact.setArtifactUUID("mock");
-		Deencapsulation.invoke(testSubject, "addArtifactsIdToCollection", artifactUuids, artifact);
-	}
 }