Add completion to messenger:setup-transports command

Tayfun74 · Tayfun74 · commit 199626a0c55a · 2021-10-21T23:44:25.000+02:00
diff --git a/src/Symfony/Component/Messenger/Command/SetupTransportsCommand.php b/src/Symfony/Component/Messenger/Command/SetupTransportsCommand.php
@@ -87,9 +87,9 @@ protected function execute(InputInterface $input, OutputInterface $output)
     public function complete(CompletionInput $input, CompletionSuggestions $suggestions): void
     {
         if (!$input->mustSuggestArgumentValuesFor('transport')) {
+            $suggestions->suggestValues($this->transportNames);
+
             return;
         }
-
-        $suggestions->suggestValues($this->transportNames);
     }
 }
diff --git a/src/Symfony/Component/Messenger/Tests/Command/SetupTransportsCommandTest.php b/src/Symfony/Component/Messenger/Tests/Command/SetupTransportsCommandTest.php
@@ -94,7 +94,6 @@ public function testReceiverNameArgumentNotFound()
      */
     public function testComplete(array $input, array $expectedSuggestions)
     {
-        /** @var MockObject&ServiceLocator $serviceLocator */
         $serviceLocator = $this->createMock(ServiceLocator::class);
         $command = new SetupTransportsCommand($serviceLocator, ['amqp', 'other_transport']);
         $tester = new CommandCompletionTester($command);

Original file line number	Diff line number	Diff line change
`@@ -87,9 +87,9 @@ protected function execute(InputInterface $input, OutputInterface $output)`
`87`	`87`	`public function complete(CompletionInput $input, CompletionSuggestions $suggestions): void`
`88`	`88`	`{`
`89`	`89`	`if (!$input->mustSuggestArgumentValuesFor('transport')) {`
	`90`	`+ $suggestions->suggestValues($this->transportNames);`
	`91`	`+`
`90`	`92`	`return;`
`91`	`93`	`}`
`92`		`-`
`93`		`- $suggestions->suggestValues($this->transportNames);`
`94`	`94`	`}`
`95`	`95`	`}`
Original file line number	Diff line number	Diff line change
`@@ -94,7 +94,6 @@ public function testReceiverNameArgumentNotFound()`
`94`	`94`	`*/`
`95`	`95`	`public function testComplete(array $input, array $expectedSuggestions)`
`96`	`96`	`{`
`97`		`- /** @var MockObject&ServiceLocator $serviceLocator */`
`98`	`97`	`$serviceLocator = $this->createMock(ServiceLocator::class);`
`99`	`98`	`$command = new SetupTransportsCommand($serviceLocator, ['amqp', 'other_transport']);`
`100`	`99`	`$tester = new CommandCompletionTester($command);`