早期返品（防御プログラミング）に対するテスト

Question

私はテストを学習しており、「early returns」を使用して関数をテストしようとしています。成功の関数は別のクラスのプロパティを設定し、失敗した場合は単に両方の場合に戻り値が返されます。この例では

class Test 
{ 
    private $fileHandler; 
    private $config; 

    public __constructor($fileHandler, $config) 
    { 
     $this->fileHandler = $fileHandler; 
     $this->config = $config; 
    } 

    public function example($filePath) 
    { 
     $exists = $this->fileHandler->exists($filePath); 

     if ($exists === false) { 
      return; 
     } 

     $this->config->set($filePath); 
    } 
} 

Iは Iは、2回のユニットテストで及びfileHandlerクラスをモックすることによってこれをテストすることができると信じ。

失敗（早期復帰）の場合は、$configクラスのメソッドset()を呼び出すべきではありませんが、成功のためにメソッドを呼び出す必要があります。

しかし、私が試してみるとnever()からonce()に変更するとこのテストに合格し、テスト全体が偽であると思うようになります。

/** test */ 
public function config_is_not_set_with_missing_file() 
{ 
    $fileHandlerMock = $this->getMockBuilder(fileHandler::class)->getMock; 
    $fileHandlerMock->method('exists') 
     ->willReturn('false'); 
    $configMock = $this->getMockBuilder(config::class)->getMock; 

    $test = new Test($fileHandlerMock, $configMock); 
    $test->example('fake file path'); 

    $configMock->expects($this->never()) 
     ->method('set'); 
} 

Answer 1

あなたのファイルハンドラモックは、文字列'false'（!==〜false）を返しています。 falseとTets::exampleに変更して早めに戻ってください。

Answer 2

あなたはTestコンストラクタに$configMockを渡していないので、それが使用されていません。

テストがonceとneverの両方の期待値をパスした場合、テストは期待どおりに機能していないため、レビューが必要です。