Merge pull request #141 from canjs/fix-array-weirdness-in-error-message

Error message improvment for arrays

src/define.js

@@ -928,7 +928,7 @@ getDefinitionOrMethod = function(prop, value, defaultDefinition, typePrototype){
 			definition = { default: value, type: Function };
 		}
 	} else if( Array.isArray(value) ) {
-		definition = { type: value };
+		definition = { type: type.normalize(Array) };
 	} else if( canReflect.isPlainObject(value) ){
 		definition = value;
 	}

test/define-test.js

@@ -208,3 +208,21 @@ dev.devOnlyTest('On error include the name of the property that is being set', f
 		assert.equal(error.message, '4 is not of type Number. Property age is using "type: Number". Use "age: type.convert(Number)" to automatically convert values to Numbers when setting the "age" property.');
 	}
 });
+
+
+dev.devOnlyTest('Handle types as arrays to fix "Right-hand side of instanceof is not callable for arrays"', function(assert) {
+	class Foo extends mixinObject() {
+		static get props() {
+			return {
+				list: []
+			};
+		}
+	}
+	var foo = new Foo();
+	foo.list = [ "one", "two" ];
+	try {
+		assert.ok(true, 'No errors, it handles arrays values as type');
+	} catch (error) {
+		assert.ok(error.message);
+	}
+});