Skip to content

fix(consume): fix loading of multi-index, mixed tx type fixtures #1330

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

Merged
merged 3 commits into from
Mar 19, 2025

Conversation

danceratopz
Copy link
Member

@danceratopz danceratopz commented Mar 19, 2025

🗒️ Description

State test fixtures from ethereum/{tests,legacyTests} can be parametrized via multiple transactions that take different values of data, gas and value. In this case, the tx type can be either be an access list type transaction (0x1), or not (0x0). For the latter, the entry in the transaction accessList field is set to null. This change handles this case. An empty list can also be specified as entry in this field, but then this is a 0x1 type tx.

Here's an example:
https://github.com/ethereum/tests/blob/f4f76dc8a0bd99297c70e8a21b1e31ce4263d720/GeneralStateTests/stEIP1559/lowGasPriceOldTypes.json#L170-L173

🔗 Related Issues

None

✅ Checklist

  • All: Set appropriate labels for the changes.
  • All: Considered squashing commits to improve commit history.
  • All: Added an entry to CHANGELOG.md.

Test fixtures from ethereum/{tests,legacyTests} can be parametrized by data, gas and value index. In this case, the tx type can be either be an access list type transaction (`0x1`), or not (`0x0`). For the latter, the entry in the transaction `accessList` field is set to `null` (an empty list can also be specified, but then this is a `0x1` type tx). This change handles this case.
@danceratopz danceratopz added type:bug Something isn't working scope:consume Scope: Consume command suite labels Mar 19, 2025
@danceratopz danceratopz requested a review from marioevz March 19, 2025 11:30
Copy link
Member

@marioevz marioevz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

I think if we ever do a v2 for the state tests fixture formats, access lists is something that needs to be corrected.

@marioevz marioevz merged commit 286c026 into main Mar 19, 2025
21 checks passed
@marioevz marioevz deleted the consume-for-ethereum-tests branch March 19, 2025 16:55
felix314159 pushed a commit to felix314159/execution-spec-tests that referenced this pull request May 16, 2025
…ereum#1330)

* fix(consume): fix loading of multi-index mixed tx type fixtures

Test fixtures from ethereum/{tests,legacyTests} can be parametrized by data, gas and value index. In this case, the tx type can be either be an access list type transaction (`0x1`), or not (`0x0`). For the latter, the entry in the transaction `accessList` field is set to `null` (an empty list can also be specified, but then this is a `0x1` type tx). This change handles this case.

* docs: update changelog

* docs: update pr link in changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope:consume Scope: Consume command suite type:bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants