Код-ревью Senior Баг-репорты обновлено
Ревью баг-репорта: почти хороший
Вопрос
Перед тобой баг-репорт. Найди, чего в нём не хватает и что сформулировано плохо. Отметь строки и напиши замечания. Репорт почти хороший, ищи тонкие проблемы.
Заголовок: Оплата: при повторном нажатии «Оплатить» деньги списываются дважды
Окружение: stage, сборка 2.14.3, Chrome 129, Windows 11
Предусловия: пользователь user7@test.ru, в корзине 1 товар за 990 ₽
Шаги:
1. Перейти к оформлению заказа
2. Ввести тестовую карту 4111 1111 1111 1111
3. Дважды быстро нажать «Оплатить»
Фактический результат: создано два платежа по 990 ₽, а также не работает кнопка «Назад» и в футере опечатка
Ожидаемый результат: создаётся один платёж, повторное нажатие игнорируется
Серьёзность: Minor
Приоритет: High
Коротко
Тестовый номер карты здесь нормален: это публичная тестовая карта, а не данные клиента. Главные проблемы тоньше: смешанные дефекты и заниженная серьёзность.
Что должен найти ревьюер
- Стильстрока 8В одном репорте три разных дефекта: кнопка «Назад» и опечатка должны быть отдельными репортами
- Ошибкастрока 10Двойное списание это потеря денег: серьёзность Critical, а не Minor
- ОшибкаНе хватает доказательства: ID двух платежей, запросы из Network или логи платёжного шлюза
- ОшибкаНе указано, проверено ли на проде или только на stage с тестовым шлюзом, и воспроизводится ли через API
Разбор
Тестовый номер карты здесь нормален: это публичная тестовая карта, а не данные клиента. Главные проблемы тоньше: смешанные дефекты и заниженная серьёзность. Для денежного бага стоит приложить идентификаторы платежей, чтобы разработчик нашёл их в логах шлюза.
Ответьте сами, а не прочитайте
В тренажёре вопрос попадёт в повторения и вернётся, когда начнёт забываться. Плюс билеты с зачётом: 20 вопросов, не больше 2 ошибок.