tests: add test permutation with multiple migration configuration files

nikophil · nikophil · commit 490cf1af6220 · 2025-10-04T08:09:24.000+02:00
diff --git a/.env b/.env
@@ -11,3 +11,11 @@ USE_DAMA_DOCTRINE_TEST_BUNDLE="0"
 USE_FOUNDRY_PHPUNIT_EXTENSION="0"
 USE_PHP_84_LAZY_OBJECTS="0"
 PHPUNIT_VERSION="12" # allowed values: 9, 10, 11, 12
+
+# Only relevant for "reset-database" testsuite
+DATABASE_RESET_MODE="schema" # allowed values: schema, migrate
+
+# Only relevant for "reset-database" testsuite with DATABASE_RESET_MODE="migrate"
+MIGRATION_CONFIGURATION_FILES=[]
+#MIGRATION_CONFIGURATION_FILES=["tests/Fixture/ResetDatabase/migration-configs/migration-configuration.php"]
+#MIGRATION_CONFIGURATION_FILES=["tests/Fixture/ResetDatabase/migration-configs/migration-configuration-transactional.php", "tests/Fixture/ResetDatabase/migration-configs/migration-dummy.php"]
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -120,19 +120,25 @@ jobs:
         database: [ mysql, pgsql, sqlite, mysql|mongo ]
         use-dama: [ 0, 1 ]
         reset-database-mode: [ schema, migrate ]
-        migration-configuration-file: ['no']
+        migration-configuration-file: ['[]']
         deps: [ highest, lowest ]
         include:
-          - { database: mongo, migration-configuration-file: 'no', use-dama: 0, reset-database-mode: schema }
-          - { database: pgsql, migration-configuration-file: 'migration-configuration', use-dama: 0, reset-database-mode: migration }
-          - { database: pgsql, migration-configuration-file: 'migration-configuration-transactional', use-dama: 0, reset-database-mode: migration }
+          - { database: mongo, migration-configuration-file: '[]', use-dama: 0, reset-database-mode: schema }
+          - { database: pgsql, migration-configuration-file: '["tests/Fixture/ResetDatabase/migration-configs/migration-configuration.php"]', use-dama: 0, reset-database-mode: migrate }
+          - { database: pgsql, migration-configuration-file: '["tests/Fixture/ResetDatabase/migration-configs/migration-configuration-transactional.php"]', use-dama: 0, reset-database-mode: migrate }
+          -
+            database: pgsql
+            use-dama: 0
+            reset-database-mode: migrate
+            migration-configuration-file: '["tests/Fixture/ResetDatabase/migration-configs/migration-configuration-transactional.php", "tests/Fixture/ResetDatabase/migration-configs/migration-dummy.php"]'
+            deps: highest
     env:
       DATABASE_URL: ${{ contains(matrix.database, 'mysql') && 'mysql://root:root@localhost:3306/foundry?serverVersion=5.7.42' || contains(matrix.database, 'pgsql') && 'postgresql://root:root@localhost:5432/foundry?serverVersion=15' || 'sqlite:///%kernel.project_dir%/var/data.db' }}
       MONGO_URL: ${{ contains(matrix.database, 'mongo') && 'mongodb://127.0.0.1:27017/dbName?compressors=disabled&amp;gssapiServiceName=mongodb' || '' }}
       USE_DAMA_DOCTRINE_TEST_BUNDLE: ${{ matrix.use-dama == 1 && 1 || 0 }}
-      DATABASE_RESET_MODE: ${{ matrix.reset-database-mode == 1 && 1 || 0 }}
-      MIGRATION_CONFIGURATION_FILE: ${{ matrix.migration-configuration-file == 'no' && '' || format('tests/Fixture/MigrationTests/configs/{0}.php', matrix.migration-configuration-file) }}
-      PHPUNIT_VERSION: 11
+      DATABASE_RESET_MODE: ${{ matrix.reset-database-mode }}
+      MIGRATION_CONFIGURATION_FILES: ${{ matrix.migration-configuration-file }}
+      PHPUNIT_VERSION: 12
     services:
       postgres:
         image: ${{ contains(matrix.database, 'pgsql') && 'postgres:15' || '' }}
@@ -180,7 +186,7 @@ jobs:
 
           # We should be able to run the tests twice in order to check if the second run also starts from a fresh db
           # some bugs could be detected this way
