pgsql: Fix bogus extra arguments to query_safe in test - Mailing list pgsql-committers

From Heikki Linnakangas
Subject pgsql: Fix bogus extra arguments to query_safe in test
Date
Msg-id E1vTOSg-0000w7-1W@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix bogus extra arguments to query_safe in test

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.

Branch
------
REL_17_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/807b2f261d643944120d29bfab70937ebbd4aad6

Modified Files
--------------
src/test/modules/xid_wraparound/t/004_notify_freeze.pl | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)


pgsql-committers by date:

Previous
From: Heikki Linnakangas
Date:
Subject: pgsql: Improve DDL deparsing test
Next
From: Jeff Davis
Date:
Subject: pgsql: Remove char_tolower() API.