pgsql: Revise test case added in 43746996399541ecb5c7b188725a5f097c15ce

From: Robert Haas <rhaas(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Revise test case added in 43746996399541ecb5c7b188725a5f097c15ce
Date: 2022-07-30 03:30:25
Message-ID: E1oHdBI-001s9d-KH@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: 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(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2022-07-30 15:17:13 pgsql: Adjust new pg_read_file() test cases for more portability.
Previous Message Andrew Dunstan 2022-07-29 22:19:38 Re: pgsql: Fix new recovery test for log_error_verbosity=verbose case