Skip to content

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Oct 20, 2022
Merged

Add schedule for regression tests. #82

merged 2 commits into from
Oct 20, 2022

Conversation

danolivo
Copy link
Collaborator

The real reason is to successfully pass the statement_timeout test in very slow environments.

@danolivo danolivo self-assigned this Oct 20, 2022
@MarinaPolyakova
Copy link

MarinaPolyakova commented Oct 20, 2022 •
edited
Loading

  1. Исправила запуск тестов с USE_PGXS (= подглядела, как --schedule используется в src/test/regress/GNUMakefile в исходниках postgres).
  2. Проверила, что при ошибке тест statement_timeout игнорируется.
  3. Зачем нужно выносить feature_subspace отдельно в REGRESS я не знаю - видимо, разработчики aqo знают лучше..
  4. 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 при сборке тоже не помешали бы..

@MarinaPolyakova
Copy link

MarinaPolyakova commented Oct 20, 2022 •
edited
Loading

Зачем нужно выносить feature_subspace отдельно в REGRESS я не знаю - видимо, разработчики aqo знают лучше..

Поменяла работу с переменной REGRESS, добавив фиктивный тест. @danolivo , если не нравится, то откати, пожалуйста.

@danolivo
Copy link
Collaborator Author

TAP tests not enabled. Try configuring with --enable-tap-tests
Сделал, спасибо.

@danolivo
Copy link
Collaborator Author

если не нравится, то откати, пожалуйста
В целом не нравится, но давай поживем пока с этим

@MarinaPolyakova
Copy link

если не нравится, то откати, пожалуйста
В целом не нравится, но давай поживем пока с этим

Изменение для REGRESS чисто для удобства разработки. Так что если не нравится, то его лучше откатить, оно косметическое..

@MarinaPolyakova
Copy link

Починила configure. COPT=-Werror протестирован в #85.

@danolivo
Add schedule for regression tests instead of list of tests in the REG…
aeb20b1 
…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.
@danolivo
Update github actions file (c-cpp.yml):
7f61233 
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.
@danolivo danolivo merged commit 4d6192e into master Oct 20, 2022
@danolivo danolivo deleted the add-schedule branch February 1, 2023 04:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MarinaPolyakova MarinaPolyakova Awaiting requested review from MarinaPolyakova

Assignees

@danolivo danolivo

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@danolivo @MarinaPolyakova