Как протестировать метод, который вызывает методы другого класса?

Question

[Phoen1xx]

Здравствуйте. У меня есть класс с методом, который вызывает сеттер стороннего объекта, выглядит это примерно так:

class A{
   protected $data;
   public function setData($data){
       $this->data = $data;
       file_put_contents('a.txt', $data);
   }
}

class B{
   public function doSomething(A $a, $x){
      if( $x > 10 ){
         $a->setData( $x * $x );
      }else{
         $a->setData( $x * 20 );
      }
   }
}

Как протестировать doSomething? Проблема в том, что setData запускать в тестах нельзя, но нужно узнать какие параметры были в него переданы, т.к. это и есть результат работы doSomething

Comments

[davidnum95]

передавать в doSomething не конкретную реализацию а интерфейс???

[Phoen1xx]

davidnum95, а какая разница? В тестах все равно передается mock

[Алексей Уколов]

Phoen1xx, если вы знаете о существовании моков, то в в чём тогда проблема?

[Phoen1xx]

Алексей Уколов, проблема в том, что из-за того что setData запускать нельзя, приходится создавать mock, а при создании в mock объекте заглушки данные теряются и я не могу для них вызвать assertEquals, чтобы узнать результат работы doSomething

[Алексей Уколов]

Вам стоит показать свои тесты и код приблизить к реальному, чтобы разговор был предметным. Потому что в том коде, что вы показали, ничего не теряется и setData к результату doSomething не имеет отношения. Если результат doSomething передаётся в setData и вам нужно проверить, что он был передан, то я показал в своём ответе как это сделать.

[Phoen1xx]

Алексей Уколов, да, действительно, это работает. Упустил момент с with т.к. пример из документации не работал.

Answer 1

[Алексей Уколов]

class MyTest extends \PHPUnit\Framework\TestCase
{
    public function testDoSomethingOne(): void
    {
        /** @var \PHPUnit\Framework\MockObject\MockObject $mockOfA */
        $mockOfA = $this->getMockBuilder(A::class)->getMock();

        $mockOfA->expects($this->once())
            ->method('setData')
            ->with(20);

        (new B)->doSomething($mockOfA, 1);
    }

    public function testDoSomethingTwo(): void
    {
        /** @var \PHPUnit\Framework\MockObject\MockObject $mockOfA */
        $mockOfA = $this->getMockBuilder(A::class)->getMock();

        $mockOfA->expects($this->once())
            ->method('setData')
            ->with(10000);

        (new B)->doSomething($mockOfA, 100);
    }
}

The practice of replacing an object with a test double that verifies expectations, for instance asserting that a method has been called, is referred to as mocking.

https://phpunit.de/manual/6.5/en/test-doubles.html

Comments

[SerJook]

По-моему у вас перепутаны местами числа, т.е. должно быть

->with(25); 
$b->doSomething($mockOfA, 5);

впрочем это мелочи

[Алексей Уколов]

SerJook, спасибо, поправил.