refactor(gateway-contracts): improve pause all contracts method behavior

.github/workflows/gateway-contracts-upgrade-tests.yml

@@ -121,7 +121,6 @@ jobs:
       #  See https://github.com/zama-ai/fhevm-internal/issues/379
       - name: Upgrade GatewayConfig contract
         working-directory: current-fhevm/gateway-contracts
-        if: false
         run: |
           npx hardhat task:upgradeGatewayConfig \
             --current-implementation previous-contracts/GatewayConfig.sol:GatewayConfig \

gateway-contracts/contracts/GatewayConfig.sol

@@ -35,7 +35,7 @@ contract GatewayConfig is IGatewayConfig, Ownable2StepUpgradeable, UUPSUpgradeab
      */
     string private constant CONTRACT_NAME = "GatewayConfig";
     uint256 private constant MAJOR_VERSION = 0;
-    uint256 private constant MINOR_VERSION = 4;
+    uint256 private constant MINOR_VERSION = 5;
     uint256 private constant PATCH_VERSION = 0;
 
     /**
@@ -44,7 +44,7 @@ contract GatewayConfig is IGatewayConfig, Ownable2StepUpgradeable, UUPSUpgradeab
      * This constant does not represent the number of time a specific contract have been upgraded,
      * as a contract deployed from version VX will have a REINITIALIZER_VERSION > 2.
      */
-    uint64 private constant REINITIALIZER_VERSION = 5;
+    uint64 private constant REINITIALIZER_VERSION = 6;
 
     /**
      * @notice The address of the all gateway contracts
@@ -359,21 +359,41 @@ contract GatewayConfig is IGatewayConfig, Ownable2StepUpgradeable, UUPSUpgradeab
     /**
      * @notice See {IGatewayConfig-pauseAllGatewayContracts}.
      * Contracts that are technically pausable but do not provide any pausable functions are not
-     * paused. If at least one of the contracts is already paused, the function will revert.
+     * paused. If all of the contracts are already paused, the function will revert.
      */
     function pauseAllGatewayContracts() external virtual onlyPauser {
-        DECRYPTION.pause();
-        INPUT_VERIFICATION.pause();
+        if (DECRYPTION.paused() && INPUT_VERIFICATION.paused()) {
+            revert AllGatewayContractsAlreadyPaused();
+        }
+
+        if (!DECRYPTION.paused()) {
+            DECRYPTION.pause();
+        }
+
+        if (!INPUT_VERIFICATION.paused()) {
+            INPUT_VERIFICATION.pause();
+        }
+
         emit PauseAllGatewayContracts();
     }
 
     /**
      * @notice See {IGatewayConfig-unpauseAllGatewayContracts}.
-     * If at least one of the contracts is not paused, the function will revert.
+     * If all of the contracts are not paused, the function will revert.
      */
     function unpauseAllGatewayContracts() external virtual onlyOwner {
-        DECRYPTION.unpause();
-        INPUT_VERIFICATION.unpause();
+        if (!DECRYPTION.paused() && !INPUT_VERIFICATION.paused()) {
+            revert AllGatewayContractsAlreadyUnpaused();
+        }
+
+        if (DECRYPTION.paused()) {
+            DECRYPTION.unpause();
+        }
+
+        if (INPUT_VERIFICATION.paused()) {
+            INPUT_VERIFICATION.unpause();
+        }
+
         emit UnpauseAllGatewayContracts();
     }
 

gateway-contracts/contracts/interfaces/IGatewayConfig.sol

@@ -255,6 +255,16 @@ interface IGatewayConfig {
      */
     error ChainIdNotUint64(uint256 chainId);
 
+    /**
+     * @notice Error emitted when all the pausable gateway contracts are already paused.
+     */
+    error AllGatewayContractsAlreadyPaused();
+
+    /**
+     * @notice Error emitted when all the pausable gateway contracts are already unpaused.
+     */
+    error AllGatewayContractsAlreadyUnpaused();
+
     /**
      * @notice Update the list of KMS nodes and their thresholds.
      * @dev ⚠️ This function should be used with caution as it can lead to unexpected behavior in