Skip to content

test: cover vault grant options - #2390

Open
pjpjq wants to merge 1 commit into
supabase:developfrom
pjpjq:fix/test-vault-grant-option
Open

test: cover vault grant options#2390
pjpjq wants to merge 1 commit into
supabase:developfrom
pjpjq:fix/test-vault-grant-option

Conversation

@pjpjq

@pjpjq pjpjq commented Aug 23, 2026

Copy link
Copy Markdown

Summary

  • include is_grantable in Vault relation and function privilege snapshots
  • cover the standard, Multigres, and Multigres OrioleDB test variants
  • assert that only the postgres Vault grants configured with WITH GRANT OPTION are grantable

Validation

  • ran nix/tests/sql/vault.sql against local supabase/postgres:17.6.1.107
  • compared all 37 ACL result rows with nix/tests/expected/vault.out
  • verified the three Vault SQL variants and their ACL expected-output blocks stay identical
  • full Nix/pg_regress suite not run because nix and pg_regress are not installed locally

Partial progress on #1570

@pjpjq
pjpjq requested review from a team as code owners August 23, 2026 17:19
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