release/5.36.0

@shared/api/__tests__/internal.test.ts

@@ -0,0 +1,90 @@
+import { TESTNET_NETWORK_DETAILS } from "@shared/constants/stellar";
+import * as GetLedgerKeyAccounts from "../helpers/getLedgerKeyAccounts";
+import * as internalApi from "../internal";
+
+describe("internalApi", () => {
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+  describe("getAssetDomains", () => {
+    it("should return a list of domains from a list of issuers", async () => {
+      jest
+        .spyOn(GetLedgerKeyAccounts, "getLedgerKeyAccounts")
+        .mockResolvedValue({
+          G1: {
+            account_id:
+              "GDF32CQINROD3E2LMCGZUDVMWTXCJFR5SBYVRJ7WAAIAS3P7DCVWZEFY",
+            home_domain: "stellar1.org",
+            balance: "1000000000000000000",
+            seq_num: 1,
+            num_sub_entries: 1,
+            inflation_dest: "G1",
+            flags: 1,
+            thresholds: "1000000000000000000",
+            signers: [
+              {
+                key: "GDF32CQINROD3E2LMCGZUDVMWTXCJFR5SBYVRJ7WAAIAS3P7DCVWZEFY",
+                weight: 1,
+              },
+            ],
+            sequence_number: 1,
+          },
+          G2: {
+            account_id:
+              "GBKWMR7TJ7BBICOOXRY2SWXKCWPTOHZPI6MP4LNNE5A73VP3WADGG3CH",
+            home_domain: "stellar2.org",
+            balance: "1000000000000000000",
+            seq_num: 1,
+            num_sub_entries: 1,
+            inflation_dest:
+              "GBKWMR7TJ7BBICOOXRY2SWXKCWPTOHZPI6MP4LNNE5A73VP3WADGG3CH",
+            flags: 1,
+            thresholds: "1000000000000000000",
+            signers: [
+              {
+                key: "GBKWMR7TJ7BBICOOXRY2SWXKCWPTOHZPI6MP4LNNE5A73VP3WADGG3CH",
+                weight: 1,
+              },
+            ],
+            sequence_number: 1,
+          },
+        });
+      const assetDomains = await internalApi.getAssetDomains({
+        assetIssuerDomainsToFetch: [
+          "GDF32CQINROD3E2LMCGZUDVMWTXCJFR5SBYVRJ7WAAIAS3P7DCVWZEFY",
+          "GBKWMR7TJ7BBICOOXRY2SWXKCWPTOHZPI6MP4LNNE5A73VP3WADGG3CH",
+        ],
+        networkDetails: TESTNET_NETWORK_DETAILS,
+      });
+
+      expect(assetDomains).toEqual({
+        G1: "stellar1.org",
+        G2: "stellar2.org",
+      });
+    });
+    it("should return an empty object if the fetch fails", async () => {
+      jest
+        .spyOn(GetLedgerKeyAccounts, "getLedgerKeyAccounts")
+        .mockRejectedValue(new Error("Fetch failed"));
+      const assetDomains = await internalApi.getAssetDomains({
+        assetIssuerDomainsToFetch: ["G1", "G2"],
+        networkDetails: TESTNET_NETWORK_DETAILS,
+      });
+      expect(assetDomains).toEqual({});
+    });
+    it("should return an empty object if not valid public keys are provided", async () => {
+      const getLedgerKeyAccountsSpy = jest.spyOn(
+        GetLedgerKeyAccounts,
+        "getLedgerKeyAccounts",
+      );
+      const assetDomains = await internalApi.getAssetDomains({
+        assetIssuerDomainsToFetch: [
+          "CAZXRTOKNUQ2JQQF3NCRU7GYMDJNZ2NMQN6IGN4FCT5DWPODMPVEXSND",
+        ],
+        networkDetails: TESTNET_NETWORK_DETAILS,
+      });
+      expect(getLedgerKeyAccountsSpy).not.toHaveBeenCalled();
+      expect(assetDomains).toEqual({});
+    });
+  });
+});

@shared/api/helpers/__tests__/getIconUrlFromIssuer.test.ts

