Skip to content

Commit 7283aa4

Browse files
committed
fix case sensitive address comparison for expected withdrawals in check_consensus_block_proposals task
1 parent 1aea1c4 commit 7283aa4

File tree

1 file changed

+1
-1
lines changed
  • pkg/coordinator/tasks/check_consensus_block_proposals

1 file changed

+1
-1
lines changed

pkg/coordinator/tasks/check_consensus_block_proposals/task.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -573,7 +573,7 @@ func (t *Task) checkBlockWithdrawals(block *consensus.Block, blockData *spec.Ver
573573
withdrawalAmount = withdrawalAmount.Mul(withdrawalAmount, big.NewInt(1000000000))
574574

575575
switch {
576-
case expectedWithdrawal.Address != "" && expectedWithdrawal.Address != withdrawal.Address.String():
576+
case expectedWithdrawal.Address != "" && !strings.EqualFold(expectedWithdrawal.Address, withdrawal.Address.String()):
577577
t.logger.Warnf("check failed: withdrawal found, but execution address does not match (have: %v, want: %v)", withdrawal.Address.String(), expectedWithdrawal.Address)
578578
case expectedWithdrawal.MinAmount.Cmp(big.NewInt(0)) > 0 && expectedWithdrawal.MinAmount.Cmp(withdrawalAmount) > 0:
579579
t.logger.Warnf("check failed: withdrawal found, but amount lower than minimum (have: %v, want >= %v)", withdrawalAmount, expectedWithdrawal.MinAmount)

0 commit comments

Comments
 (0)