From 075e3ab69ee1fc23239fd73202a9843631990678 Mon Sep 17 00:00:00 2001 From: hkfires <10558748+hkfires@users.noreply.github.com> Date: Thu, 5 Feb 2026 09:25:34 +0800 Subject: [PATCH] fix(test): rename test function to reflect behavior change for builtin tools --- test/builtin_tools_translation_test.go | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/test/builtin_tools_translation_test.go b/test/builtin_tools_translation_test.go index b4ca7b0d..07d76715 100644 --- a/test/builtin_tools_translation_test.go +++ b/test/builtin_tools_translation_test.go @@ -33,7 +33,7 @@ func TestOpenAIToCodex_PreservesBuiltinTools(t *testing.T) { } } -func TestOpenAIResponsesToOpenAI_PreservesBuiltinTools(t *testing.T) { +func TestOpenAIResponsesToOpenAI_IgnoresBuiltinTools(t *testing.T) { in := []byte(`{ "model":"gpt-5", "input":[{"role":"user","content":[{"type":"input_text","text":"hi"}]}], @@ -42,13 +42,7 @@ func TestOpenAIResponsesToOpenAI_PreservesBuiltinTools(t *testing.T) { out := sdktranslator.TranslateRequest(sdktranslator.FormatOpenAIResponse, sdktranslator.FormatOpenAI, "gpt-5", in, false) - if got := gjson.GetBytes(out, "tools.#").Int(); got != 1 { - t.Fatalf("expected 1 tool, got %d: %s", got, string(out)) - } - if got := gjson.GetBytes(out, "tools.0.type").String(); got != "web_search" { - t.Fatalf("expected tools[0].type=web_search, got %q: %s", got, string(out)) - } - if got := gjson.GetBytes(out, "tools.0.search_context_size").String(); got != "low" { - t.Fatalf("expected tools[0].search_context_size=low, got %q: %s", got, string(out)) + if got := gjson.GetBytes(out, "tools.#").Int(); got != 0 { + t.Fatalf("expected 0 tools (builtin tools not supported in Chat Completions), got %d: %s", got, string(out)) } }