Skip to content

fix: wrap unmount in act #706

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 1 commit into from
Mar 19, 2021
Merged

fix: wrap unmount in act #706

merged 1 commit into from
Mar 19, 2021

Conversation

michaelgmcd
Copy link
Contributor

Summary

Fixes #705

Test plan

Tested via added unit tests and the failing test provided in #705.

@thymikee thymikee changed the title fix #705: wrap unmount in act fix: wrap unmount in act Mar 19, 2021
Copy link
Member

@thymikee thymikee left a comment

Choose a reason for hiding this comment

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

Thanks!

@thymikee thymikee merged commit e0236a5 into callstack:master Mar 19, 2021
@blackbing
Copy link

will this published in version 7?

@grifotv
Copy link

grifotv commented Sep 1, 2021

What is the best way to get this fix? Both latest releases v8.0.0-rc.0 and v7.2.0 were published before this PR was merged.

@grifotv
Copy link

grifotv commented Sep 1, 2021

@blackbing an alternative would be to install v8.0.0-rc.1 which includes this fix. It's not listed in the repo tags https://github.com/callstack/react-native-testing-library/tags but it's in the NPM package versions https://www.npmjs.com/package/@testing-library/react-native/v/8.0.0-rc.1

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.

Unable to test useEffect cleanup functions (RN 0.64)
4 participants