Sprint 5 by Alexey3234 · Pull Request #1 · Alexey3234/python5

Alexey3234 · 2025-07-24T12:39:40Z

No description provided.

DolgoarshinnykhLG · 2025-07-24T16:43:59Z

confest.py

+@pytest.fixture
+def base_url():
+    """Базовый URL тестируемого приложения"""
+    return "https://stellarburgers.nomoreparties.site/"


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

DolgoarshinnykhLG · 2025-07-24T16:44:09Z

confest.py

+@pytest.fixture
+def base_url():
+    """Базовый URL тестируемого приложения"""
+    return "https://stellarburgers.nomoreparties.site/"


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

DolgoarshinnykhLG · 2025-07-24T16:44:23Z

test_constructor.py

+class TestConstructorNavigation:
+    def test_buns_section(self, driver, base_url):
+        driver.get(base_url)
+        time.sleep(2)


Нужно исправить: использование sleep не допускается. Стоит посмотреть в сторону нативных expected conditions в selenium

DolgoarshinnykhLG · 2025-07-24T16:44:37Z

test_constructor.py

+        driver.get(base_url)
+        time.sleep(2)
+
+        driver.find_element(By.XPATH, "//span[text()='Соусы']").click()


Нужно исправить: все локаторы следует хранить в отдельном модуле locators. Так мы сможем их переиспользовать в других местах и не править каждый при изменениях на странице

DolgoarshinnykhLG · 2025-07-24T16:44:55Z

test_constructor.py

+    def test_buns_section(self, driver, base_url):
+        driver.get(base_url)
+        time.sleep(2)
+
+        driver.find_element(By.XPATH, "//span[text()='Соусы']").click()
+        time.sleep(1)
+
+        driver.find_element(By.XPATH, "//span[text()='Булки']").click()
+        time.sleep(1)
+
+        active_tab = driver.find_element(By.CSS_SELECTOR, ".tab_tab__1SPyG.tab_tab_type_current__2BEPc")
+        assert "Булки" in active_tab.text, "Раздел 'Булки' не активирован"
+
+    def test_sauces_section(self, driver, base_url):
+        driver.get(base_url)
+        time.sleep(2)
+
+        driver.find_element(By.XPATH, "//span[text()='Соусы']").click()
+        time.sleep(1)
+
+        active_tab = driver.find_element(By.CSS_SELECTOR, ".tab_tab__1SPyG.tab_tab_type_current__2BEPc")
+        assert "Соусы" in active_tab.text, "Раздел 'Соусы' не активирован"
+
+    def test_toppings_section(self, driver, base_url):
+        driver.get(base_url)
+        time.sleep(2)
+
+        driver.find_element(By.XPATH, "//span[text()='Начинки']").click()
+        time.sleep(1)
+
+        active_tab = driver.find_element(By.CSS_SELECTOR, ".tab_tab__1SPyG.tab_tab_type_current__2BEPc")
+        assert "Начинки" in active_tab.text, "Раздел 'Начинки' не активирован"


Можно лучше: шаги этих тестов схожи. Можно их объединить с помощью параметризации

DolgoarshinnykhLG · 2025-07-24T16:45:50Z

test_logout.py

+from selenium.webdriver.support.ui import WebDriverWait
+from selenium.webdriver.support import expected_conditions as EC
+
+def test_logout_from_account(driver, base_url, test_credentials):


Нужно исправить здесь и далее: Для корректного запуска тестов необходимо код с шагами теста поместить в тестовый метод (нейминг начинается с test_), а метод - в тестовый класс (нейминг начинается с Test). Необходимо привести к такому формату все модули внутри пакета tests

DolgoarshinnykhLG · 2025-07-24T16:46:44Z

test_logout.py

+from selenium.webdriver.support import expected_conditions as EC
+
+def test_logout_from_account(driver, base_url, test_credentials):
+    try:


Необходимо исправить: не стоит использовать в теле теста try-except конструкцию. Если цель - сделать скриншот при падении, то стоит это делать на уровне фикстуры/хука

DolgoarshinnykhLG · 2025-07-24T16:47:08Z

test_navigation.py

+    def login(self, driver, base_url, test_credentials):
+        driver.get(f"{base_url}login")
+        time.sleep(2)
+
+        driver.find_element(By.XPATH, "//input[@name='name']").send_keys(test_credentials["email"])
+        driver.find_element(By.XPATH, "//input[@type='password']").send_keys(test_credentials["password"])
+        time.sleep(1)
+
+        driver.find_element(By.XPATH, "//button[text()='Войти']").click()
+        time.sleep(2)


Необходимо исправить: в модуле с тестами должны быть только тесты. Фикстуры стоит вынести в модуль conftest, а вспомогательные методы - в helpers

DolgoarshinnykhLG · 2025-07-24T16:47:38Z

test_registration.py

+    submit_button = driver.find_element(By.XPATH, "//button[text()='Зарегистрироваться']")
+    submit_button.click()
+
+    WebDriverWait(driver, 10).until(EC.visibility_of_element_located((By.XPATH, "//h2[text()='Вход']")))


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

DolgoarshinnykhLG · 2025-07-24T16:48:01Z

test_registration.py

+    name_input = driver.find_element(By.XPATH, "//fieldset[1]//input")
+    email_input = driver.find_element(By.XPATH, "//fieldset[2]//input")
+    password_input = driver.find_element(By.XPATH, "//fieldset[3]//input")


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

DolgoarshinnykhLG · 2025-07-28T17:46:03Z

