opencode/yao
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Improve test reliability in TestSubscribeFrom by adjusting sleep durations and timestamp calculations. Increased sleep time to 1100ms for CI environment stability and modified timestamp capture to account for millisecond precision, ensuring accurate event timing in tests.

Browse source
This commit is contained in:
Max 2025-12-11 17:50:00 +08:00
parent eebaf27f1d
commit edf921324b
1 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
trace/trace_subscription_test.go
View file

@ -121,13 +121,17 @@ func TestSubscribeFrom(t *testing.T) {
assert.NoError(t, err)
// Wait to ensure different timestamp (simulate time passing)
// Use a longer sleep to account for CI environment variability
time.Sleep(1100 * time.Millisecond)
// Record timestamp (simulate user noting current time before refresh)
resumeTimestamp := time.Now().UnixMilli()
// Subtract 1ms to ensure we capture events that happen "now"
// This accounts for millisecond precision and timing variability in CI
resumeTimestamp := time.Now().UnixMilli() - 1
// Wait again to ensure next operations are after resumeTimestamp
time.Sleep(100 * time.Millisecond)
// Wait to ensure next operations have a clearly different timestamp
// Use longer sleep for CI reliability
time.Sleep(500 * time.Millisecond)
// Continue with more operations
_, err = manager.Add("Step 2", types.TraceNodeOption{Label: "Finalizing"})