Project

General

Profile

action #121357

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 months ago. Updated 17 days ago.

Status:
Resolved
Priority:
Low
Assignee:
Category:
Feature requests
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Difficulty:

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


Related issues

Copied from openQA Project - action #94952: [easy][beginner] Increase code coverage of os-autoinst basetest.pm size:MResolved2021-06-30

Copied to openQA Project - action #122047: [tools] tesseract ocr test failsResolved2022-12-15

History

#1 Updated by okurz about 2 months ago

  • Copied from action #94952: [easy][beginner] Increase code coverage of os-autoinst basetest.pm size:M added

#2 Updated by cdywan about 2 months ago

  • 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

#3 Updated by cdywan about 2 months ago

Discussed with Ondřej, first small PR: https://github.com/os-autoinst/os-autoinst/pull/2229

#4 Updated by cdywan about 2 months ago

  • Status changed from Workable to In Progress
  • Assignee set to osukup

#7 Updated by osukup about 2 months ago

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 :(

#8 Updated by osukup about 2 months ago

https://github.com/os-autoinst/os-autoinst/pull/2233 --> add terrasect-ocr to os-autoinst test dependencies

#9 Updated by osukup about 2 months ago

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)

#10 Updated by osukup about 2 months ago

#11 Updated by osukup about 2 months ago

--> 02-test_ocr.t 94% , needs only mark skip codepath with uncoverable statement

#14 Updated by osukup 17 days ago

  • Status changed from In Progress to Resolved

Also available in: Atom PDF