Sprint_5 by DimPonLe · Pull Request #1 · DimPonLe/Sprint_5

DimPonLe · 2025-10-18T14:31:52Z

No description provided.

DolgoarshinnykhLG · 2025-10-19T14:24:23Z

__pycache__/conftest.cpython-313-pytest-8.4.2.pyc

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

DolgoarshinnykhLG · 2025-10-19T14:24:28Z

tests/__pycache__/test_constructor_section.cpython-313-pytest-8.4.2.pyc

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

DolgoarshinnykhLG · 2025-10-19T14:24:51Z

conftest.py

+# Фикстура для страницы регистрации
+@pytest.fixture()
+def page_register(browser):
+    browser.get("https://stellarburgers.education-services.ru/register")


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

DolgoarshinnykhLG · 2025-10-19T14:25:10Z

conftest.py

+@pytest.fixture()
+def logged_in_account(page_home):
+    # Переход на страницу входа
+    page_home.find_element(By.XPATH, ".//button[text()='Войти в аккаунт']").click()


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

DolgoarshinnykhLG · 2025-10-19T14:25:28Z

conftest.py

+    page_home.find_element(By.XPATH, ".//button[text()='Войти в аккаунт']").click()
+    WebDriverWait(page_home, 5).until(expected_conditions.text_to_be_present_in_element((By.XPATH, ".//h2"), "Вход"))
+    # Ввод данных пользователя
+    page_home.find_element(By.XPATH, ".//fieldset[1]//input").send_keys("1234567890@ya.ru")  # поле Email


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

DolgoarshinnykhLG · 2025-10-19T14:25:44Z

tests/test_constructor_section.py

+
+# проверка перехода в Булки, Соусы, Начинки через скроллинг
+# @pytest.mark.parametrize("index,text", [(1, "Булки"), (2, "Соусы"), (3, "Начинки")])
+def test_scrolling_bread_sauce_topping(logged_in_account, index, text): 


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

DolgoarshinnykhLG · 2025-10-19T14:27:00Z

tests/test_login_page.py

+def test_button_go_in_accaunt_valid_existing_email_password_construction_section(page_home):
+    page_home.find_element(By.XPATH, ".//button[text()='Войти в аккаунт']").click()
+    WebDriverWait(page_home, 5).until(expected_conditions.text_to_be_present_in_element((By.XPATH, ".//h2"), "Вход")) # второй заголовок Вход
+    page_home.find_element(By.XPATH, ".//fieldset[1]//input").send_keys("1234567890@ya.ru")  # поле Email


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

DolgoarshinnykhLG · 2025-10-19T14:28:29Z

tests/test_login_page.py

+    page_home.find_element(By.NAME, "Пароль").send_keys("qwerty")  # поле Пароль
+    page_home.find_element(By.XPATH, ".//button[text()='Войти']").click()  # кнопка Войти
+    WebDriverWait(page_home, 5).until(expected_conditions.text_to_be_present_in_element((By.XPATH, ".//section[2]//button"), "Оформить заказ"))
+    assert page_home.find_element(By.XPATH, ".//h2").text != "Регистрация", "Заголовок 'Регистрация' остался на странице!"


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

DolgoarshinnykhLG · 2025-10-19T14:29:27Z

tests/test_login_page.py

+    try:
+        WebDriverWait(page_register, 3).until(expected_conditions.visibility_of_element_located((By.XPATH, "//p[contains(text(), 'Такой пользователь уже существует')]")))
+        # Если сообщение появилось, меняем имя пользователя и повторяем попытку
+        new_name = f"{user_data['name']} ({random.randint(100, 999)})"
+        user_data["name"] = new_name
+        # Повторная попытка регистрации
+        page_register.find_element(By.XPATH, ".//fieldset[1]//input").clear()
+        page_register.find_element(By.XPATH, ".//fieldset[1]//input").send_keys(new_name)  # новое имя
+        page_register.find_element(By.XPATH, ".//button[text()='Зарегистрироваться']").click()  # кнопка Зарегистрироваться
+    except TimeoutException:
+        pass  # Если сообщение не появилось, продолжаем
+    # Ждём заголовок Вход


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

DolgoarshinnykhLG · 2025-10-19T14:29:59Z

tests/test_registration_page.py

+    try:
+        WebDriverWait(page_register, 3).until(expected_conditions.visibility_of_element_located((By.XPATH, "//p[contains(text(), 'Такой пользователь уже существует')]")))
+        # Если сообщение появилось, меняем имя пользователя и повторяем попытку
+        new_name = f"{user_data['name']}{random.randint(100, 999)}"
+        user_data["name"] = new_name
+        # Повторная попытка регистрации
+        page_register.find_element(By.XPATH, ".//fieldset[1]//input").clear()
+        page_register.find_element(By.XPATH, ".//fieldset[1]//input").send_keys(new_name)  # новое имя
+        page_register.find_element(By.XPATH, ".//button[text()='Зарегистрироваться']").click()  # кнопка Зарегистрироваться
+    except TimeoutException:
+        pass  # Если сообщение не появилось, продолжаем


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

DolgoarshinnykhLG · 2025-10-22T05:21:29Z

.pytest_cache/.gitignore

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

DolgoarshinnykhLG · 2025-10-22T05:21:37Z

tests/.pytest_cache/.gitignore

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

DolgoarshinnykhLG · 2025-10-22T05:22:29Z

locators.py

+REG_INPUT_NAME = (By.XPATH, ".//fieldset[1]//input")
+# поле Email
+REG_INPUT_EMAIL = (By.XPATH, ".//fieldset[2]//input")
+# поле Пароль


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

First commit

77d8803

DolgoarshinnykhLG reviewed Oct 19, 2025

View reviewed changes

Delete files .pyc

2513110

DolgoarshinnykhLG reviewed Oct 22, 2025

View reviewed changes

Comments

Conversation

DimPonLe commented Oct 18, 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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants