From 888277ece8a92929dd558ee923c23ce4af5c4f0e Mon Sep 17 00:00:00 2001 From: Brooklyn Nicholson Date: Mon, 9 Mar 2026 20:53:04 -0500 Subject: [PATCH] chore: fix vision test by mocking aux client --- tests/tools/test_vision_tools.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/tools/test_vision_tools.py b/tests/tools/test_vision_tools.py index 3bdd301786..c3021891ab 100644 --- a/tests/tools/test_vision_tools.py +++ b/tests/tools/test_vision_tools.py @@ -244,17 +244,17 @@ class TestErrorLoggingExcInfo: with patch("tools.vision_tools._validate_image_url", return_value=True), \ patch("tools.vision_tools._download_image", new_callable=AsyncMock, side_effect=Exception("download boom")), \ + patch("tools.vision_tools._aux_async_client", new="fake"), \ caplog.at_level(logging.ERROR, logger="tools.vision_tools"): result = await vision_analyze_tool( "https://example.com/img.jpg", "describe this", "test/model" ) result_data = json.loads(result) - # Error response uses "success": False, not an "error" key assert result_data["success"] is False error_records = [r for r in caplog.records if r.levelno >= logging.ERROR] - assert any(r.exc_info is not None for r in error_records) + assert any(r.exc_info and r.exc_info[0] is not None for r in error_records) @pytest.mark.asyncio async def test_cleanup_error_logs_exc_info(self, tmp_path, caplog):