chore: bitcoin adapter unit testing (#3516)

Co-authored-by: tomiir <rocchitomas@gmail.com>

.changeset/tall-carrots-jam.md

@@ -0,0 +1,23 @@
+---
+'@reown/appkit-adapter-bitcoin': patch
+'@reown/appkit-adapter-ethers': patch
+'@reown/appkit-adapter-ethers5': patch
+'@reown/appkit-adapter-solana': patch
+'@reown/appkit-adapter-wagmi': patch
+'@reown/appkit': patch
+'@reown/appkit-utils': patch
+'@reown/appkit-cdn': patch
+'@reown/appkit-cli': patch
+'@reown/appkit-common': patch
+'@reown/appkit-core': patch
+'@reown/appkit-experimental': patch
+'@reown/appkit-polyfills': patch
+'@reown/appkit-scaffold-ui': patch
+'@reown/appkit-siwe': patch
+'@reown/appkit-siwx': patch
+'@reown/appkit-ui': patch
+'@reown/appkit-wallet': patch
+'@reown/appkit-wallet-button': patch
+---
+
+Add unit testing for Bitcoin adapter and fix unused default values

packages/adapters/bitcoin/src/adapter.ts

@@ -78,15 +78,15 @@ export class BitcoinAdapter extends AdapterBlueprint<BitcoinConnector> {
     }
   }
   override async getAccounts(
-    _params: AdapterBlueprint.GetAccountsParams
+    params: AdapterBlueprint.GetAccountsParams
   ): Promise<AdapterBlueprint.GetAccountsResult> {
-    const addresses = await this.connector?.getAccountAddresses()
+    const addresses = await this.connectors
+      .find(connector => connector.id === params.id)
+      ?.getAccountAddresses()
+      .catch(() => [])
+
     const accounts = addresses?.map(a =>
-      CoreHelperUtil.createAccount(
-        'bip122',
-        a.address,
-        (a.purpose || 'payment') as BitcoinConnector.AccountAddress['purpose']
-      )
+      CoreHelperUtil.createAccount('bip122', a.address, a.purpose || 'payment')
     )
 
     return {
@@ -134,7 +134,7 @@ export class BitcoinAdapter extends AdapterBlueprint<BitcoinConnector> {
   ): Promise<AdapterBlueprint.ConnectResult> {
     return this.connect({
       id: params.id,
-      chainId: params.chainId || this.networks[0]?.id || '',
+      chainId: params.chainId,
       type: ''
     })
   }
@@ -182,8 +182,6 @@ export class BitcoinAdapter extends AdapterBlueprint<BitcoinConnector> {
     this.unbindEvents()
   }
 
-  // -- Unused => Refactor ------------------------------------------- //
-
   override async getBalance(
     params: AdapterBlueprint.GetBalanceParams
   ): Promise<AdapterBlueprint.GetBalanceResult> {
@@ -210,6 +208,8 @@ export class BitcoinAdapter extends AdapterBlueprint<BitcoinConnector> {
     })
   }
 
+  // -- Unused => Refactor ------------------------------------------- //
+
   override getProfile(
     _params: AdapterBlueprint.GetProfileParams
   ): Promise<AdapterBlueprint.GetProfileResult> {
@@ -280,8 +280,7 @@ export class BitcoinAdapter extends AdapterBlueprint<BitcoinConnector> {
       const [newAccount] = data
       if (newAccount) {
         this.emit('accountChanged', {
-          address: newAccount,
-          chainId: this.networks[0]?.id || ''
+          address: newAccount
         })
       }
     }