From 4f56e8a7d500019c564ccab9eb11d556d0c6ed5e Mon Sep 17 00:00:00 2001 From: Yuneng Jiang Date: Wed, 26 Aug 2026 18:09:10 -0700 Subject: [PATCH] test(e2e): un-skip the per-model budget update case The case was skipped because /budget/update 500d on any model_max_budget. #38430 fixes that by serializing the update payload before the write, so the case now passes against a proxy carrying that change and there is nothing left for the skip to hide. Merge this after #38430; on staging alone the case still fails with the same 500 it was skipped for. --- tests/e2e/management/test_budget_customer_user_org_e2e.py | 6 ------ 1 file changed, 6 deletions(-) diff --git a/tests/e2e/management/test_budget_customer_user_org_e2e.py b/tests/e2e/management/test_budget_customer_user_org_e2e.py index 9caf042803b..6e14a2d5745 100644 --- a/tests/e2e/management/test_budget_customer_user_org_e2e.py +++ b/tests/e2e/management/test_budget_customer_user_org_e2e.py @@ -163,12 +163,6 @@ class TestBudgetManagement: f"/budget/list never included the created budget {budget_id}", ) - @pytest.mark.skip( - reason=( - "stage red: product gap, /budget/update 500s on any model_max_budget " - "(prisma Json arg + unquoted GraphQL interpolation)" - ) - ) @pytest.mark.covers("mgmt.budget.update.accepts_model_max_budget") def test_update_accepts_per_model_budgets_including_punctuated_names( self, client: ManagementClient, resources: ResourceManager