Add checks to ensure bytes arguments provided to ByteUnitObject adaptors are valid

Author: miraclx

lib/index.js

@@ -183,22 +183,26 @@ class ByteUnitObject {
  }
 
  add(bytes) {
- this.checkInternalByteVal(bytes);
+ if (!isParsable(bytes)) throw Error('<bytes> argument must be a finite value');
+ this.checkInternalByteVal();
  return new ByteUnitObject(this.bytes + parseByteOrByteArray(bytes).reduce((a, v) => a + v, 0));
  }
 
  subtract(bytes) {
- this.checkInternalByteVal(bytes);
+ if (!isParsable(bytes)) throw Error('<bytes> argument must be a finite value');
+ this.checkInternalByteVal();
  return new ByteUnitObject(this.bytes - parseByteOrByteArray(bytes).reduce((a, v) => a + v, 0));
  }
 
  multiply(bytes) {
- this.checkInternalByteVal(bytes);
+ if (!isParsable(bytes)) throw Error('<bytes> argument must be a finite value');
+ this.checkInternalByteVal();
  return new ByteUnitObject(this.bytes * parseByteOrByteArray(bytes).reduce((a, v) => a * v, 1));
  }
 
  divide(bytes) {
- this.checkInternalByteVal(bytes);
+ if (!isParsable(bytes)) throw Error('<bytes> argument must be a finite value');
+ this.checkInternalByteVal();
  return new ByteUnitObject(this.bytes / parseByteOrByteArray(bytes).reduce((a, v) => a * v, 1));
  }