test: use async/await in test-debugger-sb-before-load

PR-URL: #44697 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
nodejs · Oct 5, 2022 · eeabd23 · eeabd23
1 parent 5c63d14
commit eeabd23
Showing 1 changed file with 21 additions and 32 deletions.
diff --git a/test/sequential/test-debugger-sb-before-load.js b/test/sequential/test-debugger-sb-before-load.js
@@ -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());