test: make sure test function resolves in test-worker-debug

Use the common `.then(common.mustCall())` check to verify that. Also, we should not use `process.abort()` in `test/parallel`; if the test fails, that leaves core dumps lying around on POSIX systems. PR-URL: nodejs#28155 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]>
Trott · Jun 17, 2019 · 380494f · 380494f
1 parent db459e9
commit 380494f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/test/parallel/test-worker-debug.js b/test/parallel/test-worker-debug.js
@@ -272,7 +272,7 @@ async function testWaitForDisconnectInWorker(session, post) {
 
   session.disconnect();
   console.log('Test done');
-})().catch((err) => {
+})().then(common.mustCall()).catch((err) => {
   console.error(err);
-  process.abort();
+  process.exitCode = 1;
 });