zfc/sub2api
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

test: align codex tool-call id assertions with fc_ prefix

Browse Source
This commit is contained in:
shaw 2026-05-19 09:45:05 +08:00
parent 11870cf84f
commit a729752de6
1 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
backend/internal/service/openai_codex_transform_test.go
View File

@ -41,7 +41,7 @@ func TestApplyCodexOAuthTransform_ToolContinuationPreservesInput(t *testing.T) {
second, ok := input[1].(map[string]any) second, ok := input[1].(map[string]any)
require.True(t, ok) require.True(t, ok)
require.Equal(t, "o1", second["id"]) require.Equal(t, "o1", second["id"])
require.Equal(t, "fc1", second["call_id"]) require.Equal(t, "fc_1", second["call_id"])
} }
func TestApplyCodexOAuthTransform_MessagesBridgePromptCacheKeyIsHeaderOnly(t *testing.T) { func TestApplyCodexOAuthTransform_MessagesBridgePromptCacheKeyIsHeaderOnly(t *testing.T) {
@ -120,11 +120,11 @@ func TestApplyCodexOAuthTransform_ToolContinuationNormalizesToolReferenceIDsOnly
first, ok := input[0].(map[string]any) first, ok := input[0].(map[string]any)
require.True(t, ok) require.True(t, ok)
require.Equal(t, "fc1", first["id"]) require.Equal(t, "fc_1", first["id"])
second, ok := input[1].(map[string]any) second, ok := input[1].(map[string]any)
require.True(t, ok) require.True(t, ok)
require.Equal(t, "fc1", second["call_id"]) require.Equal(t, "fc_1", second["call_id"])
} }
func TestApplyCodexOAuthTransform_ToolSearchOutputPreservesCallID(t *testing.T) { func TestApplyCodexOAuthTransform_ToolSearchOutputPreservesCallID(t *testing.T) {
@ -144,7 +144,7 @@ func TestApplyCodexOAuthTransform_ToolSearchOutputPreservesCallID(t *testing.T)
first, ok := input[0].(map[string]any) first, ok := input[0].(map[string]any)
require.True(t, ok) require.True(t, ok)
require.Equal(t, "tool_search_output", first["type"]) require.Equal(t, "tool_search_output", first["type"])
require.Equal(t, "fc1", first["call_id"]) require.Equal(t, "fc_1", first["call_id"])
} }
func TestApplyCodexOAuthTransform_CustomAndMCPToolOutputsPreserveCallID(t *testing.T) { func TestApplyCodexOAuthTransform_CustomAndMCPToolOutputsPreserveCallID(t *testing.T) {
@ -164,11 +164,11 @@ func TestApplyCodexOAuthTransform_CustomAndMCPToolOutputsPreserveCallID(t *testi
first, ok := input[0].(map[string]any) first, ok := input[0].(map[string]any)
require.True(t, ok) require.True(t, ok)
require.Equal(t, "fccustom", first["call_id"]) require.Equal(t, "fc_custom", first["call_id"])
second, ok := input[1].(map[string]any) second, ok := input[1].(map[string]any)
require.True(t, ok) require.True(t, ok)
require.Equal(t, "fcmcp", second["call_id"]) require.Equal(t, "fc_mcp", second["call_id"])
} }
func TestApplyCodexOAuthTransform_ImageAndWebSearchCallsDoNotGainCallID(t *testing.T) { func TestApplyCodexOAuthTransform_ImageAndWebSearchCallsDoNotGainCallID(t *testing.T) {
@ -221,7 +221,7 @@ func TestApplyCodexOAuthTransform_ConvertsToolRoleMessageToFunctionCallOutput(t
item, ok := input[0].(map[string]any) item, ok := input[0].(map[string]any)
require.True(t, ok) require.True(t, ok)
require.Equal(t, "function_call_output", item["type"]) require.Equal(t, "function_call_output", item["type"])
require.Equal(t, "fc1", item["call_id"]) require.Equal(t, "fc_1", item["call_id"])
require.Equal(t, "ok", item["output"]) require.Equal(t, "ok", item["output"])
_, hasRole := item["role"] _, hasRole := item["role"]
require.False(t, hasRole) require.False(t, hasRole)
@ -340,7 +340,7 @@ func TestApplyCodexOAuthTransform_AddsFallbackNameForFunctionCallInput(t *testin
require.True(t, ok) require.True(t, ok)
require.Equal(t, "function_call", item["type"]) require.Equal(t, "function_call", item["type"])
require.Equal(t, "tool", item["name"]) require.Equal(t, "tool", item["name"])
require.Equal(t, "fc1", item["call_id"]) require.Equal(t, "fc_1", item["call_id"])
} }
func TestApplyCodexOAuthTransform_PreservesFunctionCallInputName(t *testing.T) { func TestApplyCodexOAuthTransform_PreservesFunctionCallInputName(t *testing.T) {
@ -359,7 +359,7 @@ func TestApplyCodexOAuthTransform_PreservesFunctionCallInputName(t *testing.T) {
item, ok := input[0].(map[string]any) item, ok := input[0].(map[string]any)
require.True(t, ok) require.True(t, ok)
require.Equal(t, "shell", item["name"]) require.Equal(t, "shell", item["name"])
require.Equal(t, "fc1", item["call_id"]) require.Equal(t, "fc_1", item["call_id"])
} }
func TestApplyCodexOAuthTransform_PreservesMCPToolCallIDAndName(t *testing.T) { func TestApplyCodexOAuthTransform_PreservesMCPToolCallIDAndName(t *testing.T) {
@ -384,7 +384,7 @@ func TestApplyCodexOAuthTransform_PreservesMCPToolCallIDAndName(t *testing.T) {
require.True(t, ok) require.True(t, ok)
require.Equal(t, "mcp_tool_call", item["type"]) require.Equal(t, "mcp_tool_call", item["type"])
require.Equal(t, "remote_tool", item["name"]) require.Equal(t, "remote_tool", item["name"])
require.Equal(t, "fcabc", item["call_id"]) require.Equal(t, "fc_abc", item["call_id"])
} }
func TestCodexInputItemRequiresNameTypesAllowCallID(t *testing.T) { func TestCodexInputItemRequiresNameTypesAllowCallID(t *testing.T) {