Skip to content

Instantly share code, notes, and snippets.

@borodicht
Created December 21, 2025 16:50
Show Gist options
  • Select an option

  • Save borodicht/00c62517885ee30295e88e3288e8965a to your computer and use it in GitHub Desktop.

Select an option

Save borodicht/00c62517885ee30295e88e3288e8965a to your computer and use it in GitHub Desktop.
РЕВЬЮ КОДА
Цель: Получить комплексную оценку кода автоматизации тестирования
Ты — опытный SDET с 10-летним стажем, специализирующийся на Java и фреймворке Selenide. Я хочу, чтобы ты
провел ревью следующего кода.
Проанализируй код по следующим критериям:
1. Читаемость и структура:
Насколько код понятен?
Соблюдены ли принципы чистого кода (Clean Code)?
2. Надежность:
Есть ли в коде потенциальные причины для "плавающих" падений?
3. Производительность:
Можно ли оптимизировать выполнение теста?
4. Следование паттерну Page Object:
Корректно ли вынесены локаторы и действия со страницей?
5. Ассерты:
Достаточны ли проверки? Являются ли они атомарными и осмысленными?
Представь результат в виде списка улучшений с примерами кода "было-стало"
Контекст:
Язык: Java
Фреймворк: Selenide
Тестовая библиотека: JUnit5
Тестируемая функциональность: страница переводов между счетами
Код теста для анализа:
________________________________________________________________________________________
КОММЕНТАРИЙ
Цель: Сформулировать вежливый и конструктивный комментарий для коллеги по результатам ревью.
На основе моего анализа (или на основе предоставленного списка проблем)
напиши конструктивный комментарий для код-ревью в GitHub/GitLab.
Стиль должен быть дружелюбным и поддерживающим. Начни с позитивных моменто
затем перечисли основные моменты для улучшения в виде списка. Для каждого
пункта предложи конкретное решение или задай наводящий вопрос.
Основные проблемы, которые нужно осветить:
1. Используется `thread.sleep()` вместо явных ожиданий
2. Локаторы слишком хрупкие
3. В тесте отсутствует проверка текста ошибки при неверном переводе
4.
Пример:
"Привет! Спасибо за твою работу. Тест выглядит хорошо, покрыва
основной сценарий. Есть пара моментов, которые мы могли бы улучшить..."
_______________________________________________________________________________________
ОНБОРДИНГ
Цель: Быстро понять структуру нового проекта и его ключевые компоненты.
Ты — лид автоматизации, которому нужно быстро погрузиться в новый проект.
Проанализируй предоставленную структуру папок и файлов проекта автоматизаци
Задача:
1. Опиши назначение каждой ключевой папки (`pages`, `tests`, `utils`,
`fixtures` и т.д.).
2. Предположи, какие основные паттерны проектирования применяются в проект
3. Составь список из 3-5 файлов, которые мне стоит изучить в первую очеред
чтобы понять архитектуру.
Структура проекта:
________________________________________________________________________________________
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment