-
Notifications
You must be signed in to change notification settings - Fork 3.3k
chore: vitest project config #32447
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.
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
chore: vitest project config #32447
Conversation
cypress
|
Project |
cypress
|
Branch Review |
chore/vitest-project-config
|
Run status |
|
Run duration | 10m 00s |
Commit |
|
Committer | Cacie Prins |
View all properties for this run ↗︎ |
Test results | |
---|---|
|
0
|
|
0
|
|
694
|
|
0
|
|
130
|
View all changes introduced in this branch ↗︎ |
'cli/vitest.config.*', | ||
], | ||
exclude: ['**/node_modules/**', '**/dist/**', 'system-tests/**'], | ||
coverage: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
where exactly does the coverage get output to and do we need to check if they are generated similar to our current CI process?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They aren't, yet! I've been using it personally to check coverage locally, but not anything in CI.
Additional details
Some vitest extensions have a pretty hard time in a monorepo with multiple vitest configs.
This project config is set up to quiet those warnings. It's not super useful to run vitest from root at this point - it takes quite some time, partially because some of our vitest suites are more integration tests than unit tests (particularly in
npm/cypress-schematic
).Steps to test
How has the user experience changed?
PR Tasks
cypress-documentation
?type definitions
?