#326 property names with pattern (#374)

* #326: Pattern validation for propertyNames

* #326: Adjust test case descriptions.

Co-authored-by: leif.rilbe <[email protected]>

Author: LeifRilbe

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

@@ -20,6 +20,7 @@
 import org.slf4j.LoggerFactory;
 
 import java.util.*;
+import java.util.regex.Pattern;
 
 public class PropertyNamesValidator extends BaseJsonValidator implements JsonValidator {
     private static final Logger logger = LoggerFactory.getLogger(PropertyNamesValidator.class);
@@ -58,6 +59,9 @@ public Set<ValidationMessage> validate(JsonNode node, JsonNode rootNode, String
                     if("minLength".equals(entry.getKey()) && pname.length() < entry.getValue().getSchemaNode().intValue()) {
                         errors.add(buildValidationMessage(at + "." + pname, "minLength"));
                     }
+                    if("pattern".equals(entry.getKey()) && !Pattern.matches(entry.getValue().getSchemaNode().textValue(),pname)) {
+                        errors.add(buildValidationMessage(at + "." + pname, "pattern"));
+                    }
                 }
             }
         } else {

src/test/resources/draft2019-09/propertyNames.json

@@ -50,6 +50,35 @@
       }
     ]
   },
+  {
+      "description": "propertyNames validation with pattern",
+      "schema": {
+          "propertyNames": { "pattern": "^a+$" }
+      },
+      "tests": [
+          {
+              "description": "matching property names are valid",
+              "data": {
+                  "a": {},
+                  "aa": {},
+                  "aaa": {}
+              },
+              "valid": true
+          },
+          {
+              "description": "non-matching property name is invalid",
+              "data": {
+                  "aaA": {}
+              },
+              "valid": false
+          },
+          {
+              "description": "object without properties is valid",
+              "data": {},
+              "valid": true
+          }
+      ]
+  },
   {
     "description": "propertyNames with boolean schema true",
     "schema": {

src/test/resources/draft6/propertyNames.json

@@ -50,6 +50,35 @@
       }
     ]
   },
+  {
+      "description": "propertyNames validation with pattern",
+      "schema": {
+          "propertyNames": { "pattern": "^a+$" }
+      },
+      "tests": [
+          {
+              "description": "matching property names are valid",
+              "data": {
+                  "a": {},
+                  "aa": {},
+                  "aaa": {}
+              },
+              "valid": true
+          },
+          {
+              "description": "non-matching property name is invalid",
+              "data": {
+                  "aaA": {}
+              },
+              "valid": false
+          },
+          {
+              "description": "object without properties is valid",
+              "data": {},
+              "valid": true
+          }
+      ]
+  },
   {
     "description": "propertyNames with boolean schema true",
     "schema": {

src/test/resources/draft7/propertyNames.json

@@ -50,6 +50,35 @@
       }
     ]
   },
+  {
+      "description": "propertyNames validation with pattern",
+      "schema": {
+          "propertyNames": { "pattern": "^a+$" }
+      },
+      "tests": [
+          {
+              "description": "matching property names are valid",
+              "data": {
+                  "a": {},
+                  "aa": {},
+                  "aaa": {}
+              },
+              "valid": true
+          },
+          {
+              "description": "non-matching property name is invalid",
+              "data": {
+                  "aaA": {}
+              },
+              "valid": false
+          },
+          {
+              "description": "object without properties is valid",
+              "data": {},
+              "valid": true
+          }
+      ]
+  },
   {
     "description": "propertyNames with boolean schema true",
     "schema": {