From 484a257067f1d56b1796f44aa8996d67de666906 Mon Sep 17 00:00:00 2001 From: hongming Date: Thu, 4 Jun 2026 16:49:06 -0700 Subject: [PATCH] =?UTF-8?q?test(providers):=20unbreak=20main=20=E2=80=94?= =?UTF-8?q?=20namespaced=20vendor=20id=20is=20now=20BYOK-routable=20(cp#52?= =?UTF-8?q?9)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit core#2262 merged via a race on the pre-fix commit, so main carries the stale `platform_shared_openai_namespaced_still_rejected` assertion while the byok-vendor providers (also in that merge) make hermes openai/gpt-4o routable via the tenant's key. Flip the assertion to allowed. Unbreaks CI/Platform(Go). cp#529 Co-Authored-By: Claude Opus 4.8 (1M context) --- .../internal/handlers/model_registry_validation_test.go | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/workspace-server/internal/handlers/model_registry_validation_test.go b/workspace-server/internal/handlers/model_registry_validation_test.go index f9a4658d7..3d7a82234 100644 --- a/workspace-server/internal/handlers/model_registry_validation_test.go +++ b/workspace-server/internal/handlers/model_registry_validation_test.go @@ -115,12 +115,13 @@ func TestValidateRegisteredModelForRuntime(t *testing.T) { wantOK: false, }, { - // A platform-shared namespaced id that MUST remain unroutable on - // hermes (billing guardrail: openai vendor not wired) → still 422. - name: "platform_shared_openai_namespaced_still_rejected", + // A namespaced vendor id NOW routable on hermes via the dedicated + // byok-openai provider (cp#529 BYOK-vendor arms): routes with the + // tenant's OPENAI_API_KEY → BYOK billing, never the platform key. + name: "byok_openai_namespaced_routable_now_allowed", runtime: "hermes", model: "openai/gpt-4o", - wantOK: false, + wantOK: true, }, } for _, c := range cases { -- 2.52.0