bug symfony#45947 [FrameworkBundle] [Command] Fix debug:router --no-interaction error … (WilliamBoulle)

This PR was squashed before being merged into the 4.4 branch.

Discussion
----------

[FrameworkBundle] [Command] Fix `debug:router --no-interaction` error …

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix symfony#45777
| License       | MIT

When command `debug:router` is used in non-interactive mode with a name that matches more than one route, render the list of matching routes
![debug_router_profiler_nointeraction](https://user-images.githubusercontent.com/18715733/161756561-cb2b5f75-111d-4920-8e81-8066b302a887.png)

Commits
-------

e92e261 [FrameworkBundle] [Command] Fix `debug:router --no-interaction` error …

Author: fabpot

src/Symfony/Bundle/FrameworkBundle/Command/RouterDebugCommand.php

@@ -81,7 +81,21 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $routes = $this->router->getRouteCollection();
 
         if ($name) {
-            if (!($route = $routes->get($name)) && $matchingRoutes = $this->findRouteNameContaining($name, $routes)) {
+            $route = $routes->get($name);
+            $matchingRoutes = $this->findRouteNameContaining($name, $routes);
+
+            if (!$input->isInteractive() && !$route && \count($matchingRoutes) > 1) {
+                $helper->describe($io, $this->findRouteContaining($name, $routes), [
+                    'format' => $input->getOption('format'),
+                    'raw_text' => $input->getOption('raw'),
+                    'show_controllers' => $input->getOption('show-controllers'),
+                    'output' => $io,
+                ]);
+
+                return 0;
+            }
+
+            if (!$route && $matchingRoutes) {
                 $default = 1 === \count($matchingRoutes) ? $matchingRoutes[0] : null;
                 $name = $io->choice('Select one of the matching routes', $matchingRoutes, $default);
                 $route = $routes->get($name);
@@ -120,4 +134,16 @@ private function findRouteNameContaining(string $name, RouteCollection $routes):
 
         return $foundRoutesNames;
     }
+
+    private function findRouteContaining(string $name, RouteCollection $routes): RouteCollection
+    {
+        $foundRoutes = new RouteCollection();
+        foreach ($routes as $routeName => $route) {
+            if (false !== stripos($routeName, $name)) {
+                $foundRoutes->add($routeName, $route);
+            }
+        }
+
+        return $foundRoutes;
+    }
 }

src/Symfony/Bundle/FrameworkBundle/Tests/Functional/RouterDebugCommandTest.php

@@ -61,6 +61,23 @@ public function testSearchMultipleRoutes()
         $this->assertStringContainsString('/test', $tester->getDisplay());
     }
 
+    public function testSearchMultipleRoutesWithoutInteraction()
+    {
+        $tester = $this->createCommandTester();
+        $ret = $tester->execute(['name' => 'routerdebug'], ['interactive' => false]);
+
+        $this->assertSame(0, $ret, 'Returns 0 in case of success');
+        $this->assertStringNotContainsString('Select one of the matching routes:', $tester->getDisplay());
+        $this->assertStringContainsString('routerdebug_session_welcome', $tester->getDisplay());
+        $this->assertStringContainsString('/session', $tester->getDisplay());
+        $this->assertStringContainsString('routerdebug_session_welcome_name', $tester->getDisplay());
+        $this->assertStringContainsString('/session/{name} ', $tester->getDisplay());
+        $this->assertStringContainsString('routerdebug_session_logout', $tester->getDisplay());
+        $this->assertStringContainsString('/session_logout', $tester->getDisplay());
+        $this->assertStringContainsString('routerdebug_test', $tester->getDisplay());
+        $this->assertStringContainsString('/test', $tester->getDisplay());
+    }
+
     public function testSearchWithThrow()
     {
         $this->expectException(\InvalidArgumentException::class);