pgsql: Fix some issues with TAP tests for postgres -C

Started by Michael Paquieralmost 5 years ago1 messagescomitters
Jump to latest
#1Michael Paquier
michael@paquier.xyz

Fix some issues with TAP tests for postgres -C

This addresses two issues with the tests added in 0c39c292 for runtime
GUCs:
- Re-enable the test on Msys. The test could fail because of \r\n
generated by Msys perl. 0d91c52a has taken care of this issue.
- Allow the test to run in the context of a privileged account. CIs
running under privileged accounts would fail on permission failures, as
reported by Andres Freund. This issue is fixed by wrapping the postgres
command within pg_ctl as the latter will take care of any permissions
needed. The test checking a failure of postgres -C for a runtime
parameter with an instance running is removed, as pg_ctl produces an
unstable error code (no need for a CI to reproduce that).

Discussion: /messages/by-id/20210921032040.lyl4lcax37aedx2x@alap3.anarazel.de

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/1a9d802828110c87a207785aaf6b00d8917a86ad

Modified Files
--------------
src/bin/pg_checksums/t/002_actions.pl | 37 +++++++++++++----------------------
1 file changed, 14 insertions(+), 23 deletions(-)