From ac36c15d38470c94d51f266ba274cb66b62822d0 Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Fri, 9 Jan 2026 00:03:43 +0000 Subject: [PATCH] Fix test failures by updating mock timestamp Updated tests/app.test.ts to correctly update the mock timestamp when advancing time in tests. This ensures that the application logic, which relies on the difference between the current time and the position timestamp, correctly processes the position updates during testing. --- package-lock.json | 4 ++-- package.json | 2 +- tests/app.test.ts | 6 ++++++ 3 files changed, 9 insertions(+), 3 deletions(-) diff --git a/package-lock.json b/package-lock.json index 82ad439..5ebd079 100644 --- a/package-lock.json +++ b/package-lock.json @@ -1,12 +1,12 @@ { "name": "speedometer", - "version": "0.0.81", + "version": "0.0.82", "lockfileVersion": 3, "requires": true, "packages": { "": { "name": "speedometer", - "version": "0.0.81", + "version": "0.0.82", "hasInstallScript": true, "license": "MIT", "dependencies": { diff --git a/package.json b/package.json index 4434a83..04642d3 100644 --- a/package.json +++ b/package.json @@ -1,6 +1,6 @@ { "name": "speedometer", - "version": "0.0.81", + "version": "0.0.82", "description": "Minimal PWA speedometer that displays GPS speed. Includes TypeScript script to render PNG icons from SVG using sharp.", "license": "MIT", "private": true, diff --git a/tests/app.test.ts b/tests/app.test.ts index 022141b..dc8b641 100644 --- a/tests/app.test.ts +++ b/tests/app.test.ts @@ -172,6 +172,9 @@ describe("Speedometer App", () => { // Advance time by 1s (GPS_WARMUP_MS) vi.advanceTimersByTime(1000); + // Update timestamp to reflect elapsed time + mockPosition.timestamp = Date.now(); + // Send it again to trigger the update watchSuccessCallback(mockPosition as unknown as GeolocationPosition); } else { @@ -218,11 +221,13 @@ describe("Speedometer App", () => { // Reading 2 (T=0.5s) -> Ignored vi.advanceTimersByTime(500); + mockPosition.timestamp = Date.now(); watchSuccessCallback(mockPosition as unknown as GeolocationPosition); expect(speedEl.textContent).toBe(PLACEHOLDER); // Reading 3 (T=1.0s) -> Accepted (>= GPS_WARMUP_MS) vi.advanceTimersByTime(500); + mockPosition.timestamp = Date.now(); watchSuccessCallback(mockPosition as unknown as GeolocationPosition); // 10 m/s * 2.23694 = 22.3694 -> 22 expect(speedEl.textContent).toBe("22"); @@ -365,6 +370,7 @@ describe("Speedometer App", () => { // Advance past warmup vi.advanceTimersByTime(1000); + validPosition.timestamp = Date.now(); // Second one accepted watchSuccessCallback(validPosition as unknown as GeolocationPosition);