Skip to content

Commit 1e5a863

Browse files
committed
fix: use catalog-marked model in Responses-API auth-failure regression test
The Responses API fallback regression test used openai/gpt-5 to exercise the stream() method's Responses-API path. After switching routing to read use_responses_api from the catalog, gpt-5 (the base model, which the catalog explicitly marks use_responses_api: false) now routes to Chat Completions and never hits the Responses-API code under test. Switch to openai/gpt-5-pro, which the catalog marks for Responses-API routing. The test now exercises the path it claims to cover.
1 parent 421ed91 commit 1e5a863

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

tests/regression/test_bug_stale_client_after_invalidation.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -130,11 +130,14 @@ async def test_bug_litellm_client_reset_on_auth_failure_stream_chat_completions(
130130

131131
@pytest.mark.asyncio
132132
async def test_bug_litellm_client_reset_on_auth_failure_stream_responses_api() -> None:
133-
"""LiteLLMClient.stream() GPT-5 Responses API fallback path (litellm.py line ~547).
133+
"""LiteLLMClient.stream() Responses API fallback path.
134134
135-
GPT-5 models fall back to non-streaming Responses API in the stream() method.
135+
Models marked use_responses_api: true in the catalog fall back to
136+
non-streaming Responses API in the stream() method. Pick a model
137+
the catalog actually marks (not gpt-5 base) so the routing decision
138+
follows the catalog's source of truth.
136139
"""
137-
client = _make_litellm_client("openai/gpt-5")
140+
client = _make_litellm_client("openai/gpt-5-pro")
138141
mock_openai = _inject_mock_client(client)
139142
mock_openai.responses.create = AsyncMock(side_effect=Exception("unauthorized request"))
140143

0 commit comments

Comments
 (0)