releases/ClaudeDo
2
0
Fork 0
Code Issues 1 Pull Requests 2 Actions Packages Projects Releases 24 Wiki Activity

fix(worker-tests): pin LogRingBuffer clock in Does_not_throw_when_detached

Browse Source 
The test built the buffer with the real DateTime.UtcNow clock but emitted
an event with the file's fixed EvtTime timestamp (2026-06-23). As real
time drifted more than the 1h window past that fixed timestamp, the
just-appended record was evicted inside the same Append call, before
Snapshot() ran. BroadcastLogSink.Emit itself buffers unconditionally
regardless of attach state, so this was a test bug, not a sink bug. Pin
the buffer's clock to EvtTime, matching every sibling test's NewSink setup.
This commit is contained in:
mika kuns
2026-07-29 12:10:58 +02:00
parent db447f36da
commit 58f8b11dbb
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/ClaudeDo.Worker.Tests/Logging/BroadcastLogSinkTests.cs
+1 -1
View File
@@ -98,7 +98,7 @@ public class BroadcastLogSinkTests
[Fact] [Fact]
public void Does_not_throw_when_detached() public void Does_not_throw_when_detached()
{ {
var buffer = new LogRingBuffer(TimeSpan.FromHours(1)); var buffer = new LogRingBuffer(TimeSpan.FromHours(1), utcNow: () => EvtTime.UtcDateTime);
var sink = new BroadcastLogSink(buffer); var sink = new BroadcastLogSink(buffer);
sink.Emit(Evt(LogEventLevel.Error, "no subscriber")); sink.Emit(Evt(LogEventLevel.Error, "no subscriber"));