Fixes #110 Validation Error when using OneOf in OpenAPI specs

Author: ddobrin

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

@@ -130,7 +130,12 @@ public OneOfValidator(String schemaPath, JsonNode schemaNode, JsonSchema parentS
 
     public Set<ValidationMessage> validate(JsonNode node, JsonNode rootNode, String at) {
         debug(logger, node, rootNode, at);
-
+        
+        // this validator considers a missing node as an error
+        // set it here to true, however re-set it to its original value upon finishing the validation
+        boolean missingNodeAsError = config.isMissingNodeAsError();
+        config.setMissingNodeAsError(true);
+        
         int numberOfValidSchema = 0;
         Set<ValidationMessage> errors = new LinkedHashSet<ValidationMessage>();
 
@@ -171,6 +176,9 @@ public Set<ValidationMessage> validate(JsonNode node, JsonNode rootNode, String
             errors = Collections.singleton(buildValidationMessage(at, ""));
         }
 
+        // reset the flag for error handling
+        config.setMissingNodeAsError(missingNodeAsError);
+        
         return Collections.unmodifiableSet(errors);
     }
 

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

@@ -46,7 +46,11 @@ public Set<ValidationMessage> validate(JsonNode node, JsonNode rootNode, String
             JsonNode propertyNode = node.get(entry.getKey());
 
             if (propertyNode != null) {
-                errors.addAll(propertySchema.validate(propertyNode, rootNode, at + "." + entry.getKey()));
+                errors.addAll(propertySchema.validate(propertyNode, rootNode, at + "." + entry.getKey())); 
+            } else {
+            	// if a node could not be found, treat is as error/continue, depending on the SchemaValidatorsConfig
+            	if(config.isMissingNodeAsError())
+            		errors.add(buildValidationMessage(at, node.toString()));
             }
         }
 

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

@@ -5,7 +5,13 @@ public class SchemaValidatorsConfig {
      * when validate type, if TYPE_LOOSE = true, will try to convert string to different types to match the type defined in schema.
      */
     private boolean typeLoose;
-
+    
+    /**
+     * if IS_MISSING_NODE_AS_ERROR = true, the validator will ignore the missing node.
+     * if set to false, then the validator will report an error
+     */
+    private boolean missingNodeAsError = false;
+    
     public boolean isTypeLoose() {
         return typeLoose;
     }
@@ -14,6 +20,14 @@ public void setTypeLoose(boolean typeLoose) {
         this.typeLoose = typeLoose;
     }
 
+    public boolean isMissingNodeAsError() {
+    	return missingNodeAsError;
+    }
+    
+    public void setMissingNodeAsError(boolean missingNodeAsError) {
+    	this.missingNodeAsError = missingNodeAsError;
+    }
+    
     public SchemaValidatorsConfig() {
         loadDefaultConfig();
     }