Skip to content

Добавлены файлы по заданию 2 дипломного проекта#1

Open
komorebi-ps wants to merge 2 commits intomainfrom
develop2
Open

Добавлены файлы по заданию 2 дипломного проекта#1
komorebi-ps wants to merge 2 commits intomainfrom
develop2

Conversation

@komorebi-ps
Copy link
Owner

No description provided.

@allure.title('Проверка регистрации пользователя')
def test_create_user(self, create_delete_user):

user_data = create_delete_user
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нужно исправить: Здесь и везде. Фикстура - это объект, который можно рассматривать, как набор условий необходимых тесту для выполнения. То есть она не должна заменять шаги теста. В этом случае тестируются шаги регистрации, поэтому нельзя использовать фикстуру с регистрацией

# создаем пользователей, не указывая обязательное поле
response = UserCreateMethods().create_new_user_and_return_data(user_data)

try:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

нужно исправить: ассерты не должны быть внутри try иначе не получим результатов тестирования


response_data, status_code = UserEditMethods().user_edit(edited_data, user_data['accessToken'])

# проверки при изменении пароля:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нужно исправить: условий в тестах быть не может, тест всегда однозначен. Если это не вписывается в параметризацию, значит ее быть не должно и это просто несколько отдельных тестовых методов

# передаем данные из теста в фикстуру, если они есть
user_data = getattr(request, 'param', None)

if user_data is None:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

нужно вынести в хэлпер генерацию пейлоада, сейчас получилось очень запутанно и сложно понять что в итоге будет протестировано

@@ -0,0 +1,10 @@
REGISTER_URL = "https://stellarburgers.education-services.ru/api/auth/register"

USER_URL = "https://stellarburgers.education-services.ru/api/auth/user"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно лучше: Код станет более поддерживаемым, если вынести адрес сервера в отдельную переменную, а в переменных эндпоинтов оставить только их уникальный путь. Таким образом, если нужно будет тестировать не продакшн или если поменяется url будет достаточно обновить одну переменную

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants