@ledgerhq/coin-xrp 7.1.0-nightly.2 → 7.1.0

package/src/logic/getTransactionStatus.test.ts

@@ -1,14 +1,8 @@
 import { getTransactionStatus } from "./getTransactionStatus";
 import * as utils from "./utils";
 
-const mockGetBalance = jest.fn();
-
 const mockGetServerInfos = jest.fn();
 
-jest.mock("./getBalance", () => ({
-  getBalance: () => mockGetBalance(),
-}));
-
 jest.mock("../network", () => ({
   getServerInfos: () => mockGetServerInfos(),
 }));
@@ -26,9 +20,21 @@ const SENDER = "rPSCfmnX3t9jQJG5RNcZtSaP5UhExZDue4";
 const RECIPIENT = "rPT1Sjq2YGrBMTttX4GZHjKu9dyfzbpAYe";
 const RECIPIENT_NEW = "rDKsbvy9uaNpPtvVFraJyNGfjvTw8xivgK";
 
+const account = {
+  address: SENDER,
+  balance: 50_000_000n,
+  currencyUnit: {
+    code: "XRP",
+    magnitude: 6,
+    name: "XRP",
+    symbol: "XRP",
+  },
+  currencyName: "XRP",
+};
+
 describe("getTransactionStatus", () => {
   afterEach(() => {
-    jest.clearAllMocks();
+    mockGetServerInfos.mockReset();
   });
 
   it("returns no errors on valid transaction", async () => {
@@ -39,22 +45,13 @@ describe("getTransactionStatus", () => {
         },
       },
     });
-    mockGetBalance.mockResolvedValue([
-      {
-        value: 50_000_000n,
-        asset: { type: "native" },
-        locked: 0n,
-      },
-    ]);
 
     const result = await getTransactionStatus(
-      // account as any,
+      account as any,
       {
-        sender: SENDER,
         amount: 20_000_000n,
-        fees: 10_000n,
+        fee: 10_000n,
         recipient: RECIPIENT,
-        asset: { unit: { code: "XRP", magnitude: 6 } },
       } as any,
     );
 
@@ -71,22 +68,13 @@ describe("getTransactionStatus", () => {
         },
       },
     });
-    mockGetBalance.mockResolvedValue([
-      {
-        value: 50_000_000n,
-        asset: { type: "native" },
-        locked: 0n,
-      },
-    ]);
 
     const result = await getTransactionStatus(
-      // account as any,
+      account as any,
       {
-        sender: SENDER,
         amount: 1_000_000n,
-        fees: 200_000n, // 20%
+        fee: 200_000n, // 20%
         recipient: RECIPIENT,
-        asset: { unit: { code: "XRP", magnitude: 6 } },
       } as any,
     );
 
@@ -102,21 +90,12 @@ describe("getTransactionStatus", () => {
         },
       },
     });
-    mockGetBalance.mockResolvedValue([
-      {
-        value: 30_000_000n,
-        asset: { type: "native" },
-        locked: 0n,
-      },
-    ]);
 
     const result = await getTransactionStatus(
-      // account as any,
+      account as any,
       {
-        sender: SENDER,
         amount: 10_000_000n,
         recipient: RECIPIENT,
-        asset: { unit: { code: "XRP", magnitude: 6 } },
       } as any,
     );
 
@@ -131,22 +110,13 @@ describe("getTransactionStatus", () => {
         },
       },
     });
-    mockGetBalance.mockResolvedValue([
-      {
-        value: 50_000_000n,
-        asset: { type: "native" },
-        locked: 0n,
-      },
-    ]);
 
     const result = await getTransactionStatus(
-      // account as any,
+      account as any,
       {
-        sender: SENDER,
         amount: 10_000_000n,
-        fees: 10_000n,
+        fee: 10_000n,
         recipient: SENDER,
-        asset: { unit: { code: "XRP", magnitude: 6 } },
       } as any,
     );
 
@@ -161,22 +131,13 @@ describe("getTransactionStatus", () => {
         },
       },
     });
-    mockGetBalance.mockResolvedValue([
-      {
-        value: 50_000_000n,
-        asset: { type: "native" },
-        locked: 0n,
-      },
-    ]);
 
     const result = await getTransactionStatus(
-      // account as any,
+      account as any,
       {
-        sender: SENDER,
         amount: 5_000_000n,
-        fees: 10_000n,
+        fee: 10_000n,
         recipient: RECIPIENT_NEW,
-        asset: { unit: { code: "XRP", magnitude: 6 } },
       } as any,
     );
 
