Optimize sharing test connection

doctrine · Nov 15, 2021 · 06f5925 · 06f5925
1 parent 10df50f
commit 06f5925
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 7 deletions.
diff --git a/tests/Functional/Schema/MySQLSchemaManagerTest.php b/tests/Functional/Schema/MySQLSchemaManagerTest.php
@@ -21,11 +21,6 @@ public static function setUpBeforeClass(): void
         Type::addType('point', PointType::class);
     }
 
-    protected function tearDown(): void
-    {
-        $this->markConnectionNotReusable();
-    }
-
     protected function supportsPlatform(AbstractPlatform $platform): bool
     {
         return $platform instanceof MySQLPlatform;
@@ -106,6 +101,9 @@ public function testSpatialIndex(): void
 
         $this->schemaManager->dropAndCreateTable($table);
 
+        // see https://github.com/doctrine/dbal/issues/4983
+        $this->markConnectionNotReusable();
+
         $indexes = $this->schemaManager->listTableIndexes('spatial_index');
         self::assertArrayHasKey('s_index', $indexes);
         self::assertTrue($indexes['s_index']->hasFlag('spatial'));

diff --git a/tests/Functional/Schema/PostgreSQLSchemaManagerTest.php b/tests/Functional/Schema/PostgreSQLSchemaManagerTest.php
@@ -198,6 +198,8 @@ public function testFilterSchemaExpression(): void
         $testTable->addColumn('id', 'integer');
         $this->schemaManager->createTable($testTable);
 
+        $this->markConnectionNotReusable();
+
         $this->connection->getConfiguration()->setSchemaAssetsFilter(static function (string $name): bool {
             return preg_match('#^dbal204_#', $name) === 1;
         });

diff --git a/tests/Functional/Schema/SchemaManagerFunctionalTestCase.php b/tests/Functional/Schema/SchemaManagerFunctionalTestCase.php
@@ -78,8 +78,6 @@ protected function tearDown(): void
             $this->connection->executeStatement('DROP SCHEMA testschema');
         } catch (Exception $e) {
         }
-
-        $this->markConnectionNotReusable();
     }
 
     public function testDropAndCreateSequence(): void