[FEATURE] Suppress diff output in quiet mode

Author: andreaswolf

e2e/run-test.sh

@@ -18,7 +18,7 @@ cd $TEST_DIR
 cp -r fixtures/ output/
 
 cd $TESTS_BASE_DIR
-./vendor/bin/fractor process -c $TESTS_BASE_DIR/$TEST_DIR/fractor.php
+./vendor/bin/fractor process --quiet -c $TESTS_BASE_DIR/$TEST_DIR/fractor.php
 
 # TODO remove -b once we keep the output format when re-writing the file
 diff -rub $TEST_DIR/expected-output/ $TEST_DIR/output/

fractor/src/Application/FractorRunner.php

@@ -39,20 +39,23 @@ public function run(Output $output, Configuration $configuration): void
     {
         $filePaths = $this->fileFinder->findFiles($configuration->getPaths(), $configuration->getFileExtensions());
 
-        $output->progressStart(count($filePaths));
+        if (! $configuration->isQuiet()) {
+            $output->progressStart(count($filePaths));
+        }
 
         foreach ($filePaths as $filePath) {
             $file = new File($filePath, FileSystem::read($filePath));
             $this->fileCollector->addFile($file);
 
             foreach ($this->processors as $processor) {
-                if (! $processor->canHandle($file)) {
+                if (! $configuration->isQuiet()) {
                     $output->progressAdvance();
+                }
+                if (! $processor->canHandle($file)) {
                     continue;
                 }
 
                 $processor->handle($file);
-                $output->progressAdvance();
             }
 
             if (! $file->hasChanged()) {
@@ -62,21 +65,25 @@ public function run(Output $output, Configuration $configuration): void
             $file->setFileDiff($this->fileDiffFactory->createFileDiff($file));
         }
 
-        $output->progressFinish();
+        if (! $configuration->isQuiet()) {
+            $output->progressFinish();
+        }
 
         foreach ($this->fileCollector->getFiles() as $file) {
             if ($file->getFileDiff() === null) {
                 continue;
             }
 
-            $output->write($file->getFileDiff()->getDiffConsoleFormatted());
-            if ($file->getAppliedRules() !== []) {
-                $fractorsChangelogsLines = $this->fractorsChangelogLinesResolver->createFractorChangelogLines(
-                    $file->getAppliedRules()
-                );
-                $output->write('<options=underscore>Applied rules:</>');
-                $output->listing($fractorsChangelogsLines);
-                $output->newLine();
+            if (! $configuration->isQuiet()) {
+                $output->write($file->getFileDiff()->getDiffConsoleFormatted());
+                if ($file->getAppliedRules() !== []) {
+                    $fractorsChangelogsLines = $this->fractorsChangelogLinesResolver->createFractorChangelogLines(
+                        $file->getAppliedRules()
+                    );
+                    $output->write('<options=underscore>Applied rules:</>');
+                    $output->listing($fractorsChangelogsLines);
+                    $output->newLine();
+                }
             }
 
             if ($configuration->isDryRun()) {

fractor/src/Configuration/ConfigurationFactory.php

@@ -23,7 +23,8 @@ public function createFromInput(InputInterface $input): Configuration
             $this->allowedFileExtensionsResolver->resolve(),
             (array) $this->parameterBag->get(Option::PATHS),
             (array) $this->parameterBag->get(Option::SKIP),
-            (bool) $input->getOption(Option::DRY_RUN)
+            (bool) $input->getOption(Option::DRY_RUN),
+            (bool) $input->getOption(Option::QUIET)
         );
     }
 

fractor/src/Configuration/Option.php

@@ -21,6 +21,16 @@ final class Option
      */
     public const DRY_RUN = 'dry-run';
 
+    /**
+     * @var string
+     */
+    public const QUIET = 'quiet';
+
+    /**
+     * @var string
+     */
+    public const QUIET_SHORT = 'q';
+
     /**
      * @var string
      */

fractor/src/Configuration/ValueObject/Configuration.php

@@ -17,7 +17,8 @@ public function __construct(
         private array $fileExtensions,
         private array $paths,
         private array $skip,
-        private bool $dryRun
+        private bool $dryRun,
+        private bool $quiet
     ) {
         Assert::allStringNotEmpty($this->paths, 'No directories given');
     }
@@ -50,4 +51,9 @@ public function isDryRun(): bool
     {
         return $this->dryRun;
     }
+
+    public function isQuiet(): bool
+    {
+        return $this->quiet;
+    }
 }

fractor/src/Console/Command/ProcessCommand.php

@@ -39,6 +39,12 @@ protected function configure(): void
             InputOption::VALUE_NONE,
             'Only see the diff of changes, do not save them to files.'
         );
+        $this->addOption(
+            Option::QUIET,
+            Option::QUIET_SHORT,
+            InputOption::VALUE_NONE,
+            'Do not output diff of changes.'
+        );
     }
 
     protected function execute(InputInterface $input, OutputInterface $output): int