Merge branch 'dev' into jaxxjj/ecdsa-stake-registry

Author: jaxxjj

test/events/IStakeRegistryEvents.sol

@@ -18,4 +18,6 @@ interface IStakeRegistryEvents {
     event StrategyMultiplierUpdated(
         uint8 indexed quorumNumber, IStrategy strategy, uint256 multiplier
     );
+    /// @notice Emitted when the look ahead period for checking operator shares is updated.
+    event LookAheadPeriodChanged(uint32 oldLookAheadBlocks, uint32 newLookAheadBlocks);
 }

test/unit/IndexRegistryUnit.t.sol

@@ -388,6 +388,42 @@ contract IndexRegistryUnitTests_configAndGetters is IndexRegistryUnitTests {
         cheats.expectRevert();
         indexRegistry.getOperatorListAtBlockNumber(quorumNumber, 0);
     }
+
+    function test_getQuorumUpdateAtIndex() public {
+        uint8 quorumNumber = nextQuorum;
+        _initializeQuorum();
+
+        uint32 blockNum = uint32(block.number);
+
+        // Roll forward to force update
+        vm.roll(block.number + 1);
+
+        // Check quorum update at index 0
+        IIndexRegistry.QuorumUpdate memory quorumUpdate =
+            indexRegistry.getQuorumUpdateAtIndex(quorumNumber, 0);
+        assertEq(quorumUpdate.numOperators, 0, "numOperators not 0");
+        assertEq(quorumUpdate.fromBlockNumber, blockNum, "fromBlockNumber not correct");
+
+        // Register operator
+        (, bytes32 operatorId) = _selectNewOperator();
+        _registerOperatorSingleQuorum(operatorId, quorumNumber);
+
+        // Check quorum update at index 1
+        quorumUpdate = indexRegistry.getQuorumUpdateAtIndex(quorumNumber, 1);
+        assertEq(quorumUpdate.numOperators, 1, "numOperators not 1");
+        assertEq(quorumUpdate.fromBlockNumber, block.number, "fromBlockNumber not correct");
+
+        // Roll forward to force update
+        vm.roll(block.number + 1);
+
+        // Deregister
+        _deregisterOperatorSingleQuorum(operatorId, quorumNumber);
+
+        // Check quorum update at index 2
+        quorumUpdate = indexRegistry.getQuorumUpdateAtIndex(quorumNumber, 2);
+        assertEq(quorumUpdate.numOperators, 0, "numOperators not 0");
+        assertEq(quorumUpdate.fromBlockNumber, block.number, "fromBlockNumber not correct");
+    }
 }
 
 contract IndexRegistryUnitTests_registerOperator is IndexRegistryUnitTests {

test/unit/ServiceManagerBase.t.sol

@@ -5,6 +5,7 @@ import "@openzeppelin/contracts/token/ERC20/presets/ERC20PresetFixedSupply.sol";
 import {
     RewardsCoordinator,
     IRewardsCoordinator,
+    IRewardsCoordinatorEvents,
     IRewardsCoordinatorTypes,
     IERC20
 } from "eigenlayer-contracts/src/contracts/core/RewardsCoordinator.sol";
@@ -928,4 +929,39 @@ contract ServiceManagerBase_UnitTests is MockAVSDeployer, IServiceManagerBaseEve
         cheats.expectRevert(IServiceManagerErrors.OnlyRegistryCoordinator.selector);
         serviceManager.deregisterOperatorFromOperatorSets(operator, operatorSetIds);
     }
+
+    function testFuzz_setClaimerFor_revert_notOwner(
+        address claimer,
+        address caller
+    ) public filterFuzzedAddressInputs(claimer) filterFuzzedAddressInputs(caller) {
+        cheats.assume(caller != serviceManagerOwner);
+
+        cheats.expectRevert("Ownable: caller is not the owner");
+        cheats.prank(caller);
+        serviceManager.setClaimerFor(claimer);
+    }
+
+    function testFuzz_setClaimerFor_upateClaimer(
+        address claimer
+    ) public filterFuzzedAddressInputs(claimer) {
+        cheats.prank(serviceManagerOwner);
+        serviceManager.setClaimerFor(claimer);
+        assertEq(
+            claimer, rewardsCoordinator.claimerFor(address(serviceManager)), "claimer not updated"
+        );
+    }
+
+    function testFuzz_setClaimerFor_emitEvent(
+        address claimer
+    ) public filterFuzzedAddressInputs(claimer) {
+        // Retrieve previous claimer.
+        address prevClaimer = rewardsCoordinator.claimerFor(address(serviceManager));
+
+        // Expect an event on the RewardCoordinator.
+        cheats.expectEmit(true, true, true, true);
+        emit IRewardsCoordinatorEvents.ClaimerForSet(address(serviceManager), prevClaimer, claimer);
+
+        cheats.prank(serviceManagerOwner);
+        serviceManager.setClaimerFor(claimer);
+    }
 }