Skip to content

Conversation

@b3b
Copy link

@b3b b3b commented Nov 1, 2025

What kind of change does this PR introduce?

Tests: adds an integration suite using the psql client to verify PGOPTIONS parsing and propagation.

What is the current behavior?

Existing integration tests only cover protocol-level behavior.
PGOPTIONS handling is currently untested end-to-end, which makes it easy for regressions in startup option parsing to go unnoticed.

What is the new behavior?

This suite launches real psql sessions against Supavisor and verifies the fixes introduced in previous PRs:

CI and configuration changes

  • Ensures psql client is installed in GitHub Actions workflow.
  • Increases max_pools up to 10 to accommodate new test tenants.
  • Tests will skip gracefully if psql is unavailable locally.

Additional context

Tests for known issues that are not yet fixed are marked as skipped:

- Adds an integration suite that verifies correct behavior of PGOPTIONS
  parsing and propagation.
- Updates CI workflow to ensure the psql client is available for integration runs.
- Increases test max_pools to accommodate additional tenants.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant