Skip to content

chore(tests): update test deps#419

Merged
eddywashere merged 1 commit intoreactstrap:masterfrom
TheSharpieOne:chore/update-test-deps
May 14, 2017
Merged

chore(tests): update test deps#419
eddywashere merged 1 commit intoreactstrap:masterfrom
TheSharpieOne:chore/update-test-deps

Conversation

@TheSharpieOne
Copy link
Copy Markdown
Member

@TheSharpieOne TheSharpieOne commented May 8, 2017

This updates enzyme to remove the deprectation Warning: ReactTestUtils has been moved to react-dom/test-utils. Update references to remove this warning. and Warning: Shallow renderer has been moved to react-test-renderer/shallow. Update references to remove this warning.
keeping addon test utils for backwards compatibility until the min react version for this project is updated. Enzyme will use it if needed.

This updates enzyme to remove the depractation Warning: ReactTestUtils has been moved to react-dom/test-utils. Update references to remove this warning.
keeping addon test utils for backwards compatibility until the min react version for this project is updated. Enzyme will use it if needed.
Comment thread package.json
"css-loader": "^0.25.0",
"ejs": "^2.5.1",
"enzyme": "^2.4.1",
"enzyme": "^2.8.2",
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note: This is just a minimum version change, previously if you did a fresh npm install, you would be able to get this version. This just forces it to be at least this version as enzyme fixed some things in this version to allow it to work with react 15.5

@TheSharpieOne
Copy link
Copy Markdown
Member Author

Test fail due to linter errors which are fixed in #418. Checkout of the travis output to see that no tests actually failed and that the deprecation warnings (ReactTestUtils has been moved to react-dom/test-utils. Update references to remove this warning. and Shallow renderer has been moved to react-test-renderer/shallow. Update references to remove this warning) are gone.

@eddywashere
Copy link
Copy Markdown
Member

Thanks!

@eddywashere eddywashere merged commit 33a4f5b into reactstrap:master May 14, 2017
@TheSharpieOne TheSharpieOne deleted the chore/update-test-deps branch October 3, 2017 17:52
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.

2 participants