-
Notifications
You must be signed in to change notification settings - Fork 633
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
feat(internal,expect,testing): @std/expect
jest compatible fn and unified mock experience with @std/testing/mock
#6317
base: main
Are you sure you want to change the base?
Conversation
@std/expect
jest compatible fn and unified mock experience with @std/testing/mock
@std/expect
jest compatible fn and unified mock experience with @std/testing/mock
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6317 +/- ##
==========================================
+ Coverage 96.34% 96.37% +0.02%
==========================================
Files 547 555 +8
Lines 41671 42970 +1299
Branches 6314 6524 +210
==========================================
+ Hits 40147 41411 +1264
- Misses 1482 1517 +35
Partials 42 42 ☔ View full report in Codecov by Sentry. |
@std/expect
jest compatible fn and unified mock experience with @std/testing/mock
@std/expect
jest compatible fn and unified mock experience with @std/testing/mock
c5e6556
to
605a7a7
Compare
605a7a7
to
7b4cf2a
Compare
@kt3k Hi! Maybe you will have any thoughts on this? |
Sorry for the delay in review. Addition of Jest compatible methods are welcome, but it doesn't seem desirable to make |
Summary
Made an implementation of jest compatible
@std/expect
'sfn
utility.Implementation made with side eyeing latest version of this api in jest https://jestjs.io/docs/mock-function-api .
Issues that this pr covers:
fn
function signature with Jest'sfn
type #6247