tests(core): improvements

Author: Guikingone

.github/workflows/code-style.yml

@@ -7,7 +7,7 @@ on:
     branches:
       - main
   schedule:
-    - cron: "0 0 * * *"
+    - cron: "5 0 * * *"
 
 jobs:
   php-cs-fixer:

.github/workflows/infection.yml

@@ -7,7 +7,7 @@ on:
     branches:
       - main
   schedule:
-    - cron: "0 0 * * *"
+    - cron: "5 0 * * *"
 
 jobs:
   infection:

.github/workflows/phpunit.yml

@@ -7,7 +7,7 @@ on:
     branches:
       - main
   schedule:
-    - cron: "0 0 * * *"
+    - cron: "5 0 * * *"
 
 jobs:
   phpunit:

.github/workflows/rector.yml

@@ -7,7 +7,7 @@ on:
     branches:
       - main
   schedule:
-    - cron: "0 0 * * *"
+    - cron: "5 0 * * *"
 
 jobs:
   phpstan:

.github/workflows/security.yml

@@ -7,7 +7,7 @@ on:
     branches:
       - main
   schedule:
-    - cron: "0 0 * * *"
+    - cron: "5 0 * * *"
 
 jobs:
   security:

.github/workflows/static-analysis.yml

@@ -7,7 +7,7 @@ on:
     branches:
       - main
   schedule:
-    - cron: "0 0 * * *"
+    - cron: "5 0 * * *"
 
 jobs:
   phpstan:

src/Command/ExportCommand.php

@@ -5,23 +5,31 @@
 namespace SchedulerBundle\Command;
 
 use SchedulerBundle\Export\ExporterRegistryInterface;
+use SchedulerBundle\SchedulerInterface;
+use SchedulerBundle\Task\TaskInterface;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
+use Throwable;
 
 /**
  * @author Guillaume Loulier <[email protected]>
  */
 final class ExportCommand extends Command
 {
     private ExporterRegistryInterface $exporterRegistry;
+    private SchedulerInterface $scheduler;
 
     protected static $defaultName = 'scheduler:export';
 
-    public function __construct(ExporterRegistryInterface $exporterRegistry)
-    {
+    public function __construct(
+        ExporterRegistryInterface $exporterRegistry,
+        SchedulerInterface $scheduler
+    ) {
         $this->exporterRegistry = $exporterRegistry;
+        $this->scheduler = $scheduler;
 
         parent::__construct();
     }
@@ -33,6 +41,10 @@ protected function configure(): void
     {
         $this
             ->setDescription('Export tasks to a specific format')
+            ->setDefinition([
+                new InputOption('format', null, InputOption::VALUE_OPTIONAL, 'The format used to export tasks', 'crontab'),
+                new InputOption('filename', null, InputOption::VALUE_OPTIONAL, 'The name of the filename used to export tasks', 'crontab'),
+            ])
         ;
     }
 
@@ -43,7 +55,30 @@ protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $style = new SymfonyStyle($input, $output);
 
-        // TODO
+        try {
+            $tasks = $this->scheduler->getTasks();
+            if (0 === $tasks->count()) {
+                $style->warning('No tasks found');
+
+                return Command::FAILURE;
+            }
+
+            $filename = $input->getOption('filename');
+            $exporter = $this->exporterRegistry->find($input->getOption('format'));
+
+            $tasks->walk(function (TaskInterface $task) use ($exporter, $filename): void {
+                $exporter->export($filename, $task);
+            });
+        } catch (Throwable $throwable) {
+            $style->error([
+                'An error occurred when exporting tasks',
+                $throwable->getMessage(),
+            ]);
+
+            return Command::FAILURE;
+        }
+
+        $style->success('The export has succeed');
 
         return Command::SUCCESS;
     }

src/DependencyInjection/SchedulerBundleExtension.php

@@ -17,6 +17,7 @@
 use SchedulerBundle\Command\DebugProbeCommand;
 use SchedulerBundle\Command\ExecuteExternalProbeCommand;
 use SchedulerBundle\Command\ExecuteTaskCommand;
+use SchedulerBundle\Command\ExportCommand;
 use SchedulerBundle\Command\ListFailedTasksCommand;
 use SchedulerBundle\Command\ListTasksCommand;
 use SchedulerBundle\Command\RebootSchedulerCommand;
@@ -606,6 +607,17 @@ private function registerCommands(ContainerBuilder $container): void
             ])
         ;
 
+        $container->register(ExportCommand::class, ExportCommand::class)
+            ->setArguments([
+                new Reference(ExporterRegistryInterface::class, ContainerInterface::EXCEPTION_ON_INVALID_REFERENCE),
+                new Reference(SchedulerInterface::class, ContainerInterface::EXCEPTION_ON_INVALID_REFERENCE),
+            ])
+            ->addTag('console.command')
+            ->addTag('container.preload', [
+                'class' => ExportCommand::class,
+            ])
+        ;
+
         $container->register(ListFailedTasksCommand::class, ListFailedTasksCommand::class)
             ->setArguments([
                 new Reference(WorkerInterface::class, ContainerInterface::EXCEPTION_ON_INVALID_REFERENCE),

src/Export/CronTabExporter.php

@@ -5,6 +5,7 @@
 namespace SchedulerBundle\Export;
 
 use SchedulerBundle\Task\TaskInterface;
+use function file_exists;
 
 /**
  * @author Guillaume Loulier <[email protected]>
@@ -13,8 +14,14 @@ final class CronTabExporter implements ExporterInterface
 {
     public function export(string $filename, TaskInterface $task): void
     {
+        if (file_exists($filename)) {
+            return;
+        }
     }
 
+    /**
+     * {@inheritdoc}
+     */
     public function support(string $format): bool
     {
         return 'crontab' === $format;

src/Export/ExporterInterface.php

@@ -13,5 +13,8 @@ interface ExporterInterface
 {
     public function export(string $filename, TaskInterface $task): void;
 
+    /**
+     * Determine if the exporter support the current @param string $format.
+     */
     public function support(string $format): bool;
 }