-
Notifications
You must be signed in to change notification settings - Fork 2
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
Update jest-dom, make TS aware #637
Conversation
See testing-library/jest-dom#517 for more info
@@ -15,6 +15,7 @@ | |||
"skipLibCheck": true, | |||
"forceConsistentCasingInFileNames": true, | |||
"resolveJsonModule": true, | |||
"preserveWatchOutput": true | |||
"preserveWatchOutput": true, | |||
"types": ["node", "jest", "@testing-library/jest-dom"] |
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.
@testing-library/jest-dom adds more matchers to jest's expect(...)
; for example, expect(element).toBeInTheDocument()
. In v6, the mechanism that informs typescript about the new matchers changed.
See testing-library/jest-dom#517 for a little more info.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #637 +/- ##
=======================================
Coverage 75.02% 75.02%
=======================================
Files 248 248
Lines 11318 11318
Branches 1963 1963
=======================================
Hits 8491 8491
Misses 2653 2653
Partials 174 174 ☔ 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.
changes look good. tests pass.
See testing-library/jest-dom#517 for more info
What are the relevant tickets?
Closes #353
Description (What does it do?)
This PR updates jest and @testing-library/jest-dom.
How can this be tested?
Tests and CI should pass.