fix: remove window.location mocking to fix CI test failures

Remove all window.location mocking from AuthDebugger tests as it was causing "Cannot redefine property" errors in CI. The tests work correctly with JSDOM's default window.location implementation.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: jerome3o-anthropic

client/src/components/__tests__/AuthDebugger.test.tsx

@@ -84,21 +84,6 @@ Object.defineProperty(window, "sessionStorage", {
   value: sessionStorageMock,
 });
 
-// Try to mock window.location, but don't fail if it doesn't work (e.g., in CI)
-try {
-  Object.defineProperty(window, "location", {
-    value: {
-      origin: "http://localhost:3000",
-    },
-  });
-} catch {
-  // Ignore error - tests that depend on this will be skipped
-}
-
-// Skip location-dependent tests in CI where window.location mocking fails
-const isCI = process.env.CI === "true";
-const skipIfCI = isCI ? it.skip : it;
-
 describe("AuthDebugger", () => {
   const defaultAuthState = EMPTY_DEBUGGER_STATE;
 
@@ -113,7 +98,7 @@ describe("AuthDebugger", () => {
     jest.clearAllMocks();
     sessionStorageMock.getItem.mockReturnValue(null);
 
-    // Supress
+    // Suppress console errors in tests to avoid JSDOM navigation noise
     jest.spyOn(console, "error").mockImplementation(() => {});
 
     mockDiscoverOAuthMetadata.mockResolvedValue(mockOAuthMetadata);
@@ -453,65 +438,58 @@ describe("AuthDebugger", () => {
   });
 
   describe("OAuth State Persistence", () => {
-    skipIfCI(
-      "should store auth state to sessionStorage before redirect in Quick OAuth Flow",
-      async () => {
-        const updateAuthState = jest.fn();
-
-        // This test depends on window.location being mocked at the top of the file
-
-        // Setup mocks for OAuth flow
-        mockStartAuthorization.mockResolvedValue({
-          authorizationUrl: new URL(
-            "https://oauth.example.com/authorize?client_id=test_client_id&redirect_uri=http%3A%2F%2Flocalhost%3A3000%2Foauth%2Fcallback%2Fdebug",
-          ),
-          codeVerifier: "test_verifier",
-        });
+    it("should store auth state to sessionStorage before redirect in Quick OAuth Flow", async () => {
+      const updateAuthState = jest.fn();
 
-        await act(async () => {
-          renderAuthDebugger({
-            updateAuthState,
-            authState: { ...defaultAuthState },
-          });
-        });
+      // Setup mocks for OAuth flow
+      mockStartAuthorization.mockResolvedValue({
+        authorizationUrl: new URL(
+          "https://oauth.example.com/authorize?client_id=test_client_id&redirect_uri=http%3A%2F%2Flocalhost%3A3000%2Foauth%2Fcallback%2Fdebug",
+        ),
+        codeVerifier: "test_verifier",
+      });
 
-        // Click Quick OAuth Flow
-        await act(async () => {
-          fireEvent.click(screen.getByText("Quick OAuth Flow"));
+      await act(async () => {
+        renderAuthDebugger({
+          updateAuthState,
+          authState: { ...defaultAuthState },
         });
+      });
 
-        // Wait for the flow to reach the authorization step
-        await waitFor(() => {
-          expect(sessionStorage.setItem).toHaveBeenCalledWith(
-            SESSION_KEYS.AUTH_DEBUGGER_STATE,
-            expect.stringContaining('"oauthStep":"authorization_code"'),
-          );
-        });
+      // Click Quick OAuth Flow
+      await act(async () => {
+        fireEvent.click(screen.getByText("Quick OAuth Flow"));
+      });
 
-        // Verify the stored state includes all the accumulated data
-        const storedStateCall = (
-          sessionStorage.setItem as jest.Mock
-        ).mock.calls.find(
-          (call) => call[0] === SESSION_KEYS.AUTH_DEBUGGER_STATE,
+      // Wait for the flow to reach the authorization step
+      await waitFor(() => {
+        expect(sessionStorage.setItem).toHaveBeenCalledWith(
+          SESSION_KEYS.AUTH_DEBUGGER_STATE,
+          expect.stringContaining('"oauthStep":"authorization_code"'),
         );
+      });
 
-        expect(storedStateCall).toBeDefined();
-        const storedState = JSON.parse(storedStateCall![1] as string);
+      // Verify the stored state includes all the accumulated data
+      const storedStateCall = (
+        sessionStorage.setItem as jest.Mock
+      ).mock.calls.find((call) => call[0] === SESSION_KEYS.AUTH_DEBUGGER_STATE);
 
-        expect(storedState).toMatchObject({
-          oauthStep: "authorization_code",
-          authorizationUrl: expect.stringMatching(
-            /^https:\/\/oauth\.example\.com\/authorize/,
-          ),
-          oauthMetadata: expect.objectContaining({
-            token_endpoint: "https://oauth.example.com/token",
-          }),
-          oauthClientInfo: expect.objectContaining({
-            client_id: "test_client_id",
-          }),
-        });
-      },
-    );
+      expect(storedStateCall).toBeDefined();
+      const storedState = JSON.parse(storedStateCall![1] as string);
+
+      expect(storedState).toMatchObject({
+        oauthStep: "authorization_code",
+        authorizationUrl: expect.stringMatching(
+          /^https:\/\/oauth\.example\.com\/authorize/,
+        ),
+        oauthMetadata: expect.objectContaining({
+          token_endpoint: "https://oauth.example.com/token",
+        }),
+        oauthClientInfo: expect.objectContaining({
+          client_id: "test_client_id",
+        }),
+      });
+    });
   });
 
   describe("OAuth Protected Resource Metadata", () => {