Files
Enterprise-Knowledge-Base/internal/handler/knowledge_spans_test.go
T
wizardchen 5ffc755e76 fix(knowledge): infer synthesized stage status from parse_status
Pre-tracker historical knowledge has zero rows in
knowledge_processing_spans but parse_status correctly reads
"completed" or "failed". The /spans handler was synthesizing five
"pending" placeholders unconditionally, so legacy completed documents
rendered as if they were stuck waiting in the queue forever.

buildSpanTree now takes parse_status and chooses the placeholder
status accordingly:
- ParseStatusCompleted -> done
- ParseStatusFailed    -> failed
- everything else      -> pending (existing behaviour)

Real rows always take precedence; this only changes what we put in
the gaps. So healthy in-flight parses (parse_status=processing,
some real rows, some still pending) keep showing pending placeholders
exactly as before — the synthesized "completed" inference only fires
when the parse already hit terminal state.

Adds TestBuildSpanTree_LegacyCompletedRendersAsDone covering both
the completed-legacy and failed-legacy branches.
2026-05-28 15:14:45 +08:00

125 lines
5.8 KiB
Go

package handler
import (
"testing"
"time"
"github.com/Tencent/WeKnora/internal/types"
"github.com/stretchr/testify/assert"
)
// TestBuildSpanTree_AssemblesParentChild covers the basic shape: a root
// with stage children and an image generation grandchild. The handler's
// JSON contract depends on this tree topology; if the linking logic
// regresses, the UI sees a flat list and renders nothing.
func TestBuildSpanTree_AssemblesParentChild(t *testing.T) {
now := time.Now()
rows := []types.KnowledgeProcessingSpan{
{KnowledgeID: "kid", Attempt: 1, SpanID: "root", Name: "knowledge_processing", Kind: types.SpanKindRoot, Status: types.SpanStatusRunning, StartedAt: &now},
{KnowledgeID: "kid", Attempt: 1, SpanID: "doc", ParentSpanID: "root", Name: types.StageDocReader, Kind: types.SpanKindStage, Status: types.SpanStatusDone, StartedAt: &now},
{KnowledgeID: "kid", Attempt: 1, SpanID: "mm", ParentSpanID: "root", Name: types.StageMultimodal, Kind: types.SpanKindStage, Status: types.SpanStatusRunning, StartedAt: &now},
{KnowledgeID: "kid", Attempt: 1, SpanID: "img0", ParentSpanID: "mm", Name: "multimodal.image[0]", Kind: types.SpanKindGeneration, Status: types.SpanStatusRunning, StartedAt: &now},
}
tree, currentStage, lastFail := buildSpanTree("kid", 1, rows, types.ParseStatusProcessing)
require := assert.New(t)
require.NotNil(tree)
require.Equal("root", tree.SpanID)
require.Equal(types.StageMultimodal, currentStage,
"running stage span surfaces as current_stage")
require.Nil(lastFail)
// The 5 canonical stages must all appear under root (real or
// synthesized placeholder).
stageNames := map[string]string{}
for _, child := range tree.Children {
if child.Kind == types.SpanKindStage {
stageNames[child.Name] = child.Status
}
}
for _, name := range types.AllStages {
_, ok := stageNames[name]
require.True(ok, "stage %s must appear in tree", name)
}
require.Equal(types.SpanStatusDone, stageNames[types.StageDocReader])
require.Equal(types.SpanStatusRunning, stageNames[types.StageMultimodal])
require.Equal(types.SpanStatusPending, stageNames[types.StageEmbedding],
"missing stage rows must synthesize as pending placeholders")
// The image subspan must hang under multimodal, not at root level.
var mmNode *types.SpanTreeNode
for _, c := range tree.Children {
if c.Name == types.StageMultimodal {
mmNode = c
break
}
}
require.NotNil(mmNode)
require.Len(mmNode.Children, 1, "image generation subspan must be a child of multimodal stage")
require.Equal("multimodal.image[0]", mmNode.Children[0].Name)
}
// TestBuildSpanTree_NoRows_SynthesizesPlaceholderRoot ensures the API
// keeps a stable shape for fresh / never-parsed knowledge: the frontend
// always gets a `trace` with five pending stage children, never a
// nil/empty response.
func TestBuildSpanTree_NoRows_SynthesizesPlaceholderRoot(t *testing.T) {
tree, currentStage, lastFail := buildSpanTree("kid-empty", 0, nil, "")
a := assert.New(t)
a.NotNil(tree)
a.Equal(types.SpanKindRoot, tree.Kind)
a.Equal(types.SpanStatusPending, tree.Status)
a.Equal("", currentStage, "no rows means no running stage")
a.Nil(lastFail)
// All 5 stages must be present as pending placeholders so the UI
// renders a complete timeline even pre-parse.
a.Len(tree.Children, len(types.AllStages))
for _, child := range tree.Children {
a.Equal(types.SpanStatusPending, child.Status)
a.Equal(types.SpanKindStage, child.Kind)
}
}
// TestBuildSpanTree_LegacyCompletedRendersAsDone records the regression
// for historical knowledge parsed before span tracking was wired: rows
// is empty but parse_status is "completed", so the synthesized timeline
// must reflect the actual terminal state instead of looking forever
// "pending". Same contract for failed parses — synthesizes "failed".
func TestBuildSpanTree_LegacyCompletedRendersAsDone(t *testing.T) {
a := assert.New(t)
completedTree, _, _ := buildSpanTree("kid-legacy", 0, nil, types.ParseStatusCompleted)
a.Equal(types.SpanStatusDone, completedTree.Status,
"legacy completed knowledge with no rows must render the synthesized root as done")
a.Len(completedTree.Children, len(types.AllStages))
for _, child := range completedTree.Children {
a.Equal(types.SpanStatusDone, child.Status,
"legacy completed knowledge: every synthesized stage placeholder must be done, not pending")
}
failedTree, _, _ := buildSpanTree("kid-legacy-fail", 0, nil, types.ParseStatusFailed)
a.Equal(types.SpanStatusFailed, failedTree.Status)
for _, child := range failedTree.Children {
a.Equal(types.SpanStatusFailed, child.Status)
}
}
// TestBuildSpanTree_LastFailureSurfaces records that a failed stage is
// reported as last_error so the UI can highlight the responsible step
// even if a later stage was cancelled by cascade.
func TestBuildSpanTree_LastFailureSurfaces(t *testing.T) {
now := time.Now()
finished := now.Add(5 * time.Second)
rows := []types.KnowledgeProcessingSpan{
{KnowledgeID: "kid", Attempt: 1, SpanID: "root", Name: "knowledge_processing", Kind: types.SpanKindRoot, Status: types.SpanStatusFailed, StartedAt: &now, FinishedAt: &finished},
{KnowledgeID: "kid", Attempt: 1, SpanID: "doc", ParentSpanID: "root", Name: types.StageDocReader, Kind: types.SpanKindStage, Status: types.SpanStatusFailed, ErrorCode: "DOCREADER_TIMEOUT", ErrorMessage: "slow", StartedAt: &now, FinishedAt: &finished},
{KnowledgeID: "kid", Attempt: 1, SpanID: "chunk", ParentSpanID: "root", Name: types.StageChunking, Kind: types.SpanKindStage, Status: types.SpanStatusCancelled, ErrorCode: "UPSTREAM_FAILED"},
}
_, _, lastFail := buildSpanTree("kid", 1, rows, types.ParseStatusFailed)
a := assert.New(t)
a.NotNil(lastFail)
a.Equal(types.StageDocReader, lastFail.Name,
"last_error must point at the actually-failed span, not the cascade-cancelled downstream")
a.Equal("DOCREADER_TIMEOUT", lastFail.ErrorCode)
}