Unit tests for L1BitcoinDepositor contract

keep-network · Mar 8, 2024 · 4616e6d · 4616e6d
1 parent d7c0955
commit 4616e6d
Show file tree

Hide file tree

Showing 3 changed files with 1,485 additions and 41 deletions.
diff --git a/solidity/contracts/l2/L1BitcoinDepositor.sol b/solidity/contracts/l2/L1BitcoinDepositor.sol
@@ -470,7 +470,21 @@ contract L1BitcoinDepositor is
             ? tx.gasprice
             : maxGasPrice;
 
-        return (refund / gasPrice) - staticGas;
+        // Should not happen but check just in case of weird ReimbursementPool
+        // configuration.
+        if (gasPrice == 0) {
+            return 0;
+        }
+
+        uint256 gasSpent = (refund / gasPrice);
+
+        // Should not happen but check just in case of weird ReimbursementPool
+        // configuration.
+        if (staticGas > gasSpent) {
+            return 0;
+        }
+
+        return gasSpent - staticGas;
     }
 
     /// @notice Quotes the payment that must be attached to the `finalizeDeposit`