Fix up unused files for jest-unit (resolves #841)

packages/knip/fixtures/plugins/jest3/index.spec.ts

@@ -0,0 +1 @@
+import 'jest';

packages/knip/fixtures/plugins/jest3/jest.config.js

@@ -0,0 +1,13 @@
+module.exports = {
+  testEnvironment: 'node',
+  reporters: [
+    'default',
+    [
+      'jest-junit',
+      {
+        outputDirectory: 'reports/junit',
+        outputName: 'junit.xml',
+      },
+    ],
+  ],
+};

packages/knip/fixtures/plugins/jest3/package.json

@@ -0,0 +1,8 @@
+{
+  "name": "@fixtures/jest3",
+  "scripts": {},
+  "devDependencies": {
+    "jest": "*",
+    "jest-junit": "*"
+  }
+}

packages/knip/src/plugins/jest/helpers.ts

@@ -32,16 +32,17 @@ export const getReportersDependencies = (config: JestInitialOptions, options: Pl
         testSuitePropertiesDirectory,
       } = reporter[1];
 
-      const testCaseFileName = getStringPropOrFallback(testCasePropertiesFile, 'junitProperties.js');
-      const testCaseDirectory = getStringPropOrFallback(testCasePropertiesDirectory, options.rootCwd);
-      const testCaseFilePath = join(testCaseDirectory, testCaseFileName);
-
-      const testSuiteFileName = getStringPropOrFallback(testSuitePropertiesFile, 'junitTestCaseProperties.js');
-      const testSuiteDirectory = getStringPropOrFallback(testSuitePropertiesDirectory, options.rootCwd);
-      const testSuiteFilePath = join(testSuiteDirectory, testSuiteFileName);
-
-      jUnitReporterDeps.push(testCaseFilePath);
-      jUnitReporterDeps.push(testSuiteFilePath);
+      if (testCasePropertiesFile) {
+        const fileName = getStringPropOrFallback(testCasePropertiesFile, 'junitProperties.js');
+        const dir = getStringPropOrFallback(testCasePropertiesDirectory, options.rootCwd);
+        jUnitReporterDeps.push(join(dir, fileName));
+      }
+
+      if (testSuitePropertiesFile) {
+        const fileName = getStringPropOrFallback(testSuitePropertiesFile, 'junitTestCaseProperties.js');
+        const dir = getStringPropOrFallback(testSuitePropertiesDirectory, options.rootCwd);
+        jUnitReporterDeps.push(join(dir, fileName));
+      }
     }
   }
 

packages/knip/test/plugins/jest.test.ts

@@ -31,7 +31,7 @@ test('Find dependencies with the Jest plugin', async () => {
     devDependencies: 1,
     unlisted: 3,
     unresolved: 9,
-    processed: 8,
-    total: 8,
+    processed: 7,
+    total: 7,
   });
 });

packages/knip/test/plugins/jest2.test.ts

@@ -1,34 +1,26 @@
 import { test } from 'bun:test';
 import assert from 'node:assert/strict';
 import { main } from '../../src/index.js';
-import { join, resolve } from '../../src/util/path.js';
+import { resolve } from '../../src/util/path.js';
 import baseArguments from '../helpers/baseArguments.js';
 import baseCounters from '../helpers/baseCounters.js';
 
 const cwd = resolve('fixtures/plugins/jest2');
 
-test('Find dependencies with the Jest plugin', async () => {
+test('Find dependencies with the Jest plugin (2)', async () => {
   const { issues, counters } = await main({
     ...baseArguments,
     cwd,
   });
 
   assert(issues.devDependencies['package.json']['jest']);
 
-  // Correctly identifies setup file in a non-root jest.config.js which uses
-  // <rootDir> to reference the root directory.
-  assert(!issues.files.has(join(cwd, 'project1/setupFiles/setup.js')));
-
-  // Correctly identifies a local `testEnvironment` file.
-  assert(!issues.files.has(join(cwd, 'jest.environment.js')));
-
   assert.deepEqual(counters, {
     ...baseCounters,
-    files: 0,
     devDependencies: 1,
     unlisted: 0,
     unresolved: 0,
-    processed: 7,
-    total: 7,
+    processed: 6,
+    total: 6,
   });
 });

packages/knip/test/plugins/jest3.test.ts

@@ -0,0 +1,21 @@
+import { test } from 'bun:test';
+import assert from 'node:assert/strict';
+import { main } from '../../src/index.js';
+import { resolve } from '../../src/util/path.js';
+import baseArguments from '../helpers/baseArguments.js';
+import baseCounters from '../helpers/baseCounters.js';
+
+const cwd = resolve('fixtures/plugins/jest3');
+
+test('Find dependencies with the Jest plugin (3)', async () => {
+  const { counters } = await main({
+    ...baseArguments,
+    cwd,
+  });
+
+  assert.deepEqual(counters, {
+    ...baseCounters,
+    processed: 2,
+    total: 2,
+  });
+});