Skip to content

Commit 1375559

Browse files
Trotttargos
authored andcommitted
test: remove workaround code in debugger test
Remove code that made a check more lenient to account for a known issue that is no longer reproducible. Refs: nodejs/node-inspect#35 PR-URL: #39238 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Jan Krems <[email protected]>
1 parent 0673ede commit 1375559

File tree

1 file changed

+3
-13
lines changed

1 file changed

+3
-13
lines changed

Diff for: test/sequential/test-debugger-preserve-breaks.js

+3-13
Original file line numberDiff line numberDiff line change
@@ -53,19 +53,9 @@ const path = require('path');
5353
})
5454
.then(() => cli.command('breakpoints'))
5555
.then(() => {
56-
// TODO: There is a known issue on AIX and some other operating systems
57-
// where the breakpoints aren't properly resolved yet when we reach this
58-
// point. Eventually that should be figured out but for now we don't
59-
// want to fail builds because of it.
60-
// What it should be:
61-
//
62-
// const msg = `SCRIPT: ${script}, OUTPUT: ${cli.output}`;
63-
// assert.ok(cli.output.includes(`#0 ${script}:2`), msg);
64-
// assert.ok(cli.output.includes(`#1 ${script}:3`), msg);
65-
//
66-
// What we're doing for now instead:
67-
assert.match(cli.output, /#0 [^\n]+three-lines\.js\$?:2/);
68-
assert.match(cli.output, /#1 [^\n]+three-lines\.js\$?:3/);
56+
const msg = `SCRIPT: ${script}, OUTPUT: ${cli.output}`;
57+
assert.ok(cli.output.includes(`#0 ${script}:2`), msg);
58+
assert.ok(cli.output.includes(`#1 ${script}:3`), msg);
6959
})
7060
.then(() => cli.quit())
7161
.then(null, onFatal);

0 commit comments

Comments
 (0)