Fix metadata uri signature (#205)

* updating for compliance with interface * fix: updatng comment to reflect function rename
Layr-Labs · Mar 11, 2024 · 036e9d4 · 036e9d4
1 parent 9236bb7
commit 036e9d4
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/src/ServiceManagerBase.sol b/src/ServiceManagerBase.sol
@@ -49,11 +49,11 @@ abstract contract ServiceManagerBase is IServiceManager, OwnableUpgradeable {
     }
 
     /**
-     * @notice Sets the metadata URI for the AVS
+     * @notice Updates the metadata URI for the AVS
      * @param _metadataURI is the metadata URI for the AVS
      * @dev only callable by the owner
      */
-    function setMetadataURI(string memory _metadataURI) public virtual onlyOwner {
+    function updateAVSMetadataURI(string memory _metadataURI) public virtual onlyOwner {
         _avsDirectory.updateAVSMetadataURI(_metadataURI);
     }
 

diff --git a/src/interfaces/IServiceManager.sol b/src/interfaces/IServiceManager.sol
@@ -10,10 +10,10 @@ import {IDelegationManager} from "eigenlayer-contracts/src/contracts/interfaces/
  */
 interface IServiceManager {
     /**
-     * @notice Sets the metadata URI for the AVS
+     * @notice Updates the metadata URI for the AVS
      * @param _metadataURI is the metadata URI for the AVS
      */
-    function setMetadataURI(string memory _metadataURI) external;
+    function updateAVSMetadataURI(string memory _metadataURI) external;
 
     /**
      * @notice Forwards a call to EigenLayer's DelegationManager contract to confirm operator registration with the AVS

diff --git a/test/integration/CoreRegistration.t.sol b/test/integration/CoreRegistration.t.sol
@@ -170,7 +170,7 @@ contract Test_CoreRegistration is MockAVSDeployer {
         require(operator != serviceManager.owner(), "bad test setup");
         cheats.prank(operator);
         cheats.expectRevert("Ownable: caller is not the owner");
-        serviceManager.setMetadataURI("Test MetadataURI");
+        serviceManager.updateAVSMetadataURI("Test MetadataURI");
     }
 
     event AVSMetadataURIUpdated(address indexed avs, string metadataURI);
@@ -180,7 +180,7 @@ contract Test_CoreRegistration is MockAVSDeployer {
         cheats.prank(toPrankFrom);
         cheats.expectEmit(true, true, true, true);
         emit AVSMetadataURIUpdated(address(serviceManager), "Test MetadataURI");
-        serviceManager.setMetadataURI("Test MetadataURI");
+        serviceManager.updateAVSMetadataURI("Test MetadataURI");
     }
 
     // Utils