Skip to content

chore(test): document the Vitest project split #3531

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
setchy merged 1 commit into from
Oct 25, 2026
Merged

chore(test): document the Vitest project split #3531

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 9 additions & 2 deletions vitest.config.mts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,8 @@ export default defineConfig({
plugins: [react()],
test: {
globals: true,
// happy-dom tests import CSS from @primer/react. The threads and forks pools
// cannot load those .css files (ERR_UNKNOWN_FILE_EXTENSION), so keep vmThreads.
pool: 'vmThreads',
isolate: true,
attachmentsDir: '.vitest-attachments',
Expand All @@ -43,8 +45,10 @@ export default defineConfig({
reportOnFailure: true,
reporter: ['html', 'lcovonly'],
include: ['src/**/*'],
exclude: ['**/*.html', '**/*.graphql', '**/graphql/generated/**', '**/.DS_Store'],
exclude: ['**/*.html', '**/*.md', '**/*.graphql', '**/graphql/generated/**', '**/.DS_Store'],
},
// Projects mirror the runtime boundary: Node for main/shared, happy-dom for
// preload/renderer, and a real browser for pixel tests.
projects: [
{
extends: true,
Expand All @@ -65,7 +69,10 @@ export default defineConfig({
},
},
{
// TODO - Opportunity in future to move some of the renderer util tests to node environment
// Renderer tests stay in happy-dom even when they look like pure utils:
// many touch window.gitify, localStorage, navigator, document, or Audio,
// and the environment is only ~2% of suite runtime. A third node project
// would still need a renderer setup and would not remove that coupling.
extends: true,
test: {
name: 'node [main, shared]',
Expand Down
Loading