test: update test-debugger-scripts to use await/async - fix lint errors

nodejs · Sep 17, 2022 · 222d949 · 222d949
1 parent 4d5fe19
commit 222d949
Showing 1 changed file with 40 additions and 6 deletions.
diff --git a/test/sequential/test-debugger-scripts.js b/test/sequential/test-debugger-scripts.js
@@ -9,11 +9,43 @@ const startCLI = require('../common/debugger');
 const assert = require('assert');
 
 // List scripts.
-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;
+  }
+
+  // return cli.waitForInitialBreak()
+  //   .then(() => cli.waitForPrompt())
+  //   .then(() => cli.command('scripts'))
+  //   .then(() => {
+  //     assert.match(
+  //       cli.output,
+  //       /^\* \d+: \S+debugger(?:\/|\\)three-lines\.js/m,
+  //       'lists the user script');
+  //     assert.doesNotMatch(
+  //       cli.output,
+  //       /\d+: node:internal\/buffer/,
+  //       'omits node-internal scripts');
+  //   })
+  //   .then(() => cli.command('scripts(true)'))
+  //   .then(() => {
+  //     assert.match(
+  //       cli.output,
+  //       /\* \d+: \S+debugger(?:\/|\\)three-lines\.js/,
+  //       'lists the user script');
+  //     assert.match(
+  //       cli.output,
+  //       /\d+: node:internal\/buffer/,
+  //       'includes node-internal scripts');
+  //   })
+  //   .then(() => cli.quit())
+  //   .then(null, onFatal);
 
 (async () => {
-try{
   await cli.waitForInitialBreak();
   await cli.waitForPrompt();
   await cli.command('scripts');
@@ -34,7 +66,9 @@ try{
     cli.output,
     /\d+: node:internal\/buffer/,
     'includes node-internal scripts');
- } finally {
-    cli.quit()
- }
+
 })()
+.then(() => cli.quit())
+.then(null, onFatal);
+
+}