[ticket/17176] Do not always create a new connection in database tests

PHPBB3-17176
This commit is contained in:
Marc Alexander 2023-08-21 18:46:41 +02:00
parent 8a2199ed6c
commit eed92e766f
No known key found for this signature in database
GPG key ID: 50E0D2423696F995
2 changed files with 3 additions and 3 deletions

View file

@ -431,8 +431,8 @@ class postgres extends \phpbb\db\driver\driver
*/ */
protected function _sql_close(): bool protected function _sql_close(): bool
{ {
// Released resources are already closed, return true in this case // Skip if connection is already closed or not persistent
if (!$this->db_connect_id instanceof \PgSql\Connection) if (!$this->persistency || !$this->db_connect_id instanceof \PgSql\Connection)
{ {
return true; return true;
} }

View file

@ -305,7 +305,7 @@ abstract class phpbb_database_test_case extends TestCase
/** @var \phpbb\db\driver\driver_interface $db */ /** @var \phpbb\db\driver\driver_interface $db */
$db = new $config['dbms'](); $db = new $config['dbms']();
$db->sql_connect($config['dbhost'], $config['dbuser'], $config['dbpasswd'], $config['dbname'], $config['dbport'], false, true); $db->sql_connect($config['dbhost'], $config['dbuser'], $config['dbpasswd'], $config['dbname'], $config['dbport']);
$this->db_connections[] = $db; $this->db_connections[] = $db;