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

test: the regression test queries now run faster #50

Merged
merged 1 commit into from
Oct 19, 2024

Conversation

laurenceisla
Copy link
Member

@laurenceisla laurenceisla commented Oct 18, 2024

It doesn't call the main function that generates the OpenAPI spec directly anymore. It saves the output in a table and the tests will query that table instead (via a function).

Test duration before this change:

test info                         ... ok          504 ms
test main                         ... ok          221 ms
test parameters                   ... ok          692 ms
test paths                        ... ok         8585 ms
test reqbodies                    ... ok          794 ms
test responses                    ... ok         3781 ms
test schemas                      ... ok         4682 ms
test security                     ... ok          238 ms
test servers                      ... ok          435 ms

Test duration after this change:

test info                         ... ok            8 ms
test main                         ... ok          221 ms
test parameters                   ... ok            9 ms
test paths                        ... ok           27 ms
test reqbodies                    ... ok            8 ms
test responses                    ... ok           15 ms
test schemas                      ... ok           17 ms
test security                     ... ok            7 ms
test servers                      ... ok          431 ms

It doesn't call the main function that generates the OpenAPI spec directly anymore.
It saves the output in a table and the tests will query that table instead (via a function).
@steve-chavez steve-chavez merged commit b6161a0 into PostgREST:main Oct 19, 2024
3 checks passed
@laurenceisla laurenceisla deleted the tests-perf branch October 19, 2024 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants