fixup!: add xfail cases to all reporter tests

Author: JakobJingleheimer

test/fixtures/test-runner/output/dot_reporter.js

@@ -3,5 +3,12 @@ require('../../../common');
 const fixtures = require('../../../common/fixtures');
 const spawn = require('node:child_process').spawn;
 
-spawn(process.execPath,
-      ['--no-warnings', '--test-reporter', 'dot', fixtures.path('test-runner/output/output.js')], { stdio: 'inherit' });
+spawn(
+  process.execPath,
+  [
+    '--no-warnings',
+    '--test-reporter', 'dot',
+    fixtures.path('test-runner/output/output.js'),
+  ],
+  { stdio: 'inherit' },
+);

test/fixtures/test-runner/output/junit_reporter.js

@@ -3,9 +3,12 @@ require('../../../common');
 const fixtures = require('../../../common/fixtures');
 const spawn = require('node:child_process').spawn;
 
-spawn(process.execPath,
-      [
-        '--no-warnings', '--test-reporter', 'junit',
-        fixtures.path('test-runner/output/output.js'),
-      ],
-      { stdio: 'inherit' });
+spawn(
+  process.execPath,
+  [
+    '--no-warnings',
+    '--test-reporter', 'junit',
+    fixtures.path('test-runner/output/output.js'),
+  ],
+  { stdio: 'inherit' },
+);

test/fixtures/test-runner/output/lcov_reporter.js

@@ -3,14 +3,14 @@ require('../../../common');
 const fixtures = require('../../../common/fixtures');
 const spawn = require('node:child_process').spawn;
 
-spawn(process.execPath,
-      [
-        '--no-warnings',
-        '--experimental-test-coverage',
-        '--test-coverage-exclude=!test/**',
-        '--test-reporter',
-        'lcov',
-        fixtures.path('test-runner/output/output.js'),
-      ],
-      { stdio: 'inherit' },
+spawn(
+  process.execPath,
+  [
+    '--no-warnings',
+    '--experimental-test-coverage',
+    '--test-coverage-exclude=!test/**',
+    '--test-reporter', 'lcov',
+    fixtures.path('test-runner/output/output.js'),
+  ],
+  { stdio: 'inherit' },
 );

test/fixtures/test-runner/output/output.js

@@ -5,6 +5,30 @@ const assert = require('node:assert');
 const test = require('node:test');
 const util = require('util');
 
+test.xfail('sync expect fail (method)', () => {
+  throw new Error('should pass');
+});
+
+test('sync expect fail (options)', { xfail: true }, () => {
+  throw new Error('should pass');
+});
+
+test.xfail('async expect fail (method)', async () => {
+  throw new Error('should pass');
+});
+
+test('async expect fail (options)', { xfail: true }, async () => {
+  throw new Error('should pass');
+});
+
+test.todo('sync todo with expect fail', { xfail: true }, () => {
+  throw new Error('should not count as an expected failure');
+});
+
+test.skip('sync skip expect fail', { xfail: true }, () => {
+  throw new Error('should not fail');
+});
+
 test('sync pass todo', (t) => {
   t.todo();
 });

test/fixtures/test-runner/output/output.snapshot

@@ -3,10 +3,14 @@ require('../../../common');
 const fixtures = require('../../../common/fixtures');
 const spawn = require('node:child_process').spawn;
 
-spawn(process.execPath,
-      [
-        '--no-warnings', '--test', '--test-reporter', 'tap',
-        fixtures.path('test-runner/output/output.js'),
-        fixtures.path('test-runner/output/single.js'),
-      ],
-      { stdio: 'inherit' });
+spawn(
+  process.execPath,
+  [
+    '--no-warnings',
+    '--test-reporter', 'tap',
+    '--test',
+    fixtures.path('test-runner/output/output.js'),
+    fixtures.path('test-runner/output/single.js'),
+  ],
+  { stdio: 'inherit' },
+);

test/fixtures/test-runner/output/output_cli.js

@@ -3,9 +3,16 @@ require('../../../common');
 const fixtures = require('../../../common/fixtures');
 const spawn = require('node:child_process').spawn;
 
-const child = spawn(process.execPath,
-                    ['--no-warnings', '--test-reporter', 'spec', fixtures.path('test-runner/output/output.js')],
-                    { stdio: 'pipe' });
+const child = spawn(
+  process.execPath,
+  [
+    '--no-warnings',
+    '--test-reporter', 'spec',
+    fixtures.path('test-runner/output/output.js'),
+  ],
+  { stdio: 'pipe' },
+);
+
 // eslint-disable-next-line no-control-regex
 child.stdout.on('data', (d) => process.stdout.write(d.toString().replace(/[^\x00-\x7F]/g, '').replace(/\u001b\[\d+m/g, '')));
 child.stderr.pipe(process.stderr);

test/fixtures/test-runner/output/spec_reporter.js

@@ -3,12 +3,17 @@ require('../../../common');
 const fixtures = require('../../../common/fixtures');
 const spawn = require('node:child_process').spawn;
 
-const child = spawn(process.execPath,
-                    [
-                      '--no-warnings', '--test', '--test-reporter', 'spec',
-                      fixtures.path('test-runner/output/output.js'),
-                    ],
-                    { stdio: 'pipe' });
+const child = spawn(
+  process.execPath,
+  [
+    '--no-warnings',
+    '--test-reporter', 'spec',
+    '--test',
+    fixtures.path('test-runner/output/output.js'),
+  ],
+  { stdio: 'pipe' },
+);
+
 // eslint-disable-next-line no-control-regex
 child.stdout.on('data', (d) => process.stdout.write(d.toString().replace(/[^\x00-\x7F]/g, '').replace(/\u001b\[\d+m/g, '')));
 child.stderr.pipe(process.stderr);