From b18499a3c19862e642fb72d120011bce01403134 Mon Sep 17 00:00:00 2001 From: Colin Walters Date: Fri, 20 Sep 2024 09:02:42 -0400 Subject: [PATCH] copy: Don't print "skipped: 0.0b = 0.00%" I think this is just visually noisy. If we didn't do a partial fetch then let's just not say anything about it. Signed-off-by: Colin Walters --- copy/progress_bars.go | 13 +++++++++---- copy/progress_bars_test.go | 2 +- 2 files changed, 10 insertions(+), 5 deletions(-) diff --git a/copy/progress_bars.go b/copy/progress_bars.go index 08128ce8d..59f41d216 100644 --- a/copy/progress_bars.go +++ b/copy/progress_bars.go @@ -24,13 +24,18 @@ func (c *copier) newProgressPool() *mpb.Progress { // customPartialBlobDecorFunc implements mpb.DecorFunc for the partial blobs retrieval progress bar func customPartialBlobDecorFunc(s decor.Statistics) string { + current := decor.SizeB1024(s.Current) + total := decor.SizeB1024(s.Total) + refill := decor.SizeB1024(s.Refill) if s.Total == 0 { - pairFmt := "%.1f / %.1f (skipped: %.1f)" - return fmt.Sprintf(pairFmt, decor.SizeB1024(s.Current), decor.SizeB1024(s.Total), decor.SizeB1024(s.Refill)) + return fmt.Sprintf("%.1f / %.1f (skipped: %.1f)", current, total, refill) + } + // If we didn't do a partial fetch then let's not output a distracting ("skipped: 0.0b = 0.00%") + if s.Refill == 0 { + return fmt.Sprintf("%.1f / %.1f", current, total) } - pairFmt := "%.1f / %.1f (skipped: %.1f = %.2f%%)" percentage := 100.0 * float64(s.Refill) / float64(s.Total) - return fmt.Sprintf(pairFmt, decor.SizeB1024(s.Current), decor.SizeB1024(s.Total), decor.SizeB1024(s.Refill), percentage) + return fmt.Sprintf("%.1f / %.1f (skipped: %.1f = %.2f%%)", current, total, refill, percentage) } // progressBar wraps a *mpb.Bar, allowing us to add extra state and methods. diff --git a/copy/progress_bars_test.go b/copy/progress_bars_test.go index dc6138315..0c863cacc 100644 --- a/copy/progress_bars_test.go +++ b/copy/progress_bars_test.go @@ -20,5 +20,5 @@ func TestCustomPartialBlobDecorFunc(t *testing.T) { // Almost complete, but no reuse s.Current = int64(float64(s.Total) * 0.95) s.Refill = 0 - assert.Equal(t, "7.5MiB / 7.9MiB (skipped: 0.0b = 0.00%)", customPartialBlobDecorFunc(s)) + assert.Equal(t, "7.5MiB / 7.9MiB", customPartialBlobDecorFunc(s)) }