-
-
Notifications
You must be signed in to change notification settings - Fork 287
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
test: migrate state-transition unit tests to vitest #6173
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## unstable #6173 +/- ##
=============================================
+ Coverage 80.02% 90.35% +10.33%
=============================================
Files 19 78 +59
Lines 1717 8087 +6370
Branches 155 490 +335
=============================================
+ Hits 1374 7307 +5933
- Misses 341 772 +431
- Partials 2 8 +6 |
…tar into nh/test-state-transition
Converting to draft to shortly fix the |
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.
lgtm, straightforward conversion with no ???s
Yes for this package it was very straight forward transition. Expectations was translated with tool and mocking was done manually. |
Performance Report✔️ no performance regression detected Full benchmark results
|
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.
lgtm
🎉 This PR is included in v1.13.0 🎉 |
Motivation
Consolidate the testing frameworks and migrate to vitest.
Description
Migrate
state-transition
unit tests tovitest
.Steps to test or reproduce