Skip to content

Commit a88c1e6

Browse files
committed
fix withdrawal amount filter for check_consensus_block_proposals task
1 parent 05bf35d commit a88c1e6

File tree

3 files changed

+8
-9
lines changed

3 files changed

+8
-9
lines changed

pkg/coordinator/tasks/check_consensus_block_proposals/task.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -640,7 +640,6 @@ func (t *Task) checkBlockWithdrawals(block *consensus.Block, blockData *spec.Ver
640640

641641
if validator.Validator.PublicKey.String() == expectedWithdrawal.PublicKey {
642642
withdrawalAmount := big.NewInt(int64(withdrawal.Amount))
643-
withdrawalAmount = withdrawalAmount.Mul(withdrawalAmount, big.NewInt(1000000000))
644643

645644
switch {
646645
case expectedWithdrawal.Address != "" && !strings.EqualFold(expectedWithdrawal.Address, withdrawal.Address.String()):

playbooks/pectra-dev/validator-lifecycle-test-v3.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -557,13 +557,13 @@ tasks:
557557
config:
558558
minWithdrawalCount: 1
559559
configVars:
560-
expectWithdrawals: "| [{publicKey: .tasks.all_pubkeys.outputs.pubkeys[1], address: .tasks.depositor_wallet.outputs.childWallet.address, minAmount: 900000000000000000}]"
560+
expectWithdrawals: "| [{publicKey: .tasks.all_pubkeys.outputs.pubkeys[1], address: .tasks.depositor_wallet.outputs.childWallet.address, minAmount: 900000000}]"
561561
- name: check_consensus_block_proposals
562562
title: "Wait for partial withdrawal (>15.9 ETH) for key 4"
563563
config:
564564
minWithdrawalCount: 1
565565
configVars:
566-
expectWithdrawals: "| [{publicKey: .tasks.all_pubkeys.outputs.pubkeys[4], address: .tasks.depositor_wallet.outputs.childWallet.address, minAmount: 15900000000000000000}]"
566+
expectWithdrawals: "| [{publicKey: .tasks.all_pubkeys.outputs.pubkeys[4], address: .tasks.depositor_wallet.outputs.childWallet.address, minAmount: 15900000000}]"
567567

568568
# key status:
569569
# index 0: active 64 ETH (32 ETH EB), 0x00 withdrawal creds
@@ -846,7 +846,7 @@ tasks:
846846
config:
847847
minWithdrawalCount: 1
848848
configVars:
849-
expectWithdrawals: "| [{publicKey: .tasks.all_pubkeys.outputs.pubkeys[8], address: .tasks.depositor_wallet.outputs.childWallet.address, minAmount: 31900000000000000000}]"
849+
expectWithdrawals: "| [{publicKey: .tasks.all_pubkeys.outputs.pubkeys[8], address: .tasks.depositor_wallet.outputs.childWallet.address, minAmount: 31900000000}]"
850850

851851
# generate EL triggered exit
852852
- name: run_tasks

playbooks/stable/validator-lifecycle-test-v2.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,7 @@ tasks:
143143
config:
144144
minWithdrawalCount: 1
145145
configVars:
146-
expectWithdrawals: "| [{publicKey: .validatorPubkeys[0], address: .depositorAddress, minAmount: 31000000000000000000}]"
146+
expectWithdrawals: "| [{publicKey: .validatorPubkeys[0], address: .depositorAddress, minAmount: 31000000000}]"
147147

148148
# slash 2 validators before exitability delay (1 with withdrawal address, 1 without)
149149
- name: run_tasks
@@ -313,7 +313,7 @@ tasks:
313313
config:
314314
minWithdrawalCount: 1
315315
configVars:
316-
expectWithdrawals: "| [{publicKey: .validatorPubkey, address: .depositorAddress, minAmount: 31000000000000000000}]"
316+
expectWithdrawals: "| [{publicKey: .validatorPubkey, address: .depositorAddress, minAmount: 31000000000}]"
317317

318318

319319
# status:
@@ -436,19 +436,19 @@ tasks:
436436
config:
437437
minWithdrawalCount: 1
438438
configVars:
439-
expectWithdrawals: "| [{publicKey: .validatorPubkeys[5], address: .depositorAddress, minAmount: 63500000000000000000}]"
439+
expectWithdrawals: "| [{publicKey: .validatorPubkeys[5], address: .depositorAddress, minAmount: 63500000000}]"
440440
- name: check_consensus_block_proposals
441441
title: "Wait for partial withdrawal for ${{ validatorPubkeys[8] }}"
442442
config:
443443
minWithdrawalCount: 1
444444
configVars:
445-
expectWithdrawals: "| [{publicKey: .validatorPubkeys[8], address: .depositorAddress, minAmount: 31500000000000000000}]"
445+
expectWithdrawals: "| [{publicKey: .validatorPubkeys[8], address: .depositorAddress, minAmount: 31500000000}]"
446446
- name: check_consensus_block_proposals
447447
title: "Wait for partial withdrawal for ${{ validatorPubkeys[9] }}"
448448
config:
449449
minWithdrawalCount: 1
450450
configVars:
451-
expectWithdrawals: "| [{publicKey: .validatorPubkeys[9], address: .depositorAddress, minAmount: 31500000000000000000}]"
451+
expectWithdrawals: "| [{publicKey: .validatorPubkeys[9], address: .depositorAddress, minAmount: 31500000000}]"
452452

453453

454454
# all keys should be extied now.

0 commit comments

Comments
 (0)