feat: add features flags on every test and set maxRun to 1

LedgerHQ · Jan 16, 2025 · a7cbb03 · a7cbb03
1 parent ce9b984
commit a7cbb03
Show file tree

Hide file tree

Showing 30 changed files with 113 additions and 20 deletions.
diff --git a/libs/coin-modules/coin-algorand/src/specs.ts b/libs/coin-modules/coin-algorand/src/specs.ts
@@ -70,7 +70,8 @@ const algorand: AppSpec<AlgorandTransaction> = {
   mutations: [
     {
       name: "move ~50%",
-      maxRun: 2,
+      feature: "send",
+      maxRun: 1,
       testDestination: genericTestDestination,
       transaction: ({ account, siblings, bridge, maxSpendable }) => {
         invariant(maxSpendable.gt(0), "Spendable balance is too low");
@@ -104,6 +105,7 @@ const algorand: AppSpec<AlgorandTransaction> = {
     },
     {
       name: "send max",
+      feature: "sendMax",
       maxRun: 1,
       testDestination: genericTestDestination,
       transaction: ({ account, siblings, bridge, maxSpendable }) => {
@@ -135,7 +137,8 @@ const algorand: AppSpec<AlgorandTransaction> = {
     },
     {
       name: "send ASA ~50%",
-      maxRun: 2,
+      feature: "tokens",
+      maxRun: 1,
       transaction: ({ account, siblings, bridge, maxSpendable }) => {
         invariant(maxSpendable.gt(0), "Spendable balance is too low");
         const subAccount = sample(getAssetsWithBalance(account));
@@ -178,6 +181,7 @@ const algorand: AppSpec<AlgorandTransaction> = {
     },
     {
       name: "opt-In ASA available",
+      feature: "tokens",
       maxRun: 1,
       transaction: ({ account, bridge, maxSpendable }) => {
         // maxSpendable is expected to be greater than 100,000 micro-Algos
@@ -217,6 +221,7 @@ const algorand: AppSpec<AlgorandTransaction> = {
     },
     {
       name: "claim rewards",
+      feature: "staking",
       maxRun: 1,
       transaction: ({ account, bridge, maxSpendable }) => {
         const rewards = (account as AlgorandAccount).algorandResources?.rewards;

diff --git a/libs/coin-modules/coin-bitcoin/src/specs.ts b/libs/coin-modules/coin-bitcoin/src/specs.ts
@@ -145,6 +145,7 @@ const bitcoinLikeMutations = ({
   },
   {
     name: "optimize-size",
+    feature: "send",
     maxRun: 1,
     transaction: ({ account, siblings, bridge, maxSpendable }) => {
       invariant(maxSpendable.gt(minimalAmount), "balance is too low");
@@ -177,6 +178,7 @@ const bitcoinLikeMutations = ({
   },
   {
     name: "send 1 utxo",
+    feature: "send",
     maxRun: 1,
     transaction: ({ account, bridge, siblings, maxSpendable }) => {
       invariant(maxSpendable.gt(minimalAmount), "balance is too low");
@@ -250,6 +252,7 @@ const bitcoinLikeMutations = ({
   },
   {
     name: "send OP_RETURN transaction",
+    feature: "send",
     maxRun: 1,
     transaction: ({ account, bridge, siblings, maxSpendable }) => {
       invariant(maxSpendable.gt(minimalAmount), "balance is too low");

diff --git a/libs/coin-modules/coin-cardano/src/specs.ts b/libs/coin-modules/coin-cardano/src/specs.ts
@@ -35,6 +35,7 @@ const cardano: AppSpec<Transaction> = {
   mutations: [
     {
       name: "move ~10% token",
+      feature: "tokens",
       maxRun: 1,
       transaction: ({ account, siblings, bridge, maxSpendable }) => {
         invariant(maxSpendable.gte(minSpendableRequiredForTokenTx), "balance is too low");
@@ -77,6 +78,7 @@ const cardano: AppSpec<Transaction> = {
     {
       testDestination: genericTestDestination,
       name: "move ~50%",
+      feature: "send",
       maxRun: 1,
       transaction: ({ account, siblings, bridge, maxSpendable }) => {
         invariant(maxSpendable.gt(minBalanceRequired), "balance is too low");
@@ -125,6 +127,7 @@ const cardano: AppSpec<Transaction> = {
     },
     {
       name: "send max",
+      feature: "sendMax",
       maxRun: 1,
       testDestination: genericTestDestination,
       transaction: ({ account, siblings, bridge, maxSpendable }) => {
@@ -163,6 +166,7 @@ const cardano: AppSpec<Transaction> = {
     },
     {
       name: "delegate to pool",
+      feature: "staking",
       maxRun: 1,
       transaction: ({ account, bridge, maxSpendable }) => {
         invariant(maxSpendable.gte(minBalanceRequiredForDelegate), "balance is too low");
@@ -195,6 +199,7 @@ const cardano: AppSpec<Transaction> = {
     },
     {
       name: "redelegate to pool",
+      feature: "staking",
       maxRun: 1,
       transaction: ({ account, bridge, maxSpendable }) => {
         invariant(maxSpendable.gte(minBalanceRequiredForDelegate), "balance is too low");
@@ -225,6 +230,7 @@ const cardano: AppSpec<Transaction> = {
     },
     {
       name: "undelegate",
+      feature: "staking",
       maxRun: 1,
       transaction: ({ account, bridge, maxSpendable }) => {
         invariant(maxSpendable.gte(minBalanceRequiredForDelegate), "balance is too low");

diff --git a/libs/coin-modules/coin-cosmos/src/specs.ts b/libs/coin-modules/coin-cosmos/src/specs.ts
@@ -174,6 +174,7 @@ function cosmosLikeMutations(minimalTransactionAmount: BigNumber): MutationSpec<
     },
     {
       name: "delegate new validators",
+      feature: "staking",
       maxRun: 1,
       transaction: ({ account, bridge, siblings }) => {
         expectSiblingsHaveSpendablePartGreaterThan(siblings, 0.5);
@@ -242,7 +243,8 @@ function cosmosLikeMutations(minimalTransactionAmount: BigNumber): MutationSpec<
     },
     {
       name: "undelegate",
-      maxRun: 5,
+      feature: "staking",
+      maxRun: 1,
       transaction: ({ account, bridge, maxSpendable }) => {
         invariant(canUndelegate(account as CosmosAccount), "can undelegate");
         const { cosmosResources } = account as CosmosAccount;
@@ -307,6 +309,7 @@ function cosmosLikeMutations(minimalTransactionAmount: BigNumber): MutationSpec<
     },
     {
       name: "redelegate",
+      feature: "staking",
       maxRun: 1,
       transaction: ({ account, bridge, maxSpendable }) => {
         const { cosmosResources } = account as CosmosAccount;

diff --git a/libs/coin-modules/coin-elrond/src/specs.ts b/libs/coin-modules/coin-elrond/src/specs.ts
@@ -170,6 +170,7 @@ const elrond: AppSpec<Transaction> = {
   mutations: [
     {
       name: "send 50%~",
+      feature: "send",
       maxRun: 1,
       deviceAction: acceptMoveBalanceTransaction,
       transaction: ({ account, siblings, bridge }) => {
@@ -204,6 +205,7 @@ const elrond: AppSpec<Transaction> = {
     },
     {
       name: "send max",
+      feature: "sendMax",
       maxRun: 1,
       deviceAction: acceptMoveBalanceTransaction,
       transaction: ({ account, siblings, bridge }) => {
@@ -236,6 +238,7 @@ const elrond: AppSpec<Transaction> = {
     },
     {
       name: "move some ESDT",
+      feature: "tokens",
       maxRun: 1,
       deviceAction: acceptEsdtTransferTransaction,
       transaction: ({ account, siblings, bridge }) => {
@@ -273,6 +276,7 @@ const elrond: AppSpec<Transaction> = {
     },
     {
       name: "delegate 1 EGLD",
+      feature: "staking",
       maxRun: 1,
       deviceAction: acceptDelegateTransaction,
       transaction: ({ account, bridge }) => {
@@ -301,6 +305,7 @@ const elrond: AppSpec<Transaction> = {
     },
     {
       name: "unDelegate 1 EGLD",
+      feature: "staking",
       maxRun: 1,
       deviceAction: acceptUndelegateTransaction,
       transaction: ({ account, bridge }) => {
@@ -331,6 +336,7 @@ const elrond: AppSpec<Transaction> = {
     },
     {
       name: "withdraw all EGLD",
+      feature: "staking",
       maxRun: 1,
       deviceAction: acceptWithdrawTransaction,
       transaction: ({ account, bridge }) => {

diff --git a/libs/coin-modules/coin-evm/src/specs.ts b/libs/coin-modules/coin-evm/src/specs.ts
@@ -225,7 +225,8 @@ const evmBasicMutations: ({
 }) => MutationSpec<EvmTransaction>[] = ({ maxAccount }) => [
   {
     name: "move 50%",
-    maxRun: 2,
+    feature: "send",
+    maxRun: 1,
     testDestination: testCoinDestination,
     transaction: ({ account, siblings, bridge, maxSpendable }): TransactionRes<EvmTransaction> => {
       const sibling = pickSiblings(siblings, maxAccount);
@@ -266,6 +267,7 @@ const evmBasicMutations: ({
   },
   {
     name: "send max",
+    feature: "sendMax",
     maxRun: 1,
     testDestination: testCoinDestination,
     transaction: ({ account, siblings, bridge }): TransactionRes<EvmTransaction> => {
@@ -311,6 +313,7 @@ const evmBasicMutations: ({
 
 const moveErc20Mutation: MutationSpec<EvmTransaction> = {
   name: "move some ERC20 like (ERC20, BEP20, etc...)",
+  feature: "tokens",
   maxRun: 1,
   testDestination: testTokenDestination,
   transaction: ({ account, siblings, bridge }): TransactionRes<EvmTransaction> => {

diff --git a/libs/coin-modules/coin-filecoin/src/test/bot-specs.ts b/libs/coin-modules/coin-filecoin/src/test/bot-specs.ts
@@ -30,6 +30,7 @@ const filecoinSpecs: AppSpec<Transaction> = {
   mutations: [
     {
       name: "Send small to f4 address",
+      feature: "send",
       maxRun: 1,
       deviceAction: generateDeviceActionFlow(BotScenario.F4_RECIPIENT),
       testDestination: genericTestDestination,
@@ -51,6 +52,7 @@ const filecoinSpecs: AppSpec<Transaction> = {
     },
     {
       name: "Send small to eth address",
+      feature: "send",
       maxRun: 1,
       deviceAction: generateDeviceActionFlow(BotScenario.ETH_RECIPIENT),
       testDestination: genericTestDestination,
@@ -72,6 +74,7 @@ const filecoinSpecs: AppSpec<Transaction> = {
     },
     {
       name: "Send 50%~",
+      feature: "send",
       maxRun: 1,
       testDestination: genericTestDestination,
       transaction: ({ account, siblings, bridge }) => {
@@ -98,6 +101,7 @@ const filecoinSpecs: AppSpec<Transaction> = {
     },
     {
       name: "Transfer Max",
+      feature: "sendMax",
       maxRun: 1,
       transaction: ({ account, siblings, bridge }) => {
         return {
@@ -115,6 +119,7 @@ const filecoinSpecs: AppSpec<Transaction> = {
     },
     {
       name: "Send ~50% WFIL",
+      feature: "tokens",
       maxRun: 1,
       deviceAction: generateDeviceActionFlow(BotScenario.TOKEN_TRANSFER),
       transaction: ({ account, bridge, maxSpendable }) => {

diff --git a/libs/coin-modules/coin-hedera/src/test/bot-specs.ts b/libs/coin-modules/coin-hedera/src/test/bot-specs.ts
@@ -49,7 +49,8 @@ const hedera: AppSpec<Transaction> = {
   mutations: [
     {
       name: "Send ~50%",
-      maxRun: 2,
+      feature: "send",
+      maxRun: 1,
       testDestination: genericTestDestination,
       transaction: ({
         account,
@@ -83,7 +84,8 @@ const hedera: AppSpec<Transaction> = {
     },
     {
       name: "Send max",
-      maxRun: 2,
+      feature: "sendMax",
+      maxRun: 1,
       testDestination: genericTestDestination,
       transaction: ({
         account,
@@ -109,7 +111,8 @@ const hedera: AppSpec<Transaction> = {
     },
     {
       name: "Memo",
-      maxRun: 2,
+      feature: "send",
+      maxRun: 1,
       transaction: ({
         account,
         siblings,

diff --git a/libs/coin-modules/coin-icon/src/specs.ts b/libs/coin-modules/coin-icon/src/specs.ts
@@ -57,6 +57,7 @@ const icon: AppSpec<Transaction> = {
   mutations: [
     {
       name: "send 50%~",
+      feature: "send",
       maxRun: 1,
       transaction: ({ account, siblings, bridge }) => {
         invariant(account.spendableBalance.gt(0), "balance is 0");
@@ -94,6 +95,7 @@ const icon: AppSpec<Transaction> = {
     },
     {
       name: "send max",
+      feature: "sendMax",
       maxRun: 1,
       transaction: ({ account, siblings, bridge }) => {
         invariant(account.spendableBalance.gt(0), "balance is 0");

diff --git a/libs/coin-modules/coin-internet_computer/src/test/bot-specs.ts b/libs/coin-modules/coin-internet_computer/src/test/bot-specs.ts
@@ -28,6 +28,7 @@ const internetComputerSpecs: AppSpec<Transaction> = {
   mutations: [
     {
       name: "Send ~50%",
+      feature: "send",
       maxRun: 1,
       testDestination: genericTestDestination,
       transaction: ({ account, siblings, bridge, maxSpendable }) => {
@@ -74,6 +75,7 @@ const internetComputerSpecs: AppSpec<Transaction> = {
     },
     {
       name: "Transfer Max",
+      feature: "sendMax",
       maxRun: 1,
       testDestination: genericTestDestination,
       transaction: ({ account, siblings, bridge }) => {

diff --git a/libs/coin-modules/coin-near/src/specs.ts b/libs/coin-modules/coin-near/src/specs.ts
@@ -27,6 +27,7 @@ const near: AppSpec<Transaction> = {
   mutations: [
     {
       name: "Move 50% to another account",
+      feature: "send",
       maxRun: 1,
       transaction: ({ account, siblings, bridge, maxSpendable }) => {
         invariant(maxSpendable.gt(minimalAmount), "balance is too low");
@@ -48,6 +49,7 @@ const near: AppSpec<Transaction> = {
     },
     {
       name: "Send max to another account",
+      feature: "sendMax",
       maxRun: 1,
       transaction: ({ account, siblings, bridge, maxSpendable }) => {
         invariant(maxSpendable.gt(minimalAmount), "balance is too low");
@@ -66,6 +68,7 @@ const near: AppSpec<Transaction> = {
     },
     {
       name: "Stake",
+      feature: "staking",
       maxRun: 1,
       transaction: ({ account, bridge, maxSpendable }) => {
         invariant(maxSpendable.gt(minimalAmount.plus(stakingFee)), "balance is too low");
@@ -99,6 +102,7 @@ const near: AppSpec<Transaction> = {
     },
     {
       name: "Unstake",
+      feature: "staking",
       maxRun: 1,
       transaction: ({ account, bridge, maxSpendable }) => {
         invariant(maxSpendable.gt(stakingFee), "balance is too low for fees");
@@ -134,6 +138,7 @@ const near: AppSpec<Transaction> = {
     },
     {
       name: "Withdraw",
+      feature: "staking",
       maxRun: 1,
       transaction: ({ account, bridge, maxSpendable }) => {
         invariant(maxSpendable.gt(stakingFee), "balance is too low for fees");