WIP: Only convert to array types when it is possible

Author: gselzer

src/main/java/org/scijava/convert/DefaultConverter.java

@@ -292,7 +292,11 @@ private Collection<Object> createCollection(final Class<?> type) {
 	public boolean canConvert(final Class<?> src, final Type dest) {
 
 		// Handle array types, including generic array types.
-		if (isArray(dest)) return true;
+		if (isArray(dest)){
+			if (Collection.class.isAssignableFrom(src)) return true;
+			if (src.isArray()) return true;
+			return Types.isAssignable(src, Types.component(dest));
+		}
 
 		// Handle parameterized collection types.
 		if (dest instanceof ParameterizedType && isCollection(dest) &&

src/test/java/org/scijava/convert/ConvertServiceTest.java

@@ -457,7 +457,7 @@ class Struct {
 	/**
 	 * Tests setting an incompatible element value for a primitive array.
 	 */
-	@Test(expected = IllegalArgumentException.class)
+	@Test
 	public void testBadPrimitiveArray() {
 		class Struct {
 
@@ -467,6 +467,7 @@ class Struct {
 		final Struct struct = new Struct();
 
 		setFieldValue(struct, "intArray", "not an int array");
+		assertEquals(null, struct.intArray);
 	}
 
 	/**
@@ -486,7 +487,7 @@ class Struct {
 
 		// Test abnormal behavior for an object array
 		setFieldValue(struct, "doubleArray", "not a double array");
-		assertEquals(null, struct.doubleArray[0]);
+		assertEquals(null, struct.doubleArray);
 
 		// Test abnormal behavior for a list
 		setFieldValue(struct, "nestedArray", "definitely not a set of char arrays");