-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
chore(internals): Upgrade enzyme to v3 #1966
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you, this is awesome! solid start for #1950
looks to have no effect on templates, so LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should update enzyme to the lastest version and use enzyme-adapter-react-16 instead of enzyme-adapter-react-15
@@ -264,7 +265,8 @@ | |||
"circular-dependency-plugin": "3.0.0", | |||
"coveralls": "2.13.1", | |||
"css-loader": "0.28.4", | |||
"enzyme": "2.8.2", | |||
"enzyme": "3.1.0", | |||
"enzyme-adapter-react-15": "1.0.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should use enzyme-adapter-react-16
instead of enzyme-adapter-react-15
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
En, I am sorry, Maybe should do this in a follow up PR.
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Upgrade to Enzyme 3
This resolves what I believe to be the major blocker for React 16
find
behaviour for new static rendererResolves #1950