Skip to content

Commit

Permalink
test: update test-debugger-breakpoint-exists.js to use async/await
Browse files Browse the repository at this point in the history
PR-URL: nodejs/node#44682
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
  • Loading branch information
archana-kamath authored and guangwong committed Jan 3, 2023
1 parent cca6b12 commit 6f9fe30
Showing 1 changed file with 12 additions and 15 deletions.
27 changes: 12 additions & 15 deletions test/sequential/test-debugger-breakpoint-exists.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,20 +8,17 @@ const fixtures = require('../common/fixtures');
const startCLI = require('../common/debugger');

// Test for "Breakpoint at specified location already exists" error.
{
const script = fixtures.path('debugger', 'three-lines.js');
const cli = startCLI([script]);
const script = fixtures.path('debugger', 'three-lines.js');
const cli = startCLI([script]);

function onFatal(error) {
cli.quit();
throw error;
(async () => {
try {
await cli.waitForInitialBreak();
await cli.waitForPrompt();
await cli.command('setBreakpoint(1)');
await cli.command('setBreakpoint(1)');
await cli.waitFor(/Breakpoint at specified location already exists/);
} finally {
await cli.quit();
}

cli.waitForInitialBreak()
.then(() => cli.waitForPrompt())
.then(() => cli.command('setBreakpoint(1)'))
.then(() => cli.command('setBreakpoint(1)'))
.then(() => cli.waitFor(/Breakpoint at specified location already exists/))
.then(() => cli.quit())
.then(null, onFatal);
}
})().then(common.mustCall());

0 comments on commit 6f9fe30

Please sign in to comment.