test(vitest): update configs to reflect pkg updates

Author: trevor-anderson

vitest.config.ts

@@ -1,45 +1,45 @@
 import { defineConfig, coverageConfigDefaults } from "vitest/config";
 import GithubActionsReporter from "vitest-github-actions-reporter";
-import type { EmptyObject } from "type-fest";
+
+const isCI = !!process.env.CI;
+const isGitHubActionWorkflow = !!process.env.GITHUB_ACTIONS;
 
 export default defineConfig({
   test: {
     /**
      * `restoreMocks` accomplishes the following:
-     * - clears all spies of `spy.mock.calls` and `spy.mock.results` (same as clearMocks:true)
-     * - removes any mocked implementations (same as mockReset:true)
-     * - restores the original implementation so fns don't return undefined like with mockReset
+     *   - Removes any mocked implementations (same as mockReset:true).
+     *   - Restores the original implementation so fns don't return undefined like with mockReset.
+     *   - Clears mock-state for spies created "manually" via `vi.spyOn` (same as clearMocks:true).
+     *     > Mock-state is NOT cleared for spies created via *automocking* (`mockReset` does this).
+     *
+     * `mockReset` accomplishes the following:
+     *   - Resets mock-state (call counts, arguments, etc.) for all mocks, including automocks.
      */
     restoreMocks: true,
+    mockReset: true,
     unstubGlobals: true,
     globals: true,
     silent: true,
     hideSkippedTests: true,
     watch: false,
-
+    bail: isCI ? 1 : 0, // If in CI, bail on first test failure, else run all tests
     environment: "node",
-    setupFiles: "./vitest.setup.ts",
-
-    include: ["**/?(*.){test,spec}.?(c|m)[tj]s?(x)"],
-
+    setupFiles: ["./vitest.setup.ts"],
     reporters: [
       "default",
       // GithubActionsReporter is used to format test results for GitHub Actions
-      ...(process.env.GITHUB_ACTIONS ? [new GithubActionsReporter()] : []),
+      ...(isGitHubActionWorkflow ? [new GithubActionsReporter()] : []),
     ],
-
     coverage: {
       include: ["src/**/*.{js,ts}"],
-      exclude: [
-        ...coverageConfigDefaults.exclude,
-        "**/tests/**/*",
-        "**/__mocks__/**/*",
-        "__mocks__/**/*",
-      ],
+      exclude: [...coverageConfigDefaults.exclude, "**/index.ts"],
       reporter: [
-        ...(coverageConfigDefaults.reporter as Array<[string, EmptyObject]>),
-        // "json-summary" is used by the vitest-coverage-report GitHub Action
-        ...(process.env.GITHUB_ACTIONS ? ["json-summary"] : []),
+        "text", // <-- for console output (even in CI, for logging/debugging purposes)
+        "lcov", // <-- for uploading coverage info to Codecov
+        ...(isGitHubActionWorkflow
+          ? ["json-summary"] // <-- for the vitest-coverage-report GitHub Action
+          : []),
       ],
     },
   },