Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

Added react-test-renderer dependency #27

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

samblake
Copy link

Fix for #25

@JReinhold
Copy link

I think it replaces react-addon-test-utils as per this comment, which should be removed instead. At least my setup works fine when I remove the react-addon-test-utils and replaces it with react-test-renderer. Do you have any situations where it breaks if you remove the react-addon-test-utils dependency completely @samblake ?

@samblake
Copy link
Author

Removing it completely seems to work fine.

@Vikaskumargd
Copy link

must be merged now and closed now

@Dev-Dipesh
Copy link

Just faced the same issue, glad I checked for open PRs before creating one. This should be merged, check here for more details:

enzymejs/enzyme#875

@samblake also update the Readme.md with the new command by replacing old one:

Old

npm install -D enzyme @types/enzyme react-addons-test-utils

New

npm install -D enzyme @types/enzyme react-addons-test-utils react-test-renderer

@hyunjun19
Copy link

@JReinhold I remove the react-addon-test-utils and install react-test-renderer.
It works well.

@doyle273
Copy link

doyle273 commented Jan 7, 2019

Good

1 similar comment
@doyle273
Copy link

doyle273 commented Jan 7, 2019

Good

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants