Fixes for Mocha plugin (#1089)

Author: apeloquin-agilysys

packages/knip/fixtures/plugins/mocha/.mocharc.json

@@ -1,4 +1,4 @@
 {
-  "extension": ["ts"],
-  "require": "ts-node/register"
+  "spec": ["**/*.test.ts"],
+  "require": ["ts-node/register", "hooks.ts"]
 }

packages/knip/fixtures/plugins/mocha/.mocharc.yml

@@ -1,3 +1,5 @@
-extension:
-  - ts
-require: ts-node/register
+spec:
+  - "**/*.test.ts"
+require:
+  - ts-node/register
+  - hooks.ts

packages/knip/fixtures/plugins/mocha/example.test.ts

@@ -0,0 +1,3 @@
+import 'mocha';
+
+describe('Tests', () => {});

packages/knip/fixtures/plugins/mocha/hooks.ts

@@ -0,0 +1,3 @@
+export const mochaHooks = {
+  beforeAll: () => {},
+};

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

@@ -1,7 +1,12 @@
 {
   "name": "@fixtures/mocha",
   "mocha": {
-    "extension": ["ts"],
-    "require": "ts-node/register"
+    "spec": ["**/*.test.ts"],
+    "require": ["ts-node/register", "hooks.ts"]
+  },
+  "devDependencies": {
+    "@types/mocha": "*",
+    "mocha": "*",
+    "ts-node": "*"
   }
 }

packages/knip/src/plugins/mocha/index.ts

@@ -1,5 +1,5 @@
 import type { IsPluginEnabled, Plugin, ResolveConfig } from '../../types/config.js';
-import { toEntry } from '../../util/input.js';
+import { type Input, toDeferResolve, toEntry } from '../../util/input.js';
 import { hasDependency } from '../../util/plugin.js';
 import type { MochaConfig } from './types.js';
 
@@ -18,7 +18,11 @@ const entry = ['**/test/*.{js,cjs,mjs}'];
 const resolveConfig: ResolveConfig<MochaConfig> = localConfig => {
   const entryPatterns = localConfig.spec ? [localConfig.spec].flat() : entry;
   const require = localConfig.require ? [localConfig.require].flat() : [];
-  return [...entryPatterns, ...require].map(id => toEntry(id));
+
+  const inputs: Input[] = [];
+  inputs.push(...entryPatterns.map(id => toEntry(id)));
+  inputs.push(...require.map(id => toDeferResolve(id)));
+  return inputs;
 };
 
 const args = {

packages/knip/test/plugins/mocha.test.ts

@@ -15,7 +15,7 @@ test('Find dependencies with the Mocha plugin', async () => {
 
   assert.deepEqual(counters, {
     ...baseCounters,
-    processed: 0,
-    total: 0,
+    processed: 2,
+    total: 2,
   });
 });