1

假设我有我指定的这个类(遵循 BDD 方法)

class Logger
{
    private $em;

    public function __construct(EntityManager $em)
    {
        $this->em = $em;
    }

    public function logMessageAsRead(Message $message)
    {
        $log = new LoggedMessage($message);
        $this->em->persist($message);
    }
}

LoggedMessage定义如下

class LoggedMessage
{
    private $date;

    private $message;

    public function __construct(Message $message)
    {
        $this->date = new \DateTime();
        $this->message = $message;
    }
}

Message有时我的规范示例会由于规范中实例化的日期与类中的日期不一致而失败Logger

class LoggerSpec
{
    public function it_logs_a_message(Message $message, EntityManager $em)
    {
        $log = new LoggedMessage($message);
        $em->persist($log)->shouldBeCalled(1);

        $this->logMessageAsRead($message);
    }
}

问题一:我的代码中是否有异味,所以我是否需要创建一个协作者(即:一个工厂)并将其注入Logger以创建一个新的LoggedMessage

问题二:如果不需要注入新的合作者,我如何确保我的规范每次都能正常工作,并且不会由于日期时间差异而随机失败?

4

1 回答 1

0
  1. s的注入工厂LoggedMessage将是一个很好的解决方案,特别是如果您不想关闭LoggedMessage反对修改的构造函数。一般来说,最好将创建对象的关注点与使用它的关注点分开。

  2. 最简单的解决方案是检查特定类型而不是具体实例:

    $em->persist(Argument::type(LoggedMessage::class))->shouldBeCalled(1);

如果您希望您的期望更具体,您可以使用Argument::whichor Argument::that

$em->persist(Argument::which('getMessage', $message))->shouldBeCalled(1);
$em->persist(Argument::that(function($arg) {
    return $arg->getDate() instanceof \DateTime;
}))->shouldBeCalled(1);
于 2016-11-04T14:23:52.080 回答