From 1f8bb65e275e92b2b887b86a7f5ebcf74931c48f Mon Sep 17 00:00:00 2001 From: Tin Chi Lo Date: Mon, 15 Jun 2026 16:53:39 -0700 Subject: [PATCH] test(mcp): drop discovery default regression tests --- .../mcp_server/test_mcp_discovery.py | 30 ------------------- .../mcp_tools/create_mcp_server.test.tsx | 24 --------------- 2 files changed, 54 deletions(-) diff --git a/tests/test_litellm/proxy/_experimental/mcp_server/test_mcp_discovery.py b/tests/test_litellm/proxy/_experimental/mcp_server/test_mcp_discovery.py index 3d3f9cfd781..9a741a3f861 100644 --- a/tests/test_litellm/proxy/_experimental/mcp_server/test_mcp_discovery.py +++ b/tests/test_litellm/proxy/_experimental/mcp_server/test_mcp_discovery.py @@ -92,38 +92,8 @@ class TestMCPRegistryFile: linear = next(s for s in data["servers"] if s["name"] == "linear") assert linear["transport"] == "http" assert linear["url"] == "https://mcp.linear.app/mcp" - assert linear["auth_type"] == "oauth2" assert "/sse" not in linear["url"] - def test_slack_defaults_to_streamable_http_oauth(self, registry_path): - """Slack's MCP server should default to streamable HTTP with the hosted /mcp endpoint and OAuth.""" - with open(registry_path, "r") as f: - data = json.load(f) - slack = next(s for s in data["servers"] if s["name"] == "slack") - assert slack["transport"] == "http" - assert slack["url"] == "https://mcp.slack.com/mcp" - assert slack["auth_type"] == "oauth2" - assert "command" not in slack - assert "args" not in slack - - def test_notion_defaults_to_streamable_http_oauth(self, registry_path): - """Notion's MCP server should default to streamable HTTP at /mcp with OAuth, not SSE at /sse.""" - with open(registry_path, "r") as f: - data = json.load(f) - notion = next(s for s in data["servers"] if s["name"] == "notion") - assert notion["transport"] == "http" - assert notion["url"] == "https://mcp.notion.com/mcp" - assert notion["auth_type"] == "oauth2" - assert "/sse" not in notion["url"] - - def test_atlassian_uses_authv2_endpoint(self, registry_path): - """Atlassian's MCP server should default to the streamable HTTP authv2 endpoint.""" - with open(registry_path, "r") as f: - data = json.load(f) - atlassian = next(s for s in data["servers"] if s["name"] == "atlassian") - assert atlassian["transport"] == "http" - assert atlassian["url"] == "https://mcp.atlassian.com/v1/mcp/authv2" - def test_well_known_servers_present(self, registry_path): """Ensure key well-known MCPs are in the registry.""" with open(registry_path, "r") as f: diff --git a/ui/litellm-dashboard/src/components/mcp_tools/create_mcp_server.test.tsx b/ui/litellm-dashboard/src/components/mcp_tools/create_mcp_server.test.tsx index 72f9b1a6b2a..e70548d6a96 100644 --- a/ui/litellm-dashboard/src/components/mcp_tools/create_mcp_server.test.tsx +++ b/ui/litellm-dashboard/src/components/mcp_tools/create_mcp_server.test.tsx @@ -754,30 +754,6 @@ describe("CreateMCPServer", () => { expect(nameInput).toHaveValue("github_mcp"); }); }); - - it("prefills streamable HTTP, URL, and OAuth from a discovery preset like Slack", async () => { - const prefillData = { - name: "slack", - title: "Slack", - description: "Channel management, messaging, and Slack workspace integration", - category: "Communication", - transport: "http", - url: "https://mcp.slack.com/mcp", - auth_type: "oauth2", - }; - - render(); - - await waitFor(() => { - const urlInput = screen.getByPlaceholderText("https://your-mcp-server.com"); - expect(urlInput).toHaveValue("https://mcp.slack.com/mcp"); - }); - - // Setting auth_type to oauth2 should render the OAuth form fields - await waitFor(() => { - expect(screen.getByText("OAuth Flow Type")).toBeInTheDocument(); - }); - }); }); describe("with back to discovery button", () => {