Skip to content

tests: enable subprocess coverage#1916

Merged
Bastian-Krause merged 1 commit into
labgrid-project:masterfrom
Emantor:fix/subprocess-coverage
Jun 30, 2026
Merged

tests: enable subprocess coverage#1916
Bastian-Krause merged 1 commit into
labgrid-project:masterfrom
Emantor:fix/subprocess-coverage

Conversation

@Emantor

@Emantor Emantor commented Jun 30, 2026

Copy link
Copy Markdown
Member

Subprocess coverage used to be provided by default by pytest-cov, however that was removed in newer versions in favor of using the coverage patch functionality. Use the coverage patch functionality to re-enable coverage of subprocesses, re-enabling coverage for the remote tests.

Checklist

  • PR has been tested

Subprocess coverage used to be provided by default by pytest-cov,
however that was removed in newer versions in favor of using the
coverage patch functionality. Use the coverage patch functionality to
re-enable coverage of subprocesses, re-enabling coverage for the remote
tests.

Signed-off-by: Rouven Czerwinski <rouven.czerwinski@linaro.org>
@codecov

codecov Bot commented Jun 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.2%. Comparing base (6b40400) to head (b76559f).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##           master   #1916      +/-   ##
=========================================
+ Coverage    45.9%   61.2%   +15.2%     
=========================================
  Files         180     182       +2     
  Lines       14500   14781     +281     
=========================================
+ Hits         6666    9053    +2387     
+ Misses       7834    5728    -2106     
Flag Coverage Δ
3.10 61.2% <ø> (+15.2%) ⬆️
3.11 61.2% <ø> (+15.2%) ⬆️
3.12 61.2% <ø> (+15.2%) ⬆️
3.13 61.1% <ø> (+15.2%) ⬆️
3.14 61.1% <ø> (+15.2%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@Emantor

Emantor commented Jun 30, 2026

Copy link
Copy Markdown
Member Author

+15,2%, not bad 😁

@Bastian-Krause Bastian-Krause left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice, thank you for looking into that!

@Bastian-Krause
Bastian-Krause merged commit adcd323 into labgrid-project:master Jun 30, 2026
11 checks passed
@Emantor
Emantor deleted the fix/subprocess-coverage branch June 30, 2026 10:13
@Emantor Emantor added the picked to stable Already picked to the latest stable branch. label Jul 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI picked to stable Already picked to the latest stable branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants