fix: creation of registry coordinator

Author: stevennevins

test/harnesses/RegistryCoordinatorHarness.t.sol

@@ -12,8 +12,9 @@ contract RegistryCoordinatorHarness is RegistryCoordinator, Test {
         IStakeRegistry _stakeRegistry,
         IBLSApkRegistry _blsApkRegistry,
         IIndexRegistry _indexRegistry,
-        IAVSDirectory _avsDirectory
-    ) RegistryCoordinator(_serviceManager, _stakeRegistry, _blsApkRegistry, _indexRegistry, _avsDirectory, pauserRegistry) {
+        IAVSDirectory _avsDirectory,
+        IPauserRegistry _pauserRegistry
+    ) RegistryCoordinator(_serviceManager, _stakeRegistry, _blsApkRegistry, _indexRegistry, _avsDirectory, _pauserRegistry) {
         _transferOwnership(msg.sender);
     }
 

test/integration/CoreRegistration.t.sol

@@ -82,7 +82,8 @@ contract Test_CoreRegistration is MockAVSDeployer {
             stakeRegistry,
             blsApkRegistry,
             indexRegistry,
-            avsDirectory
+            avsDirectory,
+            pauserRegistry
         );
 
         // Upgrade Registry Coordinator & ServiceManager

test/mocks/DelegationMock.sol

@@ -2,6 +2,7 @@
 pragma solidity ^0.8.12;
 
 import {IERC20} from "@openzeppelin/contracts/token/ERC20/IERC20.sol";
+import {console2 as console} from "forge-std/Test.sol";
 
 import {IDelegationManager} from "eigenlayer-contracts/src/contracts/interfaces/IDelegationManager.sol";
 import {IStrategyManager} from "eigenlayer-contracts/src/contracts/interfaces/IStrategyManager.sol";
@@ -174,9 +175,13 @@ contract DelegationMock is IDelegationManager {
         address operator,
         IStrategy strategy,
         uint256 shares
-    ) external {}
+    ) external {
+        console.log("HERE");
+    }
 
-    function setIsOperator(address, bool) external {}
+    function setIsOperator(address, bool) external {
+        console.log("HERE");
+    }
 
     function minWithdrawalDelayBlocks() external returns (uint32) {}
 }

test/unit/StakeRegistryUnit.t.sol

