АД
Может быть вам стоит начать с изучения архитектурной темы? Там и про приватники найдете все что требуется и про контракты
Size: a a a
2021 May 12
В
В том-то и печаль, что не найду.
OS
Вброшу
Бывают ситуации, когда приватный метод очень сложный, но выносить его не хочется просто ради выноса. Ситуация, скорее, исключительная, но бывает
Бывают ситуации, когда приватный метод очень сложный, но выносить его не хочется просто ради выноса. Ситуация, скорее, исключительная, но бывает
OS
А что за колбаса?
АД
Это говорит лишь о том, что кто-то наговнокодил и сделал приватный метод - бог
ДМ
если его надо тестировать, то вероятнее всего он не должен тут находиться
OS
Вынос кода ради тестирования?
OS
Абсолютно нет, точнее необязательно
ДМ
никто не говорит ради, скорее всего он должен быть вынесен, потому что нарушает solid
АД
Еще как обязательно. Починайте про объектную калистенику
АД
ну и про солид, да
OS
Вы ж пример в вакуума разбираете, откуда тут 'обязательно'?
ДМ
контракт должен быть простой для тестирования, если вам приходится писать сотни тест кейсов, чтобы понять, что какой-то из приватных методов работает корректно - значит сущность реализована не верно
АД
Потому что это бинарная тема: либо хорошо, либо плохо. Третьего не бывает в этом вопросе. Можно мириться с действительностью и обслуживать такие методы, но это не делает их менее говнокодными
OS
Контракт должен быть простой для пользования, а не тестирования
В
колбаса колбас: написал логику внутри компонента - написал тесты логики(функций) - получил замечание на ревью: сделай функции приватными - но как их тестировать - сделай хак someComponentFixture['initData'].toBeCalledOnce() - но мы выключим проверку типов - ну тогда сделай публичный метод поверх приватного - но чтобы тестировать логику приватного метода надо её повторить в публичном
ДМ
+ это затрудняет понимание, как правило, хороший тест - это как дока. Он должен показывать, что делает ваша сущность
OS
Окей, закончим на зтом) с булевой логикой не поспоришь
IB
Ты так говоришь как будто тесты не граждане первого сорта в твоём проекте, а какие то маргинальные элементы
OS
Это не так должно быть
Вы сразу должны скрыть все лишние методы
Если есть публичный метод, который не юзается- почему он публичный?
Вы сразу должны скрыть все лишние методы
Если есть публичный метод, который не юзается- почему он публичный?