refactor(ATokenVaultMerklRewardClaimer): remove operator functionality

- the feature is not needed if rewards can only be claimed
  to the ATokenVault

Author: aa-eyup

src/ATokenVaultMerklRewardClaimer.sol

@@ -60,14 +60,6 @@ contract ATokenVaultMerklRewardClaimer is ATokenVault, IATokenVaultMerklRewardCl
         emit MerklDistributorUpdated(currentMerklDistributor, merklDistributor);
     }
 
-    /// @inheritdoc IATokenVaultMerklRewardClaimer
-    function toggleOperator(address operator) external override onlyOwner {
-        require(_s.merklDistributor != address(0), "MERKL_DISTRIBUTOR_NOT_SET");
-        require(operator != address(0), "ZERO_ADDRESS_NOT_VALID");
-        IMerklDistributor(_s.merklDistributor).toggleOperator(address(this), operator);
-        emit MerklRewardsOperatorToggled(_s.merklDistributor, operator);
-    }
-
     /// @inheritdoc IATokenVaultMerklRewardClaimer
     function getMerklDistributor() external view override returns (address) {
         return _s.merklDistributor;

src/interfaces/IATokenVaultMerklRewardClaimer.sol

@@ -20,13 +20,6 @@ interface IATokenVaultMerklRewardClaimer {
      */
     event MerklRewardsClaimed(address indexed distributor, address[] tokens, uint256[] amounts);
 
-    /**
-     * @dev Emitted when the operator status for the vault is toggled
-     * @param distributor Address of the Merkl distributor contract
-     * @param operator Address of the operator to toggle
-     */
-    event MerklRewardsOperatorToggled(address indexed distributor, address indexed operator);
-
     /**
      * @dev Emitted when the Merkl distributor address is updated
      * @param oldMerklDistributor The old address of the Merkl distributor contract
@@ -54,13 +47,6 @@ interface IATokenVaultMerklRewardClaimer {
      */
     function setMerklDistributor(address merklDistributor) external;
 
-    /**
-     * @notice Toggles the operator status for the vault
-     * @dev Only callable by the owner
-     * @param operator Address of the operator to toggle
-     */
-    function toggleOperator(address operator) external;
-
     /**
      * @notice Getter for the contract address called to claim Merkl rewards
      * @return Address of the Merkl distributor contract

test/ATokenVaultMerklRewardClaimer.t.sol

@@ -105,48 +105,6 @@ contract ATokenVaultMerklRewardClaimerTest is ATokenVaultBaseTest {
         vaultMerklRewardClaimer.claimMerklRewards(rewardTokens, amounts, proofs);
     }
 
-    function testToggleOperator() public {
-        _setMerklDistributor();
-        address operator = makeAddr("newOperator");
-        vm.prank(OWNER);
-        vaultMerklRewardClaimer.toggleOperator(operator);
-
-        assertEq(merklDistributor.getOperator(address(vaultMerklRewardClaimer), operator), true);
-
-        vm.prank(OWNER);
-        vaultMerklRewardClaimer.toggleOperator(operator);
-        assertEq(merklDistributor.getOperator(address(vaultMerklRewardClaimer), operator), false);
-    }
-
-    function testToggleOperatorEmitsEvent() public {
-        _setMerklDistributor();
-        address operator = makeAddr("newOperator");
-        vm.prank(OWNER);
-        vm.expectEmit(true, true, false, true, address(vaultMerklRewardClaimer));
-        emit IATokenVaultMerklRewardClaimer.MerklRewardsOperatorToggled(address(merklDistributor), operator);
-        vaultMerklRewardClaimer.toggleOperator(operator);
-    }
-
-    function testToggleOperatorRevertsIfOperatorIsZeroAddress() public {
-        _setMerklDistributor();
-        vm.prank(OWNER);
-        vm.expectRevert(bytes("ZERO_ADDRESS_NOT_VALID"));
-        vaultMerklRewardClaimer.toggleOperator(address(0));
-    }
-
-    function testToggleOperatorRevertsIfMerklDistributorNotSet() public {
-        address operator = makeAddr("operator");
-        vm.prank(OWNER);
-        vm.expectRevert(bytes("MERKL_DISTRIBUTOR_NOT_SET"));
-        vaultMerklRewardClaimer.toggleOperator(operator);
-    }
-
-    function testToggleOperatorRevertsIfNotOwner() public {
-        address operator = makeAddr("operator");
-        vm.expectRevert(bytes("Ownable: caller is not the owner"));
-        vaultMerklRewardClaimer.toggleOperator(operator);
-    }
-
     function testSetMerklDistributor() public {
         vm.prank(OWNER);
         vaultMerklRewardClaimer.setMerklDistributor(address(merklDistributor));