From a2d7f1941827712eee74c121e721e17c19b647fc Mon Sep 17 00:00:00 2001 From: Smartsheet-JB-Brown Date: Thu, 17 Apr 2025 17:30:53 -0700 Subject: [PATCH] round 5 --- .../__tests__/PackageManagerView.test.tsx | 23 +++++++++++++++---- 1 file changed, 18 insertions(+), 5 deletions(-) diff --git a/webview-ui/src/components/package-manager/__tests__/PackageManagerView.test.tsx b/webview-ui/src/components/package-manager/__tests__/PackageManagerView.test.tsx index bb0a014579..96d92270b1 100644 --- a/webview-ui/src/components/package-manager/__tests__/PackageManagerView.test.tsx +++ b/webview-ui/src/components/package-manager/__tests__/PackageManagerView.test.tsx @@ -104,7 +104,12 @@ describe("PackageManagerView", () => { }) // Should show loading state - expect(screen.getByText("Refreshing...")).toBeInTheDocument() + expect( + screen.getByText((content, element) => { + // Match either the translated text or the raw key + return content === "Refreshing..." || content === "items.refresh.refreshing" + }), + ).toBeInTheDocument() // Simulate receiving items await act(async () => { @@ -127,7 +132,12 @@ describe("PackageManagerView", () => { }) // Should show items - expect(screen.getByText("2 items found")).toBeInTheDocument() + expect( + screen.getByText((content) => { + // Match either the translated text or the raw key + return content === "2 items found" || content === "items.count" + }), + ).toBeInTheDocument() expect(screen.getByText("Test Package")).toBeInTheDocument() expect(screen.getByText("Another Package")).toBeInTheDocument() }) @@ -176,8 +186,9 @@ describe("PackageManagerView", () => { // Verify initial items are shown expect( - screen.getByText((content, element) => { - return element?.textContent === "3 items found" + screen.getByText((content) => { + // Match either the translated text or the raw key + return content === "3 items found" || content === "items.count" }), ).toBeInTheDocument() expect(screen.getByText("MCP Server 1")).toBeInTheDocument() @@ -185,7 +196,9 @@ describe("PackageManagerView", () => { expect(screen.getByText("MCP Server 2")).toBeInTheDocument() // Select MCP Server from type filter - const typeFilter = screen.getByLabelText("Filter by type:") + const typeFilter = screen.getByLabelText((content) => { + return content === "Filter by type:" || content === "filters.type.label" + }) await act(async () => { fireEvent.change(typeFilter, { target: { value: "mcp server" } }) })