action #121357
closed
coordination #109668: [saga][epic] Stable and updated non-qemu backends for SLE validation
coordination #109740: [epic] Stable os-autoinst unit tests with good coverage
[easy][beginner] Ensure 100% test coverage of t/ in os-autoinst size:M
Added by okurz about 2 years ago.
Updated almost 2 years ago.
Category:
Feature requests
Description
Motivation¶
All code should be fully covered or marked as uncoverable so that we can spot gaps easily. As of this writing we have 98.86%.
Acceptance criteria¶
- AC1: All places that could be covered by tests but aren't have been identified
- AC2: Codecov reports 100%
Comments¶
- Copied from action #94952: [easy][beginner] Increase code coverage of os-autoinst basetest.pm size:M added
- Subject changed from [easy][beginner] Ensure 100% test coverage of t/ to [easy][beginner] Ensure 100% test coverage of t/ in os-autoinst size:M
- Description updated (diff)
- Status changed from New to Workable
- Status changed from Workable to In Progress
- Assignee set to osukup
after merging @chris PR's -> 99.04%
it should be more ->
codecov reports:
Commit YAML is invalid
Coverage data is unable to be displayed, as the commit YAML appears to be invalid.
so it contains old data :(
so new PR uncovered https://github.com/os-autoinst/os-autoinst/actions/runs/3703495822/jobs/6275014804
7: "tesseract" unexpectedly returned exit value 1 at /opt/ppmclibs/blib/lib/tinycv.pm line 113.
631
7: at /opt/ocr.pm line 20
632
7: # Tests were run but no plan was declared and done_testing() was not seen.
633
7: # Looks like your test exited with 1 just after 2.
634
7: [11:10:03] ./t/02-test_ocr.t ..........................
635
7: Dubious, test returned 1 (wstat 256, 0x100)
--> 02-test_ocr.t 94% , needs only mark skip codepath with uncoverable statement
- Status changed from In Progress to Resolved
Also available in: Atom
PDF