fix: prevent modifying test.exclude when same object passed in coverage.exclude (#7774)

Author: AriPerkkio

packages/vitest/src/node/config/resolveConfig.ts

@@ -349,16 +349,23 @@ export function resolveConfig(
   resolved.globalSetup = toArray(resolved.globalSetup || []).map(file =>
     resolvePath(file, resolved.root),
   )
-  resolved.coverage.exclude.push(
+
+  // override original exclude array for cases where user re-uses same object in test.exclude
+  resolved.coverage.exclude = [
+    ...resolved.coverage.exclude,
+
+    // Exclude setup files
     ...resolved.setupFiles.map(
       file =>
         `${resolved.coverage.allowExternal ? '**/' : ''}${relative(
           resolved.root,
           file,
         )}`,
     ),
-  )
-  resolved.coverage.exclude.push(...resolved.include)
+
+    // Exclude test files
+    ...resolved.include,
+  ]
 
   resolved.forceRerunTriggers = [
     ...resolved.forceRerunTriggers,

test/config/test/public.test.ts

@@ -42,3 +42,21 @@ test('respects custom config', async () => {
   expect(vitestConfig.name).toBe('custom config')
   expect(vitestConfig.reporters).toEqual([['default', {}]])
 })
+
+test('default value changes of coverage.exclude do not reflect to test.exclude', async () => {
+  const exclude = ['**/custom-exclude/**']
+
+  const { vitestConfig } = await resolveConfig({
+    include: ['**/example.test.ts'],
+    exclude,
+    coverage: {
+      exclude,
+    },
+  })
+
+  expect(exclude).toStrictEqual(['**/custom-exclude/**'])
+
+  expect(vitestConfig.include).toStrictEqual(['**/example.test.ts'])
+  expect(vitestConfig.exclude).toStrictEqual(['**/custom-exclude/**'])
+  expect(vitestConfig.coverage.exclude).toStrictEqual(['**/custom-exclude/**', '**/example.test.ts'])
+})