@@ -49,7 +49,8 @@ contract StakeRegistryUnitTests is MockAVSDeployer, IStakeRegistryEvents {
             stakeRegistry,
             IBLSApkRegistry(blsApkRegistry),
             IIndexRegistry(indexRegistry),
-            IAVSDirectory(avsDirectory)
+            IAVSDirectory(avsDirectory),
+            pauserRegistry
         );
 
         stakeRegistryImplementation = new StakeRegistryHarness(

test/utils/MockAVSDeployer.sol

@@ -135,24 +135,43 @@ contract MockAVSDeployer is Test {
 
     function _deployMockEigenLayerAndAVS(uint8 numQuorumsToAdd) internal {
         emptyContract = new EmptyContract();
+        vm.label(address(emptyContract), "EmptyContract");
 
         defaultOperatorId = defaultPubKey.hashG1Point();
 
         cheats.startPrank(proxyAdminOwner);
         proxyAdmin = new ProxyAdmin();
+        vm.label(address(proxyAdmin), "ProxyAdmin");
 
         address[] memory pausers = new address[](1);
         pausers[0] = pauser;
         pauserRegistry = new PauserRegistry(pausers, unpauser);
+        vm.label(address(pauserRegistry), "PauserRegistry");
 
         delegationMock = new DelegationMock();
+        vm.label(address(delegationMock), "DelegationMock");
+
         avsDirectoryMock = new AVSDirectoryMock();
+        vm.label(address(avsDirectoryMock), "AVSDirectoryMock");
+
         eigenPodManagerMock = new EigenPodManagerMock(pauserRegistry);
+        vm.label(address(eigenPodManagerMock), "EigenPodManagerMock");
+
         strategyManagerMock = new StrategyManagerMock(delegationMock);
+        vm.label(address(strategyManagerMock), "StrategyManagerMock");
+
         allocationManagerMock = new AllocationManagerMock();
+        vm.label(address(allocationManagerMock), "AllocationManagerMock");
+
         avsDirectoryMock = new AVSDirectoryMock();
+        vm.label(address(avsDirectoryMock), "AVSDirectoryMock");
+
         allocationManagerMock = new AllocationManagerMock();
+        vm.label(address(allocationManagerMock), "AllocationManagerMock");
+
         avsDirectoryImplementation = new AVSDirectory(delegationMock, pauserRegistry); // TODO: config value
+        vm.label(address(avsDirectoryImplementation), "AVSDirectoryImplementation");
+
         avsDirectory = AVSDirectory(
             address(
                 new TransparentUpgradeableProxy(
@@ -167,7 +186,10 @@ contract MockAVSDeployer is Test {
                 )
             )
         );
+        vm.label(address(avsDirectory), "AVSDirectory");
+
         rewardsCoordinatorMock = new RewardsCoordinatorMock();
+        vm.label(address(rewardsCoordinatorMock), "RewardsCoordinatorMock");
 
         strategyManagerMock.setDelegationManager(delegationMock);
         cheats.stopPrank();
@@ -178,57 +200,66 @@ contract MockAVSDeployer is Test {
                 new TransparentUpgradeableProxy(address(emptyContract), address(proxyAdmin), "")
             )
         );
+        vm.label(address(registryCoordinator), "RegistryCoordinator");
 
         stakeRegistry = StakeRegistryHarness(
             address(
                 new TransparentUpgradeableProxy(address(emptyContract), address(proxyAdmin), "")
             )
         );
+        vm.label(address(stakeRegistry), "StakeRegistry");
 
         indexRegistry = IndexRegistry(
             address(
                 new TransparentUpgradeableProxy(address(emptyContract), address(proxyAdmin), "")
             )
         );
+        vm.label(address(indexRegistry), "IndexRegistry");
 
         blsApkRegistry = BLSApkRegistryHarness(
             address(
                 new TransparentUpgradeableProxy(address(emptyContract), address(proxyAdmin), "")
             )
         );
+        vm.label(address(blsApkRegistry), "BLSApkRegistry");
 
         serviceManager = ServiceManagerMock(
             address(
                 new TransparentUpgradeableProxy(address(emptyContract), address(proxyAdmin), "")
             )
         );
+        vm.label(address(serviceManager), "ServiceManager");
 
         allocationManager = AllocationManagerMock(
             address(
                 new TransparentUpgradeableProxy(address(emptyContract), address(proxyAdmin), "")
             )
         );
+        vm.label(address(allocationManager), "AllocationManager");
 
         cheats.stopPrank();
 
         cheats.startPrank(proxyAdminOwner);
 
         stakeRegistryImplementation =
             new StakeRegistryHarness(IRegistryCoordinator(registryCoordinator), delegationMock, avsDirectory, serviceManager);
+        vm.label(address(stakeRegistryImplementation), "StakeRegistryImplementation");
 
         proxyAdmin.upgrade(
             TransparentUpgradeableProxy(payable(address(stakeRegistry))),
             address(stakeRegistryImplementation)
         );
 
         blsApkRegistryImplementation = new BLSApkRegistryHarness(registryCoordinator);
+        vm.label(address(blsApkRegistryImplementation), "BLSApkRegistryImplementation");
 
         proxyAdmin.upgrade(
             TransparentUpgradeableProxy(payable(address(blsApkRegistry))),
             address(blsApkRegistryImplementation)
         );
 
         indexRegistryImplementation = new IndexRegistry(registryCoordinator);
+        vm.label(address(indexRegistryImplementation), "IndexRegistryImplementation");
 
         proxyAdmin.upgrade(
             TransparentUpgradeableProxy(payable(address(indexRegistry))),
@@ -242,13 +273,15 @@ contract MockAVSDeployer is Test {
             stakeRegistry,
             allocationManager
         );
+        vm.label(address(serviceManagerImplementation), "ServiceManagerImplementation");
 
         proxyAdmin.upgrade(
             TransparentUpgradeableProxy(payable(address(serviceManager))),
             address(serviceManagerImplementation)
         );
 
         allocationManagerImplementation = new AllocationManagerMock();
+        vm.label(address(allocationManagerImplementation), "AllocationManagerImplementation");
 
         proxyAdmin.upgrade(
             TransparentUpgradeableProxy(payable(address(allocationManager))),
@@ -281,7 +314,7 @@ contract MockAVSDeployer is Test {
         }
 
         registryCoordinatorImplementation = new RegistryCoordinatorHarness(
-            serviceManager, stakeRegistry, blsApkRegistry, indexRegistry, avsDirectory
+            serviceManager, stakeRegistry, blsApkRegistry, indexRegistry, avsDirectory, pauserRegistry
         );
         {
             delete operatorSetParams;