Created
December 21, 2025 16:50
-
-
Save borodicht/00c62517885ee30295e88e3288e8965a to your computer and use it in GitHub Desktop.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| РЕВЬЮ КОДА | |
| Цель: Получить комплексную оценку кода автоматизации тестирования | |
| Ты — опытный 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