Skip to content

Commit

Permalink
test: use async/await in test-debugger-sb-before-load
Browse files Browse the repository at this point in the history
PR-URL: #44697
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
  • Loading branch information
hopeolaide authored and juanarbol committed Oct 10, 2022
1 parent ac00be1 commit c38945b
Showing 1 changed file with 21 additions and 32 deletions.
53 changes: 21 additions & 32 deletions test/sequential/test-debugger-sb-before-load.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,35 +10,24 @@ const assert = require('assert');
const path = require('path');

// Using sb before loading file.
{
const scriptFullPath = fixtures.path('debugger', 'cjs', 'index.js');
const script = path.relative(process.cwd(), scriptFullPath);

const otherScriptFullPath = fixtures.path('debugger', 'cjs', 'other.js');
const otherScript = path.relative(process.cwd(), otherScriptFullPath);

const cli = startCLI([script]);

function onFatal(error) {
cli.quit();
throw error;
}

cli.waitForInitialBreak()
.then(() => cli.waitForPrompt())
.then(() => cli.command('sb("other.js", 2)'))
.then(() => {
assert.match(
cli.output,
/not loaded yet/,
'warns that the script was not loaded yet');
})
.then(() => cli.stepCommand('cont'))
.then(() => {
assert.ok(
cli.output.includes(`break in ${otherScript}:2`),
'found breakpoint in file that was not loaded yet');
})
.then(() => cli.quit())
.then(null, onFatal);
}

const scriptFullPath = fixtures.path('debugger', 'cjs', 'index.js');
const script = path.relative(process.cwd(), scriptFullPath);

const otherScriptFullPath = fixtures.path('debugger', 'cjs', 'other.js');
const otherScript = path.relative(process.cwd(), otherScriptFullPath);

const cli = startCLI([script]);

(async () => {
await cli.waitForInitialBreak();
await cli.waitForPrompt();
await cli.command('sb("other.js", 2)');
assert.match(cli.output, /not loaded yet/,
'warns that the script was not loaded yet');
await cli.stepCommand('cont');
assert.ok(cli.output.includes(`break in ${otherScript}:2`),
'found breakpoint in file that was not loaded yet');
})()
.then(common.mustCall())
.finally(() => cli.quit());

0 comments on commit c38945b

Please sign in to comment.