Do not throw when testing invalid version strings

Range.test() and Comparator.test() would throw if passed an invalid
SemVer string.  It's better in those cases to just treat it as not a
match.  (This is what npm does internally when using those functions,
wrapping in a try/catch.)

Author: isaacs

semver.js

@@ -810,7 +810,11 @@ Comparator.prototype.test = function (version) {
   }
 
   if (typeof version === 'string') {
-    version = new SemVer(version, this.options)
+    try {
+      version = new SemVer(version, this.options)
+    } catch (er) {
+      return false
+    }
   }
 
   return cmp(version, this.operator, this.semver, this.options)
@@ -1261,7 +1265,11 @@ Range.prototype.test = function (version) {
   }
 
   if (typeof version === 'string') {
-    version = new SemVer(version, this.options)
+    try {
+      version = new SemVer(version, this.options)
+    } catch (er) {
+      return false
+    }
   }
 
   for (var i = 0; i < this.set.length; i++) {

test/index.js

@@ -335,6 +335,9 @@ test('negative range tests', function (t) {
     ['^1.2.3', '2.0.0-pre'],
     ['0.x', undefined],
     ['*', undefined],
+    // invalid versions never satisfy, but shouldn't throw
+    ['*', 'not a version'],
+    ['>=2', 'glorp'],
   ].forEach(function (v) {
     var range = v[0]
     var ver = v[1]
@@ -1014,6 +1017,9 @@ test('comparator testing', function (t) {
   t.ok(c2.test('1.2.4'))
   var c3 = new Comparator(c, true)
   t.ok(c3.test('1.2.4'))
+  // test an invalid version, should not throw
+  var c4 = new Comparator(c)
+  t.notOk(c4.test('not a version string'))
   t.end()
 })