Merge pull request #863 from epoberezkin/fix-861-property-names

Fix #861: propertyNames with empty schema
ajv-validator · Sep 23, 2018 · 5c41a84 · 5c41a84
2 parents 12e1655 + c1f929b
commit 5c41a84
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 2 deletions.
diff --git a/lib/dot/propertyNames.jst b/lib/dot/propertyNames.jst
@@ -3,6 +3,8 @@
 {{# def.setupKeyword }}
 {{# def.setupNextLevel }}
 
+var {{=$errs}} = errors;
+
 {{? {{# def.nonEmptySchema:$schema }} }}
   {{
     $it.schema = $schema;
@@ -22,8 +24,6 @@
       , $currentBaseId = it.baseId;
   }}
 
-  var {{=$errs}} = errors;
-
   {{? $ownProperties }}
     var {{=$dataProperties}} = undefined;
   {{?}}

diff --git a/spec/tests/issues/861_empty_propertynames.json b/spec/tests/issues/861_empty_propertynames.json
@@ -0,0 +1,23 @@
+[
+  {
+    "description": "propertyNames with empty schema (#861)",
+    "schema": {
+      "properties": {
+        "foo": {"type": "string"}
+      },
+      "propertyNames": {}
+    },
+    "tests": [
+      {
+        "description": "valid",
+        "data": {"foo": "bar"},
+        "valid": true
+      },
+      {
+        "description": "invalid",
+        "data": {"foo": 1},
+        "valid": false
+      }
+    ]
+  }
+]