Skip to content

Commit 60225ab

Browse files
authored
test: assert the prefixed model the azure responses bridge now hands back (#37749)
a369cb0 made completion() hand the prefixed model back to responses(), so that responses() running get_llm_provider() a second time becomes a no-op instead of stripping a prefix the model id owns. That was deliberate, and it shipped with its own unit test, but it left two older assertions behind still expecting the bare id. #37744 corrected the openai one in test_openai.py. This is its azure sibling, which llm_translation_testing has been failing on ever since. Only the expected value moves. The neighbouring custom_llm_provider assertion already passes and stays as it is.
1 parent 4e02e7e commit 60225ab

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

tests/llm_translation/test_azure_openai.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -650,7 +650,7 @@ def test_azure_openai_responses_bridge():
650650
mock_responses.assert_called_once()
651651
assert (
652652
mock_responses.call_args.kwargs["model"]
653-
== "test-azure-computer-use-preview"
653+
== "azure/test-azure-computer-use-preview"
654654
)
655655
assert mock_responses.call_args.kwargs["custom_llm_provider"] == "azure"
656656

0 commit comments

Comments
 (0)