@jmruthers/pace-core 0.6.6 → 0.6.7

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

@@ -90,12 +90,12 @@ const TestComponent = () => {
   const { events, isLoading, error, selectedEvent, setSelectedEvent, refreshEvents } = useEvents();
   
   return (
-    <div data-testid="test-component">
-      <div data-testid="events-count">{events.length}</div>
-      <div data-testid="is-loading">{isLoading ? 'true' : 'false'}</div>
-      <div data-testid="error">{error?.message || 'no-error'}</div>
-      <div data-testid="selected-event">{selectedEvent?.event_name || 'no-event'}</div>
-      <div data-testid="selected-event-id">{selectedEvent?.event_id || 'no-id'}</div>
+    <section data-testid="test-component">
+      <p data-testid="events-count">{events.length}</p>
+      <p data-testid="is-loading">{isLoading ? 'true' : 'false'}</p>
+      <p data-testid="error">{error?.message || 'no-error'}</p>
+      <p data-testid="selected-event">{selectedEvent?.event_name || 'no-event'}</p>
+      <p data-testid="selected-event-id">{selectedEvent?.event_id || 'no-id'}</p>
       <button data-testid="select-first-btn" onClick={() => setSelectedEvent(events[0] || null)}>
         Select First Event
       </button>
@@ -105,7 +105,7 @@ const TestComponent = () => {
       <button data-testid="refresh-events-btn" onClick={() => refreshEvents()}>
         Refresh Events
       </button>
-    </div>
+    </section>
   );
 };
 
@@ -169,7 +169,7 @@ describe('EventProvider', () => {
         </TestWrapper>
       );
 
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
       expect(screen.getByTestId('events-count')).toHaveTextContent('0');
       expect(screen.getByTestId('is-loading')).toHaveTextContent('false');
       expect(screen.getByTestId('error')).toHaveTextContent('no-error');
@@ -188,9 +188,9 @@ describe('EventProvider', () => {
       const clearBtn = screen.getByTestId('clear-event-btn');
       const refreshBtn = screen.getByTestId('refresh-events-btn');
 
-      expect(selectBtn).toBeInTheDocument();
-      expect(clearBtn).toBeInTheDocument();
-      expect(refreshBtn).toBeInTheDocument();
+      expect(selectBtn).toBeDefined();
+      expect(clearBtn).toBeDefined();
+      expect(refreshBtn).toBeDefined();
     });
 
     it('handles event selection', () => {
@@ -232,7 +232,7 @@ describe('EventProvider', () => {
       fireEvent.click(refreshBtn);
 
       // Should handle the click without errors
-      expect(screen.getByTestId('refresh-events-btn')).toBeInTheDocument();
+      expect(screen.getByTestId('refresh-events-btn')).toBeDefined();
     });
   });
 
@@ -242,14 +242,14 @@ describe('EventProvider', () => {
         const { events, isLoading, error, selectedEvent, setSelectedEvent, refreshEvents } = useEvents();
         
         return (
-          <div>
-            <div data-testid="hasEvents">{Array.isArray(events) ? 'true' : 'false'}</div>
-            <div data-testid="hasIsLoading">{typeof isLoading === 'boolean' ? 'true' : 'false'}</div>
-            <div data-testid="hasError">{error !== null ? 'true' : 'false'}</div>
-            <div data-testid="hasSelectedEvent">{selectedEvent !== null ? 'true' : 'false'}</div>
-            <div data-testid="hasSetSelectedEvent">{typeof setSelectedEvent === 'function' ? 'true' : 'false'}</div>
-            <div data-testid="hasRefreshEvents">{typeof refreshEvents === 'function' ? 'true' : 'false'}</div>
-          </div>
+          <section>
+            <p data-testid="hasEvents">{Array.isArray(events) ? 'true' : 'false'}</p>
+            <p data-testid="hasIsLoading">{typeof isLoading === 'boolean' ? 'true' : 'false'}</p>
+            <p data-testid="hasError">{error !== null ? 'true' : 'false'}</p>
+            <p data-testid="hasSelectedEvent">{selectedEvent !== null ? 'true' : 'false'}</p>
+            <p data-testid="hasSetSelectedEvent">{typeof setSelectedEvent === 'function' ? 'true' : 'false'}</p>
+            <p data-testid="hasRefreshEvents">{typeof refreshEvents === 'function' ? 'true' : 'false'}</p>
+          </section>
         );
       };
 
@@ -273,9 +273,9 @@ describe('EventProvider', () => {
       const TestOutsideProvider = () => {
         try {
           useEvents();
-          return <div data-testid="no-error">No error</div>;
+          return <p data-testid="no-error">No error</p>;
         } catch (error) {
-          return <div data-testid="error">{error.message}</div>;
+          return <p data-testid="error">{error.message}</p>;
         }
       };
 
@@ -293,7 +293,7 @@ describe('EventProvider', () => {
         </TestWrapper>
       );
 
-      expect(screen.getByTestId('test-component')).toBeInTheDocument();
+      expect(screen.getByTestId('test-component')).toBeDefined();
       expect(screen.getByTestId('events-count')).toHaveTextContent('0');
     });
   });
@@ -304,10 +304,10 @@ describe('EventProvider', () => {
         const { events, isLoading } = useEvents();
         
         return (
-          <div>
-            <div data-testid="events-list">{events.length}</div>
-            <div data-testid="loading-state">{isLoading ? 'loading' : 'ready'}</div>
-          </div>
+          <section>
+            <p data-testid="events-list">{events.length}</p>
+            <p data-testid="loading-state">{isLoading ? 'loading' : 'ready'}</p>
+          </section>
         );
       };
 
@@ -318,8 +318,8 @@ describe('EventProvider', () => {
       );
 
       // Initial state should be ready (mocks don't trigger real async)
-      expect(screen.getByTestId('events-list')).toBeInTheDocument();
-      expect(screen.getByTestId('loading-state')).toBeInTheDocument();
+      expect(screen.getByTestId('events-list')).toBeDefined();
+      expect(screen.getByTestId('loading-state')).toBeDefined();
     });
 
     it('handles error states gracefully', () => {
@@ -327,9 +327,9 @@ describe('EventProvider', () => {
         const { error } = useEvents();
         
         return (
-          <div data-testid="error-state">
+          <section data-testid="error-state">
             {error ? error.message : 'no-error'}
-          </div>
+          </section>
         );
       };
 
@@ -339,7 +339,7 @@ describe('EventProvider', () => {
         </TestWrapper>
       );
 
-      expect(screen.getByTestId('error-state')).toBeInTheDocument();
+      expect(screen.getByTestId('error-state')).toBeDefined();
     });
 
     it('provides refresh functionality', () => {
@@ -347,11 +347,11 @@ describe('EventProvider', () => {
         const { refreshEvents } = useEvents();
         
         return (
-          <div>
+          <section>
             <button data-testid="refresh-btn" onClick={() => refreshEvents()}>
               Refresh
             </button>
-          </div>
+          </section>
         );
       };
 
@@ -362,7 +362,7 @@ describe('EventProvider', () => {
       );
 
       const refreshBtn = screen.getByTestId('refresh-btn');
-      expect(refreshBtn).toBeInTheDocument();
+      expect(refreshBtn).toBeDefined();
       fireEvent.click(refreshBtn);
       // Should not throw
     });
@@ -374,8 +374,8 @@ describe('EventProvider', () => {
         const { events, selectedEvent, setSelectedEvent } = useEvents();
         
         return (
-          <div>
-            <div data-testid="current-selection">{selectedEvent?.event_name || 'none'}</div>
+          <section>
+            <p data-testid="current-selection">{selectedEvent?.event_name || 'none'}</p>
             <button 
               data-testid="select-btn"
               onClick={() => setSelectedEvent(events[0] || null)}
@@ -388,7 +388,7 @@ describe('EventProvider', () => {
             >
               Clear
             </button>
-          </div>
+          </section>
         );
       };
 
@@ -399,8 +399,8 @@ describe('EventProvider', () => {
       );
 
       expect(screen.getByTestId('current-selection')).toHaveTextContent('none');
-      expect(screen.getByTestId('select-btn')).toBeInTheDocument();
-      expect(screen.getByTestId('clear-btn')).toBeInTheDocument();
+      expect(screen.getByTestId('select-btn')).toBeDefined();
+      expect(screen.getByTestId('clear-btn')).toBeDefined();
     });
 
     it('maintains event context across interactions', () => {
@@ -408,16 +408,16 @@ describe('EventProvider', () => {
         const { events, selectedEvent, setSelectedEvent } = useEvents();
         
         return (
-          <div>
-            <div data-testid="total-events">{events.length}</div>
-            <div data-testid="has-selection">{selectedEvent ? 'yes' : 'no'}</div>
+          <section>
+            <p data-testid="total-events">{events.length}</p>
+            <p data-testid="has-selection">{selectedEvent ? 'yes' : 'no'}</p>
             <button 
               data-testid="toggle-selection"
               onClick={() => setSelectedEvent(events[0] || null)}
             >
               Toggle
             </button>
-          </div>
+          </section>
         );
       };
 
@@ -427,7 +427,7 @@ describe('EventProvider', () => {
         </TestWrapper>
       );
 
-      expect(screen.getByTestId('total-events')).toBeInTheDocument();
+      expect(screen.getByTestId('total-events')).toBeDefined();
       expect(screen.getByTestId('has-selection')).toHaveTextContent('no');
     });
   });
@@ -440,11 +440,11 @@ describe('EventProvider', () => {
         const { events, selectedEvent } = useEvents();
         
         return (
-          <div>
-            <div data-testid="render-count">{renderCount}</div>
-            <div data-testid="events-stable">{events ? 'stable' : 'unstable'}</div>
-            <div data-testid="selection-stable">{selectedEvent !== undefined ? 'stable' : 'unstable'}</div>
-          </div>
+          <section>
+            <p data-testid="render-count">{renderCount}</p>
+            <p data-testid="events-stable">{events ? 'stable' : 'unstable'}</p>
+            <p data-testid="selection-stable">{selectedEvent !== undefined ? 'stable' : 'unstable'}</p>
+          </section>
         );
       };
 
@@ -471,7 +471,7 @@ describe('EventProvider', () => {
         const { setSelectedEvent, refreshEvents } = useEvents();
         
         return (
-          <div>
+          <section>
             <button 
               data-testid="rapid-change-btn"
               onClick={() => {
@@ -481,7 +481,7 @@ describe('EventProvider', () => {
             >
               Rapid Change
             </button>
-          </div>
+          </section>
         );
       };
 
@@ -497,7 +497,7 @@ describe('EventProvider', () => {
       fireEvent.click(btn);
       fireEvent.click(btn);
       
-      expect(btn).toBeInTheDocument();
+      expect(btn).toBeDefined();
     });
   });
 
@@ -507,10 +507,10 @@ describe('EventProvider', () => {
         const { events, selectedEvent } = useEvents();
         
         return (
-          <div>
-            <div data-testid="has-events">{events.length > 0 ? 'yes' : 'no'}</div>
-            <div data-testid="org-context-available">context-available</div>
-          </div>
+          <section>
+            <p data-testid="has-events">{events.length > 0 ? 'yes' : 'no'}</p>
+            <p data-testid="org-context-available">context-available</p>
+          </section>
         );
       };
 
@@ -525,7 +525,7 @@ describe('EventProvider', () => {
         </TestWrapper>
       );
 
-      expect(screen.getByTestId('org-context-available')).toBeInTheDocument();
+      expect(screen.getByTestId('org-context-available')).toBeDefined();
     });
 
     it('handles missing organisation context gracefully', () => {
@@ -533,9 +533,9 @@ describe('EventProvider', () => {
         const { events } = useEvents();
         
         return (
-          <div data-testid="no-org-handled">
+          <section data-testid="no-org-handled">
             Events available: {events.length}
-          </div>
+          </section>
         );
       };
 
@@ -545,7 +545,7 @@ describe('EventProvider', () => {
         </TestWrapper>
       );
 
-      expect(screen.getByTestId('no-org-handled')).toBeInTheDocument();
+      expect(screen.getByTestId('no-org-handled')).toBeDefined();
     });
   });
 });

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>
       );