Skip to content

Conversation

@spencer-tb
Copy link
Collaborator

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

🗒️ Description

Adds some basic tests for EIP-7742:

  • Test duplicates for the invalid Cancun blob tests that are now valid for Prague.
  • Test for zero blobs in a type 3 txs.
  • 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 0 to N.
  • Tests with multiple blocks descending from N to 0.
  • Tests with multiple blocks ascending from 0 to N then descending from N to 0.

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": "857c3fa4d733535f32e9f17250d5790dd7ab5a76"
    }

🔗 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 added scope:tests Scope: Changes EL client test cases in `./tests` fork:prague Prague hardfork labels Dec 3, 2024
@spencer-tb spencer-tb self-assigned this Dec 3, 2024
@spencer-tb spencer-tb merged commit e2ca766 into pectra-devnet-5 Dec 4, 2024
1 of 6 checks passed
@spencer-tb spencer-tb deleted the pectra-7742-tests branch December 4, 2024 18:27
@spencer-tb spencer-tb restored the pectra-7742-tests branch December 4, 2024 18:29
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: Changes EL client test cases in `./tests`

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants