Mockery\Exception\NoMatchingExpectationException PHPUnit测试失败

Question

我正在使用PHPUnit进行测试，但我在一个函数上的测试失败了。但我不知道为什么。我想模拟的函数：

public function subscribe($email)
{
    $message = new SubscribeMessage();
    $message->setEmailaddress($email);
    $message->setLocale(Locale::getDefault());
    $this->getAmqpProducer()->publish($message, 'newsletter-subscribe');
    return true;
}

和我的单元测试：

public function testSubscribeSendsAmqpMessage()
{
    $email = 'email@email.nl';
    $locale = 'nl';

    $this->amqpProducerMock
        ->shouldReceive('publish')
        ->once()
        ->with(

            \Mockery::on(
                function ($message, $routingkey) use (&$publishedMessage) {
                    $publishedMessage = $message;
                    return $routingkey == 'newsletter-subscribe';
                }
            )
        );

    $this->service->subscribe($email, $locale);
}

但测试结果显示：

Mockery\Exception\NoMatchingExpectationException : No matching handler found for AcsiRabbitMq\Producer\Producer::publish(AcsiNewsletter\RabbitMq\Message\SubscribeMessage, "newsletter-subscribe"). Either the method was unexpected or its arguments matched no expected argument list for this method

我如何修复我的单元测试？或者我如何重构我的测试？

Answer 1

您可以模拟订阅，而不是内部发布。当您运行测试并调用->subscribe时，它将尝试执行类中的代码。因此，它将尝试运行subscribe()方法，您似乎对Mock有一个奇怪的引用。

通常，您的测试将模拟订阅，因此您可以为断言测试返回一个值，该值是硬编码的。

您似乎试图模拟常规代码中的GetAmqpProducer()对象。您需要能够将要使用的模拟对象传递到您的类中，或者能够为其赋值。

简化示例：

class Email
{
    private $MsgObject;

    // Constructor Injection
    public __construct(SubscribeMessage $MessageObject)
    {
        $this->MsgObject = $MessageObject;
        ...
    }

    // Setter Injection
    public function SetSubscribeMessage(Subscribe $MessageObject)
    {
        $this->MsgObject = $MessageObject;
    }   

    public function setEmailaddress($email)
    {
        $this->MsgObject->emailAddress = $email;
        ...
    }

    public function setLocale($Locale)
    {
        $this->MsgObject->Locale = $Locale;
        ...
    }

    ...

}

上面的类示例有太多的内部对象和依赖项需要测试，因为测试实际上会调用它们。您将使用依赖项注入来传递具有已知状态的对象，并让它们正确返回。

请注意，我不是在嘲笑中展示如何做到这一点，因为我没有使用它，但这个简单的例子应该可以帮助您理解我试图表达的内容。

因此，一个简单的测试可能如下所示：

public function testSubscribeMessage()
{
    $email = 'email@email.nl';
    $this->Mock(
        ->shouldReceive('setEmailAddress')
        ->once()
        ->will_return($email)
        );

    $SubscribeMessage = new SubscribeMessage($this->Mock);
    $SetEmail = $SubscribeMessage->setEmailAddress($email);
    $this->assertEquals($email, $SetEmail);
}