fix cloning of DB connection for sqlite in-memory db

fixes #14131

Author: cebe

framework/db/Connection.php

@@ -1090,8 +1090,11 @@ public function __clone()
 
         $this->_master = false;
         $this->_slave = false;
-        $this->pdo = null;
         $this->_schema = null;
         $this->_transaction = null;
+        if (strncmp($this->dsn, 'sqlite::memory:', 15) === 0) {
+            // reset PDO connection, unless its sqlite in-memory, which can only have one connection
+            $this->pdo = null;
+        }
     }
 }

tests/framework/db/ConnectionTest.php

@@ -351,21 +351,36 @@ public function testClone()
         $this->assertNotNull($connection->pdo);
 
         $this->assertNull($conn2->transaction);
-        $this->assertNull($conn2->pdo);
+        if ($this->driverName === 'sqlite') {
+            // in-memory sqlite should not reset PDO
+            $this->assertNotNull($conn2->pdo);
+        } else {
+            $this->assertNull($conn2->pdo);
+        }
 
         $connection->beginTransaction();
 
         $this->assertNotNull($connection->transaction);
         $this->assertNotNull($connection->pdo);
 
         $this->assertNull($conn2->transaction);
-        $this->assertNull($conn2->pdo);
+        if ($this->driverName === 'sqlite') {
+            // in-memory sqlite should not reset PDO
+            $this->assertNotNull($conn2->pdo);
+        } else {
+            $this->assertNull($conn2->pdo);
+        }
 
         $conn3 = clone $connection;
 
         $this->assertNotNull($connection->transaction);
         $this->assertNotNull($connection->pdo);
         $this->assertNull($conn3->transaction);
-        $this->assertNull($conn3->pdo);
+        if ($this->driverName === 'sqlite') {
+            // in-memory sqlite should not reset PDO
+            $this->assertNotNull($conn3->pdo);
+        } else {
+            $this->assertNull($conn3->pdo);
+        }
     }
 }