Skip to content

Commit

Permalink
test: remove error allowance in debugger test
Browse files Browse the repository at this point in the history
Remove allowance for FATAL ERROR. It is no longer needed.

PR-URL: #41640
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jan Krems <jan.krems@gmail.com>
  • Loading branch information
MrJithil authored and danielleadams committed Mar 14, 2022
1 parent 3e93cc3 commit 2b87f97
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions test/sequential/test-debugger-exceptions.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,7 @@ const path = require('path');
})
// Making sure it will die by default:
.then(() => cli.command('c'))
// TODO: Remove FATAL ERROR once node doesn't show a FATAL ERROR anymore.
.then(() => cli.waitFor(/disconnect|FATAL ERROR/))
.then(() => cli.waitFor(/disconnect/))

// Next run: With `breakOnException` it pauses in both places.
.then(() => cli.stepCommand('r'))
Expand Down Expand Up @@ -66,9 +65,7 @@ const path = require('path');
assert.deepStrictEqual(cli.breakInfo, { filename: script, line: 1 });
})
.then(() => cli.command('c'))
// TODO: Remove FATAL ERROR once node doesn't show a FATAL ERROR anymore
.then(() => cli.waitFor(/disconnect|FATAL ERROR/))

.then(() => cli.waitFor(/disconnect/))
.then(() => cli.quit())
.then(null, onFatal);
}

0 comments on commit 2b87f97

Please sign in to comment.