From 951df07c0fb4662e96cb92e1ecbe3e8486058e06 Mon Sep 17 00:00:00 2001
From: stranger2904 <stranger2904@users.noreply.github.com>
Date: Wed, 14 Jan 2026 15:10:45 -0500
Subject: [PATCH] fix: correct test syntax for headers verification

Fix syntax error where expect().rejects.toThrow() was not properly closed
before the headers assertion.
---
 src/features/skill-mcp-manager/manager.test.ts | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/features/skill-mcp-manager/manager.test.ts b/src/features/skill-mcp-manager/manager.test.ts
index ff9816c5f..b77e74efe 100644
--- a/src/features/skill-mcp-manager/manager.test.ts
+++ b/src/features/skill-mcp-manager/manager.test.ts
@@ -475,15 +475,13 @@ describe("SkillMcpManager", () => {
       // Headers are passed through to the transport
       await expect(manager.getOrCreateClient(info, config)).rejects.toThrow(
         /Failed to connect/
+      )
 
       // Verify headers were forwarded to transport
       expect(lastTransportInstance.options?.requestInit?.headers).toEqual({
         Authorization: "Bearer test-token",
         "X-Custom-Header": "custom-value",
       })
-
-
-      )
     })
 
     it("works without headers (optional)", async () => {