search

ethereum / execution-specs

Specification for the Execution Layer. Tracking network upgrades.
Creative Commons Zero v1.0 Universal
832 stars 234 forks source link

Make `evm tools` compatible with `execution-spec-tests` `tf` tool #762

Closed gurukamath closed 1 year ago

gurukamath commented 1 year ago

( closes #759 )

What was wrong?

Currently, the evm tools are not fully compatible with the tf tool on the execution-spec-tests repo.

Related to Issue #759

How was it fixed?

The following updates are included in this PR

  1. evm tools tests are implemented for Shanghai
  2. t8n tool updated to accept transaction rlp input and provide transaction rlp output.
  3. Other minor fixes that allow the evm tools from go-ethereum to be swapped 1-1 with the evm tools from the specs in execution-spec-tests

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

codecov-commenter commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (4708ccd) 74.01% compared to head (427e3da) 74.01%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #762 +/- ## ======================================= Coverage 74.01% 74.01% ======================================= Files 570 570 Lines 31557 31557 ======================================= Hits 23356 23356 Misses 8201 8201 ``` | Flag | Coverage Δ | | |---|---|---| | unittests | `74.01% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ethereum#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.