Skip to content

Conversation

@PeeachPie
Copy link
Collaborator

No description provided.

@PeeachPie
Copy link
Collaborator Author

При попытке сделать POST запрос в файле test_training.py получается ошибка
err.txt.

Что я делаю:

  • запускаю проект при помощи docker compose, как описано на вики с конфигом config.ini
  • собираю контейнер с селениумом docker build -t selenium -f Dockerfile_selenium .
  • запускаю селениум docker run selenium

@PeeachPie PeeachPie linked an issue Jul 7, 2025 that may be closed by this pull request
Copy link
Collaborator

Choose a reason for hiding this comment

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

Для информации (исправлять не нужно) - для разных образов (точнее Dockerfile'ов) можно указывать свои .dockerignore (ссылка)

Copy link
Collaborator

Choose a reason for hiding this comment

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

  1. Организуйте запуск тест-кейсов аналогично примеру - в дальнейшем у нас будут появляться новые тестовые сценарии, для которых потребуется централизованный запуск (хотя сейчас это и будет только один)
  2. Вынесите базовую логику теста в отдельный класс
    • так будет проще реализовывать следующие тесты
    • "TestBasicTraining" по факту уже не "basic", а конкретный TestTraining

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Залил коммит. Запуск тест кейсов как в примере, наверное, характерен для unittest.
Я сделал 3 файла:

  • selenium_session - базовая логика для любых будущих селениум тестов
  • training_session - базовая логика для тестирования тренировок, основанная на selenium_session, возможно избыточно
  • test_simple_training - тест, с которого все началось

Мне кажется, для новых тестов будет проще скопировать test_simple_training и использовать SeleniumSession или TrainingSession

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Просто main.py в случае pytest как будто бы избыточен, я могу ошибаться

Copy link
Collaborator Author

@PeeachPie PeeachPie Jul 10, 2025

Choose a reason for hiding this comment

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

Сделал немного мудрее. Теперь Training это просто обертка на SeleniumSession использующая экземпляр последнего. Соотвественно обертки можно менять по ходу тестирования. Более того тесты могут наследоваться, что поможет сформировать четкую иерархию

Copy link
Collaborator

@HadronCollider HadronCollider left a comment

Choose a reason for hiding this comment

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

Логика и код - ок, добавьте readme для тестов с описанием классов/логики и запуска

@HadronCollider HadronCollider merged commit 981dc1c into master Jul 22, 2025
3 checks passed
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.

Исправить селениум тесты

3 participants