diff options
| author | Michael Paquier <michael@paquier.xyz> | 2024-01-03 10:28:05 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2024-01-03 10:28:05 +0900 |
| commit | 4b66d7b4e0b3bb55c04fe9466b428c24d7cb0980 (patch) | |
| tree | 300a559560fc824be472504ad97fd4ab6339a3e9 /src/bin/scripts | |
| parent | ae6bc39317f59dff5eb549e77d3f7e330e9ba075 (diff) | |
| download | postgresql-4b66d7b4e0b3bb55c04fe9466b428c24d7cb0980.tar.gz postgresql-4b66d7b4e0b3bb55c04fe9466b428c24d7cb0980.zip | |
Remove unnecessary PGDATABASE settings from TAP tests
Some of the TAP tests have been historically setting the environment
variable PGDATABASE to 'postgres', which is not needed because
PostgreSQL::Test::Cluster already sets it when initialized. This commit
removes these explicit setups.
Note that the dependency of cluster -a with PGDATABASE (from
1caef31d9e55) is still documented.
Author: Bharath Rupireddy
Discussion: https://postgr.es/m/CALj2ACXLAz5dW3ZP+Fec8g6jQMMmDyCVT+qdbye2h7QJJmhsdw@mail.gmail.com
Diffstat (limited to 'src/bin/scripts')
| -rw-r--r-- | src/bin/scripts/t/011_clusterdb_all.pl | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/bin/scripts/t/011_clusterdb_all.pl b/src/bin/scripts/t/011_clusterdb_all.pl index 26315d0a510..f1cb5e3f155 100644 --- a/src/bin/scripts/t/011_clusterdb_all.pl +++ b/src/bin/scripts/t/011_clusterdb_all.pl @@ -12,10 +12,8 @@ my $node = PostgreSQL::Test::Cluster->new('main'); $node->init; $node->start; -# clusterdb -a is not compatible with -d, hence enforce environment variable -# correctly. -$ENV{PGDATABASE} = 'postgres'; - +# clusterdb -a is not compatible with -d. This relies on PGDATABASE to be +# set, something PostgreSQL::Test::Cluster does. $node->issues_sql_like( [ 'clusterdb', '-a' ], qr/statement: CLUSTER.*statement: CLUSTER/s, |