linting and test fixes

Author: caseyjhol

index.ts

@@ -56,7 +56,10 @@ program
   .option('-x, --exclude <ids>', 'Exceptions or the vulnerabilities ID(s) to exclude.')
   .option('-m, --module-ignore <moduleNames>', 'Names of modules to ignore.')
   .option('-l, --level <auditLevel>', 'The minimum audit level to validate.')
-  .option('-f, --filter-table [level]', 'Filter table to show only vulnerabilities at or above specified level (defaults to audit level if no value provided).')
+  .option(
+    '-f, --filter-table [level]',
+    'Filter table to show only vulnerabilities at or above specified level (defaults to audit level if no value provided).',
+  )
   .option('-p, --production', 'Skip checking the devDependencies.')
   .option('-r, --registry <url>', 'The npm registry url to use.')
   .option('-i, --include-columns <columnName1>,<columnName2>,..,<columnNameN>', 'Columns to include in report.')

test/utils/color.test.ts

@@ -2,6 +2,21 @@ import { expect } from 'chai';
 import { color, getSeverityBgColor } from '../../src/utils/color';
 
 describe('Color utils', () => {
+  let originalNoColor: string | undefined;
+
+  before(() => {
+    originalNoColor = process.env.NO_COLOR;
+    delete process.env.NO_COLOR;
+  });
+
+  after(() => {
+    if (originalNoColor !== undefined) {
+      process.env.NO_COLOR = originalNoColor;
+    } else {
+      delete process.env.NO_COLOR;
+    }
+  });
+
   describe('#color', () => {
     it('should handle correctly without given colors specificed', () => {
       expect(color('message')).to.equal('message\u001b[0m');

test/utils/vulnerability.test.ts

@@ -21,6 +21,22 @@ import V7_JSON_BUFFER_EMPTY from '../__mocks__/v7-json-buffer-empty.json';
 import { v6Advisory, v7VulnerabilityVia } from '../../src/types';
 
 describe('Vulnerability utils', () => {
+  let originalNoColor: string | undefined;
+
+  before(() => {
+    // Ensure color codes are enabled to match fixtures
+    originalNoColor = process.env.NO_COLOR;
+    delete process.env.NO_COLOR;
+  });
+
+  after(() => {
+    if (originalNoColor !== undefined) {
+      process.env.NO_COLOR = originalNoColor;
+    } else {
+      delete process.env.NO_COLOR;
+    }
+  });
+
   describe('validateV6Vulnerability', () => {
     it('should be able to validate an exception by ID', () => {
       const v6Advisory: v6Advisory = {
@@ -152,6 +168,17 @@ describe('Vulnerability utils', () => {
   });
 
   describe('#processExceptions', () => {
+    let clock: sinon.SinonFakeTimers;
+
+    before(() => {
+      // Freeze "now" so date-based coloring (e.g., 2030 expiries) stays deterministic
+      clock = sinon.useFakeTimers(new Date('2024-01-01T00:00:00Z').getTime());
+    });
+
+    after(() => {
+      clock.restore();
+    });
+
     it('should be able to process exceptions correctly', () => {
       const cmdExceptions = ['1165', '1890'];
       const result = processExceptions(NSPRC, cmdExceptions);