-          ./phpunit --testsuite reset-database --bootstrap tests/bootstrap-reset-database.php
+          DATABASE_GENERATE_MIGRATIONS=0 ./phpunit --testsuite reset-database --bootstrap tests/bootstrap-reset-database.php
         shell: bash
 
   test-no-framework:
@@ -293,7 +299,7 @@ jobs:
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: 8.2
+          php-version: 8.4
           coverage: xdebug
           ini-values: xdebug.mode=coverage
 
diff --git a/README.md b/README.md
@@ -52,7 +52,7 @@ $ composer update
 $ ./phpunit
 
 # run "migrate" testsuite (with "migrate" reset database strategy)
-$ ./phpunit --testsuite reset-database
+$ ./phpunit --testsuite reset-database --bootstrap tests/bootstrap-reset-database.php
 ```
 
 ### Overriding the default configuration
@@ -73,7 +73,7 @@ PHPUNIT_VERSION="11" # possible values: 9, 10, 11, 11.4
 
 # test reset database with migrations,
 # only relevant for "reset-database" testsuite
-MIGRATION_CONFIGURATION_FILE="tests/Fixture/MigrationTests/configs/migration-configuration.php"
+MIGRATION_CONFIGURATION_FILES="[\"tests/Fixture/ResetDatabase/migration-configs/migration-configuration.php\"]"
 
 # run test suite with postgreSQL
 $ vendor/bin/phpunit
diff --git a/composer.json b/composer.json
@@ -33,6 +33,7 @@
         "doctrine/common": "^3.2.2",
         "doctrine/doctrine-bundle": "^2.10",
         "doctrine/doctrine-migrations-bundle": "^2.2|^3.0",
+        "doctrine/migrations": "dev-feat/unfroze-dependencies-after-command as 3.9.5",
         "doctrine/mongodb-odm": "^2.4",
         "doctrine/mongodb-odm-bundle": "^4.6|^5.0",
         "doctrine/orm": "^2.16|^3.0",
@@ -64,6 +65,12 @@
             "src/symfony_console.php"
         ]
     },
+    "repositories": [
+        {
+            "type": "vcs",
+            "url": "https://github.com/nikophil/migrations"
+        }
+    ],
     "autoload-dev": {
         "psr-4": {
             "Zenstruck\\Foundry\\Tests\\": ["tests/"],
diff --git a/tests/Fixture/DoctrineCascadeRelationship/ChangesEntityRelationshipCascadePersist.php b/tests/Fixture/DoctrineCascadeRelationship/ChangesEntityRelationshipCascadePersist.php
@@ -38,6 +38,9 @@ trait ChangesEntityRelationshipCascadePersist
 {
     use RequiresORM;
 
+    // Because we're handling the provided data manually in a #[Before] method,
+    // PHPUnit triggers a warning, which we ignore with #[IgnorePhpunitWarnings]
+
     private static string $methodName = '';
 
     #[Before]
diff --git a/tests/Fixture/DoctrineCascadeRelationship/DoctrineCascadeRelationshipMetadata.php b/tests/Fixture/DoctrineCascadeRelationship/DoctrineCascadeRelationshipMetadata.php
@@ -56,7 +56,7 @@ public static function allCombinations(array $relationshipFields): iterable
         if (!\getenv('USE_DAMA_DOCTRINE_TEST_BUNDLE')) {
             $metadata = self::fromArray($relationshipFields[0]);
 
-            yield "{$metadata}\n" => [$metadata];
+            yield "{$metadata}" => [$metadata];
 
             return;
         }
diff --git a/tests/Fixture/FoundryTestKernel.php b/tests/Fixture/FoundryTestKernel.php
@@ -60,11 +60,6 @@ public static function hasMongo(): bool
         return (bool) \getenv('MONGO_URL');
     }
 
-    public static function usesMigrations(): bool
-    {
-        return 'migrate' === \getenv('DATABASE_RESET_MODE');
-    }
-
     public static function usesDamaDoctrineTestBundle(): bool
     {
         return (bool) \getenv('USE_DAMA_DOCTRINE_TEST_BUNDLE');
diff --git a/tests/Fixture/ResetDatabase/DummyMigration.php b/tests/Fixture/ResetDatabase/DummyMigration.php
@@ -0,0 +1,19 @@
+<?php
+
+namespace Zenstruck\Foundry\Tests\Fixture\ResetDatabase;
+
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+final class DummyMigration extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return 'Dummy migration to test migration reset with multiple configuration files.';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->addSql('INSERT INTO address (city) VALUES (\'city\')');
+    }
+}
diff --git a/tests/Fixture/ResetDatabase/ResetDatabaseTestKernel.php b/tests/Fixture/ResetDatabase/ResetDatabaseTestKernel.php
@@ -28,7 +28,7 @@ public function registerBundles(): iterable
     {
         yield from parent::registerBundles();
 
-        if (FoundryTestKernel::usesMigrations()) {
+        if (self::usesMigrations()) {
             yield new DoctrineMigrationsBundle();
         }
     }
@@ -38,23 +38,25 @@ protected function configureContainer(ContainerBuilder $c, LoaderInterface $load
         parent::configureContainer($c, $loader);
 
         $c->loadFromExtension('zenstruck_foundry', [
+            'persistence' => ['flush_once' => true],
+            'enable_auto_refresh_with_lazy_objects' => self::usePHP84LazyObjects(),
             'global_state' => [
                 GlobalStory::class,
                 GlobalInvokableService::class,
             ],
             'orm' => [
-                'reset' => FoundryTestKernel::usesMigrations()
+                'reset' => self::usesMigrations()
                         ? [
                             'mode' => ResetDatabaseMode::MIGRATE,
                             'migrations' => [
-                                'configurations' => ($configFile = \getenv('MIGRATION_CONFIGURATION_FILE')) ? [$configFile] : [],
+                                'configurations' => ($configFile = self::migrationFiles()) ? $configFile : [],
                             ],
                         ]
                         : ['mode' => ResetDatabaseMode::SCHEMA],
             ],
         ]);
 
-        if (FoundryTestKernel::usesMigrations() && !\getenv('MIGRATION_CONFIGURATION_FILE')) {
+        if (self::usesMigrations() && self::migrationFiles() === []) {
             // if no configuration file was given in Foundry's config, let's use the main one as default.
             $c->loadFromExtension(
                 'doctrine_migrations',
@@ -76,4 +78,34 @@ protected function configureContainer(ContainerBuilder $c, LoaderInterface $load
             $c->register(MongoResetterDecorator::class)->setAutowired(true)->setAutoconfigured(true);
         }
     }
+
+    public static function usesMigrations(): bool
+    {
+        return 'migrate' === \getenv('DATABASE_RESET_MODE');
+    }
+
+    public static function shouldGenerateMigrations(): bool
+    {
+        return '0' !== \getenv('DATABASE_GENERATE_MIGRATIONS');
+    }
+
+    /**
+     * @return list<string>
+     */
+    public static function migrationFiles(): array
+    {
+        $files = json_decode(\getenv('MIGRATION_CONFIGURATION_FILES')?: '[]', flags: JSON_THROW_ON_ERROR);
+
+        if (!\is_array($files)) {
+            throw new \InvalidArgumentException('MIGRATION_CONFIGURATION_FILES must be a JSON array.');
+        }
+
+        foreach ($files as $file) {
+            if (!\file_exists($file)) {
+                throw new \InvalidArgumentException(\sprintf('Migration configuration file "%s" does not exist.', $file));
+            }
+        }
+
+        return $files; // @phpstan-ignore return.type
+    }
 }
diff --git a/tests/Fixture/ResetDatabase/migration-configs/migration-configuration-transactional.php b/tests/Fixture/ResetDatabase/migration-configs/migration-configuration-transactional.php
@@ -11,7 +11,7 @@
 
 return [
     'migrations_paths' => [
-        'Zenstruck\\Foundry\\Tests\\Fixture\\ResetDatabase\\Migrations' => \dirname(__DIR__, 4).'/var/cache/Migrations',
+        'Zenstruck\\Foundry\\Tests\\Fixture\\ResetDatabase\\Migrations' => \dirname(__DIR__, 4).'/var/Migrations',
     ],
     'transactional' => true,
 ];
diff --git a/tests/Fixture/ResetDatabase/migration-configs/migration-configuration.php b/tests/Fixture/ResetDatabase/migration-configs/migration-configuration.php
@@ -11,7 +11,7 @@
 
 return [
     'migrations_paths' => [
-        'Zenstruck\\Foundry\\Tests\\Fixture\\ResetDatabase\\Migrations' => \dirname(__DIR__, 4).'/var/cache/Migrations',
+        'Zenstruck\\Foundry\\Tests\\Fixture\\ResetDatabase\\Migrations' => \dirname(__DIR__, 4).'/var/Migrations',
     ],
     'transactional' => false,
 ];
diff --git a/tests/Fixture/ResetDatabase/migration-configs/migration-dummy.php b/tests/Fixture/ResetDatabase/migration-configs/migration-dummy.php
@@ -0,0 +1,17 @@
+<?php
+
+/*
+ * This file is part of the zenstruck/foundry package.
+ *
+ * (c) Kevin Bond <kevinbond@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+return [
+    'migrations_paths' => [
+        'Zenstruck\\Foundry\\Tests\\Fixture\\ResetDatabase' => \dirname(__DIR__),
+    ],
+    'transactional' => false,
+];
diff --git a/tests/Integration/ORM/AutoRefreshTest.php b/tests/Integration/ORM/AutoRefreshTest.php
@@ -35,7 +35,6 @@
 use Zenstruck\Foundry\Tests\Fixture\Factories\Entity\GenericEntityFactory;
 use Zenstruck\Foundry\Tests\Integration\Persistence\AutoRefreshTestCase;
 use Zenstruck\Foundry\Tests\Integration\RequiresORM;
-
 use function Zenstruck\Foundry\Persistence\persistent_factory;
 use function Zenstruck\Foundry\Persistence\refresh_all;
 
@@ -97,7 +96,7 @@ public function tracker_keeps_reference_only_for_objects_in_current_scope(): voi
     }
 
     #[Test]
-    #[IgnorePhpunitWarnings(EdgeCasesRelationshipTest::DATA_PROVIDER_WARNING_REGEX)]
+    #[IgnorePhpunitWarnings(EntityRelationship\EntityFactoryRelationshipTestCase::DATA_PROVIDER_WARNING_REGEX)]
     #[UsingRelationships(Contact::class, ['category', 'address'])]
     #[DataProvider('provideCascadeRelationshipsCombinations')]
     public function it_can_refresh_objects_with_relationships(): void
diff --git a/tests/Integration/ORM/EdgeCasesRelationshipTest.php b/tests/Integration/ORM/EdgeCasesRelationshipTest.php
@@ -35,7 +35,6 @@
 use Zenstruck\Foundry\Tests\Fixture\Entity\EdgeCases\RichDomainMandatoryRelationship;
 use Zenstruck\Foundry\Tests\Fixture\Factories\Entity\EdgeCases\MultipleMandatoryRelationshipToSameEntity;
 use Zenstruck\Foundry\Tests\Integration\RequiresORM;
-
 use function Zenstruck\Foundry\Persistence\persistent_factory;
 
 /**
@@ -45,16 +44,12 @@ final class EdgeCasesRelationshipTest extends KernelTestCase
 {
     use ChangesEntityRelationshipCascadePersist, Factories, RequiresORM, ResetDatabase;
 
-    // Because we're handling the provided data manually in a #[Before] method,
-    // PHPUnit triggers a warning, which we ignore with #[IgnorePhpunitWarnings]
-    public const DATA_PROVIDER_WARNING_REGEX = 'Data set "(.)*" provided by Zenstruck(.)*::provideCascadeRelationshipsCombinations has more arguments \(\d\) than the test method accepts \(0\)';
-
     /** @test */
     #[Test]
     #[DataProvider('provideCascadeRelationshipsCombinations')]
     #[UsingRelationships(RichDomainMandatoryRelationship\OwningSide::class, ['main'])]
     #[RequiresPhpunit('>=11.4')]
-    #[IgnorePhpunitWarnings(self::DATA_PROVIDER_WARNING_REGEX)]
+    #[IgnorePhpunitWarnings(EntityRelationship\EntityFactoryRelationshipTestCase::DATA_PROVIDER_WARNING_REGEX)]
     public function inversed_relationship_mandatory(): void
     {
         $owningSideEntityFactory = persistent_factory(RichDomainMandatoryRelationship\OwningSide::class);
@@ -74,7 +69,7 @@ public function inversed_relationship_mandatory(): void
     #[DataProvider('provideCascadeRelationshipsCombinations')]
     #[UsingRelationships(InversedOneToOneWithNonNullableOwning\OwningSide::class, ['inverseSide'])]
     #[RequiresPhpunit('>=11.4')]
-    #[IgnorePhpunitWarnings(self::DATA_PROVIDER_WARNING_REGEX)]
+    #[IgnorePhpunitWarnings(EntityRelationship\EntityFactoryRelationshipTestCase::DATA_PROVIDER_WARNING_REGEX)]
     public function inverse_one_to_one_with_non_nullable_inverse_side(): void
     {
         $owningSideFactory = persistent_factory(InversedOneToOneWithNonNullableOwning\OwningSide::class);
@@ -93,7 +88,7 @@ public function inverse_one_to_one_with_non_nullable_inverse_side(): void
     #[DataProvider('provideCascadeRelationshipsCombinations')]
     #[UsingRelationships(InversedOneToOneWithSetter\OwningSide::class, ['inverseSide'])]
     #[RequiresPhpunit('>=11.4')]
-    #[IgnorePhpunitWarnings(self::DATA_PROVIDER_WARNING_REGEX)]
+    #[IgnorePhpunitWarnings(EntityRelationship\EntityFactoryRelationshipTestCase::DATA_PROVIDER_WARNING_REGEX)]
     public function inverse_one_to_one_with_both_nullable(): void
     {
         $owningSideFactory = persistent_factory(InversedOneToOneWithSetter\OwningSide::class);
@@ -113,7 +108,7 @@ public function inverse_one_to_one_with_both_nullable(): void
     #[UsingRelationships(InversedOneToOneWithOneToMany\OwningSide::class, ['inverseSide'])]
     #[UsingRelationships(InversedOneToOneWithOneToMany\Item::class, ['owningSide'])]
     #[RequiresPhpunit('>=11.4')]
-    #[IgnorePhpunitWarnings(self::DATA_PROVIDER_WARNING_REGEX)]
+    #[IgnorePhpunitWarnings(EntityRelationship\EntityFactoryRelationshipTestCase::DATA_PROVIDER_WARNING_REGEX)]
     public function inverse_one_to_one_with_one_to_many(): void
     {
         $inverseSideFactory = persistent_factory(InversedOneToOneWithOneToMany\InverseSide::class);
@@ -154,7 +149,7 @@ public function many_to_many_to_self_referencing_inverse_side(): void
     #[DataProvider('provideCascadeRelationshipsCombinations')]
     #[UsingRelationships(IndexedOneToMany\ParentEntity::class, ['items'])]
     #[RequiresPhpunit('>=11.4')]
-    #[IgnorePhpunitWarnings(self::DATA_PROVIDER_WARNING_REGEX)]
+    #[IgnorePhpunitWarnings(EntityRelationship\EntityFactoryRelationshipTestCase::DATA_PROVIDER_WARNING_REGEX)]
     public function indexed_one_to_many(): void
     {
         $parentFactory = persistent_factory(IndexedOneToMany\ParentEntity::class);
@@ -177,7 +172,7 @@ public function indexed_one_to_many(): void
     #[DataProvider('provideCascadeRelationshipsCombinations')]
     #[UsingRelationships(InversedOneToOneWithManyToOne\InverseSide::class, ['owningSide', 'item'])]
     #[RequiresPhpunit('>=11.4')]
-    #[IgnorePhpunitWarnings(self::DATA_PROVIDER_WARNING_REGEX)]
+    #[IgnorePhpunitWarnings(EntityRelationship\EntityFactoryRelationshipTestCase::DATA_PROVIDER_WARNING_REGEX)]
     public function inversed_one_to_one_can_be_used_after_other_relationship(): void
     {
         $inverseSideFactory = persistent_factory(InversedOneToOneWithManyToOne\InverseSide::class);
@@ -206,7 +201,7 @@ public function inversed_one_to_one_can_be_used_after_other_relationship(): void
     #[DataProvider('provideCascadeRelationshipsCombinations')]
     #[UsingRelationships(InversedOneToOneWithoutAutoGeneratedId\OwningSide::class, ['inverseSide'])]
     #[RequiresPhpunit('>=11.4')]
-    #[IgnorePhpunitWarnings(self::DATA_PROVIDER_WARNING_REGEX)]
+    #[IgnorePhpunitWarnings(EntityRelationship\EntityFactoryRelationshipTestCase::DATA_PROVIDER_WARNING_REGEX)]
     public function inverse_one_to_one_with_custom_id(): void
     {
         $owningSideFactory = persistent_factory(InversedOneToOneWithoutAutoGeneratedId\OwningSide::class);
@@ -225,7 +220,7 @@ public function inverse_one_to_one_with_custom_id(): void
     #[DataProvider('provideCascadeRelationshipsCombinations')]
     #[UsingRelationships(OneToManyWithUnionType\OwningSideEntity::class, ['item'])]
     #[RequiresPhpunit('>=11.4')]
-    #[IgnorePhpunitWarnings(self::DATA_PROVIDER_WARNING_REGEX)]
+    #[IgnorePhpunitWarnings(EntityRelationship\EntityFactoryRelationshipTestCase::DATA_PROVIDER_WARNING_REGEX)]
     public function after_instantiate_flushing_using_current_object_in_relationship_one_to_one(): void
     {
         $owningSideFactory = persistent_factory(OneToManyWithUnionType\OwningSideEntity::class);
@@ -283,7 +278,7 @@ public function inverse_one_to_one_without_types_throws(): void
     #[DataProvider('provideCascadeRelationshipsCombinations')]
     #[UsingRelationships(InversedOneToOneWithNonNullableOwning\OwningSide::class, ['inverseSide'])]
     #[RequiresPhpunit('>=11.4')]
-    #[IgnorePhpunitWarnings(self::DATA_PROVIDER_WARNING_REGEX)]
+    #[IgnorePhpunitWarnings(EntityRelationship\EntityFactoryRelationshipTestCase::DATA_PROVIDER_WARNING_REGEX)]
     public function after_instantiate_flushing_using_current_object_in_relationship_inverse_one_to_one(): void
     {
         $owningSideFactory = persistent_factory(InversedOneToOneWithNonNullableOwning\OwningSide::class);
diff --git a/tests/Integration/ORM/EntityRelationship/EntityFactoryRelationshipTestCase.php b/tests/Integration/ORM/EntityRelationship/EntityFactoryRelationshipTestCase.php
diff --git a/tests/Integration/ORM/EntityRelationship/ProxyEntityFactoryRelationshipTest.php b/tests/Integration/ORM/EntityRelationship/ProxyEntityFactoryRelationshipTest.php
diff --git a/tests/Integration/ResetDatabase/OrmEdgeCaseTest.php b/tests/Integration/ResetDatabase/OrmEdgeCaseTest.php
diff --git a/tests/Integration/ResetDatabase/ResetDatabaseTest.php b/tests/Integration/ResetDatabase/ResetDatabaseTest.php
diff --git a/tests/bootstrap-reset-database.php b/tests/bootstrap-reset-database.php

Original file line number	Diff line number	Diff line change
`@@ -38,6 +38,9 @@ trait ChangesEntityRelationshipCascadePersist`
`38`	`38`	`{`
`39`	`39`	`use RequiresORM;`
`40`	`40`
	`41`	`+ // Because we're handling the provided data manually in a #[Before] method,`
	`42`	`+ // PHPUnit triggers a warning, which we ignore with #[IgnorePhpunitWarnings]`
	`43`	`+`
`41`	`44`	`private static string $methodName = '';`
`42`	`45`
`43`	`46`	`#[Before]`
Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ public static function allCombinations(array $relationshipFields): iterable`
`56`	`56`	`if (!\getenv('USE_DAMA_DOCTRINE_TEST_BUNDLE')) {`
`57`	`57`	`$metadata = self::fromArray($relationshipFields[0]);`
`58`	`58`
`59`		`- yield "{$metadata}\n" => [$metadata];`
	`59`	`+ yield "{$metadata}" => [$metadata];`
`60`	`60`
`61`	`61`	`return;`
`62`	`62`	`}`
Original file line number	Diff line number	Diff line change
`@@ -60,11 +60,6 @@ public static function hasMongo(): bool`
`60`	`60`	`return (bool) \getenv('MONGO_URL');`
`61`	`61`	`}`
`62`	`62`
`63`		`- public static function usesMigrations(): bool`
`64`		`- {`
`65`		`- return 'migrate' === \getenv('DATABASE_RESET_MODE');`
`66`		`- }`
`67`		`-`
`68`	`63`	`public static function usesDamaDoctrineTestBundle(): bool`
`69`	`64`	`{`
`70`	`65`	`return (bool) \getenv('USE_DAMA_DOCTRINE_TEST_BUNDLE');`