Merge pull request #808 from cakephp/4.x-require-bake-for-baking

check if bake is loaded in BakeSimpleMigrationCommand

Author: markstory

src/Command/BakeSimpleMigrationCommand.php

@@ -20,6 +20,7 @@
 use Cake\Console\Arguments;
 use Cake\Console\ConsoleIo;
 use Cake\Console\ConsoleOptionParser;
+use Cake\Core\Plugin;
 use Cake\Utility\Inflector;
 use Migrations\Util\Util;
 
@@ -105,6 +106,10 @@ public function getPath(Arguments $args): string
      */
     public function execute(Arguments $args, ConsoleIo $io): ?int
     {
+        if (!Plugin::isLoaded('Bake')) {
+            $io->err('Bake plugin is not loaded. Please load it first to generate a migration.');
+            $this->abort();
+        }
         $this->extractCommonProperties($args);
         $name = $args->getArgumentAt(0);
         if (empty($name)) {

tests/TestCase/Command/BakeMigrationCommandTest.php

@@ -386,4 +386,16 @@ public function testActionWithoutValidPrefix()
         $this->assertExitCode(BaseCommand::CODE_ERROR);
         $this->assertErrorContains('When applying fields the migration name should start with one of the following prefixes: `Create`, `Drop`, `Add`, `Remove`, `Alter`.');
     }
+
+    public function testBakeMigrationWithoutBake()
+    {
+        // Make sure to unload the Bake plugin
+        $this->createApp()->getEventManager()->on('Console.buildCommands', function ($event, $commands) {
+            Plugin::getCollection()->remove('Bake');
+        });
+
+        $this->exec('bake migration CreateUsers name:string --connection test');
+        $this->assertExitCode(BaseCommand::CODE_ERROR);
+        $this->assertErrorContains('Bake plugin is not loaded. Please load it first to generate a migration.');
+    }
 }