-
-
Notifications
You must be signed in to change notification settings - Fork 347
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 reqresp unit tests to vitest #6191
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## unstable #6191 +/- ##
============================================
- Coverage 90.35% 88.55% -1.80%
============================================
Files 78 109 +31
Lines 8087 10302 +2215
Branches 490 648 +158
============================================
+ Hits 7307 9123 +1816
- Misses 772 1169 +397
- Partials 8 10 +2 |
Performance Report✔️ no performance regression detected Full benchmark results
|
bc10914
to
6e6b061
Compare
@wemeetagain Please re-review this PR. I went through the changes again and made sure only the absolute necessary changes are included from 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.
looks good, would let @wemeetagain decided on Buffer.from usage on the src/responseEncode.ts, afaik we didn't want any actual Buffer usage in the main src
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.
Looks good to me, PR no longer modifies src/
* Migrate reqresp unit tests to vitest * Fix the build step * Fix the types * Fix the source changes * Remove the usage in src
🎉 This PR is included in v1.14.0 🎉 |
* Migrate reqresp unit tests to vitest * Fix the build step * Fix the types * Fix the source changes * Remove the usage in src
Motivation
Consolidate the testing frameworks and migrate to vitest.
Description
Migrate
reqresp
unit tests to vitest.Steps to test or reproduce