anvilresearch / connect

A modern authorization server built to authenticate your users and protect your APIs
http://anvil.io
MIT License
361 stars 85 forks source link

Improve coveralls in travis build #293

Closed henrjk closed 8 years ago

henrjk commented 8 years ago

Add coveralls badge and improve build script.

Update original comment!

The introduction of the coveralls build somewhat unintentionally had all tests running at once. This used to fail which was the reason that the coverage.sh script was introduced so that the coverage would run separately for unit and integration tests and the results were combined.

However it should really be possible to run all tests at once, so that it makes sense to do so in the build. This should allow to see regressions in this respect.

The coverage script itself is kept as it can still be useful during development if only to see how one can run the commands in a shell without installing istanbul or mocha globally.

Finally the travis script now sends coverage data in a second step to coveralls with the intent to easier distinguish cases where the tests fail and where coveralls fails.

henrjk commented 8 years ago

The build bage an only be seen in master.

This should be good to be pulled now.

christiansmith commented 8 years ago

Thanks!