Size: a a a

2019 December 27

BT

Bohdan Turchyk in PHP
сайт написан по swebok, как я понял
источник

SP

Sergey Protko in PHP
Константин Грачев
Путаница в категориях тестов просто пипец
Не, просто отсутствие понимания чё зачем и неумение в декомпозицию. Зато в реюзе кода все преуспели
источник

КГ

Константин Грачев in PHP
Sergey Protko
Не, просто отсутствие понимания чё зачем и неумение в декомпозицию. Зато в реюзе кода все преуспели
Ага, у меня тут весь проект сплошной реюз блет.
Причём в тестах мокомрак, по 8 зависимостей на класс
источник

КГ

Константин Грачев in PHP
Ещё я видел тест где тестируется мок)
источник

КГ

Константин Грачев in PHP
Хрупкий тест это когда от смены реализации всё валится?
источник

SP

Sergey Protko in PHP
Константин Грачев
Хрупкий тест это когда от смены реализации всё валится?
да. и беда с ними что если тестов нет то хотя бы знаешь что ты мог сломать. А если тесты есть но хрупкие то ты не заметишь разницы между "опять надо тесты обновить" и "регрессия"
источник

SP

Sergey Protko in PHP
ложное чувство безопасности формируется
источник

SP

Sergey Protko in PHP
Константин Грачев
Ну в данном конкретном случае в одном модуле находятся и абстрактные машины, типа модели производителей, и сразу клиентские автомобили.
Вот клиентским тут не место, плюс возможно рекомендациям по клиентским машина не особо место рядом с самой машиной, это ведь по сути функционал поверх автомобиля. Но тут я не уверен
суть в том насколько модуль независим. Цель разделения - что бы весь остальной проект минимум знал о наличии твоего модуля. полностью связи убрать не выйдет, но их должно быть минимум
источник

КГ

Константин Грачев in PHP
Sergey Protko
да. и беда с ними что если тестов нет то хотя бы знаешь что ты мог сломать. А если тесты есть но хрупкие то ты не заметишь разницы между "опять надо тесты обновить" и "регрессия"
У нас тут больше половины таких тестов. Убрал зависимость от формы, всё посыпалось, потому что тест проверят сколько раз какой метод формы был вызван 🤷🏻‍♂️
источник

КГ

Константин Грачев in PHP
Sergey Protko
суть в том насколько модуль независим. Цель разделения - что бы весь остальной проект минимум знал о наличии твоего модуля. полностью связи убрать не выйдет, но их должно быть минимум
Сейчас есть несколько причин туда лезть, по этим причинам как раз хочу разделить
источник

КГ

Константин Грачев in PHP
туда лезть - всмыле другим модулям
источник

DS

Dmitriy Simushev in PHP
Коллеги, у кого-нибудь Amp\Sql  с pq либой для postgres вытаскивал из базы вместо пустого массива (колонка TEXT[]) значение [''] ?
источник

DS

Dmitriy Simushev in PHP
или это у меня какие-то локальные глюки?
источник

DS

Dmitriy Simushev in PHP
причем в базе "{}"
источник
2019 December 28

SP

Sergey Protko in PHP
Константин Грачев
У нас тут больше половины таких тестов. Убрал зависимость от формы, всё посыпалось, потому что тест проверят сколько раз какой метод формы был вызван 🤷🏻‍♂️
причем как правило проблема в коде. Хотя и разработчики часто пишут тесты не для того что бы протестить поведение или обосновать наличие кода а потому что "надо ж тесты писать". Я не так давно общался с человеком который специально пишет тесты завязанные на реализацию мол "ну а вдруг они криво смерджат".
источник

КГ

Константин Грачев in PHP
Sergey Protko
причем как правило проблема в коде. Хотя и разработчики часто пишут тесты не для того что бы протестить поведение или обосновать наличие кода а потому что "надо ж тесты писать". Я не так давно общался с человеком который специально пишет тесты завязанные на реализацию мол "ну а вдруг они криво смерджат".
У коллег примерно такие же оправдания. Надо же убедится что форму проверили на валидность вызвав isValid
источник

SP

Sergey Protko in PHP
не ну вдруг они дебилы и им действительно надо)
источник

КГ

Константин Грачев in PHP
Ну такое)
источник

SP

Sergey Protko in PHP
такое обычно связано с тем что "как-то раз забыли, что бы не забывать"
источник

SP

Sergey Protko in PHP
например я помню как лет 5 назад у меня были похожие мысли потому что пару раз флаш забыли вызвать и вылезли баги. Но по факту такое приемочные тесты закрывают
источник