From d85aaa3049a919a9e04bfd05796a444a64af745c Mon Sep 17 00:00:00 2001 From: teknium1 <127238744+teknium1@users.noreply.github.com> Date: Mon, 14 Sep 2026 19:50:34 -0700 Subject: [PATCH] test(desktop): stub the new settings-scope atoms in the ConfigSettings test config-settings.test.tsx mocks @/store/settings-scope with a hand-written subset of exports; SettingsProfileScope now also reads $settingsScopeProfile and $settingsScopeEditsNonDefault, so the partial mock threw at render time in CI (JS & TS checks red). --- apps/desktop/src/app/settings/config-settings.test.tsx | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/apps/desktop/src/app/settings/config-settings.test.tsx b/apps/desktop/src/app/settings/config-settings.test.tsx index fb42ea448a..f9b2ed4832 100644 --- a/apps/desktop/src/app/settings/config-settings.test.tsx +++ b/apps/desktop/src/app/settings/config-settings.test.tsx @@ -26,10 +26,14 @@ vi.mock('../hooks/use-on-profile-switch', () => ({ // The real stores pull in the gateway/profile stack, which needs a live // backend connection. This page only reads the "applies to" scope override -// and the repo-discovery signature, neither of which this test touches. +// and the repo-discovery signature, neither of which this test touches. The +// scope chip it renders also reads the selected profile and the loud-note +// selector, so those are stubbed to the single-profile default shape. vi.mock('@/store/settings-scope', () => ({ $settingsRequestProfile: atom(undefined), - $settingsScopeOverride: atom(null) + $settingsScopeEditsNonDefault: atom(false), + $settingsScopeOverride: atom(null), + $settingsScopeProfile: atom('default') })) vi.mock('@/store/projects', () => ({