fix: added a test and fixed bad test setup

Author: Leemalin Moodley

src/main/frontend/common/components/status-icon.spec.tsx

@@ -127,6 +127,23 @@ describe("StatusIcon", () => {
         expect(result.current).to.equal(0);
         unmount();
       });
+      it("should continue progress when waitingForInput is true", async () => {
+        const { result, unmount } = renderHook(() => {
+          return useStageProgress({
+            ...mockStage,
+            state: Result.running,
+            startTimeMillis: now - 2_000,
+            previousTotalDurationMillis: 20_000,
+            waitingForInput: true,
+          });
+        });
+        expect(result.current).to.equal(10);
+        await act(() => vi.advanceTimersByTime(1_000));
+        expect(result.current).to.equal(15);
+        await act(() => vi.advanceTimersByTime(1_000));
+        expect(result.current).to.equal(20);
+        unmount();
+      });
     });
     describe("other states", function () {
       for (const state in Result) {

src/main/frontend/setupTests.ts

@@ -8,3 +8,20 @@ const IntersectionObserverMock = vi.fn(() => ({
 }));
 
 vi.stubGlobal("IntersectionObserver", IntersectionObserverMock);
+
+// Ensure a functional localStorage implementation for tests that persist user preferences.
+if (
+  !('localStorage' in window) ||
+  typeof window.localStorage?.getItem !== 'function' ||
+  typeof window.localStorage?.setItem !== 'function'
+) {
+  const store: Record<string, string> = {};
+  const localStoragePolyfill = {
+    getItem: (key: string) => (key in store ? store[key] : null),
+    setItem: (key: string, value: string) => { store[key] = String(value); },
+    removeItem: (key: string) => { delete store[key]; },
+    clear: () => { Object.keys(store).forEach(k => delete store[k]); },
+  };
+  // @ts-expect-error override for test environment
+  window.localStorage = localStoragePolyfill;
+}