pgsql: Revise test case added in 43746996399541ecb5c7b188725a5f097c15ce - Mailing list pgsql-committers

From Robert Haas
Subject pgsql: Revise test case added in 43746996399541ecb5c7b188725a5f097c15ce
Date
Msg-id E1oHdBI-001s9d-KH@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Revise test case added in 43746996399541ecb5c7b188725a5f097c15ceae.

Instead of using command_ok() to run psql, use safe_psql(). wrasse
isn't happy, and it be because of failure to pass -X to the psql
invocation, which safe_psql() will do automatically.

Since safe_psql() returns standard output instead of writing it to
a file, this requires some changes to the incantation for running
'diff'.

Test against the 'regression' database rather than 'postgres' so
we test more than just one table. That also means we need to record
the horizons later, after the test does "VACUUM FULL pg_largeobject".

Add an ORDER BY clause to the horizon query for stability.

Patch by me, reviewed by Tom Lane.

Discussion: http://postgr.es/m/CA+TgmoaGBbpzgu3=du1f9zDUbkfycO0y=_uWrLFy=KKEqXWeLQ@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/212bdc0cbc32ae5041bf1cd609f7b7a0c8930fc3

Modified Files
--------------
src/bin/pg_upgrade/t/002_pg_upgrade.pl | 62 ++++++++++++++++------------------
1 file changed, 30 insertions(+), 32 deletions(-)


pgsql-committers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: pgsql: Fix new recovery test for log_error_verbosity=verbose case
Next
From: Tom Lane
Date:
Subject: pgsql: Adjust new pg_read_file() test cases for more portability.