Sprint 6 by Airflp · Pull Request #2 · Airflp/Sprint_6

Airflp · 2026-03-05T16:39:06Z

No description provided.

DolgoarshinnykhLG · 2026-03-06T11:00:01Z

@@ -0,0 +1,15 @@
+[


Необходимо исправить: эта директория лишняя в проекте. Необходимо убрать её из ветки и добавить в .gitignore в корне проекта

DolgoarshinnykhLG · 2026-03-06T11:00:52Z

+    ORDER_BUTTON_TOP = (By.XPATH, "//button[contains(text(), 'Заказать')]")
+    ORDER_BUTTON_BOTTOM = (By.XPATH, "//button[contains(text(), 'Заказать')]")


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

DolgoarshinnykhLG · 2026-03-06T11:01:10Z

+
+
+class BasePage:
+    BASE_URL = "https://qa-scooter.praktikum-services.ru/"


Можно лучше здесь и далее: адреса лучше вынести во внешний модуль(например, urls). Они могут измениться, что затруднит поддержку тестов

DolgoarshinnykhLG · 2026-03-06T11:01:20Z

+    def __init__(self, driver):
+        self.driver = driver
+
+    def find_element(self, locator, time=10):


Необходимо исправить здесь и далее: не хватает аннотации allure для шага - allure.step

DolgoarshinnykhLG · 2026-03-06T11:03:09Z

+
+    # Локаторы для кнопок заказа
+    ORDER_BUTTON_TOP = (By.XPATH, "//button[@class='Button_Button__ra12g']")
+    ORDER_BUTTON_BOTTOM = (By.XPATH, "(//button[text()='Заказать'])[2]")


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

DolgoarshinnykhLG · 2026-03-06T11:03:43Z

@@ -0,0 +1,9 @@
+from locators.main_page_locators import MainPageLocators
+
+class MainPageLocators:


Необходимо исправить: классы этого пакета нигде не используются

DolgoarshinnykhLG · 2026-03-06T11:04:42Z

+        for domain in ['dzen.ru', 'yandex.ru', 'ya.ru']:
+            if self.wait_for_url_contains(domain, timeout=5):
+                return True
+


Необходимо исправить: проверка для теста получилась не атомарной. Стоит проверять редирект на конкретный адрес

DolgoarshinnykhLG · 2026-03-06T11:05:08Z

+        if order_button == "top":
+            main_page.click_order_button_top()
+        else:
+            main_page.click_order_button_bottom()


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

DolgoarshinnykhLG · 2026-03-06T11:05:35Z

+        assert main_page.check_yandex_redirect_url(), "Не открылась страница Яндекса/Дзена"
+
+        # Закрываем новое окно и возвращаемся к основному
+        main_page.close_current_window_and_switch_to(main_window)


Необходимо исправить: браузер переоткрывается каждый тест. Можно избавиться от этого шага

DolgoarshinnykhLG · 2026-03-08T09:37:01Z

+    def __init__(self, driver):
+        self.driver = driver
+
+    def find_element(self, locator, timeout=10):


Необходимо исправить здесь и далее: не хватает аннотации allure для шага - allure.step

DolgoarshinnykhLG · 2026-03-08T09:38:06Z

+        WebDriverWait(self.driver, 10).until(
+            EC.number_of_windows_to_be(2)
+        )
+
+        for window in self.driver.window_handles:
+            if window != main_window:
+                self.driver.switch_to.window(window)
+                break


Необходимо исправить здесь и далее: все вызовы к driver/WebDriverWait должны быть описаны в методах BasePage, а затем вызываться через эти методы в *Page-классах через наследование. Стоит посмотреть в сторону EC.new_window_is_opened

DolgoarshinnykhLG · 2026-03-08T09:39:45Z

+        if button == "top":
+            main_page.click_order_button_top()
+        else:
+            main_page.click_order_button_bottom()


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

DolgoarshinnykhLG · 2026-03-08T09:40:10Z

+        main_page.click_order_button_top()
+        main_page.click_scooter_logo()
+
+        assert "scooter" in driver.current_url


Нужно исправить: здесь и далее, все вызовы к driver/WebDriverWait напрямую из теста или фикстуры не допустимы (кроме get). Необходимо реализовать соответствующий метод в BasePage и вызывать его

DolgoarshinnykhLG · 2026-03-08T09:40:21Z

+        main_window = driver.current_window_handle
+
+        main_page.click_yandex_logo_and_wait_for_new_window(main_window)
+
+        assert "dzen" in driver.current_url


Нужно исправить: здесь и далее, все вызовы к driver/WebDriverWait напрямую из теста или фикстуры не допустимы (кроме get). Необходимо реализовать соответствующий метод в BasePage и вызывать его

DolgoarshinnykhLG · 2026-03-08T09:41:08Z

+from selenium import webdriver
+
+
+@pytest.fixture(scope="class")


Необходимо исправить: такой скоуп создает зависимые тесты. Стоит использовать скоуп function

Не исправлено

Airflp added 6 commits March 4, 2026 21:45

Sprint 6 project

e2c9f0b

prepare PR

a32b8b0

update project files

7fe6e74

remove all files

531ae62

restore sprint 6 project

9effe1a

Merge branch 'master' into sprint_6

ccf38b4

DolgoarshinnykhLG reviewed Mar 6, 2026

View reviewed changes

Airflp added 2 commits March 8, 2026 10:34

fix review comments

e792131

remove pytest_cache and allure_results from repo

3017e58

DolgoarshinnykhLG reviewed Mar 8, 2026

View reviewed changes

Airflp added 2 commits March 8, 2026 15:07

Fix reviewer comments and stabilize tests

4e3dc06

Fix reviewer comments and stabilize tests

586d37b

		ORDER_BUTTON_TOP = (By.XPATH, "//button[contains(text(), 'Заказать')]")
		ORDER_BUTTON_BOTTOM = (By.XPATH, "//button[contains(text(), 'Заказать')]")



		class BasePage:
		BASE_URL = "https://qa-scooter.praktikum-services.ru/"

		@@ -0,0 +1,9 @@
		from locators.main_page_locators import MainPageLocators

		class MainPageLocators:

		from selenium import webdriver


		@pytest.fixture(scope="class")

Conversation

Airflp commented Mar 5, 2026

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