Revert "fix: add tests for WalletConnectContext"

This reverts commit 5c08405.

src/services/walletconnect/WalletConnectContext.tsx

@@ -6,10 +6,10 @@ import WalletConnectWallet from './WalletConnectWallet'
 import { asError } from '../exceptions/utils'
 import { stripEip155Prefix } from './utils'
 
-const walletConnectSingleton = WalletConnectWallet
+const walletConnectSingleton = new WalletConnectWallet()
 
 export const WalletConnectContext = createContext<{
-  walletConnect: typeof WalletConnectWallet | null
+  walletConnect: WalletConnectWallet | null
   error: Error | null
 }>({
   walletConnect: null,
@@ -21,7 +21,7 @@ export const WalletConnectProvider = ({ children }: { children: ReactNode }) =>
     safe: { chainId },
     safeAddress,
   } = useSafeInfo()
-  const [walletConnect, setWalletConnect] = useState<typeof WalletConnectWallet | null>(null)
+  const [walletConnect, setWalletConnect] = useState<WalletConnectWallet | null>(null)
   const [error, setError] = useState<Error | null>(null)
   const safeWalletProvider = useSafeWalletProvider()
 

src/services/walletconnect/WalletConnectWallet.test.ts

@@ -2,7 +2,7 @@ import { hexZeroPad } from 'ethers/lib/utils'
 import type { ProposalTypes, SessionTypes, SignClientTypes, Verify } from '@walletconnect/types'
 import type { IWeb3Wallet, Web3WalletTypes } from '@walletconnect/web3wallet'
 
-import type WalletConnectWallet from './WalletConnectWallet'
+import WalletConnectWallet from './WalletConnectWallet'
 
 jest.mock('@walletconnect/core', () => ({
   Core: jest.fn(),
@@ -45,20 +45,14 @@ jest.mock('@walletconnect/web3wallet', () => {
 })
 
 describe('WalletConnectWallet', () => {
-  let wallet: typeof WalletConnectWallet
+  let wallet: WalletConnectWallet
 
   beforeEach(async () => {
-    // Reset import
-    wallet = (await import('./WalletConnectWallet')).default
+    wallet = new WalletConnectWallet()
 
     await wallet.init()
   })
 
-  afterEach(() => {
-    // Reset instance to avoid side leaking mocks
-    jest.resetModules()
-  })
-
   describe('connect', () => {
     it('should call pair with the correct parameters', async () => {
       const pairSpy = jest.spyOn(((wallet as any).web3Wallet as IWeb3Wallet).core.pairing, 'pair')

src/services/walletconnect/WalletConnectWallet.ts

@@ -251,5 +251,4 @@ class WalletConnectWallet {
   }
 }
 
-// Instance needs to be exported for testing purposes
-export default new WalletConnectWallet()
+export default WalletConnectWallet

src/services/walletconnect/useWalletConnectSessions.test.tsx

@@ -28,7 +28,7 @@ describe('useWalletConnectSessions', () => {
       getActiveSessions: jest.fn().mockReturnValue(sessions),
       onSessionAdd: jest.fn(),
       onSessionDelete: jest.fn(),
-    } as unknown as typeof WalletConnectWallet
+    } as unknown as WalletConnectWallet
 
     const wrapper = ({ children }: any) => (
       <WalletConnectContext.Provider value={{ walletConnect: mockWalletConnect, error: null }}>
@@ -46,7 +46,7 @@ describe('useWalletConnectSessions', () => {
       getActiveSessions: jest.fn().mockReturnValue([]),
       onSessionAdd: jest.fn(),
       onSessionDelete: jest.fn(),
-    } as unknown as typeof WalletConnectWallet
+    } as unknown as WalletConnectWallet
 
     const wrapper = ({ children }: any) => (
       <WalletConnectContext.Provider value={{ walletConnect: mockWalletConnect, error: null }}>
@@ -94,7 +94,7 @@ describe('useWalletConnectSessions', () => {
       getActiveSessions: jest.fn().mockReturnValue([]),
       onSessionAdd: jest.fn(),
       onSessionDelete: jest.fn(),
-    } as unknown as typeof WalletConnectWallet
+    } as unknown as WalletConnectWallet
 
     const wrapper = ({ children }: any) => (
       <WalletConnectContext.Provider value={{ walletConnect: mockWalletConnect, error: null }}>