Re: Making background psql nicer to use in tap tests

Daniel Gustafsson <daniel@yesql.se>

From: Daniel Gustafsson <daniel@yesql.se>

To: Andrew Dunstan <andrew@dunslane.net>, Andres Freund <andres@anarazel.de>

Cc: Tom Lane <tgl@sss.pgh.pa.us>, PostgreSQL Hackers <pgsql-hackers@postgresql.org>, Melanie Plageman <melanieplageman@gmail.com>

Date: 2023-04-02T20:24:16Z

Lists: pgsql-hackers

Attachments

v5-0001-Refactor-background-psql-TAP-functions.patch (application/octet-stream) patch v5-0001
v5-0002-Add-test-SCRAM-iteration-changes-with-psql-passwo.patch (application/octet-stream) patch v5-0002

> On 31 Mar 2023, at 22:33, Daniel Gustafsson <daniel@yesql.se> wrote:
> 
> The attached v4 fixes some incorrect documentation (added by me in v3), and
> fixes that background_psql() didn't honor on_error_stop and extraparams passed
> by the user.  I've also added a commit which implements the \password test from
> the SCRAM iteration count patchset as well as cleaned up a few IPC::Run
> includes from test scripts.

And a v5 to fix a test failure in recovery tests.

--
Daniel Gustafsson

Commits

Fix missing installation/uninstallation rules for BackgroundPsql.pm
- 241ef53b6a95 12.20 landed
- 0d80e59a1b2c 13.16 landed
- 1185be355462 14.13 landed
- 9d7506a2f7cb 15.8 landed
Backport BackgroundPsql perl test module
- 4b467a6581af 12.20 landed
- 12f327b210b3 13.16 landed
- 31877cd8ec8c 14.13 landed
- d5fd7865f068 15.8 landed
- 187b8991f70f 16.4 landed
Add missing uninstallation rule for BackgroundPsql.pm
- 7039c7cff673 16.0 cited
Fix missing installation rules for BackgroundPsql.pm
- a4c17c86176c 16.0 cited
Skip \password TAP test on old IPC::Run versions
- 2e57ffe12f6b 16.0 landed
Test SCRAM iteration changes with psql \password
- bf5a894c5571 16.0 landed
Refactor background psql TAP functions
- 664d757531e1 16.0 landed