From 4c1a6309c2d677036fbb86e5acb670775dc43f2c Mon Sep 17 00:00:00 2001
From: Joshua Valluru <326636767+joshua-berri@users.noreply.github.com>
Date: Mon, 21 Sep 2026 14:31:28 -0700
Subject: [PATCH] test(mcp): type server fixtures and role queries

---
 .../mcp-servers/_components/mcp_servers.test.tsx          | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/ui/litellm-dashboard/src/app/(dashboard)/mcp-servers/_components/mcp_servers.test.tsx b/ui/litellm-dashboard/src/app/(dashboard)/mcp-servers/_components/mcp_servers.test.tsx
index 7198cb3cce9..1217d878489 100644
--- a/ui/litellm-dashboard/src/app/(dashboard)/mcp-servers/_components/mcp_servers.test.tsx
+++ b/ui/litellm-dashboard/src/app/(dashboard)/mcp-servers/_components/mcp_servers.test.tsx
@@ -184,6 +184,8 @@ describe("MCPServers", () => {
 
   it.each(["cancel", "success", "failure", "unnamed"])("preserves delete confirmation on %s", async (outcome) => {
     const server: MCPServer = {
+      created_at: "",
+      updated_at: "",
       server_id: "delete-server",
       server_name: outcome === "unnamed" ? null : "Delete fixture",
       alias: "delete-alias",
@@ -220,7 +222,7 @@ describe("MCPServers", () => {
       await userEvent.click(within(dialog).getByRole("button", { name: "Cancel" }));
       expect(networking.deleteMCPServer).not.toHaveBeenCalled();
     } else {
-      await userEvent.click(within(dialog).getByRole("button", { name: "Delete", exact: true }));
+      await userEvent.click(within(dialog).getByRole("button", { name: "Delete" }));
       expect(within(dialog).getByRole("button", { name: "Deleting..." })).toBeDisabled();
       expect(within(dialog).getByRole("button", { name: "Cancel" })).toBeDisabled();
       expect(networking.deleteMCPServer).toHaveBeenCalledWith("123", "delete-server");
@@ -262,7 +264,7 @@ describe("MCPServers", () => {
     );
     await screen.findByText("String group");
     await userEvent.click(screen.getByRole("combobox", { name: "Access Group" }));
-    await userEvent.click(await screen.findByRole("option", { name: "shared", exact: true }));
+    await userEvent.click(await screen.findByRole("option", { name: "shared" }));
     expect(screen.getByText("String group")).toBeVisible();
     expect(screen.getByText("Legacy group")).toBeVisible();
     expect(screen.queryByText("Other group")).not.toBeInTheDocument();
@@ -270,6 +272,8 @@ describe("MCPServers", () => {
 
   it.each(["server_name", "alias", "url", "server_id"] as const)("searches by %s case-insensitively", async (field) => {
     const server: MCPServer = {
+      created_at: "",
+      updated_at: "",
       server_id: "search-server",
       server_name: "Search fixture",
       created_by: "user",