Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

PHPUnit: how do I mock multiple method calls with multiple arguments?

I am writing a unit test for a method using PHPUnit. The method I am testing makes a call to the same method on the same object 3 times but with different sets of arguments. My question is similar to the questions asked here and here

The questions asked in the other posts have to do with mocking methods that only take one argument.

However, my method takes multiple arguments and I need something like this:

$mock->expects($this->exactly(3)) ->method('MyMockedMethod')     ->with(         $this->logicalOr(             $this->equalTo($arg1, $arg2, arg3....argNb),             $this->equalTo($arg1b, $arg2b, arg3b....argNb),             $this->equalTo($arg1c, $arg2c, arg3c....argNc)         )     ); 

This code doesn't work because equalTo() validates only one argument. Giving it more than one argument throws an exception:

Argument #2 of PHPUnit_Framework_Constraint_IsEqual::__construct() must be a numeric

Is there a way to do a logicalOr mocking for a method with more than one argument?

Thanks in advance.

like image 677
Thomas Avatar asked Jun 08 '12 18:06

Thomas


2 Answers

In my case the answer turned out to be quite simple:

$this->expects($this->at(0))     ->method('write')     ->with(/* first set of params */);  $this->expects($this->at(1))     ->method('write')     ->with(/* second set of params */); 

The key is to use $this->at(n), with n being the Nth call of the method. I couldn't do anything with any of the logicalOr() variants I tried.

like image 145
dr Hannibal Lecter Avatar answered Sep 21 '22 08:09

dr Hannibal Lecter


For others who are looking to both match input parameters and provide return values for multiple calls.. this works for me:

    $mock->method('myMockedMethod')          ->withConsecutive([$argA1, $argA2], [$argB1, $argB2], [$argC1, $argC2])          ->willReturnOnConsecutiveCalls($retValue1, $retValue2, $retValue3); 
like image 27
Cody A. Ray Avatar answered Sep 20 '22 08:09

Cody A. Ray