Skip to content

Sprint_7 - #1

Open
Interlude312 wants to merge 3 commits into
mainfrom
develop
Open

Sprint_7#1
Interlude312 wants to merge 3 commits into
mainfrom
develop

Conversation

@Interlude312

Copy link
Copy Markdown
Owner

No description provided.

Евгений added 2 commits August 31, 2026 18:46
Comment thread courier_api.py
login_url = COURIER_LOGIN_URL

def create(self, payload):
return requests.post(self.url, json=payload, timeout=REQUEST_TIMEOUT)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Нужно исправить: здесь и далее над каждым методом с http запросом должен стоять декоратор allure.step

Comment thread test_courier_login.py
@allure.title("Авторизация с неверным паролем возвращает ошибку")
def test_courier_login_with_wrong_password_returns_error_status(self, courier_client, registered_courier):
response = courier_client.login({"login": registered_courier["login"], "password": "wrongPassword123"})
assert response.status_code == STATUS_CODE_NOT_FOUND

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Нужно исправить: здесь и во всех тестах нужно проверить не только статус-код, но и распарсить ответ

@Interlude312

Copy link
Copy Markdown
Owner Author

Исправил

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