Fix test name and username used in failed connection attempts
authorHeikki Linnakangas <[email protected]>
Sun, 9 Mar 2025 17:47:55 +0000 (19:47 +0200)
committerHeikki Linnakangas <[email protected]>
Sun, 9 Mar 2025 17:47:55 +0000 (19:47 +0200)
The first failed connection tests the "regular" connections limit, not
the reserved limit.

In the second failed connection, the username doesn't really matter,
but since the previous successful connections used "regress_reserved",
it seems weird to switch back to "regress_regular" for the
expected-to-fail attempt.

Discussion: https://www.postgresql.org/message-id/fd5e9523-78d3-4270-86b2-fd1b1eeb4fc9@iki.fi

src/test/postmaster/t/002_connection_limits.pl

index 2c185eef6ebda470d5b6538ac0bd5fef64eba170..fd670f3a8f5f9640381f574b514577a6e0ac65c4 100644 (file)
@@ -80,7 +80,7 @@ push(@sessions, background_psql_as_user('regress_regular'));
 connect_fails_wait(
    $node,
    "dbname=postgres user=regress_regular",
-   "reserved_connections limit",
+   "regular connections limit",
    expected_stderr =>
      qr/FATAL:  remaining connection slots are reserved for roles with privileges of the "pg_use_reserved_connections" role/
 );
@@ -89,7 +89,7 @@ push(@sessions, background_psql_as_user('regress_reserved'));
 push(@sessions, background_psql_as_user('regress_reserved'));
 connect_fails_wait(
    $node,
-   "dbname=postgres user=regress_regular",
+   "dbname=postgres user=regress_reserved",
    "reserved_connections limit",
    expected_stderr =>
      qr/FATAL:  remaining connection slots are reserved for roles with the SUPERUSER attribute/