Merge pull request #83 from synolia/fix/execute-immediately

fix execute immediately

Makefile

@@ -51,6 +51,9 @@ endif
 update-dependencies:
 	${COMPOSER} config extra.symfony.require "~${SYMFONY_VERSION}"
 	${COMPOSER} require symfony/asset:~${SYMFONY_VERSION} --no-scripts --no-update
+ifeq ($(shell [[ $(SYLIUS_VERSION) == 1.10.0 ]] && echo true ),true)
+	${COMPOSER} require php-http/message-factory --no-scripts --no-update
+endif
 	${COMPOSER} update --no-progress -n
 
 install-plugin:
@@ -80,7 +83,7 @@ phpunit-configure:
 	cp phpunit.xml.dist ${TEST_DIRECTORY}/phpunit.xml
 
 phpunit-run:
-	cd ${TEST_DIRECTORY} && ./vendor/bin/phpunit
+	cd ${TEST_DIRECTORY} && ./vendor/bin/phpunit --testdox
 
 behat-configure: ## Configure Behat
 	(cd ${TEST_DIRECTORY} && cp behat.yml.dist behat.yml)

src/Command/SynoliaSchedulerRunCommand.php

@@ -73,6 +73,8 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             // delayed execution just after, to keep cron comparison effective
             if ($this->shouldExecuteCommand($command, $io)) {
                 $this->scheduledCommandPlanner->plan($command);
+                // The execution is planned, and does not need to be launched again in the future.
+                $command->setExecuteImmediately(false);
             }
         }
 

tests/PHPUnit/Command/SynoliaSchedulerRunCommandTest.php

@@ -92,6 +92,7 @@ public function testExecuteWithANewCommandThatItMarkedAsExecuteNow(): void
         self::assertStringContainsString('Immediately execution asked for : about', $commandTester->getDisplay());
         self::assertStringContainsString('Execute Command "about" - last execution : never', $commandTester->getDisplay());
         self::assertStringNotContainsString('Nothing to do', $commandTester->getDisplay());
+        self::assertFalse($scheduledCommand->isExecuteImmediately());
     }
 
     public function testExecuteNonExistentCommand(): void