From 1cb77542e1af3c12b389cb0feb596ba5bc171a0a Mon Sep 17 00:00:00 2001 From: Smartsheet-JB-Brown Date: Thu, 17 Apr 2025 17:25:28 -0700 Subject: [PATCH] round 4 --- .../package-manager/__tests__/PackageManagerView.test.tsx | 8 ++++++-- .../components/__tests__/PackageManagerItemCard.test.tsx | 2 +- 2 files changed, 7 insertions(+), 3 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 053d949229..bb0a014579 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,7 @@ describe("PackageManagerView", () => { }) // Should show loading state - expect(screen.getByText("Loading items...")).toBeInTheDocument() + expect(screen.getByText("Refreshing...")).toBeInTheDocument() // Simulate receiving items await act(async () => { @@ -175,7 +175,11 @@ describe("PackageManagerView", () => { }) // Verify initial items are shown - expect(screen.getByText("3 items found")).toBeInTheDocument() + expect( + screen.getByText((content, element) => { + return element?.textContent === "3 items found" + }), + ).toBeInTheDocument() expect(screen.getByText("MCP Server 1")).toBeInTheDocument() expect(screen.getByText("Mode 1")).toBeInTheDocument() expect(screen.getByText("MCP Server 2")).toBeInTheDocument() diff --git a/webview-ui/src/components/package-manager/components/__tests__/PackageManagerItemCard.test.tsx b/webview-ui/src/components/package-manager/components/__tests__/PackageManagerItemCard.test.tsx index 375db3980c..24dd6fd690 100644 --- a/webview-ui/src/components/package-manager/components/__tests__/PackageManagerItemCard.test.tsx +++ b/webview-ui/src/components/package-manager/components/__tests__/PackageManagerItemCard.test.tsx @@ -163,7 +163,7 @@ describe("PackageManagerItemCard", () => { name: "View", }) expect(button.querySelector(".codicon-link-external")).toBeInTheDocument() - expect(button).toHaveTextContent(/Source/i) + expect(button).toHaveTextContent("View") }) })