From 94e943144ea5f237fc158d85f00c67ef9fe72c08 Mon Sep 17 00:00:00 2001 From: Yuneng Jiang Date: Fri, 14 Aug 2026 16:40:16 -0700 Subject: [PATCH] refactor(ui): drop the explanatory comment from the badge variant test --- .../src/components/permissions/MCPServerPermissions.test.tsx | 5 ----- 1 file changed, 5 deletions(-) diff --git a/ui/litellm-dashboard/src/components/permissions/MCPServerPermissions.test.tsx b/ui/litellm-dashboard/src/components/permissions/MCPServerPermissions.test.tsx index 1d8a0a6dd84..c2df945f367 100644 --- a/ui/litellm-dashboard/src/components/permissions/MCPServerPermissions.test.tsx +++ b/ui/litellm-dashboard/src/components/permissions/MCPServerPermissions.test.tsx @@ -374,11 +374,6 @@ describe("MCPServerPermissions", () => { }); it("should use the neutral badge variant unless MCP access is blocked", async () => { - /** - * The header badge sits next to the Vector Stores and Agents badges, which both render - * variant="secondary". "default" renders solid bg-primary (black), so it only belongs on - * the blocked state, which uses "destructive". - */ vi.mocked(networking.fetchMCPServers).mockResolvedValue([]); const { rerender } = render(