Skip to content

fix(tooling): include namespace packages in coverage reports#2982

Open
danceratopz wants to merge 1 commit into
ethereum:forks/amsterdamfrom
danceratopz:include-namespace-packages-in-coverage
Open

fix(tooling): include namespace packages in coverage reports#2982
danceratopz wants to merge 1 commit into
ethereum:forks/amsterdamfrom
danceratopz:include-namespace-packages-in-coverage

Conversation

@danceratopz

@danceratopz danceratopz commented Jun 11, 2026

Copy link
Copy Markdown
Member

🗒️ Description

Enable coverage.py's include_namespace_packages option so that never-imported modules under the ethereum.forks namespace package count towards the coverage denominator (at 0%); this gives all coverage uploads the same denominator and explains the apparent coverage drop seen in #2975, where fill-only reports silently excluded the never-imported spurious_dragon and tangerine_whistle fork trees.

🔗 Related Issues or PRs

Related to #2975.

✅ Checklist

  • All: Ran fast static checks to avoid unnecessary CI fails, see also Code Standards and Enabling Pre-commit Checks:
    just static
  • All: PR title adheres to the repo standard - it will be used as the squash commit message and should start type(scope):.
  • All: Considered updating the online docs in the ./docs/ directory.
  • All: Set appropriate labels for the changes (only maintainers can apply labels).

Cute Animal Picture

A cute dog

@danceratopz danceratopz added C-bug Category: this is a bug, deviation, or other problem A-tooling Area: Improvements or changes to auxiliary tooling such as uv, ruff, mypy, ... labels Jun 11, 2026
@codecov

codecov Bot commented Jun 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.29%. Comparing base (5f132e7) to head (321d465).
⚠️ Report is 10 commits behind head on forks/amsterdam.

Additional details and impacted files
@@                 Coverage Diff                 @@
##           forks/amsterdam    #2982      +/-   ##
===================================================
- Coverage            90.52%   81.29%   -9.24%     
===================================================
  Files                  535      620      +85     
  Lines                32893    36635    +3742     
  Branches              3021     3311     +290     
===================================================
+ Hits                 29778    29782       +4     
- Misses                2596     6335    +3739     
+ Partials               519      518       -1     
Flag Coverage Δ
unittests 81.29% <ø> (-9.24%) ⬇️

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.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-tooling Area: Improvements or changes to auxiliary tooling such as uv, ruff, mypy, ... C-bug Category: this is a bug, deviation, or other problem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant