Issue 12

jjve.js

@@ -1,6 +1,23 @@
 (function() {
   'use strict';
 
+  function isArray(obj) {
+    if (typeof Array.isArray === 'function') {
+      return Array.isArray(obj);
+    }
+    return Object.prototype.toString.call(obj) === '[object Array]';
+  }
+
+  function allowsType(schema, type) {
+    if (typeof schema.type === 'string') {
+      return schema.type === type;
+    }
+    if (isArray(schema.type)) {
+      return schema.type.indexOf(type) !== -1;
+    }
+    return false;
+  }
+
   function make(o) {
     var errors = [];
 
@@ -278,23 +295,6 @@
     return errors;
   }
 
-  function allowsType(schema, type) {
-    if (typeof schema.type === 'string') {
-      return schema.type === type;
-    }
-    if (isArray(schema.type)) {
-      return schema.type.indexOf(type) !== -1;
-    }
-    return false;
-  }
-
-  function isArray(obj) {
-    if (typeof Array.isArray === 'function') {
-      return Array.isArray(obj);
-    }
-    return Object.prototype.toString.call(obj) === '[object Array]';
-  }
-
   function formatPath(options) {
     var root = options.hasOwnProperty('root') ?
       options.root : '$';

test/errors.js

@@ -497,6 +497,43 @@ describe('jjve', function() {
       ]);
     });
 
+    it('should handle schema ref which is an array and has definition with required properties', function() {
+      var data = { one: [{}] };
+      var ref = {
+        id: 'one',
+        type: 'array',
+        items: { $ref: '#/definitions/rest' },
+          definitions: {
+            rest: {
+              properties: {
+                two: {
+                  type: 'boolean'
+                }
+              },
+              required: ['two']
+            }
+          }
+      };
+
+      var schema = {
+        type: 'object',
+        properties: {
+          one: { $ref: 'one' }
+        },
+      };
+
+      this.env.addSchema(ref);
+
+      this.run(schema, data).should.eql([
+        {
+          code: 'VALIDATION_MIN_LENGTH',
+          message: 'String is too short (0 chars), minimum 1',
+          data: '',
+          path: '$.one.two'
+        }
+      ]);
+    });
+
     it('should handle type property given as array where leaf schema is needed',
        function() {
       var data = { test: 'xyz' };