На что вы в первую очередь обращаете внимание при ревью чужого кода и автотестов?

При проведении code review автоматизированных тестов обращаю внимание на следующие моменты:

• Покрытие сценариев: тесты должны покрывать как позитивные, так и негативные кейсы.
• Надёжность: тесты не должны быть флейки (нестабильными), важно, чтобы они давали предсказуемый результат.
• Читаемость и поддерживаемость: код тестов должен быть понятен, с логичной структурой и комментариями при необходимости.
• Изоляция: тесты должны быть независимыми друг от друга, не влиять на состояние внешних систем без необходимости.
• Использование правильных ассерт-методов: проверки должны быть точными и информативными.
• Отсутствие избыточных ожиданий и таймаутов: чтобы тесты не тормозили сборку.

Красные флаги:

• Жёстко захардкоженные данные, которые сложно менять.
• Сложные и длинные тесты, которые трудно понять.
• Использование sleep вместо явных ожиданий.
• Тесты, которые зависят от внешних сервисов без моков.
• Отсутствие очистки после теста (например, не удаляются созданные данные).

Пример плохого паттерна:

// Использование Thread.sleep вместо ожидания
Thread.sleep(5000);
assertTrue(element.isDisplayed());

Лучше использовать явные ожидания.