Skip to content

Commit ce9b3e3

Browse files
SidestreamSweatyPumpkinyondonfu
authored andcommitted
fix: enhance delegator fee validation to assert significant fee loss
1 parent cf383a8 commit ce9b3e3

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

src/test/BondingManagerRetroactiveRewardCalculationPoC.sol

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -142,8 +142,11 @@ contract BondingManagerRetroactiveRewardCalculationPoC is GovernorBaseTest {
142142
}
143143

144144
function _validateDelegatorFees(uint256 consecutive, uint256 missed) internal virtual {
145-
// Assert delegator fee loss exists
146-
assertNotEq(consecutive, missed, "Delegator fee loss NOT detected");
145+
// Calculate the absolute difference
146+
uint256 diff = consecutive > missed ? consecutive - missed : missed - consecutive;
147+
148+
// Assert that the fee loss is greater than 1e1 wei
149+
assertGt(diff, 1e1, "Delegator fee loss is negligible (<= 10 wei)");
147150
}
148151

149152
function testCompareConsecutiveAndMissedRewardClaims() public virtual {

0 commit comments

Comments
 (0)