Merge pull request #69 from ehrmann/required-on-objects

Only apply required validator to objects

Author: stevehu

src/main/java/com/networknt/schema/RequiredValidator.java

@@ -35,9 +35,8 @@ public RequiredValidator(String schemaPath, JsonNode schemaNode, JsonSchema pare
 
         super(schemaPath, schemaNode, parentSchema, ValidatorTypeCode.REQUIRED, validationContext);
         if (schemaNode.isArray()) {
-            int size = schemaNode.size();
-            for (int i = 0; i < size; i++) {
-                fieldNames.add(schemaNode.get(i).asText());
+            for (JsonNode fieldNme : schemaNode) {
+                fieldNames.add(fieldNme.asText());
             }
         }
 
@@ -47,6 +46,10 @@ public RequiredValidator(String schemaPath, JsonNode schemaNode, JsonSchema pare
     public Set<ValidationMessage> validate(JsonNode node, JsonNode rootNode, String at) {
         debug(logger, node, rootNode, at);
 
+        if (!node.isObject()) {
+            return Collections.emptySet();
+        }
+
         Set<ValidationMessage> errors = new LinkedHashSet<ValidationMessage>();
 
         for (String fieldName : fieldNames) {

src/test/resources/tests/required.json

@@ -35,5 +35,18 @@
                 "valid": true
             }
         ]
+    },
+    {
+        "description": "required validation on non-object",
+        "schema": {
+            "required": ["foo"]
+        },
+        "tests": [
+            {
+                "description": "required not applied on null",
+                "data": null,
+                "valid": true
+            }
+        ]
     }
 ]