test: update test-debugger-breakpoint-exists.js to use async/await · nodejs/node@e9e1309 · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 635
• Pull requests 755
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit e9e1309

Browse filesBrowse the repository at this point in the historyBrowse files

archana-kamath

authored and

juanarbol

committed

test: update test-debugger-breakpoint-exists.js to use async/await

PR-URL: #44682 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>

1 parent 510365a commit e9e1309
Copy full SHA for e9e1309

1 file changed

+12-15Lines changed: 12 additions & 15 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/sequential
  • test-debugger-breakpoint-exists.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/sequential/test-debugger-breakpoint-exists.js‎

Copy file name to clipboardExpand all lines: test/sequential/test-debugger-breakpoint-exists.js

+12-15Lines changed: 12 additions & 15 deletions

Original file line number	Diff line number	Diff line change
@@ -8,20 +8,17 @@ const fixtures = require('../common/fixtures');
8	8	const startCLI = require('../common/debugger');
9	9
10	10	// Test for "Breakpoint at specified location already exists" error.
11		- {
12		- const script = fixtures.path('debugger', 'three-lines.js');
13		- const cli = startCLI([script]);
	11	+ const script = fixtures.path('debugger', 'three-lines.js');
	12	+ const cli = startCLI([script]);
14	13
15		- function onFatal(error) {
16		- cli.quit();
17		- throw error;
	14	+ (async () => {
	15	+ try {
	16	+ await cli.waitForInitialBreak();
	17	+ await cli.waitForPrompt();
	18	+ await cli.command('setBreakpoint(1)');
	19	+ await cli.command('setBreakpoint(1)');
	20	+ await cli.waitFor(/Breakpoint at specified location already exists/);
	21	+ } finally {
	22	+ await cli.quit();
18	23	}
19		-
20		- cli.waitForInitialBreak()
21		- .then(() => cli.waitForPrompt())
22		- .then(() => cli.command('setBreakpoint(1)'))
23		- .then(() => cli.command('setBreakpoint(1)'))
24		- .then(() => cli.waitFor(/Breakpoint at specified location already exists/))
25		- .then(() => cli.quit())
26		- .then(null, onFatal);
27		- }
	24	+ })().then(common.mustCall());

0 commit comments

Comments

0 (0)