0

目前我正在测试一个基于值对象生成 XML、通过 HTTP 发送 XML 并将 XML 响应解析回第二个值对象的类。在这种情况下，我想测试生成的 XML 和基于给定 XML 解析的值对象。

该类看起来像：

class MyClient
{
    public function send(RequestValues $request)
    {
        $document = $this->generateMessage($request);
        $response = $this->request($document);

        return $this->parseResponse($response);
    }

    protected function generateMessage(RequestValues $request)
    {
        $document = new DomDocument;
        // Do stuff with $request

        return $document;
    }

    public function request(DomDocument $document)
    {
        $client = $this->getHttpClient();
        $client->setRawBody($document->saveXml());
        // Configure client

        return $client->send();
    }

    public function parseResponse(Response $response)
    {
        $parameters = new ResponseValues;

        $document = new DomDocument;
        $document->loadXml($response->getBody());

        // Fill in $parameters
        return $parameters;
    }
}

我想测试两件事：

  1. 给定某个RequestValues参数，生成的 XML 必须看起来像$string
  2. 给定一个特定的 XML 响应值（HTTP 客户端将被模拟），ResponseValues必须等于$object

我现在正在为#1 编写测试，但我认为我只能通过回调来实现这一点。然而，当测试失败时，回调并没有给我非常有用的信息。只有这条消息：

断言 DOMDocument Object () 被指定的回调接受失败。

测试看起来像这样：

public function testRequestContainsValidXml()
{
    $client  = $this->getMock('MyClient', array('request'));

    $message = '';
    $client->expects($this->once())
           ->method('request')
           ->with($this->callback(function($object) use ($message) {
                return
                    ($object instanceof DomDocument)
                 && ($object->saveXml() === $message);
            }));

    $request = new DirectoryRequest;
    $client->send($request);
}

问题是：如何改进测试以便可以进行正常的字符串比较？我很想让 phpunit 说“字符串 X 不等于 Y”，这极大地简化了调试。

PS。这个类的完整代码可以在 GitHub 上找到。当然，上面的例子是一个简化版本。这是实际的课程：https ://github.com/juriansluiman/SlmIdealPayment/blob/master/src/SlmIdealPayment/Client/StandardClient.php#L58

聚苯乙烯。如果必须更改代码才能对其进行测试，那不是问题。我只想保持公共 API 相同（即调用ResponseValues send(RequestValues $request)

4

1 回答 1

0

你的班级MyClient暗示它是一个门面。要在 UnitTests 的上下文中测试外观，通常首先需要测试作为该外观的协作者的所有单元。

正如您通常模拟协作者而不是被测单元一样，您的测试设置看起来错误，因为您正在模拟被测MyClient单元而不是协作者。

例如：

您想测试该MyClient::parseResponse()方法是否返回预期的ResponseValues对象。因此，您会嘲笑它，Response因为在这种情况下它是一个合作者。

您可能还想模拟其他协作者（ResponseValues），但是您不能，因为它是一个无法注入的隐藏依赖项（您可以通过注入一个MyClient可以控制此类创建的工厂来解决此问题ResponseValues）。

测试就这么多MyClient::parseResponse()：您将通过Response模拟注入夹具 XML 并在返回值上运行您的断言。

对于测试请求是否包含有效的 XML ( testRequestContainsValidXml()) 的案例，我认为也不应该以那种复杂的形式完成。听起来您在这里有一个 HTTP 客户端，您应该只测试它是否工作，而不关心它是否也适用于 XML，因为如果它工作，它也适用于 XML。您测试有效 XML 的原因不是因为您想测试客户端。因此，请将该测试排除在客户端单元测试之外。

顺便说一句，字符串的断言。是：

$this->assertSame($expected, $actual);

Phpunit 将向您展示字符串之间的一个很好的差异。其他一些程序员/测试人员在比较 XML 时也应用了一些 XML 规范化，以使差异更具可读性。如果您的 XML 在处理无关紧要的空白方面没有问题，您可能会发现类似PHP XML 如何输出良好格式和相关有用的问答。

我希望我能稍微指出您面临的问题案例，并扩大您对问题的看法。我认为您在批判性地质疑您当前的模拟使用方面走得最远，对于我给出的示例，我已将模拟完全减少为提供协作者，而不是进行断言。断言仅在被测单元上完成，这里是示例性的MyClient.

于 2013-09-27T14:32:42.203 回答