Как протестировать метод, создающий внутри http клиент через статический метод фабрики?

Question

[Сергей Чазов]

Есть метод примерно следующего содержания (упростил содержание для ясности):

public function handle(array $array): void
{
    foreach ($array as $item) {
        $client = ClientFactory::createClient($item->getToken());
        $this->handleOrders($client->call());
    }
}

Я бы хотел замокать метод call(), чтобы посмотреть, правильно ли отрабатывает приватный метод handleOrders. Правильно ли я понимаю, что единственный выход - сделать handleOrders публичным через рефлекцию и протестировать уже его?

Comments

[Дмитрий]

Приватный не нужно тестировать, нужно набить ватой клиенты и сравнивать ожидания

[Сергей Чазов]

Дмитрий, это понятно. Но как я набью ватой клиент, который создается статическим методом?

[Дмитрий]

Сергей Чазов, ну у вас же задача не протестировать метод фабрики или получение токена. Внутри статического метода же скорее всего обычный клиент через new

[Сергей Чазов]

Дмитрий, да, там обычный клиент через new. Но что это дает? Я не понимаю.

[Дмитрий]

Сергей Чазов, ну его мокайте в цикле на метод call нужный return

[Сергей Чазов]

Дмитрий, я могу создать мок клиента. Что мне сделать дальше, чтобы использовать его в методе handle? Ведь метод handle в любом случае вызовет фабрику, которая создаст новый клиент через статический вызов метода createClient у класса ClientFactory. А createClient я мокнуть не могу.

Если я чего-то не понимаю, то можете дать пример или ссылку на документацию. Буду благодарен.

[Дмитрий]

Сергей Чазов,

$mock = $this->getMock('myObject', 'methodToMock');
$mock->expects($this->any))
     ->method('methodToMock')
     ->will($this->returnValue('return value'));

Фабрика идёт лесом, в цикле делайте мок клиента и прокидывайте возвращаемые значения, далее где нужно сравнивайте ассертами что пришло после с ожидаемым
Возможно можно подглядеть респонз

[Сергей Чазов]

Мне нужно протестировать по сути приватный метод $this->handleOrders($client->call()). Он принимает результат вызова клиента. Я не смогу его протестировать через публичный метод handle

[Дмитрий]

Сергей Чазов, ну вот и поменяйте выхлоп call на свой, и сравнивайте ожидаемое

[Сергей Чазов]

Дмитрий, окей, я мокнул клиент. Но как мне вызвать в тесте приватный метод handleOrders, если только не сделав его публичным через рефлекцию?
В этом и заключался вопрос: можно ли обойтись без рефлекции.

[Дмитрий]

Сергей Чазов, его же вызывает другой метод
handleOrders как я понял
Вы тестируете только публичные методы, если приватные coverage не пометил, то метод можно удалить, так как он не участвует.
--coverage-text флаг

Time: 00:00.058, Memory: 12.00 MB

OK (1 test, 2 assertions)


Code Coverage Report:
  2021-01-20 02:14:24

 Summary:
  Classes: 50.00% (1/2)
  Methods: 75.00% (3/4)
  Lines:   83.33% (5/6)

Answer 1

[Дмитрий Гординский]

Проще переписать чем тестировать такой метод со статическими вызовами.
Сделайте фабрику без статического вызова и заинжекте её через конструктор этого хэндлера.

Comments

[Дмитрий]

Там клиенты в цикле же, будет не много не удобно вычислять количество элементов и прокидывать