From 3180505776fde46852d15ff29212b80326d68d33 Mon Sep 17 00:00:00 2001 From: Dawei Gu Date: Wed, 6 May 2026 17:09:30 -0700 Subject: [PATCH] test(bedrock): move retrieve_batch dispatch test to tests/test_litellm/ MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The dispatch test landed under `tests/test_litellm/batches/`, a new directory that no upstream `test-unit-*.yml` workflow's `test-path` allow-list includes. As a result, the test was never executed in CI and codecov reported `litellm/batches/main.py` patch coverage at 11.11% (8 lines uncovered) — the lines belonging to this PR's dispatch refactor itself. Move the file up one level so it matches the `tests/test_litellm/test_*.py` glob that `test-unit-misc.yml` already runs, and adjust `sys.path.insert` for the new depth. The companion handler tests under `tests/test_litellm/llms/bedrock/batches/test_handler.py` are unaffected — they're picked up by the `llms` directory in `test-unit-llm-providers.yml`. Made-with: Cursor --- tests/test_litellm/batches/__init__.py | 0 .../{batches => }/test_retrieve_batch_bedrock_dispatch.py | 2 +- 2 files changed, 1 insertion(+), 1 deletion(-) delete mode 100644 tests/test_litellm/batches/__init__.py rename tests/test_litellm/{batches => }/test_retrieve_batch_bedrock_dispatch.py (99%) diff --git a/tests/test_litellm/batches/__init__.py b/tests/test_litellm/batches/__init__.py deleted file mode 100644 index e69de29bb2d..00000000000 diff --git a/tests/test_litellm/batches/test_retrieve_batch_bedrock_dispatch.py b/tests/test_litellm/test_retrieve_batch_bedrock_dispatch.py similarity index 99% rename from tests/test_litellm/batches/test_retrieve_batch_bedrock_dispatch.py rename to tests/test_litellm/test_retrieve_batch_bedrock_dispatch.py index 1ef789a0615..9df18a9f0f0 100644 --- a/tests/test_litellm/batches/test_retrieve_batch_bedrock_dispatch.py +++ b/tests/test_litellm/test_retrieve_batch_bedrock_dispatch.py @@ -20,7 +20,7 @@ from unittest.mock import MagicMock, patch import pytest -sys.path.insert(0, os.path.abspath("../../..")) +sys.path.insert(0, os.path.abspath("../..")) import litellm # noqa: E402