Conversation
There was a problem hiding this comment.
Необходимо исправить: эта директория лишняя в проекте. Необходимо убрать её из ветки и добавить в .gitignore в корне проекта
There was a problem hiding this comment.
Необходимо исправить: эта директория лишняя в проекте. Необходимо убрать её из ветки и добавить в .gitignore в корне проекта
2_task/tests/test_orders.py
Outdated
|
|
||
|
|
||
| @allure.title("Создание заказа с авторизацией") | ||
| def test_create_order_with_auth(): |
There was a problem hiding this comment.
Нужно исправить здесь и далее: Для корректного запуска тестов необходимо код с шагами теста поместить в тестовый метод (нейминг начинается с test_), а метод - в тестовый класс (нейминг начинается с Test). Необходимо привести к такому формату все модули внутри пакета tests
2_task/tests/test_orders.py
Outdated
| @allure.title("Создание заказа с авторизацией") | ||
| def test_create_order_with_auth(): | ||
| user_data, _ = register_user() | ||
| login_response = login_user(user_data["email"], user_data["password"]) |
There was a problem hiding this comment.
Необходимо исправить здесь и далее: необходимо сделать allure-аннотации для каждого метода отправки запроса. При такой реализации подходит как контекст-менеджер with allure.step, так навесить декоратор на соответствующий метод отправки
2_task/tests/test_orders.py
Outdated
|
|
||
| @allure.title("Создание заказа с авторизацией") | ||
| def test_create_order_with_auth(): | ||
| user_data, _ = register_user() |
There was a problem hiding this comment.
Можно лучше здесь и далее: создание пользователя тут является предусловием к тесту, можно вынести его в фикстуры
2_task/tests/test_user_create.py
Outdated
| import allure | ||
| from data.user_data import get_valid_user | ||
|
|
||
| BASE_URL = "https://stellarburgers.education-services.ru" |
There was a problem hiding this comment.
Необходимо исправить: в модулях с тестами должны быть только тесты. Адрес можно вынести в отдельный модуль urls
2_task/tests/test_user_create.py
Outdated
| response = requests.post(f"{BASE_URL}/api/auth/register", json=payload) | ||
|
|
||
| assert response.status_code == 200 | ||
| assert response.json()["success"] is True |
There was a problem hiding this comment.
Необходимо исправить здесь и далее: проверка success is True является хрупкой. Стоит проверять специфичный для конкретного ресурса атрибут в ответе
2_task/tests/test_user_create.py
Outdated
|
|
||
|
|
||
| @allure.title("Создание пользователя без обязательного поля") | ||
| def test_create_user_without_required_field(): |
There was a problem hiding this comment.
Необходимо исправить: проверяются не все обязательные поля
2_task/tests/test_user_login.py
Outdated
| assert "accessToken" in response.json() | ||
|
|
||
|
|
||
| @allure.title("Логин с неверным логином и паролем") |
There was a problem hiding this comment.
Необходимо исправить: не хватает теста на проверку авторизации без обязательных полей
2 participants
No description provided.