Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EIP-7702 - devnet-6 version #1114

Closed
wants to merge 2 commits into from
Closed

Conversation

gumb0
Copy link
Member

@gumb0 gumb0 commented Jan 23, 2025

This implements ethereum/EIPs#9248

Blockchain tests are currently failing, because system contract addresses need update.

@gumb0 gumb0 changed the base branch from master to eip-7702 January 23, 2025 16:24
@gumb0 gumb0 self-assigned this Jan 23, 2025
Copy link

codecov bot commented Jan 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.36%. Comparing base (c19fdd2) to head (a92a4ed).
Report is 2 commits behind head on eip-7702.

Additional details and impacted files
@@             Coverage Diff              @@
##           eip-7702    #1114      +/-   ##
============================================
+ Coverage     93.51%   94.36%   +0.85%     
============================================
  Files           160      161       +1     
  Lines         17691    17689       -2     
============================================
+ Hits          16544    16693     +149     
+ Misses         1147      996     -151     
Flag Coverage Δ
eof_execution_spec_tests 15.76% <55.00%> (+<0.01%) ⬆️
ethereum_tests 26.50% <45.00%> (+<0.01%) ⬆️
ethereum_tests_silkpre 18.70% <43.75%> (-0.01%) ⬇️
execution_spec_tests 22.04% <90.00%> (?)
unittests 89.24% <100.00%> (-0.01%) ⬇️

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

Files with missing lines Coverage Δ
lib/evmone/delegation.hpp 100.00% <100.00%> (ø)
lib/evmone/instructions_calls.cpp 97.90% <100.00%> (ø)
test/state/host.cpp 100.00% <100.00%> (ø)
test/state/host.hpp 100.00% <ø> (ø)
test/state/state.cpp 98.70% <100.00%> (+4.20%) ⬆️
test/state/state.hpp 100.00% <ø> (ø)
test/unittests/state_transition_eip7702_test.cpp 100.00% <100.00%> (ø)

... and 8 files with indirect coverage changes

@chfast chfast added the Prague Changes for Prague upgrade label Jan 24, 2025
@gumb0 gumb0 force-pushed the eip-7702-observe-delegation branch from acdf6d8 to 39f23b3 Compare January 24, 2025 17:26
@gumb0 gumb0 force-pushed the eip-7702-observe-delegation branch 3 times, most recently from e998fb7 to 0f4db7f Compare January 28, 2025 09:36
@gumb0 gumb0 force-pushed the eip-7702-observe-delegation branch 3 times, most recently from 6599453 to e63c272 Compare January 28, 2025 11:17
@gumb0 gumb0 force-pushed the eip-7702-observe-delegation branch from e63c272 to a92a4ed Compare January 31, 2025 12:39
@gumb0
Copy link
Member Author

gumb0 commented Jan 31, 2025

Merged into #961

@gumb0 gumb0 closed this Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Prague Changes for Prague upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants