Adding formatted option for migration command

symfony · Aug 14, 2023 · 5f9232b · 5f9232b
1 parent 79c710b
commit 5f9232b
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 0 deletions.
diff --git a/src/FileManager.php b/src/FileManager.php
@@ -194,6 +194,7 @@ private function realPath(string $absolutePath): string
         }
 
         $finalPath = implode('/', $finalParts);
+
         // Normalize: // => /
         // Normalize: /./ => /
         return str_replace(['//', '/./'], '/', $finalPath);

diff --git a/src/Maker/MakeMigration.php b/src/Maker/MakeMigration.php
@@ -70,6 +70,9 @@ public function configureCommand(Command $command, InputConfiguration $inputConf
                 ->addOption('shard', null, InputOption::VALUE_REQUIRED, 'The shard connection name')
             ;
         }
+
+        $command
+            ->addOption('formatted', null, InputOption::VALUE_NONE, 'Format the generated SQL');
     }
 
     public function generate(InputInterface $input, ConsoleStyle $io, Generator $generator)
@@ -88,6 +91,10 @@ public function generate(InputInterface $input, ConsoleStyle $io, Generator $gen
         }
         // end 2.x support
 
+        if ($input->hasOption('formatted') && null !== $input->getOption('formatted')) {
+            $options[] = '--formatted';
+        }
+
         $generateMigrationCommand = $this->application->find('doctrine:migrations:diff');
         $generateMigrationCommandInput = new ArgvInput($options);
 

diff --git a/tests/Maker/MakeMigrationTest.php b/tests/Maker/MakeMigrationTest.php
@@ -131,5 +131,16 @@ public function getTestDetails(): \Generator
                 $this->assertStringNotContainsString('Success', $output);
             }),
         ];
+
+        yield 'it_generate_a_formatted_migration' => [$this->createMakeMigrationTest()
+            ->addRequiredPackageVersion('doctrine/doctrine-migrations-bundle', '>=3')
+            ->run(function (MakerTestRunner $runner) {
+                $runner->runConsole('make:migration', [], '--formatted');
+
+                $output = $runner->runMaker([/* no input */]);
+
+                $this->assertStringContainsString('Success', $output);
+            }),
+        ];
     }
 }