skipping tests before DoctrineMigraationsBundle 3

Author: weaverryan

tests/Maker/MakeMigrationTest.php

@@ -11,6 +11,7 @@
 
 namespace Symfony\Bundle\MakerBundle\Tests\Maker;
 
+use Doctrine\Bundle\MigrationsBundle\Command\MigrationsDiffDoctrineCommand;
 use Symfony\Bundle\MakerBundle\Maker\MakeMigration;
 use Symfony\Bundle\MakerBundle\Test\MakerTestCase;
 use Symfony\Bundle\MakerBundle\Test\MakerTestDetails;
@@ -61,39 +62,41 @@ public function getTestDetails()
             }),
         ];
 
-        yield 'migration_with_previous_migration_question' => [MakerTestDetails::createTest(
-            $this->getMakerInstance(MakeMigration::class),
-            [
-                // confirm migration
-                'y',
-            ])
-            ->setFixtureFilesPath(__DIR__.'/../fixtures/MakeMigration')
-            ->configureDatabase(false)
-            ->addExtraDependencies('doctrine/orm:@stable')
-            // generate a migration first
-            ->addPreMakeCommand('php bin/console make:migration')
-            ->assert(function (string $output, string $directory) {
-                $this->assertStringContainsString('You have 1 available migrations to execute', $output);
-                $this->assertStringContainsString('Success', $output);
-                $this->assertCount(14, explode("\n", $output), 'Asserting that very specific output is shown - some should be hidden');
-                var_dump($output);
-            }),
-        ];
+        // these tests are for DoctrineMigrationsBundle v3 only
+        if (!class_exists(MigrationsDiffDoctrineCommand::class)) {
+            yield 'migration_with_previous_migration_question' => [MakerTestDetails::createTest(
+                $this->getMakerInstance(MakeMigration::class),
+                [
+                    // confirm migration
+                    'y',
+                ])
+                ->setFixtureFilesPath(__DIR__ . '/../fixtures/MakeMigration')
+                ->configureDatabase(false)
+                ->addExtraDependencies('doctrine/orm:@stable')
+                // generate a migration first
+                ->addPreMakeCommand('php bin/console make:migration')
+                ->assert(function (string $output, string $directory) {
+                    $this->assertStringContainsString('You have 1 available migrations to execute', $output);
+                    $this->assertStringContainsString('Success', $output);
+                    $this->assertCount(14, explode("\n", $output), 'Asserting that very specific output is shown - some should be hidden');
+                }),
+            ];
 
-        yield 'migration_with_previous_migration_decline_question' => [MakerTestDetails::createTest(
-            $this->getMakerInstance(MakeMigration::class),
-            [
-                // no to confirm
-                'n',
-            ])
-            ->setFixtureFilesPath(__DIR__.'/../fixtures/MakeMigration')
-            ->configureDatabase(false)
-            ->addExtraDependencies('doctrine/orm:@stable')
-            // generate a migration first
-            ->addPreMakeCommand('php bin/console make:migration')
-            ->assert(function (string $output, string $directory) {
-                $this->assertStringNotContainsString('Success', $output);
-            }),
-        ];
+            yield 'migration_with_previous_migration_decline_question' => [MakerTestDetails::createTest(
+                $this->getMakerInstance(MakeMigration::class),
+                [
+                    // no to confirm
+                    'n',
+                ])
+                ->setFixtureFilesPath(__DIR__ . '/../fixtures/MakeMigration')
+                ->configureDatabase(false)
+                ->addExtraDependencies('doctrine/orm:@stable')
+                // generate a migration first
+                ->addPreMakeCommand('php bin/console make:migration')
+                ->assert(function (string $output, string $directory) {
+                    $this->assertStringNotContainsString('Success', $output);
+                }),
+            ];
+        }
     }
 }