Код-ревью Senior Баг-репорты обновлено
Ревью баг-репорта с проды
Вопрос
Перед тобой баг-репорт. Найди, чего в нём не хватает и что сформулировано плохо. Отметь строки и напиши замечания.
Заголовок: Клиент не видит заказ в истории
Окружение: прод
Предусловия: клиент Иванова Мария, тел. +7 912 000-00-00, пароль Masha1990
Шаги:
1. Войти под клиентом
2. Открыть «Мои заказы»
Фактический результат: заказа от вчера нет
Ожидаемый результат: заказ есть
Вложение: HAR-файл сессии клиента
Коротко
Самое опасное в этом репорте не неполнота, а утечка: имя, телефон и пароль клиента и токены сессии в трекере, который видят десятки людей. Идентифицировать клиента достаточно по внутреннему ID, а заказ по номеру.
Что должен найти ревьюер
- Безопасностьстрока 3Персональные данные и пароль клиента в открытом виде в трекере: нарушение 152-ФЗ и безопасности, нужно удалить и сменить пароль
- Безопасностьстрока 3Тестировщик не должен входить под чужой учёткой с паролем клиента: нужен идентификатор клиента и вход через служебный доступ
- Безопасностьстрока 9HAR содержит куки и токены сессии клиента: вычистить перед вложением
- Ошибкастрока 7Нет номера заказа, даты и времени, статуса оплаты: без них заказ не найти в логах и БД
- Ошибкастрока 2Нет версии приложения или сайта и канала (веб, iOS, Android)
Разбор
Самое опасное в этом репорте не неполнота, а утечка: имя, телефон и пароль клиента и токены сессии в трекере, который видят десятки людей. Идентифицировать клиента достаточно по внутреннему ID, а заказ по номеру.
Ответьте сами, а не прочитайте
В тренажёре вопрос попадёт в повторения и вернётся, когда начнёт забываться. Плюс билеты с зачётом: 20 вопросов, не больше 2 ошибок.