-
Notifications
You must be signed in to change notification settings - Fork 20
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: switch Popover
tests to react testing library
#1505
Conversation
This pull request is automatically being deployed by Amplify Hosting (learn more). |
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.
There's a bunch of places where we're looking for .Popover
class names. Generally speaking, we should avoid querying by class names because changing the class name should not break tests that do not explicitly depend on the class name of the component.
Popover
tests to react testing library
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.
There's still an unresolved convo regarding setting aria-labelledby
a791005
to
e11f449
Compare
e11f449
to
fa4d9e3
Compare
Issue #1159
This PR switches the
Popover
tests to React Testing Library.Validation steps
Run yarn test
See that there are coverage percentages in the table
Verify that the tests are converted properly