alter output so escaped namespace works

update expected test output fix cs checks use " instead of ' amend test expectatioon to expect double quotes add escape slashes again fix expected testoutput
doctrine · Nov 16, 2024 · f09f881 · f09f881
1 parent a3af231
commit f09f881
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 9 deletions.
diff --git a/src/Tools/Console/Command/DiffCommand.php b/src/Tools/Console/Command/DiffCommand.php
@@ -128,7 +128,6 @@ protected function execute(
         }
 
         $fqcn = $this->getDependencyFactory()->getClassNameGenerator()->generateClassName($namespace);
-
         $diffGenerator = $this->getDependencyFactory()->getDiffGenerator();
 
         try {
@@ -154,12 +153,12 @@ protected function execute(
             sprintf('Generated new migration class to "<info>%s</info>"', $path),
             '',
             sprintf(
-                'To run just this migration for testing purposes, you can use <info>migrations:execute --up \'%s\'</info>',
+                'To run just this migration for testing purposes, you can use <info>migrations:execute --up "%s"</info>',
                 addslashes($fqcn),
             ),
             '',
             sprintf(
-                'To revert the migration you can use <info>migrations:execute --down \'%s\'</info>',
+                'To revert the migration you can use <info>migrations:execute --down "%s"</info>',
                 addslashes($fqcn),
             ),
             '',

diff --git a/src/Tools/Console/Command/DumpSchemaCommand.php b/src/Tools/Console/Command/DumpSchemaCommand.php
@@ -105,12 +105,12 @@ public function execute(
             sprintf('Dumped your schema to a new migration class at "<info>%s</info>"', $path),
             '',
             sprintf(
-                'To run just this migration for testing purposes, you can use <info>migrations:execute --up \'%s\'</info>',
+                'To run just this migration for testing purposes, you can use <info>migrations:execute --up "%s"</info>',
                 addslashes($fqcn),
             ),
             '',
             sprintf(
-                'To revert the migration you can use <info>migrations:execute --down \'%s\'</info>',
+                'To revert the migration you can use <info>migrations:execute --down "%s"</info>',
                 addslashes($fqcn),
             ),
             '',

diff --git a/tests/Tools/Console/Command/DiffCommandTest.php b/tests/Tools/Console/Command/DiffCommandTest.php
@@ -82,9 +82,9 @@ public function testExecute(): void
         self::assertSame([
             'Generated new migration class to "/path/to/migration.php"',
             '',
-            'To run just this migration for testing purposes, you can use migrations:execute --up \'FooNs\\\\Version1234\'',
+            'To run just this migration for testing purposes, you can use migrations:execute --up "FooNs\\\\Version1234"',
             '',
-            'To revert the migration you can use migrations:execute --down \'FooNs\\\\Version1234\'',
+            'To revert the migration you can use migrations:execute --down "FooNs\\\\Version1234"',
         ], array_map(trim(...), explode("\n", trim($output))));
     }
 

diff --git a/tests/Tools/Console/Command/DumpSchemaCommandTest.php b/tests/Tools/Console/Command/DumpSchemaCommandTest.php
@@ -91,9 +91,9 @@ public function testExecute(): void
             [
                 'Dumped your schema to a new migration class at ""',
                 '',
-                'To run just this migration for testing purposes, you can use migrations:execute --up \'FooNs\\\\Version1234\'',
+                'To run just this migration for testing purposes, you can use migrations:execute --up "FooNs\\\\Version1234"',
                 '',
-                'To revert the migration you can use migrations:execute --down \'FooNs\\\\Version1234\'',
+                'To revert the migration you can use migrations:execute --down "FooNs\\\\Version1234"',
                 '',
                 'To use this as a rollup migration you can use the migrations:rollup command.',
             ],