@jmruthers/pace-core 0.5.68 → 0.5.70

package/src/__tests__/rbac/PagePermissionGuard.test.tsx

@@ -71,16 +71,22 @@ describe('PagePermissionGuard', () => {
   });
 
   describe('UI State Management', () => {
-    it('should render without crashing', () => {
+    it('should render without crashing', async () => {
       render(
-        <PagePermissionGuard pageName="test" operation="read">
+        <PagePermissionGuard 
+          pageName="test" 
+          operation="read"
+          scope={mockScope}
+        >
           <div>Protected Content</div>
         </PagePermissionGuard>
       );
 
       // Should render something (either loading, content, or fallback)
       // With default mock (can: false), should show access denied
-      expect(screen.getByText('Access Denied')).toBeInTheDocument();
+      await waitFor(() => {
+        expect(screen.getByText('Access Denied')).toBeInTheDocument();
+      });
     });
 
     it('should show protected content when permission is granted', async () => {
@@ -92,14 +98,18 @@ describe('PagePermissionGuard', () => {
       });
 
       render(
-        <PagePermissionGuard pageName="test" operation="read">
+        <PagePermissionGuard 
+          pageName="test" 
+          operation="read"
+          scope={mockScope}
+        >
           <div>Protected Content</div>
         </PagePermissionGuard>
       );
 
       await waitFor(() => {
         expect(screen.getByText('Protected Content')).toBeInTheDocument();
-      });
+      }, { timeout: 3000 });
     });
   });
 
@@ -113,14 +123,18 @@ describe('PagePermissionGuard', () => {
       });
 
       render(
-        <PagePermissionGuard pageName="test" operation="read">
+        <PagePermissionGuard 
+          pageName="test" 
+          operation="read"
+          scope={mockScope}
+        >
           <div>Protected Content</div>
         </PagePermissionGuard>
       );
 
       await waitFor(() => {
         expect(screen.getByText('Protected Content')).toBeInTheDocument();
-      });
+      }, { timeout: 3000 });
     });
   });
 
@@ -134,13 +148,19 @@ describe('PagePermissionGuard', () => {
       });
 
       render(
-        <PagePermissionGuard pageName="test" operation="read">
+        <PagePermissionGuard 
+          pageName="test" 
+          operation="read"
+          scope={mockScope}
+        >
           <div>Protected Content</div>
         </PagePermissionGuard>
       );
 
       // Should render access denied when there's an error
-      expect(screen.getByText('Access Denied')).toBeInTheDocument();
+      await waitFor(() => {
+        expect(screen.getByText('Access Denied')).toBeInTheDocument();
+      }, { timeout: 3000 });
     });
   });
 
@@ -151,7 +171,11 @@ describe('PagePermissionGuard', () => {
       const TestComponent = () => {
         renderCount++;
         return (
-          <PagePermissionGuard pageName="test" operation="read">
+          <PagePermissionGuard 
+            pageName="test" 
+            operation="read"
+            scope={mockScope}
+          >
             <div>Protected Content</div>
           </PagePermissionGuard>
         );
@@ -178,13 +202,19 @@ describe('PagePermissionGuard', () => {
       } as any);
 
       render(
-        <PagePermissionGuard pageName="test" operation="read">
+        <PagePermissionGuard 
+          pageName="test" 
+          operation="read"
+          scope={mockScope}
+        >
           <div>Protected Content</div>
         </PagePermissionGuard>
       );
 
       // Should render access denied with default mock (can: false)
-      expect(screen.getByText('Access Denied')).toBeInTheDocument();
+      await waitFor(() => {
+        expect(screen.getByText('Access Denied')).toBeInTheDocument();
+      }, { timeout: 3000 });
     });
   });
 });

package/src/__tests__/setup.ts

@@ -6,54 +6,60 @@
 import '@testing-library/jest-dom';
 import { vi } from 'vitest';
 import { cleanup } from '@testing-library/react';
+import { setupFastTimers, cleanupTimers } from './helpers/timer-utils';
 
-// Global test setup
+// Global test setup - optimized for performance
 beforeEach(() => {
-  // Clear all mocks before each test
+  // Only clear mocks that are actually used
   vi.clearAllMocks();
+  // Setup optimized timers for faster tests
+  setupFastTimers();
 });
 
 afterEach(() => {
   // Clean up DOM after each test
   cleanup();
+  // Cleanup timers
+  cleanupTimers();
 });
 
-// Mock console methods to reduce noise in tests
-global.console = {
-  ...console,
-  // Uncomment to suppress console.log in tests
-  // log: vi.fn(),
-  // warn: vi.fn(),
-  // error: vi.fn(),
-};
+// Optimized console mocking - only mock when needed
+// global.console = {
+//   ...console,
+//   // Uncomment to suppress console.log in tests
+//   // log: vi.fn(),
+//   // warn: vi.fn(),
+//   // error: vi.fn(),
+// };
 
