在phpunit中测试构造函数参数


Testing constructor args in phpunit

我们使用Varien_Http_Client从Magento扩展发出http请求,如下所示:

public function callApi(…)
{
    <SNIP>
    // Set default factory adapter to socket in case curl isn't installed.
    $client = new Varien_Http_Client($apiUri, array(
        'adapter' => 'Zend_Http_Client_Adapter_Socket',
        'timeout' => $timeout, 
    ));
    $client->setRawData($xmlDoc->saveXML())->setEncType('text/xml');
    $response = $client->request($method);
    $results = '';
    if ($response->isSuccessful()) {
        $results = $response->getBody();
    }
    return $results;
}

我知道我应该避免测试Varien_Http_Client的内部;相反,我应该测试我们是否向它发送了正确的输入,并正确地处理了它的输出。我可以很容易地模拟Varien_Http_Client,但即使我重构这段代码,让我用它的mock替换Varien_Http_Client,我也不明白如何测试构造函数是用预期的参数调用的,因为构造函数是由PHPUnit::getMock调用的。

我不需要模拟对象;我需要一个模拟。如何测试是否使用预期的参数调用了构造函数?

*(在这种情况下,我知道解决Varien_Http_Client特定问题的方法,但我能用更不透明的第三方代码做什么?)

这就是我们所说的"不稳定"代码。当您在方法中构建依赖项时,没有办法模拟它们。在模型中每次使用"new"关键字都是一个信号,表明您应该考虑注入对象,而不是在内部创建对象。在我看来,该规则的唯一例外是创建"数据容器"对象或工厂类。但在这些情况下,你可能可以测试对象,因为方法会返回它

正如您所说,您展示的方法需要进行一些重构,例如:

class ApiClass
{
    protected $client;
    public function __construct(Varien_Http_Client $client)
    {
        $this->client = $client;
    }
    public function callApi()
    {
        $this->client->setRawData($xmlDoc->saveXML())->setEncType('text/xml');
        (...)

最好!