fix: chain swaps to unconfidential Liquid addresses

BoltzExchange · May 28, 2024 · 8e3fa41 · 8e3fa41
1 parent f305eaf
commit 8e3fa41
Show file tree

Hide file tree

Showing 2 changed files with 45 additions and 11 deletions.
diff --git a/src/components/Fees.tsx b/src/components/Fees.tsx
@@ -37,6 +37,11 @@ const Fees = () => {
         addressValid,
     } = useCreateContext();
 
+    const isToUnconfidentialLiquid = () =>
+        assetReceive() === LBTC &&
+        addressValid() &&
+        !isConfidentialAddress(onchainAddress());
+
     createEffect(() => {
         if (pairs()) {
             const cfg = getPair(
@@ -62,25 +67,26 @@ const Fees = () => {
                     let fee =
                         reverseCfg.fees.minerFees.claim +
                         reverseCfg.fees.minerFees.lockup;
-                    if (
-                        assetReceive() === LBTC &&
-                        addressValid() &&
-                        !isConfidentialAddress(onchainAddress())
-                    ) {
+                    if (isToUnconfidentialLiquid()) {
                         fee += 1;
                     }
 
                     setMinerFee(fee);
                     break;
 
-                case SwapType.Chain:
+                case SwapType.Chain: {
                     const chainCfg = cfg as ChainPairTypeTaproot;
-                    setMinerFee(
+                    let fee =
                         chainCfg.fees.minerFees.server +
-                            chainCfg.fees.minerFees.user.lockup +
-                            chainCfg.fees.minerFees.user.claim,
-                    );
+                        chainCfg.fees.minerFees.user.lockup +
+                        chainCfg.fees.minerFees.user.claim;
+                    if (isToUnconfidentialLiquid()) {
+                        fee += 1;
+                    }
+
+                    setMinerFee(fee);
                     break;
+                }
             }
 
             const calculateLimit = (limit: number): number => {

diff --git a/tests/components/Fees.spec.tsx b/tests/components/Fees.spec.tsx
@@ -70,7 +70,7 @@ describe("Fees component", () => {
         );
     });
 
-    test("should increase the miner fee by 1 when sending to an unconfidential Liquid address", () => {
+    test("should increase the miner fee for reverse swaps by 1 when sending to an unconfidential Liquid address", () => {
         render(
             () => (
                 <>
@@ -93,4 +93,32 @@ describe("Fees component", () => {
             fees.minerFees.lockup + fees.minerFees.claim + 1,
         );
     });
+
+    test("should increase the miner fee for chain swaps by 1 when sending to an unconfidential Liquid address", () => {
+        render(
+            () => (
+                <>
+                    <TestComponent />
+                    <Fees />
+                </>
+            ),
+            { wrapper: contextWrapper },
+        );
+
+        globalSignals.setPairs(pairs);
+        signals.setAssetSend(BTC);
+        signals.setAssetReceive(LBTC);
+        signals.setAddressValid(true);
+        signals.setOnchainAddress(
+            "ert1q2vf850cshpedhvn9x0lv33j8az4ela04afuzp0",
+        );
+
+        const fees = pairs.chain[BTC][LBTC].fees;
+        expect(signals.minerFee()).toEqual(
+            fees.minerFees.server +
+                fees.minerFees.user.lockup +
+                fees.minerFees.user.claim +
+                1,
+        );
+    });
 });