From 406c0232c12085e1e2274017117612838b54b1db Mon Sep 17 00:00:00 2001 From: Mathias Date: Fri, 2 Oct 2026 17:19:32 +0200 Subject: [PATCH] test: a shortfall is noted on the audio step, not refused; the 80%, two-minute and 30-second tolerances are pinned (red; second review finding 1) Co-Authored-By: Claude Sonnet 5.5 --- internal/progress/tracker_test.go | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/internal/progress/tracker_test.go b/internal/progress/tracker_test.go index 9a0d0e1..52b00ee 100644 --- a/internal/progress/tracker_test.go +++ b/internal/progress/tracker_test.go @@ -387,3 +387,16 @@ func TestWithAndFrom_ACarriedReporterIsTheOneReturned(t *testing.T) { require.Equal(t, domain.StepRunning, stateOf(sink.last(t), "audio")) } + +func TestTracker_AnnotateAttachesANoteToAStepAndSavesItAtOnce(t *testing.T) { + tr, sink, _, _ := newTracker(t) + tr.Start() + tr.Begin("audio", 0) + before := sink.saveCount() + + tr.Annotate("audio", "20 min of the 79 the file claimed") + + require.Equal(t, before+1, sink.saveCount()) + require.Equal(t, "20 min of the 79 the file claimed", sink.last(t)[0].Note) + require.NotPanics(t, func() { tr.Annotate("no-such-step", "x") }) +}