test: drop a laziness assertion the existing suite already makes

test_provider_lookup_skipped_for_models_below_gpt_5_6 already pins that
_resolve_provider is not called for an unflagged model, and it covers the new
flag lookup unchanged. The duplicate patched a litellm internal for no added
coverage, which the test-quality gate counts against TQ008.
This commit is contained in:
Priyansh Nandwana 2026-08-29 11:02:27 +05:30
parent 45601ff1d2
commit 79b24c70c9

View file

@ -3117,12 +3117,6 @@ class TestHostedOpenAIDialectFlag:
assert AnthropicCacheControlHook._targets_openai_prompt_cache_breakpoint("gpt-5.6", "azure") is False
assert AnthropicCacheControlHook._targets_openai_prompt_cache_breakpoint("azure/gpt-5.6", None) is False
def test_provider_is_not_resolved_when_no_flag_is_present(self):
"""The cheap model-map lookup must short-circuit before the provider lookup."""
with patch.object(AnthropicCacheControlHook, "_resolve_provider") as resolve:
assert AnthropicCacheControlHook._targets_openai_prompt_cache_breakpoint("gpt-4.1", None) is False
resolve.assert_not_called()
class TestRecordGatewayInjection:
"""The injection marker spend accounting gates prompt-caching savings on."""