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

Avoid uncovered lines in t/24-worker-engine.t #4128

Merged
merged 1 commit into from Aug 18, 2021

Conversation

Martchus
Copy link
Contributor

The package … lines for sub packages sometimes show up as uncovered lines
so this change marks them as uncoverable like we already do in other
places.

The `package …` lines for sub packages sometimes show up as uncovered lines
so this change marks them as uncoverable like we already do in other
places.
@codecov
Copy link

codecov bot commented Aug 17, 2021

Codecov Report

Merging #4128 (fecdeca) into master (b0b22cf) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4128   +/-   ##
=======================================
  Coverage   97.79%   97.80%           
=======================================
  Files         371      371           
  Lines       33182    33182           
=======================================
+ Hits        32450    32453    +3     
+ Misses        732      729    -3     
Impacted Files Coverage Δ
t/24-worker-engine.t 100.00% <100.00%> (ø)
t/05-scheduler-full.t 100.00% <0.00%> (+1.59%) ⬆️

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 b0b22cf...fecdeca. Read the comment docs.

@kalikiana kalikiana merged commit 6dc5381 into os-autoinst:master Aug 18, 2021
@Martchus Martchus deleted the coverage branch August 18, 2021 14:36
openqabot pushed a commit to openqabot/openQA that referenced this pull request Aug 19, 2021
commit 6dc5381
Merge: 7299db3 fecdeca
Author:     Cris Dywan (they/them) <cdywan@suse.de>
AuthorDate: Wed Aug 18 16:28:13 2021 +0200
Commit:     GitHub <noreply@github.com>
CommitDate: Wed Aug 18 16:28:13 2021 +0200

    Merge pull request os-autoinst#4128 from Martchus/coverage

    Avoid uncovered lines in `t/24-worker-engine.t`
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

2 participants