pgsql: Make PostgresNode::append_conf append a newline automatically.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Make PostgresNode::append_conf append a newline automatically.
Date: 2017-04-22 20:58:23
Message-ID: E1d226p-0006oC-Fg@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Make PostgresNode::append_conf append a newline automatically.

Although the documentation for append_conf said clearly that it didn't
add a newline, many test authors seem to have forgotten that ... or maybe
they just consulted the example at the top of the POD documentation,
which clearly shows adding a config entry without bothering to add a
trailing newline. The worst part of that is that it works, as long as
you don't do it more than once, since the backend isn't picky about
whether config files end with newlines. So there's not a strong forcing
function reminding test authors not to do it like that. Upshot is that
this is a terribly fragile way to go about things, and there's at least
one existing test case that is demonstrably broken and not testing what
it thinks it is.

Let's just make append_conf append a newline, instead; that is clearly
way safer than the old definition.

I also cleaned up a few call sites that were unnecessarily ugly.
(I left things alone in places where it's plausible that additional
config lines would need to be added someday.)

Back-patch the change in append_conf itself to 9.6 where it was added,
as having a definitional inconsistency between branches would obviously
be pretty hazardous for back-patching TAP tests. The other changes are
just cosmetic and don't need to be back-patched.

Discussion: https://postgr.es/m/19751.1492892376@sss.pgh.pa.us

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/8a19c1a3736c6b928ebc3503edaf0bd289366bcc

Modified Files
--------------
src/test/modules/commit_ts/t/004_restart.pl | 15 +++------------
src/test/perl/PostgresNode.pm | 4 ++--
src/test/recovery/t/001_stream_rep.pl | 12 ++++++------
src/test/recovery/t/002_archiving.pl | 6 ++----
src/test/recovery/t/003_recovery_targets.pl | 4 +---
5 files changed, 14 insertions(+), 27 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2017-04-22 22:18:37 pgsql: Make PostgresNode.pm check server status more carefully.
Previous Message Tom Lane 2017-04-22 18:40:28 Re: [COMMITTERS] pgsql: Replication lag tracking for walsenders