postgres.git / summary / log / commit / refs

commit    22c6a44f01fd751c2ed229be4003f7c26d671933
Author:   Noah Misch <noah@leadboat.com>
Date:     Sat Aug 23 23:46:20 2025 +0000
First released in: 13.23, 14.20, 15.15, 16.11, 17.7, 18.0

    Rewrite previous commit's test for TestUpgradeXversion compatibility.
    
    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 | 4 ++--
 src/test/regress/sql/privileges.sql      | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/test/regress/expected/privileges.out b/src/test/regress/expected/privileges.out
index 543efe5cb87..06c99fd9e63 100644
--- a/src/test/regress/expected/privileges.out
+++ b/src/test/regress/expected/privileges.out
@@ -2528,9 +2528,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);
diff --git a/src/test/regress/sql/privileges.sql b/src/test/regress/sql/privileges.sql
index bcc49e88e05..be14731ec24 100644
--- a/src/test/regress/sql/privileges.sql
+++ b/src/test/regress/sql/privileges.sql
@@ -1536,9 +1536,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


[parent: 4948bb9df421]