RR
Все данные должен готовить слой выше него
Size: a a a
2020 September 04
IN
У нас тоже тесты пишут разрабы, обязательнвй контроль коверейж при форсировании изменений, каждая строчка должна быть покрыта тестами, качество тестов тоже проверять стараемся
да, но у нас как-то был хитрый джун-макак, который тесты постоянно не правильно писал)
Шанс того что на кодревью это увидят - не так уж и огромен
Шанс того что на кодревью это увидят - не так уж и огромен
IN
Ты не понял что я имею ввиду, я говорю что слой получения данных должен быть до сериалайзера, он тупо принял структуру и отформатировал ее как нужно, все
ну, он выше, объект user
RR
да, но у нас как-то был хитрый джун-макак, который тесты постоянно не правильно писал)
Шанс того что на кодревью это увидят - не так уж и огромен
Шанс того что на кодревью это увидят - не так уж и огромен
Ну смотря как смотреть) особенно зная такого человека, можно с особым пристрастием смотреть тесты, у нас тоже бывает пытаюся хитрить и грязные хаки делать
IN
Ну смотря как смотреть) особенно зная такого человека, можно с особым пристрастием смотреть тесты, у нас тоже бывает пытаюся хитрить и грязные хаки делать
когда жопа горит не всегда до этого)
RR
ну, он выше, объект user
Да, но если это АР то запросы выполнятся в момент обращения к релейшенам, а я говорю, что ты должен сделать доменный объект, который полностью уже пожгрузился
RR
когда жопа горит не всегда до этого)
Ну это понятно, но как практика показывает краткосрочные плюсы вылить какашку аукаются
IN
Ну это понятно, но как практика показывает краткосрочные плюсы вылить какашку аукаются
Согласен, но без этого видимо никак, когда с тобой джуны работают ахах
RR
Ну мне кажется в таких случаях надо вмешиваться и быстро фиксануть самому или сразу завести задачу и отправить джуна делать уже не такой горящей обстановке
IN
Ну мне кажется в таких случаях надо вмешиваться и быстро фиксануть самому или сразу завести задачу и отправить джуна делать уже не такой горящей обстановке
да, но тут у нас просто рук не хватает, типичная проблема нехватки разрабов
RR
Как у вас проходит процесс сборки релиза?
IN
Как у вас проходит процесс сборки релиза?
Сейчас очень плохо - если сказали "надо было сделать вчера" и прикрылись ЧТЗ, то в срочном порядке собираем на специальной машине и отдаем архив, который они должны подложить и запустить скрипт
IN
Я пытаюсь все автоматизировать это, но фзб не дает такое удовольствие ((
RR
Ну я больше имею ввиду, что флоу задачи - разработка с обязательными тестами - ревью - тестирование/мониторинг на тестовой среде - ревью перед мержем в мастер
IN
Ну я больше имею ввиду, что флоу задачи - разработка с обязательными тестами - ревью - тестирование/мониторинг на тестовой среде - ревью перед мержем в мастер
а, это сейчас есть все, но на код ревью сейчас все пальцы ставят, а по факту, я знаю что никто не смотрит)
IN
я как-то пасхалку в проект запихнул - никто даже не оценил пока я не сказал((
IN
просто все ленивые))
IN
Еще хотел провернуть такую практику - мы садимся с джунами и начинаем делать код ревью и показывать им че не так и почему, но пацанам эта идея не понравилась, говорят что слишком много времени занимает)
Хотя просто никто не хочет время тратить на ревью ахах
Хотя просто никто не хочет время тратить на ревью ахах
RR
Ну это да, проблема, когда команда не видит пользы в ревью, тут как вариант принцип пряника и кнута, в духе делать плюшки за качество и пиздюлины всем, если в задаче косяк, особенно тем, кто ревьюит
IN
Хотя ревью вроде как инструмент который выявляет больше багов чем юнит тестирование