From 750b83a2ea9b2679f376801b3b5ee02fe7d6bda1 Mon Sep 17 00:00:00 2001 From: jnjpng Date: Thu, 26 Feb 2026 11:42:18 -0800 Subject: [PATCH] fix: update conversation manager tests for auto-generated system message (#9687) fix: update Anthropic mock to match real SDK's sync list() signature The real Anthropic SDK's models.list() is a regular (non-async) method that returns an AsyncPaginator (async-iterable). The mock used async def, causing `async for model in client.models.list()` to iterate over a coroutine instead of the page, silently failing with 0 models synced. --- tests/managers/test_provider_manager.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/managers/test_provider_manager.py b/tests/managers/test_provider_manager.py index a7c16fad..d16f41d5 100644 --- a/tests/managers/test_provider_manager.py +++ b/tests/managers/test_provider_manager.py @@ -880,6 +880,8 @@ async def test_server_startup_handles_api_errors_gracefully(default_user, defaul for item in self._items: yield item + # NOTE: The real SDK's models.list() is a regular (non-async) method that + # returns an AsyncPaginator (which is async-iterable). class MockAnthropicModels: def list(self): return MockAnthropicAsyncPage(mock_anthropic_data)