test: ignore stale process cleanup failures on Windows

In some tests we try to clean up stale child processes on Windows, but they don't necessarily exist, in that case we should ignore any failures from the WMIC.exe command. PR-URL: #44480 Reviewed-By: Luigi Pinca <[email protected]>
nodejs · Oct 4, 2022 · 63033ec · 63033ec
1 parent 72d1070
commit 63033ec
Showing 1 changed file with 12 additions and 8 deletions.
diff --git a/test/common/child_process.js b/test/common/child_process.js
@@ -12,14 +12,18 @@ function cleanupStaleProcess(filename) {
   }
   process.once('beforeExit', () => {
     const basename = filename.replace(/.*[/\\]/g, '');
-    require('child_process')
-      .execFileSync(`${process.env.SystemRoot}\\System32\\wbem\\WMIC.exe`, [
-        'process',
-        'where',
-        `commandline like '%${basename}%child'`,
-        'delete',
-        '/nointeractive',
-      ]);
+    try {
+      require('child_process')
+        .execFileSync(`${process.env.SystemRoot}\\System32\\wbem\\WMIC.exe`, [
+          'process',
+          'where',
+          `commandline like '%${basename}%child'`,
+          'delete',
+          '/nointeractive',
+        ]);
+    } catch {
+      // Ignore failures, there might not be any stale process to clean up.
+    }
   });
 }