From 4aa19eee8640410bb8158feee25a669ceaa365e2 Mon Sep 17 00:00:00 2001 From: Developer Date: Fri, 10 Apr 2026 12:01:11 -0700 Subject: [PATCH] Fix test: align remote.mode in no-reload settings test The default remote.mode changed from 'local' to 'byok', causing the apply_settings test to detect a mode mismatch and trigger an unexpected engine reload. Pin remote.mode to 'local' in the test to match the controller's assumed current mode. Co-Authored-By: Claude Opus 4.6 (1M context) --- backend/tests/test_app_controller.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/backend/tests/test_app_controller.py b/backend/tests/test_app_controller.py index d02b23a..ed206cd 100644 --- a/backend/tests/test_app_controller.py +++ b/backend/tests/test_app_controller.py @@ -125,6 +125,8 @@ def test_apply_settings_no_reload_when_same(controller): # Ensure config returns the same values controller.config.set("transcription.model", "base.en") controller.config.set("transcription.device", "auto") + # Remote mode must also match (no engine means current mode is 'local') + controller.config.set("remote.mode", "local") controller.reload_engine = MagicMock(return_value=(True, "reloaded"))