@jmruthers/pace-core 0.2.7 → 0.5.1

package/src/components/FileUpload/__tests__/FileUpload.real.test.tsx

@@ -1,927 +0,0 @@
-/**
- * @vitest-environment jsdom
- */
-
-import React from 'react';
-import { screen, waitFor, fireEvent } from '@testing-library/react';
-import userEvent from '@testing-library/user-event';
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { errorTestingUtils, TestErrorBoundary } from '../../__tests__/ErrorTesting.enhanced.test';
-import { FileUpload } from '../FileUpload';
-import { SupabaseClient } from '@supabase/supabase-js';
-import { renderWithProviders, testUserWorkflow, testDataFlow } from '../../../__tests__/shared';
-
-// Mock the storage utilities at the module level
-vi.mock('../../../utils/storage', () => ({
-  validateFileSize: vi.fn(),
-  formatFileSize: vi.fn()
-}));
-
-// Mock the useFileUpload hook to return a real implementation
-vi.mock('../../../hooks/useStorage', () => ({
-  useFileUpload: vi.fn()
-}));
-
-// Mock the Button component
-vi.mock('../../Button/Button', () => ({
-  Button: ({ children, onClick, disabled, className, type, variant, ...props }: any) => (
-    <button
-      onClick={onClick}
-      disabled={disabled}
-      className={className}
-      type={type}
-      data-variant={variant}
-      {...props}
-    >
-      {children}
-    </button>
-  )
-}));
-
-// Mock the Progress component
-vi.mock('../../Progress/Progress', () => ({
-  Progress: ({ value, max, className, ...props }: any) => (
-    <div className={className} data-value={value} data-max={max} {...props}>
-      Progress: {value}%
-    </div>
-  )
-}));
-
-// Mock the Alert component
-vi.mock('../../Alert/Alert', () => ({
-  Alert: ({ children, variant, className, ...props }: any) => (
-    <div className={className} data-variant={variant} {...props}>
-      {children}
-    </div>
-  )
-}));
-
-// Import mocked functions
-import { validateFileSize, formatFileSize } from '../../../utils/storage';
-import { useFileUpload } from '../../../hooks/useStorage';
-
-// Mock data
-const mockSupabase = {} as SupabaseClient;
-const mockAppName = 'TEST_APP';
-const mockOrgId = 'test-org-id';
-
-// Test file factory
-const createMockFile = (name: string, size: number, type: string = 'image/jpeg'): File => {
-  const file = new File(['test content'], name, { type });
-  Object.defineProperty(file, 'size', { value: size });
-  return file;
-};
-
-// Helper function to get file input
-const getFileInput = (container: HTMLElement): HTMLInputElement => {
-  return container.querySelector('input[type="file"]') as HTMLInputElement;
-};
-
-describe('FileUpload Real Component Tests', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-    
-    // Setup default mocks
-    (validateFileSize as any).mockReturnValue({ isValid: true, error: null });
-    (formatFileSize as any).mockReturnValue('1.0 MB');
-    
-    // Mock useFileUpload to return a real implementation that we can control
-    const mockUploadWithProgress = vi.fn();
-    (useFileUpload as any).mockReturnValue({
-      uploadWithProgress: mockUploadWithProgress,
-      uploadProgress: 0,
-      isUploading: false,
-      uploadError: null
-    });
-  });
-
-  afterEach(() => {
-    vi.clearAllTimers();
-  });
-
-  describe('Component Initialization', () => {
-    it('should initialize with default state', () => {
-      renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      // Should render the default content
-      expect(screen.getByText('Drag and drop files here or click to select')).toBeInTheDocument();
-      expect(screen.getByRole('button', { name: 'Select Files' })).toBeInTheDocument();
-      
-      // Should not show files section initially
-      expect(screen.queryByText('Selected Files:')).not.toBeInTheDocument();
-      
-      // Should not show validation errors initially
-      expect(screen.queryByText(/File \d+:/)).not.toBeInTheDocument();
-    });
-
-    it('should initialize with custom props', () => {
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-          accept="image/*"
-          multiple={true}
-          maxSize={1024 * 1024}
-          disabled={false}
-          className="custom-class"
-        />
-      );
-
-      // Should apply custom className
-      expect(container.querySelector('.file-upload')).toHaveClass('custom-class');
-      
-      // Should have correct input attributes
-      const fileInput = getFileInput(container);
-      expect(fileInput).toHaveAttribute('accept', 'image/*');
-      expect(fileInput).toHaveAttribute('multiple');
-    });
-  });
-
-  describe('File Selection and Validation', () => {
-    it('should handle single file selection and validation', () => {
-      const mockFile = createMockFile('test.jpg', 1024);
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      
-      // Trigger file selection
-      fireEvent.change(fileInput, { target: { files: [mockFile] } });
-
-      // Should validate the file
-      expect(validateFileSize).toHaveBeenCalledWith(mockFile);
-      
-      // Should show selected files
-      expect(screen.getByText('Selected Files:')).toBeInTheDocument();
-      expect(screen.getByText('test.jpg')).toBeInTheDocument();
-      expect(screen.getByText('1.0 MB')).toBeInTheDocument();
-      
-      // Should show upload and clear buttons
-      expect(screen.getByRole('button', { name: 'Upload 1 file' })).toBeInTheDocument();
-      expect(screen.getByRole('button', { name: 'Clear' })).toBeInTheDocument();
-    });
-
-    it('should handle multiple file selection', () => {
-      const mockFiles = [
-        createMockFile('test1.jpg', 1024),
-        createMockFile('test2.jpg', 2048)
-      ];
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-          multiple={true}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      
-      // Trigger file selection
-      fireEvent.change(fileInput, { target: { files: mockFiles } });
-
-      // Should validate each file
-      expect(validateFileSize).toHaveBeenCalledTimes(2);
-      expect(validateFileSize).toHaveBeenNthCalledWith(1, mockFiles[0]);
-      expect(validateFileSize).toHaveBeenNthCalledWith(2, mockFiles[1]);
-      
-      // Should show selected files
-      expect(screen.getByText('Selected Files:')).toBeInTheDocument();
-      expect(screen.getByText('test1.jpg')).toBeInTheDocument();
-      expect(screen.getByText('test2.jpg')).toBeInTheDocument();
-      
-      // Should show upload button for multiple files
-      expect(screen.getByRole('button', { name: 'Upload 2 files' })).toBeInTheDocument();
-    });
-
-    it('should handle file validation errors', () => {
-      (validateFileSize as any).mockReturnValue({ 
-        isValid: false, 
-        error: 'File too large' 
-      });
-
-      const mockFile = createMockFile('test.jpg', 1024);
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      
-      // Trigger file selection
-      fireEvent.change(fileInput, { target: { files: [mockFile] } });
-
-      // Should show validation error
-      expect(screen.getByText('File 1: File too large')).toBeInTheDocument();
-      
-      // Should disable upload button
-      const uploadButton = screen.getByRole('button', { name: 'Upload 1 file' });
-      expect(uploadButton).toBeDisabled();
-    });
-
-    it('should handle custom max size validation', () => {
-      const mockFile = createMockFile('test.jpg', 5 * 1024 * 1024); // 5MB
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-          maxSize={2 * 1024 * 1024} // 2MB limit
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      
-      // Trigger file selection
-      fireEvent.change(fileInput, { target: { files: [mockFile] } });
-
-      // Should show size validation error
-      expect(screen.getByText('File 1: Size (5MB) exceeds limit (2MB)')).toBeInTheDocument();
-    });
-
-    it('should handle mixed validation results for multiple files', () => {
-      (validateFileSize as any)
-        .mockReturnValueOnce({ isValid: false, error: 'File too large' })
-        .mockReturnValueOnce({ isValid: true, error: null });
-
-      const mockFiles = [
-        createMockFile('test1.jpg', 1024),
-        createMockFile('test2.jpg', 2048)
-      ];
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-          multiple={true}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      
-      // Trigger file selection
-      fireEvent.change(fileInput, { target: { files: mockFiles } });
-
-      // Should show validation error for first file
-      expect(screen.getByText('File 1: File too large')).toBeInTheDocument();
-      
-      // Should show second file (valid)
-      expect(screen.getByText('test2.jpg')).toBeInTheDocument();
-      
-      // Should disable upload due to validation errors
-      const uploadButton = screen.getByRole('button', { name: 'Upload 2 files' });
-      expect(uploadButton).toBeDisabled();
-    });
-  });
-
-  describe('Drag and Drop Functionality', () => {
-    it('should handle drag enter and show drop message', () => {
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const dropzone = container.querySelector('.file-upload__dropzone');
-      
-      // Trigger drag enter
-      fireEvent.dragEnter(dropzone!);
-      
-      // Should show drop message
-      expect(screen.getByText('Drop files here')).toBeInTheDocument();
-    });
-
-    it('should handle drag leave and hide drop message', () => {
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const dropzone = container.querySelector('.file-upload__dropzone');
-      
-      // Trigger drag enter then leave
-      fireEvent.dragEnter(dropzone!);
-      expect(screen.getByText('Drop files here')).toBeInTheDocument();
-      
-      fireEvent.dragLeave(dropzone!);
-      expect(screen.getByText('Drag and drop files here or click to select')).toBeInTheDocument();
-    });
-
-    it('should handle drag over and show drop message', () => {
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const dropzone = container.querySelector('.file-upload__dropzone');
-      
-      // Trigger drag over
-      fireEvent.dragOver(dropzone!);
-      
-      // Should show drop message
-      expect(screen.getByText('Drop files here')).toBeInTheDocument();
-    });
-
-    it('should handle file drop and process files', () => {
-      const mockFile = createMockFile('test.jpg', 1024);
-      const mockDataTransfer = {
-        files: [mockFile]
-      };
-
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const dropzone = container.querySelector('.file-upload__dropzone');
-      
-      // Trigger file drop
-      fireEvent.drop(dropzone!, { dataTransfer: mockDataTransfer });
-
-      // Should validate the file
-      expect(validateFileSize).toHaveBeenCalledWith(mockFile);
-      
-      // Should show selected files
-      expect(screen.getByText('Selected Files:')).toBeInTheDocument();
-      expect(screen.getByText('test.jpg')).toBeInTheDocument();
-    });
-
-    it('should not handle file drop when disabled', () => {
-      const mockFile = createMockFile('test.jpg', 1024);
-      const mockDataTransfer = {
-        files: [mockFile]
-      };
-
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-          disabled={true}
-        />
-      );
-
-      const dropzone = container.querySelector('.file-upload__dropzone');
-      
-      // Trigger file drop
-      fireEvent.drop(dropzone!, { dataTransfer: mockDataTransfer });
-
-      // Should not validate the file
-      expect(validateFileSize).not.toHaveBeenCalled();
-      
-      // Should not show selected files
-      expect(screen.queryByText('Selected Files:')).not.toBeInTheDocument();
-    });
-  });
-
-  describe('File Display and Formatting', () => {
-    it('should display files with formatted sizes', () => {
-      (formatFileSize as any).mockReturnValue('1.5 MB');
-      
-      const mockFile = createMockFile('test.jpg', 1024);
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      
-      // Trigger file selection
-      fireEvent.change(fileInput, { target: { files: [mockFile] } });
-
-      // Should show file name and formatted size
-      expect(screen.getByText('test.jpg')).toBeInTheDocument();
-      expect(screen.getByText('1.5 MB')).toBeInTheDocument();
-      expect(formatFileSize).toHaveBeenCalledWith(1024);
-    });
-
-    it('should not display files section when no files selected', () => {
-      renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      expect(screen.queryByText('Selected Files:')).not.toBeInTheDocument();
-    });
-  });
-
-  describe('Upload Flow', () => {
-    it('should call onUploadStart when upload begins', async () => {
-      const onUploadStart = vi.fn();
-      const mockFile = createMockFile('test.jpg', 1024);
-      const mockUploadWithProgress = vi.fn().mockResolvedValue({ success: true, path: 'test/path' });
-      
-      (useFileUpload as any).mockReturnValue({
-        uploadWithProgress: mockUploadWithProgress,
-        uploadProgress: 0,
-        isUploading: false,
-        uploadError: null
-      });
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-          onUploadStart={onUploadStart}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      fireEvent.change(fileInput, { target: { files: [mockFile] } });
-
-      const uploadButton = screen.getByRole('button', { name: 'Upload 1 file' });
-      await userEvent.click(uploadButton);
-
-      expect(onUploadStart).toHaveBeenCalled();
-    });
-
-    it('should call onUploadComplete for each file', async () => {
-      const onUploadComplete = vi.fn();
-      const mockFiles = [
-        createMockFile('test1.jpg', 1024),
-        createMockFile('test2.jpg', 2048)
-      ];
-      const mockUploadWithProgress = vi.fn()
-        .mockResolvedValueOnce({ success: true, path: 'test/path1' })
-        .mockResolvedValueOnce({ success: true, path: 'test/path2' });
-      
-      (useFileUpload as any).mockReturnValue({
-        uploadWithProgress: mockUploadWithProgress,
-        uploadProgress: 0,
-        isUploading: false,
-        uploadError: null
-      });
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-          onUploadComplete={onUploadComplete}
-          multiple={true}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      fireEvent.change(fileInput, { target: { files: mockFiles } });
-
-      const uploadButton = screen.getByRole('button', { name: 'Upload 2 files' });
-      await userEvent.click(uploadButton);
-
-      expect(onUploadComplete).toHaveBeenCalledTimes(2);
-      expect(onUploadComplete).toHaveBeenNthCalledWith(1, { success: true, path: 'test/path1' });
-      expect(onUploadComplete).toHaveBeenNthCalledWith(2, { success: true, path: 'test/path2' });
-    });
-
-    it('should not upload when validation errors exist', async () => {
-      (validateFileSize as any).mockReturnValue({ 
-        isValid: false, 
-        error: 'File too large' 
-      });
-      
-      const mockFile = createMockFile('test.jpg', 1024);
-      const mockUploadWithProgress = vi.fn();
-      
-      (useFileUpload as any).mockReturnValue({
-        uploadWithProgress: mockUploadWithProgress,
-        uploadProgress: 0,
-        isUploading: false,
-        uploadError: null
-      });
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      fireEvent.change(fileInput, { target: { files: [mockFile] } });
-
-      const uploadButton = screen.getByRole('button', { name: 'Upload 1 file' });
-      expect(uploadButton).toBeDisabled();
-      
-      await userEvent.click(uploadButton);
-      expect(mockUploadWithProgress).not.toHaveBeenCalled();
-    });
-
-    it('should reset files after upload', async () => {
-      const mockFile = createMockFile('test.jpg', 1024);
-      const mockUploadWithProgress = vi.fn().mockResolvedValue({ success: true, path: 'test/path' });
-      
-      (useFileUpload as any).mockReturnValue({
-        uploadWithProgress: mockUploadWithProgress,
-        uploadProgress: 0,
-        isUploading: false,
-        uploadError: null
-      });
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      fireEvent.change(fileInput, { target: { files: [mockFile] } });
-
-      expect(screen.getByText('Selected Files:')).toBeInTheDocument();
-
-      const uploadButton = screen.getByRole('button', { name: 'Upload 1 file' });
-      await userEvent.click(uploadButton);
-
-      await waitFor(() => {
-        expect(screen.queryByText('Selected Files:')).not.toBeInTheDocument();
-      });
-    });
-  });
-
-  describe('Upload Progress and Error States', () => {
-    it('should display upload progress when uploading', () => {
-      (useFileUpload as any).mockReturnValue({
-        uploadWithProgress: vi.fn(),
-        uploadProgress: 50,
-        isUploading: true,
-        uploadError: null
-      });
-
-      renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      expect(screen.getByText('Progress: 50%')).toBeInTheDocument();
-      expect(screen.getByText('Uploading... 50%')).toBeInTheDocument();
-    });
-
-    it('should display upload error', () => {
-      (useFileUpload as any).mockReturnValue({
-        uploadWithProgress: vi.fn(),
-        uploadProgress: 0,
-        isUploading: false,
-        uploadError: 'Upload failed'
-      });
-
-      renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      expect(screen.getByText('Upload failed')).toBeInTheDocument();
-    });
-  });
-
-  describe('Actions and State Management', () => {
-    it('should show upload and clear buttons when files selected', () => {
-      const mockFile = createMockFile('test.jpg', 1024);
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      fireEvent.change(fileInput, { target: { files: [mockFile] } });
-
-      expect(screen.getByRole('button', { name: 'Upload 1 file' })).toBeInTheDocument();
-      expect(screen.getByRole('button', { name: 'Clear' })).toBeInTheDocument();
-    });
-
-    it('should disable buttons when uploading', () => {
-      (useFileUpload as any).mockReturnValue({
-        uploadWithProgress: vi.fn(),
-        uploadProgress: 0,
-        isUploading: true,
-        uploadError: null
-      });
-
-      const mockFile = createMockFile('test.jpg', 1024);
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      fireEvent.change(fileInput, { target: { files: [mockFile] } });
-
-      const uploadButton = screen.getByRole('button', { name: 'Uploading...' });
-      const clearButton = screen.getByRole('button', { name: 'Clear' });
-      
-      expect(uploadButton).toBeDisabled();
-      expect(clearButton).toBeDisabled();
-    });
-
-    it('should clear files when clear button clicked', async () => {
-      const user = userEvent.setup();
-      const mockFile = createMockFile('test.jpg', 1024);
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      fireEvent.change(fileInput, { target: { files: [mockFile] } });
-
-      expect(screen.getByText('Selected Files:')).toBeInTheDocument();
-
-      const clearButton = screen.getByRole('button', { name: 'Clear' });
-      await user.click(clearButton);
-
-      expect(screen.queryByText('Selected Files:')).not.toBeInTheDocument();
-    });
-
-    it('should show correct upload button text for single file', () => {
-      const mockFile = createMockFile('test.jpg', 1024);
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      fireEvent.change(fileInput, { target: { files: [mockFile] } });
-
-      expect(screen.getByRole('button', { name: 'Upload 1 file' })).toBeInTheDocument();
-    });
-
-    it('should show correct upload button text for multiple files', () => {
-      const mockFiles = [
-        createMockFile('test1.jpg', 1024),
-        createMockFile('test2.jpg', 2048)
-      ];
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-          multiple={true}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      fireEvent.change(fileInput, { target: { files: mockFiles } });
-
-      expect(screen.getByRole('button', { name: 'Upload 2 files' })).toBeInTheDocument();
-    });
-  });
-
-  describe('Edge Cases', () => {
-    it('should handle null file list', () => {
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      
-      fireEvent.change(fileInput, { target: { files: null } });
-
-      expect(screen.queryByText('Selected Files:')).not.toBeInTheDocument();
-    });
-
-    it('should handle empty file list', () => {
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      
-      fireEvent.change(fileInput, { target: { files: [] } });
-
-      expect(screen.queryByText('Selected Files:')).not.toBeInTheDocument();
-    });
-
-    it('should handle upload error during upload', async () => {
-      const onUploadComplete = vi.fn();
-      const mockFile = createMockFile('test.jpg', 1024);
-      const mockUploadWithProgress = vi.fn().mockResolvedValue({ 
-        success: false, 
-        error: 'Network error' 
-      });
-      
-      (useFileUpload as any).mockReturnValue({
-        uploadWithProgress: mockUploadWithProgress,
-        uploadProgress: 0,
-        isUploading: false,
-        uploadError: null
-      });
-      
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-          onUploadComplete={onUploadComplete}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      fireEvent.change(fileInput, { target: { files: [mockFile] } });
-
-      const uploadButton = screen.getByRole('button', { name: 'Upload 1 file' });
-      await userEvent.click(uploadButton);
-
-      expect(onUploadComplete).toHaveBeenCalledWith({ 
-        success: false, 
-        error: 'Network error' 
-      });
-    });
-  });
-
-  describe('Accessibility', () => {
-    it('should have proper ARIA attributes', () => {
-      const { container } = renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const fileInput = getFileInput(container);
-      expect(fileInput).toHaveAttribute('type', 'file');
-    });
-
-    it('should be keyboard accessible', async () => {
-      const user = userEvent.setup();
-      
-      renderWithProviders(
-        <FileUpload
-          supabase={mockSupabase}
-          appName={mockAppName}
-          orgId={mockOrgId}
-        />
-      );
-
-      const selectButton = screen.getByRole('button', { name: 'Select Files' });
-      await user.click(selectButton);
-
-      // The file input should be triggered
-      expect(selectButton).toBeInTheDocument();
-    });
-  });
-describe('FileUpload Error Handling', () => {
-  const user = userEvent.setup();
-  let restoreConsole: (() => void) | null = null;
-
-  beforeEach(() => {
-    restoreConsole = errorTestingUtils.suppressConsoleErrors();
-  });
-
-  afterEach(() => {
-    if (restoreConsole) {
-      restoreConsole();
-    }
-  });
-
-  it('should handle component errors gracefully', () => {
-    const ErrorComponent = () => {
-      throw new Error('FileUpload component error');
-    };
-
-    renderWithProviders(
-      <TestErrorBoundary>
-        <ErrorComponent />
-      </TestErrorBoundary>
-    );
-
-    expect(screen.getByTestId('error-boundary')).toBeInTheDocument();
-    expect(screen.getByText('Error: FileUpload component error')).toBeInTheDocument();
-  });
-
-  it('should handle async errors', async () => {
-    const AsyncErrorComponent = ({ shouldFail }: { shouldFail: boolean }) => {
-      const [error, setError] = React.useState<Error | null>(null);
-
-      React.useEffect(() => {
-        if (shouldFail) {
-          setError(new Error('Async error'));
-        }
-      }, [shouldFail]);
-
-      if (error) {
-        throw error;
-      }
-
-      return <div>Async component</div>;
-    };
-
-    renderWithProviders(
-      <TestErrorBoundary>
-        <AsyncErrorComponent shouldFail={true} />
-      </TestErrorBoundary>
-    );
-
-    await waitFor(() => {
-      expect(screen.getByTestId('error-boundary')).toBeInTheDocument();
-    });
-  });
-
-  it('should handle user interaction errors', async () => {
-    const ErrorComponent = errorTestingUtils.createErrorThrowingComponent(
-      'FileUpload interaction error',
-      'click'
-    );
-
-    renderWithProviders(
-      <TestErrorBoundary>
-        <ErrorComponent />
-      </TestErrorBoundary>
-    );
-
-    await user.click(screen.getByTestId('error-button'));
-
-    await waitFor(() => {
-      expect(screen.getByTestId('error-display')).toBeInTheDocument();
-    });
-  });
-
-  it('should recover from errors when props change', async () => {
-    const TestComponent = ({ shouldError }: { shouldError: boolean }) => {
-      if (shouldError) {
-        throw new Error('FileUpload prop error');
-      }
-      return <div>Normal content</div>;
-    };
-
-    await errorTestingUtils.testErrorRecovery(
-      TestComponent,
-      { shouldError: true },
-      { shouldError: false }
-    );
-
-    expect(screen.getByText('Normal content')).toBeInTheDocument();
-  });
-});
-});