adds fix for ci coverage step

smartcontractkit · Jan 16, 2024 · 2ecad87 · 2ecad87
1 parent f40ddbb
commit 2ecad87
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 11 deletions.
diff --git a/contracts/src/v0.8/l2ep/dev/CrossDomainGovernor.sol b/contracts/src/v0.8/l2ep/dev/CrossDomainGovernor.sol
@@ -38,5 +38,12 @@ abstract contract CrossDomainGovernor is
   }
 
   /// @notice The call MUST come from either the L1 owner (via cross-chain message) or the L2 owner. Reverts otherwise.
-  modifier onlyLocalOrCrossDomainOwner() virtual;
+  modifier onlyLocalOrCrossDomainOwner() virtual {
+    // solhint-disable-next-line custom-errors
+    require(
+      msg.sender == this.crossDomainMessenger() || msg.sender == owner(),
+      "Sender is not the L2 messenger or owner"
+    );
+    _;
+  }
 }
diff --git a/contracts/src/v0.8/l2ep/dev/arbitrum/ArbitrumCrossDomainGovernor.sol b/contracts/src/v0.8/l2ep/dev/arbitrum/ArbitrumCrossDomainGovernor.sol
@@ -23,16 +23,6 @@ contract ArbitrumCrossDomainGovernor is CrossDomainGovernor {
     return AddressAliasHelper.applyL1ToL2Alias(l1Owner());
   }
 
-  /// @notice The call MUST come from either the L1 owner (via cross-chain message) or the L2 owner. Reverts otherwise.
-  modifier onlyLocalOrCrossDomainOwner() override {
-    // solhint-disable-next-line custom-errors
-    require(
-      msg.sender == this.crossDomainMessenger() || msg.sender == owner(),
-      "Sender is not the L2 messenger or owner"
-    );
-    _;
-  }
-
   /// @notice The call MUST come from the L1 owner (via cross-chain message.) Reverts otherwise.
   modifier onlyL1Owner() override {
     // solhint-disable-next-line custom-errors