Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix/pytest git #124

Merged
merged 7 commits into from
Jan 25, 2024
Merged

Bugfix/pytest git #124

merged 7 commits into from
Jan 25, 2024

Conversation

Pogorelii1993
Copy link

Description

Пофиксил pytest:

  • откорректировал тест на проверку страницы logout, после того как мы сделали редирект он работал некорректно.
  • откорректировал / дописал тест на проверку url: team/1, судя по всему он никогда и не работал, т.к. во время запуска теста не создавалось ни одной команды и естественно по id=1 база выдавала ошибку и тест падал.
  • поставил в прекоммит pytest
  • поставил pytest в workflow

Type of change

Пожалуйста, удалите варианты, которые не относятся к ПР-у.

  • Documentation (опечатки, примеры кода или любое обновление документации)
  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

  • проверил работу pytest локально по команде make pytest из корневой директории проекта
  • проверил работу pytest локально по команде poetry run pytest из adaptive_hockey_federation
  • проверил работу pytest при прекоммите.
  • проверить workflow смогу только сейчас при pull request (если будут проблемы сделаю новый коммит)

Checklist:

  • Мой код соответствует code-style данного проекта
  • Я провел самоанализ собственного кода

@@ -76,3 +76,28 @@ jobs:
- name: mypy
run: |
poetry run mypy .

pytest:
Copy link
Member

Choose a reason for hiding this comment

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

Я вроде в таске просил отдельным файлом) Давай отдельным оформим и добавим именно на пулреквест ручку. а в прекоммите оставим. То есть не надо проводить его постоянно при каждом пуше

@@ -36,6 +36,13 @@ repos:

- repo: local
hooks:
- id: run-make-pytest
Copy link
Member

Choose a reason for hiding this comment

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

а еще не знаешь почему такое вылезает?
2024-01-24_21-26

Copy link
Author

@Pogorelii1993 Pogorelii1993 Jan 24, 2024

Choose a reason for hiding this comment

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

Запуская pytest командой poetry run pytest из "adaptive_hockey_federation/adaptive_hockey_federation" работает нормально, если запускать из корня, то да, он не находит файл.
Перепроверь дирректорию мне кажется запускаешь не из той.

Если запускать командой make pytest, то будет нормально и из той дирректории, из которой запускаешь на скрине.

@KonstantinRaikhert KonstantinRaikhert merged commit 0d695ec into dev Jan 25, 2024
7 checks passed
@KonstantinRaikhert KonstantinRaikhert deleted the bugfix/pytest-git branch September 20, 2024 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants