Fix patternProperties annotation (#955)

networknt · Feb 6, 2024 · 937105f · 937105f
1 parent fb40864
commit 937105f
Show file tree

Hide file tree

Showing 2 changed files with 50 additions and 2 deletions.
diff --git a/src/main/java/com/networknt/schema/PatternPropertiesValidator.java b/src/main/java/com/networknt/schema/PatternPropertiesValidator.java
@@ -86,7 +86,10 @@ public Set<ValidationMessage> validate(ExecutionContext executionContext, JsonNo
             executionContext.getAnnotations()
                     .put(JsonNodeAnnotation.builder().instanceLocation(instanceLocation)
                             .evaluationPath(this.evaluationPath).schemaLocation(this.schemaLocation)
-                            .keyword(getKeyword()).value(matchedInstancePropertyNames).build());
+                            .keyword(getKeyword())
+                            .value(matchedInstancePropertyNames != null ? matchedInstancePropertyNames
+                                    : Collections.emptySet())
+                            .build());
         }
         return errors == null ? Collections.emptySet() : Collections.unmodifiableSet(errors);
     }

diff --git a/src/test/java/com/networknt/schema/PatternPropertiesValidatorTest.java b/src/test/java/com/networknt/schema/PatternPropertiesValidatorTest.java
@@ -18,14 +18,17 @@
 
 import com.fasterxml.jackson.databind.JsonNode;
 import com.networknt.schema.SpecVersion.VersionFlag;
+import com.networknt.schema.output.OutputUnit;
 
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.Test;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
+import java.util.HashSet;
 import java.util.Set;
 
 /**
@@ -108,7 +111,49 @@ void message() {
         assertEquals("999", message.getInstanceNode().toString());
         assertEquals("/valid_array/0: integer found, string expected", message.getMessage());
         assertNull(message.getProperty());
+    }
 
-
+    @SuppressWarnings("unchecked")
+    @Test
+    void annotation() {
+        String schemaData = "{\n"
+                + "  \"$id\": \"https://www.example.org/schema\",\n"
+                + "  \"type\": \"object\",\n"
+                + "  \"patternProperties\": {\n"
+                + "    \"^valid_\": {\n"
+                + "      \"type\": [\"array\", \"string\"],\n"
+                + "      \"items\": {\n"
+                + "        \"type\": \"string\"\n"
+                + "      }\n"
+                + "    }\n"
+                + "  }\n"
+                + "}";
+        JsonSchemaFactory factory = JsonSchemaFactory.getInstance(VersionFlag.V202012);
+        SchemaValidatorsConfig config = new SchemaValidatorsConfig();
+        config.setPathType(PathType.JSON_POINTER);
+        JsonSchema schema = factory.getSchema(schemaData, config);
+        String inputData = "{\n"
+                + "  \"test\": 5\n"
+                + "}";
+        OutputUnit outputUnit = schema.validate(inputData, InputFormat.JSON, OutputFormat.HIERARCHICAL, executionContext -> {
+            executionContext.getExecutionConfig().setAnnotationCollectionEnabled(true);
+            executionContext.getExecutionConfig().setAnnotationCollectionFilter(keyword -> true);
+        });
+        Set<String> patternProperties = (Set<String>) outputUnit.getAnnotations().get("patternProperties");
+        assertTrue(patternProperties.isEmpty());
+
+        inputData = "{\n"
+                + "  \"valid_array\": [\"999\", \"2\"],\n"
+                + "  \"valid_string\": \"string_value\""
+                + "}";
+        outputUnit = schema.validate(inputData, InputFormat.JSON, OutputFormat.HIERARCHICAL, executionContext -> {
+            executionContext.getExecutionConfig().setAnnotationCollectionEnabled(true);
+            executionContext.getExecutionConfig().setAnnotationCollectionFilter(keyword -> true);
+        });
+        patternProperties = (Set<String>) outputUnit.getAnnotations().get("patternProperties");
+        Set<String> all = new HashSet<>();
+        all.add("valid_array");
+        all.add("valid_string");
+        assertTrue(patternProperties.containsAll(patternProperties));
     }
 }