bug symfony#60741 [Scheduler] Fix #[AsCronTask] not passing arguments to command (Jan Pintr, jan-pintr)

This PR was merged into the 7.3 branch.

Discussion
----------

[Scheduler] Fix `#[AsCronTask]` not passing arguments to command

| Q             | A
| ------------- | ---
| Branch?       | 7.3
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | -
| License       | MIT

This MR fixes passing `AsCronTask` arguments to `Command`. Reproduced can be by following command. It wont receive `--all` option or any other argument specified in `AsCronTask` when executed by Scheduler.
```
#[AsCommand(name: 'debug:as-cron-task')]
#[AsCronTask('* * * * *', arguments: '--all')]
class SampleCommand extends Command
{
    protected function configure(): void
    {
        $this->addOption('all', null, InputOption::VALUE_NONE);
    }

    protected function execute(InputInterface $input, OutputInterface $output): int
    {
        dump($input->getOption('all'));

        return Command::SUCCESS;
    }
}
```

Fix description: When command name was found in `AsCommand` attribute then attributes were not appended to message definition due to missing brackets.

Commits
-------

b57a815 Fix AsCronTask not passing arguments to command

Author: nicolas-grekas

src/Symfony/Bundle/FrameworkBundle/Tests/Fixtures/Messenger/DummyCommand.php

@@ -0,0 +1,30 @@
+<?php
+
+namespace Symfony\Bundle\FrameworkBundle\Tests\Fixtures\Messenger;
+
+use Symfony\Component\Console\Attribute\AsCommand;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputArgument;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
+use Symfony\Component\Scheduler\Attribute\AsCronTask;
+
+#[AsCronTask(expression: '* * * * *', schedule: 'dummy_command')]
+#[AsCronTask(expression: '0 * * * *', arguments: 'test', schedule: 'dummy_command')]
+#[AsCommand(name: 'test:dummy-command')]
+class DummyCommand extends Command
+{
+    public static array $calls = [];
+
+    public function configure(): void
+    {
+        $this->addArgument('dummy-argument', InputArgument::OPTIONAL);
+    }
+
+    public function execute(InputInterface $input, ?OutputInterface $output = null): int
+    {
+        self::$calls[__FUNCTION__][] = $input->getArgument('dummy-argument');
+
+        return Command::SUCCESS;
+    }
+}

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

@@ -12,6 +12,7 @@
 namespace Symfony\Bundle\FrameworkBundle\Tests\Functional;
 
 use Symfony\Bundle\FrameworkBundle\Tests\Fixtures\Messenger\BarMessage;
+use Symfony\Bundle\FrameworkBundle\Tests\Fixtures\Messenger\DummyCommand;
 use Symfony\Bundle\FrameworkBundle\Tests\Fixtures\Messenger\DummySchedule;
 use Symfony\Bundle\FrameworkBundle\Tests\Fixtures\Messenger\DummyTask;
 use Symfony\Bundle\FrameworkBundle\Tests\Fixtures\Messenger\FooMessage;
@@ -88,6 +89,29 @@ public function testAutoconfiguredScheduler()
         $this->assertSame([['5', 6], ['7', 8]], $calls['attributesOnMethod']);
     }
 
+    public function testAutoconfiguredSchedulerCommand()
+    {
+        $container = self::getContainer();
+        $container->set('clock', $clock = new MockClock('2023-10-26T08:59:59Z'));
+
+        $this->assertTrue($container->get('receivers')->has('scheduler_dummy_command'));
+        $this->assertInstanceOf(SchedulerTransport::class, $cron = $container->get('receivers')->get('scheduler_dummy_command'));
+        $bus = $container->get(MessageBusInterface::class);
+
+        $getCalls = static function (float $sleep) use ($clock, $cron, $bus) {
+            DummyCommand::$calls = [];
+            $clock->sleep($sleep);
+            foreach ($cron->get() as $message) {
+                $bus->dispatch($message->with(new ReceivedStamp('scheduler_dummy_command')));
+            }
+
+            return DummyCommand::$calls;
+        };
+
+        $this->assertSame([], $getCalls(0));
+        $this->assertSame(['execute' => [0 => null, 1 => 'test']], $getCalls(1));
+    }
+
     public function testSchedulerWithCustomTransport()
     {
         $container = self::getContainer();

src/Symfony/Bundle/FrameworkBundle/Tests/Functional/app/Scheduler/config.yml

@@ -16,6 +16,9 @@ services:
     Symfony\Bundle\FrameworkBundle\Tests\Fixtures\Messenger\DummyTaskWithCustomReceiver:
         autoconfigure: true
 
+    Symfony\Bundle\FrameworkBundle\Tests\Fixtures\Messenger\DummyCommand:
+        autoconfigure: true
+
     clock:
         synthetic: true
 

src/Symfony/Component/Scheduler/DependencyInjection/AddScheduleMessengerPass.php

@@ -58,7 +58,9 @@ public function process(ContainerBuilder $container): void
                 if ($serviceDefinition->hasTag('console.command')) {
                     /** @var AsCommand|null $attribute */
                     $attribute = ($container->getReflectionClass($serviceDefinition->getClass())->getAttributes(AsCommand::class)[0] ?? null)?->newInstance();
-                    $message = new Definition(RunCommandMessage::class, [$attribute?->name ?? $serviceDefinition->getClass()::getDefaultName().(empty($tagAttributes['arguments']) ? '' : " {$tagAttributes['arguments']}")]);
+                    $commandName = $attribute?->name ?? $serviceDefinition->getClass()::getDefaultName();
+
+                    $message = new Definition(RunCommandMessage::class, [$commandName.($tagAttributes['arguments'] ? " {$tagAttributes['arguments']}" : '')]);
                 } else {
                     $message = new Definition(ServiceCallMessage::class, [$serviceId, $tagAttributes['method'] ?? '__invoke', (array) ($tagAttributes['arguments'] ?? [])]);
                 }