test: stop default exporting CustomDefaultReporter (#7084)

* test: stop default exporting `CustomDefaultReporter`

* fix type default import

Author: Bertie690

test/reporters/custom-default-reporter.ts

@@ -9,11 +9,12 @@ const IGNORE_STACKTRACE: boolean = true;
 
 const F_POINTER = "❯";
 
+// NB: If we ever want to use benchmark support, we will need to move the console log overriding to a mixin (different reporter base classes)
+
 /**
  * Custom Vitest reporter to strip file name headers and `console.log` stack traces from logging output.
  */
-// biome-ignore lint/style/noDefaultExport: Required by Vitest
-export default class CustomDefaultReporter extends DefaultReporter {
+export class CustomDefaultReporter extends DefaultReporter {
   public override onUserConsoleLog(log: UserConsoleLog, taskState?: TestState): void {
     // This code is more or less copied verbatim from the `vitest/reporters` bundled source code, with minor tweaks to use
     // dependencies we actually _have_ (i.e. chalk) rather than ones we don't (i.e. tinyrainbow).

test/setup/test-end-log.ts

@@ -6,7 +6,7 @@
  */
 
 // biome-ignore lint/correctness/noUnusedImports: Biome can't detect unused imports in preceding doc comments
-import type CustomDefaultReporter from "#test/reporters/custom-default-reporter";
+import type { CustomDefaultReporter } from "#test/reporters/custom-default-reporter";
 import { join, relative } from "path";
 import chalk from "chalk";
 import type { RunnerTask, RunnerTaskResult, RunnerTestCase } from "vitest";

vitest.config.ts

@@ -10,10 +10,9 @@ import type { UserConfig } from "vite";
 import { defineConfig } from "vitest/config";
 import { BaseSequencer, type TestSpecification } from "vitest/node";
 import { TEST_TIMEOUT } from "./test/constants";
+import { CustomDefaultReporter } from "./test/reporters/custom-default-reporter";
 import { sharedConfig } from "./vite.config";
 
-const customReporterFile = "./test/reporters/custom-default-reporter.ts" as const;
-
 // biome-ignore lint/style/noDefaultExport: required for vitest
 export default defineConfig(async config => {
   const viteConfig = await sharedConfig(config);
@@ -22,10 +21,10 @@ export default defineConfig(async config => {
     test: {
       passWithNoTests: false,
       reporters: process.env.MERGE_REPORTS
-        ? ["github-actions", customReporterFile]
+        ? ["github-actions", new CustomDefaultReporter()]
         : process.env.GITHUB_ACTIONS
-          ? ["blob", customReporterFile]
-          : [customReporterFile],
+          ? ["blob", new CustomDefaultReporter()]
+          : [new CustomDefaultReporter()],
       env: {
         TZ: "UTC",
       },
@@ -57,7 +56,11 @@ export default defineConfig(async config => {
       coverage: {
         provider: "v8",
         reportsDirectory: "coverage",
-        reporter: process.env.MERGE_REPORTS ? ["text-summary", "json-summary"] : [],
+        reporter: process.env.MERGE_REPORTS
+          ? ["text-summary", "json-summary"]
+          : process.env.GITHUB_ACTIONS
+            ? []
+            : ["text-summary", "html"],
         exclude: ["{src,test}/**/*.d.ts"],
         include: ["src/**/*.ts", "test/utils/**/*.ts"],
       },