Skip to content

Conversation

@pdobacz
Copy link
Contributor

@pdobacz pdobacz commented Apr 23, 2025

🗒️ Description

(requires evmone with this update to fill) EDIT: Merged

Also NOTE that, same as #1475 this PR adds tests and facilities for behavior of "legacy creation txs" (to: nil, EOF container in data), which is likely to change based on recently obtained feedback.

I'd like to merge this as a baby step (and #1475) and then update the behavior in a proper PR

🔗 Related Issues

NA

✅ Checklist

  • All: Set appropriate labels for the changes.
  • All: Considered squashing commits to improve commit history.
  • All: Added an entry to CHANGELOG.md.
  • All: Considered updating the online docs in the ./docs/ directory.
  • Tests: All converted JSON/YML tests from ethereum/tests have been added to converted-ethereum-tests.txt.
  • Tests: A PR with removal of converted JSON/YML tests from ethereum/tests have been opened.
  • Tests: Included the type and version of evm t8n tool used to locally execute test cases: e.g., ref with commit hash or geth 1.13.1-stable-3f40e65.
  • Tests: Ran mkdocs serve locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.

@pdobacz pdobacz added the scope:tests Scope: Changes EL client test cases in `./tests` label Apr 23, 2025
@pdobacz pdobacz self-assigned this Apr 23, 2025
@pdobacz pdobacz requested a review from chfast April 23, 2025 13:18
Copy link
Contributor

@rodiazet rodiazet left a comment

Choose a reason for hiding this comment

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

LGTM

@pdobacz pdobacz merged commit 33d4484 into ethereum:main Apr 24, 2025
11 checks passed
@pdobacz pdobacz deleted the creation-tx-7873 branch April 24, 2025 11:46
pacrob pushed a commit to pacrob/execution-spec-tests that referenced this pull request May 5, 2025
felix314159 pushed a commit to felix314159/execution-spec-tests that referenced this pull request May 16, 2025
kclowes pushed a commit to kclowes/execution-spec-tests that referenced this pull request Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:tests Scope: Changes EL client test cases in `./tests`

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants