fix: resolve comments and fix e2e tests

Author: Zygimantass

contracts/interfaces/IIBCPausable.sol

@@ -2,10 +2,10 @@
 pragma solidity ^0.8.28;
 
 interface IIBCPausable {
-    /// @notice The role identifier for the unpauser role
+    /// @notice The role identifier for the pauser role
     function PAUSER_ROLE() external view returns (bytes32);
 
-    /// @notice The role identifier for the ununpauser role
+    /// @notice The role identifier for the unpauser role
     function UNPAUSER_ROLE() external view returns (bytes32);
 
     /// @notice Pauses the contract
@@ -16,12 +16,12 @@ interface IIBCPausable {
     /// @dev The caller must have the unpauser role
     function unpause() external;
 
-    /// @notice Grants the unpauser role to an account
+    /// @notice Grants the pauser role to an account
     /// @dev The caller must be authorized by the derived contract
     /// @param account The account to grant the role to
     function grantPauserRole(address account) external;
 
-    /// @notice Revokes the unpauser role from an account
+    /// @notice Revokes the pauser role from an account
     /// @dev The caller must be authorized by the derived contract
     /// @param account The account to revoke the role from
     function revokePauserRole(address account) external;

contracts/utils/IBCPausableUpgradeable.sol

@@ -16,10 +16,12 @@ abstract contract IBCPausableUpgradeable is
 {
     /// @inheritdoc IIBCPausable
     bytes32 public constant PAUSER_ROLE = keccak256("PAUSER_ROLE");
+    /// @inheritdoc IIBCPausable
     bytes32 public constant UNPAUSER_ROLE = keccak256("UNPAUSER_ROLE");
 
     /// @dev Initializes the contract in unpaused state.
-    /// @param pauser The address that can pause and unpause the contract
+    /// @param pauser The address that is granted the `PAUSER_ROLE`
+    /// @param unpauser The address that is granted the `UNPAUSER_ROLE`
     function __IBCPausable_init(address pauser, address unpauser) internal onlyInitializing {
         __Pausable_init();
         __AccessControl_init();
@@ -39,7 +41,7 @@ abstract contract IBCPausableUpgradeable is
     }
 
     /// @inheritdoc IIBCPausable
-    function unpause() external virtual onlyRole(UNPAUSER_ROLE) {
+    function unpause() external onlyRole(UNPAUSER_ROLE) {
         _unpause();
     }
 

scripts/E2ETestDeploy.s.sol

@@ -77,6 +77,7 @@ contract E2ETestDeploy is Script, IICS07TendermintMsgs, DeploySP1ICS07Tendermint
             escrowImplementation: escrowLogic,
             ibcERC20Implementation: ibcERC20Logic,
             pauser: address(0),
+            unpauser: address(0),
             permit2: address(0)
         });