Skip to content

Commit eeabd23

Browse files
hopeolaidedanielleadams
authored andcommitted
test: use async/await in test-debugger-sb-before-load
PR-URL: #44697 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de>
1 parent 5c63d14 commit eeabd23

File tree

1 file changed

+21
-32
lines changed

1 file changed

+21
-32
lines changed

test/sequential/test-debugger-sb-before-load.js

+21-32
Original file line numberDiff line numberDiff line change
@@ -10,35 +10,24 @@ const assert = require('assert');
1010
const path = require('path');
1111

1212
// Using sb before loading file.
13-
{
14-
const scriptFullPath = fixtures.path('debugger', 'cjs', 'index.js');
15-
const script = path.relative(process.cwd(), scriptFullPath);
16-
17-
const otherScriptFullPath = fixtures.path('debugger', 'cjs', 'other.js');
18-
const otherScript = path.relative(process.cwd(), otherScriptFullPath);
19-
20-
const cli = startCLI([script]);
21-
22-
function onFatal(error) {
23-
cli.quit();
24-
throw error;
25-
}
26-
27-
cli.waitForInitialBreak()
28-
.then(() => cli.waitForPrompt())
29-
.then(() => cli.command('sb("other.js", 2)'))
30-
.then(() => {
31-
assert.match(
32-
cli.output,
33-
/not loaded yet/,
34-
'warns that the script was not loaded yet');
35-
})
36-
.then(() => cli.stepCommand('cont'))
37-
.then(() => {
38-
assert.ok(
39-
cli.output.includes(`break in ${otherScript}:2`),
40-
'found breakpoint in file that was not loaded yet');
41-
})
42-
.then(() => cli.quit())
43-
.then(null, onFatal);
44-
}
13+
14+
const scriptFullPath = fixtures.path('debugger', 'cjs', 'index.js');
15+
const script = path.relative(process.cwd(), scriptFullPath);
16+
17+
const otherScriptFullPath = fixtures.path('debugger', 'cjs', 'other.js');
18+
const otherScript = path.relative(process.cwd(), otherScriptFullPath);
19+
20+
const cli = startCLI([script]);
21+
22+
(async () => {
23+
await cli.waitForInitialBreak();
24+
await cli.waitForPrompt();
25+
await cli.command('sb("other.js", 2)');
26+
assert.match(cli.output, /not loaded yet/,
27+
'warns that the script was not loaded yet');
28+
await cli.stepCommand('cont');
29+
assert.ok(cli.output.includes(`break in ${otherScript}:2`),
30+
'found breakpoint in file that was not loaded yet');
31+
})()
32+
.then(common.mustCall())
33+
.finally(() => cli.quit());

0 commit comments

Comments
 (0)