Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mark uncoverage statements in OpenQA::Worker::WebUIConnection #3718

Merged
merged 3 commits into from Feb 14, 2021

Conversation

okurz
Copy link
Member

@okurz okurz commented Feb 11, 2021

Unreliable coverage observed in tests for
#3713

Related progress issue: https://progress.opensuse.org/issues/55364

@okurz
Copy link
Member Author

okurz commented Feb 11, 2021

Anyone has an idea about the codecov related failure:

406 Not Acceptable
{'detail': ErrorDetail(string='Could not satisfy the request Accept header.', code='not_acceptable')}

?

@Martchus
Copy link
Contributor

No, the same happens on #3714 and persisted after re-triggering codecov.

@kraih
Copy link
Member

kraih commented Feb 11, 2021

Seems to happen for all our PRs now.

@codecov
Copy link

codecov bot commented Feb 14, 2021

Codecov Report

Merging #3718 (4c2e8c8) into master (02712a5) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3718      +/-   ##
==========================================
- Coverage   96.38%   96.38%   -0.01%     
==========================================
  Files         369      369              
  Lines       32279    32274       -5     
==========================================
- Hits        31111    31106       -5     
  Misses       1168     1168              
Impacted Files Coverage Δ
lib/OpenQA/Worker/WebUIConnection.pm 89.90% <100.00%> (-0.24%) ⬇️
t/25-cache.t 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7880be7...4c2e8c8. Read the comment docs.

@mergify mergify bot merged commit 525d96f into os-autoinst:master Feb 14, 2021
openqabot pushed a commit to openqabot/openQA that referenced this pull request Feb 15, 2021
commit 525d96f
Merge: 86968e7 4c2e8c8
Author:     mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
AuthorDate: Sun Feb 14 15:55:10 2021 +0000
Commit:     GitHub <noreply@github.com>
CommitDate: Sun Feb 14 15:55:10 2021 +0000

    Merge pull request os-autoinst#3718 from okurz/fix/coverage

    Mark uncoverage statements in OpenQA::Worker::WebUIConnection
@okurz okurz deleted the fix/coverage branch February 15, 2021 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants