Skip to content

Commit 090e76b

Browse files
committedFeb 10, 2025
Ignore built-in test reporters (resolve #935)
1 parent 1019b99 commit 090e76b

File tree

3 files changed

+8
-1
lines changed

3 files changed

+8
-1
lines changed
 

‎packages/knip/src/binaries/plugins.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import { resolve as fallbackResolve } from './fallback.js';
99
const isGlobLikeMatch = /(^!|[*+\\(|{^$])/;
1010
const isGlobLike = (value: string) => isGlobLikeMatch.test(value);
1111

12-
const nodeLoadersArgs = { import: ['r', 'experimental-loader', 'require', 'loader', 'test-reporter'] };
12+
const nodeLoadersArgs = { import: ['r', 'experimental-loader', 'require', 'loader'] };
1313

1414
export const resolve: BinaryResolver = (binary, _args, options) => {
1515
const { fromArgs, containingFilePath } = options;

‎packages/knip/src/plugins/node/index.ts

+2
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,8 @@ const resolveEntryPaths: ResolveEntryPaths<PackageJson> = localConfig => {
2525
const args = {
2626
positional: true,
2727
nodeImportArgs: true,
28+
resolve: ['test-reporter'],
29+
args: (args: string[]) => args.filter(arg => !/--test-reporter[= ](spec|tap|dot|junit|lcov)/.test(arg)),
2830
};
2931

3032
export default {

‎packages/knip/test/util/get-inputs-from-scripts.test.ts

+5
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,11 @@ test('getInputsFromScripts (node)', () => {
5454
t('node ./script.js build', [toBinary('node'), js]);
5555
});
5656

57+
test('getInputsFromScripts (node --test)', () => {
58+
t('node --test --test-reporter=reporter', [toBinary('node'), toDeferResolve('reporter')]);
59+
t('node --test --test-reporter=spec', [toBinary('node')]);
60+
});
61+
5762
test('getInputsFromScripts (node -r)', () => {
5863
t('node -r script.js', [toBinary('node'), toDeferResolve('script.js')]);
5964
t('node -r package/script', [toBinary('node'), toDeferResolve('package/script')]);

0 commit comments

Comments
 (0)
Please sign in to comment.