Skip to content

Commit aefbab0

Browse files
committed
fix(jest-worker): filtering --inspect from execArgv is unnecessary for thread works
1 parent 9c8c3b3 commit aefbab0

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

packages/jest-worker/src/workers/NodeThreadsWorker.ts

+1-2
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,7 @@ export default class ExperimentalWorker implements WorkerInterface {
6565
JEST_WORKER_ID: String(this._options.workerId + 1), // 0-indexed workerId, 1-indexed JEST_WORKER_ID
6666
} as NodeJS.ProcessEnv,
6767
eval: false,
68-
// Suppress --debug / --inspect flags while preserving others (like --harmony).
69-
execArgv: process.execArgv.filter(v => !/^--(debug|inspect)/.test(v)),
68+
execArgv: process.execArgv,
7069
// @ts-expect-error: added in newer versions
7170
resourceLimits: this._options.resourceLimits,
7271
stderr: true,

packages/jest-worker/src/workers/__tests__/NodeThreadsWorker.test.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ it('passes fork options down to worker_threads.Worker, adding the defaults', ()
7373
expect(workerThreads.mock.calls[0][1]).toEqual({
7474
env: process.env, // Default option.
7575
eval: false,
76-
execArgv: ['-p'], // Filtered option.
76+
execArgv: ['--inspect', '-p'],
7777
execPath: 'hello', // Added option.
7878
resourceLimits: undefined,
7979
stderr: true,

0 commit comments

Comments
 (0)