From d80c4f49a42772b2b73d123f19ad055f616b37ca Mon Sep 17 00:00:00 2001 From: mateo-berri <277851410+mateo-berri@users.noreply.github.com> Date: Sat, 20 Jun 2026 07:51:17 +0000 Subject: [PATCH] test(e2e): mark known pricing-leak isolation test xfail(strict) test_custom_pricing_is_isolated_from_sibling_deployment documents a real proxy gap (a deployment's custom per-token pricing leaks into the shared cost map for sibling deployments of the same underlying model) and was left unconditionally failing, which pollutes the suite's pass/fail signal. Mark it xfail(strict=True) so the suite stays green while the leak persists and turns into a failure the moment isolation is fixed, prompting the marker's removal. --- tests/e2e/llm_translation/test_custom_pricing_e2e.py | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/tests/e2e/llm_translation/test_custom_pricing_e2e.py b/tests/e2e/llm_translation/test_custom_pricing_e2e.py index 6c8619a075e..4b3e87b78e1 100644 --- a/tests/e2e/llm_translation/test_custom_pricing_e2e.py +++ b/tests/e2e/llm_translation/test_custom_pricing_e2e.py @@ -10,8 +10,9 @@ back here from the same config file. Three behaviors are checked independently: - reporting: /model/info surfaces those rates for the model - isolation: gemini-2.5-flash shares the same underlying gemini/gemini-2.5-flash but sets no override, so it must keep its own price; an override that leaks into - the shared cost map misprices it. This currently fails on a real gap and is left - failing rather than weakened to pass. + the shared cost map misprices it. This fails on a real proxy gap today, so it is + marked xfail(strict=True): the suite stays green while the leak persists and + flips to a failure the moment isolation is fixed and the marker should be removed. """ import time @@ -191,6 +192,12 @@ def test_model_info_reports_custom_pricing(client: PassthroughClient) -> None: ) +@pytest.mark.xfail( + strict=True, + reason="known proxy bug: a deployment's custom per-token pricing leaks into the " + "shared cost map for sibling deployments of the same underlying model; remove " + "this marker once isolation is fixed", +) def test_custom_pricing_is_isolated_from_sibling_deployment( client: PassthroughClient, ) -> None: