fix: throw on bad version

Author: wraithgar

functions/diff.js

@@ -1,8 +1,8 @@
-const parse = require('./parse')
+const parse = require('./parse.js')
 
 const diff = (version1, version2) => {
-  const v1 = parse(version1)
-  const v2 = parse(version2)
+  const v1 = parse(version1, null, true)
+  const v2 = parse(version2, null, true)
   const comparison = v1.compare(v2)
 
   if (comparison === 0) {

functions/parse.js

@@ -1,23 +1,25 @@
 const { MAX_LENGTH } = require('../internal/constants')
 const SemVer = require('../classes/semver')
-const parse = (version, options) => {
+const parse = (version, options, throwErrors = false) => {
   if (version instanceof SemVer) {
     return version
   }
+  if (!throwErrors) {
+    if (typeof version !== 'string') {
+      return null
+    }
 
-  if (typeof version !== 'string') {
-    return null
-  }
-
-  if (version.length > MAX_LENGTH) {
-    return null
-  }
+    if (version.length > MAX_LENGTH) {
+      return null
+    }
 
-  try {
-    return new SemVer(version, options)
-  } catch (er) {
-    return null
+    try {
+      return new SemVer(version, options)
+    } catch (er) {
+      return null
+    }
   }
+  return new SemVer(version, options)
 }
 
 module.exports = parse

test/functions/diff.js

@@ -43,3 +43,13 @@ test('diff versions test', (t) => {
 
   t.end()
 })
+
+test('throws on bad version', (t) => {
+  t.throws(() => {
+    diff('bad', '1.2.3')
+  }, {
+    message: 'Invalid Version: bad',
+    name: 'TypeError',
+  })
+  t.end()
+})

test/functions/parse.js

@@ -9,6 +9,16 @@ t.test('returns null instead of throwing when presented with garbage', t => {
     t.equal(parse(v, opts), null, msg))
 })
 
+t.test('throw errors if asked to', t => {
+  t.throws(() => {
+    parse('bad', null, true)
+  })
+  t.throws(() => {
+    parse([], null, true)
+  })
+  t.end()
+})
+
 t.test('parse a version into a SemVer object', t => {
   t.match(parse('1.2.3'), new SemVer('1.2.3'))
   const s = new SemVer('4.5.6')