Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit a096813

Browse files
committed
Log regression.diffs in 027_stream_regress.pl.
To help diagnose the reasons for a regression test failure inside this TAP test, dump the contents of regression.diffs to the log. While the CI scripts show it automatically, the build farm client does not. Reviewed-by: Andrew Dunstan <andrew@dunslane.net> Discussion: https://postgr.es/m/CA%2BhUKGLsrWbiCcMxBLRBAP6Z%3DykFRHWzdmP9YKujSKoSnEJECQ%40mail.gmail.com
1 parent 7fa3db3 commit a096813

File tree

1 file changed

+21
-9
lines changed

1 file changed

+21
-9
lines changed

src/test/recovery/t/027_stream_regress.pl

Lines changed: 21 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -53,15 +53,27 @@
5353

5454
# Run the regression tests against the primary.
5555
my $extra_opts = $ENV{EXTRA_REGRESS_OPTS} || "";
56-
system_or_bail($ENV{PG_REGRESS} . " $extra_opts " .
57-
"--dlpath=\"$dlpath\" " .
58-
"--bindir= " .
59-
"--host=" . $node_primary->host . " " .
60-
"--port=" . $node_primary->port . " " .
61-
"--schedule=../regress/parallel_schedule " .
62-
"--max-concurrent-tests=20 " .
63-
"--inputdir=../regress " .
64-
"--outputdir=\"$outputdir\"");
56+
my $rc = system($ENV{PG_REGRESS} . " $extra_opts " .
57+
"--dlpath=\"$dlpath\" " .
58+
"--bindir= " .
59+
"--host=" . $node_primary->host . " " .
60+
"--port=" . $node_primary->port . " " .
61+
"--schedule=../regress/parallel_schedule " .
62+
"--max-concurrent-tests=20 " .
63+
"--inputdir=../regress " .
64+
"--outputdir=\"$outputdir\"");
65+
if ($rc != 0)
66+
{
67+
# Dump out the regression diffs file, if there is one
68+
my $diffs = "$outputdir/regression.diffs";
69+
if (-e $diffs)
70+
{
71+
print "=== dumping $diffs ===\n";
72+
print slurp_file($diffs);
73+
print "=== EOF ===\n";
74+
}
75+
}
76+
is($rc, 0, 'regression tests pass');
6577

6678
# Clobber all sequences with their next value, so that we don't have
6779
# differences between nodes due to caching.

0 commit comments

Comments
 (0)