Project

General

Profile

Actions

action #137525

closed

os-autoinst codecov "fully_covered" check fails in unrelated PRs, maybe since https://github.com/os-autoinst/os-autoinst/pull/2364

Added by okurz 7 months ago. Updated 7 months ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Category:
Regressions/Crashes
Target version:
Start date:
2023-10-06
Due date:
% Done:

0%

Estimated time:

Description

Observation

https://github.com/os-autoinst/os-autoinst/pull/2377 failed the check "codecov/project/fully_covered 99.99% (target 100.00%)" though there are no related changes in this PR. According to https://app.codecov.io/gh/os-autoinst/os-autoinst/pull/2377/tree the folder backend is not covered 100%. https://app.codecov.io/gh/os-autoinst/os-autoinst/pull/2377/blob/backend/ipmi.pm#L54 shows the problem in detail

# DELL BMCs are touchy
sub dell_sleep ($self) {
    return unless ($bmwqemu::vars{IPMI_HW} || '') eq 'dell';
    sleep 4; <-- this line
}

likely caused by https://github.com/os-autoinst/os-autoinst/pull/2364 which I merged by mistake.

Acceptance criteria

  • AC1: "fully_covered" check passes again

Suggestions

  • Ensure that there is 100% statement coverage again in backend/ipmi.pm. For that you could try to set IPMI_HW=dell. Likely the 4s sleep is already mocked by Test::Mock::Time used in t/29-backend-ipmi.t
Actions #1

Updated by okurz 7 months ago

  • Status changed from New to In Progress
  • Assignee set to okurz
Actions #2

Updated by okurz 7 months ago

  • Due date set to 2023-10-20
  • Status changed from In Progress to Feedback
Actions #3

Updated by okurz 7 months ago

  • Due date deleted (2023-10-20)
  • Status changed from Feedback to Resolved

merged. Another later PR https://github.com/os-autoinst/os-autoinst/pull/2379 showed no problems so I assume we are good

Actions

Also available in: Atom PDF