action #10262
closed
action #10260: Improve test coverage
Prevent further test coverage drop
Added by okurz almost 9 years ago.
Updated over 8 years ago.
Description
user story¶
As openqa backend as well as test developer I get feedback on the coverage change in my PRs not to introduce coverage regressions (to prevent technical debt increase)
acceptance criteria¶
suggestions¶
- check how coverage is currently gathered or was e.g. in https://github.com/os-autoinst/openQA/pull/51
- research why it does not work for current changes anymore
- re-add coverage analysis to local tests
- re-add coverage analysis to travis/coveralls calls
further notes¶
seems like there was a notification on coverage using coveralls some time ago but it seems it got either disabled or somehow broke. It should be re-enabled and extended to other repositories as available.
- Priority changed from High to Normal
- Status changed from New to In Progress
- Checklist item changed from [ ] os-autoinst, [ ] openQA, [ ] os-autoinst-distri-opensuse/master, [ ] os-autoinst-distri-opensuse/sle12 to [ ] os-autoinst, [x] openQA, [ ] os-autoinst-distri-opensuse/master, [ ] os-autoinst-distri-opensuse/sle12
with gh#540 merged we got openQA covered, other projects still missing
- Checklist item changed from [ ] os-autoinst, [x] openQA, [ ] os-autoinst-distri-opensuse/master, [ ] os-autoinst-distri-opensuse/sle12 to [ ] os-autoinst, [x] openQA, [ ] os-autoinst-distri-opensuse/master
- Assignee changed from oholecek to okurz
- Checklist item changed from [ ] os-autoinst, [x] openQA, [ ] os-autoinst-distri-opensuse/master to [x] os-autoinst, [x] openQA, [ ] os-autoinst-distri-opensuse/master
- Checklist item changed from [x] os-autoinst, [x] openQA, [ ] os-autoinst-distri-opensuse/master to [x] os-autoinst, [x] openQA
- Status changed from In Progress to Resolved
we have proper coveralls check for os-autoinst now, too. openQA and os-autoinst track test coverage changes so all good. The tests never had a coverage analysis so this is a totally different issue and must be tracked elsewhere.
Also available in: Atom
PDF