chore: use viteOverrides instead of the cli options (#8144)

Author: sheremet-va

test/benchmark/test/basic.test.ts

@@ -16,7 +16,7 @@ it('basic', { timeout: 60_000 }, async () => {
 
     // Verify that type testing cannot be used with benchmark
     typecheck: { enabled: true },
-  }, [], 'benchmark')
+  }, [], { mode: 'benchmark' })
   expect(result.stderr).toBe('')
   expect(result.exitCode).toBe(0)
 

test/benchmark/test/compare.test.ts

@@ -11,7 +11,7 @@ test('compare', { timeout: 60_000 }, async () => {
       root: './fixtures/compare',
       outputJson: './bench.json',
       reporters: ['default'],
-    }, [], 'benchmark')
+    }, [], { mode: 'benchmark' })
     expect(result.exitCode).toBe(0)
     expect(fs.existsSync('./fixtures/compare/bench.json')).toBe(true)
   }
@@ -22,7 +22,7 @@ test('compare', { timeout: 60_000 }, async () => {
       root: './fixtures/compare',
       compare: './bench.json',
       reporters: ['default'],
-    }, [], 'benchmark')
+    }, [], { mode: 'benchmark' })
     expect(result.exitCode).toBe(0)
     const lines = result.stdout.split('\n').slice(4).slice(0, 6)
     const expected = `

test/benchmark/test/reporter.test.ts

@@ -4,15 +4,15 @@ import { runVitest } from '../../test-utils'
 
 it('summary', async () => {
   const root = pathe.join(import.meta.dirname, '../fixtures/reporter')
-  const result = await runVitest({ root }, ['summary.bench.ts'], 'benchmark')
+  const result = await runVitest({ root }, ['summary.bench.ts'], { mode: 'benchmark' })
   expect(result.stderr).toBe('')
   expect(result.stdout).not.toContain('NaNx')
   expect(result.stdout.split('BENCH  Summary')[1].replaceAll(/[0-9.]+x/g, '(?)')).toMatchSnapshot()
 })
 
 it('non-tty', async () => {
   const root = pathe.join(import.meta.dirname, '../fixtures/basic')
-  const result = await runVitest({ root }, ['base.bench.ts'], 'benchmark')
+  const result = await runVitest({ root }, ['base.bench.ts'], { mode: 'benchmark' })
   const lines = result.stdout.split('\n').slice(4).slice(0, 11)
   const expected = `\
  ✓ base.bench.ts > sort
@@ -40,7 +40,7 @@ it.for([true, false])('includeSamples %s', async (includeSamples) => {
       benchmark: { includeSamples },
     },
     ['summary.bench.ts'],
-    'benchmark',
+    { mode: 'benchmark' },
   )
   assert(result.ctx)
   const allSamples = [...result.ctx.state.idMap.values()]

test/benchmark/test/sequential.test.ts

@@ -5,7 +5,7 @@ import { runVitest } from '../../test-utils'
 
 it('sequential', async () => {
   const root = pathe.join(import.meta.dirname, '../fixtures/sequential')
-  await runVitest({ root }, [], 'benchmark')
+  await runVitest({ root }, [], { mode: 'benchmark' })
   const testLog = await fs.promises.readFile(pathe.join(root, 'test.log'), 'utf-8')
   expect(testLog).toMatchSnapshot()
 })

test/browser/specs/benchmark.test.ts

@@ -4,7 +4,7 @@ import { runVitest } from '../../test-utils'
 const IS_PLAYWRIGHT = process.env.PROVIDER === 'playwright'
 
 test('benchmark', async () => {
-  const result = await runVitest({ root: 'fixtures/benchmark' }, [], 'benchmark')
+  const result = await runVitest({ root: 'fixtures/benchmark' }, [], { mode: 'benchmark' })
   expect(result.stderr).toReportNoErrors()
 
   if (IS_PLAYWRIGHT) {

test/cli/test/annotations.test.ts

@@ -58,6 +58,7 @@ describe('API', () => {
         'basic.test.ts': annotationTest,
         'test-3.js': test3Content,
         'test-4.js': '',
+        'vitest.config.js': { test: options },
       },
       {
         includeTaskLocation: true,
@@ -97,10 +98,6 @@ describe('API', () => {
           },
         ],
       },
-      {},
-      {
-        test: options,
-      },
     )
 
     expect(stderr).toBe('')

test/cli/test/artifacts.test.ts

@@ -58,6 +58,7 @@ describe('API', () => {
         'test-4.js': '',
       },
       {
+        ...options,
         includeTaskLocation: true,
         reporters: [
           'default',
@@ -100,10 +101,6 @@ describe('API', () => {
           },
         ],
       },
-      {},
-      {
-        test: options,
-      },
     )
 
     expect(stderr).toBe('')

test/cli/test/fails.test.ts

@@ -116,14 +116,12 @@ it('prints a warning if the assertion is not awaited in the browser mode', async
       expect(Promise.resolve(1)).resolves.toBe(1)
     })
     `,
-  }, {}, {}, {
-    test: {
-      browser: {
-        enabled: true,
-        instances: [{ browser: 'chromium' }],
-        provider: playwright(),
-        headless: true,
-      },
+  }, {
+    browser: {
+      enabled: true,
+      instances: [{ browser: 'chromium' }],
+      provider: playwright(),
+      headless: true,
     },
   })
   expect(stderr).toContain('Promise returned by \`expect(actual).resolves.toBe(expected)\` was not awaited')

test/cli/test/group-order.test.ts

@@ -8,7 +8,7 @@ test('tests run according to the group order', async () => {
     'example.2-2.test.ts': `test('2-2', () => {})`,
     'example.3.test.ts': `test('3', () => {})`,
   }, {
-    globals: true,
+    $cliOptions: { globals: true },
     // run projects in the opposite order!
     projects: [
       {

test/cli/test/open-telemetry.test.ts

@@ -48,8 +48,7 @@ describe.for([
       // root needs to be set before vitest sets up,
       // but browser options need to be in the config already
       root: './fixtures/otel-tests',
-    }, [], 'test', {
-      test: (type === 'root'
+      ...(type === 'root'
         ? config
         : { projects: [{ test: config }] }),
     })