jedib0t / go-pretty / 26110093750

100%

main: 100%

Pull #406

github

progress: fix speed decay on done trackers and log overwrite; fixes #405

1. Per-tracker speed for a done tracker decayed toward zero on every
subsequent render tick. Done trackers are now repainted each cycle
(previously they were painted once at the active->done transition),
but renderTrackerStatsSpeed still used time.Since(timeStart) as the
denominator. Use timeStop-timeStart for done trackers, matching
renderTrackerStatsTime.

2. Log messages were overwritten by a stale tracker render on the next
tick. Logs were emitted between the tracker block and pinned messages
but were not counted by moveCursorToTheTop, leaving the cursor rewind
short by len(logs) lines so trackers re-painted on top of the log
row. Emit logs above the rewindable region instead, so they sit as
permanent scrollback (the pre-v6.7.8 layout).

Also consolidates timeStart/timeStop/done access into a single
timeStartStopAndDone() helper so callers see a consistent snapshot — the
previous two-call pattern could observe timeStop=zero with done=true
during an active->done transition, producing a negative duration.

92 of 92 new or added lines in 2 files covered. (100.0%)

4889 of 4889 relevant lines covered (100.0%)

1.21 hits per line