Conversation
| payload = { | ||
| "ingredients": random_ingredients | ||
| } | ||
| headers = {"Authorization": user['accessToken']} if need_auth else {} |
There was a problem hiding this comment.
Нужно исправить: if, else в тестовых сценариях недопустимы, т.к. условия предполагают разные варианты исхода. А значит это должны быть два отдельных тестовых метода
| } | ||
| response = requests.post(CREATE_USER_URL, data=payload) | ||
| body = response.json() | ||
| return { |
There was a problem hiding this comment.
Нужно исправить: отсутствует функционал удаления пользователя после выполнения тестового метода.
| } | ||
| response = requests.post(CREATE_ORDER_URL, data=payload, headers={"Authorization": user['accessToken']}) | ||
|
|
||
| assert response.status_code == 500 |
There was a problem hiding this comment.
Нужно исправить: проверки лишь статуса кода недостаточно. Необходимо также парсить ответ.
|
|
||
| class TestCreateUser: | ||
|
|
||
| payload = [ |
There was a problem hiding this comment.
Можно лучше: здесь и далее: тестовые данные стоит хранить отдельно в data модуле, так поддерживать и переиспользовать будет проще
|
|
||
| class TestUpdateUserData: | ||
|
|
||
| fake = Faker("ru_RU") |
There was a problem hiding this comment.
Можно лучше: объект фейкера создается уже не первый раз. По хорошему можно создать хелпер для генерации ланных.
| fake = Faker("ru_RU") | ||
| payload = { | ||
| "email": fake.email(), | ||
| "password": fake.password(), | ||
| "name": fake.name() | ||
| } |
There was a problem hiding this comment.
Можно лучше: этот участок кода повторяется в 3 местах в проекте. СОздавать объект данных лучше в отдельном модуле по типу helpers, откуда просто импортировать по необходимости
| import pytest | ||
|
|
||
|
|
||
| MESSAGE_NO_INGREDIENTS = "Ingredient ids must be provided" |
There was a problem hiding this comment.
Можно лучше: тексты ответов стоит хранить снаружи в data модулях для поддержания и актуализации, а тут вызывать переменные в которые они записаны
| } | ||
| response = requests.post(CREATE_ORDER_URL, data=payload, headers={"Authorization": user['accessToken']}) | ||
|
|
||
| assert response.status_code == 500 |
There was a problem hiding this comment.
В данном тесте сервер просто вернет код ответа, тела у него не будет
2 participants
No description provided.