Copy constructor checker didn't know about Double[]

SAFEHR-data · Jul 3, 2024 · 182d2c2 · 182d2c2
1 parent 7b9b01a
commit 182d2c2
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/emap-star/emap-star/src/test/java/uk/ac/ucl/rits/inform/informdb/TestCopyConstructor.java b/emap-star/emap-star/src/test/java/uk/ac/ucl/rits/inform/informdb/TestCopyConstructor.java
@@ -32,6 +32,7 @@ public class TestCopyConstructor {
             "boolean",
             "java.lang.Boolean",
             "java.lang.Double",
+            "java.lang.Double[]",
             "java.lang.Long",
             "java.lang.String",
             "java.time.Instant",
@@ -134,7 +135,7 @@ Class<?> classFromName(String className) {
 
         } catch (ClassNotFoundException e) {
             e.printStackTrace();
-            throw new RuntimeException("Failed to generate an instance of " + className);
+            throw new RuntimeException("Failed to generate an instance of " + className, e);
         }
     }
 
@@ -149,7 +150,7 @@ Object defaultConstructedInstance(Class<?> entity) {
         } catch (InvocationTargetException | InstantiationException
                  | IllegalAccessException | NoSuchMethodException e) {
             e.printStackTrace();
-            throw new RuntimeException("Failed to generate an instance of " + entity.getName());
+            throw new RuntimeException("Failed to generate an instance of " + entity.getName(), e);
         }
     }
 
@@ -176,6 +177,8 @@ Object newBaseInstance(String className) {
                 return Instant.now();
             case "java.lang.Double":
                 return (double) INTEGERS.get(index);
+            case "java.lang.Double[]":
+                return new Double[]{(double) INTEGERS.get(index)};
             case "long":
             case "java.lang.Long":
                 return (long) INTEGERS.get(index);
@@ -189,7 +192,7 @@ Object newBaseInstance(String className) {
                 return List.of();
         }
 
-        throw new StringIndexOutOfBoundsException("Did not find " + className + " in base types");
+        throw new IllegalArgumentException("Need to add an implementation for initialising object of type: " + className);
     }
 
     /**
@@ -224,7 +227,7 @@ void setAllFieldsOf(Class<?> entity, Object instance) {
         for (Method method : setterMethodsOf(entity)) {
 
             Object[] params = Arrays.stream(method.getParameterTypes())
-                    .map(p -> newInstance(p.getName()))
+                    .map(p -> newInstance(p.getTypeName()))
                     .toArray();
 
             try {