Skip to content

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