Skip to content

Conversation

@jeltz
Copy link
Collaborator

@jeltz jeltz commented Nov 12, 2025

Including the feutils directory in the totals in the coverage report does not make too much sense since we only test the features relevant to pg_tde.

It is a bit tricky to test if a config change like this actually works but let's take stab at it.

Also update components to match current realities.

@jeltz jeltz marked this pull request as draft November 12, 2025 09:55
@codecov-commenter
Copy link

codecov-commenter commented Nov 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.69%. Comparing base (2f0315d) to head (7c64ec7).
⚠️ Report is 1 commits behind head on main.

❌ Your project status has failed because the head coverage (59.69%) is below the target coverage (90.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #464   +/-   ##
=======================================
  Coverage   59.69%   59.69%           
=======================================
  Files          67       67           
  Lines       10471    10471           
  Branches     1813     1813           
=======================================
  Hits         6251     6251           
  Misses       3524     3524           
  Partials      696      696           
Components Coverage Δ
access 84.89% <ø> (ø)
catalog 87.93% <ø> (ø)
common 77.77% <ø> (ø)
encryption 71.56% <ø> (ø)
keyring 73.54% <ø> (ø)
src 94.20% <ø> (ø)
smgr 94.06% <ø> (ø)
transam ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

I am not sure this split in componenets makes sense but at least let's
update it to match the current directories.
Including the feutils directory in the totals in the coverage report
does not make too much sense since we only test the features relevant to
pg_tde.

It is a bit tricky to test if a config change like this actually works
but let's take stab at it.
@jeltz jeltz changed the title Fix coverage Try to exclude feutils directory from Codecov report and update componenets Nov 12, 2025
@jeltz jeltz marked this pull request as ready for review November 12, 2025 11:07
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.

2 participants