Merge pull request #1009 from cakephp/fix/dump-missing-migrations-dir

Fix dump command error when migrations directory does not exist

Author: dereuromark

src/Command/DumpCommand.php

@@ -112,6 +112,12 @@ public function execute(Arguments $args, ConsoleIo $io): ?int
         ]);
         $config = $factory->createConfig();
         $path = $config->getMigrationPath();
+
+        if (!is_dir($path)) {
+            $io->verbose('<info>No migrations directory found, skipping dump.</info>');
+
+            return self::CODE_SUCCESS;
+        }
         $connectionName = (string)$config->getConnection();
         $connection = ConnectionManager::get($connectionName);
         assert($connection instanceof Connection);

tests/TestCase/Command/DumpCommandTest.php

@@ -84,6 +84,13 @@ public function testExecuteSuccess(): void
         $this->assertEquals(['id', 'letter'], $generatedDump['letters']->columns());
     }
 
+    public function testExecuteNoMigrationsDirectory(): void
+    {
+        $this->exec('migrations dump --connection test --source NonExistentMigrations');
+
+        $this->assertExitSuccess();
+    }
+
     public function testExecutePlugin(): void
     {
         $this->loadPlugins(['Migrator']);