Skip to content

Commit

Permalink
feat(pytests): OmmersRoot is EmptyOmmersRoot by default instead of 0 (e…
Browse files Browse the repository at this point in the history
…thereum#890)

* put EmptyOmmersRoot in env file instead of 0 by default

* fix(types): tests

---------

Co-authored-by: Mario Vega <[email protected]>
  • Loading branch information
winsvega and marioevz authored Oct 10, 2024
1 parent e6d68b0 commit 41c7d4e
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/ethereum_test_types/tests/test_types.py
Original file line number Diff line number Diff line change
Expand Up @@ -486,7 +486,7 @@ def test_account_merge(
"blockHashes": {},
"ommers": [],
"parentUncleHash": (
"0x0000000000000000000000000000000000000000000000000000000000000000"
"0x1dcc4de8dec75d7aab85b567b6ccd41ad312451b948a7413f0a142fd40d49347"
),
},
id="environment_1",
Expand Down
3 changes: 2 additions & 1 deletion src/ethereum_test_types/types.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@
BLSSignature,
Bytes,
CamelModel,
EmptyOmmersRoot,
Hash,
HexNumber,
Number,
Expand Down Expand Up @@ -402,7 +403,7 @@ class Environment(EnvironmentGeneric[Number]):
"""

blob_gas_used: Number | None = Field(None, alias="currentBlobGasUsed")
parent_ommers_hash: Hash = Field(Hash(0), alias="parentUncleHash")
parent_ommers_hash: Hash = Field(Hash(EmptyOmmersRoot), alias="parentUncleHash")
parent_blob_gas_used: Number | None = Field(None)
parent_excess_blob_gas: Number | None = Field(None)
parent_beacon_block_root: Hash | None = Field(None)
Expand Down

0 comments on commit 41c7d4e

Please sign in to comment.