Skip to content

Добавлены API-тесты#1

Open
aporguneva wants to merge 1 commit into
mainfrom
develop2
Open

Добавлены API-тесты#1
aporguneva wants to merge 1 commit into
mainfrom
develop2

Conversation

@aporguneva

Copy link
Copy Markdown
Owner

No description provided.

@klushcheva klushcheva left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Сильные стороны:

  • Код запускается без ошибок
  • Написанные тесты атомарны, проверяется тело и код отчета, там где это необходимо
  • Тестовые данные создаются перед тестом и удаляются после того, как он выполнится

Comment thread requirements.txt
pytest
requests
Faker
python-dotenv

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Можно лучше: использовать pip freeze для фиксации версий библиотек

Comment thread tests/test_create_user.py
@allure.title("Ошибка при отсутствии почты")
@allure.description("Проверка, что система не позволяет создать пользователя при отсутствии обязательного поля email, система возвращает ошибку 403")

def test_create_missing_email(self, cleanup_user):

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Можно лучше: этот тест параметризовать вместе с test_create_missing_password, так как логика одинаковая, отличие только в тестовых данных

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