-
Notifications
You must be signed in to change notification settings - Fork 55
Add schedule for regression tests. #82
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
Conversation
fbd2a90 to
e3b8514
Compare
- Исправила запуск тестов с USE_PGXS (= подглядела, как --schedule используется в src/test/regress/GNUMakefile в исходниках postgres).
- Проверила, что при ошибке тест statement_timeout игнорируется.
- Зачем нужно выносить feature_subspace отдельно в REGRESS я не знаю - видимо, разработчики aqo знают лучше..
- FYI у вас в CI не тестируются TAP тесты: см. https://github.com/postgrespro/aqo/actions/runs/3286965942/jobs/5415628695:
TAP tests not enabled. Try configuring with --enable-tap-tests
--enable-cassert и COPT=-Werror при сборке тоже не помешали бы..
Зачем нужно выносить feature_subspace отдельно в REGRESS я не знаю - видимо, разработчики aqo знают лучше..
Поменяла работу с переменной REGRESS, добавив фиктивный тест. @danolivo , если не нравится, то откати, пожалуйста.
TAP tests not enabled. Try configuring with --enable-tap-tests
Сделал, спасибо.
если не нравится, то откати, пожалуйста
В целом не нравится, но давай поживем пока с этим
MarinaPolyakova
commented
Oct 20, 2022
если не нравится, то откати, пожалуйста
В целом не нравится, но давай поживем пока с этим
Изменение для REGRESS чисто для удобства разработки. Так что если не нравится, то его лучше откатить, оно косметическое..
MarinaPolyakova
commented
Oct 20, 2022
Починила configure. COPT=-Werror протестирован в #85.
...RESS variable. The real reason is to successfully pass the statement_timeout test in very slow environments. We must inialize REGRESS. So, add an empty dummy test just to define the variable. regress_schedule contains the full list of real tests. So all changes for real tests will be made in a general way in regress_schedule. Authors: a.lepikhov, m.polyakova.
1. Enable TAP-tests 2. Add some useful options for configure and build stages. 3. Parameterize github CI, just to reduce code duplication. Authors: m.polyakova (mostly), and a.lepikhov.
67e8c73 to
7f61233
Compare
The real reason is to successfully pass the statement_timeout test in very slow environments.