Skip to content

Commit

Permalink
test: avoid left behind child processes
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
richardlau committed Jan 22, 2023
1 parent d73ea4d commit a5f8798
Showing 1 changed file with 3 additions and 3 deletions.
Expand Up @@ -11,9 +11,9 @@ const invalidArgTypeError = {
name: 'TypeError'
};

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

{
const ac = new AbortController();
Expand Down

0 comments on commit a5f8798

Please sign in to comment.