Cover scenario `BEGIN TRANSACTION; COMMIT; BEGIN TRANSACTION --failure;

doctrine#6545 (comment)
simPod · Oct 20, 2024 · e21875a · e21875a
1 parent 378a9a4
commit e21875a
Showing 1 changed file with 30 additions and 0 deletions.
diff --git a/tests/ConnectionTest.php b/tests/ConnectionTest.php
@@ -32,6 +32,7 @@
 use PHPUnit\Framework\TestCase;
 use Psr\Cache\CacheItemInterface;
 use Psr\Cache\CacheItemPoolInterface;
+use RuntimeException;
 use stdClass;
 
 /** @requires extension pdo_mysql */
@@ -455,6 +456,35 @@ public function testCommitStartsTransactionInNoAutoCommitMode(): void
         self::assertTrue($conn->isTransactionActive());
     }
 
+    public function testBeginTransactionFailureAfterCommitInNoAutoCommitMode(): void
+    {
+        $driverConnectionMock = $this->createMock(DriverConnection::class);
+        $driverConnectionMock->expects(self::exactly(2))
+            ->method('beginTransaction')
+            ->willReturnOnConsecutiveCalls(
+                true,
+                self::throwException(new RuntimeException()),
+            );
+
+        $driverMock = $this->createMock(Driver::class);
+        $driverMock
+            ->method('connect')
+            ->willReturn(
+                $driverConnectionMock,
+            );
+        $conn = new Connection([], $driverMock);
+
+        $conn->setAutoCommit(false);
+
+        $conn->connect();
+        try {
+            $conn->commit();
+        } catch (RuntimeException $e) {
+        }
+
+        self::assertTrue($conn->isTransactionActive());
+    }
+
     /** @dataProvider resultProvider */
     public function testCommitReturn(bool $expectedResult): void
     {