From 4d4fbf7257c91a7b1088df95c7ddb8752255f8b9 Mon Sep 17 00:00:00 2001 From: Mohamed Boudra Date: Thu, 7 May 2026 18:35:44 +0700 Subject: [PATCH] Drop dead test for removed Update installed branch The "shows only the changelog action once the update is installed" case in update-callout-source.test.tsx pinned the rendering path that 2d9c7747 deleted. The callout now early-returns for any status other than available/installing/error, so the assertion that the container contains "Update installed" can never be satisfied. --- .../updates/update-callout-source.test.tsx | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/packages/app/src/desktop/updates/update-callout-source.test.tsx b/packages/app/src/desktop/updates/update-callout-source.test.tsx index 033cb3ead..207b32302 100644 --- a/packages/app/src/desktop/updates/update-callout-source.test.tsx +++ b/packages/app/src/desktop/updates/update-callout-source.test.tsx @@ -184,21 +184,6 @@ describe("UpdateCalloutSource", () => { expect(container?.querySelector('[data-testid="update-callout"]')).toBeNull(); }); - it("shows only the changelog action once the update is installed", async () => { - updaterState.value = { - ...updaterState.value, - status: "installed", - availableUpdate: null, - }; - await renderHarness(root!); - - expect(container?.textContent).toContain("Update installed"); - expect( - container?.querySelector('[data-testid="update-callout-action-0"]')?.textContent, - ).toContain("What's new"); - expect(container?.querySelector('[data-testid="update-callout-action-1"]')).toBeNull(); - }); - it("disables the install action and shows Installing... while installing", async () => { updaterState.value = { ...updaterState.value,