@@ -191,22 +152,13 @@ describe("getTransactionStatus", () => {
         },
       },
     });
-    mockGetBalance.mockResolvedValue([
-      {
-        value: 50_000_000n,
-        asset: { type: "native" },
-        locked: 0n,
-      },
-    ]);
 
     const result = await getTransactionStatus(
-      // account as any,
+      account as any,
       {
-        sender: SENDER,
         amount: 0n,
-        fees: 10_000n,
+        fee: 10_000n,
         recipient: RECIPIENT,
-        asset: { unit: { code: "XRP", magnitude: 6 } },
       } as any,
     );
 
@@ -221,21 +173,12 @@ describe("getTransactionStatus", () => {
         },
       },
     });
-    mockGetBalance.mockResolvedValue([
-      {
-        value: 50_000_000n,
-        asset: { type: "native" },
-        locked: 0n,
-      },
-    ]);
 
     const result = await getTransactionStatus(
-      // account as any,
+      account as any,
       {
-        sender: SENDER,
-        asset: { unit: { code: "XRP", magnitude: 6 } },
         amount: 1_000_000n,
-        fees: 10_000n,
+        fee: 10_000n,
         recipient: "not-an-address",
       } as any,
     );
@@ -251,21 +194,12 @@ describe("getTransactionStatus", () => {
         },
       },
     });
-    mockGetBalance.mockResolvedValue([
-      {
-        value: 50_000_000n,
-        asset: { type: "native" },
-        locked: 0n,
-      },
-    ]);
 
     const result = await getTransactionStatus(
-      // account as any,
+      account as any,
       {
-        sender: SENDER,
-        asset: { unit: { code: "XRP", magnitude: 6 } },
         amount: 1_000_000n,
-        fees: 10_000n,
+        fee: 10_000n,
         recipient: "",
       } as any,
     );

package/src/logic/getTransactionStatus.ts

@@ -14,12 +14,11 @@ import { formatCurrencyUnit } from "@ledgerhq/coin-framework/currencies/index";
 import { getServerInfos } from "../network";
 import { cachedRecipientIsNew } from "./utils";
 import { parseAPIValue } from "./common";
