fixup! test: fix skipping behavior for test-runner-run-files-undefined

Author: aduh95

test/parallel/test-runner-run-files-undefined.mjs

@@ -1,58 +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.isAIX) {
-  common.skip('test runner without specifying files fails on AIX');
-}
-
-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 {
-  fork(import.meta.filename, [], {
-    env: { ...process.env, CHILD: 'true' }
-  }).on('exit', common.mustCall((code) => {
-    assert.strictEqual(code, 0);
-  }));
-}
+  await (${runTests})(run, reporters);
+`], { cwd: tmpdir.path });