Skip to content

Commit 1a78632

Browse files
joyeecheungRafaelGSS
authored andcommitted
test: use random port in NodeInstance.startViaSignal()
Which makes it possible to run the tests that use this in parallel. PR-URL: #47412 Refs: #47146 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Rich Trott <[email protected]>
1 parent 7d2d40e commit 1a78632

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/common/inspector-helper.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -372,7 +372,7 @@ class NodeInstance extends EventEmitter {
372372

373373
static async startViaSignal(scriptContents) {
374374
const instance = new NodeInstance(
375-
['--expose-internals'],
375+
['--expose-internals', '--inspect-port=0'],
376376
`${scriptContents}\nprocess._rawDebug('started');`, undefined);
377377
const msg = 'Timed out waiting for process to start';
378378
while (await fires(instance.nextStderrString(), msg, TIMEOUT) !== 'started');

0 commit comments

Comments
 (0)