From: Tatsuo Ishii Date: Fri, 27 Jun 2025 06:50:33 +0000 (+0900) Subject: Test: more fix to 038.pcp_commands regression test. X-Git-Url: http://git.postgresql.org/gitweb/?a=commitdiff_plain;h=b76c6876de4bca2da5e009f5cc64212060ccb1fb;p=pgpool2.git Test: more fix to 038.pcp_commands regression test. Commit 04e09df17 was not enough fix. The test calls pcp_proc_info() pgpool_adm function along with user name and password (in the test password is the same string as user name). Problem is, the user name is obtained from a user name that runs the test, and we use psql -a to submit the SQL, which prints the user name. Of course the user name can vary depending on the environment, and it makes the test fail. To fix the issue, run psql without -a option. --- diff --git a/src/test/regression/tests/038.pcp_commands/expected b/src/test/regression/tests/038.pcp_commands/expected index 861fabb87..61df0b958 100644 --- a/src/test/regression/tests/038.pcp_commands/expected +++ b/src/test/regression/tests/038.pcp_commands/expected @@ -1,8 +1,4 @@ -CREATE EXTENSION pgpool_adm; CREATE EXTENSION -SELECT database, status, case when client_host = '127.0.0.1' or client_host = '::1' then 'localhost' end, statement FROM pcp_proc_info -(host => '', port => 11001, username => 't-ishii', password => 't-ishii') -WHERE connected = '1' AND backend_id = '0' AND statement = 'SELECT pg_sleep(1)' database | status | case | statement ----------+-----------------+-----------+-------------------- test | Execute command | localhost | SELECT pg_sleep(1) diff --git a/src/test/regression/tests/038.pcp_commands/test.sh b/src/test/regression/tests/038.pcp_commands/test.sh index 21896e44b..89c4dbc92 100755 --- a/src/test/regression/tests/038.pcp_commands/test.sh +++ b/src/test/regression/tests/038.pcp_commands/test.sh @@ -58,7 +58,7 @@ WHOAMI=`whoami` ./startall wait_for_pgpool_startup $PSQL -a -h localhost -c "SELECT pg_sleep(1)" test & -$PSQL -a -h localhost test > result 2>&1 < result 2>&1 < '', port => $PCP_PORT, username => '$WHOAMI', password => '$WHOAMI')