test: fix test-debugger-profile for coverage generation

The child process should not inherit NODE_V8_COVERAGE because that clobbers the inspector output the test is checking. PR-URL: #51816 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Ethan Arrowood <[email protected]>
nodejs · Feb 27, 2024 · 2bf03ee · 2bf03ee
1 parent d47a95f
commit 2bf03ee
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/test/parallel/test-debugger-profile.js b/test/parallel/test-debugger-profile.js
@@ -14,7 +14,14 @@ function delay(ms) {
 
 // Profiles.
 {
-  const cli = startCLI(['--port=0', fixtures.path('debugger/empty.js')]);
+  const cli = startCLI(['--port=0', fixtures.path('debugger/empty.js')], [], {
+    env: {
+      ...process.env,
+      // When this test is run with NODE_V8_COVERAGE, it clobbers the inspector
+      // output, so override to disable coverage for the child process.
+      NODE_V8_COVERAGE: undefined,
+    }
+  });
 
   function onFatal(error) {
     cli.quit();