Improve display of progress bar fraction completed (#204)

Author: jessarcher

src/Themes/Default/ProgressRenderer.php

@@ -27,31 +27,31 @@ public function __invoke(Progress $progress): string
                 ->box(
                     $this->dim($this->truncate($progress->label, $progress->terminal()->cols() - 6)),
                     $this->dim($filled),
-                    info: $progress->progress.'/'.$progress->total,
+                    info: $this->fractionCompleted($progress),
                 ),
 
             'error' => $this
                 ->box(
                     $this->truncate($progress->label, $progress->terminal()->cols() - 6),
                     $this->dim($filled),
                     color: 'red',
-                    info: $progress->progress.'/'.$progress->total,
+                    info: $this->fractionCompleted($progress),
                 ),
 
             'cancel' => $this
                 ->box(
                     $this->truncate($progress->label, $progress->terminal()->cols() - 6),
                     $this->dim($filled),
                     color: 'red',
-                    info: $progress->progress.'/'.$progress->total,
+                    info: $this->fractionCompleted($progress),
                 )
                 ->error($progress->cancelMessage),
 
             default => $this
                 ->box(
                     $this->cyan($this->truncate($progress->label, $progress->terminal()->cols() - 6)),
                     $this->dim($filled),
-                    info: $progress->progress.'/'.$progress->total,
+                    info: $this->fractionCompleted($progress),
                 )
                 ->when(
                     $progress->hint,
@@ -60,4 +60,12 @@ public function __invoke(Progress $progress): string
                 )
         };
     }
+
+    /**
+     * @param  Progress<int|iterable<mixed>>  $progress
+     */
+    protected function fractionCompleted(Progress $progress): string
+    {
+        return number_format($progress->progress).' / '.number_format($progress->total);
+    }
 }

tests/Feature/ProgressTest.php

@@ -16,28 +16,28 @@
     Prompt::assertStrippedOutputContains(<<<'OUTPUT'
      ┌ Adding States ───────────────────────────────────────────────┐
      │                                                              │
-     └───────────────────────────────────────────────────────── 0/4 ┘
+     └─────────────────────────────────────────────────────── 0 / 4 ┘
     OUTPUT);
 
     Prompt::assertStrippedOutputContains(<<<'OUTPUT'
      │ ███████████████                                              │
-     └───────────────────────────────────────────────────────── 1/4 ┘
+     └─────────────────────────────────────────────────────── 1 / 4 ┘
     OUTPUT);
 
     Prompt::assertStrippedOutputContains(<<<'OUTPUT'
      │ ██████████████████████████████                               │
-     └───────────────────────────────────────────────────────── 2/4 ┘
+     └─────────────────────────────────────────────────────── 2 / 4 ┘
     OUTPUT);
 
     Prompt::assertStrippedOutputContains(<<<'OUTPUT'
      │ █████████████████████████████████████████████                │
-     └───────────────────────────────────────────────────────── 3/4 ┘
+     └─────────────────────────────────────────────────────── 3 / 4 ┘
     OUTPUT);
 
     Prompt::assertStrippedOutputContains(<<<'OUTPUT'
      ┌ Adding States ───────────────────────────────────────────────┐
      │ ████████████████████████████████████████████████████████████ │
-     └───────────────────────────────────────────────────────── 4/4 ┘
+     └─────────────────────────────────────────────────────── 4 / 4 ┘
     OUTPUT);
 })->with([
     'array' => [['Alabama', 'Alaska', 'Arizona', 'Arkansas']],
@@ -60,7 +60,7 @@
     Prompt::assertStrippedOutputContains(<<<'OUTPUT'
      ┌──────────────────────────────────────────────────────────────┐
      │                                                              │
-     └───────────────────────────────────────────────────────── 0/6 ┘
+     └─────────────────────────────────────────────────────── 0 / 6 ┘
     OUTPUT);
 });