Conversation
| response = requests.post(f'{MAIN_URL}/api/v1/courier/login', data=payload) | ||
| id_courier = response.json()['id'] | ||
|
|
||
| assert response.json()['id'] == id_courier No newline at end of file |
There was a problem hiding this comment.
Нужно исправить: нужно добавить сценарий с некорректным логином/паролем
There was a problem hiding this comment.
еще разок обращаю твое внимание, что авторизация с некорректным логином/паролем и авторизация несуществующего пользователя -это разные сценарии
| id_courier = response.json()['id'] | ||
|
|
||
| # Удалить курьера, который был создан | ||
| requests.delete(f'{MAIN_URL}{CREATE_COURIER_URL}/{id_courier}') |
There was a problem hiding this comment.
Нужно исправить: Фикстура пытается удалить курьера, но логика неверна: она сначала авторизуется, а потом удаляет по ID, что может не сработать, если курьер не создан.
There was a problem hiding this comment.
отлично, а если есть фикстура, которая и создает и удаляет курьера, не очень оптимально иметь еще две других отдельных..
| def create_courier(): | ||
| # Создать нового курьера | ||
| payload = {"login": LOGIN, "password": PASSWORD, "firstName": FIRST_NAME} | ||
| requests.post(f'{MAIN_URL}{CREATE_COURIER_URL}', data=payload) No newline at end of file |
There was a problem hiding this comment.
Нужно исправить: лучше переписать фикстуру create_courier, чтобы она создавала курьера и удаляла его после теста через yield.
| response = requests.put(f'{MAIN_URL}{ACCEPT_ORDER_URL}/{order_id}', params=params) | ||
|
|
||
| assert response.status_code == 400 and response.json() == { | ||
| 'code': 400, 'message': 'Недостаточно данных для поиска'} |
There was a problem hiding this comment.
Можно улучшить: здесь и далее тестовые данные, сообщения и урлы лучше хранить в отдельном модуле. Так проще поддерживать
|
|
||
| # Принять заказ | ||
| params = {'courierId': ""} | ||
| response = requests.put(f'{MAIN_URL}{ACCEPT_ORDER_URL}/{order_id}', params=params) |
There was a problem hiding this comment.
Нужно исправить: здесь и далее над всеми методами с http запросами должен стоять декоратор allure step
2 participants
No description provided.