Skip to content

Conversation

@koba04
Copy link
Contributor

@koba04 koba04 commented Aug 17, 2018

Fixes #1766. Fixes #1765.

I missed that ShallowWrapper's constructor is called when finding child components.
In this case, the renderer is a renderer of the root, which means its instance is the instance of the root component. So cDM of the instance is called when creating a ShallowWrapper instance for child components.

@xitter
Copy link

xitter commented Aug 17, 2018

Looking forward to getting this released soon.

@ljharb ljharb force-pushed the fix-unexpeced-cDM branch from c61f1fd to d2f6603 Compare August 17, 2018 20:48
@ljharb ljharb merged commit d2f6603 into enzymejs:master Aug 17, 2018
@selbekk
Copy link

selbekk commented Aug 17, 2018

Incredible work! Thanks a ton! 👏👏👏

ljharb pushed a commit that referenced this pull request Aug 17, 2018
@koba04 koba04 deleted the fix-unexpeced-cDM branch August 17, 2018 22:56
@koba04
Copy link
Contributor Author

koba04 commented Aug 17, 2018

Thank you for the release!!
If there are any problems, please open a new issue!
(A test case to reproduce it is very helpful)

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.

4 participants