From b268c17fe25365027bd69e5bfcc608bb4274ae37 Mon Sep 17 00:00:00 2001 From: Mohamed Boudra Date: Sat, 11 Apr 2026 04:39:22 +0000 Subject: [PATCH] test(cli): replace OpenAI-specific assertions with generic third-party check MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The opencode provider test asserted models with "openai/" or "openrouter/openai/" prefixes, but these are environment-dependent — opencode returns whatever providers are connected, and CI may not have OpenAI configured. Replace with a generic check that at least one non-opencode/ namespaced model exists. Co-Authored-By: Claude Opus 4.6 --- packages/cli/tests/15-provider.test.ts | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/packages/cli/tests/15-provider.test.ts b/packages/cli/tests/15-provider.test.ts index e01ac7ae2..f890141d3 100644 --- a/packages/cli/tests/15-provider.test.ts +++ b/packages/cli/tests/15-provider.test.ts @@ -224,15 +224,8 @@ try { "opencode output should include at least one first-party opencode model", ); assert( - ids.some((id) => id.startsWith("openai/") || id.startsWith("openrouter/openai/")), - "opencode output should include at least one OpenAI-backed model", - ); - assert( - ids.some( - (id) => - (id.startsWith("openai/") || id.startsWith("openrouter/openai/")) && id.includes("codex"), - ), - "opencode output should include at least one codex-optimized OpenAI model", + ids.some((id) => id.includes("/") && !id.startsWith("opencode/")), + "opencode output should include at least one third-party provider model", ); assert( data.every((m) => m.model && m.id && m.description !== undefined),