Mockery shouldReceive()->once() doesn't seem to work

Question

I'm trying to get Mockery to assert that a given method is called at least once.

My test class is:

use \Mockery as m;

class MyTest extends \PHPUnit_Framework_TestCase
{

    public function testSetUriIsCalled()
    {
        $uri = 'http://localhost';
        $httpClient = m::mock('Zend\Http\Client');
        $httpClient->shouldReceive('setUri')->with($uri)->atLeast()->once();
    }

}

As you can see, there's one test that (hopefully) creates an expectation that setUri will be called. Since there isn't any other code involved, I can't imagine that it could be called and yet my test passes. Can anyone explain why?

score 61 · Accepted Answer · edited Apr 30 '14 at 08:41

61

You need to call Mockery:close() to run verifications for your expectations. It also handles the cleanup of the mockery container for the next testcase.

public function tearDown()
{
    parent::tearDown();
    m::close();
}

edited Apr 30 '14 at 08:41

Mārtiņš Briedis

17,396
5
54
76

answered Apr 30 '13 at 20:53

Bram Gerritsen

7,178
4
35
45

score 2 · Answer 2 · answered Mar 27 '16 at 14:58

To avoid having to call the close method in every test class, you can just add the TestListener to your phpunit config like so:

<listeners>
    <listener class="\Mockery\Adapter\Phpunit\TestListener"></listener>
</listeners>

This approach is explained in the docs.

One thing to note from the linked docs is:

Make sure Composer’s or Mockery’s autoloader is present in the bootstrap file or you will need to also define a “file” attribute pointing to the file of the above TestListener class.

This is no longer supported in PHPUnit 7 – SeanJA Feb 26 '19 at 18:09 — SeanJA, Feb 26 '19 at 18:09

score 0 · Answer 3 · answered Nov 15 '21 at 05:57

Just a sidenote: If you use Laravel: the make:test --unit generates a test class that extends the original PhpUnit Testcase class and not the included Tests\Testcase, which loads the laravel app and runs the Mockery::close(). It is also the reason why in some cases your tests fail if you use Laravel specific code (like Cache, DB or Storage) in the units you're testing.

so if you need to test units with Laravel specific code, just swap out the 'extends Testcase' and there is no need to call Mockery::close() manually

Mockery shouldReceive()->once() doesn't seem to work

3 Answers3

Linked