Fix return value for un-parsable strings

fixes #61

Author: dougwilson

History.md

@@ -1,3 +1,8 @@
+unreleased
+==========
+
+  * Fix return value for un-parsable strings
+
 3.1.1 / 2021-11-15
 ==================
 

index.js

@@ -162,5 +162,9 @@ function parse(val) {
     unit = results[4].toLowerCase();
   }
 
+  if (isNaN(floatValue)) {
+    return null;
+  }
+
   return Math.floor(map[unit] * floatValue);
 }

test/byte-parse.js

@@ -12,6 +12,7 @@ describe('Test byte parse function', function(){
     assert.strictEqual(bytes.parse(NaN), null);
     assert.strictEqual(bytes.parse(function(){}), null);
     assert.strictEqual(bytes.parse({}), null);
+    assert.strictEqual(bytes.parse('foobar'), null);
   });
 
   it('Should parse raw number', function(){

test/bytes.js

@@ -16,6 +16,7 @@ describe('Test constructor', function(){
     assert.strictEqual(bytes(NaN), null);
     assert.strictEqual(bytes(function(){}), null);
     assert.strictEqual(bytes({}), null);
+    assert.strictEqual(bytes('foobar'), null);
   });
 
   it('Should be able to parse a string into a number', function(){