Skip to content

Commit

Permalink
test: pause child until parent is ready
Browse files Browse the repository at this point in the history
Pause child on startup using inspect-brk=0 until the parent debugger
is ready.

PR-URL: #15774
Fixes: #14897
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
  • Loading branch information
jBarz authored and gibfahn committed Nov 14, 2017
1 parent 1bf6250 commit 1e8a421
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 0 additions & 2 deletions test/sequential/sequential.status
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ prefix sequential
test-inspector-async-call-stack : PASS, FLAKY
test-inspector-bindings : PASS, FLAKY
test-inspector-debug-end : PASS, FLAKY
test-inspector-stop-profile-after-done: PASS, FLAKY

[$system==linux]

Expand All @@ -21,4 +20,3 @@ test-inspector-stop-profile-after-done: PASS, FLAKY
[$system==freebsd]

[$system==aix]
test-inspector-stop-profile-after-done: PASS, FLAKY
2 changes: 1 addition & 1 deletion test/sequential/test-inspector-stop-profile-after-done.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ const assert = require('assert');
const { NodeInstance } = require('../common/inspector-helper.js');

async function runTests() {
const child = new NodeInstance(['--inspect=0'],
const child = new NodeInstance(['--inspect-brk=0'],
`let c = 0;
const interval = setInterval(() => {
console.log(new Object());
Expand Down

0 comments on commit 1e8a421

Please sign in to comment.