fix: remove unused value from event (TRST-R01)

graphprotocol · Dec 4, 2024 · 9271b99 · 9271b99
1 parent c9f8a2f
commit 9271b99
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/packages/horizon/contracts/interfaces/internal/IHorizonStakingMain.sol b/packages/horizon/contracts/interfaces/internal/IHorizonStakingMain.sol
@@ -303,9 +303,8 @@ interface IHorizonStakingMain {
 
     /**
      * @notice Emitted when the delegation slashing global flag is set.
-     * @param enabled Whether delegation slashing is enabled or disabled.
      */
-    event DelegationSlashingEnabled(bool enabled);
+    event DelegationSlashingEnabled();
 
     // -- Errors: tokens
 

diff --git a/packages/horizon/contracts/staking/HorizonStaking.sol b/packages/horizon/contracts/staking/HorizonStaking.sol
@@ -585,7 +585,7 @@ contract HorizonStaking is HorizonStakingBase, IHorizonStakingMain {
      */
     function setDelegationSlashingEnabled() external override onlyGovernor {
         _delegationSlashingEnabled = true;
-        emit DelegationSlashingEnabled(_delegationSlashingEnabled);
+        emit DelegationSlashingEnabled();
     }
 
     /**

diff --git a/packages/horizon/test/shared/horizon-staking/HorizonStakingShared.t.sol b/packages/horizon/test/shared/horizon-staking/HorizonStakingShared.t.sol
@@ -1367,7 +1367,7 @@ abstract contract HorizonStakingSharedTest is GraphBaseTest {
     function _setDelegationSlashingEnabled() internal {
         // setDelegationSlashingEnabled
         vm.expectEmit();
-        emit IHorizonStakingMain.DelegationSlashingEnabled(true);
+        emit IHorizonStakingMain.DelegationSlashingEnabled();
         staking.setDelegationSlashingEnabled();
 
         // after