Skip to content

🧹 chore(tests): Remove duplicate frame tx coverage - #3946

Merged
gurukamath merged 1 commit into
ethereum:eips/bogota/eip-8141from
raxhvl:frames/1
Nov 3, 2026
Merged

gurukamath merged 1 commit into
ethereum:eips/bogota/eip-8141from
raxhvl:frames/1

Conversation

@raxhvl

@raxhvl raxhvl commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

The other case of test_value_transfer covers deleted test, except asserting verify frame as no logs - which has been carried over.

@codecov

codecov Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.12%. Comparing base (bac6e1a) to head (5b90b0c).
⚠️ Report is 1 commits behind head on eips/bogota/eip-8141.

Additional details and impacted files
@@                  Coverage Diff                  @@
##           eips/bogota/eip-8141    #3946   +/-   ##
=====================================================
  Coverage                 95.12%   95.12%           
=====================================================
  Files                       677      677           
  Lines                     41914    41914           
  Branches                   3941     3941           
=====================================================
  Hits                      39870    39870           
  Misses                     1418     1418           
  Partials                    626      626           
Flag Coverage Δ
unittests 95.11% <ø> (ø)

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.

@gurukamath
gurukamath merged commit 8992fc5 into ethereum:eips/bogota/eip-8141 Nov 3, 2026
20 checks passed
@raxhvl
raxhvl deleted the frames/1 branch November 3, 2026 07:55
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