#12 - check-path option addition (#14)

Co-authored-by: marek <[email protected]>
lexik · Sep 27, 2023 · 1595e0e · 1595e0e
1 parent 2fafc7a
commit 1595e0e
Show file tree

Hide file tree

Showing 10 changed files with 80 additions and 27 deletions.
diff --git a/Command/GenerateCronCommand.php b/Command/GenerateCronCommand.php
@@ -4,6 +4,7 @@
 
 use Lexik\Bundle\CronFileGeneratorBundle\Cron\DumpFileFactory;
 use Lexik\Bundle\CronFileGeneratorBundle\Exception\CronEmptyException;
+use Lexik\Bundle\CronFileGeneratorBundle\Exception\WrongConsoleBinPathException;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
@@ -33,6 +34,7 @@ protected function configure(): void
 EOPHP
             )
             ->addOption('dry-run', null, InputOption::VALUE_NONE, 'Execute the dump file as a dry run.')
+            ->addOption('check-path', 'c', InputOption::VALUE_NONE, 'Check absolute path on crontab generation.')
             ->addArgument('env-mode', InputArgument::REQUIRED, 'Env config')
         ;
     }
@@ -44,6 +46,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $io->title('Generated cron file');
 
         $dryRun = (bool) $input->getOption('dry-run');
+        $checkPath = (bool) $input->getOption('check-path');
 
         try {
             $dumpFile = $this->dumpFileFactory->createWithEnv($input->getArgument('env-mode'));
@@ -63,6 +66,16 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             return 0;
         }
 
+        if ($checkPath) {
+            try {
+                $dumpFile->checkPath();
+            } catch (WrongConsoleBinPathException $e) {
+                $output->writeln(sprintf('<error>%s.</error>', $e->getMessage()));
+
+                return 1;
+            }
+        }
+
         $filename = $dumpFile->dumpFile();
 
         $io->success('File generated');

diff --git a/Cron/DumpFile.php b/Cron/DumpFile.php
@@ -3,6 +3,7 @@
 namespace Lexik\Bundle\CronFileGeneratorBundle\Cron;
 
 use Lexik\Bundle\CronFileGeneratorBundle\Exception\CronEmptyException;
+use Lexik\Bundle\CronFileGeneratorBundle\Exception\WrongConsoleBinPathException;
 use Symfony\Component\Filesystem\Filesystem;
 use Twig\Environment;
 
@@ -32,6 +33,13 @@ public function dumpFile(): string
         return $destination;
     }
 
