From d0e7b1914fb2d2b38368bf337a6647094c57093e Mon Sep 17 00:00:00 2001 From: "pre-commit-ci[bot]" <66853113+pre-commit-ci[bot]@users.noreply.github.com> Date: Tue, 28 Jul 2026 20:24:58 +0000 Subject: [PATCH] [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --- .../backend/tests/test_openai_auto_switch.py | 16 ++++++++------ tests/studio/test_model_picker_contracts.py | 22 +++++-------------- 2 files changed, 15 insertions(+), 23 deletions(-) diff --git a/studio/backend/tests/test_openai_auto_switch.py b/studio/backend/tests/test_openai_auto_switch.py index 9e913de11f..42ea6ee957 100644 --- a/studio/backend/tests/test_openai_auto_switch.py +++ b/studio/backend/tests/test_openai_auto_switch.py @@ -701,7 +701,13 @@ def _mock_override_store(monkeypatch): store = {} - def _merge_entry(key, entry_key, entry_value, *, only_if_absent = False): + def _merge_entry( + key, + entry_key, + entry_value, + *, + only_if_absent = False, + ): current = dict(store.get(key) or {}) if only_if_absent: # Create only: an entry already there wins and nothing is deleted. @@ -5196,9 +5202,7 @@ def test_only_if_absent_put_never_replaces_a_newer_server_entry(monkeypatch): _mock_override_store(monkeypatch) # The other tab's save lands first. - newer = settings_route.ModelOverridePayload( - model_id = "unsloth/B-GGUF", max_seq_length = 8192 - ) + newer = settings_route.ModelOverridePayload(model_id = "unsloth/B-GGUF", max_seq_length = 8192) settings_route.update_openai_auto_switch_override(newer, "tester") # The backfill's write, carrying this browser's older localStorage value. @@ -5261,9 +5265,7 @@ def test_only_if_absent_does_not_break_the_empty_payload_removal(monkeypatch): _mock_override_store(monkeypatch) - stored = settings_route.ModelOverridePayload( - model_id = "unsloth/B-GGUF", max_seq_length = 4096 - ) + stored = settings_route.ModelOverridePayload(model_id = "unsloth/B-GGUF", max_seq_length = 4096) settings_route.update_openai_auto_switch_override(stored, "tester") empty = settings_route.ModelOverridePayload(model_id = "unsloth/B-GGUF") resp = settings_route.update_openai_auto_switch_override(empty, "tester") diff --git a/tests/studio/test_model_picker_contracts.py b/tests/studio/test_model_picker_contracts.py index d80ff4af37..ca8c8bfc9a 100644 --- a/tests/studio/test_model_picker_contracts.py +++ b/tests/studio/test_model_picker_contracts.py @@ -942,9 +942,7 @@ def test_the_chat_picker_marks_ollama_targets_unloadable_by_the_api(): advertised a load the API can never make.""" picker = " ".join(_read("features/model-picker/components/model-selector.tsx").split()) assert "apiLoadable: isGguf && !isOllamaLinkPath(id)," in picker - sidebar = " ".join( - _read("features/model-picker/components/sidebar-model-config.tsx").split() - ) + sidebar = " ".join(_read("features/model-picker/components/sidebar-model-config.tsx").split()) assert "apiLoadable: isGguf && !isOllamaLinkPath(modelId)," in sidebar # The same classification gates the one-time backfill, or a config saved before # the upgrade still reaches the server on the next start. @@ -974,9 +972,7 @@ def test_the_backfill_writes_are_creates_not_replacements(): # Ordinary saves must stay unconditional, or a settings edit would never land. assert "syncModelOverride" in api and "only_if_absent: true" in api - route = (WORKDIR / "studio" / "backend" / "routes" / "settings.py").read_text( - encoding = "utf-8" - ) + route = (WORKDIR / "studio" / "backend" / "routes" / "settings.py").read_text(encoding = "utf-8") assert "only_if_absent: bool = False" in route, "the rule this mirrors" assert "only_if_absent = payload.only_if_absent," in route # A write mode must not leak into the saved fields, or "only model_id means @@ -1000,9 +996,9 @@ def test_the_hub_settings_page_matches_a_resident_path_loaded_model(): identity = _read("features/hub/lib/model-identity.ts") assert "export function publicModelId(" in identity assert "models--" in identity and "snapshots" in identity - backend = ( - WORKDIR / "studio" / "backend" / "core" / "inference" / "model_ids.py" - ).read_text(encoding = "utf-8") + backend = (WORKDIR / "studio" / "backend" / "core" / "inference" / "model_ids.py").read_text( + encoding = "utf-8" + ) assert "def public_model_id(" in backend, "the rule this mirrors" @@ -1018,13 +1014,7 @@ def test_a_standalone_gguf_has_one_settings_key(): assert 'row.kind === "local" && row.path.toLowerCase().endsWith(".gguf")' in helper common = ( - WORKDIR - / "studio" - / "backend" - / "hub" - / "services" - / "models" - / "common.py" + WORKDIR / "studio" / "backend" / "hub" / "services" / "models" / "common.py" ).read_text(encoding = "utf-8") # The rule this mirrors: a variant is derived only for a single scanned file. assert "extract_quant_label(gguf_files[0].name)" in common