test(sdk): coinbase wallet

Author: gregfromstl

packages/thirdweb/src/exports/wallets.native.ts

@@ -107,7 +107,7 @@ export const authenticateWithRedirect = () => {
 export type {
   CoinbaseWalletCreationOptions,
   CoinbaseSDKWalletConnectionOptions,
-} from "../wallets/coinbase/coinbaseWebSDK.js";
+} from "../wallets/coinbase/coinbase-web.js";
 
 export type {
   WalletEmitter,

packages/thirdweb/src/exports/wallets.ts

@@ -117,7 +117,7 @@ export {
 export type {
   CoinbaseWalletCreationOptions,
   CoinbaseSDKWalletConnectionOptions,
-} from "../wallets/coinbase/coinbaseWebSDK.js";
+} from "../wallets/coinbase/coinbase-web.js";
 
 export type {
   WalletEmitter,

packages/thirdweb/src/react/web/utils/usePreloadWalletProviders.ts

@@ -21,7 +21,7 @@ export function usePreloadWalletProviders({
           switch (true) {
             case COINBASE === w.id: {
               const { getCoinbaseWebProvider } = await import(
-                "../../../wallets/coinbase/coinbaseWebSDK.js"
+                "../../../wallets/coinbase/coinbase-web.js"
               );
               await getCoinbaseWebProvider(
                 w.getConfig() as CreateWalletArgs<typeof COINBASE>[1],

packages/thirdweb/src/wallets/coinbase/coinbase-mobile.test.ts

@@ -0,0 +1,69 @@
+import type { ProviderInterface } from "@coinbase/wallet-sdk";
+import { beforeEach, describe, expect, test, vi } from "vitest";
+import { getCoinbaseMobileProvider } from "./coinbase-mobile.js";
+import type { CoinbaseWalletCreationOptions } from "./coinbase-web.js";
+
+// Mock dependencies
+vi.mock("expo-linking", () => ({
+  addEventListener: vi.fn(),
+}));
+
+vi.mock("@mobile-wallet-protocol/client", () => ({
+  EIP1193Provider: class {
+    constructor() {
+      // biome-ignore lint/correctness/noConstructorReturn: Inside tests
+      return {
+        request: vi.fn(),
+      };
+    }
+  },
+  Wallets: {
+    CoinbaseSmartWallet: {},
+  },
+  handleResponse: vi.fn(),
+}));
+
+vi.mock("@coinbase/wallet-mobile-sdk", () => ({
+  configure: vi.fn(),
+  handleResponse: vi.fn(),
+}));
+
+vi.mock("@coinbase/wallet-mobile-sdk/build/WalletMobileSDKEVMProvider", () => ({
+  WalletMobileSDKEVMProvider: class {
+    constructor() {
+      // biome-ignore lint/correctness/noConstructorReturn: Inside tests
+      return {
+        request: vi.fn(),
+      };
+    }
+  },
+}));
+
+describe("Coinbase Mobile", () => {
+  let provider: undefined | ProviderInterface;
+
+  beforeEach(() => {
+    provider = undefined;
+  });
+
+  test("getCoinbaseMobileProvider initializes smart wallet provider", async () => {
+    const options: CoinbaseWalletCreationOptions = {
+      walletConfig: { options: "smartWalletOnly" },
+      mobileConfig: { callbackURL: "https://example.com" },
+    };
+
+    provider = await getCoinbaseMobileProvider(options);
+    expect(provider).toBeDefined();
+    expect(provider.request).toBeInstanceOf(Function);
+  });
+
+  test("getCoinbaseMobileProvider initializes coinbase app provider", async () => {
+    const options: CoinbaseWalletCreationOptions = {
+      mobileConfig: { callbackURL: "https://example.com" },
+    };
+
+    provider = await getCoinbaseMobileProvider(options);
+    expect(provider).toBeDefined();
+    expect(provider.request).toBeInstanceOf(Function);
+  });
+});

packages/thirdweb/src/wallets/coinbase/coinbaseMobileSDK.ts renamed to packages/thirdweb/src/wallets/coinbase/coinbase-mobile.ts

@@ -1,5 +1,5 @@
 import type { ProviderInterface } from "@coinbase/wallet-sdk";
-import type { CoinbaseWalletCreationOptions } from "./coinbaseWebSDK.js";
+import type { CoinbaseWalletCreationOptions } from "./coinbase-web.js";
 
 let _provider: ProviderInterface | undefined;
 

packages/thirdweb/src/wallets/coinbase/coinbase-wallet.ts

@@ -67,7 +67,7 @@ export function coinbaseWalletSDK(args: {
     getAccount: () => account,
     autoConnect: async (options) => {
       const { autoConnectCoinbaseWalletSDK } = await import(
-        "./coinbaseWebSDK.js"
+        "./coinbase-web.js"
       );
       const provider = await args.providerFactory();
       const [connectedAccount, connectedChain, doDisconnect, doSwitchChain] =
@@ -87,7 +87,7 @@ export function coinbaseWalletSDK(args: {
       return account;
     },
     connect: async (options) => {
-      const { connectCoinbaseWalletSDK } = await import("./coinbaseWebSDK.js");
+      const { connectCoinbaseWalletSDK } = await import("./coinbase-web.js");
       const provider = await args.providerFactory();
       const [connectedAccount, connectedChain, doDisconnect, doSwitchChain] =
         await connectCoinbaseWalletSDK(options, emitter, provider);

packages/thirdweb/src/wallets/coinbase/coinbase-web.test.ts

@@ -0,0 +1,105 @@
+import type { ProviderInterface } from "@coinbase/wallet-sdk";
+import { beforeEach, describe, expect, test, vi } from "vitest";
+import {} from "../../chains/utils.js";
+import { COINBASE } from "../constants.js";
+import type { Wallet } from "../interfaces/wallet.js";
+import {
+  autoConnectCoinbaseWalletSDK,
+  connectCoinbaseWalletSDK,
+  getCoinbaseWebProvider,
+  isCoinbaseSDKWallet,
+} from "./coinbase-web.js";
+
+// Mock dependencies
+vi.mock("@coinbase/wallet-sdk", () => ({
+  default: class {
+    makeWeb3Provider() {
+      return {
+        request: vi.fn(),
+        on: vi.fn(),
+        removeListener: vi.fn(),
+        disconnect: vi.fn(),
+      };
+    }
+  },
+}));
+
+vi.mock("../../utils/address.js", () => ({
+  getAddress: vi.fn((address) => address),
+}));
+
+vi.mock("../../chains/utils.js", () => ({
+  getCachedChain: vi.fn((chainId) => ({ id: chainId })),
+  getChainMetadata: vi.fn(async (_chain) => ({
+    name: "Test Chain",
+    nativeCurrency: { name: "Test Coin", symbol: "TC", decimals: 18 },
+    explorers: [{ url: "https://explorer.test" }],
+  })),
+}));
+
+vi.mock("../../utils/normalizeChainId.js", () => ({
+  normalizeChainId: vi.fn((chainId) => Number(chainId)),
+}));
+
+describe("Coinbase Web", () => {
+  let provider: ProviderInterface;
+
+  beforeEach(async () => {
+    provider = await getCoinbaseWebProvider();
+  });
+
+  test("getCoinbaseWebProvider initializes provider", async () => {
+    expect(provider).toBeDefined();
+    expect(provider.request).toBeInstanceOf(Function);
+  });
+
+  test("isCoinbaseSDKWallet returns true for Coinbase wallet", () => {
+    const wallet: Wallet = { id: COINBASE } as Wallet;
+    expect(isCoinbaseSDKWallet(wallet)).toBe(true);
+  });
+
+  test("isCoinbaseSDKWallet returns false for non-Coinbase wallet", () => {
+    const wallet: Wallet = { id: "other" } as unknown as Wallet;
+    expect(isCoinbaseSDKWallet(wallet)).toBe(false);
+  });
+
+  test("connectCoinbaseWalletSDK connects to the wallet", async () => {
+    provider.request = vi
+      .fn()
+      .mockResolvedValueOnce(["0x123"])
+      .mockResolvedValueOnce("0x1");
+    const emitter = { emit: vi.fn() };
+    const options = { client: {} };
+
+    const [account, chain] = await connectCoinbaseWalletSDK(
+      // biome-ignore lint/suspicious/noExplicitAny: Inside tests
+      options as any,
+      // biome-ignore lint/suspicious/noExplicitAny: Inside tests
+      emitter as any,
+      provider,
+    );
+
+    expect(account.address).toBe("0x123");
+    expect(chain.id).toBe(1);
+  });
+
+  test("autoConnectCoinbaseWalletSDK auto-connects to the wallet", async () => {
+    provider.request = vi
+      .fn()
+      .mockResolvedValueOnce(["0x123"])
+      .mockResolvedValueOnce("0x1");
+    const emitter = { emit: vi.fn() };
+    const options = { client: {} };
+
+    const [account, chain] = await autoConnectCoinbaseWalletSDK(
+      // biome-ignore lint/suspicious/noExplicitAny: Inside tests
+      options as any,
+      // biome-ignore lint/suspicious/noExplicitAny: Inside tests
+      emitter as any,
+      provider,
+    );
+
+    expect(account.address).toBe("0x123");
+    expect(chain.id).toBe(1);
+  });
+});

packages/thirdweb/src/wallets/coinbase/coinbaseWebSDK.ts renamed to packages/thirdweb/src/wallets/coinbase/coinbase-web.ts

@@ -17,7 +17,7 @@ import { webLocalStorage } from "../utils/storage/webStorage.js";
 import { isMobile } from "../utils/web/isMobile.js";
 import { openWindow } from "../utils/web/openWindow.js";
 import { coinbaseWalletSDK } from "./coinbase/coinbase-wallet.js";
-import { getCoinbaseWebProvider } from "./coinbase/coinbaseWebSDK.js";
+import { getCoinbaseWebProvider } from "./coinbase/coinbase-web.js";
 import { COINBASE } from "./constants.js";
 import { isEcosystemWallet } from "./ecosystem/is-ecosystem-wallet.js";
 import { ecosystemWallet } from "./in-app/web/ecosystem.js";

packages/thirdweb/src/wallets/create-wallet.ts

@@ -1,5 +1,5 @@
 import type { ThirdwebClient } from "../../client/client.js";
-import { isCoinbaseSDKWallet } from "../coinbase/coinbaseWebSDK.js";
+import { isCoinbaseSDKWallet } from "../coinbase/coinbase-web.js";
 import { isInAppWallet } from "../in-app/core/wallet/index.js";
 import { getInjectedProvider } from "../injected/index.js";
 import type { Wallet } from "../interfaces/wallet.js";
@@ -64,7 +64,7 @@ export async function getCallsStatus({
 
   if (isCoinbaseSDKWallet(wallet)) {
     const { coinbaseSDKWalletGetCallsStatus } = await import(
-      "../coinbase/coinbaseWebSDK.js"
+      "../coinbase/coinbase-web.js"
     );
     return coinbaseSDKWalletGetCallsStatus({ wallet, bundleId });
   }