action #168370
closedcoordination #127031: [saga][epic] openQA for SUSE customers
coordination #130414: [epic] Improved code coverage in os-autoinst
Improved code coverage in os-autoinst - consoles/sshVirtsh.pm size:M
0%
Description
Motivation¶
To be able to safely improve and extend os-autoinst, we need to have reliable and stable unit tests with even better code coverage, at least 100% statement coverage. Continue with the directory "consoles/"
Acceptance criteria¶
- AC1: codecov.yml ensures that we have at least 100% statement coverage (with explicit Devel::Cover exceptions allowed) in consoles/sshVirtsh.pm
Suggestions¶
- Identify areas showing less than 100.00% on
https://app.codecov.io/gh/os-autoinst/os-autoinst/blob/master/consoles%2FsshVirtsh.pm
- Remove dead code
- Add according test coverage
- Add explicit exception statements on unreachable code
- Add consoles/ to codecov.yml "fully_covered" until the complete project reaches 100%
Updated by okurz 2 months ago
- Copied from action #168367: Improved code coverage in os-autoinst - consoles/sshIucvconn.pm size:S added
Updated by ybonatakis about 1 month ago
- Status changed from Workable to In Progress
- Assignee set to ybonatakis
Updated by ybonatakis about 1 month ago
- Status changed from In Progress to Workable
Updated by ybonatakis 14 days ago
- Status changed from In Progress to Feedback
Updated by ybonatakis 10 days ago
tinita wrote in #note-9:
Is it 100% covered? Then it should be added to codecov.yml
I keep forgetting about it. I will do
Updated by ybonatakis 10 days ago
- Status changed from Workable to Feedback
ybonatakis wrote in #note-10:
tinita wrote in #note-9:
Is it 100% covered? Then it should be added to codecov.yml
I keep forgetting about it. I will do
Actually is 99.46% https://app.codecov.io/gh/os-autoinst/os-autoinst/tree/master/consoles.
Updated by ybonatakis 10 days ago
Updated by okurz 9 days ago
- Copied to action #174244: Improved code coverage in os-autoinst - consoles/sshXterm.pm size:S added
Updated by okurz 9 days ago
- Status changed from Feedback to Resolved
https://github.com/os-autoinst/os-autoinst/pull/2585 merged. Automatic check is in place and effective.