Skip to content

Commit c6271b6

Browse files
authored
Merge pull request #9829 from ChrisDryden/pr-tests-enable
pr: Enable pr-tests.pl with suppressed diff output
2 parents 398b9c1 + 3a4de80 commit c6271b6

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

util/build-gnu.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -223,9 +223,9 @@ sed -i -e "s|---dis ||g" tests/tail/overlay-headers.sh
223223
# Do not FAIL, just do a regular ERROR
224224
"${SED}" -i -e "s|framework_failure_ 'no inotify_add_watch';|fail=1;|" tests/tail/inotify-rotate-resources.sh
225225

226-
# pr produces very long log and this command isn't super interesting
227-
# SKIP for now
228-
"${SED}" -i -e "s|my \$prog = 'pr';$|my \$prog = 'pr';CuSkip::skip \"\$prog: SKIP for producing too long logs\";|" tests/pr/pr-tests.pl
226+
# pr-tests.pl: Override the comparison function to suppress diff output
227+
# This prevents the test from overwhelming logs while still reporting failures
228+
"${SED}" -i '/^my $fail = run_tests/i no warnings "redefine"; *Coreutils::_compare_files = sub { my ($p, $t, $io, $a, $e) = @_; my $d = File::Compare::compare($a, $e); warn "$p: test $t: mismatch\\n" if $d; return $d; };' tests/pr/pr-tests.pl
229229

230230
# We don't have the same error message and no need to be that specific
231231
"${SED}" -i -e "s|invalid suffix in --pages argument|invalid --pages argument|" \

0 commit comments

Comments
 (0)