Project

General

Profile

Actions

action #80298

closed

coordination #80142: [saga][epic] Scale out: Redundant/load-balancing deployments of openQA, easy containers, containers on kubernetes

coordination #55364: [epic] Let's make codecov reports reliable

Fix flaky coverage - lib/OpenQA/WebSockets.pm

Added by okurz over 3 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Feature requests
Target version:
Start date:
2020-11-24
Due date:
% Done:

0%

Estimated time:

Description

Motivation

See #55364 : codecov reports often report about coverage changes which are obviously not related to the actual changes of a PR, e.g. when documentation is changed. We can already trust our coverage analysis more but should have only coverage changes reported for actual changes we introduced in a pull request.

Acceptance criteria

  • AC1: lib/OpenQA/WebSockets.pm does not appear anymore as changing code coverage in unrelated changes

Suggestions

  • One can start with rm -rf cover_db/ && make coverage KEEP_DB=1 TESTS=t/27-websockets.t but more than just this single test module is needed for complete coverage of lib/OpenQA/WebSockets.pm
  • check coverage details in generated html report
  • Fix uncovered lines with "uncoverable" statements or other means
  • retry multiple times to check for flakyness

Related issues 1 (0 open1 closed)

Copied from openQA Project - action #80274: Fix flaky coverage - t/lib/OpenQA/Test/Utils.pm size:MResolvedkraih2020-11-24

Actions
Actions #1

Updated by okurz over 3 years ago

  • Copied from action #80274: Fix flaky coverage - t/lib/OpenQA/Test/Utils.pm size:M added
Actions #2

Updated by okurz over 3 years ago

  • Status changed from Workable to Feedback
  • Assignee set to okurz

Let's see if https://github.com/os-autoinst/openQA/pull/3588 is enough, otherwise someone else can try to apply a proper solution. I am merely adding "uncoverable" statements :)

Actions #3

Updated by okurz over 3 years ago

  • Status changed from Feedback to Resolved

https://codecov.io/gh/os-autoinst/openQA/src/master/lib/OpenQA/WebSockets.pm shows that the file should have a consistent 100% statement coverage now.

Actions

Also available in: Atom PDF