Skip to content

Commit a5f8798

Browse files
committedJan 22, 2023
test: avoid left behind child processes
Extend the Linux logic to all POSIX platforms in test-child-process-exec-abortcontroller-promisified. PR-URL: #46276 Fixes: nodejs/build#3154 Refs: #37518 Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com> Reviewed-By: Ruy Adorno <ruyadorno@google.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Michael Dawson <midawson@redhat.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent d73ea4d commit a5f8798

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed
 

‎test/parallel/test-child-process-exec-abortcontroller-promisified.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,9 @@ const invalidArgTypeError = {
1111
name: 'TypeError'
1212
};
1313

14-
const waitCommand = common.isLinux ?
15-
'sleep 2m' :
16-
`${process.execPath} -e "setInterval(()=>{}, 99)"`;
14+
const waitCommand = common.isWindows ?
15+
`${process.execPath} -e "setInterval(()=>{}, 99)"` :
16+
'sleep 2m';
1717

1818
{
1919
const ac = new AbortController();

0 commit comments

Comments
 (0)
Please sign in to comment.