🐛 [Test] Fixed tests for KapuaEntityUniquenessException

Signed-off-by: Alberto Codutti <alberto.codutti@eurotech.com>
eclipse · May 17, 2024 · fb78627 · fb78627
1 parent ecb21f2
commit fb78627
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 42 deletions.
diff --git a/service/api/src/test/java/org/eclipse/kapua/KapuaEntityUniquenessExceptionTest.java b/service/api/src/test/java/org/eclipse/kapua/KapuaEntityUniquenessExceptionTest.java
@@ -17,66 +17,36 @@
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.experimental.categories.Category;
-import org.mockito.Mockito;
 
+import java.util.AbstractMap;
 import java.util.LinkedList;
 import java.util.List;
 import java.util.Map;
 
-
-
 @Category(JUnitTests.class)
 public class KapuaEntityUniquenessExceptionTest {
 
     String[] entityType;
     List<Map.Entry<String, Object>> uniquesFieldValues;
-    Map.Entry mapEntry;
 
     @Before
     public void initialize() {
-        entityType = new String[]{"Entity Type", null};
+        entityType = new String[] { "AEntity", "AnotherEntity" };
         uniquesFieldValues = new LinkedList<>();
-        mapEntry = Mockito.mock(Map.Entry.class);
-        uniquesFieldValues.add(mapEntry);
-        uniquesFieldValues.add(mapEntry);
-        uniquesFieldValues.add(null);
-    }
 
-    @Test
-    public void kapuaEntityUniquenessExceptionTest() {
-        for (String type : entityType) {
-            KapuaEntityUniquenessException kapuaEntityUniquenessException1 = new KapuaEntityUniquenessException(type, uniquesFieldValues);
-            Assert.assertEquals("Expected and actual values should be the same.", type, kapuaEntityUniquenessException1.getEntityType());
-            Assert.assertEquals("Expected and actual values should be the same.", uniquesFieldValues, kapuaEntityUniquenessException1.getUniquesFieldValues());
-            Assert.assertEquals("Expected and actual values should be the same.", KapuaErrorCodes.ENTITY_UNIQUENESS, kapuaEntityUniquenessException1.getCode());
-            Assert.assertEquals("Expected and actual values should be the same.", "Error: " + uniquesFieldValues, kapuaEntityUniquenessException1.getMessage());
-            Assert.assertNull("Null expected.", kapuaEntityUniquenessException1.getCause());
-        }
+        uniquesFieldValues.add(new AbstractMap.SimpleEntry<>("aField", "aValue"));
+        uniquesFieldValues.add(new AbstractMap.SimpleEntry<>("anotherField", "anotherValue"));
     }
 
     @Test
-    public void kapuaEntityUniquenessExceptionNullFieldValuesTest() {
-        for (String type : entityType) {
-            KapuaEntityUniquenessException kapuaEntityUniquenessException2 = new KapuaEntityUniquenessException(type, null);
-            Assert.assertEquals("Expected and actual values should be the same.", type, kapuaEntityUniquenessException2.getEntityType());
-            Assert.assertNull("Null expected.", kapuaEntityUniquenessException2.getUniquesFieldValues());
-            Assert.assertEquals("Expected and actual values should be the same.", KapuaErrorCodes.ENTITY_UNIQUENESS, kapuaEntityUniquenessException2.getCode());
-            Assert.assertEquals("Expected and actual values should be the same.", "Error: null", kapuaEntityUniquenessException2.getMessage());
-            Assert.assertNull("Null expected.", kapuaEntityUniquenessException2.getCause());
-        }
-    }
-
-    @Test(expected = KapuaEntityUniquenessException.class)
-    public void throwingExceptionTest() throws KapuaEntityUniquenessException {
-        for (String type : entityType) {
-            throw new KapuaEntityUniquenessException(type, uniquesFieldValues);
-        }
-    }
-
-    @Test(expected = KapuaEntityUniquenessException.class)
-    public void throwingExceptionNullFieldValuesTest() throws KapuaEntityUniquenessException {
+    public void kapuaEntityUniquenessExceptionTest() {
         for (String type : entityType) {
-            throw new KapuaEntityUniquenessException(type, null);
+            KapuaEntityUniquenessException kapuaEntityUniquenessException = new KapuaEntityUniquenessException(type, uniquesFieldValues);
+            Assert.assertEquals(type, kapuaEntityUniquenessException.getEntityType());
+            Assert.assertEquals(uniquesFieldValues, kapuaEntityUniquenessException.getUniquesFieldValues());
+            Assert.assertEquals(KapuaErrorCodes.ENTITY_UNIQUENESS, kapuaEntityUniquenessException.getCode());
+            Assert.assertEquals("Duplicate " + type + " entry for values: " + uniquesFieldValues, kapuaEntityUniquenessException.getMessage());
+            Assert.assertNull("Null expected.", kapuaEntityUniquenessException.getCause());
         }
     }
 }  
diff --git a/service/api/src/test/java/org/eclipse/kapua/KapuaIllegalArgumentExceptionTest.java b/service/api/src/test/java/org/eclipse/kapua/KapuaIllegalArgumentExceptionTest.java
@@ -79,7 +79,7 @@ public void kapuaIllegalArgumentExceptionKapuaErrorCodesStringParametersTest() {
                         "Error: " + name + ", " + value,
                         "An entity with the same name " + name + " already exists.",
                         "An entity with the same external Id " + name + " already exists.",
-                        "Error: " + name + ", " + value,
+                        "Duplicate " + name + " entry for values: " + value,
                         "The current subject is not authorized for " + name + ".",
                         "An illegal value was provided for the argument " + name + ": " + value + ".",
                         "An illegal null value was provided for the argument " + name + ".",