Добавлены тесты by julia7774 · Pull Request #1 · julia7774/sprint_7

julia7774 · 2025-12-19T09:47:54Z

No description provided.

irisqul · 2025-12-20T05:23:10Z

tests/conftest.py

+    if response.status_code != 200:
+        pytest.fail("Ошибка авторизации тестового курьера")
+
+    response = ScooterClient.courier_delete(response.json()["id"])


почему курьер удаляется в предусловии, вероятно нужно после теста это делать?
также имеет смысл выделить фикстуру удаления отдельно, она будет нужна для тестов на создание курьера

irisqul · 2025-12-20T05:24:31Z

tests/test_courier/test_create.py

+        assert response.status_code == 201
+        assert response.json() == {"ok": True}
+
+        with allure.step("Отправить POST запрос на авторизацию созданного курьера"):


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

irisqul · 2025-12-20T05:25:01Z

tests/conftest.py

+
+    response = ScooterClient.courier_login(login, password)
+    if response.status_code != 200:
+        pytest.fail("Ошибка авторизации тестового курьера")


это постусловие, отмечать тест упавшим из-за постусловия - это немного слишком

irisqul · 2025-12-20T05:27:19Z

tests/test_courier/test_login.py

+            "empty"
+        ],
+    )
+    def test_missing(self, courier, use_login, use_password, step):


нужно исправить: название теста непонятное

irisqul · 2025-12-20T05:27:28Z

tests/test_courier/test_login.py

+
+class TestCourierLogin:
+    @allure.title("Проверка авторизации курьера")
+    def test(self, courier):


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

irisqul · 2025-12-20T05:41:14Z

tests/test_courier/test_login.py

+        login, password = courier
+
+        with allure.step(f"Отправить POST запрос на авторизацию курьера без указания: {step}"):
+            response = ScooterClient.courier_login(login if use_login else None, password if use_password else None)


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

irisqul · 2025-12-20T05:41:49Z

tests/test_courier/test_login.py

+            response = ScooterClient.courier_login(login if use_login else None, password if use_password else None)
+
+        if response.status_code == 504:
+            pytest.skip("Некорректная работа API")


ошибки должны обрабатываться в assert, если код неверный то это фейл

irisqul · 2025-12-20T06:09:56Z

tests/test_courier/test_create.py

+        ]
+    )
+    def test_with_missing_fields(self, login, password, firstName, step):
+        with allure.step(f"Отправить POST запрос на создание курьера без указания: {step}"):


отлично: понятные описания шагов для отчета

irisqul · 2025-12-21T11:27:28Z

client.py

+    @staticmethod
+    def courier_create(login, password, firstName):
+        payload = {}
+        if login:


Нужно исправить: условий в тестах быть не должно, мы хотим точно знать какие были шаги

Этот класс не является частью теста. Это просто класс-клиент в котором описаны запросы в api, чтобы их не приходилось копировать в каждом тесте

этот класс вместе со всем ветвлением вызывается в тесте. от того что мы перенесли ветвление и спрятали его внутрь вызываемого метода оно не исчезло

irisqul · 2025-12-21T11:27:46Z

client.py

+    @staticmethod
+    def courier_login(login, password):
+        payload = {}
+        if login:


нужно убрать условия из тестов

irisqul · 2025-12-21T11:27:56Z

client.py

+    @staticmethod
+    def order_list(courierId, nearestStation, limit, page):
+        params = {}
+        if courierId:


irisqul · 2025-12-21T11:29:58Z

tests/test_courier/test_create.py

+
+        assert response.status_code == 201
+        assert response.json() == {"ok": True}
+        courier_delete(login, password)


Нужно исправить: фикстура выполнится сама если в ней есть yield

Здесь не используется фикстура в которой есть yield, так как здесь мы внутри теста создаем нового курьера и вызываем отдельно фикстуру для удаления курьера (очищение данных)

мы можем передавать данные курьера которого создали в тесте обратно в фикстуру

irisqul · 2025-12-21T11:30:26Z

tests/test_courier/test_create.py

+    )
+    def test_with_missing_fields(self, login, password, firstName, step):
+        with allure.step(f"Отправить POST запрос на создание курьера без указания: {step}"):
+            response = ScooterClient.courier_create(login, password, firstName)


не исправлено - ветвление кода осталось

Здесь нет ветвления кода, просто 6 наборов входных параметров

в методе courier_create есть ветвление и оно вызывается внутри теста. пейлоад формируется внутри теста, а не передается входными параметрами.

Мы действительно можем в этом тесте передать 6 пейлоадов для теста и это будет работать. но сейчас реализовано так что внутри теста есть условия и нелинейное поведение

Добавлены тесты

2767798

irisqul reviewed Dec 20, 2025

View reviewed changes

Правки после ревью

140abe8

irisqul reviewed Dec 21, 2025

View reviewed changes

Правки

98445f1

Conversation

julia7774 commented Dec 19, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Comments