Skip to content
This repository was archived by the owner on Jul 2, 2026. It is now read-only.

test(eip6110): cover forged deposit request bodies - #2325

Closed
pirapira wants to merge 1 commit into
ethereum:mainfrom
pirapira:test/e3vib-corrupt-ssz-deposits
Closed

test(eip6110): cover forged deposit request bodies#2325
pirapira wants to merge 1 commit into
ethereum:mainfrom
pirapira:test/e3vib-corrupt-ssz-deposits

Conversation

@pirapira

Copy link
Copy Markdown
Member

Summary

  • add a paired EIP-6110 blockchain test for Engine API deposit requests bodies
  • valid case supplies a requests body matching the deposit emitted by execution logs
  • forged case supplies a different deposit body and expects INVALID_REQUESTS while the header requests_hash remains internally consistent with that forged body

Validation

  • uv run python -m py_compile tests/prague/eip6110_deposits/test_deposits.py
  • git diff --check
  • uv run pytest --collect-only tests/prague/eip6110_deposits/test_deposits.py -k test_deposit_requests_hash_uses_execution_logs -q
  • attempted: uv run fill --fork Prague -m blockchain_test -k test_deposit_requests_hash_uses_execution_logs tests/prague/eip6110_deposits/test_deposits.py -q
    • blocked before test execution because the default EELS resolver tries to clone missing branch forks/bpos for Prague
  • attempted with local resolver: EELS_RESOLUTIONS='{"Prague":{"path":"/home/zksecurity/evm-asm/execution-specs"}}' uv run fill --fork Prague -m blockchain_test -k test_deposit_requests_hash_uses_execution_logs --output /tmp/e3vib-fill tests/prague/eip6110_deposits/test_deposits.py -q
    • reached transition output validation, then failed because local Prague resolver receipt logs omit blockNumber/transactionHash/transactionIndex/blockHash/logIndex/removed fields

@pirapira pirapira closed this Jun 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant