fix(vitest): pass down CLI options to override workspace configs (#4774)

Author: sheremet-va

packages/vitest/src/node/core.ts

@@ -264,8 +264,11 @@ export class Vitest {
       'testTimeout',
       'pool',
       'globals',
-      'mode',
       'expandSnapshotDiff',
+      'retry',
+      'testNamePattern',
+      'passWithNoTests',
+      'bail',
     ] as const
 
     const cliOverrides = overridesOptions.reduce((acc, name) => {

packages/vitest/src/node/workspace.ts

@@ -41,7 +41,7 @@ export async function initializeProject(workspacePath: string | number, ctx: Vit
     logLevel: 'error',
     configFile,
     // this will make "mode": "test" | "benchmark" inside defineConfig
-    mode: options.mode || ctx.config.mode,
+    mode: options.test?.mode || options.mode || ctx.config.mode,
     plugins: [
       ...options.plugins || [],
       WorkspaceVitestPlugin(project, { ...options, root, workspacePath }),

test/config/fixtures/workspace-flags/projects/cli-config.test.ts

@@ -0,0 +1,5 @@
+import { it, expect } from 'vitest';
+
+it('math', () => {
+  expect(1).toBe(1)
+})

test/config/fixtures/workspace-flags/projects/vitest.config.js

@@ -0,0 +1 @@
+export default {}

test/config/fixtures/workspace-flags/vitest.workspace.js

@@ -0,0 +1 @@
+export default ['projects']

test/config/test/flags.test.ts

@@ -0,0 +1,38 @@
+import { expect, it } from 'vitest'
+import { runVitest } from '../../test-utils'
+
+it('correctly inherit from the cli', async () => {
+  const { vitest } = await runVitest({
+    root: 'fixtures/workspace-flags',
+    logHeapUsage: true,
+    allowOnly: true,
+    sequence: {
+      seed: 123,
+    },
+    testTimeout: 5321,
+    pool: 'forks',
+    globals: true,
+    expandSnapshotDiff: true,
+    retry: 6,
+    testNamePattern: 'math',
+    passWithNoTests: true,
+    bail: 100,
+  })
+  const project = vitest!.projects[0]
+  const config = project.getSerializableConfig()
+  expect(config).toMatchObject({
+    logHeapUsage: true,
+    allowOnly: true,
+    sequence: expect.objectContaining({
+      seed: 123,
+    }),
+    testTimeout: 5321,
+    pool: 'forks',
+    globals: true,
+    expandSnapshotDiff: true,
+    retry: 6,
+    passWithNoTests: true,
+    bail: 100,
+  })
+  expect(config.testNamePattern?.test('math')).toBe(true)
+})