⚓ T179507 Compare coverage reports of migrating to pytest

• Ladsgroup created this task.Nov 1 2017, 4:38 PM

Restricted Application added projects: User-Ladsgroup, artificial-intelligence. · View Herald TranscriptNov 1 2017, 4:38 PM

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

• Ladsgroup moved this task from Incoming to In progress on the User-Ladsgroup board.Nov 2 2017, 4:44 PM

For revscoring: https://github.com/wiki-ai/revscoring/pull/370 https://codecov.io/gh/wiki-ai/revscoring/pulls?_pjax=%23pjax-outer&state=all The coverage has improved a little bit.

• Ladsgroup mentioned this in rORES4ecf6b994336: Use pytest-cov for proper coverage report.Nov 4 2017, 5:52 PM

For ORES https://codecov.io/gh/wiki-ai/ores/pull/224 this is interesting, it seems that by migrating to pytest we lost coverage of routs files but we never tested those before, my hypothesis is that old coverage collector counted lines we ran while we tried to load the service which is not correct and we should have unit tests for routs too. So The coverage now is real.

For Wikilabels: https://github.com/wiki-ai/wikilabels/pull/210 I was checking and amazingly I realized the integration between travis-ci and codecov.io is broken, after fixing it, it seems we have 1.7% of coverage which is unfortunately correct.