Files
ragflow/internal/handler/memory_message_test.go

92 lines
2.8 KiB
Go
Raw Permalink Normal View History

feat: Implement API of ragflow server in Go (#15256) ## Summary - Implemented the Go API endpoint for Memory message forgetting: - `DELETE /api/v1/messages/{memory_id}:{message_id}` - Added route registration for the Memory message DELETE endpoint only. - Added request path validation for `memory_id:message_id`. - Added service logic to mark a message as forgotten by setting `forget_at`. - Preserved Python-compatible response behavior: - Success returns `code: 0`, `message: true`, `data: null`. - Added focused unit tests for message path parsing and invalid message ID handling. - Fixed Linux cgo linker config to use the installed shared PCRE2 library so Go tests/builds can run in this environment. ## Related Issue Closes: #15240 ## Change Type - [x] Feature - [x] Test - [x] Build / CI compatibility ## Implemented API - `DELETE /api/v1/messages/{memory_id}:{message_id}` ## Real Behavior Proof Validated with targeted Go tests: ```bash /tmp/go1.25.0/bin/go test ./internal/handler ./internal/router ``` Result: ```text ok ragflow/internal/handler ? ragflow/internal/router [no test files] ``` Validated server entrypoint build: ```bash /tmp/go1.25.0/bin/go build -o /tmp/ragflow-server-main ./cmd/server_main.go ``` Result: ```text build succeeded ``` Validated patch formatting: ```bash git diff --check ``` Result: ```text no whitespace errors ``` ## Checklist - [x] Implemented only `DELETE /api/v1/messages/{memory_id}:{message_id}`. - [x] Did not implement unrelated Memory message APIs. - [x] Added route registration. - [x] Added handler validation. - [x] Added service-level memory access check. - [x] Added tests. - [x] Ran targeted Go tests. - [x] Ran server build validation. - [x] Ran `git diff --check`.
2026-06-10 20:27:35 +07:00
package handler
import (
"encoding/json"
"fmt"
"net/http"
"net/http/httptest"
"testing"
"github.com/gin-gonic/gin"
"ragflow/internal/common"
"ragflow/internal/entity"
"ragflow/internal/service"
)
func TestIsMemoryServiceNotFound(t *testing.T) {
notFoundErr := &service.ResourceNotFoundError{Resource: "Memory", ID: "memory-1"}
if !isMemoryServiceNotFound(fmt.Errorf("wrapped: %w", notFoundErr)) {
t.Fatal("expected wrapped service not found error to map to not found")
}
messageNotFoundErr := &service.ResourceNotFoundError{Resource: "Message", ID: "message-1"}
if isMemoryServiceNotFound(messageNotFoundErr) {
t.Fatal("expected non-memory resource not found error to avoid memory 404 mapping")
}
if isMemoryServiceNotFound(fmt.Errorf("backend index does not exist")) {
t.Fatal("backend text should not map to not found without service error type")
}
}
func TestParseMemoryMessagePath(t *testing.T) {
tests := []struct {
name string
value string
wantMemoryID string
wantMessageID int64
wantErr bool
}{
{name: "valid", value: "memory-1:42", wantMemoryID: "memory-1", wantMessageID: 42},
{name: "empty", value: "", wantErr: true},
{name: "missing message id", value: "memory-1:", wantErr: true},
{name: "missing memory id", value: ":42", wantErr: true},
{name: "invalid message id", value: "memory-1:not-int", wantErr: true},
{name: "negative message id", value: "memory-1:-1", wantErr: true},
{name: "too many separators", value: "memory-1:2:3", wantErr: true},
}
for _, tt := range tests {
t.Run(tt.name, func(t *testing.T) {
memoryID, messageID, err := parseMemoryMessagePath(tt.value)
if tt.wantErr {
if err == nil {
t.Fatalf("expected error")
}
return
}
if err != nil {
t.Fatalf("unexpected error: %v", err)
}
if memoryID != tt.wantMemoryID || messageID != tt.wantMessageID {
t.Fatalf("got (%q, %d), want (%q, %d)", memoryID, messageID, tt.wantMemoryID, tt.wantMessageID)
}
})
}
}
func TestForgetMessageRejectsMalformedPath(t *testing.T) {
gin.SetMode(gin.TestMode)
router := gin.New()
h := NewMemoryHandler(service.NewMemoryService())
router.DELETE("/api/v1/messages/:memory_message", func(c *gin.Context) {
c.Set("user", &entity.User{ID: "user-1"})
h.ForgetMessage(c)
})
req := httptest.NewRequest(http.MethodDelete, "/api/v1/messages/memory-1:not-int", nil)
w := httptest.NewRecorder()
router.ServeHTTP(w, req)
if w.Code != http.StatusOK {
t.Fatalf("status = %d, want %d", w.Code, http.StatusOK)
}
var res map[string]interface{}
if err := json.Unmarshal(w.Body.Bytes(), &res); err != nil {
t.Fatalf("decode response: %v", err)
}
if code := common.ErrorCode(res["code"].(float64)); code != common.CodeArgumentError {
t.Fatalf("code = %v, want %v; body=%s", code, common.CodeArgumentError, w.Body.String())
}
}