Fix wrapper.unmount not calling useEffect cleanup

[Iskander508]

fix for issue with unmount not running useEffect cleanup: #12

[vdwijngaert]

Would be helpful if this pull request was merged :)

[MikeMcMahon]

we are using this library as well and it would be very useful

[gisaklement]

Hi @wojtekmaj
I found your adapter today 🥳 and it seems like the only things that's blocking our pipeline is this PR 🙂
Any chance this pull request can be merged?

Thank you so much helping us out!

[gisaklement]

Nevermind I just saw your comment here

I'm afraid this will need to wait for an official package to be resolved

Thank you anyway though for this package!!

[wojtekmaj]

@gisaklement I started discussing this change in a PR that adds official React 17 adapter this package is based on. Let's see how it goes.

[gisaklement]

@gisaklement I started discussing this change in a PR that adds official React 17 adapter this package is based on. Let's see how it goes.

Thank you so much!! I appreciate it 🙂

[wojtekmaj]

Looks like we won't get an official response anytime soon. Screw this then. @Iskander508 Please kindly rebase your branch off current master, or allow edits from maintainers, so I could check if this doesn't break anything, since tests were added recently.

[Iskander508]

Looks like we won't get an official response anytime soon. Screw this then. @Iskander508 Please kindly rebase your branch off current master, or allow edits from maintainers, so I could check if this doesn't break anything, since tests were added recently.

Edits should be allowed already

[Iskander508]

rebased

[wojtekmaj]

Thanks so much.

Interestingly this doesn't seem to affect neither positively nor negatively any unit tests originally added to Enzyme.

Let's go for it.

[gisaklement]

Again, thank you very much 🙂