fix: Added support for the UNLICENSED NPM magic value (fixes #113) (#134

)

* added handling of the UNLICENSED edge case

* added tests for UNLICENSED being non-green

src/checker.ts

@@ -196,6 +196,12 @@ export class LicenseChecker extends EventEmitter {
   }
 
   private correctLicenseName(license: string): string | null {
+    // NPM specific value.
+    if (license === 'UNLICENSED' || license === 'UNLICENCED') {
+      console.warn(`Unlicensed package, specified license: ${license}`);
+      return 'UNLICENSED';
+    }
+
     const corrected = spdxCorrect(license);
     if (this.opts.verbose && corrected && corrected !== license) {
       console.warn(`Correcting ${license} to ${corrected}`);

test/checker-test.ts

@@ -446,4 +446,41 @@ describe(__filename, () => {
       }
     );
   });
+
+  it('treats UNLICENSED as non-green', () => {
+    const primaryPackageJson = JSON.stringify({
+      name: 'hello',
+      version: '1.0.0',
+      license: 'UNLICENSED',
+    });
+
+    const pathToPrimary = path.join('path', 'to', 'primary');
+    return withFixtures(
+      {
+        [pathToPrimary]: {
+          'package.json': primaryPackageJson,
+        },
+      },
+      async () => {
+        requestedPackages = [];
+        const nonGreenPackages: string[] = [];
+        const packageJsonPaths: string[] = [];
+        const checker = new LicenseChecker();
+        checker
+          .on('non-green-license', arg => {
+            nonGreenPackages.push(`${arg.packageName}@${arg.version}`);
+          })
+          .on('package.json', filePath => {
+            packageJsonPaths.push(filePath);
+          });
+        await checker.checkLocalDirectory(pathToPrimary);
+        console.log('requested packages: ', requestedPackages);
+        assert.deepStrictEqual(requestedPackages, []);
+        assert.deepStrictEqual(nonGreenPackages, ['hello@1.0.0']);
+        assert.deepStrictEqual(packageJsonPaths, [
+          path.join(pathToPrimary, 'package.json'),
+        ]);
+      }
+    );
+  });
 });