Skip to content

XXX Test better mok coverage - #821

Draft
vathpela wants to merge 2 commits into
rhboot:mainfrom
vathpela:test-better-mok-coverage
Draft

XXX Test better mok coverage#821
vathpela wants to merge 2 commits into
rhboot:mainfrom
vathpela:test-better-mok-coverage

Conversation

@vathpela

Copy link
Copy Markdown
Member

don't merge this yet.

@vathpela
vathpela force-pushed the test-better-mok-coverage branch from 51cc826 to 74e69e7 Compare July 27, 2026 19:51
@codecov

codecov Bot commented Jul 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.77%. Comparing base (72fd4b7) to head (1f66299).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #821      +/-   ##
==========================================
- Coverage   59.48%   58.77%   -0.71%     
==========================================
  Files          24       24              
  Lines        4226     4226              
  Branches      655      655              
==========================================
- Hits         2514     2484      -30     
  Misses       1471     1471              
- Partials      241      271      +30     

see 6 files with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

This moves the definition for STATIC so that it's always defined as
nothing for SHIM_UNIT_TEST and as "static" for everything else.

In doing so, it adds a header "shim-unit-test.h" which is at the very
end of shim.h .

Signed-off-by: Peter Jones <pjones@redhat.com>
@vathpela
vathpela force-pushed the test-better-mok-coverage branch from 74e69e7 to 76545bd Compare July 27, 2026 20:02
Signed-off-by: Peter Jones <pjones@redhat.com>
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.

1 participant