test: change promises to async/await in test-debugger-backtrace.js · nodejs/node@5bed2d1 · GitHub

nodejs / node Public

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

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

Additional navigation options

Commit 5bed2d1

Browse filesBrowse the repository at this point in the historyBrowse files

zhangjuliet

authored and

juanarbol

committed

test: change promises to async/await in test-debugger-backtrace.js

PR-URL: #44677 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de>

1 parent f630881 commit 5bed2d1
Copy full SHA for 5bed2d1

1 file changed

+13-16Lines changed: 13 additions & 16 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/sequential
  • test-debugger-backtrace.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/sequential/test-debugger-backtrace.js‎

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

+13-16Lines changed: 13 additions & 16 deletions

Original file line number	Diff line number	Diff line change
@@ -15,22 +15,19 @@ const path = require('path');
15	15	const script = path.relative(process.cwd(), scriptFullPath);
16	16	const cli = startCLI([script]);
17	17
18		- function onFatal(error) {
19		- cli.quit();
20		- throw error;
21		- }
22		-
23		- return cli.waitForInitialBreak()
24		- .then(() => cli.waitForPrompt())
25		- .then(() => cli.stepCommand('c'))
26		- .then(() => cli.command('bt'))
27		- .then(() => {
	18	+ async function runTest() {
	19	+ try {
	20	+ await cli.waitForInitialBreak();
	21	+ await cli.waitForPrompt();
	22	+ await cli.stepCommand('c');
	23	+ await cli.command('bt');
28	24	assert.ok(cli.output.includes(`#0 topFn ${script}:7:2`));
29		- })
30		- .then(() => cli.command('backtrace'))
31		- .then(() => {
	25	+ await cli.command('backtrace');
32	26	assert.ok(cli.output.includes(`#0 topFn ${script}:7:2`));
33		- })
34		- .then(() => cli.quit())
35		- .then(null, onFatal);
	27	+ } finally {
	28	+ await cli.quit();
	29	+ }
	30	+ }
	31	+
	32	+ runTest();
36	33	}

0 commit comments

Comments

0 (0)