Skip to content

Commit 380494f

Browse files
addaleaxTrott
authored andcommitted
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 <cjihrig@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent db459e9 commit 380494f

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/parallel/test-worker-debug.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -272,7 +272,7 @@ async function testWaitForDisconnectInWorker(session, post) {
272272

273273
session.disconnect();
274274
console.log('Test done');
275-
})().catch((err) => {
275+
})().then(common.mustCall()).catch((err) => {
276276
console.error(err);
277-
process.abort();
277+
process.exitCode = 1;
278278
});

0 commit comments

Comments
 (0)