Skip to content

Commit 2fed087

Browse files
pa-lemclaude
andcommitted
fix(test): pre-bundle lucide-react via Vite optimizeDeps (correct key for vitest 4 browser mode)
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 23e5527 commit 2fed087

1 file changed

Lines changed: 6 additions & 9 deletions

File tree

frontend/app/vitest.config.ts

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,12 @@ import viteConfig from "./vite.config";
66
export default mergeConfig(
77
viteConfig,
88
defineConfig({
9+
// @infrahub/ui is consumed as workspace SOURCE, so transitive deps it pulls in (e.g.
10+
// lucide-react via FloatingPanel) must be pre-bundled up front. Otherwise Vite discovers
11+
// them mid-run and triggers a page reload that resets vi.mock(), breaking unrelated tests.
12+
optimizeDeps: {
13+
include: ["lucide-react"],
14+
},
915
test: {
1016
browser: {
1117
enabled: true,
@@ -36,15 +42,6 @@ export default mergeConfig(
3642
],
3743
},
3844
exclude: ["**/node_modules/**", "**/dist/**", "**/e2e/**", "**/playwright-report/**"],
39-
deps: {
40-
optimizer: {
41-
web: {
42-
// Pre-bundle deps pulled in by @infrahub/ui workspace-source so Vite
43-
// doesn't discover them mid-run and trigger a reload that resets vi.mock().
44-
include: ["lucide-react"],
45-
},
46-
},
47-
},
4845
},
4946
})
5047
);

0 commit comments

Comments
 (0)