From 22e8e8f463cd5b51124ea5db4551b22b620eec00 Mon Sep 17 00:00:00 2001 From: Daniel Han Date: Sun, 19 Jul 2026 16:21:30 +0000 Subject: [PATCH] tests: anchor the inheritance order check on the call, not the definition source.index("_resolve_inherited_extra_args(") matched the function definition, which always precedes the endpoint, so the ordering assertion was vacuously true. Anchoring on "= _resolve_inherited_ extra_args(" pins the first call site inside the load endpoint (line 4505), which is the statement whose position relative to the GGUF branch the test is meant to guard. 32 tests pass. --- studio/backend/tests/test_gguf_load_cache_reuse.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/studio/backend/tests/test_gguf_load_cache_reuse.py b/studio/backend/tests/test_gguf_load_cache_reuse.py index 6e707f6c76..6c39f813b1 100644 --- a/studio/backend/tests/test_gguf_load_cache_reuse.py +++ b/studio/backend/tests/test_gguf_load_cache_reuse.py @@ -730,7 +730,7 @@ class TestLoadHubDownloadExclusion: # Pass-through inheritance runs before the GGUF branch, so a carried # --no-mmproj shapes the hub guard's companion requirement. - assert source.index("_resolve_inherited_extra_args(") < source.index("if config.is_gguf:") + assert source.index("= _resolve_inherited_extra_args(") < source.index("if config.is_gguf:") assert ( gguf_branch.index("enter_context(gguf_load_in_flight") < gguf_branch.index("_hub_download_blocks_gguf_load")