Page MenuHomePhabricator

Compare coverage reports of migrating to pytest
Closed, ResolvedPublic


Mostly in revscoring

Event Timeline

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

For ORES 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: I was checking and amazingly I realized the integration between travis-ci and is broken, after fixing it, it seems we have 1.7% of coverage which is unfortunately correct.