From 49170695ce7ec7b3baafe8af70db25da20bac79d Mon Sep 17 00:00:00 2001 From: Yuneng Jiang Date: Thu, 27 Aug 2026 14:50:17 -0700 Subject: [PATCH] test(e2e): drop the fixture helper docstring The why belongs in the commit message and the PR, not above a one-line helper whose name already says what it returns. --- .../test_chat_completions_regression_e2e.py | 7 ------- 1 file changed, 7 deletions(-) diff --git a/tests/e2e/llm_translation/test_chat_completions_regression_e2e.py b/tests/e2e/llm_translation/test_chat_completions_regression_e2e.py index 584dbce8089..68c0dfab897 100644 --- a/tests/e2e/llm_translation/test_chat_completions_regression_e2e.py +++ b/tests/e2e/llm_translation/test_chat_completions_regression_e2e.py @@ -88,13 +88,6 @@ OPENAI_VISION_BACKEND = "openai/gpt-4o" def _cat_image_data_url() -> str: - """The vision image as a data URL, read from a fixture we own. - - An https URL would make every vision run depend on a third-party host staying - up and unthrottled, and a 429 from that host reads as a gateway failure. It also - changes what is under test per provider: litellm downloads the image itself for - bedrock, while openai is handed the link and fetches it from its own servers. A - data URL removes the host and puts both providers on the same bytes.""" return "data:image/jpeg;base64," + base64.b64encode(CAT_IMAGE.read_bytes()).decode()