В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

symfony

1885 membersпожаловаться на группу
2020 October 03

ПГ

Павел Г. in symfony
The Ant 🐜
потому и сыпятся, потому что тестируете непонятно что ради просто чтобы тесты были...
Сыпятся из-за множества зависисмотей. Я выше писал кейс: добавляем новый функционал, который требует новую зависимоть  - падают все тесты, и их нужно все править. так как у хэндлера изменился конструктор.
источник
09:46пожаловаться#1

ПГ

Павел Г. in symfony
Это банальная проблема сервис классов больших при использовании анемик моделей. Которые имеют потенциал для реализации нового метода потребвать новую зависимость. Тянем ее в конструктор. Падают тесты других методов.
источник
09:51пожаловаться#2

ПГ

Павел Г. in symfony
Но тут решение банальное - разделить на более мелкие независмые сервис классы.
источник
09:52пожаловаться#3

D

Dmitry in symfony
The Ant 🐜
потому и сыпятся, потому что тестируете непонятно что ради просто чтобы тесты были...
при изменении конструктора посыпется что угодно :)
источник
09:54пожаловаться#4

D

Dmitry in symfony
Павел Г.
Сыпятся из-за множества зависисмотей. Я выше писал кейс: добавляем новый функционал, который требует новую зависимоть  - падают все тесты, и их нужно все править. так как у хэндлера изменился конструктор.
нужно править всего один раз, там где создается инстанс для тестирования в setUp
источник
09:55пожаловаться#5

ПГ

Павел Г. in symfony
Dmitry
нужно править всего один раз, там где создается инстанс для тестирования в setUp
Так условия могут быть разные.  Мы по разному мокаем.
источник
09:56пожаловаться#6

ПГ

Павел Г. in symfony
Для одного метода мы сущность из find() отдаем одну, а для другого метода - другую
источник
09:56пожаловаться#7

T🐜

The Ant 🐜 in symfony
find & replace по регулярке... неужели сложно?
источник
09:56пожаловаться#8

D

Dmitry in symfony
Павел Г.
Для одного метода мы сущность из find() отдаем одну, а для другого метода - другую
и чо ? контроллер от этого не зависит. вы просто в контроллер передаете MyFinderRepository
источник
09:57пожаловаться#9

D

Dmitry in symfony
а потом в конкретном тесте уже делаете
expects()->method(find)->willReturn(neededEntity)
источник
09:57пожаловаться#10

D

Dmitry in symfony
The Ant 🐜
find & replace по регулярке... неужели сложно?
я бы поглядел на регулярку которая найдет все создания инстансов и добавит туда мок, создав его выше :)
источник
09:58пожаловаться#11

ПГ

Павел Г. in symfony
Dmitry
и чо ? контроллер от этого не зависит. вы просто в контроллер передаете MyFinderRepository
Контроллер не зависит, Зависит метод который его создает. Если будет какой то prepare в который мы будет отдавать конкретные созданные для этгого зависимости, то изменится этот prepare при добавлении новой зависимости.
источник
09:58пожаловаться#12

D

Dmitry in symfony
Павел Г.
Контроллер не зависит, Зависит метод который его создает. Если будет какой то prepare в который мы будет отдавать конкретные созданные для этгого зависимости, то изменится этот prepare при добавлении новой зависимости.
тут согласен, но без этого никуда. если вы меняете поведение метода, то очевидно что тесты упадут и их надо править. с этим ничего не поделать
источник
09:59пожаловаться#13

VS

Vlad Sobenko in symfony
Dmitry
а потом в конкретном тесте уже делаете
expects()->method(find)->willReturn(neededEntity)
Лучше фейки юзать всегда вместо стабов.
источник
10:00пожаловаться#14

ПГ

Павел Г. in symfony
Спасибо за дискуссию и ответы, с удовольствием продолжил бы, надо уходить :(
источник
10:00пожаловаться#15

D

Dmitry in symfony
Vlad Sobenko
Лучше фейки юзать всегда вместо стабов.
можно вообще моки юзать вместо стабов и фейков :)
источник
10:01пожаловаться#16

IG

Ivan Grigoriev in symfony
Павел Г.
Сыпятся из-за множества зависисмотей. Я выше писал кейс: добавляем новый функционал, который требует новую зависимоть  - падают все тесты, и их нужно все править. так как у хэндлера изменился конструктор.
Если новый функционал добавляется модификацией методов, то это явное нарушение OCP. Хотите так делать - возитесь с тестами: изменили класс - очевидно, что надо менять тесты, его проверяющие.
источник
10:01пожаловаться#17

VS

Vlad Sobenko in symfony
Dmitry
можно вообще моки юзать вместо стабов и фейков :)
Можно, но не нужно. Очень ломкие тесты с ними.
источник
10:01пожаловаться#18

ПГ

Павел Г. in symfony
Ivan Grigoriev
Если новый функционал добавляется модификацией методов, то это явное нарушение OCP. Хотите так делать - возитесь с тестами: изменили класс - очевидно, что надо менять тесты, его проверяющие.
ОСР решается добавлением диспатчера. Этот вариант я выше описывал.
источник
10:02пожаловаться#19

D

Dmitry in symfony
Vlad Sobenko
Можно, но не нужно. Очень ломкие тесты с ними.
вообще нет. но там зависит...не будем спорить по этому поводу
источник
10:02пожаловаться#20