Skip to content

Develop#1

Open
MixaTorres7 wants to merge 10 commits intomainfrom
develop
Open

Develop#1
MixaTorres7 wants to merge 10 commits intomainfrom
develop

Conversation

@MixaTorres7
Copy link
Owner

No description provided.


def test_login_from_main_page(self, driver, base_url):
"""Вход через кнопку «Войти в аккаунт» на главной."""
print(f"=== Тест: Вход с главной страницы ===")

Choose a reason for hiding this comment

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

Нужно исправить здесь и далее:
в проекте не должно быть принтов. Для логирования можно использовать логгеры

print("✓ Форма регистрации загружена")

# Используем реальные данные пользователя
test_data = {

Choose a reason for hiding this comment

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

Нужно исправить:
нужно использовать сгенерированный email

class TestNavigation:

@pytest.fixture
def login_user(self, driver, base_url):

Choose a reason for hiding this comment

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

Нужно исправить:
фикстуры должны находиться в conftest

print("✓ Кнопка 'Зарегистрироваться' найдена и кликабельна")

# Делаем скриншот перед кликом
driver.save_screenshot("before_registration.png")

Choose a reason for hiding this comment

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

Нужно исправить
скриншоты не нужны

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