-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
React Native unit tests: migrate getByA11yLabel usages #45454
Conversation
|
Size Change: 0 B Total Size: 1.28 MB ℹ️ View Unchanged
|
bfb49a4
to
dd0dd4e
Compare
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.
Nice work, LGTM 🚀
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.
LGTM! Thanks for making these changes!
The
@testing-library/react-native
package, version 10.0, removes thegetByA11yLabel
query methods (and their siblings likefind*
orqueryAll*
). They were aliases ofgetByLabelText
methods, which are the only variant that continues to be supported.This PR migrates all usages of
ByA11yLabel
toByLabelText
. They are aliases, there's no behavior change. We're doing this in preparation to upgrade the library from currently used 9.x to the latest 11.x.See also:
ByA11yLabel
aliases@testing-library/react-native
version 10.0.0Part of React 18 migration in #45235.