@ledgerhq/live-common 27.2.0 → 27.3.0-next.0

package/src/families/cosmos/specs.ts

@@ -13,7 +13,12 @@ import type {
 } from "../../families/cosmos/types";
 import { getCurrentCosmosPreloadData } from "../../families/cosmos/preloadedData";
 import { getCryptoCurrencyById } from "../../currencies";
-import { pickSiblings, botTest } from "../../bot/specs";
+import {
+  pickSiblings,
+  botTest,
+  expectSiblingsHaveSpendablePartGreaterThan,
+  genericTestDestination,
+} from "../../bot/specs";
 import type { AppSpec } from "../../bot/types";
 import { toOperationRaw } from "../../account";
 import {
@@ -56,6 +61,7 @@ const cosmos: AppSpec<Transaction> = {
   },
   genericDeviceAction: acceptTransaction,
   testTimeout: 2 * 60 * 1000,
+  minViableAmount: minAmount,
   transactionCheck: ({ maxSpendable }) => {
     invariant(maxSpendable.gt(minAmount), "balance is too low");
   },
@@ -92,6 +98,7 @@ const cosmos: AppSpec<Transaction> = {
   mutations: [
     {
       name: "send some",
+      testDestination: genericTestDestination,
       test: ({ account, accountBeforeTransaction, operation }) => {
         expect(account.balance.toString()).toBe(
           accountBeforeTransaction.balance.minus(operation.value).toString()
@@ -123,6 +130,7 @@ const cosmos: AppSpec<Transaction> = {
     {
       name: "send max",
       maxRun: 1,
+      testDestination: genericTestDestination,
       transaction: ({ account, siblings, bridge }) => {
         return {
           transaction: bridge.createTransaction(account),
@@ -145,7 +153,8 @@ const cosmos: AppSpec<Transaction> = {
     {
       name: "delegate new validators",
       maxRun: 1,
-      transaction: ({ account, bridge }) => {
+      transaction: ({ account, bridge, siblings }) => {
+        expectSiblingsHaveSpendablePartGreaterThan(siblings, 0.5);
         invariant(
           account.index % 2 > 0,
           "only one out of 2 accounts is not going to delegate"

package/src/families/crypto_org/specs.ts

@@ -6,6 +6,14 @@ import type { AppSpec } from "../../bot/types";
 import { DeviceModelId } from "@ledgerhq/devices";
 import { acceptTransaction } from "./speculos-deviceActions";
 
+const minAmount = parseCurrencyUnit(
+  getCryptoCurrencyById("crypto_org").units[0],
+  "0.01"
+);
+const transactionCheck = ({ maxSpendable }) => {
+  invariant(maxSpendable.gt(minAmount), "balance is too low");
+};
+
 const sharedMutations = ({ maxAccount }) => [
   {
     name: "move 50%",
@@ -31,17 +39,8 @@ const crypto_org_croeseid: AppSpec<Transaction> = {
   },
   genericDeviceAction: acceptTransaction,
   testTimeout: 4 * 60 * 1000,
-  transactionCheck: ({ maxSpendable }) => {
-    invariant(
-      maxSpendable.gt(
-        parseCurrencyUnit(
-          getCryptoCurrencyById("crypto_org_croeseid").units[0],
-          "0.01"
-        )
-      ),
-      "balance is too low"
-    );
-  },
+  minViableAmount: minAmount,
+  transactionCheck,
   mutations: sharedMutations({ maxAccount: 5 }),
 };
 
@@ -54,14 +53,8 @@ const crypto_org: AppSpec<Transaction> = {
   },
   genericDeviceAction: acceptTransaction,
   testTimeout: 4 * 60 * 1000,
-  transactionCheck: ({ maxSpendable }) => {
-    invariant(
-      maxSpendable.gt(
-        parseCurrencyUnit(getCryptoCurrencyById("crypto_org").units[0], "0.01")
-      ),
-      "balance is too low"
-    );
-  },
+  minViableAmount: minAmount,
+  transactionCheck,
   mutations: sharedMutations({ maxAccount: 5 }),
 };
 

package/src/families/elrond/specs.ts

@@ -28,6 +28,7 @@ const elrondSpec: AppSpec<Transaction> = {
   },
   genericDeviceAction: acceptTransaction,
   testTimeout: 2 * 60 * 1000,
+  minViableAmount: ELROND_MIN_SAFE,
   transactionCheck: ({ maxSpendable }) => {
     invariant(maxSpendable.gt(ELROND_MIN_SAFE), "balance is too low");
   },

package/src/families/elrond/speculos-deviceActions.ts

@@ -1,7 +1,6 @@
 import type { DeviceAction } from "../../bot/types";
 import type { Transaction } from "./types";
-import { deviceActionFlow } from "../../bot/specs";
-import { formatCurrencyUnit } from "../../currencies";
+import { deviceActionFlow, formatDeviceAmount } from "../../bot/specs";
 
 export const acceptTransaction: DeviceAction<Transaction, any> =
   deviceActionFlow({
@@ -15,13 +14,9 @@ export const acceptTransaction: DeviceAction<Transaction, any> =
         title: "Amount",
         button: "Rr",
         expectedValue: ({ account, transaction }) =>
-          formatCurrencyUnit(account.unit, transaction.amount, {
-            showCode: true,
-            disableRounding: true,
-            joinFragmentsSeparator: " ",
-          })
-            .replace(/\s+/g, " ")
-            .replace("egld", "EGLD"), // FIXME
+          formatDeviceAmount(account.currency, transaction.amount, {
+            postfixCode: true,
+          }),
       },
       {
         title: "Fee",

package/src/families/ethereum/specs.ts

@@ -13,7 +13,7 @@ import {
   getAccountCapabilities,
 } from "../../compound/logic";
 import { getSupplyMax } from "./modules/compound";
-import { botTest, pickSiblings } from "../../bot/specs";
+import { botTest, genericTestDestination, pickSiblings } from "../../bot/specs";
 import type { AppSpec } from "../../bot/types";
 import { getGasLimit } from "./transaction";
 import { DeviceModelId } from "@ledgerhq/devices";
@@ -27,6 +27,7 @@ const ethereumBasicMutations = ({ maxAccount }) => [
   {
     name: "move 50%",
     maxRun: 2,
+    testDestination: genericTestDestination,
     transaction: ({ account, siblings, bridge, maxSpendable }) => {
       const sibling = pickSiblings(siblings, maxAccount);
       const recipient = sibling.freshAddress;
@@ -119,26 +120,28 @@ function getCompoundResult({ account, transaction, accountBeforeTransaction }) {
   };
 }
 
+const minAmountETH = parseCurrencyUnit(
+  getCryptoCurrencyById("ethereum").units[0],
+  "0.01"
+);
+
 const ethereum: AppSpec<Transaction> = {
   name: "Ethereum",
   currency: getCryptoCurrencyById("ethereum"),
   appQuery: {
     model: DeviceModelId.nanoS,
     appName: "Ethereum",
+    appVersion: "1.9.20-dev", // FIXME remove this line once 1.9.20 lands on coin-apps (branch ledger-live-bot)
   },
   genericDeviceAction: acceptTransaction,
   testTimeout,
+  minViableAmount: minAmountETH,
   transactionCheck: ({ maxSpendable }) => {
-    invariant(
-      maxSpendable.gt(
-        parseCurrencyUnit(getCryptoCurrencyById("ethereum").units[0], "0.01")
-      ),
-      "balance is too low"
-    );
+    invariant(maxSpendable.gt(minAmountETH), "balance is too low");
   },
   // @ts-expect-error seriously we have to do somehting
   mutations: ethereumBasicMutations({
-    maxAccount: 3,
+    maxAccount: 7,
   }).concat([
     {
       name: "allow MAX a compound token",
@@ -371,6 +374,11 @@ const ethereum: AppSpec<Transaction> = {
     },
   ]),
 };
+
+const minAmountETC = parseCurrencyUnit(
+  getCryptoCurrencyById("ethereum_classic").units[0],
+  "0.05"
+);
 const ethereumClassic: AppSpec<Transaction> = {
   name: "Ethereum Classic",
   currency: getCryptoCurrencyById("ethereum_classic"),
@@ -381,16 +389,9 @@ const ethereumClassic: AppSpec<Transaction> = {
   genericDeviceAction: acceptTransaction,
   dependency: "Ethereum",
   testTimeout,
+  minViableAmount: minAmountETC,
   transactionCheck: ({ maxSpendable }) => {
-    invariant(
-      maxSpendable.gt(
-        parseCurrencyUnit(
-          getCryptoCurrencyById("ethereum_classic").units[0],
-          "0.05"
-        )
-      ),
-      "balance is too low"
-    );
+    invariant(maxSpendable.gt(minAmountETC), "balance is too low");
   },
   mutations: ethereumBasicMutations({
     maxAccount: 4,
@@ -402,25 +403,23 @@ const ethereumGoerli: AppSpec<Transaction> = {
   appQuery: {
     model: DeviceModelId.nanoS,
     appName: "Ethereum",
+    appVersion: "1.9.20-dev", // FIXME remove this line once 1.9.20 lands on coin-apps (branch ledger-live-bot)
   },
   genericDeviceAction: acceptTransaction,
   testTimeout,
+  minViableAmount: minAmountETH,
   transactionCheck: ({ maxSpendable }) => {
-    invariant(
-      maxSpendable.gt(
-        parseCurrencyUnit(
-          getCryptoCurrencyById("ethereum_goerli").units[0],
-          "0.01"
-        )
-      ),
-      "balance is too low"
-    );
+    invariant(maxSpendable.gt(minAmountETH), "balance is too low");
   },
   mutations: ethereumBasicMutations({
     maxAccount: 8,
   }),
 };
 
+const minAmountBSC = parseCurrencyUnit(
+  getCryptoCurrencyById("bsc").units[0],
+  "0.005"
+);
 const bsc: AppSpec<Transaction> = {
   name: "BSC",
   currency: getCryptoCurrencyById("bsc"),
@@ -431,13 +430,9 @@ const bsc: AppSpec<Transaction> = {
   genericDeviceAction: acceptTransaction,
   dependency: "Ethereum",
   testTimeout,
+  minViableAmount: minAmountBSC,
   transactionCheck: ({ maxSpendable }) => {
-    invariant(
-      maxSpendable.gt(
-        parseCurrencyUnit(getCryptoCurrencyById("bsc").units[0], "0.005")
-      ),
-      "balance is too low"
-    );
+    invariant(maxSpendable.gt(minAmountBSC), "balance is too low");
   },
   mutations: ethereumBasicMutations({ maxAccount: 8 }).concat([
     {
@@ -469,6 +464,11 @@ const bsc: AppSpec<Transaction> = {
   ]),
 };
 
+const minAmountPolygon = parseCurrencyUnit(
+  getCryptoCurrencyById("polygon").units[0],
+  "0.005"
+);
+
 const polygon: AppSpec<Transaction> = {
   name: "Polygon",
   currency: getCryptoCurrencyById("polygon"),
@@ -479,13 +479,9 @@ const polygon: AppSpec<Transaction> = {
   genericDeviceAction: acceptTransaction,
   dependency: "Ethereum",
   testTimeout,
+  minViableAmount: minAmountPolygon,
   transactionCheck: ({ maxSpendable }) => {
-    invariant(
-      maxSpendable.gt(
-        parseCurrencyUnit(getCryptoCurrencyById("polygon").units[0], "0.005")
-      ),
-      "balance is too low"
-    );
+    invariant(maxSpendable.gt(minAmountPolygon), "balance is too low");
   },
   mutations: ethereumBasicMutations({ maxAccount: 8 }).concat([
     {

package/src/families/ethereum/speculos-deviceActions.ts

@@ -1,7 +1,6 @@
 import type { DeviceAction } from "../../bot/types";
 import type { Transaction } from "./types";
-import { formatCurrencyUnit, findCompoundToken } from "../../currencies";
-import { deviceActionFlow } from "../../bot/specs";
+import { deviceActionFlow, formatDeviceAmount } from "../../bot/specs";
 
 function subAccount(subAccountId, account) {
   const sub = (account.subAccounts || []).find((a) => a.id === subAccountId);
@@ -10,28 +9,8 @@ function subAccount(subAccountId, account) {
   return sub;
 }
 
-function expectedCompoundToken(t) {
-  if (!t) {
-    throw new Error("compound token was expected");
-  }
-
-  return t;
-}
-
 const maxFeesExpectedValue = ({ account, status }) =>
-  formatCurrencyUnit(
-    {
-      ...account.unit,
-      code: account.currency.deviceTicker || account.unit.code,
-      prefixCode: true,
-    },
-    status.estimatedFees,
-    {
-      showCode: true,
-      disableRounding: true,
-      joinFragmentsSeparator: " ",
-    }
-  ).replace(/\s/g, " ");
+  formatDeviceAmount(account.currency, status.estimatedFees);
 
 export const acceptTransaction: DeviceAction<Transaction, any> =
   deviceActionFlow({
@@ -66,16 +45,6 @@ export const acceptTransaction: DeviceAction<Transaction, any> =
             return "Unlimited " + a.token.ticker;
           }
 
-          const unit = !a
-            ? {
-                ...account.unit,
-                code: account.currency.deviceTicker || account.unit.code,
-              }
-            : (transaction.mode === "compound.withdraw" &&
-              transaction.useAllAmount
-                ? expectedCompoundToken(findCompoundToken(a.token))
-                : a.token
-              ).units[0];
           const amount =
             a &&
             a.compoundBalance &&
@@ -83,19 +52,20 @@ export const acceptTransaction: DeviceAction<Transaction, any> =
             transaction.useAllAmount
               ? a.compoundBalance
               : status.amount;
-          return formatCurrencyUnit(
-            {
-              ...unit,
-              code: account.currency.deviceTicker || account.unit.code,
-              prefixCode: true,
-            },
-            amount,
-            {
-              showCode: true,
-              disableRounding: true,
-              joinFragmentsSeparator: " ",
-            }
-          ).replace(/\s/g, " ");
+
+          if (
+            a &&
+            transaction.mode === "compound.withdraw" &&
+            transaction.useAllAmount
+          ) {
+            return formatDeviceAmount(a.compoundBalance, amount);
+          }
+
+          if (a) {
+            return formatDeviceAmount(a.token, amount);
+          }
+
+          return formatDeviceAmount(account.currency, amount);
         },
       },
       {

package/src/families/evm/specs.ts

@@ -3,7 +3,7 @@ import invariant from "invariant";
 import { DeviceModelId } from "@ledgerhq/devices";
 import { getCryptoCurrencyById, parseCurrencyUnit } from "../../currencies";
 import { acceptTransaction } from "./speculos-deviceActions";
-import { pickSiblings } from "../../bot/specs";
+import { botTest, genericTestDestination, pickSiblings } from "../../bot/specs";
 import type { AppSpec } from "../../bot/types";
 import type { Transaction } from "./types";
 
@@ -14,7 +14,7 @@ const transactionCheck =
   ({ maxSpendable }) => {
     invariant(
       maxSpendable.gt(
-        parseCurrencyUnit(getCryptoCurrencyById(currencyId).units[0], "0.01")
+        parseCurrencyUnit(getCryptoCurrencyById(currencyId).units[0], "1")
       ),
       `${currencyId} balance is too low`
     );
@@ -24,6 +24,7 @@ const evmBasicMutations = ({ maxAccount }) => [
   {
     name: "move 50%",
     maxRun: 2,
+    testDestination: genericTestDestination,
     transaction: ({ account, siblings, bridge, maxSpendable }) => {
       const sibling = pickSiblings(siblings, maxAccount);
       const recipient = sibling.freshAddress;
@@ -45,13 +46,17 @@ const evmBasicMutations = ({ maxAccount }) => [
         "operation time to be older than 60s"
       );
       const estimatedGas = transaction.gasLimit.times(
-        transaction.gasPrice || 0
+        transaction.gasPrice || transaction.maxFeePerGas || 0
       );
-      expect(operation.fee.toNumber()).toBeLessThanOrEqual(
-        estimatedGas.toNumber()
+      botTest("operation fee is not exceeding estimated gas", () =>
+        expect(operation.fee.toNumber()).toBeLessThanOrEqual(
+          estimatedGas.toNumber()
+        )
       );
-      expect(account.balance.toString()).toBe(
-        accountBeforeTransaction.balance.minus(operation.value).toString()
+      botTest("account balance moved with operation value", () =>
+        expect(account.balance.toString()).toBe(
+          accountBeforeTransaction.balance.minus(operation.value).toString()
+        )
       );
     },
   },
@@ -63,6 +68,7 @@ const cronos: AppSpec<Transaction> = {
   appQuery: {
     model: DeviceModelId.nanoS,
     appName: "Ethereum",
+    appVersion: "1.9.20-dev", // FIXME remove this line once 1.9.20 lands on coin-apps (branch ledger-live-bot)
   },
   testTimeout,
   transactionCheck: transactionCheck("cronos"),
@@ -78,6 +84,7 @@ const fantom: AppSpec<Transaction> = {
   appQuery: {
     model: DeviceModelId.nanoS,
     appName: "Ethereum",
+    appVersion: "1.9.20-dev", // FIXME remove this line once 1.9.20 lands on coin-apps (branch ledger-live-bot)
   },
   testTimeout,
   transactionCheck: transactionCheck("fantom"),
@@ -93,6 +100,7 @@ const moonbeam: AppSpec<Transaction> = {
   appQuery: {
     model: DeviceModelId.nanoS,
     appName: "Ethereum",
+    appVersion: "1.9.20-dev", // FIXME remove this line once 1.9.20 lands on coin-apps (branch ledger-live-bot)
   },
   testTimeout,
   transactionCheck: transactionCheck("moonbeam"),
@@ -108,6 +116,7 @@ const songbird: AppSpec<Transaction> = {
   appQuery: {
     model: DeviceModelId.nanoS,
     appName: "Ethereum",
+    appVersion: "1.9.20-dev", // FIXME remove this line once 1.9.20 lands on coin-apps (branch ledger-live-bot)
   },
   testTimeout,
   transactionCheck: transactionCheck("songbird"),
@@ -118,11 +127,12 @@ const songbird: AppSpec<Transaction> = {
 };
 
 const flare: AppSpec<Transaction> = {
-  name: "Moonbeam",
+  name: "Flare",
   currency: getCryptoCurrencyById("flare"),
   appQuery: {
     model: DeviceModelId.nanoS,
     appName: "Ethereum",
+    appVersion: "1.9.20-dev", // FIXME remove this line once 1.9.20 lands on coin-apps (branch ledger-live-bot)
   },
   testTimeout,
   transactionCheck: transactionCheck("flare"),

package/src/families/evm/speculos-deviceActions.ts

@@ -1,7 +1,6 @@
 import type { DeviceAction } from "../../bot/types";
 import type { Transaction } from "./types";
-import { formatCurrencyUnit } from "../../currencies";
-import { deviceActionFlow } from "../../bot/specs";
+import { deviceActionFlow, formatDeviceAmount } from "../../bot/specs";
 
 function subAccount(subAccountId, account) {
   const sub = (account.subAccounts || []).find((a) => a.id === subAccountId);
@@ -11,19 +10,7 @@ function subAccount(subAccountId, account) {
 }
 
 const maxFeesExpectedValue = ({ account, status }) =>
-  formatCurrencyUnit(
-    {
-      ...account.unit,
-      code: account.currency.deviceTicker || account.unit.code,
-      prefixCode: true,
-    },
-    status.estimatedFees,
-    {
-      showCode: true,
-      disableRounding: true,
-      joinFragmentsSeparator: " ",
-    }
-  ).replace(/\s/g, " ");
+  formatDeviceAmount(account.currency, status.estimatedFees);
 
 export const acceptTransaction: DeviceAction<Transaction, any> =
   deviceActionFlow({
@@ -44,26 +31,11 @@ export const acceptTransaction: DeviceAction<Transaction, any> =
             ? subAccount(transaction.subAccountId, account)
             : null;
 
-          const unit = !a
-            ? {
-                ...account.unit,
-                code: account.currency.deviceTicker || account.unit.code,
-              }
-            : a.token.units[0];
-          const amount = status.amount;
-          return formatCurrencyUnit(
-            {
-              ...unit,
-              code: account.currency.deviceTicker || account.unit.code,
-              prefixCode: true,
-            },
-            amount,
-            {
-              showCode: true,
-              disableRounding: true,
-              joinFragmentsSeparator: " ",
-            }
-          ).replace(/\s/g, " ");
+          if (a) {
+            return formatDeviceAmount(a.token, status.amount);
+          }
+
+          return formatDeviceAmount(account.currency, status.amount);
         },
       },
       {

package/src/families/filecoin/specs.ts

@@ -4,7 +4,7 @@ import BigNumber from "bignumber.js";
 
 import type { Transaction } from "../../families/filecoin/types";
 import { getCryptoCurrencyById } from "../../currencies";
-import { pickSiblings } from "../../bot/specs";
+import { genericTestDestination, pickSiblings } from "../../bot/specs";
 import type { AppSpec } from "../../bot/types";
 import { acceptTransaction } from "./speculos-deviceActions";
 
@@ -20,6 +20,7 @@ const filecoinSpecs: AppSpec<Transaction> = {
   },
   genericDeviceAction: acceptTransaction,
   testTimeout: 5 * 60 * 1000,
+  minViableAmount: MIN_SAFE,
   transactionCheck: ({ maxSpendable }) => {
     invariant(maxSpendable.gt(MIN_SAFE), "balance is too low");
   },
@@ -27,6 +28,7 @@ const filecoinSpecs: AppSpec<Transaction> = {
     {
       name: "Send 50%~",
       maxRun: 1,
+      testDestination: genericTestDestination,
       transaction: ({ account, siblings, bridge }) => {
         const sibling = pickSiblings(siblings, maxAccount);
         let amount = account.spendableBalance
@@ -57,6 +59,7 @@ const filecoinSpecs: AppSpec<Transaction> = {
     {
       name: "Transfer Max",
       maxRun: 1,
+      testDestination: genericTestDestination,
       transaction: ({ account, siblings, bridge }) => {
         return {
           transaction: bridge.createTransaction(account),

package/src/families/filecoin/speculos-deviceActions.ts

@@ -1,7 +1,6 @@
 import type { DeviceAction } from "../../bot/types";
 import type { Transaction } from "./types";
-import { deviceActionFlow } from "../../bot/specs";
-import { formatCurrencyUnit } from "../../currencies";
+import { deviceActionFlow, formatDeviceAmount } from "../../bot/specs";
 
 export const acceptTransaction: DeviceAction<Transaction, any> =
   deviceActionFlow({
@@ -25,8 +24,8 @@ export const acceptTransaction: DeviceAction<Transaction, any> =
         title: "Value",
         button: "Rr",
         expectedValue: ({ account, status }) =>
-          formatCurrencyUnit(account.unit, status.amount, {
-            disableRounding: true,
+          formatDeviceAmount(account.currency, status.amount, {
+            hideCode: true,
             showAllDigits: true,
           }),
       },
@@ -39,8 +38,8 @@ export const acceptTransaction: DeviceAction<Transaction, any> =
         title: "Gas Premium",
         button: "Rr",
         expectedValue: ({ account, transaction }) =>
-          formatCurrencyUnit(account.unit, transaction.gasPremium, {
-            disableRounding: true,
+          formatDeviceAmount(account.currency, transaction.gasPremium, {
+            hideCode: true,
             showAllDigits: true,
           }),
       },
@@ -48,8 +47,8 @@ export const acceptTransaction: DeviceAction<Transaction, any> =
         title: "Gas Fee Cap",
         button: "Rr",
         expectedValue: ({ account, transaction }) =>
-          formatCurrencyUnit(account.unit, transaction.gasFeeCap, {
-            disableRounding: true,
+          formatDeviceAmount(account.currency, transaction.gasFeeCap, {
+            hideCode: true,
             showAllDigits: true,
           }),
       },

package/src/families/hedera/specs.ts

@@ -9,7 +9,7 @@ import type {
   TransactionRes,
 } from "../../bot/types";
 import type { Transaction } from "./types";
-import { botTest, pickSiblings } from "../../bot/specs";
+import { botTest, genericTestDestination, pickSiblings } from "../../bot/specs";
 import { isAccountEmpty } from "../../account";
 import { acceptTransaction } from "./speculos-deviceActions";
 
@@ -49,6 +49,7 @@ const hedera: AppSpec<Transaction> = {
     {
       name: "Send ~50%",
       maxRun: 2,
+      testDestination: genericTestDestination,
       transaction: ({
         account,
         siblings,
@@ -85,6 +86,7 @@ const hedera: AppSpec<Transaction> = {
     {
       name: "Send max",
       maxRun: 2,
+      testDestination: genericTestDestination,
       transaction: ({
         account,
         siblings,

package/src/families/osmosis/specs.ts

@@ -1,7 +1,7 @@
 import invariant from "invariant";
 import type { Transaction } from "./types";
 import { getCryptoCurrencyById, parseCurrencyUnit } from "../../currencies";
-import { botTest, pickSiblings } from "../../bot/specs";
+import { botTest, genericTestDestination, pickSiblings } from "../../bot/specs";
 import type { AppSpec } from "../../bot/types";
 import { DeviceModelId } from "@ledgerhq/devices";
 import type {
@@ -51,6 +51,7 @@ const osmosis: AppSpec<Transaction> = {
   },
   genericDeviceAction: acceptTransaction,
   testTimeout: 2 * 60 * 1000,
+  minViableAmount: minimalAmount,
   transactionCheck: ({ maxSpendable }) => {
     invariant(maxSpendable.gt(minimalAmount), "balance is too low");
   },
@@ -89,6 +90,7 @@ const osmosis: AppSpec<Transaction> = {
     {
       name: "move 50% to another account",
       maxRun: 2,
+      testDestination: genericTestDestination,
       transaction: ({ account, siblings, bridge, maxSpendable }) => {
         invariant(maxSpendable.gt(minimalAmount), "balance is too low");
         const sibling = pickSiblings(siblings, maxAccount);
@@ -111,6 +113,7 @@ const osmosis: AppSpec<Transaction> = {
     {
       name: "send max to another account",
       maxRun: 1,
+      testDestination: genericTestDestination,
       transaction: ({ account, siblings, bridge, maxSpendable }) => {
         invariant(maxSpendable.gt(minimalAmount), "balance is too low");
         const sibling = pickSiblings(siblings, maxAccount);