Fix Jest mock factory out-of-scope variable errors

- Fixed Buffer import in setupTests.js by moving it inside jest.mock() factory for @solana/web3.js
- Fixed Connection import in App.test.tsx by moving it inside jest.mock() factory
- Added missing clusterApiUrl mock function to @solana/web3.js mock
- Added comprehensive mocks for Ledger hardware wallet libraries
- Added mock for @project-serum/serum to prevent initialization issues
- Made clipboard API configurable to prevent redefinition errors
- All Jest mock factories now comply with out-of-scope variable restrictions

Co-authored-by: 0xrinegade <[email protected]>

Author: Copilot

src/App.test.tsx

@@ -2,7 +2,6 @@ import React from 'react';
 import { render, screen, waitFor } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import { MemoryRouter } from 'react-router-dom';
-import { Connection } from '@solana/web3.js';
 import App from './App';
 import { ConnectionProvider } from './utils/connection';
 import { WalletProvider } from './utils/wallet';
@@ -21,10 +20,13 @@ jest.mock('./utils/connection', () => ({
   ConnectionProvider: ({ children }: { children: React.ReactNode }) => (
     <div data-testid="connection-provider">{children}</div>
   ),
-  useConnection: () => ({
-    connection: new Connection('https://api.devnet.solana.com'),
-    endpoint: 'https://api.devnet.solana.com',
-  }),
+  useConnection: () => {
+    const { Connection } = require('@solana/web3.js'); // Move import inside mock factory
+    return {
+      connection: new Connection('https://api.devnet.solana.com'),
+      endpoint: 'https://api.devnet.solana.com',
+    };
+  },
 }));
 
 // Mock wallet

src/setupTests.js

@@ -3,7 +3,6 @@
 // expect(element).toHaveTextContent(/react/i)
 // learn more: https://github.com/testing-library/jest-dom
 import '@testing-library/jest-dom';
-import { Buffer } from 'buffer';
 
 // Polyfill for TextEncoder/TextDecoder in Jest
 global.TextEncoder = require('util').TextEncoder;
@@ -32,6 +31,7 @@ Object.defineProperty(navigator, 'clipboard', {
     writeText: jest.fn(() => Promise.resolve()),
     readText: jest.fn(() => Promise.resolve('')),
   },
+  configurable: true, // Allow reconfiguration for tests
 });
 
 // Mock ResizeObserver
@@ -92,15 +92,15 @@ jest.mock('scrypt-js', () => jest.fn((password, salt, N, r, p, keylen, callback)
   callback(null, new Uint8Array(keylen));
 }));
 jest.mock('@solana/web3.js', () => {
-  const mockBuffer = require('buffer').Buffer;
+  const { Buffer } = require('buffer'); // Move Buffer import inside mock factory
 
   const mockPublicKey = {
     toBase58: jest.fn(() => 'mock-public-key'),
     toString: jest.fn(() => 'mock-public-key'),
     equals: jest.fn(() => false),
     toJSON: jest.fn(() => 'mock-public-key'),
     toBytes: jest.fn(() => new Uint8Array(32)),
-    toBuffer: jest.fn(() => mockBuffer.alloc(32)),
+    toBuffer: jest.fn(() => Buffer.alloc(32)),
     isOnCurve: jest.fn(() => true),
   };
 
@@ -126,10 +126,46 @@ jest.mock('@solana/web3.js', () => {
     SystemProgram: {
       transfer: jest.fn(() => ({})),
     },
+    clusterApiUrl: jest.fn((cluster) => `https://api.${cluster}.solana.com`),
     LAMPORTS_PER_SOL: 1000000000,
   };
 });
 
+// Mock Ledger hardware wallet libraries to avoid hardware-specific dependencies
+jest.mock('@ledgerhq/hw-transport-webhid', () => ({
+  __esModule: true,
+  default: jest.fn().mockImplementation(() => ({
+    getPublicKey: jest.fn(() => Promise.resolve('mock-ledger-public-key')),
+    signTransaction: jest.fn(() => Promise.resolve('mock-signature')),
+    close: jest.fn(() => Promise.resolve()),
+  })),
+}));
+
+jest.mock('@ledgerhq/hw-transport-webusb', () => ({
+  __esModule: true,
+  default: jest.fn().mockImplementation(() => ({
+    getPublicKey: jest.fn(() => Promise.resolve('mock-ledger-public-key')),
+    signTransaction: jest.fn(() => Promise.resolve('mock-signature')),
+    close: jest.fn(() => Promise.resolve()),
+  })),
+}));
+
+// Mock @project-serum/serum to avoid initialization issues
+jest.mock('@project-serum/serum', () => ({
+  TokenInstructions: {
+    initializeMint: jest.fn(),
+    initializeAccount: jest.fn(),
+    transfer: jest.fn(),
+  },
+  Market: {
+    load: jest.fn(() => Promise.resolve({
+      address: 'mock-market-address',
+      baseMintAddress: 'mock-base-mint',
+      quoteMintAddress: 'mock-quote-mint',
+    })),
+  },
+}));
+
 // Mock SVM-Pay to avoid network calls in tests
 jest.mock('svm-pay', () => ({
   SVMPay: jest.fn().mockImplementation(() => ({

yarn.lock

@@ -1197,7 +1197,7 @@
 
 "@craco/craco@^7.1.0":
   version "7.1.0"
-  resolved "https://registry.npmjs.org/@craco/craco/-/craco-7.1.0.tgz"
+  resolved "https://registry.yarnpkg.com/@craco/craco/-/craco-7.1.0.tgz#12bd394c7f0334e214302e4d35a1768f68042fbb"
   integrity sha512-oRAcPIKYrfPXp9rSzlsDNeOaVtDiKhoyqSXUoqiK24jCkHr4T8m/a2f74yXIzCbIheoUWDOIfWZyRgFgT+cpqA==
   dependencies:
     autoprefixer "^10.4.12"