From 9c4aef143f04f9fbd7e0fdeeeac25c7454d105db Mon Sep 17 00:00:00 2001
From: Pietro Marchini <pietro.marchini94@gmail.com>
Date: Wed, 11 Sep 2024 18:22:13 +0200
Subject: [PATCH] test_runner: refactor cwd handling in test runner

---
 doc/api/cli.md                       | 11 -----------
 lib/internal/test_runner/coverage.js |  2 +-
 lib/internal/test_runner/harness.js  |  6 ++++--
 lib/internal/test_runner/runner.js   |  6 +++---
 lib/internal/test_runner/utils.js    |  2 --
 src/node_options.cc                  |  4 ----
 src/node_options.h                   |  1 -
 7 files changed, 8 insertions(+), 24 deletions(-)

diff --git a/doc/api/cli.md b/doc/api/cli.md
index ce9cb06b84a686..01efb97e31be23 100644
--- a/doc/api/cli.md
+++ b/doc/api/cli.md
@@ -1097,17 +1097,6 @@ generated as part of the test runner output. If no tests are run, a coverage
 report is not generated. See the documentation on
 [collecting code coverage from tests][] for more details.
 
-### `--experimental-test-cwd=directory`
-
-<!-- YAML
-added: CHANGEME
--->
-
-> Stability: 1.0 - Early development
-
-Set the current working directory for the test runner. If not specified, the current
-working directory is used.
-
 ### `--experimental-test-isolation=mode`
 
 <!-- YAML
diff --git a/lib/internal/test_runner/coverage.js b/lib/internal/test_runner/coverage.js
index e3961d9bf06747..0ffbcb479ff45c 100644
--- a/lib/internal/test_runner/coverage.js
+++ b/lib/internal/test_runner/coverage.js
@@ -477,7 +477,7 @@ function sortCoverageFiles(a, b) {
 
 function setupCoverage(options) {
   let originalCoverageDirectory = process.env.NODE_V8_COVERAGE;
-  const cwd = options.testCwd;
+  const cwd = options.cwd ?? process.cwd();
 
   if (originalCoverageDirectory) {
     // NODE_V8_COVERAGE was already specified. Convert it to an absolute path
diff --git a/lib/internal/test_runner/harness.js b/lib/internal/test_runner/harness.js
index dc396c724c84fa..d52f1d98e9aa03 100644
--- a/lib/internal/test_runner/harness.js
+++ b/lib/internal/test_runner/harness.js
@@ -196,10 +196,12 @@ function setupProcessState(root, globalOptions) {
 
   hook.enable();
 
+  const cwd = globalOptions.cwd || process.cwd();
+
   const exceptionHandler =
-    createProcessEventHandler('uncaughtException', root, globalOptions.testCwd);
+    createProcessEventHandler('uncaughtException', root, cwd);
   const rejectionHandler =
-    createProcessEventHandler('unhandledRejection', root, globalOptions.testCwd);
+    createProcessEventHandler('unhandledRejection', root, cwd);
   const coverage = configureCoverage(root, globalOptions);
   const exitHandler = async () => {
     if (root.subtests.length === 0 && (root.hooks.before.length > 0 || root.hooks.after.length > 0)) {
diff --git a/lib/internal/test_runner/runner.js b/lib/internal/test_runner/runner.js
index 4139fa69799aca..0272230f0fb4bc 100644
--- a/lib/internal/test_runner/runner.js
+++ b/lib/internal/test_runner/runner.js
@@ -95,7 +95,7 @@ let debug = require('internal/util/debuglog').debuglog('test_runner', (fn) => {
 });
 
 const kIsolatedProcessName = Symbol('kIsolatedProcessName');
-const kFilterArgs = ['--test', '--experimental-test-coverage', '--watch', '--experimental-test-cwd'];
+const kFilterArgs = ['--test', '--experimental-test-coverage', '--watch'];
 const kFilterArgValues = ['--test-reporter', '--test-reporter-destination'];
 const kDiagnosticsFilterArgs = ['tests', 'suites', 'pass', 'fail', 'cancelled', 'skipped', 'todo', 'duration_ms'];
 
@@ -626,6 +626,7 @@ function run(options = kEmptyObject) {
     // behavior has relied on it, so removing it must be done in a semver major.
     ...parseCommandLine(),
     setup,  // This line can be removed when parseCommandLine() is removed here.
+    cwd,
   };
   const root = createTestTree(rootTestOptions, globalOptions);
   let testFiles = files ?? createTestFileList(globPatterns, cwd);
@@ -678,8 +679,7 @@ function run(options = kEmptyObject) {
     };
   } else if (isolation === 'none') {
     if (watch) {
-      // TODO 
-      const absoluteTestFiles = ArrayPrototypeMap(testFiles, (file) => isAbsolute(file) ? file : resolve(cwd, file));
+      const absoluteTestFiles = ArrayPrototypeMap(testFiles, (file) => (isAbsolute(file) ? file : resolve(cwd, file)));
       filesWatcher = watchFiles(absoluteTestFiles, opts);
       runFiles = async () => {
         root.harness.bootstrapPromise = null;
diff --git a/lib/internal/test_runner/utils.js b/lib/internal/test_runner/utils.js
index f3cd8838c0422c..0d6fab11d10d82 100644
--- a/lib/internal/test_runner/utils.js
+++ b/lib/internal/test_runner/utils.js
@@ -200,7 +200,6 @@ function parseCommandLine() {
   let functionCoverage;
   let destinations;
   let isolation;
-  let testCwd = getOptionValue('--experimental-test-cwd') || process.cwd()
   let only = getOptionValue('--test-only');
   let reporters;
   let shard;
@@ -312,7 +311,6 @@ function parseCommandLine() {
     destinations,
     forceExit,
     isolation,
-    testCwd,
     branchCoverage,
     functionCoverage,
     lineCoverage,
diff --git a/src/node_options.cc b/src/node_options.cc
index 5ba315582b7c35..ee92d31e2e0d95 100644
--- a/src/node_options.cc
+++ b/src/node_options.cc
@@ -670,10 +670,6 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
             "the line coverage minimum threshold",
             &EnvironmentOptions::test_coverage_lines,
             kAllowedInEnvvar);
-
-  AddOption("--experimental-test-cwd", 
-            "specify the working directory for the test runner",
-            &EnvironmentOptions::test_runner_cwd);
   AddOption("--experimental-test-isolation",
             "configures the type of test isolation used in the test runner",
             &EnvironmentOptions::test_isolation);
diff --git a/src/node_options.h b/src/node_options.h
index 529997c2cae58a..5f186ae86f0bcc 100644
--- a/src/node_options.h
+++ b/src/node_options.h
@@ -195,7 +195,6 @@ class EnvironmentOptions : public Options {
   bool test_only = false;
   bool test_udp_no_try_send = false;
   std::string test_isolation = "process";
-  std::string test_runner_cwd;
   std::string test_shard;
   std::vector<std::string> test_skip_pattern;
   std::vector<std::string> coverage_include_pattern;