Skip to content

test: improve test-debugger-util-regression

Rodrigo Muino Tomonari requested to merge github/fork/santigimeno/fix_debugger_util_regression into master
Checklist
  • make -j8 test (UNIX), or vcbuild test nosign (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

test

Description of change

Avoid the exit command to be sent more than once. It prevents from undesired errors emitted on proc.stdin. Remove the watchdog timer so the test does not fail in case it takes longer to complete.

There was a failure on the CI freebsd box caused by the watchdog timer: https://ci.nodejs.org/job/node-test-commit-freebsd/5177/nodes=freebsd10-64/console

not ok 236 parallel/test-debugger-util-regression
# 
# assert.js:85
#   throw new assert.AssertionError({
#   ^
# AssertionError: the program should not hang
#     at Object.exports.fail (/usr/home/iojs/build/workspace/node-test-commit-freebsd/nodes/freebsd10-64/test/common.js:443:10)
#     at Timeout.fail [as _onTimeout] (/usr/home/iojs/build/workspace/node-test-commit-freebsd/nodes/freebsd10-64/test/parallel/test-debugger-util-regression.js:23:10)
#     at ontimeout (timers.js:365:14)
#     at tryOnTimeout (timers.js:237:5)
#     at Timer.listOnTimeout (timers.js:207:5)

Merge request reports