我正在使用 Zend Framework 3 应用程序编写集成/数据库测试

• zendframework/zend-test 3.1.0,
• phpunit/phpunit 6.2.2，和
• phpunit/dbunit 3.0.0

由于

Connect Error: SQLSTATE[HY000] [1040] Too many connections

我设置了一些断点并查看了数据库:

SHOW STATUS WHERE `variable_name` = 'Threads_connected';

我实际上已经看到超过 100 个打开的连接。

我通过在 tearDown() 中断开连接来减少它们:

protected function tearDown()
{
    parent::tearDown();
    if ($this->dbAdapter && $this->dbAdapter instanceof Adapter) {
        $this->dbAdapter->getDriver()->getConnection()->disconnect();
    }
}

但我仍然有超过 80 打开的连接。

如何将测试中的数据库连接数减少到可能的最小值？

更多信息

(1) 我有很多测试，其中我 dispatch 一个 URI。每个这样的请求都会导致至少一个数据库请求，从而导致新的数据库连接。这些连接似乎没有关闭。这可能会导致最多的连接。 (但我还没有找到让应用程序在处理完请求后关闭连接的方法。)

(2) 其中一个问题可能是我对数据库的测试:

protected function retrieveActualData($table, $idColumn, $idValue)
{
    $sql = new Sql($this->dbAdapter);
    $select = $sql->select($table);
    $select->where([$table . '.' . $idColumn . ' = ?' => $idValue]);
    $statement = $sql->prepareStatementForSqlObject($select);
    $result = $statement->execute();
    $data = $result->current();
    return $data;
}

但是在 return 之前调用 $this->dbAdapter->getDriver()->getConnection()->disconnect() 什么也没做。

测试方法中的用法示例:

public function testInputDataActionSaving()
{
    // The getFormParams(...) returns an array with the needed input.
    $formParams = $this->getFormParams(self::FORM_CREATE_CLUSTER);

    $createWhateverUrl = '/whatever/create';
    $this->dispatch($createWhateverUrl, Request::METHOD_POST, $formParams);

    $this->assertEquals(
        $formParams['whatever']['some_param'],
        $this->retrieveActualData('whatever', 'id', 2)['some_param']
    );
}

(3) 另一个问题可能出在 PHPUnit(或者我对它的配置？)。(删除，因为“PHPUnit 不做任何与数据库连接相关的事情。”，参见 this 评论.) 无论如何，即使这不是 PHPUnit 问题，事实是，在该行之后

$testSuite = $configuration->getTestSuiteConfiguration($this->arguments['testsuite'] ?? null);

在 PHPUnit\TextUI\Command 中，我获得了 31 个新连接。

最佳答案

干净和适当的方法

如果“您的代码是以一种难以测试的方式编写的”，这似乎是一个问题。数据库连接应该由 DIC 或(在某些连接池的情况下)某些特殊类处理。基本上，包含 retrieveActualData() 的类应该将 Sql 实例作为构造函数中的依赖项传递。

相反，您的 Sql 类看起来是一个有害的 PDO 包装器，它(很可能)在您创建实例时建立了一个数据库连接。相反，您应该在多个类之间共享相同的 PDO 实例。这样您既可以控制建立的连接数量，又可以在(某些)隔离的情况下测试您的代码。

因此，主要解决方案是 - 您的代码很糟糕，但您可以清理它。

不要将新的 片段深入执行树中，而是将连接作为依赖项传递并共享它。

通过这种方式，您的测试可以转向使用各种模拟和 stub ，帮助您隔离测试结构。

在数据库绑定(bind)逻辑和 gremlins 的情况下

但是还有一个更实际的方面，您应该考虑。在集成测试中使用 SQLite 而不是真正的数据库。 PDO 支持该选项(您只需为测试代码提供不同的 DSN)。

如果您改用 SQLite 作为您的“测试数据库”，您将能够拥有一个定义明确的数据库状态(多个)，您可以根据这些状态测试您的代码。

您有类似文件 integration-002.db 的文件，其中包含准备好的数据库状态。在集成测试的 Bootstrap 中，您只需将准备好的 sqlite 数据库文件从 integration-0902.db 复制到 live-002.db 并运行所有测试。

use PHPUnit\Framework\TestCase;

final class CombinedTest extends TestCase
{
    public static function setUpBeforeClass()
    {
        copy(FIXTURE_PATH . '/integration-02.db', FIXTURE_PATH . '/live-02.db');
    }


    // your test go here

}

这样您将更好地控制您的持久性状态并且您的测试将运行得更快，因为不涉及网络堆栈。

您还可以准备任意数量的测试数据库，并在发现新错误时添加新的。这种方法可以让您在数据库中重新创建更复杂的场景，甚至可以模拟数据损坏。

您可以在 this 中看到这种方法的实际应用项目。

P.S. 个人经验 - 在集成测试中使用 SQLite 还可以提高 SQL 代码的总体质量(如果您不使用查询构建器，而是编写自定义数据映射器)。因为它迫使您考虑 SQLite 中可用功能与 MariaDB 或 PostgreSQL 之间的差异。但这是“你的里程可能会有所不同”的事情之一。

P.P.S. 您可以同时使用这两种建议的方法，因为它们只会相互促进。

关于php - 如何减少 PHPUnit 和 ZF3 测试中的数据库连接数？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/45651158/