fix: add support for Bitcoin test net in OKX Wallet (#4691)

Author: svenvoskamp

.changeset/warm-planets-work.md

@@ -0,0 +1,5 @@
+---
+'@reown/appkit-adapter-bitcoin': patch
+---
+
+Adds support for Bitcoin testnet in OKX wallet

packages/adapters/bitcoin/src/connectors/OKXConnector.ts

@@ -3,13 +3,18 @@ import { type CaipNetwork, ConstantsUtil as CommonConstantsUtil } from '@reown/a
 import { CoreHelperUtil, type RequestArguments } from '@reown/appkit-controllers'
 import { PresetsUtil } from '@reown/appkit-utils'
 import type { BitcoinConnector } from '@reown/appkit-utils/bitcoin'
-import { bitcoin } from '@reown/appkit/networks'
+import { bitcoin, bitcoinTestnet } from '@reown/appkit/networks'
 
 import { MethodNotSupportedError } from '../errors/MethodNotSupportedError.js'
 import { AddressPurpose } from '../utils/BitcoinConnector.js'
 import { ProviderEventEmitter } from '../utils/ProviderEventEmitter.js'
 import { UnitsUtil } from '../utils/UnitsUtil.js'
 
+const OKX_NETWORK_KEYS = {
+  [bitcoin.caipNetworkId]: 'bitcoin',
+  [bitcoinTestnet.caipNetworkId]: 'bitcoinTestnet'
+} as const
+
 export class OKXConnector extends ProviderEventEmitter implements BitcoinConnector {
   public readonly id = 'OKX'
   public readonly name = 'OKX Wallet'
@@ -21,7 +26,7 @@ export class OKXConnector extends ProviderEventEmitter implements BitcoinConnect
 
   public readonly provider = this
 
-  private readonly wallet: OKXConnector.Wallet
+  private wallet: OKXConnector.Wallet
   private readonly requestedChains: CaipNetwork[] = []
   private readonly getActiveNetwork: () => CaipNetwork | undefined
 
@@ -116,8 +121,25 @@ export class OKXConnector extends ProviderEventEmitter implements BitcoinConnect
     }
   }
 
-  public async switchNetwork(_caipNetworkId: string): Promise<void> {
-    throw new Error(`${this.name} wallet does not support network switching`)
+  public async switchNetwork(_caipNetworkId: CaipNetwork['caipNetworkId']): Promise<void> {
+    const connector = OKXConnector.getWallet({
+      requestedChains: this.requestedChains,
+      getActiveNetwork: this.getActiveNetwork,
+      requestedCaipNetworkId: _caipNetworkId
+    })
+
+    if (!connector) {
+      throw new Error(`${this.name} wallet does not support network switching`)
+    }
+
+    this.unbindEvents()
+    this.wallet = connector.wallet
+
+    try {
+      await this.connect()
+    } catch (error) {
+      throw new Error(`${this.name} wallet does not support network switching`)
+    }
   }
 
   public request<T>(_args: RequestArguments): Promise<T> {
@@ -146,9 +168,17 @@ export class OKXConnector extends ProviderEventEmitter implements BitcoinConnect
       return undefined
     }
 
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    let wallet: any = undefined
+
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
     const okxwallet = (window as any)?.okxwallet
-    const wallet = okxwallet?.bitcoin
+
+    const networkKey =
+      OKX_NETWORK_KEYS[params.requestedCaipNetworkId as keyof typeof OKX_NETWORK_KEYS]
+
+    wallet = okxwallet?.[networkKey] || okxwallet?.bitcoin
+
     /**
      * OKX doesn't provide a way to get the image URL specifally for bitcoin
      * so we use the icon for cardano as a fallback
@@ -173,6 +203,7 @@ export namespace OKXConnector {
     requestedChains: CaipNetwork[]
     getActiveNetwork: () => CaipNetwork | undefined
     imageUrl: string
+    requestedCaipNetworkId?: CaipNetwork['caipNetworkId']
   }
 
   export type Wallet = {

packages/adapters/bitcoin/tests/connectors/OKXConnector.test.ts

@@ -245,10 +245,33 @@ describe('OKXConnector', () => {
   })
 
   describe('switchNetwork', () => {
-    it('should throw error saying network switching is not supported', async () => {
-      await expect(
-        connector.switchNetwork('bip122:000000000019d6689c085ae165831e93')
-      ).rejects.toThrow('OKX Wallet wallet does not support network switching')
+    it('should switch to testnet network and connect', async () => {
+      const testnetWallet = mockOKXWallet()
+      const testnetConnector = new OKXConnector({
+        wallet: testnetWallet,
+        requestedChains,
+        getActiveNetwork,
+        imageUrl: 'mock_image'
+      })
+
+      vi.spyOn(OKXConnector, 'getWallet').mockReturnValue(testnetConnector)
+
+      const accountsChangedListener = vi.fn()
+      connector.on('accountsChanged', accountsChangedListener)
+
+      // Switch to testnet bitcoin
+      await connector.switchNetwork('bip122:000000000933ea01ad0ee984209779ba')
+
+      expect(testnetWallet.connect).toHaveBeenCalled()
+      expect(accountsChangedListener).toHaveBeenCalledWith(['mock_address'])
+    })
+
+    it('should throw error when wallet is not available', async () => {
+      vi.spyOn(OKXConnector, 'getWallet').mockReturnValue(undefined)
+
+      await expect(connector.switchNetwork('bip122:fake-network')).rejects.toThrow(
+        'OKX Wallet wallet does not support network switching'
+      )
     })
   })
 })