minor: displaying number of checked files

docker-compose.override.yaml.dist

@@ -10,7 +10,7 @@ services:
     extra_hosts:
       # Required for Docker Linux until natively supported.
       # See https://github.com/docker/for-linux/issues/264
-      host.docker.internal: 172.17.0.1
+      - 'host.docker.internal: 172.17.0.1'
   php-8.0:
     <<: *php
   php-8.1:

src/Console/Command/FixCommand.php

@@ -183,7 +183,7 @@ public function getHelp(): string
 * 64 - Exception raised within the application.
 
 EOF
-        ;
+            ;
     }
 
     /**
@@ -315,6 +315,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
         $reportSummary = new ReportSummary(
             $changed,
+            count($finder),
             $fixEvent->getDuration(),
             $fixEvent->getMemory(),
             OutputInterface::VERBOSITY_VERBOSE <= $verbosity,

src/Console/Report/FixReport/ReportSummary.php

@@ -26,6 +26,8 @@ final class ReportSummary
      */
     private array $changed;
 
+    private int $files;
+
     private int $time;
 
     private int $memory;
@@ -43,13 +45,15 @@ final class ReportSummary
      */
     public function __construct(
         array $changed,
+        int $files,
         int $time,
         int $memory,
         bool $addAppliedFixers,
         bool $isDryRun,
         bool $isDecoratedOutput
     ) {
         $this->changed = $changed;
+        $this->files = $files;
         $this->time = $time;
         $this->memory = $memory;
         $this->addAppliedFixers = $addAppliedFixers;
@@ -85,6 +89,11 @@ public function getTime(): int
         return $this->time;
     }
 
+    public function getFiles(): int
+    {
+        return $this->files;
+    }
+
     public function shouldAddAppliedFixers(): bool
     {
         return $this->addAppliedFixers;

src/Console/Report/FixReport/TextReporter.php

@@ -54,7 +54,7 @@ public function generate(ReportSummary $reportSummary): string
             $output .= PHP_EOL;
         }
 
-        return $output.$this->getFooter($reportSummary->getTime(), $reportSummary->getMemory(), $reportSummary->isDryRun());
+        return $output.$this->getFooter($reportSummary->getTime(), count($reportSummary->getChanged()), $reportSummary->getFiles(), $reportSummary->getMemory(), $reportSummary->isDryRun());
     }
 
     /**
@@ -83,17 +83,18 @@ private function getDiff(bool $isDecoratedOutput, string $diff): string
         return PHP_EOL.$diffFormatter->format($diff).PHP_EOL;
     }
 
-    private function getFooter(int $time, int $memory, bool $isDryRun): string
+    private function getFooter(int $time, int $changed, int $files, int $memory, bool $isDryRun): string
     {
         if (0 === $time || 0 === $memory) {
             return '';
         }
 
         return PHP_EOL.sprintf(
-            '%s all files in %.3f seconds, %.3f MB memory used'.PHP_EOL,
-            $isDryRun ? 'Checked' : 'Fixed',
-            $time / 1000,
-            $memory / 1024 / 1024
-        );
+                '%s %d files in %.3f seconds, %.3f MB memory used'.PHP_EOL,
+                $isDryRun ? 'Checked' : sprintf('Fixed %d of', $changed),
+                $files,
+                $time / 1000,
+                $memory / 1024 / 1024
+            );
     }
 }

tests/Console/Report/FixReport/AbstractReporterTestCase.php

@@ -69,6 +69,7 @@ final public function provideGenerateCases(): array
                 $this->createNoErrorReport(),
                 new ReportSummary(
                     [],
+                    10,
                     0,
                     0,
                     false,
@@ -85,6 +86,7 @@ final public function provideGenerateCases(): array
                             'diff' => '',
                         ],
                     ],
+                    10,
                     0,
                     0,
                     false,
@@ -101,6 +103,7 @@ final public function provideGenerateCases(): array
                             'diff' => 'this text is a diff ;)',
                         ],
                     ],
+                    10,
                     0,
                     0,
                     false,
@@ -117,6 +120,7 @@ final public function provideGenerateCases(): array
                             'diff' => '',
                         ],
                     ],
+                    10,
                     0,
                     0,
                     true,
@@ -133,6 +137,7 @@ final public function provideGenerateCases(): array
                             'diff' => '',
                         ],
                     ],
+                    10,
                     1234,
                     2621440, // 2.5 * 1024 * 1024
                     false,
@@ -153,6 +158,7 @@ final public function provideGenerateCases(): array
                             'diff' => 'another diff here ;)',
                         ],
                     ],
+                    10,
                     1234,
                     2621440, // 2.5 * 1024 * 1024
                     true,

tests/Console/Report/FixReport/ReportSummaryTest.php

@@ -32,6 +32,7 @@ public function testReportSummary(): void
                 'diff' => 'this text is a diff ;)',
             ],
         ];
+        $files = 10;
         $time = time();
         $memory = 123456789;
         $addAppliedFixers = true;
@@ -40,6 +41,7 @@ public function testReportSummary(): void
 
         $reportSummary = new ReportSummary(
             $changed,
+            $files,
             $time,
             $memory,
             $addAppliedFixers,
@@ -48,6 +50,7 @@ public function testReportSummary(): void
         );
 
         static::assertSame($changed, $reportSummary->getChanged());
+        static::assertSame($files, $reportSummary->getFiles());
         static::assertSame($time, $reportSummary->getTime());
         static::assertSame($memory, $reportSummary->getMemory());
         static::assertSame($addAppliedFixers, $reportSummary->shouldAddAppliedFixers());

tests/Console/Report/FixReport/TextReporterTest.php

@@ -81,7 +81,7 @@ protected function createWithTimeAndMemoryReport(): string
             <<<'TEXT'
    1) someFile.php
 
-Fixed all files in 1.234 seconds, 2.500 MB memory used
+Fixed 1 of 10 files in 1.234 seconds, 2.500 MB memory used
 
 TEXT
         );
@@ -104,7 +104,7 @@ protected function createComplexReport(): string
 <comment>      ----------- end diff -----------</comment>
 
 
-Checked all files in 1.234 seconds, 2.500 MB memory used
+Checked 10 files in 1.234 seconds, 2.500 MB memory used
 
 TEXT
         );

tests/Smoke/CiIntegrationTest.php

@@ -194,7 +194,7 @@ public function testIntegration(
         static::assertSame(substr_count($expectedResult3FilesDots, 'S'), substr_count($matches[1], 'S'));
 
         static::assertMatchesRegularExpression(
-            '/^\s*Checked all files in \d+\.\d+ seconds, \d+\.\d+ MB memory used\s*$/',
+            '/^\s*Checked \d+ files in \d+\.\d+ seconds, \d+\.\d+ MB memory used\s*$/',
             $result3->getOutput()
         );
     }

tests/Smoke/StdinTest.php

@@ -75,7 +75,7 @@ public function testFixingStdin(): void
     private function unifyFooter(string $output): string
     {
         return preg_replace(
-            '/Checked all files in \d+\.\d+ seconds, \d+\.\d+ MB memory used/',
+            '/Checked \d+ files in \d+\.\d+ seconds, \d+\.\d+ MB memory used/',
             'Footer',
             $output
         );