MS
по-хорошему контейнер не должен присутствовать в тестах
+
Size: a a a
2020 February 14
AG
Ребят, вот я читаю всё это и задаюсь риторическим вопросом а нужны ли вам вообще контейнеры? Мне кажется вы с ними больше страдаете 🙃
так можно сказать, что рельсы ну нужны, в ответ на вопрос про них :)
и вообще про что угодно
и вообще про что угодно
MS
так можно сказать, что рельсы ну нужны, в ответ на вопрос про них :)
и вообще про что угодно
и вообще про что угодно
Так в чем преимущество контейнеров против ручной инициализации?
AD
по-хорошему контейнер не должен присутствовать в тестах
это холиварный вопрос так то, я считаю, что сам контейнер проверять в тестах не нужно, как и то, что зависимости правильно достаются. обычно мне удается не мокать контейнер из-за ханами (там в конструктор зависимостти прокидыватся), но в рельсе/грейпе/синатре/роде так не сделать, поэтому костылить приходиться
AG
Так в чем преимущество контейнеров против ручной инициализации?
в вопросе и ответ. помогает автоматизировать ручной труд
AG
при явной/ручной передаче параметров, было бы довольно много "лишнего" когда вокруг
AG
наверное, правильнее ставить вопрос: а нужно ли заморачиваться с DI?
NS
надо инжектить все через эффекты и забыть про проблемы с мемоизацией
AG
Если не включать мемоизацию и стабы одновременно в dry-контейнере, то кое-где прикольно получается.
Например, в интеграционных тестах можно подменить какой-то внутренний компонент, там где до этого был vcr/webmock.
Мне это больше нравится, потому что я стаблю "свой" код.
Конечно, можно сделать и без контейнера.
Так что хз. Даже рспековские стабы где-то выигрывают, потому что они проверяют названия методов, которые стабятся и их сигнатуры.
Но это такое.
Например, в интеграционных тестах можно подменить какой-то внутренний компонент, там где до этого был vcr/webmock.
Мне это больше нравится, потому что я стаблю "свой" код.
Конечно, можно сделать и без контейнера.
allow_any_instance_of().to_receive().and_return никто не отменял.Так что хз. Даже рспековские стабы где-то выигрывают, потому что они проверяют названия методов, которые стабятся и их сигнатуры.
Но это такое.
AG
надо инжектить все через эффекты и забыть про проблемы с мемоизацией
Я даже не знаю, шутишь ты или нет @flash_gordon :)
Не работал еще с этим.
Не работал еще с этим.
AD
Я даже не знаю, шутишь ты или нет @flash_gordon :)
Не работал еще с этим.
Не работал еще с этим.
не, он не шутит
NS
нууу, все что можно я перевел на эффекты, жизнь прежней не будет
AG
а сколько людей еще с кодом работают? они тоже прониклись или есть подводные камни?
NS
то есть ко всему остальному вопросов нет?) Подводные камни есть в интеграции, например, с тестами, где есть больше одного треда. Типа когда из браузера тыкается что-нибудь. То есть в одном месте нужно сконфигурировать, в остальном нет проблем, код, написанный с dry-system, менять не нужно
NS
просто нет этих проблем с конструкторами, мемоизацией, стабами и т.п.
AG
то есть ко всему остальному вопросов нет?) Подводные камни есть в интеграции, например, с тестами, где есть больше одного треда. Типа когда из браузера тыкается что-нибудь. То есть в одном месте нужно сконфигурировать, в остальном нет проблем, код, написанный с dry-system, менять не нужно
да куча вопросов. Но ты же скажешь - попробуй и узнаешь.
Просто интересно, что говорит кто-нибудь другой. Понятно, что для автора все просто :)
Просто интересно, что говорит кто-нибудь другой. Понятно, что для автора все просто :)
NS
видишь, никто не поверит, я поэтому даже и не продаю 🙂
AD
то есть ко всему остальному вопросов нет?) Подводные камни есть в интеграции, например, с тестами, где есть больше одного треда. Типа когда из браузера тыкается что-нибудь. То есть в одном месте нужно сконфигурировать, в остальном нет проблем, код, написанный с dry-system, менять не нужно
а ты уже пробовал систему на эффекты перевести?
AD
я про сам код системы
NS
