Alternative shouldReceive syntax

[davedevelopment]

Not sure what people think about this, it just gets away from using strings for method names...

$mock->shouldReceive()->foo("bar")->andReturn(123);

[GrahamCampbell]

phpdoc needs updating to string|null ;)

[coveralls]

Coverage decreased (-0.1%) to 75.416% when pulling 8eae1a0 on davedevelopment:alternative-syntax into ad31ff9 on padraic:master.

[davedevelopment]

<?php

require "vendor/autoload.php";

$mock = Mockery::mock();

$mock->shouldReceive()->foo(123)->andReturn(456);
$mock->shouldNotReceive()->bar(567);

var_dump($mock->foo(123));

$mock->shouldHaveReceived()->foo(123);
$mock->shouldNotHaveReceived()->bar(456);

Mockery::close();

[davedevelopment]

Riffing on the same thing, I could leave the old school methods alone and do

<?php

require "vendor/autoload.php";

$mock = Mockery::mock();

// $mock->shouldReceive("foo")->with(123)->once()->andReturn(456);
$mock->expect()->foo(123)->andReturn(456);

// $mock->shouldReceive("bar")->with(567)->andReturn(789);
$mock->allow()->bar(567)->andReturn(789);

Might collide with some userland code. Not sure what equivalent I'd want to use for the spies.

[coveralls]

Coverage decreased (-0.5%) to 75.039% when pulling 27fd287 on davedevelopment:alternative-syntax into ad31ff9 on padraic:master.

[usox]

+1
A bit late but I would really prefer having real methods instead of dealing with method names a string-parameters.

[coveralls]

Coverage decreased (-0.6%) to 75.509% when pulling 2e4373e on davedevelopment:alternative-syntax into b4617ef on padraic:master.