Skip to content

Commit

Permalink
Add extra assert to ensure withdrawal creds are different
Browse files Browse the repository at this point in the history
  • Loading branch information
jtraglia committed Jan 20, 2025
1 parent a079dbf commit 13d9aa1
Showing 1 changed file with 4 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -278,6 +278,9 @@ def test_deposit_transition__deposit_with_same_pubkey_different_withdrawal_crede
indices_with_same_pubkey = [1, 3]
for index in indices_with_same_pubkey:
block.body.execution_requests.deposits[index].pubkey = block.body.deposits[0].data.pubkey
# ensure top-up deposit request withdrawal credentials are different than the deposit
assert (block.body.execution_requests.deposits[index].withdrawal_credentials
!= block.body.deposits[0].data.withdrawal_credentials)

block.body.execution_payload.block_hash = compute_el_block_hash_for_block(spec, block)

Expand All @@ -288,5 +291,6 @@ def test_deposit_transition__deposit_with_same_pubkey_different_withdrawal_crede
assert len(state.pending_deposits) == deposit_request_count + deposit_count
for index in indices_with_same_pubkey:
assert state.pending_deposits[deposit_count + index].pubkey == deposit_requests[index].pubkey
# ensure withdrawal credentials are retained, rather than being made the same
assert (state.pending_deposits[deposit_count + index].withdrawal_credentials
== deposit_requests[index].withdrawal_credentials)

0 comments on commit 13d9aa1

Please sign in to comment.