@@ -0,0 +1,93 @@
+import { TESTNET_NETWORK_DETAILS } from "@shared/constants/stellar";
+import { getIconUrlFromIssuer } from "../getIconUrlFromIssuer";
+import * as ExtensionMessaging from "@shared/api/helpers/extensionMessaging";
+import * as ApiInternal from "@shared/api/internal";
+
+jest.mock("stellar-sdk", () => {
+  const original = jest.requireActual("stellar-sdk");
+  return {
+    ...original,
+    StellarToml: {
+      Resolver: {
+        resolve: jest.fn().mockResolvedValue({
+          CURRENCIES: [
+            {
+              code: "USDC",
+              issuer:
+                "GBKWMR7TJ7BBICOOXRY2SWXKCWPTOHZPI6MP4LNNE5A73VP3WADGG3CH",
+              image: "http://tomldomain.com/baz.png",
+            },
+          ],
+        }),
+      },
+    },
+  };
+});
+
+describe("getIconUrlFromIssuer", () => {
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+  it("should return the icon url for a given issuer from the background cache", async () => {
+    jest
+      .spyOn(ExtensionMessaging, "sendMessageToBackground")
+      .mockImplementationOnce(() =>
+        Promise.resolve({
+          iconUrl: "http://bgdomain.com/baz.png",
+        } as any),
+      );
+    const iconUrl = await getIconUrlFromIssuer({
+      key: "GBKWMR7TJ7BBICOOXRY2SWXKCWPTOHZPI6MP4LNNE5A73VP3WADGG3CH",
+      code: "USDC",
+      networkDetails: TESTNET_NETWORK_DETAILS,
+    });
+    expect(iconUrl).toBe("http://bgdomain.com/baz.png");
+  });
+  it("should return the icon url for a given issuer from a passed home domain", async () => {
+    // mock both calls to the background
+    jest
+      .spyOn(ExtensionMessaging, "sendMessageToBackground")
+      .mockImplementationOnce(() => Promise.resolve({} as any));
+    jest
+      .spyOn(ExtensionMessaging, "sendMessageToBackground")
+      .mockImplementationOnce(() => Promise.resolve({} as any));
+    const iconUrl = await getIconUrlFromIssuer({
+      key: "GBKWMR7TJ7BBICOOXRY2SWXKCWPTOHZPI6MP4LNNE5A73VP3WADGG3CH",
+      code: "USDC",
+      networkDetails: TESTNET_NETWORK_DETAILS,
+      homeDomain: "http://home.com/baz.png",
+    });
+    expect(iconUrl).toBe("http://tomldomain.com/baz.png");
+  });
+  it("should return the icon url for a given issuer from a passed home domain", async () => {
+    // mock both calls to the background
+    jest
+      .spyOn(ExtensionMessaging, "sendMessageToBackground")
+      .mockImplementationOnce(() => Promise.resolve({} as any));
+    jest
+      .spyOn(ExtensionMessaging, "sendMessageToBackground")
+      .mockImplementationOnce(() => Promise.resolve({} as any));
+
+    const getAssetDomainsSpy = jest
+      .spyOn(ApiInternal, "getAssetDomains")
+      .mockImplementationOnce(() =>
+        Promise.resolve({
+          GBKWMR7TJ7BBICOOXRY2SWXKCWPTOHZPI6MP4LNNE5A73VP3WADGG3CH:
+            "http://ledgerkeydomain.com/baz.png",
+        }),
+      );
+    const iconUrl = await getIconUrlFromIssuer({
+      key: "GBKWMR7TJ7BBICOOXRY2SWXKCWPTOHZPI6MP4LNNE5A73VP3WADGG3CH",
+      code: "USDC",
+      networkDetails: TESTNET_NETWORK_DETAILS,
+    });
+
+    expect(getAssetDomainsSpy).toHaveBeenCalledWith({
+      assetIssuerDomainsToFetch: [
+        "GBKWMR7TJ7BBICOOXRY2SWXKCWPTOHZPI6MP4LNNE5A73VP3WADGG3CH",
+      ],
+      networkDetails: TESTNET_NETWORK_DETAILS,
+    });
+    expect(iconUrl).toBe("http://tomldomain.com/baz.png");
+  });
+});

@shared/api/helpers/__tests__/getLedgerKeyAccounts.test.ts

