action #167941
opencoordination #127031: [saga][epic] openQA for SUSE customers
coordination #130414: [epic] Improved code coverage in os-autoinst
Cover code of os-autoinst path distribution.pm fully (statement coverage) size:S
Description
Acceptance criteria¶
- AC1: the path distribution.pm is listed in codecov.yml under "fully_covered"
Suggestions¶
- Take a look into https://app.codecov.io/gh/os-autoinst/os-autoinst/blob/master/distribution.pm about what is missing
- Add unit test statement coverage for that to bump the target
Updated by okurz 3 months ago
- Copied from action #167938: Cover code of os-autoinst path lockapi.pm fully (statement coverage) size:S added
Updated by okurz 3 months ago
- Copied to action #167944: Cover code of os-autoinst path cv.pm fully (statement coverage) added
Updated by gpathak 8 days ago
Pull request Created: https://github.com/os-autoinst/os-autoinst/pull/2597
Code Coverage on PR: https://app.codecov.io/gh/os-autoinst/os-autoinst/pull/2597/blob/distribution.pm
Updated by gpathak 3 days ago
Pull request Merged: https://github.com/os-autoinst/os-autoinst/pull/2597
Codecov: https://app.codecov.io/gh/os-autoinst/os-autoinst/blob/master/distribution.pm
There are still some uncovered bits https://app.codecov.io/gh/os-autoinst/os-autoinst/blob/master/distribution.pm#L68, they are related to fedora and debian distribution.
To cover that we need to write or create something similar to https://github.com/os-autoinst/os-autoinst-distri-opensuse.git
Updated by gpathak about 12 hours ago
Created a new PR: https://github.com/os-autoinst/os-autoinst/pull/2605
Now, almost everytihing is covered: https://app.codecov.io/gh/os-autoinst/os-autoinst/blob/gauravpathak%3Adistribution-codecov/distribution.pm?trend=3%20months
There are no uncovered lines or statements.