Rewrite previous commit's test for TestUpgradeXversion compatibility.
authorNoah Misch <noah@leadboat.com>
Sat, 23 Aug 2025 23:46:20 +0000 (16:46 -0700)
committerNoah Misch <noah@leadboat.com>
Sat, 23 Aug 2025 23:46:24 +0000 (16:46 -0700)
v17 introduced the MAINTAIN ON TABLES privilege.  That changed the
applicable "baseacls" reaching buildACLCommands().  That yielded
spurious TestUpgradeXversion diffs.  Change to use a TYPES privilege.
Types have the same one privilege in all supported versions, so they
avoid the problem.  Per buildfarm.  Back-patch to v13, like that commit.

Discussion: https://postgr.es/m/20250823144505.88.nmisch@google.com
Backpatch-through: 13

src/test/regress/expected/privileges.out
src/test/regress/sql/privileges.sql

index 697fd9d43d66afe22a03ecb77c86e3c89c7b9b38..1c87f03a79d729e3a6517e0a1b97ee312522bbb3 100644 (file)
@@ -2827,9 +2827,9 @@ DROP USER regress_priv_user8; -- does not exist
 ERROR:  role "regress_priv_user8" does not exist
 -- leave some default ACLs for pg_upgrade's dump-restore test input.
 ALTER DEFAULT PRIVILEGES FOR ROLE pg_signal_backend
-   REVOKE INSERT ON TABLES FROM pg_signal_backend;
+   REVOKE USAGE ON TYPES FROM pg_signal_backend;
 ALTER DEFAULT PRIVILEGES FOR ROLE pg_read_all_settings
-   REVOKE INSERT ON TABLES FROM pg_read_all_settings;
+   REVOKE USAGE ON TYPES FROM pg_read_all_settings;
 -- permissions with LOCK TABLE
 CREATE USER regress_locktable_user;
 CREATE TABLE lock_table (a int);
index d964d91ebf01130c755007e94257f658c82814de..0b417955b6abe4a5cf78b814385899394325bd9a 100644 (file)
@@ -1766,9 +1766,9 @@ DROP USER regress_priv_user8; -- does not exist
 
 -- leave some default ACLs for pg_upgrade's dump-restore test input.
 ALTER DEFAULT PRIVILEGES FOR ROLE pg_signal_backend
-   REVOKE INSERT ON TABLES FROM pg_signal_backend;
+   REVOKE USAGE ON TYPES FROM pg_signal_backend;
 ALTER DEFAULT PRIVILEGES FOR ROLE pg_read_all_settings
-   REVOKE INSERT ON TABLES FROM pg_read_all_settings;
+   REVOKE USAGE ON TYPES FROM pg_read_all_settings;
 
 
 -- permissions with LOCK TABLE