This repository was archived by the owner on Jul 2, 2026. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 200
new(tests): add more test cases for request types #1340
Merged
Merged
Changes from 5 commits
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
a7d548e
new(tests): added more cases (two contract req, one EOA req) in test_β¦
felix314159 f67fbb2
fixed comments
felix314159 e22fe91
rename test_deposits_withdrawals_consolidations.py to test_multi_typeβ¦
felix314159 4310e6c
changed type from ParameterSet to a union of the three request types,β¦
felix314159 fb157d6
added draft test of trying out a modified withdrawal contract that alβ¦
felix314159 4c48495
fixed docstrings
felix314159 c0f885d
tried to implement marios feedback
felix314159 b61fe3e
moved test to a better suited folder (eip7002)
felix314159 d2519c4
parameterized test and bugfixes
felix314159 e9a5740
added pseudo contract that just returns withdrawals requests
felix314159 c1e9fc9
removed unnecessary offset update
felix314159 85ac1ff
macro MSTORE now used instead of manually splitting into chunks
felix314159 1d5b358
Update tests/prague/eip7002_el_triggerable_withdrawals/test_modified_β¦
marioevz 0ceffe2
Datatype is ParameterSet
felix314159 10f41b0
added pytest marker for prague, and some minor fixes
felix314159 8c3adbd
removed comment and updated CHANGELOG.md
felix314159 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
97 changes: 97 additions & 0 deletions
97
tests/prague/eip7685_general_purpose_el_requests/test_modified_withdrawal_contract.py
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,97 @@ | ||
| """ | ||
| abstract: Tests [EIP-7685: General purpose execution layer requests](https://eips.ethereum.org/EIPS/eip-7685) | ||
| Cross testing for withdrawal and deposit request for [EIP-7685: General purpose execution layer requests](https://eips.ethereum.org/EIPS/eip-7685). | ||
|
|
||
| """ # noqa: E501 | ||
|
|
||
| import pytest | ||
|
|
||
| from ethereum_test_tools import ( | ||
| Account, | ||
| Alloc, | ||
| Block, | ||
| BlockchainTestFiller, | ||
| Header, | ||
| Transaction, | ||
| ) | ||
| from ethereum_test_types import Requests | ||
| from ethereum_test_types.types import EOA | ||
| from tests.prague.eip7685_general_purpose_el_requests.test_multi_type_requests import ( | ||
| single_withdrawal_from_contract, | ||
| single_withdrawal_from_eoa, | ||
| ) | ||
|
|
||
| from ..eip7002_el_triggerable_withdrawals.helpers import ( | ||
| WithdrawalRequestContract, | ||
| ) | ||
| from ..eip7002_el_triggerable_withdrawals.spec import Spec as Spec_EIP7002 | ||
| from .spec import ref_spec_7685 | ||
|
|
||
| REFERENCE_SPEC_GIT_PATH: str = ref_spec_7685.git_path | ||
| REFERENCE_SPEC_VERSION: str = ref_spec_7685.version | ||
|
|
||
|
|
||
| @pytest.mark.parametrize( | ||
| "requests", | ||
| [ | ||
| # create 18 withdrawal contract requests (usually 16 is max, but modified_code dequeues 18) | ||
| [ | ||
| single_withdrawal_from_contract(i) | ||
|
marioevz marked this conversation as resolved.
Outdated
|
||
| for i in range( | ||
| 0, | ||
| 18, | ||
| ) | ||
| ], | ||
| # create 18 withdrawal requests | ||
| [ | ||
| single_withdrawal_from_eoa(i) | ||
| for i in range( | ||
| 0, | ||
| 18, | ||
| ) | ||
| ], | ||
| ], | ||
| ) | ||
| def test_extra_withdrawals( | ||
| blockchain_test: BlockchainTestFiller, | ||
| pre: Alloc, | ||
| requests: WithdrawalRequestContract, | ||
| ): | ||
| """Test how clients were to behave when more than 16 withdrawals would be allowed per block.""" | ||
| modified_code: bytes = b"3373fffffffffffffffffffffffffffffffffffffffe1460cb5760115f54807fffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff146101f457600182026001905f5b5f82111560685781019083028483029004916001019190604d565b909390049250505036603814608857366101f457346101f4575f5260205ff35b34106101f457600154600101600155600354806003026004013381556001015f35815560010160203590553360601b5f5260385f601437604c5fa0600101600355005b6003546002548082038060121160df575060125b5f5b8181146101835782810160030260040181604c02815460601b8152601401816001015481526020019060020154807fffffffffffffffffffffffffffffffff00000000000000000000000000000000168252906010019060401c908160381c81600701538160301c81600601538160281c81600501538160201c81600401538160181c81600301538160101c81600201538160081c81600101535360010160e1565b910180921461019557906002556101a0565b90505f6002555f6003555b5f54807fffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff14156101cd57505f5b6001546002828201116101e25750505f6101e8565b01600290035b5f555f600155604c025ff35b5f5ffd" # noqa: E501 | ||
| pre[Spec_EIP7002.WITHDRAWAL_REQUEST_PREDEPLOY_ADDRESS] = Account( | ||
| code=modified_code, | ||
| nonce=1, | ||
| balance=0, | ||
| ) | ||
|
|
||
| sender: EOA = pre.fund_eoa() | ||
| tx: Transaction = Transaction( | ||
|
marioevz marked this conversation as resolved.
Outdated
|
||
| to=Spec_EIP7002.WITHDRAWAL_REQUEST_PREDEPLOY_ADDRESS, | ||
| sender=sender, | ||
| gas_limit=100_000, | ||
| ) | ||
|
|
||
| blockchain_test( | ||
| pre=pre, | ||
| blocks=[ | ||
| Block( | ||
| txs=[tx], | ||
| # header_verify=Header( | ||
| # requests_hash=Requests( | ||
|
marioevz marked this conversation as resolved.
Outdated
|
||
| # *[ | ||
| # request.with_source_address( | ||
| # Spec_EIP7002.WITHDRAWAL_REQUEST_PREDEPLOY_SENDER | ||
| # ) | ||
| # for request in sorted(requests, key=lambda r: r.type) | ||
| # ], | ||
| # ) | ||
| # | ||
| # requests_hash=requests, | ||
| # | ||
| # withdrawals_root=requests, | ||
| # ), | ||
| ), | ||
| ], | ||
| post={}, | ||
| ) | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.