From 556c7f6b68ec5e38bc13a3d9ad10b58a300ef7bc Mon Sep 17 00:00:00 2001 From: Yuneng Jiang Date: Sun, 20 Sep 2026 00:50:05 -0700 Subject: [PATCH] ci: keep real-database coverage for both migration resolvers The Postgres-backed smoke job previously exercised one resolver. Running the default and the legacy variants in it covers v2 now that it is the default, without losing v1's coverage. --- .circleci/config.yml | 3 ++- tests/local_testing/test_basic_python_version.py | 4 ++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/.circleci/config.yml b/.circleci/config.yml index 6d5b9a2258e..ba107472b8d 100644 --- a/.circleci/config.yml +++ b/.circleci/config.yml @@ -1561,9 +1561,10 @@ jobs: url: tcp://localhost:5432 timeout: "60" - run: - name: Run legacy migration resolver proxy smoke test + name: Run both migration resolvers against Postgres command: | uv run --no-sync python -m pytest -vv \ + tests/local_testing/test_basic_python_version.py::test_litellm_proxy_server_config_no_general_settings \ tests/local_testing/test_basic_python_version.py::test_litellm_proxy_server_config_no_general_settings_legacy_resolver helm_chart_testing: diff --git a/tests/local_testing/test_basic_python_version.py b/tests/local_testing/test_basic_python_version.py index 0ce59332417..ef500fdff42 100644 --- a/tests/local_testing/test_basic_python_version.py +++ b/tests/local_testing/test_basic_python_version.py @@ -312,7 +312,7 @@ def test_litellm_proxy_server_config_no_general_settings(): def test_litellm_proxy_server_config_no_general_settings_legacy_resolver(): """Exercises the opt-out legacy (v1) migration resolver. - Runs in a separate CI job against a local Postgres to avoid collisions - with the default variant when they share a database. + Runs after the default variant in the CI job that provides a local + Postgres, so both resolvers get real-database proxy-boot coverage. """ _run_proxy_server_smoke_test(extra_proxy_args=["--use_legacy_migration_resolver"])