diff --git a/workout-tracker/e2e/timer-notification.spec.ts b/workout-tracker/e2e/timer-notification.spec.ts index d3a3c63..ee784c5 100644 --- a/workout-tracker/e2e/timer-notification.spec.ts +++ b/workout-tracker/e2e/timer-notification.spec.ts @@ -101,7 +101,7 @@ test.describe('Rest Timer Notification Reliability', () => { const vibrateCount = await page.evaluate(() => (window as any).__vibrateCount); expect(vibrateCount).toBe(1); - // Timer should be hidden - await expect(page.locator('#rest-timer')).toBeHidden(); + // Timer should show expired state (active countdown stopped) + await expect(page.locator('[data-testid="timer-expired"]')).toBeVisible(); }); }); diff --git a/workout-tracker/src/ui/workout.ts b/workout-tracker/src/ui/workout.ts index 8a82430..53ec5ba 100644 --- a/workout-tracker/src/ui/workout.ts +++ b/workout-tracker/src/ui/workout.ts @@ -374,7 +374,6 @@ export async function renderWorkout(container: HTMLElement): Promise { el.dataset.testid = 'timer-expired'; const timerValue = document.getElementById('timer-value'); if (timerValue) timerValue.textContent = "Time's Up!"; - // Hide skip button during expired state const skipBtn = document.getElementById('skip-timer-btn'); if (skipBtn) skipBtn.classList.add('hidden'); @@ -386,7 +385,6 @@ export async function renderWorkout(container: HTMLElement): Promise { }; el.addEventListener('click', dismiss); - // Auto-dismiss after 10 seconds setTimeout(dismiss, 10000); } @@ -586,7 +584,7 @@ export async function renderWorkout(container: HTMLElement): Promise { await putTimerState(null); cancelBackgroundTimerNotification(); fireTimerNotification(); - showTimerExpired(timerEl); + timerEl.classList.add('hidden'); } }