chore: test coverage

Author: Chrilleweb

test/unit/ui/scan/printMissing.test.ts

@@ -1,5 +1,4 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import chalk from 'chalk';
 import { printMissing } from '../../../../src/ui/scan/printMissing.js';
 import type { EnvUsage } from '../../../../src/config/types.js';
 
@@ -21,6 +20,7 @@ describe('printMissing', () => {
 
   it('returns false when no missing variables', () => {
     const result = printMissing([], [], '.env');
+
     expect(result).toBe(false);
     expect(logSpy).not.toHaveBeenCalled();
   });
@@ -29,6 +29,70 @@ describe('printMissing', () => {
     const used: EnvUsage[] = [];
 
     const result = printMissing(['UNKNOWN'], used, '.env');
+
+    expect(result).toBe(true);
+    expect(logSpy).toHaveBeenCalledTimes(4);
+    expect(logSpy).toHaveBeenNthCalledWith(
+      2,
+      expect.stringContaining('Missing in .env'),
+    );
+  });
+
+  it('prints grouped missing usages and falls back to environment file label', () => {
+    const used: EnvUsage[] = [
+      {
+        variable: 'API_KEY',
+        file: 'src/a.ts',
+        line: 10,
+        column: 1,
+        pattern: 'process.env',
+        context: 'process.env.API_KEY',
+      },
+      {
+        variable: 'API_KEY',
+        file: 'src/a.ts',
+        line: 12,
+        column: 1,
+        pattern: 'process.env',
+        context: 'process.env.API_KEY',
+      },
+      {
+        variable: 'JWT_SECRET',
+        file: 'src/b.ts',
+        line: 7,
+        column: 1,
+        pattern: 'process.env',
+        context: 'process.env.JWT_SECRET',
+      },
+      {
+        variable: 'UNUSED',
+        file: 'src/c.ts',
+        line: 1,
+        column: 1,
+        pattern: 'process.env',
+        context: 'process.env.UNUSED',
+      },
+    ];
+
+    const result = printMissing(['API_KEY', 'JWT_SECRET'], used, '');
+
     expect(result).toBe(true);
+    expect(logSpy).toHaveBeenCalledTimes(7);
+    expect(logSpy).toHaveBeenNthCalledWith(
+      2,
+      expect.stringContaining('Missing in environment file'),
+    );
+    expect(logSpy).toHaveBeenNthCalledWith(
+      4,
+      expect.stringContaining('normalized:src/a.ts:10'),
+    );
+    expect(logSpy).toHaveBeenNthCalledWith(
+      5,
+      expect.stringContaining('normalized:src/a.ts:12'),
+    );
+    expect(logSpy).toHaveBeenNthCalledWith(
+      6,
+      expect.stringContaining('normalized:src/b.ts:7'),
+    );
   });
 });

test/unit/ui/scan/printUnused.test.ts

@@ -1,5 +1,4 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import chalk from 'chalk';
 import { printUnused } from '../../../../src/ui/scan/printUnused.js';
 
 describe('printUnused', () => {
@@ -17,4 +16,50 @@ describe('printUnused', () => {
     printUnused([], '.env');
     expect(logSpy).not.toHaveBeenCalled();
   });
+
+  it('prints unused variables with provided file name in non-strict mode', () => {
+    printUnused(['API_KEY', 'JWT_SECRET'], '.env');
+
+    expect(logSpy).toHaveBeenCalledTimes(6);
+    expect(logSpy).toHaveBeenNthCalledWith(
+      2,
+      expect.stringContaining('Unused in .env'),
+    );
+    expect(logSpy).toHaveBeenNthCalledWith(
+      4,
+      expect.stringContaining('API_KEY'),
+    );
+    expect(logSpy).toHaveBeenNthCalledWith(
+      5,
+      expect.stringContaining('JWT_SECRET'),
+    );
+  });
+
+  it('uses fallback file label when comparedAgainst is empty', () => {
+    printUnused(['SOME_VAR'], '');
+
+    expect(logSpy).toHaveBeenCalledTimes(5);
+    expect(logSpy).toHaveBeenNthCalledWith(
+      2,
+      expect.stringContaining('Unused in environment file'),
+    );
+  });
+
+  it('prints output in strict mode', () => {
+    printUnused(['SOME_VAR'], '.env', true);
+
+    expect(logSpy).toHaveBeenCalledTimes(5);
+    expect(logSpy).toHaveBeenNthCalledWith(
+      2,
+      expect.stringContaining('Unused in .env'),
+    );
+    expect(logSpy).toHaveBeenNthCalledWith(
+      4,
+      expect.stringContaining('SOME_VAR'),
+    );
+    expect(logSpy).toHaveBeenNthCalledWith(
+      4,
+      expect.stringContaining('unused'),
+    );
+  });
 });