7

我想第一次使用Prophecy ("phpspec/prophecy-phpunit") 为我的类创建单元测试。我想测试一个在同一服务中调用另一个函数的函数，代码如下：

class UserManager
{
    private $em;
    private $passwordHelper;

    public function __construct(\Doctrine\ORM\EntityManager $em, \MainBundle\Helper\PasswordHelper $passwordHelper)
     {
         $this->em = $em;
         $this->passwordHelper = $passwordHelper;
     }

     public function getUserForLdapLogin($ldapUser)
     {
          $dbUser = $this
              ->em
              ->getRepository('MainBundle:User')
              ->findOneBy(array('username' => $ldapUser->getUsername()));

         return (!$dbUser) ?
              $this->createUserFromLdap($ldapUser) :
              $this->updateUserFromLdap($ldapUser, $dbUser);
     }

我遇到的第一个问题是我正在使用findOneByUsernameProphecy，据我所知，它不允许您：模拟魔术方法（_callfor EntityRepository），模拟不存在的方法，模拟您正在测试的类。如果这些都是真的，那我就有点麻烦了，这意味着如果不测试类中的其他函数，我就无法测试这个函数。

到目前为止，我的测试如下所示：

class UserManagerTest extends \Prophecy\PhpUnit\ProphecyTestCase
{

      public function testGetUserForLdapLoginWithNoUser()
      {
          $ldapUser = new LdapUser();
          $ldapUser->setUsername('username');

          $em = $this->prophesize('Doctrine\ORM\EntityManager');
          $passwordHelper = $this->prophesize('MainBundle\Helper\PasswordHelper');

          $repository = $this->prophesize('Doctrine\ORM\EntityRepository');
          $em->getRepository('MainBundle:User')->willReturn($repository);
          $repository->findOneBy(array('username' => 'username'))->willReturn(null);

          $em->getRepository('MainBundle:User')->shouldBeCalled();
          $repository->findOneBy(array('username' => 'username'))->shouldBeCalled();

          $service = $this->prophesize('MainBundle\Helper\UserManager')
            ->willBeConstructedWith(array($em->reveal(), $passwordHelper->reveal()));

          $service->reveal();
          $service->getUserForLdapLogin($ldapUser);
     }
}

当然，测试失败是因为对$em, 和存储库的承诺没有实现。如果我实例化我正在测试的类，测试会失败，因为该函数随后会调用createUserFromLdap()同一个类并且没有经过测试。

有什么建议么？

4

3 回答 3

1

第一个问题：

不要使用魔法，魔法是邪恶的。__call 可能导致不可预知的行为。

“对 $em 的承诺和存储库没有实现”：

不要让你的代码依赖于类而是接口。然后模拟 Interface 而不是 Class ！您应该模拟 ObjectManager 而不是 EntityManager。（不要忘记更改参数的类型）

最后一点：

揭晓前。

$service->createUserFromLdap()
   ->shouldBeCalled()
   ->willReturn(null);
于 2015-02-06T17:05:08.757 回答
1

您要实现的是部分模拟，Prophecy 不支持。更多关于它在这里https://github.com/phpspec/prophecy/issues/101和https://github.com/phpspec/prophecy/issues/61。

TL;博士; 设计您的类时要牢记单一职责，因此您不必模拟其他功能。

于 2016-08-27T12:41:01.213 回答
0

关于您无法模拟不存在的方法的问题，您可以使用

http://docs.mockery.io/en/latest/

而不是预言。嘲弄允许你这样做。严格来说，这确实打破了良好设计的一些规则，但另一方面，有时它只是非常有用。无论如何，就功能而言，嘲弄非常相似，而且它同样直观且易于使用 imo。但是，他们还没有发布稳定版本，所以如果你决定使用它，请注意这一点。

在这里你可以找到两个库的一个很好的比较

http://everzet.com/post/72910908762/conceptual-difference-between-mockery-and-prophecy

于 2016-02-25T08:14:13.630 回答