From 91c3f8da8fed6276f5e99c7a2412cdb5b41426fe Mon Sep 17 00:00:00 2001 From: Matt Toohey Date: Wed, 2 Sep 2026 16:17:49 +1000 Subject: [PATCH] test(desktop): seed the override test with a live preview feature MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The merge with main crossed two changes: main added the thread-scoped ACP session experiment and seeded its store test with `projects: false`, while this branch promoted Projects out of preview-features. `getOverrides` filters IDs that are no longer in the manifest, so `setOverride` rewrote storage without the stale `projects` key and the assertion failed. Seed with `pulse` instead — still a preview feature, so the test keeps binding what it was written to bind: setting one override preserves the others rather than clobbering them. Signed-off-by: Matt Toohey --- desktop/src/shared/features/store.test.mjs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/desktop/src/shared/features/store.test.mjs b/desktop/src/shared/features/store.test.mjs index 540cdb70f5..072629c89a 100644 --- a/desktop/src/shared/features/store.test.mjs +++ b/desktop/src/shared/features/store.test.mjs @@ -47,7 +47,7 @@ test("setOverride persists stable IDs but unknown IDs stay filtered on read", () }); test("thread-scoped ACP session preference persists without changing existing experiments", () => { - const { values } = installStorage({ workflows: true, projects: false }); + const { values } = installStorage({ workflows: true, pulse: false }); setOverride("threadScopedAcpSessions", true); @@ -55,7 +55,7 @@ test("thread-scoped ACP session preference persists without changing existing ex values.get(OVERRIDES_KEY), JSON.stringify({ workflows: true, - projects: false, + pulse: false, threadScopedAcpSessions: true, }), );