👩‍💻 Про код-ревью автотестов Код автотестов должен обязательно проходить код-ревью, а то проект превратится в тыкву рискует стать нестабильным, нечитаемым и проблемным. 🔹Первоначальное ревью Первым ревьюером является сам автор. Написав код, необходимо тщательно его проверить. Полезно составить чек-лист, учитывающий особенности проекта. Проходя по чек-листу можно детально проверить различные моменты, дабы ничего не упустить. Составление типовых ошибок на ревью также будет хорошей практикой 📝 Можно ещё попросить 📱 AI выполнить ревью кода, чтобы получить стороннее мнение на раннем этапе. 🔹Ревью коллегами На первом этапе часть моментов можно улучшить, но не всегда, поэтому далее код-ревью проводят коллеги. Некоторые компании внедряет многоуровневую систему ревью, чтобы как можно больше людей посмотрело, тем самым минимизируя количество проблем в будущем. Со стороны ревьюера важно видеть картину целиком, а не только несвязанные кусочки кода 🌐 В Merge Request (MR) обязательно должно быть описание, прикреплённая задача, вспомогательные лейблы и другая ценная информация, которая даёт контекста происходящего. Изучение всего материала – маст хэв, иначе вы просто можете не понять что-то важное. Код обязательно должен проходить через несколько Quality Gate, такие как статические анализаторы кода, юнит-тесты и другие проверки, запуски самих автотестов и т.д. Благодаря этим проверкам процесс ревью упрощается, оставляя сложную работу ревьюерам. Максимально автоматизируйте то, что можно и нужно автоматизировать 😉 В процессе ревью автотестов важно знать сценарий, который покрывается автотестом. Нужно проверить полноту покрытия, логику работы теста, все ли шаги и проверки правильно описаны. Стабильность автотеста можно проверить запусками, но не всегда это возможно. Здесь я бы посоветовал обращать внимание на ожидания элементов перед действиями. Желательно видеть UI и нужно знать, как этот функционал выглядит вживую и отрабатывает. И про стабильные локаторы не забывать 😁 Ещё должны быть правила, по которым изменения считаются блокирующими или нет для влития. Моменты, которые не блокируют, можно выносить в отдельные задачи. 🔹Ревью комментария Немаловажную роль играет сам комментарий, так что нужно самостоятельно проревьюить комментарий к ревью 😃 Никогда не критикуйте автора, пишите только про сам код. Подача комментария должна быть доброжелательной и конструктивной. Рекомендуется подкреплять комментарий ссылкой на доку или другие ресурсы. 🔹Как развиваться в ревью? Смотреть другие MR-ы, как и на что обращают внимание другие ревьюеры. Зафиксировать часто встречающиеся моменты в доке, чтобы другие знали, на что обращать внимание, попробуйте их отлавливать через линтеры. Делайте побольше качественного ревью, в том числе в других MR, если вы видите какие-то моменты. Что в итоге: ✅Самопроверка, чек-листы, AI ✅Коллективное ревью ✅Конструктивные комментарии ✅Постоянное развитие #автотесты@qa_relia #ревью@qa_relia