From 23f5fc935d0eae6d9e313856a3ef131745e56eec Mon Sep 17 00:00:00 2001 From: Kit Langton Date: Tue, 20 Oct 2026 20:55:16 -0700 Subject: [PATCH] test(app): drop the file tree state from the browser pane fixture (#57563) --- packages/app/component-tests/browser-pane-restore.spec.ts | 5 ----- packages/gui-extensions/src/browser/panel.fixture.tsx | 5 ----- 2 files changed, 10 deletions(-) diff --git a/packages/app/component-tests/browser-pane-restore.spec.ts b/packages/app/component-tests/browser-pane-restore.spec.ts index d4aa5f675df..03ab801d928 100644 --- a/packages/app/component-tests/browser-pane-restore.spec.ts +++ b/packages/app/component-tests/browser-pane-restore.spec.ts @@ -476,23 +476,18 @@ story("selects side tabs with โŒ˜1โ€“9 and โ‡งโŒ˜[ ] inside the side panel and s story("keeps a restored browser tab selected and undrawn until the desktop's first inventory", async ({ page }) => { const root = page.getByTestId("browser-region-fixture") const tabs = root.getByRole("tab") - const tree = root.getByTestId("tree") await expect(root.getByText("Registrations: 1", { exact: true })).toBeVisible() await expect(tabs).toHaveText(["alpha.ts"]) - await expect(tree).toHaveText('{"tab":"changes"}') // Beta was left on its browser tab, which the desktop has not reported yet. await root.getByRole("button", { name: "Beta", exact: true }).click() await expect(root.getByText("Registrations: 2", { exact: true })).toBeVisible() await expect(root.getByTestId("selected")).toHaveText(/^browser:tab_/) await expect(tabs).toHaveText(["beta.ts"]) - // No fallback tab was selected, so the file tab's selection never switched the tree to All files. - await expect(tree).toHaveText('{"tab":"changes"}') await root.getByRole("button", { name: "First inventory", exact: true }).click() await expect(tabs).toHaveText(["beta.ts", "Preview"]) await expect(root.getByRole("tab", { name: "Preview", exact: true })).toHaveAttribute("aria-selected", "true") - await expect(tree).toHaveText('{"tab":"changes"}') }) story( diff --git a/packages/gui-extensions/src/browser/panel.fixture.tsx b/packages/gui-extensions/src/browser/panel.fixture.tsx index 8ad564aff7e..195cc65fa50 100644 --- a/packages/gui-extensions/src/browser/panel.fixture.tsx +++ b/packages/gui-extensions/src/browser/panel.fixture.tsx @@ -688,8 +688,6 @@ export function mountBrowserRegion(input: RegionHost) { inspects: [], }) - // The file tree's state, Changes or All files, as the file extension stores it. - const [tree, setTree] = createSignal() const [shortcut, setShortcut] = createSignal>(() => "") const [sideRegion, setSideRegion] = createSignal>() const strip = (session: string) => store.strips[session] ?? { all: [] } @@ -803,8 +801,6 @@ export function mountBrowserRegion(input: RegionHost) { const keep = (key: string, initial: T) => { const [value, set] = createStore(structuredClone(initial)) - if (key === "file:tree") setTree(() => value) - return { memory: [value, (mutation: (draft: T) => void) => set(produce(mutation))] as const, store: { @@ -1106,7 +1102,6 @@ export function mountBrowserRegion(input: RegionHost) { />

Registrations: {store.registrations.length}

Beta restores: {latest()?.restore ?? 0}

-

{JSON.stringify(tree())}

{store.closed ? "closed" : "open"}

{store.reviews}

{store.opened ? "open" : "closed"}