fix: trim whitespace from fetchSpec

Fix: #35

PR-URL: #59
Credit: @isaacs
Close: #59
Reviewed-by: @wraithgar

npa.js

@@ -283,7 +283,7 @@ function fromAlias (res, where) {
 
 function fromRegistry (res) {
   res.registry = true
-  const spec = res.rawSpec === '' ? 'latest' : res.rawSpec
+  const spec = res.rawSpec === '' ? 'latest' : res.rawSpec.trim()
   // no save spec for registry components as we save based on the fetched
   // version, not on the argument so this can't compute that.
   res.saveSpec = null

test/basic.js

@@ -489,6 +489,46 @@ require('tap').test('basic', function (t) {
       fetchSpec: 'latest',
       raw: 'foo',
     },
+
+    'foo@ 1.2 ': {
+      name: 'foo',
+      escapedName: 'foo',
+      type: 'range',
+      saveSpec: null,
+      fetchSpec: '1.2',
+      raw: 'foo@ 1.2 ',
+      rawSpec: ' 1.2 ',
+    },
+
+    'foo@ 1.2.3 ': {
+      name: 'foo',
+      escapedName: 'foo',
+      type: 'version',
+      saveSpec: null,
+      fetchSpec: '1.2.3',
+      raw: 'foo@ 1.2.3 ',
+      rawSpec: ' 1.2.3 ',
+    },
+
+    '[email protected] ': {
+      name: 'foo',
+      escapedName: 'foo',
+      type: 'version',
+      saveSpec: null,
+      fetchSpec: '1.2.3',
+      raw: '[email protected] ',
+      rawSpec: '1.2.3 ',
+    },
+
+    'foo@ 1.2.3': {
+      name: 'foo',
+      escapedName: 'foo',
+      type: 'version',
+      saveSpec: null,
+      fetchSpec: '1.2.3',
+      raw: 'foo@ 1.2.3',
+      rawSpec: ' 1.2.3',
+    },
   }
 
   Object.keys(tests).forEach(function (arg) {