src: rename config file testRunner to test

marco-ippolito · aduh95 · commit f07d76613ec0 · 2026-07-21T16:57:46.000+02:00
PR-URL: #60798 Backport-PR-URL: #63659 Signed-off-by: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
diff --git a/doc/api/cli.md b/doc/api/cli.md
@@ -987,7 +987,7 @@ in the `$schema` must be replaced with the version of Node.js you are using or
     "watch-path": "src",
     "watch-preserve-output": true
   },
-  "testRunner": {
+  "test": {
     "test-isolation": "process"
   },
   "watch": {
@@ -1000,7 +1000,7 @@ The configuration file supports namespace-specific options:
 
 * The `nodeOptions` field contains CLI flags that are allowed in [`NODE_OPTIONS`][].
 
-* Namespace fields like `testRunner` contain configuration specific to that subsystem.
+* Namespace fields like `test` contain configuration specific to that subsystem.
 
 No-op flags are not supported.
 Not all V8 flags are currently supported.
diff --git a/doc/node-config-schema.json b/doc/node-config-schema.json
@@ -816,7 +816,7 @@
         }
       }
     },
-    "testRunner": {
+    "test": {
       "type": "object",
       "additionalProperties": false,
       "required": [],
diff --git a/src/node_config_file.cc b/src/node_config_file.cc
@@ -260,8 +260,20 @@ ParseResult ConfigReader::ParseConfig(const std::string_view& config_path) {
       return ParseResult::InvalidContent;
     }
 
-    // Check if this field is a valid namespace
     std::string namespace_name(field_name);
+
+    // TODO(@marco-ippolito): Remove warning for testRunner namespace
+    if (namespace_name == "testRunner") {
+      FPrintF(stderr,
+              "the \"testRunner\" namespace has been removed. "
+              "Use \"test\" instead.\n");
+      // Better to throw an error than to ignore it
+      // Otherwise users might think their test suite is green
+      // when it's not running
+      return ParseResult::InvalidContent;
+    }
+
+    // Check if this field is a valid namespace
     if (!valid_namespaces.contains(namespace_name)) {
       // If not, skip it
       continue;
diff --git a/src/node_options.h b/src/node_options.h
@@ -429,7 +429,7 @@ std::vector<std::string> MapAvailableNamespaces();
 // Define all namespace entries
 #define OPTION_NAMESPACE_LIST(V)                                               \
   V(kNoNamespace, "")                                                          \
-  V(kTestRunnerNamespace, "testRunner")                                        \
+  V(kTestRunnerNamespace, "test")                                              \
   V(kWatchNamespace, "watch")                                                  \
   V(kPermissionNamespace, "permission")
 
diff --git a/test/fixtures/options-as-flags/test-config.json b/test/fixtures/options-as-flags/test-config.json
@@ -3,7 +3,7 @@
     "experimental-transform-types": true,
     "max-http-header-size": 8192
   },
-  "testRunner": {
+  "test": {
     "test-isolation": "none"
   }
 }
diff --git a/test/fixtures/rc/deprecated-testrunner-namespace.json b/test/fixtures/rc/deprecated-testrunner-namespace.json
@@ -0,0 +1,5 @@
+{
+  "testRunner": {
+    "test-isolation": "none"
+  }
+}
diff --git a/test/fixtures/rc/duplicate-namespace-option/node.config.json b/test/fixtures/rc/duplicate-namespace-option/node.config.json
@@ -1,5 +1,5 @@
 {
-    "testRunner": {
+    "test": {
         "test-name-pattern": "first-pattern",
         "test-name-pattern": "second-pattern"
     }
diff --git a/test/fixtures/rc/empty-valid-namespace.json b/test/fixtures/rc/empty-valid-namespace.json
@@ -1,3 +1,3 @@
 {
-    "testRunner": {}
+    "test": {}
 }
diff --git a/test/fixtures/rc/namespace-with-array.json b/test/fixtures/rc/namespace-with-array.json
@@ -1,5 +1,5 @@
 {
-    "testRunner": {
+    "test": {
         "test-coverage-exclude": ["config-pattern1", "config-pattern2"]
     }
 }
diff --git a/test/fixtures/rc/namespace-with-disallowed-envvar.json b/test/fixtures/rc/namespace-with-disallowed-envvar.json
@@ -1,5 +1,5 @@
 {
-    "testRunner": {
+    "test": {
        "test-concurrency": 1,
        "experimental-test-coverage": true
     }
diff --git a/test/fixtures/rc/namespaced/node.config.json b/test/fixtures/rc/namespaced/node.config.json
@@ -1,5 +1,5 @@
 {
-    "testRunner": {
+    "test": {
         "test-isolation": "none"
     }
 }
diff --git a/test/fixtures/rc/override-namespace.json b/test/fixtures/rc/override-namespace.json
@@ -1,5 +1,5 @@
 {
-    "testRunner": {
+    "test": {
         "test-isolation": "process"
     },
     "nodeOptions": {
diff --git a/test/fixtures/rc/override-node-option-with-namespace.json b/test/fixtures/rc/override-node-option-with-namespace.json
@@ -2,7 +2,7 @@
     "nodeOptions": {
         "test-isolation": "none"
     },
-    "testRunner": {
+    "test": {
         "test-isolation": "process"
     }
 }
diff --git a/test/fixtures/rc/unknown-flag-namespace.json b/test/fixtures/rc/unknown-flag-namespace.json
@@ -1,5 +1,5 @@
 {
-    "testRunner": {
+    "test": {
         "unknown-flag": true
     }
 }
diff --git a/test/fixtures/test-runner/options-propagation/node.config.json b/test/fixtures/test-runner/options-propagation/node.config.json
@@ -1,5 +1,5 @@
 {
-    "testRunner": {
+    "test": {
         "experimental-test-coverage": true
     }
 }
diff --git a/test/parallel/test-config-file.js b/test/parallel/test-config-file.js
@@ -429,7 +429,7 @@ describe('namespace-scoped options', () => {
       fixtures.path('rc/unknown-flag-namespace.json'),
       '-p', '"Hello, World!"',
     ]);
-    assert.match(result.stderr, /Unknown or not allowed option unknown-flag for namespace testRunner/);
+    assert.match(result.stderr, /Unknown or not allowed option unknown-flag for namespace test/);
     assert.strictEqual(result.stdout, '');
     assert.strictEqual(result.code, 9);
   });
@@ -550,4 +550,16 @@ describe('namespace-scoped options', () => {
     assert.strictEqual(result.stdout, '2\n');
     assert.strictEqual(result.code, 0);
   });
+
+  it('should throw an error for removed "testRunner" namespace', async () => {
+    const result = await spawnPromisified(process.execPath, [
+      '--no-warnings',
+      '--experimental-config-file',
+      fixtures.path('rc/deprecated-testrunner-namespace.json'),
+      '-p', '"Hello, World!"',
+    ]);
+    assert.match(result.stderr, /the "testRunner" namespace has been removed\. Use "test" instead\./);
+    assert.strictEqual(result.stdout, '');
+    assert.strictEqual(result.code, 9);
+  });
 });
diff --git a/test/parallel/test-runner-flag-propagation.js b/test/parallel/test-runner-flag-propagation.js
@@ -68,19 +68,19 @@ describe('test runner flag propagation', () => {
 
   describe('via config file', () => {
     const configFilePropagationTests = [
-      ['--test-concurrency', 2, 2, 'testRunner'],
-      ['--test-timeout', 5000, 5000, 'testRunner'],
-      ['--test-coverage-branches', 100, 100, 'testRunner'],
-      ['--test-coverage-functions', 100, 100, 'testRunner'],
-      ['--test-coverage-lines', 100, 100, 'testRunner'],
-      ['--experimental-test-coverage', true, false, 'testRunner'],
-      ['--test-coverage-exclude', 'test/**', 'test/**', 'testRunner'],
-      ['--test-coverage-include', 'src/**', 'src/**', 'testRunner'],
-      ['--test-update-snapshots', true, true, 'testRunner'],
-      ['--test-concurrency', 3, 3, 'testRunner'],
-      ['--test-timeout', 2500, 2500, 'testRunner'],
-      ['--test-coverage-branches', 90, 90, 'testRunner'],
-      ['--test-coverage-functions', 85, 85, 'testRunner'],
+      ['--test-concurrency', 2, 2, 'test'],
+      ['--test-timeout', 5000, 5000, 'test'],
+      ['--test-coverage-branches', 100, 100, 'test'],
+      ['--test-coverage-functions', 100, 100, 'test'],
+      ['--test-coverage-lines', 100, 100, 'test'],
+      ['--experimental-test-coverage', true, false, 'test'],
+      ['--test-coverage-exclude', 'test/**', 'test/**', 'test'],
+      ['--test-coverage-include', 'src/**', 'src/**', 'test'],
+      ['--test-update-snapshots', true, true, 'test'],
+      ['--test-concurrency', 3, 3, 'test'],
+      ['--test-timeout', 2500, 2500, 'test'],
+      ['--test-coverage-branches', 90, 90, 'test'],
+      ['--test-coverage-functions', 85, 85, 'test'],
     ];
 
     for (const [flagName, configValue, expectedValue, namespace] of configFilePropagationTests) {

Original file line number	Diff line number	Diff line change
`@@ -816,7 +816,7 @@`
`816`	`816`	`}`
`817`	`817`	`}`
`818`	`818`	`},`
`819`		`- "testRunner": {`
	`819`	`+ "test": {`
`820`	`820`	`"type": "object",`
`821`	`821`	`"additionalProperties": false,`
`822`	`822`	`"required": [],`
Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@`
`3`	`3`	`"experimental-transform-types": true,`
`4`	`4`	`"max-http-header-size": 8192`
`5`	`5`	`},`
`6`		`- "testRunner": {`
	`6`	`+ "test": {`
`7`	`7`	`"test-isolation": "none"`
`8`	`8`	`}`
`9`	`9`	`}`
-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +{
 +  "testRunner": {
 +    "test-isolation": "none"
 +  }
 +}
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`{`
`2`		`- "testRunner": {`
	`2`	`+ "test": {`
`3`	`3`	`"test-name-pattern": "first-pattern",`
`4`	`4`	`"test-name-pattern": "second-pattern"`
`5`	`5`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,3 @@`
`1`	`1`	`{`
`2`		`- "testRunner": {}`
	`2`	`+ "test": {}`
`3`	`3`	`}`