@jmruthers/pace-core 0.6.9 → 0.6.10

package/src/components/PaceLoginPage/PaceLoginPage.tsx

@@ -126,9 +126,8 @@ import { useNavigate, useLocation } from 'react-router-dom';
 import { UnifiedAuthContext } from '../../providers/services/UnifiedAuthProvider';
 import { isSuperAdmin } from '../../rbac/api';
 import { LoginForm } from '../LoginForm';
-import { Button, Input, Label } from '..';
 import { clearPalette } from '../../theming/runtime';
-import { EventServiceContext } from '../../providers/services/EventServiceProvider';
+import { EventServiceContext } from '../../providers/services/EventServiceContext';
 import { logger } from '../../utils/core/logger';
 
 /**
@@ -221,7 +220,7 @@ export const PaceLoginPage: React.FC<PaceLoginPageProps> = ({
         if (isOnLoginPage && eventService) {
           await eventService.restorePersistedEvent();
         }
-      } catch (error) {
+      } catch (_error) {
         // Service may not be available yet or events not loaded - that's okay
       }
     };

package/src/components/PasswordChange/PasswordChangeForm.test.tsx

@@ -13,7 +13,7 @@ import React from 'react';
 import { render, screen, fireEvent, waitFor } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { PasswordChangeForm, PasswordChangeFormProps, PasswordChangeFormValues } from './PasswordChangeForm';
+import { PasswordChangeForm, PasswordChangeFormProps } from './PasswordChangeForm';
 
 // Mock child components
 vi.mock('../Button/Button', () => ({

package/src/components/Progress/Progress.test.tsx

@@ -1,8 +1,22 @@
+/**
+ * @file Progress Component Tests
+ * @package @jmruthers/pace-core
+ * @module Components/Progress
+ * @since 0.1.0
+ *
+ * Comprehensive tests for Progress component covering:
+ * - Rendering and attributes
+ * - Value and max handling
+ * - Indeterminate state
+ * - Accessibility
+ * - Edge cases
+ */
+
 import React from 'react';
 import { renderWithProviders, screen } from '../../__tests__/helpers/test-utils';
 import { Progress } from './Progress';
 
-describe('[component] Progress', () => {
+describe('Progress Component', () => {
   it('renders a native progress element with correct attributes', () => {
     renderWithProviders(<Progress value={40} max={200} />);
 
@@ -43,6 +57,118 @@ describe('[component] Progress', () => {
     const progressbar = screen.getByRole('progressbar');
     expect(progressbar).not.toHaveAttribute('value');
   });
+
+  it('renders with custom className', () => {
+    renderWithProviders(<Progress value={50} className="custom-progress-class" />);
+    const progressbar = screen.getByRole('progressbar');
+    expect(progressbar).toHaveClass('custom-progress-class');
+  });
+
+  it('forwards ref correctly', () => {
+    const ref = React.createRef<HTMLProgressElement>();
+    renderWithProviders(<Progress value={50} ref={ref} />);
+    expect(ref.current).toBeInstanceOf(HTMLProgressElement);
+    expect(ref.current).toHaveAttribute('value', '50');
+  });
+
+  it('handles value of 0', () => {
+    renderWithProviders(<Progress value={0} />);
+    const progressbar = screen.getByRole('progressbar');
+    expect(progressbar).toHaveAttribute('value', '0');
+  });
+
+  it('handles value equal to max', () => {
+    renderWithProviders(<Progress value={100} max={100} />);
+    const progressbar = screen.getByRole('progressbar');
+    expect(progressbar).toHaveAttribute('value', '100');
+    expect(progressbar).toHaveAttribute('max', '100');
+  });
+
+  it('handles value greater than max', () => {
+    renderWithProviders(<Progress value={150} max={100} />);
+    const progressbar = screen.getByRole('progressbar');
+    expect(progressbar).toHaveAttribute('value', '150');
+    expect(progressbar).toHaveAttribute('max', '100');
+  });
+
+  it('handles negative value', () => {
+    renderWithProviders(<Progress value={-10} />);
+    const progressbar = screen.getByRole('progressbar');
+    expect(progressbar).toHaveAttribute('value', '-10');
+  });
+
+  it('handles decimal values', () => {
+    renderWithProviders(<Progress value={50.5} />);
+    const progressbar = screen.getByRole('progressbar');
+    expect(progressbar).toHaveAttribute('value', '50.5');
+  });
+
+  it('handles very large max values', () => {
+    renderWithProviders(<Progress value={1000} max={10000} />);
+    const progressbar = screen.getByRole('progressbar');
+    expect(progressbar).toHaveAttribute('value', '1000');
+    expect(progressbar).toHaveAttribute('max', '10000');
+  });
+});
+
+describe('Progress Component Accessibility', () => {
+  it('has proper ARIA role', () => {
+    renderWithProviders(<Progress value={50} />);
+    const progressbar = screen.getByRole('progressbar');
+    expect(progressbar).toBeInTheDocument();
+  });
+
+  it('supports aria-label', () => {
+    renderWithProviders(<Progress value={50} aria-label="Upload progress" />);
+    const progressbar = screen.getByRole('progressbar', { name: 'Upload progress' });
+    expect(progressbar).toBeInTheDocument();
+  });
+
+  it('supports aria-valuetext for custom announcements', () => {
+    renderWithProviders(<Progress value={50} aria-valuetext="50 percent complete" />);
+    const progressbar = screen.getByRole('progressbar');
+    expect(progressbar).toHaveAttribute('aria-valuetext', '50 percent complete');
+  });
+});
+
+describe('Progress Component Edge Cases', () => {
+  it('handles rapid value changes', () => {
+    const { rerender } = renderWithProviders(<Progress value={0} />);
+    const progressbar = screen.getByRole('progressbar');
+
+    rerender(<Progress value={25} />);
+    expect(progressbar).toHaveAttribute('value', '25');
+
+    rerender(<Progress value={50} />);
+    expect(progressbar).toHaveAttribute('value', '50');
+
+    rerender(<Progress value={75} />);
+    expect(progressbar).toHaveAttribute('value', '75');
+
+    rerender(<Progress value={100} />);
+    expect(progressbar).toHaveAttribute('value', '100');
+  });
+
+  it('handles switching between determinate and indeterminate', () => {
+    const { rerender } = renderWithProviders(<Progress value={50} />);
+    const progressbar = screen.getByRole('progressbar');
+    expect(progressbar).toHaveAttribute('value', '50');
+
+    rerender(<Progress />);
+    expect(progressbar).not.toHaveAttribute('value');
+
+    rerender(<Progress value={75} />);
+    expect(progressbar).toHaveAttribute('value', '75');
+  });
+
+  it('handles max value changes', () => {
+    const { rerender } = renderWithProviders(<Progress value={50} max={100} />);
+    const progressbar = screen.getByRole('progressbar');
+    expect(progressbar).toHaveAttribute('max', '100');
+
+    rerender(<Progress value={50} max={200} />);
+    expect(progressbar).toHaveAttribute('max', '200');
+  });
 });