@@ -0,0 +1,92 @@
+import { TESTNET_NETWORK_DETAILS } from "@shared/constants/stellar";
+import { getLedgerKeyAccounts } from "../getLedgerKeyAccounts";
+
+describe("getDomainFromIssuer", () => {
+  it("should return a list of domains from a list of issuers", async () => {
+    const fetchSpy = jest.spyOn(global, "fetch").mockImplementation(() =>
+      Promise.resolve({
+        ok: true,
+        json: () =>
+          Promise.resolve({
+            data: {
+              ledger_key_accounts: {
+                G1: { account_id: "G1", home_domain: "stellar1.org" },
+                g2: { account_id: "g2", home_domain: "stellar2.org" },
+              },
+            },
+          }),
+      } as any),
+    );
+    const ledgerKeyAccounts = await getLedgerKeyAccounts({
+      accountList: ["G1", "g2"],
+      networkDetails: TESTNET_NETWORK_DETAILS,
+    });
+
+    expect(fetchSpy).toHaveBeenCalledWith(
+      new URL(
+        "http://localhost:3003/api/v1/ledger-key/accounts?network=TESTNET",
+      ),
+      {
+        body: '{"public_keys":["G1","g2"]}',
+        headers: { "Content-Type": "application/json" },
+        method: "POST",
+      },
+    );
+
+    expect(ledgerKeyAccounts).toEqual({
+      G1: { account_id: "G1", home_domain: "stellar1.org" },
+      g2: { account_id: "g2", home_domain: "stellar2.org" },
+    });
+  });
+  it("should return an empty object if the fetch fails", async () => {
+    const fetchSpy = jest.spyOn(global, "fetch").mockImplementation(() =>
+      Promise.resolve({
+        ok: false,
+      } as any),
+    );
+    const ledgerKeyAccounts = await getLedgerKeyAccounts({
+      accountList: ["G1", "g2"],
+      networkDetails: TESTNET_NETWORK_DETAILS,
+    });
+
+    expect(fetchSpy).toHaveBeenCalledWith(
+      new URL(
+        "http://localhost:3003/api/v1/ledger-key/accounts?network=TESTNET",
+      ),
+      {
+        body: '{"public_keys":["G1","g2"]}',
+        headers: { "Content-Type": "application/json" },
+        method: "POST",
+      },
+    );
+
+    expect(ledgerKeyAccounts).toEqual({});
+  });
+
+  it("should return an empty object if the fetch returns an error", async () => {
+    const fetchSpy = jest.spyOn(global, "fetch").mockImplementation(() =>
+      Promise.resolve({
+        ok: true,
+        json: () => Promise.resolve({ error: "test" }),
+      } as any),
+    );
+
+    const ledgerKeyAccounts = await getLedgerKeyAccounts({
+      accountList: ["G1", "g2"],
+      networkDetails: TESTNET_NETWORK_DETAILS,
+    });
+
+    expect(fetchSpy).toHaveBeenCalledWith(
+      new URL(
+        "http://localhost:3003/api/v1/ledger-key/accounts?network=TESTNET",
+      ),
+      {
+        body: '{"public_keys":["G1","g2"]}',
+        headers: { "Content-Type": "application/json" },
+        method: "POST",
+      },
+    );
+
+    expect(ledgerKeyAccounts).toEqual({});
+  });
+});

@shared/api/helpers/getIconFromTokenList.ts

@@ -5,6 +5,9 @@ import {
 import { NetworkDetails } from "@shared/constants/stellar";
 import { getCanonicalFromAsset } from "@shared/helpers/stellar";
 
+import { sendMessageToBackground } from "./extensionMessaging";
+import { SERVICE_TYPES } from "../../constants/services";
+
 export const getIconFromTokenLists = async ({
   issuerId,
   contractId,
@@ -36,6 +39,7 @@ export const getIconFromTokenLists = async ({
           issuerId &&
           record.issuer &&
           record.issuer === issuerId &&
+          record.code === code &&
           record.icon
         ) {
           verifiedToken = record;
@@ -46,6 +50,15 @@ export const getIconFromTokenLists = async ({
     }
   }
 
+  if (verifiedToken?.icon) {
+    await sendMessageToBackground({
+      activePublicKey: null,
+      assetCanonical: `${code}:${contractId || issuerId}`,
+      iconUrl: verifiedToken?.icon,
+      type: SERVICE_TYPES.CACHE_ASSET_ICON,
+    });
+  }
+
   return {
     icon: verifiedToken?.icon,
     canonicalAsset,