refactor(testing): enhance documentation and Vitest configuration setup

Author: Szymon.Poltorak

testing/test-setup-config/README.md

@@ -4,23 +4,23 @@ Standardized Vitest configuration for the Code PushUp monorepo.
 
 ### Usage
 
-**Unit tests:**
+#### Unit tests:
 
 ```typescript
 import { createUnitTestConfig } from '@code-pushup/test-setup-config';
 
 export default createUnitTestConfig('my-package');
 ```
 
-**Integration tests:**
+#### Integration tests:
 
 ```typescript
 import { createIntTestConfig } from '@code-pushup/test-setup-config';
 
 export default createIntTestConfig('my-package');
 ```
 
-**E2E tests:**
+#### E2E tests:
 
 ```typescript
 import { createE2ETestConfig } from '@code-pushup/test-setup-config';
@@ -32,18 +32,3 @@ export default createE2ETestConfig('my-e2e', {
   testTimeout: 60_000,
 });
 ```
-
-### Advanced: Overriding Config
-
-For edge cases, use the spread operator to override any property:
-
-```typescript
-const baseConfig = createE2ETestConfig('my-e2e');
-export default {
-  ...baseConfig,
-  test: {
-    ...(baseConfig as any).test,
-    globalSetup: ['./custom-setup.ts'],
-  },
-};
-```

testing/test-setup-config/src/lib/vitest-config-factory.ts

@@ -1,4 +1,4 @@
-import type { CoverageOptions, InlineConfig } from 'vitest';
+import type { CoverageOptions } from 'vitest';
 import { type UserConfig as ViteUserConfig, defineConfig } from 'vitest/config';
 import { getSetupFiles } from './vitest-setup-files.js';
 import { tsconfigPathAliases } from './vitest-tsconfig-path-aliases.js';
@@ -9,8 +9,6 @@ export type E2ETestOptions = {
   testTimeout?: number;
 };
 
-export type VitestConfig = ViteUserConfig & { test?: InlineConfig };
-
 function getIncludePatterns(kind: TestKind): string[] {
   switch (kind) {
     case 'unit':
@@ -25,10 +23,6 @@ function getIncludePatterns(kind: TestKind): string[] {
   }
 }
 
-function getGlobalSetup(kind: TestKind): string[] | undefined {
-  return kind === 'e2e' ? undefined : ['../../global-setup.ts'];
-}
-
 function buildCoverageConfig(
   projectKey: string,
   kind: TestKind,
@@ -37,13 +31,13 @@ function buildCoverageConfig(
     return undefined;
   }
 
-  const defaultExclude = ['mocks/**', '**/types.ts', 'perf/**'];
+  const exclude = ['mocks/**', '**/types.ts', 'perf/**'];
   const reportsDirectory = `../../coverage/${projectKey}/${kind}-tests`;
 
   return {
     reporter: ['text', 'lcov'],
     reportsDirectory,
-    exclude: defaultExclude,
+    exclude: exclude,
   };
 }
 
@@ -54,7 +48,7 @@ export function createVitestConfig(
 ): ViteUserConfig {
   const coverage = buildCoverageConfig(projectKey, kind);
 
-  const config: VitestConfig = {
+  const config: ViteUserConfig = {
     cacheDir: `../../node_modules/.vite/${projectKey}`,
     test: {
       reporters: ['basic'],
@@ -67,7 +61,7 @@ export function createVitestConfig(
       poolOptions: { threads: { singleThread: true } },
       environment: 'node',
       include: getIncludePatterns(kind),
-      globalSetup: getGlobalSetup(kind),
+      globalSetup: ['../../global-setup.ts'],
       setupFiles: [...getSetupFiles(kind)],
       ...(options?.testTimeout ? { testTimeout: options.testTimeout } : {}),
       ...(coverage ? { coverage } : {}),