From 1fd190fbb76c9bad45b1d710e646e3c94e1bb554 Mon Sep 17 00:00:00 2001
From: Jerome <jerome@anthropic.com>
Date: Fri, 13 Jun 2025 17:38:01 +0100
Subject: [PATCH] Linting, .git-blame-ignore-revs file

---
 .git-blame-ignore-revs                        |  0
 .../__tests__/AuthDebugger.test.tsx           | 97 ++++++++++---------
 2 files changed, 51 insertions(+), 46 deletions(-)
 create mode 100644 .git-blame-ignore-revs

diff --git a/.git-blame-ignore-revs b/.git-blame-ignore-revs
new file mode 100644
index 00000000..e69de29b
diff --git a/client/src/components/__tests__/AuthDebugger.test.tsx b/client/src/components/__tests__/AuthDebugger.test.tsx
index 9cd36435..e978c558 100644
--- a/client/src/components/__tests__/AuthDebugger.test.tsx
+++ b/client/src/components/__tests__/AuthDebugger.test.tsx
@@ -453,60 +453,65 @@ describe("AuthDebugger", () => {
   });
 
   describe("OAuth State Persistence", () => {
-    skipIfCI("should store auth state to sessionStorage before redirect in Quick OAuth Flow", async () => {
-      const updateAuthState = jest.fn();
+    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
+        // 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",
-      });
-
-      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"));
+        });
+
+        // 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"'),
+          );
+        });
+
+        // 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,
         );
-      });
 
-      // 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(storedStateCall).toBeDefined();
+        const storedState = JSON.parse(storedStateCall![1] as string);
 
-      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",
-        }),
-      });
-    });
+        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", () => {