Conversation
conftest.py
Outdated
| if login_response.status_code == 200: | ||
| courier_id = login_response.json().get('id') | ||
| if courier_id: | ||
| CourierMethods.delete_courier(courier_id) No newline at end of file |
There was a problem hiding this comment.
Нужно исправить: фикстура принимает список created_credentials, но не добавляет в него данные из тестов. В результате курьеры не удаляются.
В тестах, где создаётся курьер без create_and_delete_courier, нужно явно добавлять данные в delete_courier.
There was a problem hiding this comment.
изменена фикстура. Теперь создание и удаление курьера полностью инкапсулировано в фикстурах через yield, тесты не управляют очисткой данных, и курьеры гарантированно удаляются после выполнения.
tests/test_courier_login.py
Outdated
| password = courier_data['password'] | ||
|
|
||
| create_response = CourierMethods.create_courier(courier_data) | ||
| assert create_response.status_code == 201 |
There was a problem hiding this comment.
Нужно исправить: этот тест упадет, если не создать нового уникальное курьера
There was a problem hiding this comment.
Для тестов логина используется фикстура create_and_delete_courier, которая перед тестом создаёт нового уникального курьера через register_new_courier_and_return_login_password() (по заданию).
2 participants
Sprint 7: API tests and Allure report