Update examples to include Jest #183
Conversation
Also included link to my project where I am successfully implementing jest coverage and travis-ci
|
@Ethan-Arrowood thanks your script saved my life |
README.md
Outdated
| - Install [jest](https://facebook.github.io/jest/docs/en/getting-started.html) | ||
| - Use the following to run tests and push files to coveralls: | ||
| ``` | ||
| jest --coverage && cat ./coverage/lcov.info | ./node_modules/coveralls/bin/coveralls.js && rm -rf ./coverage |
There was a problem hiding this comment.
Might want to just add coverage/ to the .gitignore instead of removing the whole directory.
There was a problem hiding this comment.
I'll have to review my use case for this but I don't think adding coverage to my gitignore does the same thing.
There was a problem hiding this comment.
OK. I figure Jest writes it there for some caching and optimization. It also allows you to check it again at a later time without re-running all the test. Small stuff, no worries.
README.md
Outdated
| - Install [jest](https://facebook.github.io/jest/docs/en/getting-started.html) | ||
| - Use the following to run tests and push files to coveralls: | ||
| ``` | ||
| jest --coverage && cat ./coverage/lcov.info | ./node_modules/coveralls/bin/coveralls.js && rm -rf ./coverage |
There was a problem hiding this comment.
Sorry, but why not
jest --coverage --coverageReporters=text-lcov | coveralls
?
|
Hmm I unfortunately can't get this to work. But I have tests and Travis executes them. |
|
I've been using @Yeti-or version and it works perfectly. |
|
Thank you @Ethan-Arrowood and @Yeti-or! |
Also included link to my project where I am successfully implementing jest coverage and travis-ci