-// Mock IntersectionObserver
-global.IntersectionObserver = vi.fn().mockImplementation(() => ({
+// Optimized global mocks - create once and reuse
+const mockObserver = {
   observe: vi.fn(),
   unobserve: vi.fn(),
   disconnect: vi.fn(),
-}));
+};
+
+// Mock IntersectionObserver
+global.IntersectionObserver = vi.fn(() => mockObserver);
 
 // Mock ResizeObserver
-global.ResizeObserver = vi.fn().mockImplementation(() => ({
-  observe: vi.fn(),
-  unobserve: vi.fn(),
-  disconnect: vi.fn(),
+global.ResizeObserver = vi.fn(() => mockObserver);
+
+// Optimized matchMedia mock
+const mockMatchMedia = vi.fn(() => ({
+  matches: false,
+  media: '',
+  onchange: null,
+  addListener: vi.fn(),
+  removeListener: vi.fn(),
+  addEventListener: vi.fn(),
+  removeEventListener: vi.fn(),
+  dispatchEvent: vi.fn(),
 }));
 
-// Mock matchMedia
 Object.defineProperty(window, 'matchMedia', {
   writable: true,
-  value: vi.fn().mockImplementation(query => ({
-    matches: false,
-    media: query,
-    onchange: null,
-    addListener: vi.fn(),
-    removeListener: vi.fn(),
-    addEventListener: vi.fn(),
-    removeEventListener: vi.fn(),
-    dispatchEvent: vi.fn(),
-  })),
+  value: mockMatchMedia,
 });
 
 // Mock scrollTo

package/src/components/Alert/Alert.test.tsx

@@ -10,11 +10,7 @@ import { Alert, AlertTitle, AlertDescription } from './Alert';
 import { renderWithProviders } from '../../__tests__/helpers/test-utils';
 
 describe('Alert Component', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  describe('Rendering', () => {
+    describe('Rendering', () => {
     it('renders with default variant', () => {
       renderWithProviders(
         <Alert>

package/src/components/Avatar/Avatar.test.tsx

@@ -15,11 +15,7 @@ const getAvatarContainer = (fallbackText: string) => {
 };
 
 describe('Avatar Component', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  describe('Rendering', () => {
+    describe('Rendering', () => {
     it('renders with default props', () => {
       renderWithProviders(
         <Avatar>

package/src/components/Button/Button.test.tsx

@@ -24,11 +24,7 @@ vi.mock('../Tooltip', () => ({
 const MockIcon = () => <span data-testid="mock-icon">⚙️</span>;
 
 describe('Button Component', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  describe('Rendering', () => {
+    describe('Rendering', () => {
     it('renders with text content', () => {
       renderWithProviders(<Button>Click me</Button>);
       expect(screen.getByRole('button', { name: 'Click me' })).toBeInTheDocument();
@@ -230,11 +226,7 @@ describe('Button Component', () => {
 });
 
 describe('ButtonGroup Component', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  describe('Rendering', () => {
+    describe('Rendering', () => {
     it('renders with children buttons', () => {
       renderWithProviders(
         <ButtonGroup>
@@ -389,11 +381,7 @@ describe('ButtonGroup Component', () => {
 });
 
 describe('IconButton Component', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  describe('Rendering', () => {
+    describe('Rendering', () => {
     it('renders with icon and aria-label', () => {
       renderWithProviders(
         <IconButton icon={<MockIcon />} aria-label="Settings" />
@@ -572,11 +560,7 @@ describe('IconButton Component', () => {
 });
 
 describe('Button Component Integration', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  describe('Complex Scenarios', () => {
+    describe('Complex Scenarios', () => {
     it('handles rapid clicking without issues', async () => {
       const handleClick = vi.fn();
       const user = userEvent.setup();

package/src/components/Card/Card.test.tsx

@@ -23,11 +23,7 @@ import {
 import { renderWithProviders } from '../../__tests__/helpers/test-utils';
 
 describe('Card Component Suite', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  describe('Card Component', () => {
+    describe('Card Component', () => {
     describe('Rendering', () => {
       it('renders with default props', () => {
         renderWithProviders(<Card>Card content</Card>);

package/src/components/Checkbox/Checkbox.test.tsx

@@ -12,11 +12,7 @@ import { Checkbox } from './Checkbox';
 import { renderWithProviders } from '../../__tests__/helpers/test-utils';
 
 describe('Checkbox Component', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  describe('Rendering', () => {
+    describe('Rendering', () => {
     it('renders with default props', () => {
       renderWithProviders(<Checkbox />);
       const checkbox = screen.getByRole('checkbox');

package/src/components/DataTable/__tests__/DataTable.comprehensive.test.tsx

@@ -54,11 +54,7 @@ describe('DataTable Component', () => {
     vi.clearAllMocks();
   });
 
-  afterEach(() => {
-    vi.restoreAllMocks();
-  });
-
-  describe('Rendering', () => {
+    describe('Rendering', () => {
     it('renders with basic props', () => {
       render(
         <DataTable