diff options
| author | Andres Freund <andres@anarazel.de> | 2023-09-25 11:50:02 -0700 |
|---|---|---|
| committer | Andres Freund <andres@anarazel.de> | 2023-09-25 12:10:39 -0700 |
| commit | 9dc3c5472ed99a1046a4d7884270a35cfad29d1d (patch) | |
| tree | d037c91d7486012ad70a605b0a4423ed14c54bc1 /src | |
| parent | 0e59266a5daf31ff5c9976dae1b328b6a0e52b3e (diff) | |
| download | postgresql-9dc3c5472ed99a1046a4d7884270a35cfad29d1d.tar.gz postgresql-9dc3c5472ed99a1046a4d7884270a35cfad29d1d.zip | |
pg_dump: tests: Correct test condition for invalid databases
For some reason I used not_like = { pg_dumpall_dbprivs => 1, } in the test
condition of one of the tests added in in c66a7d75e65. That doesn't make sense
for two reasons: 1) not_like isn't a valid test condition 2) the database
should not be dumped in any of the tests. Due to 1), the test achieved its
goal, but clearly the formulation is confusing. Instead use like => {}, with
a comment explaining why.
Reported-by: Peter Eisentraut <peter@eisentraut.org>
Discussion: https://postgr.es/m/3ddf79f2-8b7b-a093-11d2-5c739bc64f86@eisentraut.org
Backpatch: 11-, like c66a7d75e65
Diffstat (limited to 'src')
| -rw-r--r-- | src/bin/pg_dump/t/002_pg_dump.pl | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/bin/pg_dump/t/002_pg_dump.pl b/src/bin/pg_dump/t/002_pg_dump.pl index 0d32882dfce..5c2d4505ae2 100644 --- a/src/bin/pg_dump/t/002_pg_dump.pl +++ b/src/bin/pg_dump/t/002_pg_dump.pl @@ -1581,9 +1581,9 @@ my %tests = ( CREATE DATABASE regression_invalid; UPDATE pg_database SET datconnlimit = -2 WHERE datname = 'regression_invalid'), regexp => qr/^CREATE DATABASE regression_invalid/m, - not_like => { - pg_dumpall_dbprivs => 1, - }, + + # invalid databases should never be dumped + like => {}, }, 'CREATE ACCESS METHOD gist2' => { |