Skip to content

Commit

Permalink
test: make debugging of inspector-port-zero easier
Browse files Browse the repository at this point in the history
If the process was killed, then the exit code will be null, in which
case knowing the signal is really helpful.

PR-URL: nodejs#16685
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Eugene Ostroukhov <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
  • Loading branch information
gibfahn committed Nov 20, 2017
1 parent 5201055 commit 5c7b1ec
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion test/sequential/test-inspector-port-zero.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,10 @@ function test(arg, port = '') {
};
proc.stdout.on('close', mustCall(() => onclose()));
proc.stderr.on('close', mustCall(() => onclose()));
proc.on('exit', mustCall((exitCode) => assert.strictEqual(exitCode, 0)));
proc.on('exit', mustCall((exitCode, signal) => assert.strictEqual(
exitCode,
0,
`exitCode: ${exitCode}, signal: ${signal}`)));
}
}

Expand Down

0 comments on commit 5c7b1ec

Please sign in to comment.