fix: returning walletClient and reconnecting mock client

.changeset/strange-hounds-rest.md

@@ -0,0 +1,5 @@
+---
+"@wagmi/core": patch
+---
+
+Fixed getting wallet client and reconnecting logic for the mock client.

packages/core/src/connectors/mock.test.ts

@@ -1,6 +1,6 @@
+import { connect, getWalletClient } from '@wagmi/core'
 import { accounts, config } from '@wagmi/test'
 import { expect, test } from 'vitest'
-
 import { mock } from './mock.js'
 
 test('setup', () => {
@@ -9,6 +9,14 @@ test('setup', () => {
   expect(connector.name).toEqual('Mock Connector')
 })
 
+test('connector client is defined', async () => {
+  const connectorFn = mock({ accounts })
+  const connector = config._internal.connectors.setup(connectorFn)
+
+  await connect(config, { connector })
+  await expect(getWalletClient(config)).resolves.toBeDefined()
+})
+
 test('behavior: features.connectError', () => {
   const connectorFn = mock({ accounts, features: { connectError: true } })
   const connector = config._internal.connectors.setup(connectorFn)

packages/core/src/connectors/mock.ts

@@ -45,15 +45,26 @@ export function mock(parameters: MockParameters) {
   type Provider = ReturnType<
     Transport<'custom', unknown, EIP1193RequestFn<WalletRpcSchema>>
   >
+  type Properties = Record<string, unknown>
+  type StorageItem = {
+    [_ in 'mock.connected']: true
+  }
   let connected = false
   let connectedChainId: number
 
-  return createConnector<Provider>((config) => ({
+  return createConnector<Provider, Properties, StorageItem>((config) => ({
     id: 'mock',
     name: 'Mock Connector',
     type: mock.type,
     async setup() {
       connectedChainId = config.chains[0].id
+
+      const isConnected = Boolean(features.reconnect
+        ? await config.storage?.getItem('mock.connected')
+        : false)
+      if (isConnected) {
+        this.connect()
+      }
     },
     async connect({ chainId } = {}) {
       if (features.connectError) {
@@ -74,6 +85,9 @@ export function mock(parameters: MockParameters) {
       }
 
       connected = true
+      if (features.reconnect) {
+        await config.storage?.setItem('mock.connected', true)
+      }
 
       return {
         accounts: accounts.map((x) => getAddress(x)),
@@ -82,6 +96,9 @@ export function mock(parameters: MockParameters) {
     },
     async disconnect() {
       connected = false
+      if (features.reconnect) {
+        await config.storage?.removeItem('mock.connected')
+      }
     },
     async getAccounts() {
       if (!connected) throw new ConnectorNotConnectedError()
@@ -125,6 +142,9 @@ export function mock(parameters: MockParameters) {
     async onDisconnect(_error) {
       config.emitter.emit('disconnect')
       connected = false
+      if (features.reconnect) {
+        await config.storage?.removeItem('mock.connected')
+      }
     },
     async getProvider({ chainId } = {}) {
       const chain =
@@ -134,7 +154,8 @@ export function mock(parameters: MockParameters) {
       const request: EIP1193RequestFn = async ({ method, params }) => {
         // eth methods
         if (method === 'eth_chainId') return numberToHex(connectedChainId)
-        if (method === 'eth_requestAccounts') return parameters.accounts
+        if (method === 'eth_accounts' || method === 'eth_requestAccounts')
+          return parameters.accounts
         if (method === 'eth_signTypedData_v4')
           if (features.signTypedDataError) {
             if (typeof features.signTypedDataError === 'boolean')