| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -557,6 +557,15 @@ export function resolveTestConfig( | |||
| 557 | 557 | resolved.deps.moduleDirectories ??= [] | |
| 558 | 558 | ||
| 559 | 559 | resolved.deps.optimizer ??= {} | |
| 560 | + // `deps.optimizer` is keyed by Vite environment name, so `web` is a valid | ||
| 561 | + // key when the user declares that environment themselves | ||
| 562 | + // (`vite.environments.web`) - only the Vitest 3 alias for `client` is | ||
| 563 | + // silently ignored | ||
| 564 | + if (resolved.deps.optimizer.web && !viteConfig.environments.web) { | ||
| 565 | + logger.deprecate( | ||
| 566 | + '`deps.optimizer.web` is deprecated. Use `deps.optimizer.client` instead (or `deps.optimizer.ssr` for `node` and `edge` environments).', | ||
| 567 | + ) | ||
| 568 | + } | ||
| 560 | 569 | resolved.deps.optimizer.ssr ??= {} | |
| 561 | 570 | resolved.deps.optimizer.ssr.enabled ??= false | |
| 562 | 571 | resolved.deps.optimizer.client ??= {} | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -89,7 +89,7 @@ test('preserves previous test results', async () => { | |||
| 89 | 89 | describe('with optimizer enabled', () => { | |
| 90 | 90 | const deps = { | |
| 91 | 91 | optimizer: { | |
| 92 | - web: { | ||
| 92 | + client: { | ||
| 93 | 93 | enabled: true, | |
| 94 | 94 | }, | |
| 95 | 95 | }, | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -0,0 +1,57 @@ | |||
| 1 | + import { expect, test } from 'vitest' | ||
| 2 | + import { resolveTestConfig, runInlineTests } from '../../../test-utils' | ||
| 3 | + | ||
| 4 | + test('deps.optimizer.web is deprecated in favour of deps.optimizer.client', async () => { | ||
| 5 | + const { stderr } = await runInlineTests( | ||
| 6 | + { 'basic.test.ts': 'test("passes", () => {})' }, | ||
| 7 | + { | ||
| 8 | + deps: { | ||
| 9 | + optimizer: { | ||
| 10 | + web: { enabled: true }, | ||
| 11 | + }, | ||
| 12 | + }, | ||
| 13 | + }, | ||
| 14 | + ) | ||
| 15 | + expect(stderr).toContain('`deps.optimizer.web` is deprecated') | ||
| 16 | + expect(stderr).toContain('Use `deps.optimizer.client` instead') | ||
| 17 | + }) | ||
| 18 | + | ||
| 19 | + test('deps.optimizer.client is not deprecated', async () => { | ||
| 20 | + const { stderr } = await runInlineTests( | ||
| 21 | + { 'basic.test.ts': 'test("passes", () => {})' }, | ||
| 22 | + { | ||
| 23 | + deps: { | ||
| 24 | + optimizer: { | ||
| 25 | + client: { enabled: false }, | ||
| 26 | + }, | ||
| 27 | + }, | ||
| 28 | + }, | ||
| 29 | + ) | ||
| 30 | + expect(stderr).not.toContain('`deps.optimizer.web` is deprecated') | ||
| 31 | + }) | ||
| 32 | + | ||
| 33 | + test('deps.optimizer.web is deprecated only without a `web` environment', async () => { | ||
| 34 | + // `deps.optimizer` is keyed by Vite environment name, so `web` is a valid key | ||
| 35 | + // when the user declares that environment themselves - only the Vitest 3 | ||
| 36 | + // alias for `client` is silently ignored | ||
| 37 | + const web = { enabled: true } | ||
| 38 | + | ||
| 39 | + const withoutEnvironment = await resolveTestConfig({ | ||
| 40 | + deps: { | ||
| 41 | + optimizer: { web }, | ||
| 42 | + }, | ||
| 43 | + }) | ||
| 44 | + expect(withoutEnvironment.stderr).toContain('`deps.optimizer.web` is deprecated') | ||
| 45 | + | ||
| 46 | + const withEnvironment = await resolveTestConfig({ | ||
| 47 | + deps: { | ||
| 48 | + optimizer: { web }, | ||
| 49 | + }, | ||
| 50 | + $viteConfig: { | ||
| 51 | + environments: { | ||
| 52 | + web: {}, | ||
| 53 | + }, | ||
| 54 | + }, | ||
| 55 | + }) | ||
| 56 | + expect(withEnvironment.stderr).not.toContain('`deps.optimizer.web` is deprecated') | ||
| 57 | + }) | ||
| Back | FazBrowse Home | New Git URL |
0 commit comments