SP
это к вопросу "а что ты мокаешь"
Size: a a a
2021 August 26
ЕК
ага, похоже на то
SP
если у тебя есть внешняя зависимость которую ты не контролируешь (не ты контракт определил и ты на него не влияешь) то это нельзя мокать
SP
например - вот у тебя есть Request условный симфоневый. или доктриновский array collection. или энвелопы у messages. Это просто не надо мокать. Более того - это и так обычно объекты без внешних зависимостей, контейнеры для данных. их вообще не надо мокать особо
SP
это как мокать DateRange например - это ж тупо
SP
ну да, если у этого чего-то есть свои зависимости ты об этом быстро узнаешь в попытке инстанцировать это что-то в тесте)
SP
Очень помогает с "а что мокать и как мокать" такое ограничение - если у тебя есть "что-то" что ты хочешь замокать - этот твой мок должен ПОЛНОСТЬЮ (без каких-либо исключений) предоставить описание контракта. повторюсь - полностью. С учетом всех нюансов аля есть у тебя array collection с методом length и isEmpty и они должны вести себя соответственно. Если length возвращает 0 то isEmpty true и т.д.
SP
я стараюсь логику тогда из этого места куда-то вынести и оставить инфраструктурную грязь там где оркестрация - нет ифов нет нужны в юнитах. приемочные тесты покроют
ЕК
согласен со всем
хотя конкретно с Envelope мне как-то неочевидно, он принимает только сообщение, но мне нужен Envelope с разным состоянием счетчика) ну я такой и пошел мокать)
хотя конкретно с Envelope мне как-то неочевидно, он принимает только сообщение, но мне нужен Envelope с разным состоянием счетчика) ну я такой и пошел мокать)
SP
ну вот я выше написал что поможет. перестать юзать моки для "я тут только этот метод юзаю - только его и хочу подменить".
контракты надо реализовывать полностью
контракты надо реализовывать полностью
SP
это должно не "приходиться" а "по дефолту". Люди херовые контракты делают потому что не соблюдают это правило
SP
потому что не воспринимают контракт как "договоренность о поведении между двумя объектами", просто делают интерфейсы в вакууме
SP
смысл в том что бы твои тесты не делали предположений о том что твоя реализация использует а что нет. что бы уменьшить связанность. Что бы проверить качество и специализацию контрактов
gp
@fes0r в проектах пишите подобные контракты для 3d party интеграций ?
наподобии етой: https://github.com/symfony/cache-contracts
наподобии етой: https://github.com/symfony/cache-contracts
SP
Нет, зачем.