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

Reduce execution time for unit tests #358

Closed
3 tasks
nicl-nno opened this issue Jul 9, 2021 · 3 comments · Fixed by #1098
Closed
3 tasks

Reduce execution time for unit tests #358

nicl-nno opened this issue Jul 9, 2021 · 3 comments · Fixed by #1098
Labels
test The additon or modification of the unit test

Comments

@nicl-nno
Copy link
Collaborator

nicl-nno commented Jul 9, 2021

The longest tests should be investigated to reduce the exection time to 7-8 min.

Obsolete proposals:

  • Define long tests with pytetst -s test/unit/module_name --durations=0
  • Extract tests with 'regression logic' and replace by unit tests
  • Add unified test data access point (Test Data Management) (module or class)
@nicl-nno nicl-nno added the test The additon or modification of the unit test label Jul 9, 2021
@nicl-nno nicl-nno changed the title Reduce execution time for unit test Reduce execution time for unit tests Jul 9, 2021
@nicl-nno
Copy link
Collaborator Author

nicl-nno commented May 2, 2023

From @gkirgizov"

"Есть предложение вынести "тяжелые" тесты и тесты с тяжелыми ML зависимостями типа CV и скачки текстовых эмбеддингов в отдельный GitHub Action, который будет крутиться каждый день, например. И на PR отдельным экшном.
Плюс этого в том, что недетерминированные ошибки с моделями будут регулярно проверяться, а не только один раз на PR"

@IIaKyJIuH
Copy link
Collaborator

Получилось вынести парочку unit-тестов в integration так, чтобы прогон unit-тестов занимал не более 6,5 минут. В то же время, теперь integration-тесты будут занимать до 60 минут реального времени).

Нормально ли это? Есть возможность запускать интеграционные тесты как при approve PR, так и раз в день (12:00 подойдёт?)

@nicl-nno
Copy link
Collaborator Author

Нормально. Думаю пока лучше раз в день.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test The additon or modification of the unit test
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants