我怎样才能改进这个功能的单元测试

Question

我想知道是否还有其他单元测试（我的意思是我可以在不依赖测试数据库的情况下进行的其他单元测试）我可以为函数 getAll 在下面 class:

<?php
namespace Example\Model;

use Example\Lib\PdoConnection;

class UserModel {

    private $pdoConn;

    function __construct()
    {
        $this->pdoConn = PdoConnection::getInstance();
    }

    function getAll()
    {
        $arrUsers = array();
        $strSql = "SELECT id, first_name, last_name FROM user";
        $arrData = array();
        try
        {
            $objRes = $this->pdoConn->prepare($strSql);
            $objRes->execute($arrData);
            $objRes->setFetchMode(\PDO::FETCH_ASSOC);
            $arrUsers = $objRes->fetchAll();
        }
        catch(\PDOException $e) {
            error_log($e->getMessage());
        }
        return $arrUsers;
    }

以下是我的测试方式：

function testGetAll()
{   
    $stubUserModel = $this->getMockBuilder('Example\Model\UserModel')
    ->disableOriginalConstructor()
    ->getMock();

    $stubUserModel->method('getAll')
    ->willReturn(array(array('id' => 1, 'first_name' => 'First1', 'last_name' => 'Last1'), array('id' => 2, 'first_name' => 'First2', 'last_name' => 'Last2')));

    $this->assertEquals(array(array('id' => 1, 'first_name' => 'First1', 'last_name' => 'Last1'), array('id' => 2, 'first_name' => 'First2', 'last_name' => 'Last2')), $stubUserModel->getAll());       
}

我知道它需要更多的测试才能被认为是良好的单元测试，而我上面写的测试是无用的。

我发现它很难测试，因为它依赖于 class PDOConnection 以及准备语句返回的 class（本例中实例化的 obj 是 $objResult）。这是否意味着我必须重构代码并将 PDOConnection 依赖项移到此函数之外，并可能将 $objRes 作为参数传递给 getAll 函数才能模拟它？或者有另一种更便宜的方法使其可测试？

欢迎所有建议。

谢谢

Answer 1

由于依赖注入，您应该将 pdo 连接作为构造函数参数传递。然后您可以考虑模拟连接并确保使用预期参数调用预期方法的预期时间。您可以使用 Mockery、Prophecy 或 phpunit 模拟对象等库。那么这不是一个模型，而是一个提供者或存储库。所以你应该称它为 UserProvider 或 UserRepository

我怎样才能改进这个功能的单元测试

How can I improve unit testing this function

php

phpunit

unit-testing