PHPunit：疑似クエリ型メソッドで期待通りに呼び出すのは悪い習慣ですか？

Question

私はPhpUnitを使用して自分のPHPコードをテストしています。多くの開発者は、クエリメソッドに期待をかけるのは悪い習慣であると言います。これは、何かを返すだけで、システムの状態（CQS）を変更しないメソッドです。期待は、コマンド型のメソッドに対してのみ使用する必要があります。例：

$this->validatorMock->expects($this->once()) 
       ->method('isValid') 
       ->with('foo') 
       ->willReturn(true) 

実装にしっかりとテストを結合するので、悪い習慣と考えられます。正しい方法：しかし

$this->validatorMock->method('isValid') 
    ->willReturn(true); 

私は例えば、ある種の検証の結果に依存して、テストクラス：

if ($this->validator->isValid($data) { 
     // some logic 
} else { 
    // some other logic 
}          

だ場合、私はその方法に期待をかけたりすべきではないでしょうか？これはクエリ型のメソッドですが、呼び出されたことをテストすることが重要です。データが検証されたことを知りたい。

もちろんisValidメソッド自体は状態を変更するが、ちょうど入力に基づいて値を返さない関数がたくさんある別々のTestClassを（バリデータクラスをテストする1）に

Answer 1

をテストしています。検証は1つのタイプであり、変換は別のタイプです。

これらの関数は、破損したくない重要なロジックを含んでいるため、これらの関数を単体テストするのには理にかなっています。私は '多くの開発者'が間違っていると思う。