Fix NaN incorrectly passing validation as a number.

apiaryio · Jun 2, 2015 · 8610045 · 8610045
1 parent 49460c7
commit 8610045
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/utils/isNumber.js b/src/utils/isNumber.js
@@ -4,5 +4,5 @@
  * @param {object} input
  */
 var isNumber = function(input) {
-  return typeof input === 'number';
+  return typeof input === 'number' && !isNaN(input);
 };
diff --git a/tests/json/attributes/type/number.js b/tests/json/attributes/type/number.js
@@ -55,6 +55,7 @@ suite('JSON/Attribute/type#number', function() {
       true,
       {},
       [],
+      NaN,
       function() {}
     ].forEach(function(instance) {
       Validator.validate(instance, schema, function(error) {
@@ -64,8 +65,8 @@ suite('JSON/Attribute/type#number', function() {
     });
   });
 
-  test('should run 21 times', function() {
-    expect(count).to.be.eql(21);
+  test('should run 22 times', function() {
+    expect(count).to.be.eql(22);
   });
 
-});
+});