@jmruthers/pace-core 0.5.68 → 0.5.69

package/src/rbac/components/__tests__/NavigationGuard.test.tsx

@@ -90,11 +90,7 @@ describe('NavigationGuard Component', () => {
     });
   });
 
-  afterEach(() => {
-    vi.restoreAllMocks();
-  });
-
-  describe('Rendering', () => {
+    describe('Rendering', () => {
     it('renders children when permission is granted', async () => {
       mockUseCan.mockReturnValue({
         can: true,
@@ -111,7 +107,7 @@ describe('NavigationGuard Component', () => {
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
         expect(screen.getByText('Navigation Link')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
 
     it('renders fallback when permission is denied', async () => {
@@ -133,7 +129,7 @@ describe('NavigationGuard Component', () => {
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
         expect(screen.queryByTestId('test-component')).not.toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
 
     it('shows loading state during permission check', () => {
@@ -171,7 +167,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByText('Access Denied')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
 
     it('uses default loading when none provided', () => {
@@ -207,7 +203,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         'user-123',
@@ -241,7 +237,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       // Should check the first permission as representative
       expect(mockUseCan).toHaveBeenCalledWith(
@@ -276,7 +272,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
 
     it('handles permission checking errors gracefully', async () => {
@@ -298,7 +294,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
   });
 
@@ -327,7 +323,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         'user-123',
@@ -353,7 +349,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         'user-123',
@@ -389,7 +385,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         'user-123',
@@ -431,7 +427,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockCreateScopeFromEvent).toHaveBeenCalledWith({}, 'event-123');
       expect(mockUseCan).toHaveBeenCalledWith(
@@ -468,7 +464,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByText('Checking...')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
 
     it('handles missing context gracefully', async () => {
@@ -490,7 +486,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByText('Checking...')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
   });
 
@@ -516,7 +512,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(consoleSpy).toHaveBeenCalledWith(
         expect.stringContaining('STRICT MODE VIOLATION'),
@@ -551,7 +547,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(consoleSpy).toHaveBeenCalledWith(
         expect.stringContaining('Navigation access attempt'),
@@ -587,7 +583,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(onDeniedSpy).toHaveBeenCalledWith(mockNavigationItem);
     });
@@ -612,7 +608,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(onDeniedSpy).not.toHaveBeenCalled();
     });
@@ -640,7 +636,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(consoleSpy).not.toHaveBeenCalledWith(
         expect.stringContaining('STRICT MODE VIOLATION')
@@ -670,7 +666,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(consoleSpy).not.toHaveBeenCalledWith(
         expect.stringContaining('Navigation access attempt')
@@ -697,7 +693,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       // Should still check the first permission as representative
       expect(mockUseCan).toHaveBeenCalledWith(
@@ -739,7 +735,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         '',
@@ -772,7 +768,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
   });
 
@@ -803,7 +799,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         'user-123',
@@ -837,7 +833,7 @@ describe('NavigationGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
   });
 });

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

@@ -104,11 +104,7 @@ describe('PagePermissionGuard Component', () => {
     });
   });
 
-  afterEach(() => {
-    vi.restoreAllMocks();
-  });
-
-  describe('Rendering', () => {
+    describe('Rendering', () => {
     it('renders children when permission is granted', async () => {
       mockUseCan.mockReturnValue({
         can: true,
@@ -128,7 +124,7 @@ describe('PagePermissionGuard Component', () => {
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
         expect(screen.getByText('Protected Page')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
 
     it('renders fallback when permission is denied', async () => {
@@ -151,7 +147,7 @@ describe('PagePermissionGuard Component', () => {
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
         expect(screen.queryByTestId('test-component')).not.toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
 
     it('shows loading state during permission check', () => {
@@ -194,7 +190,7 @@ describe('PagePermissionGuard Component', () => {
       await waitFor(() => {
         expect(screen.getByText('Access Denied')).toBeInTheDocument();
         expect(screen.getByText('You don\'t have permission to access this page.')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
 
     it('uses default loading when none provided', () => {
@@ -236,7 +232,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         'user-123',
@@ -272,7 +268,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         'user-123',
@@ -310,7 +306,7 @@ describe('PagePermissionGuard Component', () => {
 
         await waitFor(() => {
           expect(screen.getByTestId('test-component')).toBeInTheDocument();
-        });
+        }, { interval: 10 });
 
         expect(mockUseCan).toHaveBeenCalledWith(
           'user-123',
@@ -348,7 +344,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByText('Checking permissions...')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
   });
 
@@ -371,7 +367,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockGetCurrentAppName).toHaveBeenCalled();
     });
@@ -393,6 +389,7 @@ describe('PagePermissionGuard Component', () => {
         <PagePermissionGuard
           pageName={mockPageName}
           operation={mockOperation}
+          scope={mockScope}
         >
           <TestComponent>Protected Page</TestComponent>
         </PagePermissionGuard>
@@ -400,7 +397,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       // Restore NODE_ENV
       process.env.NODE_ENV = originalEnv;
@@ -424,7 +421,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByText('Checking permissions...')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       // Restore environment
       vi.unstubAllEnvs();
@@ -470,7 +467,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByText('Checking permissions...')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       // Restore NODE_ENV
       process.env.NODE_ENV = originalEnv;
@@ -503,7 +500,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         'user-123',
@@ -532,7 +529,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         'user-123',
@@ -585,7 +582,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         'user-123',
@@ -644,7 +641,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockCreateScopeFromEvent).toHaveBeenCalledWith(
         expect.any(Object),
@@ -686,7 +683,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByText('Checking permissions...')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
 
     it('handles missing context gracefully', async () => {
@@ -709,7 +706,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByText('Checking permissions...')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
   });
 
@@ -736,7 +733,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByText('Checking permissions...')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(consoleSpy).toHaveBeenCalledWith(
         expect.stringContaining('STRICT MODE VIOLATION'),
@@ -772,7 +769,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByText('Checking permissions...')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(consoleSpy).toHaveBeenCalledWith(
         expect.stringContaining('Page access attempt'),
@@ -809,7 +806,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByText('Checking permissions...')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(onDeniedSpy).toHaveBeenCalledWith(mockPageName, mockOperation);
     });
@@ -835,7 +832,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(onDeniedSpy).not.toHaveBeenCalled();
     });
@@ -864,7 +861,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByText('Checking permissions...')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(consoleSpy).not.toHaveBeenCalledWith(
         expect.stringContaining('STRICT MODE VIOLATION')
@@ -895,7 +892,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByText('Checking permissions...')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(consoleSpy).not.toHaveBeenCalledWith(
         expect.stringContaining('Page access attempt')
@@ -945,7 +942,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByText('Checking permissions...')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         '',
@@ -997,7 +994,7 @@ describe('PagePermissionGuard Component', () => {
 
       await waitFor(() => {
         expect(screen.getByText('Checking permissions...')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       // Restore NODE_ENV
       process.env.NODE_ENV = originalEnv;

package/src/rbac/components/__tests__/PermissionEnforcer.test.tsx

@@ -82,11 +82,7 @@ describe('PermissionEnforcer Component', () => {
     });
   });
 
-  afterEach(() => {
-    vi.restoreAllMocks();
-  });
-
-  describe('Rendering', () => {
+    describe('Rendering', () => {
     it('renders children when permission is granted', async () => {
       mockUseCan.mockReturnValue({
         can: true,
@@ -106,7 +102,7 @@ describe('PermissionEnforcer Component', () => {
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
         expect(screen.getByText('Protected Content')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
 
     it('renders fallback when permission is denied', async () => {
@@ -129,7 +125,7 @@ describe('PermissionEnforcer Component', () => {
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
         expect(screen.queryByTestId('test-component')).not.toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
 
     it('shows loading state during permission check', () => {
@@ -172,7 +168,7 @@ describe('PermissionEnforcer Component', () => {
       await waitFor(() => {
         expect(screen.getByText('Access Denied')).toBeInTheDocument();
         expect(screen.getByText('You don\'t have permission to perform this operation.')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
 
     it('uses default loading when none provided', () => {
@@ -216,7 +212,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         'user-123',
@@ -249,7 +245,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       // Should check the first permission as representative
       expect(mockUseCan).toHaveBeenCalledWith(
@@ -284,7 +280,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
 
     it('handles empty permissions array', async () => {
@@ -305,7 +301,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
   });
 
@@ -335,7 +331,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         'user-123',
@@ -364,7 +360,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         'user-123',
@@ -403,7 +399,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         'user-123',
@@ -448,7 +444,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockCreateScopeFromEvent).toHaveBeenCalledWith({}, 'event-123');
       expect(mockUseCan).toHaveBeenCalledWith(
@@ -486,7 +482,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
 
     it('handles missing context gracefully', async () => {
@@ -509,7 +505,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
   });
 
@@ -536,7 +532,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(consoleSpy).toHaveBeenCalledWith(
         expect.stringContaining('STRICT MODE VIOLATION'),
@@ -572,7 +568,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(consoleSpy).toHaveBeenCalledWith(
         expect.stringContaining('Permission check attempt'),
@@ -609,7 +605,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(onDeniedSpy).toHaveBeenCalledWith(mockPermissions, mockOperation);
     });
@@ -635,7 +631,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(onDeniedSpy).not.toHaveBeenCalled();
     });
@@ -664,7 +660,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(consoleSpy).not.toHaveBeenCalledWith(
         expect.stringContaining('STRICT MODE VIOLATION')
@@ -695,7 +691,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(consoleSpy).not.toHaveBeenCalledWith(
         expect.stringContaining('Permission check attempt')
@@ -723,7 +719,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-component')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       // Should still check the first permission as representative
       expect(mockUseCan).toHaveBeenCalledWith(
@@ -766,7 +762,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
 
       expect(mockUseCan).toHaveBeenCalledWith(
         '',
@@ -800,7 +796,7 @@ describe('PermissionEnforcer Component', () => {
 
       await waitFor(() => {
         expect(screen.getByTestId('test-fallback')).toBeInTheDocument();
-      });
+      }, { interval: 10 });
     });
   });
 });