Skip to content

Commit c59c186

Browse files
committed
fix: new event for minimum delegation not met (TRST-M03)
1 parent 43bc72b commit c59c186

File tree

3 files changed

+9
-2
lines changed

3 files changed

+9
-2
lines changed

packages/horizon/contracts/interfaces/internal/IHorizonStakingMain.sol

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -423,6 +423,13 @@ interface IHorizonStakingMain {
423423
*/
424424
error HorizonStakingInvalidDelegationPool(address serviceProvider, address verifier);
425425

426+
/**
427+
* @notice Thrown when the minimum token amount required for delegation is not met.
428+
* @param tokens The actual token amount
429+
* @param minTokens The minimum required token amount
430+
*/
431+
error HorizonStakingInsufficientDelegationTokens(uint256 tokens, uint256 minTokens);
432+
426433
/**
427434
* @notice Thrown when attempting to undelegate with a beneficiary that is the zero address.
428435
*/

packages/horizon/contracts/staking/HorizonStaking.sol

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -869,7 +869,7 @@ contract HorizonStaking is HorizonStakingBase, IHorizonStakingMain {
869869
function _delegate(address _serviceProvider, address _verifier, uint256 _tokens, uint256 _minSharesOut) private {
870870
// Enforces a minimum delegation amount to prevent share manipulation attacks.
871871
// This stops attackers from inflating share value and blocking other delegators.
872-
require(_tokens >= MIN_DELEGATION, HorizonStakingInsufficientTokens(_tokens, MIN_DELEGATION));
872+
require(_tokens >= MIN_DELEGATION, HorizonStakingInsufficientDelegationTokens(_tokens, MIN_DELEGATION));
873873
require(
874874
_provisions[_serviceProvider][_verifier].createdAt != 0,
875875
HorizonStakingInvalidProvision(_serviceProvider, _verifier)

packages/horizon/test/staking/delegation/delegate.t.sol

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ contract HorizonStakingDelegateTest is HorizonStakingTest {
6767
vm.startPrank(users.delegator);
6868
token.approve(address(staking), delegationAmount);
6969
bytes memory expectedError = abi.encodeWithSelector(
70-
IHorizonStakingMain.HorizonStakingInsufficientTokens.selector,
70+
IHorizonStakingMain.HorizonStakingInsufficientDelegationTokens.selector,
7171
delegationAmount,
7272
MIN_DELEGATION
7373
);

0 commit comments

Comments
 (0)