@ledgerhq/coin-tron 3.0.1-nightly.0 → 3.1.0-nightly.1

package/src/logic/listOperations.unit.test.ts

@@ -2,7 +2,7 @@ import BigNumber from "bignumber.js";
 import { defaultFetchParams, fetchTronAccountTxs, getBlock } from "../network";
 import { fromTrongridTxInfoToOperation } from "../network/trongrid/trongrid-adapters";
 import { TrongridTxInfo, TronAsset } from "../types";
-import { listOperations } from "./listOperations";
+import { defaultOptions, listOperations } from "./listOperations";
 import type { Operation } from "@ledgerhq/coin-framework/api/index";
 
 // Mock the fetchTronAccountTxs and fromTrongridTxInfoToOperation functions
@@ -30,6 +30,8 @@ describe("listOperations", () => {
   const expectedFetchParams = {
     ...defaultFetchParams,
     minTimestamp: mockBlockTimestamp,
+    hintGlobalLimit: 1000,
+    order: "desc",
   };
 
   it("should fetch transactions and return operations", async () => {
@@ -51,7 +53,7 @@ describe("listOperations", () => {
       };
     });
 
-    const [operations, token] = await listOperations(mockAddress, 0);
+    const [operations, token] = await listOperations(mockAddress, defaultOptions);
 
     expect(fetchTronAccountTxs).toHaveBeenCalledWith(
       mockAddress,
@@ -71,7 +73,7 @@ describe("listOperations", () => {
     (fetchTronAccountTxs as jest.Mock).mockResolvedValue(mockTxs);
     (fromTrongridTxInfoToOperation as jest.Mock).mockImplementation(() => null);
 
-    const [operations, token] = await listOperations(mockAddress, 0);
+    const [operations, token] = await listOperations(mockAddress, defaultOptions);
 
     expect(fetchTronAccountTxs).toHaveBeenCalledWith(
       mockAddress,
@@ -87,6 +89,8 @@ describe("listOperations", () => {
     const exampleError = new Error("Network error!");
     (fetchTronAccountTxs as jest.Mock).mockRejectedValue(exampleError);
 
-    await expect(listOperations(mockAddress, 0)).rejects.toThrow(new Error(exampleError.message));
+    await expect(listOperations(mockAddress, defaultOptions)).rejects.toThrow(
+      new Error(exampleError.message),
+    );
   });
 });

package/src/network/index.ts

@@ -483,13 +483,18 @@ export type FetchTxsStopPredicate = (
 ) => boolean;
 
 export type FetchParams = {
-  limit: number;
+  /** The maximum number of transactions to fetch per call. */
+  limitPerCall: number;
+  /** Hint about the number of transactions to be fetched in total (hint to optimize `limitPerCall`) */
+  hintGlobalLimit?: number;
   minTimestamp: number;
+  order: "asc" | "desc";
 };
 
 export const defaultFetchParams: FetchParams = {
-  limit: 100,
+  limitPerCall: 100,
   minTimestamp: 0,
+  order: "desc",
 } as const;
 
 export async function fetchTronAccountTxs(
@@ -498,12 +503,15 @@ export async function fetchTronAccountTxs(
   cacheTransactionInfoById: Record<string, TronTransactionInfo>,
   params: FetchParams,
 ): Promise<TrongridTxInfo[]> {
-  const queryParamsNativeTxs = `limit=${params.limit}&min_timestamp=${params.minTimestamp}`;
+  const adjustedLimitPerCall = params.hintGlobalLimit
+    ? Math.min(params.limitPerCall, params.hintGlobalLimit)
+    : params.limitPerCall;
+  const queryParams = `limit=${adjustedLimitPerCall}&min_timestamp=${params.minTimestamp}&order_by=block_timestamp,${params.order}`;
   const nativeTxs = (
     await getAllTransactions<
       (TransactionTronAPI & { detail?: TronTransactionInfo }) | MalformedTransactionTronAPI
     >(
-      `${getBaseApiUrl()}/v1/accounts/${addr}/transactions?${queryParamsNativeTxs}`,
+      `${getBaseApiUrl()}/v1/accounts/${addr}/transactions?${queryParams}`,
       shouldFetchMoreTxs,
       getTransactions(cacheTransactionInfoById),
     )
@@ -528,10 +536,9 @@ export async function fetchTronAccountTxs(
 
   // we need to fetch and filter trc20 transactions from another endpoint
   // doc https://developers.tron.network/reference/get-trc20-transaction-info-by-account-address
-  const queryParamsTrc20Txs = `limit=${params.limit}&min_timestamp=${params.minTimestamp}`;
   const trc20Txs = (
     await getAllTransactions<Trc20API>(
-      `${getBaseApiUrl()}/v1/accounts/${addr}/transactions/trc20?${queryParamsTrc20Txs}&get_detail=true`,
+      `${getBaseApiUrl()}/v1/accounts/${addr}/transactions/trc20?${queryParams}&get_detail=true`,
       shouldFetchMoreTxs,
       getTrc20,
     )