From 87646ae6897ff7f72c5e3b2776ce5d7b54626722 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Th=C3=A9o=20FIDRY?= Date: Sat, 9 Jun 2018 12:23:41 +0100 Subject: [PATCH] Hide suggest reason when there is not one --- .../Installer/SuggestedPackagesReporter.php | 4 ++-- .../Installer/SuggestedPackagesReporterTest.php | 14 ++++++++++++++ 2 files changed, 16 insertions(+), 2 deletions(-) diff --git a/src/Composer/Installer/SuggestedPackagesReporter.php b/src/Composer/Installer/SuggestedPackagesReporter.php index ed89ec02b..25788e547 100644 --- a/src/Composer/Installer/SuggestedPackagesReporter.php +++ b/src/Composer/Installer/SuggestedPackagesReporter.php @@ -115,10 +115,10 @@ class SuggestedPackagesReporter } $this->io->writeError(sprintf( - '%s suggests installing %s (%s)', + '%s suggests installing %s%s', $suggestion['source'], $this->escapeOutput($suggestion['target']), - $this->escapeOutput($suggestion['reason']) + $this->escapeOutput('' !== $suggestion['reason'] ? ' ('.$suggestion['reason'].')' : '') )); } diff --git a/tests/Composer/Test/Installer/SuggestedPackagesReporterTest.php b/tests/Composer/Test/Installer/SuggestedPackagesReporterTest.php index 7c80935cd..6aef0fa7d 100644 --- a/tests/Composer/Test/Installer/SuggestedPackagesReporterTest.php +++ b/tests/Composer/Test/Installer/SuggestedPackagesReporterTest.php @@ -142,6 +142,20 @@ class SuggestedPackagesReporterTest extends TestCase $this->suggestedPackagesReporter->output(); } + /** + * @covers ::output + */ + public function testOutputWithNoSuggestedPackage() + { + $this->suggestedPackagesReporter->addPackage('a', 'b', ''); + + $this->io->expects($this->once()) + ->method('writeError') + ->with('a suggests installing b'); + + $this->suggestedPackagesReporter->output(); + } + /** * @covers ::output */