Skip to content

Commit e5d7cf0

Browse files
committed
Include output of failing TAP tests into regression diff section
1 parent 712cf0a commit e5d7cf0

3 files changed

Lines changed: 14 additions & 0 deletions

File tree

.github/workflows/linux-32bit-build-and-test.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -230,6 +230,11 @@ jobs:
230230
run: |
231231
find . -name regression.diffs -exec cat {} + > regression.log
232232
233+
# TAP tests do not produce regression.diffs; prove summarises the
234+
# failing tests and subtests in installcheck.log instead. Surface that
235+
# summary here (full per-test logs are in the TAP test logs artifact).
236+
sed -n '/^Test Summary Report/,/^Result:/p' installcheck.log >> regression.log 2>/dev/null || true
237+
233238
if [[ -s regression.log ]]; then echo "regression_diff=true" >>$GITHUB_OUTPUT; fi
234239
grep -e 'FAILED' -e 'failed (ignored)' -e 'not ok' installcheck.log || true
235240
cat regression.log

.github/workflows/linux-build-and-test.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -267,6 +267,11 @@ jobs:
267267
run: |
268268
find . -name regression.diffs -exec cat {} + > regression.log
269269
270+
# TAP tests do not produce regression.diffs; prove summarises the
271+
# failing tests and subtests in installcheck.log instead. Surface that
272+
# summary here (full per-test logs are in the TAP test logs artifact).
273+
sed -n '/^Test Summary Report/,/^Result:/p' installcheck.log >> regression.log 2>/dev/null || true
274+
270275
if [[ "${{ runner.os }}" == "Linux" ]] ; then
271276
# wait in case there are in-progress coredumps
272277
sleep 10

.github/workflows/sanitizer-build-and-test.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -174,6 +174,10 @@ jobs:
174174
id: collectlogs
175175
run: |
176176
find . -name regression.diffs -exec cat {} + > regression.log
177+
# TAP tests do not produce regression.diffs; prove summarises the
178+
# failing tests and subtests in installcheck.log instead. Surface that
179+
# summary here (full per-test logs are in the TAP test logs artifact).
180+
sed -n '/^Test Summary Report/,/^Result:/p' installcheck.log >> regression.log 2>/dev/null || true
177181
if [[ "${{ runner.os }}" == "Linux" ]] ; then
178182
# wait in case there are in-progress coredumps
179183
sleep 10

0 commit comments

Comments
 (0)