From 9b556291e955dfeeae4dc0a3a008d9876485c162 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Tue, 28 Apr 2026 07:04:18 +0100 Subject: [PATCH] test(openai): split codex catalog fixtures --- .../test-support/provider-catalog.contract-test-support.ts | 4 ++-- src/plugin-sdk/provider-test-contracts.ts | 1 + src/plugin-sdk/test-helpers/provider-catalog.ts | 1 + src/plugin-sdk/testing.ts | 1 + src/plugins/provider-runtime.test-support.ts | 6 ++++++ 5 files changed, 11 insertions(+), 2 deletions(-) diff --git a/extensions/openai/test-support/provider-catalog.contract-test-support.ts b/extensions/openai/test-support/provider-catalog.contract-test-support.ts index 353c476aa84..70d201ae87e 100644 --- a/extensions/openai/test-support/provider-catalog.contract-test-support.ts +++ b/extensions/openai/test-support/provider-catalog.contract-test-support.ts @@ -4,7 +4,7 @@ import { } from "openclaw/plugin-sdk/plugin-test-runtime"; import { expectAugmentedCodexCatalog, - expectedAugmentedOpenaiCodexCatalogEntriesWithGpt55, + expectedOpenaiPluginCodexCatalogEntriesWithGpt55, expectCodexMissingAuthHint, importProviderRuntimeCatalogModule, loadBundledPluginPublicSurface, @@ -129,7 +129,7 @@ export function describeOpenAIProviderCatalogContract() { const { augmentModelCatalogWithProviderPlugins } = await contractDepsPromise; await expectAugmentedCodexCatalog( augmentModelCatalogWithProviderPlugins, - expectedAugmentedOpenaiCodexCatalogEntriesWithGpt55, + expectedOpenaiPluginCodexCatalogEntriesWithGpt55, ); }); }, diff --git a/src/plugin-sdk/provider-test-contracts.ts b/src/plugin-sdk/provider-test-contracts.ts index dd9804c29b1..2f9388cf1e6 100644 --- a/src/plugin-sdk/provider-test-contracts.ts +++ b/src/plugin-sdk/provider-test-contracts.ts @@ -6,6 +6,7 @@ export { export { expectAugmentedCodexCatalog, expectedAugmentedOpenaiCodexCatalogEntriesWithGpt55, + expectedOpenaiPluginCodexCatalogEntriesWithGpt55, expectCodexMissingAuthHint, importProviderRuntimeCatalogModule, loadBundledPluginPublicSurface, diff --git a/src/plugin-sdk/test-helpers/provider-catalog.ts b/src/plugin-sdk/test-helpers/provider-catalog.ts index 71f25e68ccf..04a96826e98 100644 --- a/src/plugin-sdk/test-helpers/provider-catalog.ts +++ b/src/plugin-sdk/test-helpers/provider-catalog.ts @@ -1,6 +1,7 @@ export { expectAugmentedCodexCatalog, expectedAugmentedOpenaiCodexCatalogEntriesWithGpt55, + expectedOpenaiPluginCodexCatalogEntriesWithGpt55, expectCodexMissingAuthHint, } from "../testing.js"; export type { ProviderPlugin } from "../provider-model-shared.js"; diff --git a/src/plugin-sdk/testing.ts b/src/plugin-sdk/testing.ts index 72cd311d66b..0896cabcb5e 100644 --- a/src/plugin-sdk/testing.ts +++ b/src/plugin-sdk/testing.ts @@ -53,6 +53,7 @@ export { resolveBundledExplicitProviderContractsFromPublicArtifacts } from "../p export { expectAugmentedCodexCatalog, expectedAugmentedOpenaiCodexCatalogEntriesWithGpt55, + expectedOpenaiPluginCodexCatalogEntriesWithGpt55, expectCodexMissingAuthHint, } from "../plugins/provider-runtime.test-support.js"; export { diff --git a/src/plugins/provider-runtime.test-support.ts b/src/plugins/provider-runtime.test-support.ts index 5ce2e175654..f47cb61f2e1 100644 --- a/src/plugins/provider-runtime.test-support.ts +++ b/src/plugins/provider-runtime.test-support.ts @@ -15,6 +15,7 @@ export const expectedAugmentedOpenaiCodexCatalogEntries = [ { provider: "openai", id: "gpt-5.4-nano", name: "gpt-5.4-nano" }, { provider: "openai-codex", id: "gpt-5.4", name: "gpt-5.4" }, { provider: "openai-codex", id: "gpt-5.4-pro", name: "gpt-5.4-pro" }, + { provider: "openai-codex", id: "gpt-5.4-mini", name: "gpt-5.4-mini" }, ]; export const expectedAugmentedOpenaiCodexCatalogEntriesWithGpt55 = [ @@ -24,6 +25,11 @@ export const expectedAugmentedOpenaiCodexCatalogEntriesWithGpt55 = [ ...expectedAugmentedOpenaiCodexCatalogEntries.slice(4), ]; +export const expectedOpenaiPluginCodexCatalogEntriesWithGpt55 = + expectedAugmentedOpenaiCodexCatalogEntriesWithGpt55.filter( + (entry) => !(entry.provider === "openai-codex" && entry.id === "gpt-5.4-mini"), + ); + export function expectCodexMissingAuthHint( buildProviderMissingAuthMessageWithPlugin: (params: { provider: string;