Fixed broken tests. One was broken by a merge from main, one was broken because the TLS verification changed. Without pinned deps (package lock) it's hard to track down the dep change actually caused this - fix was to use http instead of https example.com test urls.

This commit is contained in:
Bob Dickinson 2026-04-06 11:10:24 -07:00
commit f2a34b67de
2 changed files with 4 additions and 3 deletions

View file

@ -47,7 +47,7 @@ describe("POST /fetch endpoint", () => {
method: "POST", method: "POST",
headers: { "Content-Type": "application/json" }, headers: { "Content-Type": "application/json" },
body: JSON.stringify({ body: JSON.stringify({
url: "https://example.com/", url: "http://example.com/",
init: { method: "GET" }, init: { method: "GET" },
}), }),
}); });
@ -64,7 +64,7 @@ describe("POST /fetch endpoint", () => {
"X-MCP-Proxy-Auth": "Bearer wrong-token", "X-MCP-Proxy-Auth": "Bearer wrong-token",
}, },
body: JSON.stringify({ body: JSON.stringify({
url: "https://example.com/", url: "http://example.com/",
init: { method: "GET" }, init: { method: "GET" },
}), }),
}); });
@ -79,7 +79,7 @@ describe("POST /fetch endpoint", () => {
"X-MCP-Proxy-Auth": `Bearer ${TEST_TOKEN}`, "X-MCP-Proxy-Auth": `Bearer ${TEST_TOKEN}`,
}, },
body: JSON.stringify({ body: JSON.stringify({
url: "https://example.com/", url: "http://example.com/",
init: { method: "GET" }, init: { method: "GET" },
}), }),
}); });

View file

@ -807,6 +807,7 @@ describe("AuthDebugger", () => {
expect(mockDiscoverAuthorizationServerMetadata).toHaveBeenCalledWith( expect(mockDiscoverAuthorizationServerMetadata).toHaveBeenCalledWith(
new URL("https://custom-auth.example.com/mcp/tenant"), new URL("https://custom-auth.example.com/mcp/tenant"),
{ fetchFn: undefined },
); );
}); });