MockeryのshouldReceiveはonceをつけないと意味がない
はじめに
-
Mockeryでハマったので他の人がハマらないようにメモ書きを残す
通常サンプル
MyClass.php
class MyClass {
public function get_message($name) {
return $this->get_hello() . $name;
}
public function get_hello() {
return 'Hello ';
}
}
MyClassTest.php
class MyClassTest {
public function test_get_message(){
$object = new MyClass();
$result = $object->get_message('TEST');
$expect = 'Hello TEST';
$this->assertSame($expect, $result);
}
}
普通に使う場合はこんな感じで書けるかと思います
該当のソース
MyClassTest.php
class MyClassTest {
public function test_get_message(){
$testDouble = \Mockery::mock(MyClass::class)->makePartial();
$testDouble->shouldReceive('dummy_method');
$result = $testDouble->get_message('TEST');
$expect = 'Hello TEST';
$this->assertSame($expect, $result);
}
}
dummy_methodという存在しないメソッドがshouldReceive（呼び出されていること）をテストしています。
dummy_methodは存在しないし呼び出してもいないので当然テストは失敗する、と思いたかったんですが実はこのテスト成功してしまいます。
対策
以下のように書けばうまくいきます
MyClassTest.php
class MyClassTest {
public function test_get_message(){
$testDouble = \Mockery::mock(MyClass::class)->makePartial();
$testDouble->shouldReceive('dummy_method')->once();
$result = $testDouble->get_message('TEST');
$expect = 'Hello TEST';
$this->assertSame($expect, $result);
}
}
E 1 / 1 (100%)
Time: 12.39 seconds, Memory: 26.00MB
There was 1 error:
1) XXX::test_get_message
Mockery\Exception\InvalidCountException: Method dummy_method(<Any Arguments>) from Mockery_0_XXX should be called
exactly 1 times but called 0 times.
1回呼び出されることを想定していて0回呼び出しだったので正しく失敗してくれています
まとめ
この動きが正しいのかよく分かりません
もしかしたら使い方が間違っているだけなのかもしれないので有識者の方コメントをお願いします
ひとまずは「呼び出されたこと」をテストしたい場合は必ずonceなどの回数指定のものを設定しましょう
Author And Source
この問題について(MockeryのshouldReceiveはonceをつけないと意味がない), 我々は、より多くの情報をここで見つけました https://qiita.com/yamamoto_hiroya/items/e5812d387db1ac3e240a著者帰属：元の著者の情報は、元のURLに含まれています。著作権は原作者に属する。
Content is automatically searched and collected through network algorithms . If there is a violation . Please contact us . We will adjust (correct author information ,or delete content ) as soon as possible .