diff options
author | Andrew Dunstan <andrew@dunslane.net> | 2022-06-12 09:17:17 -0400 |
---|---|---|
committer | Andrew Dunstan <andrew@dunslane.net> | 2022-06-12 09:17:17 -0400 |
commit | 19408aae7fa2bf28866fb262b27f08405e71152e (patch) | |
tree | 0a8115ea052bc3dc21be98e1dfd6d88d717af987 | |
parent | 0d932a9d990fdb78d58abc42c03a1e84cfa2facd (diff) | |
download | postgresql-19408aae7fa2bf28866fb262b27f08405e71152e.tar.gz postgresql-19408aae7fa2bf28866fb262b27f08405e71152e.zip |
Make subscription tests pass with log_error_verbosity=verbose
Recent additions to the subscription tests check for log entries, but
fail to account for the possible presence of an SQL errror code, which
happens if log_error_verbosity is set to 'verbose'. Add this into the
regular expressions that are checked for.
-rw-r--r-- | src/test/subscription/t/027_nosuperuser.pl | 10 | ||||
-rw-r--r-- | src/test/subscription/t/029_on_error.pl | 2 |
2 files changed, 6 insertions, 6 deletions
diff --git a/src/test/subscription/t/027_nosuperuser.pl b/src/test/subscription/t/027_nosuperuser.pl index 350bc40efcb..96ec23aab70 100644 --- a/src/test/subscription/t/027_nosuperuser.pl +++ b/src/test/subscription/t/027_nosuperuser.pl @@ -179,7 +179,7 @@ revoke_superuser("regress_admin"); publish_update("alice.unpartitioned", 5 => 9); expect_failure( "alice.unpartitioned", 2, 5, 7, - qr/ERROR: permission denied for table unpartitioned/msi, + qr/ERROR: ( [A-Z0-9]+:)? permission denied for table unpartitioned/msi, "non-superuser admin fails to replicate update"); grant_superuser("regress_admin"); expect_replication("alice.unpartitioned", 2, 7, 9, @@ -214,7 +214,7 @@ expect_failure( 3, 7, 11, - qr/ERROR: permission denied for table unpartitioned/msi, + qr/ERROR: ( [A-Z0-9]+:)? permission denied for table unpartitioned/msi, "non-superuser admin without SELECT privileges fails to replicate update" ); @@ -262,7 +262,7 @@ expect_failure( 2, 11, 13, - qr/ERROR: "regress_admin" cannot replicate into relation with row-level security enabled: "unpartitioned\w*"/msi, + qr/ERROR: ( [A-Z0-9]+:)? "regress_admin" cannot replicate into relation with row-level security enabled: "unpartitioned\w*"/msi, "non-superuser admin fails to replicate insert into rls enabled table"); grant_superuser("regress_admin"); expect_replication("alice.unpartitioned", 3, 11, 15, @@ -276,7 +276,7 @@ expect_failure( 3, 11, 15, - qr/ERROR: "regress_admin" cannot replicate into relation with row-level security enabled: "unpartitioned\w*"/msi, + qr/ERROR: ( [A-Z0-9]+:)? "regress_admin" cannot replicate into relation with row-level security enabled: "unpartitioned\w*"/msi, "non-superuser admin fails to replicate update into rls enabled unpartitioned" ); @@ -291,7 +291,7 @@ expect_failure( 3, 13, 17, - qr/ERROR: "regress_admin" cannot replicate into relation with row-level security enabled: "unpartitioned\w*"/msi, + qr/ERROR: ( [A-Z0-9]+:)? "regress_admin" cannot replicate into relation with row-level security enabled: "unpartitioned\w*"/msi, "non-superuser admin without bypassrls fails to replicate delete into rls enabled unpartitioned" ); grant_bypassrls("regress_admin"); diff --git a/src/test/subscription/t/029_on_error.pl b/src/test/subscription/t/029_on_error.pl index e8b904b7452..303e8ec3fc2 100644 --- a/src/test/subscription/t/029_on_error.pl +++ b/src/test/subscription/t/029_on_error.pl @@ -48,7 +48,7 @@ sub test_skip_lsn # Check the log to ensure that the transaction is skipped, and advance the # offset of the log file for the next test. $offset = $node_subscriber->wait_for_log( - qr/LOG: done skipping logical replication transaction finished at $lsn/, + qr/LOG: ( [A-Z0-9]+:)? done skipping logical replication transaction finished at $lsn/, $offset); # Insert non-conflict data |