From 30df37f733cec977a7af26c5fe0981a9c261c800 Mon Sep 17 00:00:00 2001 From: Jithil P Ponnan Date: Tue, 25 Jan 2022 07:08:05 +0530 Subject: [PATCH] test: remove error allowance in debugger test MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Remove allowance for FATAL ERROR. It is no longer needed. PR-URL: https://github.com/nodejs/node/pull/41640 Reviewed-By: Rich Trott Reviewed-By: Tobias Nießen Reviewed-By: Darshan Sen Reviewed-By: Luigi Pinca Reviewed-By: Jan Krems --- test/sequential/test-debugger-exceptions.js | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/test/sequential/test-debugger-exceptions.js b/test/sequential/test-debugger-exceptions.js index 9b1163316268c7..f20f35d4808a47 100644 --- a/test/sequential/test-debugger-exceptions.js +++ b/test/sequential/test-debugger-exceptions.js @@ -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')) @@ -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); }