database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-12-13 14:22:43 +03:00
Code Activity

Fix bogus extra arguments to query_safe in test

Browse Source 
The test seemed to incorrectly think that query_safe() takes an
argument that describes what the query does, similar to e.g.
command_ok(). Until commit bd8d9c9bdf the extra arguments were
harmless and were just ignored, but when commit bd8d9c9bdf introduced
a new optional argument to query_safe(), the extra arguments started
clashing with that, causing the test to fail.

Backpatch to v17, that's the oldest branch where the test exists. The
extra arguments didn't cause any trouble on the older branches, but
they were clearly bogus anyway.
This commit is contained in:
Heikki Linnakangas
2025-12-10 19:38:07 +02:00
parent 343693c3c1
commit 820343bab3
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/test/modules/xid_wraparound/t/004_notify_freeze.pl
View File

@@ -25,8 +25,8 @@ $node->safe_psql('postgres',
# Start Session 1 and leave it idle in transaction
my $psql_session1 = $node->background_psql('postgres');
$psql_session1->query_safe('listen s;', "Session 1 listens to 's'");
$psql_session1->query_safe('begin;', "Session 1 starts a transaction");
$psql_session1->query_safe('listen s;');
$psql_session1->query_safe('begin;');
# Send some notifys from other sessions
for my $i (1 .. 10)
@@ -57,7 +57,7 @@ ok($datafronzenxid_freeze > $datafronzenxid, 'datfrozenxid advanced');
# On Session 1, commit and ensure that the all the notifications are
# received. This depends on correctly freezing the XIDs in the pending
# notification entries.
my $res = $psql_session1->query_safe('commit;', "commit listen s;");
my $res = $psql_session1->query_safe('commit;');
my $notifications_count = 0;
foreach my $i (split('\n', $res))
{