better-psql-invocation.txt
text/plain
Filename: better-psql-invocation.txt
Type: text/plain
Part: 0
diff --git a/src/bin/pg_upgrade/t/002_pg_upgrade.pl b/src/bin/pg_upgrade/t/002_pg_upgrade.pl
index 09af8157d0..a8df6440a4 100644
--- a/src/bin/pg_upgrade/t/002_pg_upgrade.pl
+++ b/src/bin/pg_upgrade/t/002_pg_upgrade.pl
@@ -173,14 +173,10 @@ WHERE
n.nspname NOT IN ('pg_catalog', 'information_schema', 'binary_upgrade',
'pg_toast'))
OR (n.nspname = 'pg_catalog' AND relname IN ('pg_largeobject')))
+ORDER BY c.oid::regclass::text
EOM
$horizon_query =~ s/\s+/ /g; # run it together on one line
-$newnode->command_ok(
- [
- 'psql', '-At', '-d', $oldnode->connstr('postgres'),
- '-o', "$tempdir/horizon1.txt", '-c', $horizon_query,
- ],
- 'horizons before running pg_upgrade');
+my $horizon1 = $oldnode->safe_psql('regression', $horizon_query);
# After dumping, update references to the old source tree's regress.so
# to point to the new tree.
@@ -316,12 +312,7 @@ $newnode->command_ok(
'dump after running pg_upgrade');
# And second record of horizons as well.
-$newnode->command_ok(
- [
- 'psql', '-At', '-d', $newnode->connstr('postgres'),
- '-o', "$tempdir/horizon2.txt", '-c', $horizon_query,
- ],
- 'horizons after running pg_upgrade');
+my $horizon2 = $newnode->safe_psql('regression', $horizon_query);
# Compare the two dumps, there should be no differences.
my $compare_res = compare("$tempdir/dump1.sql", "$tempdir/dump2.sql");
@@ -341,14 +332,21 @@ if ($compare_res != 0)
}
# Compare the horizons, there should be no differences.
-$compare_res = compare("$tempdir/horizon1.txt", "$tempdir/horizon2.txt");
-is($compare_res, 0, 'old and new horizons match after pg_upgrade');
+my $horizons_ok = $horizon1 eq $horizon2;
+ok($horizons_ok, 'old and new horizons match after pg_upgrade');
# Provide more context if the horizons do not match.
-if ($compare_res != 0)
+if (! $horizons_ok)
{
+ # output is long, so use diff to compare
+ open my $fh, ">", "$tempdir/horizon1.txt" or die "could not open file: $!";
+ print $fh $horizon1;
+ close $fh;
+ open $fh, ">", "$tempdir/horizon2.txt" or die "could not open file: $!";
+ print $fh $horizon2;
my ($stdout, $stderr) =
- run_command([ 'diff', "$tempdir/horizon1.txt", "$tempdir/horizon2.txt" ]);
+ run_command([ 'diff', "$tempdir/horizon1.txt", "$tempdir/horizon2.txt" ]);
+ close $fh;
print "=== diff of $tempdir/horizon1.txt and $tempdir/horizon2.txt\n";
print "=== stdout ===\n";
print $stdout;