From f10e56be7eba3c0fa767696a20c6032433d370ee Mon Sep 17 00:00:00 2001 From: Ethan0x0000 <3352979663@qq.com> Date: Tue, 24 Mar 2026 09:52:56 +0800 Subject: [PATCH] refactor(test): improve type assertions in ops endpoint context tests --- backend/internal/handler/ops_error_logger_test.go | 12 +++++++++--- .../pkg/apicompat/anthropic_to_responses_response.go | 5 ----- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/backend/internal/handler/ops_error_logger_test.go b/backend/internal/handler/ops_error_logger_test.go index f78d25a1..6ae45110 100644 --- a/backend/internal/handler/ops_error_logger_test.go +++ b/backend/internal/handler/ops_error_logger_test.go @@ -285,11 +285,15 @@ func TestSetOpsEndpointContext_SetsContextKeys(t *testing.T) { v, ok := c.Get(opsUpstreamModelKey) require.True(t, ok) - require.Equal(t, "claude-3-5-sonnet-20241022", v.(string)) + vStr, ok := v.(string) + require.True(t, ok) + require.Equal(t, "claude-3-5-sonnet-20241022", vStr) rt, ok := c.Get(opsRequestTypeKey) require.True(t, ok) - require.Equal(t, int16(2), rt.(int16)) + rtVal, ok := rt.(int16) + require.True(t, ok) + require.Equal(t, int16(2), rtVal) } func TestSetOpsEndpointContext_EmptyModelNotStored(t *testing.T) { @@ -305,7 +309,9 @@ func TestSetOpsEndpointContext_EmptyModelNotStored(t *testing.T) { rt, ok := c.Get(opsRequestTypeKey) require.True(t, ok) - require.Equal(t, int16(1), rt.(int16)) + rtVal, ok := rt.(int16) + require.True(t, ok) + require.Equal(t, int16(1), rtVal) } func TestSetOpsEndpointContext_NilContext(t *testing.T) { diff --git a/backend/internal/pkg/apicompat/anthropic_to_responses_response.go b/backend/internal/pkg/apicompat/anthropic_to_responses_response.go index 0c5e330c..9290e399 100644 --- a/backend/internal/pkg/apicompat/anthropic_to_responses_response.go +++ b/backend/internal/pkg/apicompat/anthropic_to_responses_response.go @@ -397,11 +397,6 @@ func anthToResHandleMessageDelta(evt *AnthropicStreamEvent, state *AnthropicEven } } - // Extract stop_reason from delta - if evt.Delta != nil && evt.Delta.StopReason != "" { - // We'll use this in message_stop - } - return nil }