fix debug:autowiringcommand

sez-open · Amrouche Hamza · commit cbec3df03f1e · 2019-04-06T17:21:29.000+02:00
diff --git a/src/Symfony/Bundle/FrameworkBundle/Command/DebugAutowiringCommand.php b/src/Symfony/Bundle/FrameworkBundle/Command/DebugAutowiringCommand.php
@@ -100,6 +100,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
         $hasAlias = [];
         $all = $input->getOption('all');
         $previousId = '-';
+        $serviceIdsNb = \count($serviceIds);
         foreach ($serviceIds as $serviceId) {
             $text = [];
             if (0 !== strpos($serviceId, $previousId)) {
@@ -126,7 +127,13 @@ protected function execute(InputInterface $input, OutputInterface $output)
                 if ($serviceAlias->isDeprecated()) {
                     $serviceLine .= ' - <fg=magenta>deprecated</>';
                 }
+
+            } elseif ($all && $search && $serviceIdsNb > 0) {
+                $io->text('Please use interfaces in your type-hints instead of classes to benefit from the dependency inversion principle.');
             } elseif (!$all) {
+                if ($search) {
+                    $io->text(sprintf('%s more concrete service(s) would be displayed when adding the --all option.', $serviceIdsNb));
+                }
                 continue;
             }
             $text[] = $serviceLine;
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/Functional/DebugAutowiringCommandTest.php b/src/Symfony/Bundle/FrameworkBundle/Tests/Functional/DebugAutowiringCommandTest.php
@@ -72,4 +72,29 @@ public function testSearchNoResults()
         $this->assertContains('No autowirable classes or interfaces found matching "foo_fake"', $tester->getErrorOutput());
         $this->assertEquals(1, $tester->getStatusCode());
     }
+
+    public function testSearchNotAliasedService()
+    {
+        static::bootKernel(['test_case' => 'ContainerDebug', 'root_config' => 'config.yml']);
+
+        $application = new Application(static::$kernel);
+        $application->setAutoExit(false);
+
+        $tester = new ApplicationTester($application);
+        $tester->run(['command' => 'debug:autowiring', 'search' => 'redirect']);
+
+        $this->assertContains('more concrete service(s) would be displayed when adding the --all option.', $tester->getDisplay());
+    }
+
+    public function testSearchNotAliasedServiceWithAll()
+    {
+        static::bootKernel(['test_case' => 'ContainerDebug', 'root_config' => 'config.yml']);
+
+        $application = new Application(static::$kernel);
+        $application->setAutoExit(false);
+
+        $tester = new ApplicationTester($application);
+        $tester->run(['command' => 'debug:autowiring', 'search' => 'redirect', '--all' => true]);
+        $this->assertContains('Please use interfaces in your type-hints instead of classes to benefit from the dependency inversion principle.', $tester->getDisplay());
+    }
 }