conftest.py

+@pytest.fixture
+def base_url():
+    return BASE_URL


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

DolgoarshinnykhLG · 2025-07-28T17:46:21Z

helpers.py

+
+def login(driver, base_url, test_credentials):
+    driver.get(f"{base_url}login")
+    time.sleep(2)


Нужно исправить: лучше обойтись без time.sleep и посмотреть в сторону явных ожиданий selenium`а (expected conditions)

DolgoarshinnykhLG · 2025-07-28T17:47:10Z

test_constructor.py

+        ).click()
+
+        # Финальная проверка активной вкладки "Булки"
+        WebDriverWait(driver, 10).until(


Необходимо исправить: этот тест ничего не проверяет. Стоит использовать assert-инструкцию

DolgoarshinnykhLG · 2025-07-28T17:47:44Z

test_login.py

+    def perform_login(self, driver, email, password):
+        """Вспомогательный метод для выполнения входа"""
+        email_field = WebDriverWait(driver, 10).until(
+            EC.presence_of_element_located((By.XPATH, "//input[@name='name']"))
+        )
+        email_field.clear()
+        email_field.send_keys(email)
+
+        password_field = driver.find_element(By.XPATH, "//input[@type='password']")
+        password_field.clear()
+        password_field.send_keys(password)
+
+        login_button = driver.find_element(By.XPATH, "//button[text()='Войти']")
+        login_button.click()
+
+        WebDriverWait(driver, 10).until(
+            EC.presence_of_element_located((By.XPATH, "//button[text()='Оформить заказ']"))
+        )


Необходимо исправить: в модуле с тестами должны быть только тесты. Фикстуры стоит вынести в модуль conftest, а вспомогательные методы - в helpers

DolgoarshinnykhLG · 2025-07-28T17:47:55Z

test_login.py

+    def test_login_from_main_page_button(self, driver, registered_user):
+        """Тест входа через кнопку на главной странице"""
+        driver.get(BASE_URL)
+        driver.find_element(By.XPATH, "//button[text()='Войти в аккаунт']").click()


Нужно исправить: все локаторы следует хранить в отдельном модуле locators. Так мы сможем их переиспользовать в других местах и не править каждый при изменениях на странице

DolgoarshinnykhLG · 2025-07-28T17:47:59Z

test_login.py

+    def test_login_from_personal_account_button(self, driver, registered_user):
+        """Тест входа через кнопку личного кабинета"""
+        driver.get(BASE_URL)
+        driver.find_element(By.XPATH, "//a[@href='/account']").click()


Нужно исправить: все локаторы следует хранить в отдельном модуле locators. Так мы сможем их переиспользовать в других местах и не править каждый при изменениях на странице

DolgoarshinnykhLG · 2025-07-28T17:48:03Z

test_login.py

+    def test_login_from_registration_form(self, driver, registered_user):
+        """Тест входа через форму регистрации"""
+        driver.get(f"{BASE_URL}register")
+        driver.find_element(By.XPATH, "//a[text()='Войти']").click()


Нужно исправить: все локаторы следует хранить в отдельном модуле locators. Так мы сможем их переиспользовать в других местах и не править каждый при изменениях на странице

DolgoarshinnykhLG · 2025-07-28T17:48:07Z

test_login.py

+    def test_login_from_forgot_password_form(self, driver, registered_user):
+        """Тест входа через форму восстановления пароля"""
+        driver.get(f"{BASE_URL}forgot-password")
+        driver.find_element(By.XPATH, "//a[text()='Войти']").click()


Нужно исправить: все локаторы следует хранить в отдельном модуле locators. Так мы сможем их переиспользовать в других местах и не править каждый при изменениях на странице

DolgoarshinnykhLG · 2025-07-28T17:48:56Z

test_registration.py

+    @pytest.fixture
+    def temp_credentials(self):
+        """Генерация уникальных учетных данных для теста"""
+        timestamp = int(time.time() * 1000)
+        return {
+            "name": f"User{timestamp}",
+            "email": f"user{timestamp}@example.com",
+            "password": f"Pass{timestamp % 10000}"
+        }


Необходимо исправить: в модуле с тестами должны быть только тесты. Фикстуры стоит вынести в модуль conftest, а вспомогательные методы - в helpers. Этот метод больше выглядит как функция генерации данных, которую стоит вынести в helpers

…могательные методы в helpers,убрал фикстуры с простой логикой, откорректировал тесты без time.sleep

DolgoarshinnykhLG · 2025-07-29T17:58:20Z

test_constructor.py

+        WebDriverWait(driver, 10).until(
+            EC.visibility_of_element_located(ConstructorPageLocators.BUNS_SECTION)
+        )    


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

Alexey3234 added 6 commits July 24, 2025 14:44

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

6703b5c

Обновление README перед сменой ветки

7dc84f4

Add PR trigger change

b76376c

Разрешение конфликта в README.md

93c76aa

Обновление README.md из main

b3db1a4

Явное изменение для активации сравнения веток

bc9e1e3

DolgoarshinnykhLG reviewed Jul 24, 2025

View reviewed changes

Alexey3234 added 2 commits July 28, 2025 17:10

Исправления кода согласно замечаниям ревьювера

b520602

Исправления кода согласно замечаниям ревьювера итоговая версия

179273f

DolgoarshinnykhLG reviewed Jul 28, 2025

View reviewed changes

DolgoarshinnykhLG reviewed Jul 29, 2025

View reviewed changes

Conversation

Alexey3234 commented Jul 24, 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!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Comments