Skip to content

Commit 167e873

Browse files
tkumor3ddoktorski
andauthored
Add missing field to ExecutionResources (#1301)
* Add missing field to ExecutionResources * linter * add test * rename test * Assert other execution resources fields in test * Use integers instead of felts for execution resources fields --------- Co-authored-by: Dariusz Doktorski <dariusz.doktorski@swmansion.com>
1 parent 303212e commit 167e873

File tree

3 files changed

+29
-9
lines changed

3 files changed

+29
-9
lines changed

starknet_py/net/client_models.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -363,6 +363,7 @@ class ExecutionResources:
363363
bitwise_builtin_applications: Optional[int] = None
364364
keccak_builtin_applications: Optional[int] = None
365365
memory_holes: Optional[int] = None
366+
segment_arena_builtin: Optional[int] = None
366367

367368

368369
# TODO (#1219): split into PendingTransactionReceipt and TransactionReceipt

starknet_py/net/schemas/rpc.py

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -116,29 +116,32 @@ def make_dataclass(self, data, **kwargs) -> L2toL1Message:
116116

117117

118118
class ExecutionResourcesSchema(Schema):
119-
steps = Felt(data_key="steps", required=True)
120-
range_check_builtin_applications = Felt(
119+
steps = fields.Integer(data_key="steps", required=True)
120+
range_check_builtin_applications = fields.Integer(
121121
data_key="range_check_builtin_applications", load_default=None
122122
)
123-
pedersen_builtin_applications = Felt(
123+
pedersen_builtin_applications = fields.Integer(
124124
data_key="pedersen_builtin_applications", load_default=None
125125
)
126-
poseidon_builtin_applications = Felt(
126+
poseidon_builtin_applications = fields.Integer(
127127
data_key="poseidon_builtin_applications", load_default=None
128128
)
129-
ec_op_builtin_applications = Felt(
129+
ec_op_builtin_applications = fields.Integer(
130130
data_key="ec_op_builtin_applications", load_default=None
131131
)
132-
ecdsa_builtin_applications = Felt(
132+
ecdsa_builtin_applications = fields.Integer(
133133
data_key="ecdsa_builtin_applications", load_default=None
134134
)
135-
bitwise_builtin_applications = Felt(
135+
bitwise_builtin_applications = fields.Integer(
136136
data_key="bitwise_builtin_applications", load_default=None
137137
)
138-
keccak_builtin_applications = Felt(
138+
keccak_builtin_applications = fields.Integer(
139139
data_key="keccak_builtin_applications", load_default=None
140140
)
141-
memory_holes = Felt(data_key="memory_holes", load_default=None)
141+
segment_arena_builtin = fields.Integer(
142+
data_key="segment_arena_builtin", load_default=None
143+
)
144+
memory_holes = fields.Integer(data_key="memory_holes", load_default=None)
142145

143146
@post_load
144147
def make_dataclass(self, data, **kwargs) -> ExecutionResources:

starknet_py/tests/e2e/tests_on_networks/client_test.py

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -425,3 +425,19 @@ async def test_get_events_sepolia_testnet(client_sepolia_testnet):
425425
assert isinstance(events_chunk, EventsChunk)
426426
assert len(events_chunk.events) == 10
427427
assert events_chunk.continuation_token is not None
428+
429+
430+
@pytest.mark.asyncio
431+
async def test_get_tx_receipt_with_execution_resources(client_sepolia_integration):
432+
receipt = await client_sepolia_integration.get_transaction_receipt(
433+
tx_hash=0x077E84B7C0C4CC88B778EEAEF32B7CED4500FE4AAEE62FD2F849B7DD90A87826
434+
)
435+
436+
assert receipt.execution_resources is not None
437+
assert receipt.execution_resources.steps is not None
438+
assert receipt.execution_resources.segment_arena_builtin is not None
439+
assert receipt.execution_resources.bitwise_builtin_applications is not None
440+
assert receipt.execution_resources.ec_op_builtin_applications is not None
441+
assert receipt.execution_resources.memory_holes is not None
442+
assert receipt.execution_resources.pedersen_builtin_applications is not None
443+
assert receipt.execution_resources.range_check_builtin_applications is not None

0 commit comments

Comments
 (0)