benchmark: add --test-reporter-destination to flags benchmarks

Author: brunocroh

benchmark/test_runner/global-concurrent-tests.js

@@ -2,13 +2,20 @@
 const common = require('../common');
 const { it } = require('node:test');
 
-const bench = common.createBenchmark(main, {
-  n: [100, 1000, 1e4],
-  type: ['sync', 'async'],
-}, {
-  // We don't want to test the reporter here
-  flags: ['--test-reporter=./benchmark/fixtures/empty-test-reporter.js'],
-});
+const bench = common.createBenchmark(
+  main,
+  {
+    n: [100, 1000, 1e4],
+    type: ['sync', 'async'],
+  },
+  {
+    // We don't want to test the reporter here
+    flags: [
+      '--test-reporter=./benchmark/fixtures/empty-test-reporter.js',
+      '--test-reporter-destination=stdout',
+    ],
+  },
+);
 
 async function run(n, type) {
   const promises = new Array(n);

benchmark/test_runner/global-sequential-tests.js

@@ -2,14 +2,20 @@
 const common = require('../common');
 const { it } = require('node:test');
 
-
-const bench = common.createBenchmark(main, {
-  n: [100, 1000, 1e4],
-  type: ['sync', 'async'],
-}, {
-  // We don't want to test the reporter here
-  flags: ['--test-reporter=./benchmark/fixtures/empty-test-reporter.js'],
-});
+const bench = common.createBenchmark(
+  main,
+  {
+    n: [100, 1000, 1e4],
+    type: ['sync', 'async'],
+  },
+  {
+    // We don't want to test the reporter here
+    flags: [
+      '--test-reporter=./benchmark/fixtures/empty-test-reporter.js',
+      '--test-reporter-destination=stdout',
+    ],
+  },
+);
 
 async function run(n, type) {
   // eslint-disable-next-line no-unused-vars

benchmark/test_runner/mock-fn.js

@@ -4,13 +4,20 @@ const common = require('../common');
 const assert = require('node:assert');
 const { test } = require('node:test');
 
-const bench = common.createBenchmark(main, {
-  n: [1e6],
-  mode: ['define', 'execute'],
-}, {
-  // We don't want to test the reporter here
-  flags: ['--test-reporter=./benchmark/fixtures/empty-test-reporter.js'],
-});
+const bench = common.createBenchmark(
+  main,
+  {
+    n: [1e6],
+    mode: ['define', 'execute'],
+  },
+  {
+    // We don't want to test the reporter here
+    flags: [
+      '--test-reporter=./benchmark/fixtures/empty-test-reporter.js',
+      '--test-reporter-destination=stdout',
+    ],
+  },
+);
 
 const noop = () => {};
 

benchmark/test_runner/run-single-test-file.js

@@ -31,13 +31,20 @@ function setup(numberOfTestFiles) {
  * Specifically, it compares the performance of running tests in the
  * same process versus creating multiple processes.
  */
-const bench = common.createBenchmark(main, {
-  numberOfTestFiles: [1, 10, 100],
-  isolation: ['none', 'process'],
-}, {
-  // We don't want to test the reporter here
-  flags: ['--test-reporter=./benchmark/fixtures/empty-test-reporter.js'],
-});
+const bench = common.createBenchmark(
+  main,
+  {
+    numberOfTestFiles: [1, 10, 100],
+    isolation: ['none', 'process'],
+  },
+  {
+    // We don't want to test the reporter here
+    flags: [
+      '--test-reporter=./benchmark/fixtures/empty-test-reporter.js',
+      '--test-reporter-destination=stdout',
+    ],
+  },
+);
 
 async function runBenchmark({ numberOfTestFiles, isolation }) {
   const dirPath = getTestDirPath(numberOfTestFiles);

benchmark/test_runner/suite-tests.js

@@ -6,15 +6,22 @@ const reporter = require('../fixtures/empty-test-reporter');
 
 const { describe, it } = require('node:test');
 
-const bench = common.createBenchmark(main, {
-  numberOfSuites: [10, 100],
-  testsPerSuite: [10, 100, 1000],
-  testType: ['sync', 'async'],
-  concurrency: ['yes', 'no'],
-}, {
-  // We don't want to test the reporter here
-  flags: ['--test-reporter=./benchmark/fixtures/empty-test-reporter.js'],
-});
+const bench = common.createBenchmark(
+  main,
+  {
+    numberOfSuites: [10, 100],
+    testsPerSuite: [10, 100, 1000],
+    testType: ['sync', 'async'],
+    concurrency: ['yes', 'no'],
+  },
+  {
+    // We don't want to test the reporter here
+    flags: [
+      '--test-reporter=./benchmark/fixtures/empty-test-reporter.js',
+      '--test-reporter-destination=stdout',
+    ],
+  },
+);
 
 async function run({ numberOfSuites, testsPerSuite, testType, concurrency }) {
   concurrency = concurrency === 'yes';