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,6 +1,6 @@
 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
   }
@@ -16,6 +16,9 @@ const parse = (version, options) => {
   try {
     return new SemVer(version, options)
   } catch (er) {
+    if (throwErrors) {
+      throw er
+    }
     return null
   }
 }

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,13 @@ 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.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')