Телеграмм чат группы symfony

всем привет) как считаете, https://github.com/dg/bypass-finals костыль или норм? придерживаюсь подхода "по дефолту везде final", но в тестах проблемы с моками

GitHub

GitHub - dg/bypass-finals: Removes final keywords from source code on-the-fly and allows mocking of final methods and classes. It can be used together with any test tool such as PHPUnit or Mockery.

Removes final keywords from source code on-the-fly and allows mocking of final methods and classes. It can be used together with any test tool such as PHPUnit or Mockery. - GitHub - dg/bypass-final...

источник

14:56пожаловаться #8

Andru Cherny in symfony

Мы юзаем для тестирования. Норм тема

источник

14:58пожаловаться #9

Sergey Protko in symfony

кастыль, вы явно что-то делаете не так

источник

14:58пожаловаться #10

Sergey Protko in symfony

Final classes only work effectively under following assumptions:

- There is an abstraction (interface) that the final class implements
- All of the public API of the final class is part of that interface

If one of these two pre-conditions is missing, then you will likely reach a point in time when you will make the class extensible, as your code is not truly relying on abstractions.

источник

14:59пожаловаться #11

Sergey Protko in symfony

https://ocramius.github.io/blog/when-to-declare-classes-final/

ocramius.github.io

When to declare classes final

Declaring classes as final enhances our code quality and abstraction dramatically, but is it always correct?

источник

14:59пожаловаться #12

ЕК

Евгений Котов... in symfony

ну а final'ы по дефолту - норм ведь? класс без final - только когда мы явно предусматриваем возможность наследоваться

источник

14:59пожаловаться #13

Sergey Protko in symfony

final по дефолту если у класса есть implements. иначе придется гарадить кастыли. Люди не насколько дурные что бы наследоваться так часто. Ну и еще идея - вдруг у тебя проблема с большим количеством зависимостей что ты так мокать все подряд любишь и не любишь интерфейсы

источник

15:00пожаловаться #14

Sergey Protko in symfony

короч если выбирать "тупой дефолт и кастыли" и "доверять людям даже если я им не доверяю и просто объяснить им не юзать наследование" я выберу второе.

источник

15:01пожаловаться #15

Sergey Protko in symfony

если ты с yii-никами работаешь и у них прям уж сласть как хочется сделать абстрактный Model от которого все экстендится или еще какой трэш - ну может быть дефолт не такой дурной

источник

15:02пожаловаться #16

ЕК

Евгений Котов... in symfony

имеется ввиду что я должен мокать по интерфейсу, верно?

источник

15:02пожаловаться #17

Sergey Protko in symfony

по хорошему - да. Если лень делать интерфейс - просто не делай класс final.

источник

15:02пожаловаться #18

Sergey Protko in symfony

но чаще проблема с тем что ты мокаешь и надо ли там вообще моки юзать

источник

15:03пожаловаться #19

ЕК

Евгений Котов... in symfony

столкнулся просто, кастомную ретрай стратегию для мессенджера запилил, хочу такой юнит тест запилить, а там Envelope класс финальный, без интерфейса, эх(
и вернулся к вопросу о final
со своими реализациями проблем особо нет

источник

15:03пожаловаться #20