Jest instructions cause CI to pass on failed tests

[mlms13]

The README provides these instructions for Jest integration:

jest --coverage --coverageReporters=text-lcov | coveralls

But in this case, if a test doesn't pass, Jest's non-zero exit status is swallowed by | coveralls and turned into a 0, which tricks Circle (and probably most other CIs?) into thinking the build was a success.

The command suggested in this comment shouldn't have the same issue:

jest --coverage && cat ./coverage/lcov.info | coveralls

[AndreMiras]

Yes I confirm I get the issue with Travis too. Another way to make the pipe file would be to set -o pipefail.