Ворота качества: что блокирует merge и как тесты не исчезают молча

Тимур · бэкенд, #qa-antrakt

Ребята, я в PR про возвраты удалил test_refund_frees_seat: он красный, а у меня горит релиз. Это временно, верну на следующей неделе. Ника одобрила.

Ника · фронтенд, #qa-antrakt

Я смотрела код возвратов, а не тесты. В диффе было «−14 строк в tests/», я подумала, что это рефакторинг.

Савва · продакт, #qa-antrakt

Вчера зритель вернул билет на «Белые ночи», а место так и осталось занятым. Продать его мы не смогли. Это связано?

Вера Ланская · лид QA, #qa-antrakt

Напрямую: удалённый тест ловил ровно это. Никто не сделал ничего злого, просто ничто не мешало. Конвейер показывает, что сломалось, но пока ничего не запрещает. Нужны ворота.

После прошлого урока конвейер быстрый и всё показывает в PR. Но показывать и запрещать это разные вещи. Красный PR можно влить кнопкой, а красный тест можно просто удалить, и конвейер позеленеет. Сегодня решаем, что конвейер не пропустит.

Турникет и табличка

В метро на входе можно повесить табличку «пожалуйста, оплатите проезд». Большинство честно заплатит. А в час пик, когда опаздываешь, рука сама тянется пройти мимо: «один раз, очень надо».

Турникет не уговаривает. Он просто не открывается. При этом у турникета есть служебная калитка, через которую проходит дежурный, и каждый проход через неё записан. Хорошие ворота качества устроены так же: обычный путь закрыт без исключений, а исключение возможно, но только явно, с решением конкретного человека и следом в истории.