Repository: nodejs/node
Pull request: test: fix skipping behavior for `test-runner-run-files-undefined`

Refs: https://github.com/nodejs/node/pull/62020#discussion_r2864039892

<!--
Before submitting a pull request, please read:

- the CONTRIBUTING guide at https://github.com/nodejs/node/blob/HEAD/CONTRIBUTING.md
- the commit message formatting guidelines at
  https://github.com/nodejs/node/blob/HEAD/doc/contributing/pull-requests.md#commit-message-guidelines

For code changes:
1. Include tests for any bug fixes or new features.
2. Update documentation if relevant.
3. Ensure that `make -j4 test` (UNIX), or `vcbuild test` (Windows) passes.

If you believe this PR should be highlighted in the Node.js CHANGELOG
please add the `notable-change` label.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.
-->


Changed files:
filename: test/parallel/test-runner-run-files-undefined.mjs
patch: @@ -1,61 +1,31 @@
-import * as common from '../common/index.mjs';
+import '../common/index.mjs';
 import tmpdir from '../common/tmpdir.js';
-import { describe, it, run, beforeEach } from 'node:test';
-import { dot, spec, tap } from 'node:test/reporters';
-import { fork } from 'node:child_process';
-import assert from 'node:assert';
-
-if (common.hasCrypto) {
-  console.log('1..0 # Skipped: no crypto');
-  process.exit(0);
-}
-
-if (process.env.CHILD === 'true') {
-  describe('require(\'node:test\').run with no files', { concurrency: true }, () => {
-    beforeEach(() => {
-      tmpdir.refresh();
-      process.chdir(tmpdir.path);
+import { spawnSyncAndExitWithoutError } from '../common/child_process.js';
+
+async function runTests(run, reporters) {
+  for (const reporterName of Object.keys(reporters)) {
+    if (reporterName === 'default') continue;
+    console.log({ reporterName });
+
+    const stream = run({
+      files: undefined
+    }).compose(reporters[reporterName]);
+    stream.on('test:fail', () => {
+      throw new Error('Received test:fail with ' + reporterName);
     });
-
-    it('should neither pass or fail', async () => {
-      const stream = run({
-        files: undefined
-      }).compose(tap);
-      stream.on('test:fail', common.mustNotCall());
-      stream.on('test:pass', common.mustNotCall());
-
-      // eslint-disable-next-line no-unused-vars
-      for await (const _ of stream);
+    stream.on('test:pass', () => {
+      throw new Error('Received test:pass with ' + reporterName);
     });
 
-    it('can use the spec reporter', async () => {
-      const stream = run({
-        files: undefined
-      }).compose(spec);
-      stream.on('test:fail', common.mustNotCall());
-      stream.on('test:pass', common.mustNotCall());
-
-      // eslint-disable-next-line no-unused-vars
-      for await (const _ of stream);
-    });
+    // eslint-disable-next-line no-unused-vars
+    for await (const _ of stream);
+  }
+}
 
-    it('can use the dot reporter', async () => {
-      const stream = run({
-        files: undefined
-      }).compose(dot);
-      stream.on('test:fail', common.mustNotCall());
-      stream.on('test:pass', common.mustNotCall());
+tmpdir.refresh();
+spawnSyncAndExitWithoutError(process.execPath, ['--input-type=module', '-e', `
+  import { run } from 'node:test';
+  import * as reporters from 'node:test/reporters';
 
-      // eslint-disable-next-line no-unused-vars
-      for await (const _ of stream);
-    });
-  });
-} else if (common.isAIX) {
-  console.log('1..0 # Skipped: test runner without specifying files fails on AIX');
-} else {
-  fork(import.meta.filename, [], {
-    env: { CHILD: 'true' }
-  }).on('exit', common.mustCall((code) => {
-    assert.strictEqual(code, 0);
-  }));
-}
+  await (${runTests})(run, reporters);
+`], { cwd: tmpdir.path });