T
фабриковать круговые зависимости - боль
Size: a a a
2019 September 13
v
ну в нем есть проблемы со связями
v
но в целом больно не было
v
мне понравился
A
фабриковать круговые зависимости - боль
ммм, ну это вообще непросто и можно попасть в ловушку переусложненных фабрик
T
ну вот вместо этого приходится писать в каждой спеке длинные портянки с фабрикацией каждой зависимости по-отдельности
v
ну вот вместо этого приходится писать в каждой спеке длинные портянки с фабрикацией каждой зависимости по-отдельности
иногда это хорошо, более явно)
A
я лично, предпочитаю фабрики иметь простыми, а сложную подготовку сьютов делать через шареные контексты, либо хэлперметоды в локальных контекстах делать, которые используют фабрики
T
ну вот шареные контексты - это так же не про явность, как и fabricate(:something, :with_something), кмк
A
Шареные контексты обычно лучше с семантической точки зрения, они определяют окружение их сложнее притащить туда, где их не должно быть. У
fabricate(:something, :with_something) гораздо меньше ответственности.A
Это вообще стандартная история, когда разработчик пишет новый спек и ищет по существующим, как же ему сгенерить этот :something, в итоге находит вызов фабрики с 10-ю прицепами копипастит и оставляет, т.к. тест проходит и ладно.
A
В результате раздутые фабрики с трейтами на разные случаи жизни, которые часто используются не по назначению и избыточным образом и спеки поторые выполняются сутки:)
T
Что мешает ему точно так же притащить шареный контекст?
T
Раздутый 🙂
VD
Плюс контекстов ещё и в том, что их потом можно оптимизировать (что-то вынести в фикстуры или before_all); при этом не ломая совместимость
A
Это может быть технически сложнее, будет сильнее звонить в голове о том, что "я что-то делаю не так", проще отловить на кодревью:)
A
Плюс контекстов ещё и в том, что их потом можно оптимизировать (что-то вынести в фикстуры или before_all); при этом не ломая совместимость
➕
