fix(MigrateDumpCommand::mysqlDump): Remove AUTO_INCREMENT= values from dumps.

Author: Paul Rogers

src/Commands/MigrateDumpCommand.php

@@ -71,7 +71,8 @@ private static function mysqlDump(array $db_config, string $schema_sql_path) : i
 
         // Not including connection name in file since typically only one DB.
         // Excluding any hash or date suffix since only current is relevant.
-        $command_prefix = 'mysqldump --routines --skip-add-drop-table'
+        $command_prefix = 'set -o pipefail &&'
+            . ' mysqldump --routines --skip-add-drop-table --fake'
             . ' --skip-add-locks --skip-comments --skip-set-charset --tz-utc'
             . ' --host=' . escapeshellarg($db_config['host'])
             . ' --port=' . escapeshellarg($db_config['port'])
@@ -80,8 +81,9 @@ private static function mysqlDump(array $db_config, string $schema_sql_path) : i
             . ' ' . escapeshellarg($db_config['database']);
         passthru(
             $command_prefix
-            . ' --result-file=' . escapeshellarg($schema_sql_path)
-            . ' --no-data',
+            . ' --no-data'
+            // CONSIDER: Avoiding Bash/shell and Sed by doing replacement in PHP.
+            . ' | sed -E "s/ AUTO_INCREMENT=[0-9]+ ?//g" > ' . escapeshellarg($schema_sql_path),
             $exit_code
         );
 

tests/Mysql/MigrateDumpTest.php

@@ -17,5 +17,6 @@ public function test_handle()
         $result_sql = file_get_contents($this->schemaSqlPath);
         $this->assertContains('CREATE TABLE `test_ms`', $result_sql);
         $this->assertContains('INSERT INTO `migrations`', $result_sql);
+        $this->assertNotContains(' AUTO_INCREMENT=', $result_sql);
     }
 }