fix(exit-code): account for null default (#46)

lib/index.js

@@ -15,9 +15,11 @@ module.exports = Object.assign((data, options = {}) => {
     color = true,
     unicode = true,
     indent = 2,
-    auditLevel = 'low'
   } = options
 
+  // CLI defaults this to `null` so the defaulting method above doesn't work
+  const auditLevel = options.auditLevel || 'low'
+
   if (!data)
     throw Object.assign(
       new TypeError('ENOAUDITDATA'),

test/index.js

@@ -9,6 +9,15 @@ t.equal(nar.reporters.json, require('../lib/reporters/json.js'))
 t.equal(nar.reporters.quiet, require('../lib/reporters/quiet.js'))
 
 const metadata = { vulnerabilities: {} }
+const highMeta = {
+  vulnerabilities: {
+    high: 99,
+    low: 0,
+    critical: 0,
+    total: 99
+  }
+}
+
 const fake = { reporter: 'fake' }
 t.strictSame(nar({ foo: 'bar', metadata }, fake), {
   exitCode: 0,
@@ -34,6 +43,18 @@ t.strictSame(nar({ foo: 'bar', toJSON: () => ({ bar: 'baz', metadata }) }, fake)
   }
 }, 'should call toJSON')
 
+t.strictSame(nar({ foo: 'bar', auditLevel: null, metadata: highMeta }, fake), {
+  exitCode: 1,
+  report: {
+    data: { foo: 'bar', auditLevel: null, metadata: highMeta },
+    options: {
+      color: true,
+      unicode: true,
+      indent: 2,
+    },
+  }
+}, 'null auditLevel')
+
 t.test('install is default reporter', async t => {
   const fix = fixture('one-vuln')
   t.strictSame(nar(fix).report, nar.reporters.install(fix, {