Connection::setNestTransactionsWithSavepoints() should not break lazy…

… connection
doctrine · Apr 19, 2024 · b7a701d · b7a701d
1 parent edbf307
commit b7a701d
Show file tree

Hide file tree

Showing 2 changed files with 42 additions and 4 deletions.
diff --git a/src/Connection.php b/src/Connection.php
@@ -1317,10 +1317,6 @@ public function setNestTransactionsWithSavepoints($nestTransactionsWithSavepoint
             throw ConnectionException::mayNotAlterNestedTransactionWithSavepointsInTransaction();
         }
 
-        if (! $this->getDatabasePlatform()->supportsSavepoints()) {
-            throw ConnectionException::savepointsNotSupported();
-        }
-
         $this->nestTransactionsWithSavepoints = (bool) $nestTransactionsWithSavepoints;
     }
 

diff --git a/tests/ConnectionTest.php b/tests/ConnectionTest.php
@@ -2,12 +2,14 @@
 
 namespace Doctrine\DBAL\Tests;
 
+use BadMethodCallException;
 use Doctrine\Common\EventManager;
 use Doctrine\DBAL\Cache\QueryCacheProfile;
 use Doctrine\DBAL\Configuration;
 use Doctrine\DBAL\Connection;
 use Doctrine\DBAL\ConnectionException;
 use Doctrine\DBAL\Driver;
+use Doctrine\DBAL\Driver\API\ExceptionConverter;
 use Doctrine\DBAL\Driver\Connection as DriverConnection;
 use Doctrine\DBAL\Driver\ServerInfoAwareConnection;
 use Doctrine\DBAL\DriverManager;
@@ -81,6 +83,46 @@ public function testNoTransactionActiveByDefault(): void
         self::assertFalse($this->connection->isTransactionActive());
     }
 
+    public function testSetNestTransactionsWithSavepointsDoesNotConnect(): void
+    {
+        $this->expectNotToPerformAssertions();
+
+        $connection = new Connection(
+            [],
+            new class implements VersionAwarePlatformDriver {
+                /** {@inheritDoc} */
+                public function connect(array $params): DriverConnection
+                {
+                    throw new BadMethodCallException('The connection must not be opened');
+                }
+
+                public function getDatabasePlatform(): AbstractPlatform
+                {
+                    throw new BadMethodCallException('The connection must not be opened');
+                }
+
+                public function getSchemaManager(Connection $conn, AbstractPlatform $platform): AbstractSchemaManager
+                {
+                    throw new BadMethodCallException('The connection must not be opened');
+                }
+
+                public function getExceptionConverter(): ExceptionConverter
+                {
+                    throw new BadMethodCallException('The connection must not be opened');
+                }
+
+                /** {@inheritDoc} */
+                public function createDatabasePlatformForVersion($version): AbstractPlatform
+                {
+                    throw new BadMethodCallException('The connection must not be opened');
+                }
+            },
+            new Configuration(),
+        );
+
+        $connection->setNestTransactionsWithSavepoints(true);
+    }
+
     public function testCommitWithNoActiveTransactionThrowsException(): void
     {
         $this->expectException(ConnectionException::class);