fix: Fix "outbound" spelling typo (#78)

Author: zguesmi

docs/soldoc/src/src/bridges/layerZero/IexecLayerZeroBridge.sol/contract.IexecLayerZeroBridge.md

@@ -148,7 +148,7 @@ function unpause() external onlyRole(PAUSER_ROLE);
 LEVEL 2: Pauses only outbound transfers.
 
 *Can only be called by accounts with PAUSER_ROLE
-When outbount transfers are paused:
+When outbound transfers are paused:
 - All _debit operations (outbound transfers) are blocked
 - All _credit operations (inbound transfers) still work
 - Users can still receive funds and "exit" their positions

docs/soldoc/src/src/bridges/utils/DualPausableUpgradeable.sol/abstract.DualPausableUpgradeable.md

@@ -38,24 +38,24 @@ function _getDualPausableStorage() private pure returns (DualPausableStorage sto
 
 ### whenOutboundTransfersNotPaused
 
-Use this modifier for functions that should be blocked during outbount transfer pause
+Use this modifier for functions that should be blocked during outbound transfer pause
 
-*Modifier for send operations - blocks when outbount transfer is paused*
+*Modifier for send operations - blocks when outbound transfer is paused*
 
 
 ```solidity
 modifier whenOutboundTransfersNotPaused();
 ```
 
-### whenOutbountTransfersPaused
+### whenOutboundTransfersPaused
 
-Use this modifier for administrative functions that should only work during outbount transfer pause
+Use this modifier for administrative functions that should only work during outbound transfer pause
 
 *Modifier to make a function callable only when send operations are paused*
 
 
 ```solidity
-modifier whenOutbountTransfersPaused();
+modifier whenOutboundTransfersPaused();
 ```
 
 ### __DualPausable_init
@@ -72,13 +72,13 @@ function __DualPausable_init() internal onlyInitializing;
 function __DualPausable_init_unchained() internal onlyInitializing;
 ```
 
-### outbountTransfersPaused
+### outboundTransfersPaused
 
 *Returns true if send operations are paused, false otherwise*
 
 
 ```solidity
-function outbountTransfersPaused() public view virtual returns (bool);
+function outboundTransfersPaused() public view virtual returns (bool);
 ```
 
 ### pauseStatus
@@ -94,30 +94,30 @@ function pauseStatus() public view virtual returns (bool fullyPaused, bool onlyO
 |Name|Type|Description|
 |----|----|-----------|
 |`fullyPaused`|`bool`|True if complete pause is active (blocks all operations)|
-|`onlyOutboundTransfersPaused`|`bool`|True if outbount transfer pause is active (blocks only send operations)|
+|`onlyOutboundTransfersPaused`|`bool`|True if outbound transfer pause is active (blocks only send operations)|
 
 
-### _requireOutbountTransfersNotPaused
+### _requireOutboundTransfersNotPaused
 
 *Throws if send operations are paused*
 
 
 ```solidity
-function _requireOutbountTransfersNotPaused() internal view virtual;
+function _requireOutboundTransfersNotPaused() internal view virtual;
 ```
 
-### _requireOutbountTransfersPaused
+### _requireOutboundTransfersPaused
 
 *Throws if send operations are not paused*
 
 
 ```solidity
-function _requireOutbountTransfersPaused() internal view virtual;
+function _requireOutboundTransfersPaused() internal view virtual;
 ```
 
 ### _pauseOutboundTransfers
 
-*Triggers outbount transfers pause.
+*Triggers outbound transfers pause.
 Requirements:
 - Send operations must not already be paused*
 
@@ -128,26 +128,26 @@ function _pauseOutboundTransfers() internal virtual whenOutboundTransfersNotPaus
 
 ### _unpauseOutboundTransfers
 
-*Unpauses outbount transfers.
+*Unpauses outbound transfers.
 Requirements:
 - Send operations must already be paused*
 
 
 ```solidity
-function _unpauseOutboundTransfers() internal virtual whenOutbountTransfersPaused;
+function _unpauseOutboundTransfers() internal virtual whenOutboundTransfersPaused;
 ```
 
 ## Events
 ### OutboundTransfersPaused
-*Emitted when outbount transfer pause is triggered by `account`*
+*Emitted when outbound transfer pause is triggered by `account`*
 
 
 ```solidity
 event OutboundTransfersPaused(address account);
 ```
 
 ### OutboundTransfersUnpaused
-*Emitted when outbount transfer pause is lifted by `account`*
+*Emitted when outbound transfer pause is lifted by `account`*
 
 
 ```solidity

src/bridges/layerZero/IexecLayerZeroBridge.sol

@@ -146,7 +146,7 @@ contract IexecLayerZeroBridge is
      * @notice LEVEL 2: Pauses only outbound transfers.
      * @dev Can only be called by accounts with PAUSER_ROLE
      *
-     * When outbount transfers are paused:
+     * When outbound transfers are paused:
      * - All _debit operations (outbound transfers) are blocked
      * - All _credit operations (inbound transfers) still work
      * - Users can still receive funds and "exit" their positions

src/bridges/utils/DualPausableUpgradeable.sol

@@ -42,12 +42,12 @@ abstract contract DualPausableUpgradeable is PausableUpgradeable {
     // ============ EVENTS ============
 
     /**
-     * @dev Emitted when outbount transfer pause is triggered by `account`
+     * @dev Emitted when outbound transfer pause is triggered by `account`
      */
     event OutboundTransfersPaused(address account);
 
     /**
-     * @dev Emitted when outbount transfer pause is lifted by `account`
+     * @dev Emitted when outbound transfer pause is lifted by `account`
      */
     event OutboundTransfersUnpaused(address account);
 
@@ -66,20 +66,20 @@ abstract contract DualPausableUpgradeable is PausableUpgradeable {
     // ============ MODIFIERS ============
 
     /**
-     * @dev Modifier for send operations - blocks when outbount transfer is paused
-     * @notice Use this modifier for functions that should be blocked during outbount transfer pause
+     * @dev Modifier for send operations - blocks when outbound transfer is paused
+     * @notice Use this modifier for functions that should be blocked during outbound transfer pause
      */
     modifier whenOutboundTransfersNotPaused() {
-        _requireOutbountTransfersNotPaused();
+        _requireOutboundTransfersNotPaused();
         _;
     }
 
     /**
      * @dev Modifier to make a function callable only when send operations are paused
-     * @notice Use this modifier for administrative functions that should only work during outbount transfer pause
+     * @notice Use this modifier for administrative functions that should only work during outbound transfer pause
      */
-    modifier whenOutbountTransfersPaused() {
-        _requireOutbountTransfersPaused();
+    modifier whenOutboundTransfersPaused() {
+        _requireOutboundTransfersPaused();
         _;
     }
 
@@ -101,43 +101,43 @@ abstract contract DualPausableUpgradeable is PausableUpgradeable {
     /**
      * @dev Returns true if send operations are paused, false otherwise
      */
-    function outbountTransfersPaused() public view virtual returns (bool) {
+    function outboundTransfersPaused() public view virtual returns (bool) {
         DualPausableStorage storage $ = _getDualPausableStorage();
         return $._outboundTransfersPaused;
     }
 
     /**
      * @dev Returns the overall operational state of the contract
      * @return fullyPaused True if complete pause is active (blocks all operations)
-     * @return onlyOutboundTransfersPaused True if outbount transfer pause is active (blocks only send operations)
+     * @return onlyOutboundTransfersPaused True if outbound transfer pause is active (blocks only send operations)
      */
     function pauseStatus() public view virtual returns (bool fullyPaused, bool onlyOutboundTransfersPaused) {
         fullyPaused = paused();
-        onlyOutboundTransfersPaused = outbountTransfersPaused();
+        onlyOutboundTransfersPaused = outboundTransfersPaused();
     }
 
     // ============ INTERNAL FUNCTIONS ============
 
     /**
      * @dev Throws if send operations are paused
      */
-    function _requireOutbountTransfersNotPaused() internal view virtual {
-        if (outbountTransfersPaused()) {
+    function _requireOutboundTransfersNotPaused() internal view virtual {
+        if (outboundTransfersPaused()) {
             revert EnforcedOutboundTransfersPause();
         }
     }
 
     /**
      * @dev Throws if send operations are not paused
      */
-    function _requireOutbountTransfersPaused() internal view virtual {
-        if (!outbountTransfersPaused()) {
+    function _requireOutboundTransfersPaused() internal view virtual {
+        if (!outboundTransfersPaused()) {
             revert ExpectedOutboundTransfersPause();
         }
     }
 
     /**
-     * @dev Triggers outbount transfers pause.
+     * @dev Triggers outbound transfers pause.
      * Requirements:
      * - Send operations must not already be paused
      */
@@ -148,11 +148,11 @@ abstract contract DualPausableUpgradeable is PausableUpgradeable {
     }
 
     /**
-     * @dev Unpauses outbount transfers.
+     * @dev Unpauses outbound transfers.
      * Requirements:
      * - Send operations must already be paused
      */
-    function _unpauseOutboundTransfers() internal virtual whenOutbountTransfersPaused {
+    function _unpauseOutboundTransfers() internal virtual whenOutboundTransfersPaused {
         DualPausableStorage storage $ = _getDualPausableStorage();
         $._outboundTransfersPaused = false;
         emit OutboundTransfersUnpaused(_msgSender());

test/units/bridges/layerZero/IexecLayerZeroBridge.t.sol

@@ -223,7 +223,7 @@ contract IexecLayerZeroBridgeTest is TestHelperOz5 {
         assertEq(rlcCrosschainToken.balanceOf(user1), INITIAL_BALANCE - TRANSFER_AMOUNT);
     }
 
-    // ============ LEVEL 2 PAUSE TESTS (Outbount transfer pause) ============
+    // ============ LEVEL 2 PAUSE TESTS (Outbound transfer pause) ============
 
     function test_PauseOutboundTransfers_OnlyPauserRole() public {
         vm.expectRevert();
@@ -238,7 +238,7 @@ contract IexecLayerZeroBridgeTest is TestHelperOz5 {
 
         // Verify state
         assertFalse(iexecLayerZeroBridgeChainX.paused());
-        assertTrue(iexecLayerZeroBridgeChainX.outbountTransfersPaused());
+        assertTrue(iexecLayerZeroBridgeChainX.outboundTransfersPaused());
 
         // Prepare send parameters
         (SendParam memory sendParam, MessagingFee memory fee) =
@@ -264,7 +264,7 @@ contract IexecLayerZeroBridgeTest is TestHelperOz5 {
 
         // Should now work normally
         assertFalse(iexecLayerZeroBridgeChainX.paused());
-        assertFalse(iexecLayerZeroBridgeChainX.outbountTransfersPaused());
+        assertFalse(iexecLayerZeroBridgeChainX.outboundTransfersPaused());
 
         test_SendToken_WhenOperational_WithoutApproval();
     }

test/units/bridges/utils/DualPausableUpgradeable.t.sol

@@ -30,7 +30,7 @@ contract DualPausableUpgradeableTest is Test {
     // ============ initialize ============
 
     function test_initialize_SetsCorrectInitialStates() public view {
-        assertFalse(dualPausable.outbountTransfersPaused(), "Outbound transfers should not be paused initially");
+        assertFalse(dualPausable.outboundTransfersPaused(), "Outbound transfers should not be paused initially");
 
         (bool fullyPaused, bool onlyOutboundTransfersPaused) = dualPausable.pauseStatus();
         assertFalse(fullyPaused, "Contract should not be fully paused initially");
@@ -51,7 +51,7 @@ contract DualPausableUpgradeableTest is Test {
         dualPausable.pauseOutboundTransfers();
 
         assertTrue(
-            dualPausable.outbountTransfersPaused(), "Outbound transfers should be paused after pauseOutboundTransfers"
+            dualPausable.outboundTransfersPaused(), "Outbound transfers should be paused after pauseOutboundTransfers"
         );
 
         vm.expectRevert(DualPausableUpgradeable.EnforcedOutboundTransfersPause.selector);
@@ -82,18 +82,18 @@ contract DualPausableUpgradeableTest is Test {
     function test_unpauseOutboundTransfers_RestoresOperations() public {
         vm.startPrank(OWNER);
         dualPausable.pauseOutboundTransfers();
-        assertTrue(dualPausable.outbountTransfersPaused(), "Outbound transfers should be paused before unpause");
+        assertTrue(dualPausable.outboundTransfersPaused(), "Outbound transfers should be paused before unpause");
 
         dualPausable.unpauseOutboundTransfers();
         vm.stopPrank();
 
-        assertFalse(dualPausable.outbountTransfersPaused(), "Outbound transfers should not be paused after unpause");
+        assertFalse(dualPausable.outboundTransfersPaused(), "Outbound transfers should not be paused after unpause");
 
         // Operation should work normally
         assertTrue(dualPausable.mockOperation(), "Mock operation should succeed after unpause");
     }
 
-    function test_RevertWhen_OutbountTransfersNotPaused() public {
+    function test_RevertWhen_OutboundTransfersNotPaused() public {
         vm.expectRevert(DualPausableUpgradeable.ExpectedOutboundTransfersPause.selector);
         dualPausable.unpauseOutboundTransfers();
     }
@@ -125,17 +125,17 @@ contract DualPausableUpgradeableTest is Test {
         assertFalse(fullyPaused, "Contract should not be fully paused initially");
         assertFalse(onlyOutboundTransfersPaused, "Outbound transfers should not be paused initially");
 
-        // After outbount transfer pause
+        // After outbound transfer pause
         vm.prank(OWNER);
         dualPausable.pauseOutboundTransfers();
 
         (fullyPaused, onlyOutboundTransfersPaused) = dualPausable.pauseStatus();
-        assertFalse(fullyPaused, "Contract should not be fully paused during outbount transfer pause");
+        assertFalse(fullyPaused, "Contract should not be fully paused during outbound transfer pause");
         assertTrue(
-            onlyOutboundTransfersPaused, "Outbound transfers should be paused during outbount transfers pause state"
+            onlyOutboundTransfersPaused, "Outbound transfers should be paused during outbound transfers pause state"
         );
 
-        // After full pause (from outbount transfer only pause state)
+        // After full pause (from outbound transfer only pause state)
         vm.prank(OWNER);
         dualPausable.pause();
 
@@ -148,17 +148,17 @@ contract DualPausableUpgradeableTest is Test {
 
     // Make sure `pause()` does not impact `pauseOutboundTransfers()`.
     function test_DualPause_PauseFromOnlyOutboundTransfersToFull() public {
-        // Start with outbount transfer pause
+        // Start with outbound transfer pause
         vm.startPrank(OWNER);
         dualPausable.pauseOutboundTransfers();
-        assertTrue(dualPausable.outbountTransfersPaused());
+        assertTrue(dualPausable.outboundTransfersPaused());
         assertFalse(dualPausable.paused());
 
         dualPausable.pause();
         vm.stopPrank();
 
         assertTrue(dualPausable.paused());
-        assertTrue(dualPausable.outbountTransfersPaused());
+        assertTrue(dualPausable.outboundTransfersPaused());
     }
 
     // Make sure `pauseOutboundTransfers()` does not impact `pause()`.
@@ -167,13 +167,13 @@ contract DualPausableUpgradeableTest is Test {
         vm.startPrank(OWNER);
         dualPausable.pause();
         assertTrue(dualPausable.paused());
-        assertFalse(dualPausable.outbountTransfersPaused());
+        assertFalse(dualPausable.outboundTransfersPaused());
         // pauseOutboundTransfers
         dualPausable.pauseOutboundTransfers();
         vm.stopPrank();
         // Check status
         assertTrue(dualPausable.paused());
-        assertTrue(dualPausable.outbountTransfersPaused());
+        assertTrue(dualPausable.outboundTransfersPaused());
     }
 
     function test_DualPause_PauseShouldNotImpactOutboundTransfersPause() public {
@@ -182,12 +182,12 @@ contract DualPausableUpgradeableTest is Test {
         dualPausable.pause();
         dualPausable.pauseOutboundTransfers();
         assertTrue(dualPausable.paused());
-        assertTrue(dualPausable.outbountTransfersPaused());
+        assertTrue(dualPausable.outboundTransfersPaused());
         // unpause
         dualPausable.unpause();
         assertFalse(dualPausable.paused());
         // pauseOutboundTransfers should still be active
-        assertTrue(dualPausable.outbountTransfersPaused(), "Outbound transfers should remain paused after unpause");
+        assertTrue(dualPausable.outboundTransfersPaused(), "Outbound transfers should remain paused after unpause");
         vm.stopPrank();
     }
 
@@ -197,10 +197,10 @@ contract DualPausableUpgradeableTest is Test {
         dualPausable.pause();
         dualPausable.pauseOutboundTransfers();
         assertTrue(dualPausable.paused());
-        assertTrue(dualPausable.outbountTransfersPaused());
+        assertTrue(dualPausable.outboundTransfersPaused());
         // unpauseOutboundTransfers
         dualPausable.unpauseOutboundTransfers();
-        assertFalse(dualPausable.outbountTransfersPaused());
+        assertFalse(dualPausable.outboundTransfersPaused());
         // pause should still be active
         assertTrue(dualPausable.paused(), "Pause should remain active after unpauseOutboundTransfers");
         vm.stopPrank();