fix(license): don't fail when license is whitespace-only field (#93)

Fixes: pnpm/pnpm#1067

Author: zkochan

lib/fixer.js

@@ -299,7 +299,8 @@ var fixer = module.exports = {
     } else{
       if (
         typeof(data.license) !== 'string' ||
-        data.license.length < 1
+        data.license.length < 1 ||
+        data.license.trim() === ''
       ) {
         this.warn("invalidLicense")
       } else {

test/normalize.js

@@ -156,6 +156,25 @@ tap.test("license field should be a valid SPDX expression", function(t) {
   t.end()
 })
 
+tap.test("don't fail when license is just a space", function(t) {
+  var warnings = []
+  function warn(w) {
+    warnings.push(w)
+  }
+  var a
+  normalize(a={
+    license: ' '
+  }, warn)
+
+  var expect =
+    [ warningMessages.missingDescription,
+      warningMessages.missingRepository,
+      warningMessages.missingReadme,
+      warningMessages.invalidLicense]
+  t.same(warnings, expect)
+  t.end()
+})
+
 tap.test("gist bugs url", function(t) {
   var d = {
     repository: "[email protected]:123456.git"