-
Notifications
You must be signed in to change notification settings - Fork 150
Support React Native events in prefer-user-event #957
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
Labels
enhancement
New feature or request
Comments
ddrozdov
pushed a commit
to sport24ru/eslint-plugin-testing-library
that referenced
this issue
Nov 7, 2024
1 task
Well, it actually works fine out of the box for our project, just some checks are missing which is anyway normal for a linter as there's always room for improvement :) Or am I missing something important that is required to declare React Native support? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What rule do you want to change?
prefer-user-event
Does this change cause the rule to produce more or fewer warnings?
More warnings
How will the change be implemented?
Add more fireEvent -> userEvent mappings.
React Native Testing Library has its own fireEvent function and the same idea of proposing to use userEvent if possible.
According to the docs, it's recommended to use:
press
instead offireEvent.press
type
instead offireEvent.changeText
scrollTo
instead offireEvent.scroll
Example code
How does the current rule affect the code?
It doesn't suggest any changes to
fireEvent
calls for React Native projects.How will the new rule affect the code?
It'll suggest to use User Event as per the docs.
Anything else?
No response
Do you want to submit a pull request to change the rule?
Yes
The text was updated successfully, but these errors were encountered: