This repository has been archived by the owner on May 31, 2019. It is now read-only.

feat: add code coverage reporting #48

Merged

darahayes merged 2 commits into master from coverage

Aug 8, 2018

Contributor

darahayes commented Aug 8, 2018 •

edited

Loading

This PR adds an additional step to the CI config which runs both integration and unit tests and outputs code coverage to Coveralls.io

You can see the project in coveralls here: https://coveralls.io/github/aerogear/data-sync-server


          feat: add code coverage reporting

f8deb24

darahayes requested a review from aliok as a code owner

August 8, 2018 08:29


          doc: add Readme badges

0d46962

coveralls commented Aug 8, 2018

Pull Request Test Coverage Report for Build 365

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 83.8%

Totals
Change from base Build #1:	-0.5%
Covered Lines:	798
Relevant Lines:	917

💛 - Coveralls

darahayes requested a review from wtrocki

August 8, 2018 08:42

wtrocki approved these changes

View reviewed changes

Contributor

wtrocki commented Aug 8, 2018 •

edited

Loading

verification:

Checked coveralls account 👍
Checked if reports files are picked up and include all the tests 👍
Failed coverals not blocking merge 👍
Overall impression 💯

darahayes merged commit 299609a into master

darahayes deleted the coverage branch

August 8, 2018 09:57

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.