MAGETWO-93701: Make cron:run CLI command react on cron disable configuration

Author: NadiyaS

app/code/Magento/Cron/Console/Command/CronCommand.php

@@ -14,6 +14,7 @@
 use Magento\Store\Model\Store;
 use Magento\Store\Model\StoreManager;
 use Magento\Cron\Observer\ProcessCronQueueObserver;
+use Magento\Framework\App\DeploymentConfig;
 use Magento\Framework\Console\Cli;
 use Magento\Framework\Shell\ComplexParameter;
 
@@ -35,13 +36,22 @@ class CronCommand extends Command
     private $objectManagerFactory;
 
     /**
-     * Constructor
+     * Application deployment configuration
      *
+     * @var DeploymentConfig
+     */
+    private $deploymentConfig;
+
+    /**
      * @param ObjectManagerFactory $objectManagerFactory
+     * @param DeploymentConfig $deploymentConfig Application deployment configuration
      */
-    public function __construct(ObjectManagerFactory $objectManagerFactory)
-    {
+    public function __construct(
+        ObjectManagerFactory $objectManagerFactory,
+        DeploymentConfig $deploymentConfig = null
+    ){
         $this->objectManagerFactory = $objectManagerFactory;
+        $this->deploymentConfig = $deploymentConfig;
         parent::__construct();
     }
 
@@ -71,10 +81,16 @@ protected function configure()
     }
 
     /**
+     * Runs cron jobs if cron is not disabled in Magento configurations
+     *
      * {@inheritdoc}
      */
     protected function execute(InputInterface $input, OutputInterface $output)
     {
+        if (!$this->deploymentConfig->get('cron/enabled', 1)) {
+            $output->writeln('<info>' . 'Cron is disabled. Jobs were not run.' . '</info>');
+            return;
+        }
         $omParams = $_SERVER;
         $omParams[StoreManager::PARAM_RUN_CODE] = 'admin';
         $omParams[Store::CUSTOM_ENTRY_POINT_PARAM] = true;

app/code/Magento/Cron/Test/Unit/Console/Command/CronCommandTest.php

@@ -10,15 +10,52 @@
 
 class CronCommandTest extends \PHPUnit\Framework\TestCase
 {
+    /**
+     * Test command with disables cron
+     *
+     * @return void
+     */
+    public function testExecuteWithDisabledCrons()
+    {
+        $objectManagerFactory = $this->createMock(\Magento\Framework\App\ObjectManagerFactory::class);
+        $deploymentConfigMock = $this->createMock(\Magento\Framework\App\DeploymentConfig::class);
+
+        $objectManagerFactory->expects($this->never())
+            ->method('create');
+        $deploymentConfigMock->expects($this->once())
+            ->method('get')
+            ->with('cron/enabled', 1)
+            ->willReturn(0);
+        $commandTester = new CommandTester(new CronCommand($objectManagerFactory, $deploymentConfigMock));
+        $commandTester->execute([]);
+        $expectedMsg = 'Cron is disabled. Jobs were not run.' . PHP_EOL;
+        $this->assertEquals($expectedMsg, $commandTester->getDisplay());
+    }
+
+    /**
+     * Test command with enabled cron
+     *
+     * @return void
+     */
     public function testExecute()
     {
         $objectManagerFactory = $this->createMock(\Magento\Framework\App\ObjectManagerFactory::class);
+        $deploymentConfigMock = $this->createMock(\Magento\Framework\App\DeploymentConfig::class);
         $objectManager = $this->createMock(\Magento\Framework\ObjectManagerInterface::class);
         $cron = $this->createMock(\Magento\Framework\App\Cron::class);
-        $objectManager->expects($this->once())->method('create')->willReturn($cron);
-        $cron->expects($this->once())->method('launch');
-        $objectManagerFactory->expects($this->once())->method('create')->willReturn($objectManager);
-        $commandTester = new CommandTester(new CronCommand($objectManagerFactory));
+        $objectManager->expects($this->once())
+            ->method('create')
+            ->willReturn($cron);
+        $cron->expects($this->once())
+            ->method('launch');
+        $objectManagerFactory->expects($this->once())
+            ->method('create')
+            ->willReturn($objectManager);
+        $deploymentConfigMock->expects($this->once())
+            ->method('get')
+            ->with('cron/enabled', 1)
+            ->willReturn(1);
+        $commandTester = new CommandTester(new CronCommand($objectManagerFactory, $deploymentConfigMock));
         $commandTester->execute([]);
         $expectedMsg = 'Ran jobs by schedule.' . PHP_EOL;
         $this->assertEquals($expectedMsg, $commandTester->getDisplay());