Conversation
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
| Address(0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFE): { | ||
| "nonce": 1, | ||
| "code": ( | ||
| "0x60203611603157600143035f35116029575f35612000014311602957612000" | ||
| "5f3506545f5260205ff35b5f5f5260205ff35b5f5ffd00" | ||
| ), | ||
| } | ||
| # Address(0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFE): { | ||
| # "nonce": 1, | ||
| # "code": ( | ||
| # "0x60203611603157600143035f35116029575f35612000014311602957612000" | ||
| # "5f3506545f5260205ff35b5f5f5260205ff35b5f5ffd00" | ||
| # ), | ||
| # } |
There was a problem hiding this comment.
@spencer-tb, this is a quick hack to disable the automatic inclusion of the EIP-2935 system contract. Ideally, I want this feature to avoid having this extra address in the genesis state since we want full predictability on which accounts exist for the tree conversion tests.
I think once we talked about an extra flag to do this more cleanly.
| @@ -57,7 +57,7 @@ def test_balance(blockchain_test: BlockchainTestFiller, fork: Fork, target, warm | |||
| ], | |||
There was a problem hiding this comment.
From this point forward, the changes are mostly the new tests so nothing framework related.
tests/verkle/eip7748/accounts.py
Outdated
| REFERENCE_SPEC_GIT_PATH = "EIPS/eip-7748.md" | ||
| REFERENCE_SPEC_VERSION = "TODO" | ||
|
|
||
| stride = 7 |
There was a problem hiding this comment.
Until the testing framework supports the stride as a filling option, I'm hardcoding it. This value allows me to create most of the test cases I want anyway.
See corresponding comment in go-ethereum
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
…artial conversion Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
… one converting accounts Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
| transition_tool_output.alloc = previous_alloc | ||
| # TODO: hack for now, replace with actual witness output once available from t8n | ||
| if transition_tool_output.result.verkle_conversion_ended: | ||
| if transition_tool_output.result.verkle_conversion_ended and False: |
There was a problem hiding this comment.
This needs a proper fix!
spencer-tb
added
feature:stateless
scope:tests
|
Closing for the Weld (moving EEST to EELS)! Feel free to create a PR for this in EELS in the future! |
2 participants
This PR is a work in progress for EIP-7748 (Tree conversion) tests.
uv run fill --from Shanghai --until EIP6800Transition -v -m blockchain_test -n 8 --evm-bin=<evm path> --evm-dump-dir=out ./tests/verkle/eip7748Use this branch for filling.