test_runner: use run() options with isolation="none"

When using run() programatically with isolation="none",
testNamePatterns, testSkipPattersn, and only were ignored.
This combination of options only worked when set via CLI flags,
because parseCommandLine() is still used to seed globalOptions.

Author: inukshuk

lib/internal/test_runner/runner.js

@@ -784,6 +784,19 @@ function run(options = kEmptyObject) {
     cwd,
     globalSetupPath,
   };
+
+  if (isolation === 'none') {
+    if (testNamePatterns != null) {
+      globalOptions.testNamePatterns = testNamePatterns;
+    }
+    if (testSkipPatterns != null) {
+      globalOptions.testSkipPatterns = testSkipPatterns;
+    }
+    if (only != null) {
+      globalOptions.only = only;
+    }
+  }
+
   const root = createTestTree(rootTestOptions, globalOptions);
   let testFiles = files ?? createTestFileList(globPatterns, cwd);
   const { isTestRunner } = globalOptions;

test/fixtures/test-runner/test-runner-isolation-none.mjs

@@ -0,0 +1,32 @@
+import { run } from 'node:test';
+import { tap } from 'node:test/reporters';
+import { parseArgs } from 'node:util';
+
+const {
+  values,
+} = parseArgs({
+  args: process.argv.slice(2),
+  options: {
+    file: { type: 'string' },
+    only: { type: 'boolean' },
+    'name-pattern': { type: 'string' },
+    'skip-pattern': { type: 'string' },
+  },
+});
+
+const opts = {
+  isolation: 'none',
+  files: [values.file],
+};
+
+if (values.only) {
+  opts.only = true;
+}
+if (values['name-pattern']) {
+  opts.testNamePatterns = [new RegExp(values['name-pattern'])];
+}
+if (values['skip-pattern']) {
+  opts.testSkipPatterns = [new RegExp(values['skip-pattern'])];
+}
+
+run(opts).compose(tap).pipe(process.stdout);

test/parallel/test-runner-run.mjs

@@ -689,6 +689,48 @@ describe('forceExit', () => {
   });
 });
 
+describe('with isolation="none"', () => {
+  const isolationNoneFixture = fixtures.path('test-runner', 'test-runner-isolation-none.mjs');
+
+  it('should pass only to children', { timeout: 10000 }, async () => {
+    const child = await common.spawnPromisified(process.execPath, [
+      isolationNoneFixture,
+      '--file', join(testFixtures, 'test_only.js'),
+      '--only',
+    ]);
+
+    assert.strictEqual(child.stderr, '');
+    assert.strictEqual(child.code, 0);
+    assert.match(child.stdout, /ok 1 - this should be executed/);
+    assert.match(child.stdout, /# tests 1/);
+  });
+
+  it('should skip tests not matching testNamePatterns - RegExp', { timeout: 10000 }, async () => {
+    const child = await common.spawnPromisified(process.execPath, [
+      isolationNoneFixture,
+      '--file', join(testFixtures, 'default-behavior/test/skip_by_name.cjs'),
+      '--name-pattern', 'executed',
+    ]);
+
+    assert.strictEqual(child.stderr, '');
+    assert.strictEqual(child.code, 0);
+    assert.match(child.stdout, /ok 1 - this should be executed/);
+    assert.match(child.stdout, /# tests 1/);
+  });
+
+  it('should skip tests matching testSkipPatterns - RegExp', { timeout: 10000 }, async () => {
+    const child = await common.spawnPromisified(process.execPath, [
+      isolationNoneFixture,
+      '--file', join(testFixtures, 'default-behavior/test/skip_by_name.cjs'),
+      '--skip-pattern', 'skipped',
+    ]);
+
+    assert.strictEqual(child.stderr, '');
+    assert.strictEqual(child.code, 0);
+    assert.match(child.stdout, /ok 1 - this should be executed/);
+    assert.match(child.stdout, /# tests 1/);
+  });
+});
 
 // exitHandler doesn't run until after the tests / after hooks finish.
 process.on('exit', () => {