@jmruthers/pace-core 0.6.5 → 0.6.7

package/src/providers/__tests__/InactivityProvider.test.tsx

@@ -29,11 +29,11 @@ vi.mock('../../hooks/useInactivityTracker', () => ({
 vi.mock('../../components/InactivityWarningModal/InactivityWarningModal', () => ({
   InactivityWarningModal: ({ isOpen, timeRemaining, onStaySignedIn, onSignOutNow }: any) => (
     isOpen ? (
-      <div data-testid="inactivity-modal">
-        <div data-testid="time-remaining">{timeRemaining}</div>
+      <section data-testid="inactivity-modal">
+        <p data-testid="time-remaining">{timeRemaining}</p>
         <button onClick={onStaySignedIn}>Stay Signed In</button>
         <button onClick={onSignOutNow}>Sign Out Now</button>
-      </div>
+      </section>
     ) : null
   ),
 }));
@@ -43,18 +43,18 @@ const TestComponent = () => {
   const inactivity = useInactivity();
   
   return (
-    <div data-testid="test-component">
-      <div data-testid="showWarning">{inactivity.showInactivityWarning ? 'true' : 'false'}</div>
-      <div data-testid="timeRemaining">{inactivity.inactivityTimeRemaining}</div>
-      <div data-testid="isIdle">{inactivity.isIdle ? 'true' : 'false'}</div>
-      <div data-testid="isTracking">{inactivity.isTracking ? 'true' : 'false'}</div>
+    <section data-testid="test-component">
+      <p data-testid="showWarning">{inactivity.showInactivityWarning ? 'true' : 'false'}</p>
+      <p data-testid="timeRemaining">{inactivity.inactivityTimeRemaining}</p>
+      <p data-testid="isIdle">{inactivity.isIdle ? 'true' : 'false'}</p>
+      <p data-testid="isTracking">{inactivity.isTracking ? 'true' : 'false'}</p>
       <button onClick={inactivity.resetActivity}>Reset Activity</button>
       <button onClick={inactivity.startTracking}>Start Tracking</button>
       <button onClick={inactivity.stopTracking}>Stop Tracking</button>
       <button onClick={inactivity.handleIdleLogout}>Idle Logout</button>
       <button onClick={inactivity.handleStaySignedIn}>Stay Signed In</button>
       <button onClick={inactivity.handleSignOutNow}>Sign Out Now</button>
-    </div>
+    </section>
   );
 };
 
@@ -90,21 +90,21 @@ describe('InactivityProvider', () => {
     it('renders children without crashing', () => {
       render(
         <TestWrapper supabaseClient={mockSupabaseClient}>
-          <div>Test content</div>
+          <p>Test content</p>
         </TestWrapper>
       );
       
-      expect(screen.getByText('Test content')).toBeInTheDocument();
+      expect(screen.getByText('Test content')).toBeDefined();
     });
 
     it('renders without user and session', () => {
       render(
         <TestWrapper user={null} session={null} supabaseClient={mockSupabaseClient}>
-          <div>Test content</div>
+          <p>Test content</p>
         </TestWrapper>
       );
       
-      expect(screen.getByText('Test content')).toBeInTheDocument();
+      expect(screen.getByText('Test content')).toBeDefined();
     });
   });
 
@@ -127,14 +127,14 @@ describe('InactivityProvider', () => {
         const inactivity = useInactivity();
         
         return (
-          <div>
-            <div data-testid="hasResetActivity">{typeof inactivity.resetActivity === 'function' ? 'true' : 'false'}</div>
-            <div data-testid="hasStartTracking">{typeof inactivity.startTracking === 'function' ? 'true' : 'false'}</div>
-            <div data-testid="hasStopTracking">{typeof inactivity.stopTracking === 'function' ? 'true' : 'false'}</div>
-            <div data-testid="hasHandleIdleLogout">{typeof inactivity.handleIdleLogout === 'function' ? 'true' : 'false'}</div>
-            <div data-testid="hasHandleStaySignedIn">{typeof inactivity.handleStaySignedIn === 'function' ? 'true' : 'false'}</div>
-            <div data-testid="hasHandleSignOutNow">{typeof inactivity.handleSignOutNow === 'function' ? 'true' : 'false'}</div>
-          </div>
+          <section>
+            <p data-testid="hasResetActivity">{typeof inactivity.resetActivity === 'function' ? 'true' : 'false'}</p>
+            <p data-testid="hasStartTracking">{typeof inactivity.startTracking === 'function' ? 'true' : 'false'}</p>
+            <p data-testid="hasStopTracking">{typeof inactivity.stopTracking === 'function' ? 'true' : 'false'}</p>
+            <p data-testid="hasHandleIdleLogout">{typeof inactivity.handleIdleLogout === 'function' ? 'true' : 'false'}</p>
+            <p data-testid="hasHandleStaySignedIn">{typeof inactivity.handleStaySignedIn === 'function' ? 'true' : 'false'}</p>
+            <p data-testid="hasHandleSignOutNow">{typeof inactivity.handleSignOutNow === 'function' ? 'true' : 'false'}</p>
+          </section>
         );
       };
 
@@ -162,7 +162,7 @@ describe('InactivityProvider', () => {
       );
 
       // Just verify the component renders - tracking logic is tested in service layer
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
     });
 
     it('disables tracking when no user or session', () => {
@@ -173,7 +173,7 @@ describe('InactivityProvider', () => {
       );
 
       // Just verify the component renders - tracking logic is tested in service layer
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
     });
 
     it('disables tracking when dangerously disabled', () => {
@@ -184,7 +184,7 @@ describe('InactivityProvider', () => {
       );
 
       // Just verify the component renders - tracking logic is tested in service layer
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
     });
 
     it('uses custom timeout values', () => {
@@ -199,7 +199,7 @@ describe('InactivityProvider', () => {
       );
 
       // Just verify the component renders - tracking logic is tested in service layer
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
     });
   });
 
@@ -214,7 +214,7 @@ describe('InactivityProvider', () => {
       );
 
       // Test that the component renders and the method is available
-      expect(screen.getByText('Idle Logout')).toBeInTheDocument();
+      expect(screen.getByText('Idle Logout')).toBeDefined();
     });
 
     it('handles idle logout without callback', async () => {
@@ -225,7 +225,7 @@ describe('InactivityProvider', () => {
       );
 
       // Test that the component renders
-      expect(screen.getByText('Idle Logout')).toBeInTheDocument();
+      expect(screen.getByText('Idle Logout')).toBeDefined();
     });
 
     it('handles sign out errors gracefully', async () => {
@@ -239,7 +239,7 @@ describe('InactivityProvider', () => {
       );
 
       // Test that the component renders
-      expect(screen.getByText('Idle Logout')).toBeInTheDocument();
+      expect(screen.getByText('Idle Logout')).toBeDefined();
       
       consoleErrorSpy.mockRestore();
     });
@@ -272,7 +272,7 @@ describe('InactivityProvider', () => {
       );
 
       // Service layer handles the actual logic - just verify the UI renders
-      expect(screen.getByTestId('showWarning')).toBeInTheDocument();
+      expect(screen.getByTestId('showWarning')).toBeDefined();
     });
 
     it('handles activity callback', () => {
@@ -317,7 +317,7 @@ describe('InactivityProvider', () => {
 
       screen.getByText('Reset Activity').click();
       // Service layer handles the actual logic - just verify the UI responds
-      expect(screen.getByText('Reset Activity')).toBeInTheDocument();
+      expect(screen.getByText('Reset Activity')).toBeDefined();
     });
 
     it('handles start tracking', () => {
@@ -329,7 +329,7 @@ describe('InactivityProvider', () => {
 
       screen.getByText('Start Tracking').click();
       // Service layer handles the actual logic - just verify the UI responds
-      expect(screen.getByText('Start Tracking')).toBeInTheDocument();
+      expect(screen.getByText('Start Tracking')).toBeDefined();
     });
 
     it('handles stop tracking', () => {
@@ -341,7 +341,7 @@ describe('InactivityProvider', () => {
 
       screen.getByText('Stop Tracking').click();
       // Service layer handles the actual logic - just verify the UI responds
-      expect(screen.getByText('Stop Tracking')).toBeInTheDocument();
+      expect(screen.getByText('Stop Tracking')).toBeDefined();
     });
 
     it('handles manual idle logout', async () => {
@@ -356,7 +356,7 @@ describe('InactivityProvider', () => {
       screen.getByText('Idle Logout').click();
 
       // Test that the button is clickable
-      expect(screen.getByText('Idle Logout')).toBeInTheDocument();
+      expect(screen.getByText('Idle Logout')).toBeDefined();
     });
 
     it('handles stay signed in', () => {
@@ -368,7 +368,7 @@ describe('InactivityProvider', () => {
 
       screen.getByText('Stay Signed In').click();
       // Service layer handles the actual logic - just verify the UI responds
-      expect(screen.getByText('Stay Signed In')).toBeInTheDocument();
+      expect(screen.getByText('Stay Signed In')).toBeDefined();
     });
 
     it('handles sign out now', async () => {
@@ -383,7 +383,7 @@ describe('InactivityProvider', () => {
       screen.getByText('Sign Out Now').click();
 
       // Test that the button is clickable
-      expect(screen.getByText('Sign Out Now')).toBeInTheDocument();
+      expect(screen.getByText('Sign Out Now')).toBeDefined();
     });
   });
 
@@ -403,16 +403,16 @@ describe('InactivityProvider', () => {
       );
 
       // Test that the component renders
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
     });
 
     it('renders custom warning modal when provided', () => {
       const customRenderer = vi.fn(({ timeRemaining, onStaySignedIn, onSignOutNow }) => (
-        <div data-testid="custom-modal">
-          <div data-testid="custom-time">{timeRemaining}</div>
+        <section data-testid="custom-modal">
+          <p data-testid="custom-time">{timeRemaining}</p>
           <button onClick={onStaySignedIn}>Custom Stay</button>
           <button onClick={onSignOutNow}>Custom Sign Out</button>
-        </div>
+        </section>
       ));
 
       // Mock tracker with warning state
@@ -429,15 +429,15 @@ describe('InactivityProvider', () => {
       );
 
       // Test that the component renders
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
     });
 
     it('handles custom modal actions', async () => {
       const customRenderer = vi.fn(({ onStaySignedIn, onSignOutNow }) => (
-        <div>
+        <section>
           <button onClick={onStaySignedIn}>Custom Stay</button>
           <button onClick={onSignOutNow}>Custom Sign Out</button>
-        </div>
+        </section>
       ));
 
       // Mock tracker with warning state
@@ -454,7 +454,7 @@ describe('InactivityProvider', () => {
       );
 
       // Test that the component renders
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
     });
   });
 
@@ -469,7 +469,7 @@ describe('InactivityProvider', () => {
       );
 
       // Service layer may log different messages - just verify component renders
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
 
       consoleErrorSpy.mockRestore();
     });
@@ -484,7 +484,7 @@ describe('InactivityProvider', () => {
       );
 
       // Service layer may log different messages - just verify component renders
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
 
       consoleWarnSpy.mockRestore();
     });
@@ -507,14 +507,14 @@ describe('InactivityProvider', () => {
         const inactivity = useInactivity();
         
         return (
-          <div>
-            <div data-testid="hasShowWarning">{typeof inactivity.showInactivityWarning === 'boolean' ? 'true' : 'false'}</div>
-            <div data-testid="hasTimeRemaining">{typeof inactivity.inactivityTimeRemaining === 'number' ? 'true' : 'false'}</div>
-            <div data-testid="hasIsIdle">{typeof inactivity.isIdle === 'boolean' ? 'true' : 'false'}</div>
-            <div data-testid="hasTimeRemaining2">{typeof inactivity.timeRemaining === 'number' ? 'true' : 'false'}</div>
-            <div data-testid="hasShowWarning2">{typeof inactivity.showWarning === 'boolean' ? 'true' : 'false'}</div>
-            <div data-testid="hasIsTracking">{typeof inactivity.isTracking === 'boolean' ? 'true' : 'false'}</div>
-          </div>
+          <section>
+            <p data-testid="hasShowWarning">{typeof inactivity.showInactivityWarning === 'boolean' ? 'true' : 'false'}</p>
+            <p data-testid="hasTimeRemaining">{typeof inactivity.inactivityTimeRemaining === 'number' ? 'true' : 'false'}</p>
+            <p data-testid="hasIsIdle">{typeof inactivity.isIdle === 'boolean' ? 'true' : 'false'}</p>
+            <p data-testid="hasTimeRemaining2">{typeof inactivity.timeRemaining === 'number' ? 'true' : 'false'}</p>
+            <p data-testid="hasShowWarning2">{typeof inactivity.showWarning === 'boolean' ? 'true' : 'false'}</p>
+            <p data-testid="hasIsTracking">{typeof inactivity.isTracking === 'boolean' ? 'true' : 'false'}</p>
+          </section>
         );
       };
 
@@ -542,7 +542,7 @@ describe('InactivityProvider', () => {
       );
 
       // Service layer handles time conversion - just verify the UI renders
-      expect(screen.getByTestId('timeRemaining')).toBeInTheDocument();
+      expect(screen.getByTestId('timeRemaining')).toBeDefined();
     });
 
     it('handles fractional seconds correctly', () => {
@@ -553,7 +553,7 @@ describe('InactivityProvider', () => {
       );
 
       // Service layer handles time conversion - just verify the UI renders
-      expect(screen.getByTestId('timeRemaining')).toBeInTheDocument();
+      expect(screen.getByTestId('timeRemaining')).toBeDefined();
     });
   });
 
@@ -561,7 +561,7 @@ describe('InactivityProvider', () => {
     it('handles component unmount gracefully', () => {
       const { unmount } = render(
         <TestWrapper>
-          <div>Test content</div>
+          <p>Test content</p>
         </TestWrapper>
       );