@jmruthers/pace-core 0.2.4

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

@@ -0,0 +1,937 @@
+/**
+ * @vitest-environment jsdom
+ */
+
+import React from 'react';
+import { screen, fireEvent, waitFor, act } from '@testing-library/react';
+import userEvent from '@testing-library/user-event';
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import { FileUpload } from '../FileUpload';
+import { SupabaseClient } from '@supabase/supabase-js';
+import { renderWithProviders } 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 Unmocked 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 Rendering and Initialization', () => {
+    it('should render with default props and initialize 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();
+      
+      // Should not show progress initially
+      expect(screen.queryByText(/Progress:/)).not.toBeInTheDocument();
+      
+      // Should not show error initially
+      expect(screen.queryByText(/Upload failed/)).not.toBeInTheDocument();
+    });
+
+    it('should render with custom children', () => {
+      const customContent = <div data-testid="custom-content">Custom Upload Area</div>;
+      
+      renderWithProviders(
+        <FileUpload
+          supabase={mockSupabase}
+          appName={mockAppName}
+          orgId={mockOrgId}
+        >
+          {customContent}
+        </FileUpload>
+      );
+
+      expect(screen.getByTestId('custom-content')).toBeInTheDocument();
+      expect(screen.queryByText('Drag and drop files here or click to select')).not.toBeInTheDocument();
+    });
+
+    it('should apply custom className', () => {
+      const { container } = renderWithProviders(
+        <FileUpload
+          supabase={mockSupabase}
+          appName={mockAppName}
+          orgId={mockOrgId}
+          className="custom-class"
+        />
+      );
+
+      expect(container.querySelector('.file-upload')).toHaveClass('custom-class');
+    });
+
+    it('should render with disabled state', () => {
+      renderWithProviders(
+        <FileUpload
+          supabase={mockSupabase}
+          appName={mockAppName}
+          orgId={mockOrgId}
+          disabled={true}
+        />
+      );
+
+      const fileInput = screen.getByRole('button', { name: 'Select Files' });
+      expect(fileInput).toBeDisabled();
+    });
+
+    it('should have correct input attributes', () => {
+      const { container } = renderWithProviders(
+        <FileUpload
+          supabase={mockSupabase}
+          appName={mockAppName}
+          orgId={mockOrgId}
+          accept="image/*"
+          multiple={true}
+        />
+      );
+
+      const fileInput = getFileInput(container);
+      expect(fileInput).toHaveAttribute('accept', 'image/*');
+      expect(fileInput).toHaveAttribute('multiple');
+    });
+  });
+
+  describe('File Selection and Validation Logic', () => {
+    it('should handle single file selection and validate', () => {
+      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 and validate each', () => {
+      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 and disable upload', () => {
+      (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();
+    });
+
+    it('should handle null file list gracefully', () => {
+      const { container } = renderWithProviders(
+        <FileUpload
+          supabase={mockSupabase}
+          appName={mockAppName}
+          orgId={mockOrgId}
+        />
+      );
+
+      const fileInput = getFileInput(container);
+      
+      // Trigger file selection with null files
+      fireEvent.change(fileInput, { target: { files: null } });
+
+      // Should not show files section
+      expect(screen.queryByText('Selected Files:')).not.toBeInTheDocument();
+      
+      // Should not validate anything
+      expect(validateFileSize).not.toHaveBeenCalled();
+    });
+
+    it('should handle empty file list gracefully', () => {
+      const { container } = renderWithProviders(
+        <FileUpload
+          supabase={mockSupabase}
+          appName={mockAppName}
+          orgId={mockOrgId}
+        />
+      );
+
+      const fileInput = getFileInput(container);
+      
+      // Trigger file selection with empty files
+      fireEvent.change(fileInput, { target: { files: [] } });
+
+      // Should not show files section
+      expect(screen.queryByText('Selected Files:')).not.toBeInTheDocument();
+      
+      // Should not validate anything
+      expect(validateFileSize).not.toHaveBeenCalled();
+    });
+  });
+
+  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 and Callbacks', () => {
+    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();
+      });
+    });
+
+    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' 
+      });
+    });
+
+    it('should handle upload with no callbacks', 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] } });
+
+      const uploadButton = screen.getByRole('button', { name: 'Upload 1 file' });
+      await userEvent.click(uploadButton);
+
+      // Should not throw error
+      expect(mockUploadWithProgress).toHaveBeenCalledWith(mockFile);
+    });
+  });
+
+  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 not display progress when not uploading', () => {
+      renderWithProviders(
+        <FileUpload
+          supabase={mockSupabase}
+          appName={mockAppName}
+          orgId={mockOrgId}
+        />
+      );
+
+      expect(screen.queryByText(/Progress:/)).not.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();
+    });
+
+    it('should not display error when no error', () => {
+      renderWithProviders(
+        <FileUpload
+          supabase={mockSupabase}
+          appName={mockAppName}
+          orgId={mockOrgId}
+        />
+      );
+
+      expect(screen.queryByText(/Upload failed/)).not.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('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();
+    });
+  });
+});