+    public function checkPath(): void
+    {
+        if (false === file_exists($binPath = $this->configuration->getAbsolutePath())) {
+            throw new WrongConsoleBinPathException(sprintf('Configured path "%s" does not exist', $binPath));
+        }
+    }
+
     public function dryRun(): string
     {
         return $this->render();

diff --git a/DependencyInjection/Configuration.php b/DependencyInjection/Configuration.php
@@ -7,9 +7,6 @@
 
 class Configuration implements ConfigurationInterface
 {
-    /**
-     * {@inheritdoc}
-     */
     public function getConfigTreeBuilder(): TreeBuilder
     {
         $treeBuilder = new TreeBuilder('lexik_cron_file_generator');

diff --git a/DependencyInjection/LexikCronFileGeneratorExtension.php b/DependencyInjection/LexikCronFileGeneratorExtension.php
@@ -10,9 +10,6 @@
 
 class LexikCronFileGeneratorExtension extends Extension
 {
-    /**
-     * {@inheritdoc}
-     */
     public function load(array $configs, ContainerBuilder $container): void
     {
         $configuration = new Configuration();

diff --git a/Exception/WrongConsoleBinPathException.php b/Exception/WrongConsoleBinPathException.php
@@ -0,0 +1,7 @@
+<?php
+
+namespace Lexik\Bundle\CronFileGeneratorBundle\Exception;
+
+class WrongConsoleBinPathException extends \Exception
+{
+}
diff --git a/Tests/Cron/DumpFileTest.php b/Tests/Cron/DumpFileTest.php
@@ -5,6 +5,7 @@
 use Lexik\Bundle\CronFileGeneratorBundle\Cron\Configuration;
 use Lexik\Bundle\CronFileGeneratorBundle\Cron\DumpFile;
 use Lexik\Bundle\CronFileGeneratorBundle\Exception\CronEmptyException;
+use Lexik\Bundle\CronFileGeneratorBundle\Exception\WrongConsoleBinPathException;
 use PHPUnit\Framework\TestCase;
 use Twig\Environment;
 
@@ -90,4 +91,40 @@ public function testEmptyCron()
         $dumpFile = new DumpFile($templating, $configuration);
         $dumpFile->init('staging');
     }
+
+    public function testWrongAbsolutePath()
+    {
+        self::expectException(WrongConsoleBinPathException::class);
+
+        $templating = $this->createMock(Environment::class);
+        $templating->expects($this->never())->method('render');
+
+        $configuration = new Configuration([
+            'mailto' => null,
+            'env_available' => [
+                'staging',
+            ],
+            'user' => [
+                'staging' => 'project_staging',
+            ],
+            'php_version' => 'php7.3',
+            'absolute_path' => [
+                'staging' => 'path/to/staging',
+            ],
+            'output_path' => $this->outputDir.'/cron_file',
+            'crons' => [
+                [
+                    'name' => 'test',
+                    'command' => 'app:test',
+                    'env' => [
+                        'staging' => '* * * * *',
+                    ],
+                ],
+            ],
+        ]);
+
+        $dumpFile = new DumpFile($templating, $configuration);
+        $dumpFile->init('staging');
+        $dumpFile->checkPath();
+    }
 }
diff --git a/Tests/Functional/Bundle/DependencyInjection/BundleExtension.php b/Tests/Functional/Bundle/DependencyInjection/BundleExtension.php
@@ -9,16 +9,10 @@
 
 class BundleExtension extends Extension implements PrependExtensionInterface
 {
-    /**
-     * {@inheritdoc}
-     */
     public function load(array $configs, ContainerBuilder $container): void
     {
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function prepend(ContainerBuilder $container): void
     {
         // Annotation must be disabled since this bundle doesn't use Doctrine

diff --git a/Tests/Functional/Command/GenerateCronFileCommandTest.php b/Tests/Functional/Command/GenerateCronFileCommandTest.php
@@ -56,4 +56,19 @@ public function testDryRun()
             $tester->getDisplay()
         );
     }
+
+    public function testCheckExecutivePath()
+    {
+        $kernel = $this->bootKernel();
+
+        $tester = new CommandTester(
+            $kernel->getContainer()->get('lexik_bundle_cron_file_generator.command.generate_cron_command')
+        );
+
+        $this->assertSame(1, $tester->execute(['env-mode' => 'staging', '--check-path' => true]));
+
+        $expected = 'Configured path "path/to/staging" does not exist';
+
+        $this->assertStringContainsString($expected, $tester->getDisplay());
+    }
 }
diff --git a/Tests/Functional/TestCase.php b/Tests/Functional/TestCase.php
@@ -12,19 +12,13 @@ abstract class TestCase extends WebTestCase
 
     protected static $client;
 
-    /**
-     * {@inheritdoc}
-     */
     protected static function createKernel(array $options = []): KernelInterface
     {
         require_once __DIR__.'/app/AppKernel.php';
 
         return new AppKernel('test', true, isset($options['config']) ? $options['config'] : 'base');
     }
 
-    /**
-     * {@inheritdoc}
-     */
     protected function setUp(): void
     {
         $fs = new Filesystem();

diff --git a/Tests/Functional/app/AppKernel.php b/Tests/Functional/app/AppKernel.php
@@ -22,9 +22,6 @@ public function __construct(string $environment, bool $debug, string $config = '
         $this->config = $config;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function registerBundles(): iterable
     {
         return [
@@ -35,17 +32,11 @@ public function registerBundles(): iterable
         ];
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getCacheDir(): string
     {
         return sys_get_temp_dir().'/LexikCronFileGeneratorBundle/cache';
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getLogDir(): string
     {
         return sys_get_temp_dir().'/LexikCronFileGeneratorBundle/logs';