Size: a a a

JPoint, Java-конференция

2020 July 08

s

saksonov 👀 in JPoint, Java-конференция
а если кто-то рвется в бой, то имхо надо поработать головой и придумать полезное изменение, чем просто втупую увеличивать покрытие
источник

s

saksonov 👀 in JPoint, Java-конференция
а раз есть изменение - то нужны и тесты на соответствующий модуль, в который вносятся изменения
источник

AV

Alexei Vinogradov in JPoint, Java-конференция
saksonov 👀
меньше вреда будет если разраб просто эти 3-4 часа попинает, чем замержит новое говно без тестов
Согласен. У нас практикуется попинывание регулярно.  Ведём дискуссии о целесообразности guerilla unit тестинга.
источник

s

saksonov 👀 in JPoint, Java-конференция
Alexei Vinogradov
Согласен. У нас практикуется попинывание регулярно.  Ведём дискуссии о целесообразности guerilla unit тестинга.
у нас тоже, и я хочу сказать, что это имеет полезный эффект
источник

s

saksonov 👀 in JPoint, Java-конференция
мы называем это innovation days, официально постулируется, что это время можно потратить на игрища с какими-то новыми технологиями, которые может подойдут нашему проекту, а может нет
источник

AV

Alexei Vinogradov in JPoint, Java-конференция
saksonov 👀
а если кто-то рвется в бой, то имхо надо поработать головой и придумать полезное изменение, чем просто втупую увеличивать покрытие
Ну вот об этом и дискуссия. В тупую - почему в тупую. Тесты должны были написаны ранее. Ну не написали. Ещё не конец света. Почему дописывание - тупо? По мне - обыкновенный технический долг, у кого-то такого больше,  у кого-то меньше - совсем без долга вряд ли команды бывают. Ну кроме Нетфликса и Омского Мясокомбинат (с Васей).
источник

AV

Alexei Vinogradov in JPoint, Java-конференция
saksonov 👀
мы называем это innovation days, официально постулируется, что это время можно потратить на игрища с какими-то новыми технологиями, которые может подойдут нашему проекту, а может нет
Это хорошо
источник

s

saksonov 👀 in JPoint, Java-конференция
Alexei Vinogradov
Ну вот об этом и дискуссия. В тупую - почему в тупую. Тесты должны были написаны ранее. Ну не написали. Ещё не конец света. Почему дописывание - тупо? По мне - обыкновенный технический долг, у кого-то такого больше,  у кого-то меньше - совсем без долга вряд ли команды бывают. Ну кроме Нетфликса и Омского Мясокомбинат (с Васей).
я не верю в полезность тестов после релиза
источник

AV

Alexei Vinogradov in JPoint, Java-конференция
saksonov 👀
я не верю в полезность тестов после релиза
Вопрос религиозный)
источник

s

saksonov 👀 in JPoint, Java-конференция
ну в том смысле, что момент когда придется вносить изменение в следующий раз в этот модуль - наилучший для восстановления потерянного покрытия
источник

s

saksonov 👀 in JPoint, Java-конференция
просто пойти и написать тест - увеличение времени билда в пустую, имхо
источник

s

saksonov 👀 in JPoint, Java-конференция
а если ты добавил новый инвариант, то как раз отлично бы протестировать как новый инвариант, так и то, что все предыдущие не сломались, польза максимальная
источник

NG

Nikita Gryzlov in JPoint, Java-конференция
обычно помимо сторисов с полезной новой функциональностью в головах разрабов есть мини задачки вида "вот хорошо бы эту штуку отрефакторить" или "вот там лютое говно, надо переписать все". под все это очень хорошо ложится практика запрета мержа нового кода без покрытия его на установленный порог (например, 80%). это очень хорошо автоматизируется с сонаркубом, например. получается, что неважно что именно в коде - рефакторинг или новая фича - правила едины для всех: потрогал код - на него должны быть тесты, без исключений
источник

AV

Alexei Vinogradov in JPoint, Java-конференция
saksonov 👀
ну в том смысле, что момент когда придется вносить изменение в следующий раз в этот модуль - наилучший для восстановления потерянного покрытия
Есть свои тонкости. Например оценки. Разрабы педантично описывают что для новой задачи надо сделать. Поменять тут код, добавить конфигурации, написать 2 юнит-теста, протестировать.  Нормально, берём в спринт.

А тут еще внезапно абсолютно неоцениваемые "написать 500 тестов на остальной код модуля и прорефакторить по необходимости". Да ну его к лешему, думает разраб - и мне сложно его упрекнуть.
источник

NG

Nikita Gryzlov in JPoint, Java-конференция
степень покрытия в коде расчет очень быстро. постоянно трогается старый код => на него появляются тесты. пускай и не все инварианты, но это лучше, чем вообще без покрытия
источник

s

saksonov 👀 in JPoint, Java-конференция
Alexei Vinogradov
Есть свои тонкости. Например оценки. Разрабы педантично описывают что для новой задачи надо сделать. Поменять тут код, добавить конфигурации, написать 2 юнит-теста, протестировать.  Нормально, берём в спринт.

А тут еще внезапно абсолютно неоцениваемые "написать 500 тестов на остальной код модуля и прорефакторить по необходимости". Да ну его к лешему, думает разраб - и мне сложно его упрекнуть.
сгоревшие сторипоинты не KPI
источник

s

saksonov 👀 in JPoint, Java-конференция
превысил и превысил, значит были причины
источник

AV

Alexei Vinogradov in JPoint, Java-конференция
saksonov 👀
превысил и превысил, значит были причины
Конкретно у нас - другая боль. Ладно бы превысил, просто задачу в 3 поинта (велосити 25) делал три спринта. И это без этих юнит-тестов. А теперь давайте добавим юнит-тесты...
источник

s

saksonov 👀 in JPoint, Java-конференция
Alexei Vinogradov
Конкретно у нас - другая боль. Ладно бы превысил, просто задачу в 3 поинта (велосити 25) делал три спринта. И это без этих юнит-тестов. А теперь давайте добавим юнит-тесты...
о боже
источник

s

saksonov 👀 in JPoint, Java-конференция
ну короче классическое "делать хорошо - слишком дорого для нас"
источник