-import { TransactionValidation, TransactionIntent } from "@ledgerhq/coin-framework/api/types";
-import { XrpMapMemo } from "../types";
-import { getBalance } from "./getBalance";
+import { Transaction, TransactionValidation, Account } from "@ledgerhq/coin-framework/api/types";
 
 export const getTransactionStatus = async (
-  transactionIntent: TransactionIntent<XrpMapMemo>,
+  account: Account,
+  transaction: Transaction,
 ): Promise<TransactionValidation> => {
   const errors: Record<string, Error> = {};
   const warnings: Record<string, Error> = {};
@@ -27,57 +26,47 @@ export const getTransactionStatus = async (
   const reserveBaseXRP = parseAPIValue(
     serverInfos.info.validated_ledger.reserve_base_xrp.toString(),
   );
-  const estimatedFees = transactionIntent.fees || 0n;
-  const totalSpent = transactionIntent.amount + estimatedFees;
-  const amount = transactionIntent.amount;
+  const estimatedFees = transaction.fee || 0n;
+  const totalSpent = transaction.amount + estimatedFees;
+  const amount = transaction.amount;
 
   if (amount > 0 && estimatedFees * 10n > amount) {
     warnings.feeTooHigh = new FeeTooHigh();
   }
 
-  const balances = await getBalance(transactionIntent.sender);
-  const nativeBalance = balances.find(b => b.asset.type === "native");
-  if (nativeBalance === undefined) {
-    throw Error("Shouldn't happen");
-  }
-
-  if (!transactionIntent.fees) {
+  if (!transaction.fee) {
     errors.fee = new FeeNotLoaded();
-  } else if (transactionIntent.fees == 0n) {
+  } else if (transaction.fee == 0n) {
     errors.fee = new FeeRequired();
-  } else if (totalSpent > nativeBalance.value - BigInt(reserveBaseXRP.toString())) {
+  } else if (totalSpent > account.balance - BigInt(reserveBaseXRP.toString())) {
     errors.amount = new NotEnoughSpendableBalance("", {
-      minimumAmount: transactionIntent.asset.unit
-        ? formatCurrencyUnit(transactionIntent.asset.unit, reserveBaseXRP, {
-            disableRounding: true,
-            useGrouping: false,
-            showCode: true,
-          })
-        : "Unknown unit",
+      minimumAmount: formatCurrencyUnit(account.currencyUnit, reserveBaseXRP, {
+        disableRounding: true,
+        useGrouping: false,
+        showCode: true,
+      }),
     });
   } else if (
-    transactionIntent.recipient &&
-    (await cachedRecipientIsNew(transactionIntent.recipient)) &&
-    transactionIntent.amount < BigInt(reserveBaseXRP.toString())
+    transaction.recipient &&
+    (await cachedRecipientIsNew(transaction.recipient)) &&
+    transaction.amount < BigInt(reserveBaseXRP.toString())
   ) {
     errors.amount = new NotEnoughBalanceBecauseDestinationNotCreated("", {
-      minimalAmount: transactionIntent.asset.unit
-        ? formatCurrencyUnit(transactionIntent.asset.unit, reserveBaseXRP, {
-            disableRounding: true,
-            useGrouping: false,
-            showCode: true,
-          })
-        : "Unknown unit",
+      minimalAmount: formatCurrencyUnit(account.currencyUnit, reserveBaseXRP, {
+        disableRounding: true,
+        useGrouping: false,
+        showCode: true,
+      }),
     });
   }
 
-  if (!transactionIntent.recipient) {
+  if (!transaction.recipient) {
     errors.recipient = new RecipientRequired("");
-  } else if (transactionIntent.sender === transactionIntent.recipient) {
+  } else if (account.address === transaction.recipient) {
     errors.recipient = new InvalidAddressBecauseDestinationIsAlsoSource();
-  } else if (!isValidClassicAddress(transactionIntent.recipient)) {
+  } else if (!isValidClassicAddress(transaction.recipient)) {
     errors.recipient = new InvalidAddress("", {
-      currencyName: transactionIntent.asset.name ?? "",
+      currencyName: account.currencyName,
     });
   }
 

package/src/logic/listOperations.test.ts

@@ -146,7 +146,7 @@ describe("listOperations", () => {
     async ({ address, opSender, opDestination, expectedType }) => {
       // Given
       const deliveredAmount = 100;
-      const fees = 10;
+      const fee = 10;
       mockNetworkGetTransactions.mockResolvedValue(
         mockNetworkTxs([
           {
@@ -156,7 +156,7 @@ describe("listOperations", () => {
             meta: { delivered_amount: deliveredAmount.toString() },
             tx_json: {
               TransactionType: "Payment",
-              Fee: fees.toString(),
+              Fee: fee.toString(),
               ledger_index: 1,
               date: 1000,
               Account: opSender,
@@ -172,7 +172,7 @@ describe("listOperations", () => {
             meta: { delivered_amount: deliveredAmount.toString() },
             tx_json: {
               TransactionType: "Payment",
-              Fee: fees.toString(),
+              Fee: fee.toString(),
               ledger_index: 1,
               date: 1000,
               Account: opSender,
@@ -189,7 +189,7 @@ describe("listOperations", () => {
             meta: { delivered_amount: deliveredAmount.toString() },
             tx_json: {
               TransactionType: "Payment",
-              Fee: fees.toString(),
+              Fee: fee.toString(),
               ledger_index: 1,
               date: 1000,
               Account: opSender,
@@ -217,7 +217,7 @@ describe("listOperations", () => {
       expect(mockNetworkGetTransactions).toHaveBeenCalledTimes(1);
       // if expectedType is "OUT", compute value with fees (i.e. delivered_amount + Fee)
       const expectedValue =
-        expectedType === "IN" ? BigInt(deliveredAmount) : BigInt(deliveredAmount + fees);
+        expectedType === "IN" ? BigInt(deliveredAmount) : BigInt(deliveredAmount + fee);
       expect(results).toEqual([
         {
           id: "HASH_VALUE",

package/src/logic/listOperations.ts

@@ -125,10 +125,10 @@ const convertToCoreOperation =
         ? BigInt(delivered_amount)
         : BigInt(0);
 
-    const fees = BigInt(Fee);
+    const fee = BigInt(Fee);
     if (type === "OUT") {
-      if (!Number.isNaN(fees)) {
-        value = value + fees;
+      if (!Number.isNaN(fee)) {
+        value = value + fee;
       }
     }
 
@@ -170,7 +170,7 @@ const convertToCoreOperation =
       asset: { type: "native" },
       tx: {
         hash: hash,
-        fees: fees,
+        fees: fee,
         date: new Date(toEpochDate),
         block: {
           time: new Date(close_time_iso),

package/src/test/bridgeDatasetTest.ts

@@ -40,7 +40,7 @@ export const dataset: DatasetTest<Transaction> = {
             recipient: "rB6pwovsyrFWhPYUsjj9V3CHck985QjiXi",
             amount: "15000000",
             tag: null,
-            fees: "1",
+            fee: "1",
             feeCustomUnit: null,
             networkInfo: null,
           }),
@@ -75,7 +75,7 @@ export const dataset: DatasetTest<Transaction> = {
             recipient: newAddress1,
             amount: "10000000",
             tag: null,
-            fees: "1",
+            fee: "1",
             feeCustomUnit: null,
             networkInfo: null
           }),
@@ -97,7 +97,7 @@ export const dataset: DatasetTest<Transaction> = {
                 recipient: "rageXHB6Q4VbvvWdTzKANwjeCT4HXFCKX7",
                 amount: "10000000",
                 tag: null,
-                fees: "10", // NOTE: fee is not customizable, this field is ignored
+                fee: "10", // NOTE: fee is not customizable, this field is ignored
                 feeCustomUnit: null,
                 networkInfo: null,
               }),
@@ -118,7 +118,7 @@ export const dataset: DatasetTest<Transaction> = {
                 recipient: "rB6pwovsyrFWhPYUsjj9V3CHck985QjiXi",
                 amount: "10000000",
                 tag: 12345,
-                fees: "10", // NOTE: fee is not customizable, this field is ignored
+                fee: "10", // NOTE: fee is not customizable, this field is ignored
                 feeCustomUnit: null,
                 networkInfo: null,
               }),

package/src/test/cli.ts

@@ -30,7 +30,7 @@ function inferTransactions(
     invariant(transaction.family === "xrp", "XRP family");
     return {
       ...transaction,
-      fees: inferAmount(account, opts.fee || "0.001xrp"),
+      fee: inferAmount(account, opts.fee || "0.001xrp"),
       tag: opts.tag,
     };
   });

package/src/transaction.ts

@@ -12,7 +12,7 @@ import { getAccountCurrency } from "@ledgerhq/coin-framework/account/index";
 import { formatCurrencyUnit } from "@ledgerhq/coin-framework/currencies/index";
 
 export const formatTransaction = (
-  { amount, recipient, fees, tag, useAllAmount }: Transaction,
+  { amount, recipient, fee, tag, useAllAmount }: Transaction,
   account: Account,
 ): string => `
 SEND ${
@@ -25,9 +25,9 @@ SEND ${
 }
 TO ${recipient}
 with fee=${
-  !fees
+  !fee
     ? "?"
-    : formatCurrencyUnit(getAccountCurrency(account).units[0], fees, {
+    : formatCurrencyUnit(getAccountCurrency(account).units[0], fee, {
         showCode: true,
         disableRounding: true,
       })
@@ -40,7 +40,7 @@ export const fromTransactionRaw = (tr: TransactionRaw): Transaction => {
     ...common,
     family: tr.family,
     tag: tr.tag,
-    fees: tr.fees ? new BigNumber(tr.fees) : null,
+    fee: tr.fee ? new BigNumber(tr.fee) : null,
     feeCustomUnit: tr.feeCustomUnit,
     // FIXME remove this field. this is not good.. we're dereferencing here. we should instead store an index (to lookup in currency.units on UI)
     networkInfo: networkInfo && {
@@ -58,7 +58,7 @@ export const toTransactionRaw = (t: Transaction): TransactionRaw => {
     ...common,
     family: t.family,
     tag: t.tag,
-    fees: t.fees ? t.fees.toString() : null,
+    fee: t.fee ? t.fee.toString() : null,
     feeCustomUnit: t.feeCustomUnit,
     // FIXME remove this field. this is not good.. we're dereferencing here. we should instead store an index (to lookup in currency.units on UI)
     networkInfo: networkInfo && {

package/src/types/bridge.ts

@@ -21,7 +21,7 @@ export type NetworkInfoRaw = {
 
 export type Transaction = TransactionCommon & {
   family: "xrp";
-  fees: BigNumber | null | undefined;
+  fee: BigNumber | null | undefined;
   networkInfo: NetworkInfo | null | undefined;
   tag: number | null | undefined;
   feeCustomUnit: Unit | null | undefined;
@@ -29,7 +29,7 @@ export type Transaction = TransactionCommon & {
 
 export type TransactionRaw = TransactionCommonRaw & {
   family: "xrp";
-  fees: string | null | undefined;
+  fee: string | null | undefined;
   networkInfo: NetworkInfoRaw | null | undefined;
   tag: number | null | undefined;
   feeCustomUnit: Unit | null | undefined;