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

new(tests): EIP-7742 #994

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

new(tests): EIP-7742 #994

wants to merge 10 commits into from

Conversation

spencer-tb
Copy link
Collaborator

@spencer-tb spencer-tb commented Dec 4, 2024

🗒️ Description

Adds some basic tests for EIP-7742:

  • Test duplicates for the invalid Cancun blob tests that are now valid for Prague.
  • Tests for some arbitrary values above the Cancun MAX_BLOBS_PER_BLOCK.
  • Tests with multiple blocks using varied blobs per tx.
  • Tests with multiple blocks ascending from 1 to N.
  • Tests with multiple blocks descending from N to 1.
  • Tests with multiple blocks ascending from 1 to N then descending from N to 1.

We should still add fork transition tests in the future, where we test invalid blob counts before the fork (i.e 256/0 at Cancun). And consider adding some more tests ported from Cancun.

Fills with the following EELS resolution change for Prague:

    "Prague": {
        "git_url": "https://github.com/spencer-tb/execution-specs.git",
        "branch": "forks/prague",
        "commit": "51f5476045500bcb2e902cdf8fc4b6ee3f90a0fc"
    }

🔗 Related Issues

Requires #931.

✅ 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.

@spencer-tb spencer-tb self-assigned this Dec 4, 2024
@spencer-tb spencer-tb added scope:tests Scope: Test cases fork:prague Prague hardfork labels Dec 4, 2024
Base automatically changed from pectra-devnet-5 to main December 5, 2024 00:02
@marioevz marioevz changed the title tests(eip7742): initial tests for eip7742 new(tests): EIP-7742 Dec 5, 2024
@marioevz marioevz requested review from fselmo and reedsa December 10, 2024 14:53
@marioevz marioevz marked this pull request as draft December 10, 2024 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fork:prague Prague hardfork scope:tests Scope: Test cases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants