| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325 |
- package service
- import (
- "net/http"
- "strings"
- "testing"
- "time"
- "github.com/stretchr/testify/assert"
- "github.com/stretchr/testify/require"
- "github.com/tidwall/gjson"
- )
- const keepaliveTestInterval = 10 * time.Millisecond
- // waitForKeepaliveBeats 等待至少一次心跳写出。读取 recorder 前必须先经
- // StopOpenAICompactSSEKeepaliveCommitted 停拍建立 happens-before。
- func waitForKeepaliveBeats() {
- time.Sleep(20 * keepaliveTestInterval)
- }
- // stripKeepaliveComments 去掉 SSE 注释块,返回真实事件文本。
- func stripKeepaliveComments(body string) string {
- var blocks []string
- for _, block := range strings.Split(strings.TrimSpace(body), "\n\n") {
- if strings.HasPrefix(strings.TrimSpace(block), ":") {
- continue
- }
- blocks = append(blocks, block)
- }
- return strings.Join(blocks, "\n\n")
- }
- func TestStartOpenAICompactSSEKeepalive_NoopWhenUnmarkedOrDisabled(t *testing.T) {
- // 未标记 client stream:不启动。
- c, rec := newCompactBridgeTestContext(t, false)
- stop := StartOpenAICompactSSEKeepalive(c, keepaliveTestInterval)
- waitForKeepaliveBeats()
- stop()
- require.Zero(t, rec.Body.Len())
- require.False(t, StopOpenAICompactSSEKeepaliveCommitted(c))
- // interval=0(配置禁用):不启动。
- c, rec = newCompactBridgeTestContext(t, true)
- stop = StartOpenAICompactSSEKeepalive(c, 0)
- waitForKeepaliveBeats()
- stop()
- require.Zero(t, rec.Body.Len())
- require.False(t, StopOpenAICompactSSEKeepaliveCommitted(c))
- }
- func TestOpenAICompactSSEKeepalive_CommitsHeadersAndComments(t *testing.T) {
- c, rec := newCompactBridgeTestContext(t, true)
- stop := StartOpenAICompactSSEKeepalive(c, keepaliveTestInterval)
- defer stop()
- waitForKeepaliveBeats()
- require.True(t, StopOpenAICompactSSEKeepaliveCommitted(c))
- require.Equal(t, http.StatusOK, rec.Code)
- require.Equal(t, "text/event-stream", rec.Header().Get("Content-Type"))
- require.Equal(t, "no", rec.Header().Get("X-Accel-Buffering"))
- require.Contains(t, rec.Body.String(), ": keepalive\n\n")
- }
- func TestOpenAICompactSSEKeepalive_StopBeforeFirstBeatKeepsWriterUntouched(t *testing.T) {
- c, rec := newCompactBridgeTestContext(t, true)
- stop := StartOpenAICompactSSEKeepalive(c, time.Hour)
- stop()
- waitForKeepaliveBeats()
- require.Zero(t, rec.Body.Len())
- require.False(t, StopOpenAICompactSSEKeepaliveCommitted(c))
- }
- func TestOpenAIAdjustedWrittenSizeExcludesResponsesStreamKeepalive(t *testing.T) {
- c, rec := newCompactBridgeTestContext(t, false)
- n, err := c.Writer.Write([]byte(":\n\n"))
- require.NoError(t, err)
- recordOpenAIStreamKeepaliveBytes(c, n)
- require.Equal(t, -1, OpenAICompactKeepaliveAdjustedWrittenSize(c))
- _, err = c.Writer.Write([]byte("data: semantic\n\n"))
- require.NoError(t, err)
- require.Equal(t, len("data: semantic\n\n"), OpenAICompactKeepaliveAdjustedWrittenSize(c))
- require.Equal(t, ":\n\ndata: semantic\n\n", rec.Body.String())
- }
- // 心跳已提交后,2xx 桥接续写事件而不重复提交响应头。
- func TestWriteOpenAICompactSSEBridge_AfterKeepaliveCommitAppendsEvents(t *testing.T) {
- c, rec := newCompactBridgeTestContext(t, true)
- stop := StartOpenAICompactSSEKeepalive(c, keepaliveTestInterval)
- defer stop()
- waitForKeepaliveBeats()
- finalResponse := []byte(`{"id":"resp_ka_1","output":[{"id":"cmp_ka","type":"compaction","encrypted_content":"x"}],"usage":{"input_tokens":1,"output_tokens":1,"total_tokens":2}}`)
- require.True(t, writeOpenAICompactSSEBridge(c, http.StatusOK, finalResponse))
- require.Equal(t, http.StatusOK, rec.Code)
- events := parseCompactBridgeSSE(t, stripKeepaliveComments(rec.Body.String()))
- require.Len(t, events, 2)
- require.Equal(t, "response.output_item.done", events[0][0])
- require.Equal(t, "compaction", gjson.Get(events[0][1], "item.type").String())
- require.Equal(t, "response.completed", events[1][0])
- require.Equal(t, "resp_ka_1", gjson.Get(events[1][1], "response.id").String())
- }
- // 心跳已提交后上游非 2xx:状态码无法回传,必须以 response.failed 终止事件
- // 收尾(Codex 将其作为终止事件处理),并标记流内错误供 ops 采集。
- func TestWriteOpenAICompactSSEBridge_AfterKeepaliveCommitFailureEmitsFailedEvent(t *testing.T) {
- c, rec := newCompactBridgeTestContext(t, true)
- stop := StartOpenAICompactSSEKeepalive(c, keepaliveTestInterval)
- defer stop()
- waitForKeepaliveBeats()
- require.True(t, writeOpenAICompactSSEBridge(c, http.StatusBadGateway, []byte(`{"error":{"message":"upstream exploded"}}`)))
- events := parseCompactBridgeSSE(t, stripKeepaliveComments(rec.Body.String()))
- require.Len(t, events, 1)
- require.Equal(t, "response.failed", events[0][0])
- require.Equal(t, "failed", gjson.Get(events[0][1], "response.status").String())
- require.Contains(t, gjson.Get(events[0][1], "response.error.message").String(), "upstream exploded")
- require.NotEmpty(t, gjson.Get(events[0][1], "response.id").String())
- streamErr, ok := GetOpsStreamError(c)
- require.True(t, ok)
- require.Equal(t, http.StatusBadGateway, streamErr.IntendedStatus)
- }
- // 心跳未提交时非 2xx 行为不变:返回 false,调用方按原 JSON+状态码写回。
- func TestWriteOpenAICompactSSEBridge_BeforeKeepaliveCommitFailureKeepsJSONPath(t *testing.T) {
- c, rec := newCompactBridgeTestContext(t, true)
- stop := StartOpenAICompactSSEKeepalive(c, time.Hour)
- stop()
- require.False(t, writeOpenAICompactSSEBridge(c, http.StatusBadGateway, []byte(`{"error":{"message":"fast fail"}}`)))
- require.Zero(t, rec.Body.Len())
- }
- // 未被显式拦截的写回路径(直接操作 c.Writer)也必须与心跳互斥:包装器在
- // 请求侧任何响应构造时停拍。-race 下验证无数据竞争,且停拍后不再有心跳
- // 字节写出。
- func TestOpenAICompactKeepaliveWriter_RequestSideWriteSuspendsBeats(t *testing.T) {
- c, rec := newCompactBridgeTestContext(t, true)
- stop := StartOpenAICompactSSEKeepalive(c, keepaliveTestInterval)
- defer stop()
- waitForKeepaliveBeats()
- // 模拟未拦截路径的直接写回(如 Forward 内部本地拒绝的 c.JSON)。
- _, err := c.Writer.Write([]byte(`{"error":"local reject"}`))
- require.NoError(t, err)
- lenAfterWrite := rec.Body.Len()
- waitForKeepaliveBeats()
- require.Equal(t, lenAfterWrite, rec.Body.Len(), "请求侧写回后心跳必须停止")
- require.Contains(t, rec.Body.String(), ": keepalive\n\n")
- require.Contains(t, rec.Body.String(), `{"error":"local reject"}`)
- }
- func TestOpenAICompactKeepaliveWriter_NilInnerWriter_NoPanic(t *testing.T) {
- w := &openAICompactKeepaliveWriter{
- k: &openAICompactSSEKeepalive{stop: make(chan struct{})},
- }
- w.ResponseWriter = nil
- assert.NotPanics(t, func() {
- assert.Equal(t, 0, w.Status())
- })
- assert.NotPanics(t, func() {
- assert.Equal(t, 0, w.Size())
- })
- assert.NotPanics(t, func() {
- assert.False(t, w.Written())
- })
- assert.NotPanics(t, func() {
- assert.NotNil(t, w.Header())
- })
- assert.NotPanics(t, func() {
- n, err := w.Write([]byte("test"))
- assert.Equal(t, 0, n)
- assert.NoError(t, err)
- })
- assert.NotPanics(t, func() {
- n, err := w.WriteString("test")
- assert.Equal(t, 0, n)
- assert.NoError(t, err)
- })
- assert.NotPanics(t, func() {
- w.WriteHeader(http.StatusOK)
- })
- assert.NotPanics(t, func() {
- w.WriteHeaderNow()
- })
- assert.NotPanics(t, func() {
- w.Flush()
- })
- assert.NotPanics(t, func() {
- conn, rw, err := w.Hijack()
- assert.Nil(t, conn)
- assert.Nil(t, rw)
- assert.Error(t, err)
- })
- assert.NotPanics(t, func() {
- ch := w.CloseNotify()
- assert.NotNil(t, ch)
- })
- assert.NotPanics(t, func() {
- assert.Nil(t, w.Pusher())
- })
- }
- func TestOpenAICompactKeepaliveWriter_NilKeepalive_NoPanic(t *testing.T) {
- c, rec := newCompactBridgeTestContext(t, true)
- w := &openAICompactKeepaliveWriter{ResponseWriter: c.Writer}
- assert.NotPanics(t, func() {
- assert.Equal(t, 0, w.Status())
- })
- assert.NotPanics(t, func() {
- assert.Equal(t, 0, w.Size())
- })
- assert.NotPanics(t, func() {
- assert.False(t, w.Written())
- })
- assert.NotPanics(t, func() {
- w.Header().Set("X-Test", "ok")
- })
- assert.NotPanics(t, func() {
- w.WriteHeader(http.StatusAccepted)
- })
- assert.NotPanics(t, func() {
- n, err := w.WriteString("ok")
- assert.Equal(t, 2, n)
- assert.NoError(t, err)
- })
- assert.NotPanics(t, func() {
- w.Flush()
- })
- require.Equal(t, "ok", rec.Header().Get("X-Test"))
- require.Equal(t, "ok", rec.Body.String())
- }
- func TestOpenAICompactKeepaliveWriter_DelegatesWhenReady(t *testing.T) {
- c, rec := newCompactBridgeTestContext(t, true)
- stop := StartOpenAICompactSSEKeepalive(c, time.Hour)
- defer stop()
- w, ok := c.Writer.(*openAICompactKeepaliveWriter)
- require.True(t, ok)
- w.Header().Set("X-Test", "ok")
- w.WriteHeader(http.StatusAccepted)
- n, err := w.WriteString("ready")
- require.NoError(t, err)
- require.Equal(t, len("ready"), n)
- require.Equal(t, http.StatusAccepted, w.Status())
- require.Equal(t, len("ready"), w.Size())
- require.True(t, w.Written())
- require.Equal(t, "ok", rec.Header().Get("X-Test"))
- require.Equal(t, "ready", rec.Body.String())
- }
- // fast policy block 在心跳提交后必须降级为 response.failed 终止事件。
- func TestWriteOpenAIFastPolicyBlockedResponse_AfterKeepaliveCommit(t *testing.T) {
- c, rec := newCompactBridgeTestContext(t, true)
- stop := StartOpenAICompactSSEKeepalive(c, keepaliveTestInterval)
- defer stop()
- waitForKeepaliveBeats()
- writeOpenAIFastPolicyBlockedResponse(c, &OpenAIFastBlockedError{Message: "tier blocked"})
- require.Equal(t, http.StatusOK, rec.Code)
- events := parseCompactBridgeSSE(t, stripKeepaliveComments(rec.Body.String()))
- require.Len(t, events, 1)
- require.Equal(t, "response.failed", events[0][0])
- require.Equal(t, "permission_error", gjson.Get(events[0][1], "response.error.code").String())
- require.Contains(t, gjson.Get(events[0][1], "response.error.message").String(), "tier blocked")
- }
- // failover"是否已写响应"判定的口径:心跳字节必须被排除,否则 compact 在
- // 上游等待期间发过心跳后,可换号的 failover 会被误判放弃;真实响应字节
- // 写出后口径必须变化。
- func TestOpenAICompactKeepaliveAdjustedWrittenSize_ExcludesHeartbeatBytes(t *testing.T) {
- c, rec := newCompactBridgeTestContext(t, true)
- // 无心跳的请求:等价于 c.Writer.Size()。
- require.Equal(t, c.Writer.Size(), OpenAICompactKeepaliveAdjustedWrittenSize(c))
- stop := StartOpenAICompactSSEKeepalive(c, keepaliveTestInterval)
- defer stop()
- before := OpenAICompactKeepaliveAdjustedWrittenSize(c)
- waitForKeepaliveBeats()
- require.Equal(t, before, OpenAICompactKeepaliveAdjustedWrittenSize(c), "仅心跳字节不得改变判定口径")
- // 真实响应字节写出(经包装器,先停拍再写)后口径必须变化。
- _, err := c.Writer.Write([]byte("real-bytes"))
- require.NoError(t, err)
- require.Equal(t, len("real-bytes"), OpenAICompactKeepaliveAdjustedWrittenSize(c))
- require.Contains(t, rec.Body.String(), ": keepalive\n\n")
- }
- func TestOpenAIStreamClientOutputStarted_IgnoresCompactKeepaliveBytes(t *testing.T) {
- c, _ := newCompactBridgeTestContext(t, true)
- stop := StartOpenAICompactSSEKeepalive(c, keepaliveTestInterval)
- defer stop()
- waitForKeepaliveBeats()
- require.True(t, c.Writer.Written())
- require.False(t, openAIStreamClientOutputStarted(c, false), "keepalive comments are not semantic output")
- _, err := c.Writer.Write([]byte("real-output"))
- require.NoError(t, err)
- require.True(t, openAIStreamClientOutputStarted(c, false))
- }
- // fast policy block 在心跳未提交时保持 403 JSON 原语义。
- func TestWriteOpenAIFastPolicyBlockedResponse_BeforeKeepaliveCommit(t *testing.T) {
- c, rec := newCompactBridgeTestContext(t, true)
- stop := StartOpenAICompactSSEKeepalive(c, time.Hour)
- defer stop()
- writeOpenAIFastPolicyBlockedResponse(c, &OpenAIFastBlockedError{Message: "tier blocked"})
- require.Equal(t, http.StatusForbidden, rec.Code)
- require.Equal(t, "permission_error", gjson.Get(rec.Body.String(), "error.type").String())
- }
|