fix: duplicate "type" error is reported with coerceTypes option, fixes …

…#469
ajv-validator · Apr 28, 2017 · 4ed756e · 4ed756e
1 parent 92bfeda
commit 4ed756e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/lib/dot/validate.jst b/lib/dot/validate.jst
@@ -141,7 +141,7 @@
         {{?}}
       {{? $rulesGroup.type }}
         }
-        {{? $typeSchema && $typeSchema === $rulesGroup.type }}
+        {{? $typeSchema && $typeSchema === $rulesGroup.type && !$coerceToTypes }}
           {{ var $typeChecked = true; }}
           else {
             {{
@@ -161,7 +161,7 @@
   {{~}}
 {{?}}
 
-{{? $typeSchema && !$typeChecked && !(it.opts.coerceTypes && $coerceToTypes) }}
+{{? $typeSchema && !$typeChecked && !$coerceToTypes }}
   {{# def.checkType }}
     {{# def.error:'type' }}
   }

diff --git a/spec/coercion.spec.js b/spec/coercion.spec.js
@@ -397,14 +397,14 @@ describe('Type coercion', function () {
   });
 
 
-  it.skip('should generate one error for type with coerceTypes option (issue #469)', function() {
+  it('should generate one error for type with coerceTypes option (issue #469)', function() {
     var schema = {
       "type": "number",
       "minimum": 10
     };
 
-    instances.forEach(function (ajv) {
-      var validate = ajv.compile(schema);
+    instances.forEach(function (_ajv) {
+      var validate = _ajv.compile(schema);
       validate(9). should.equal(false);
       validate.errors.length .should.equal(1);