-
Notifications
You must be signed in to change notification settings - Fork 400
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
fix(toHaveFocus): clarify message when using .not
#447
fix(toHaveFocus): clarify message when using .not
#447
Conversation
Codecov Report
@@ Coverage Diff @@
## main #447 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 26 26
Lines 624 626 +2
Branches 231 233 +2
=========================================
+ Hits 624 626 +2
Continue to review full report at Codecov.
|
@all-contributors please add @Dennis273 for code |
I've put up a pull request to add @Dennis273! 🎉 |
🎉 This PR is included in version 5.16.3 🎉 The release is available on: Your semantic-release bot 📦🚀 |
@all-contributors please add @Dennis273 for code |
I've put up a pull request to add @Dennis273! 🎉 |
What:
The output message is confusing when using
expect(element).not.toHaveFocus()
Why:
To avoid confusion.
How:
By updating the output message when
isNot
is truthy.Checklist: