merge new rtl tests and test framework #4
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…eduxjs#984) * begin work * enable testing on 4 react versions on travis * enable local testing on 4 versions * enable running wallaby if installed * Fix tests in 0.14, 15, and 16 * remove 15.4, not really necessary and it requires a tweak to the install script * fix 2 major potential issues 1) by putting all the installs in setupTestEnv into a single npm command, npm --no-save becomes possible. When they are on separate lines, the 2nd npm i removes the previous line's work 2) after test runs, we restore the node_modules to what devDependencies says * update contributing docs and add missing test:watch * re-work to use subdirectories for testing specific react versions caveat: collecting coverage does not work yet, we need to combine the lcov reports * fully working! with merged coverage! * fix linting, remove unnecessary file * fix test:watch * fix travis tests to run in parallel for each version * oops, didn't make travis run the CI test * sigh... npm syntax error * speed up test suites by only installing the specific version needed * remove unused plugin * simplify test script options in package.json use the REACT env variable to run tests for a specific version, set it to "all" to run for all supported versions * simpler gitignore * remove unnecessary coverage merging, codecov does that automagically * simplify test running * new docs on testing specific React versions * move scripts to test/, remove unused dep * revert unintentional cosmetic changes * add default version for "npm test" * revert unintentional cosmetic changes to test import order * restore the correct test renderer version * fix travis, add a note about the matrix needing update on adding a React version * Add cross-spawn dependency * Use cross-spawn for consistent NPM installations cross-platform
merge in testing
DO NOT REMOVE PLEASE TIM
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.