Merge pull request #43 from markwalet/make-command-output-consistent

Make command output consistent

Author: markwalet

CHANGELOG.md

@@ -7,6 +7,9 @@
 - Add a `changelog:install` artisan command.
 - Add a `changelog:current` artisan command.
 
+## Fixed
+- Made all command output show in a consistent manner.
+
 ## [v1.6.1 (2022-03-27)](https://github.com/markwalet/laravel-changelog/compare/v1.6.0...v1.6.1)
 
 ### Fixed

src/Commands/ChangelogCurrentCommand.php

@@ -38,6 +38,7 @@ public function handle(FeatureAdapter $adapter, GitDriver $gitState)
 
             return;
         }
+
         $feature = $adapter->read($path);
         $this->line("Changes for $branch:");
 

src/Commands/ChangelogGenerateCommand.php

@@ -49,7 +49,7 @@ public function handle(ReleaseAdapter $adapter, ChangelogFormatterFactory $facto
         }
 
         file_put_contents($writePath, $content);
-        $this->info('The changelog is written to: '.$writePath);
+        $this->info("The changelog is written to: $writePath");
     }
 
     /**

src/Commands/ChangelogListCommand.php

@@ -36,9 +36,11 @@ public function handle(ReleaseAdapter $adapter, ChangelogFormatterFactory $facto
 
         $releases = $this->releases($adapter, $path);
 
-        $formatted = $formatter->multiple($releases);
+        $lines = explode(PHP_EOL, $formatter->multiple($releases));
 
-        $this->line($formatted);
+        foreach ($lines as $line) {
+            $this->line($line);
+        }
     }
 
     /**

src/Commands/ChangelogUnreleasedCommand.php

@@ -36,8 +36,11 @@ public function handle(ReleaseAdapter $adapter, ChangelogFormatterFactory $facto
 
         $release = $this->release($adapter, $path);
 
-        $formatted = $formatter->single($release);
-        $this->line($formatted);
+        $lines = explode(PHP_EOL, $formatter->single($release));
+
+        foreach ($lines as $line) {
+            $this->line($line);
+        }
     }
 
     /**

tests/Commands/ChangelogListCommandTest.php

@@ -46,11 +46,11 @@ public function it_can_list_all_changes_for_all_releases()
         $this->fakeAdapter();
 
         $this->artisan('changelog:list')
-            ->expectsOutput('Unreleased'
-                .PHP_EOL.'  - Added: Added a new feature.'
-                .PHP_EOL.'V1.0.1'
-                .PHP_EOL.'  - Added: Added a third feature.'
-                .PHP_EOL.'  - Added: Added helper commands.'
-                .PHP_EOL.'  - Removed: Removed unused trait.');
+            ->expectsOutput('Unreleased')
+            ->expectsOutput('  - Added: Added a new feature.')
+            ->expectsOutput('V1.0.1')
+            ->expectsOutput('  - Added: Added a third feature.')
+            ->expectsOutput('  - Added: Added helper commands.')
+            ->expectsOutput('  - Removed: Removed unused trait.');
     }
 }

tests/Commands/ChangelogUnreleasedCommandTest.php

@@ -46,8 +46,8 @@ public function it_can_list_all_unreleased_changes()
         $this->fakeAdapter();
 
         $this->artisan('changelog:unreleased')
-            ->expectsOutput('Unreleased'
-                .PHP_EOL.'  - Added: Added a new feature.'
-                .PHP_EOL.'  - Removed: Removed unused trait.');
+            ->expectsOutput('Unreleased')
+            ->expectsOutput('  - Added: Added a new feature.')
+            ->expectsOutput('  - Removed: Removed unused trait.');
     }
 }