Skip to content

docs: troubleshooting guilde #1065

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

Merged
merged 4 commits into from
Aug 18, 2022
Merged

docs: troubleshooting guilde #1065

merged 4 commits into from
Aug 18, 2022

Conversation

mdjastrzebski
Copy link
Member

Summary

Let's kick of a troublshooting guide where we will explain how to fix common issues met by our users. Starting with one recent issue.

Test plan

All checks pass.

@mdjastrzebski mdjastrzebski requested a review from thymikee August 17, 2022 10:44
@thymikee
Copy link
Member

We also have this section in the readme: https://github.com/callstack/react-native-testing-library#troubleshooting We can move it into the troubleshooting docs you created here

@mdjastrzebski
Copy link
Member Author

I've updated Troublshooting guide with content from README.

@AugustinLF AugustinLF self-requested a review August 18, 2022 09:19

Normally, you should not encounter sync `act()` warnings, but if that happens this probably indicate an issue with your test and should be investigated.

In case of async `act()` function this might happen more or less randomly, especially if your components contain async logic. So far this warning does not seem to affect test correctness.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting, never payed attention to the difference between those two.

@mdjastrzebski mdjastrzebski merged commit 6108b95 into main Aug 18, 2022
@mdjastrzebski mdjastrzebski deleted the docs/troubleshooting-guide branch August 18, 2022 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants