-
Notifications
You must be signed in to change notification settings - Fork 64
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: migrate to vitest #181
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #181 +/- ##
==========================================
- Coverage 81.83% 72.89% -8.94%
==========================================
Files 21 22 +1
Lines 534 1797 +1263
Branches 40 217 +177
==========================================
+ Hits 437 1310 +873
- Misses 97 477 +380
- Partials 0 10 +10
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
Nice work!
This does a lot.
vitest
in the repovitest
patterns of mocking and assertionsscript
andscriptUtils
tests since those were all tests ofscript.js
vitests
coverage as well. Dumps all coverage in to a single coverage dir since it's no longer split between test runners. Merges coverage uploading to a single GH Action step, too.This spawned from a need to test recent fraudnet changes. So be mindful that this base branch is to our fraudnet feature branch, and that we still need to add tests for fraudnet so that is still pending.
Note: Initially done here in #117 but that was using a feature branch as the base branch. This just moves the same changes to PR against
main
.