Skip to content

Conversation

@ChrisDryden
Copy link
Collaborator

@ChrisDryden ChrisDryden commented Dec 24, 2025

This changes the patch to skip the test with a patch to override the diff functionality in the test to only output which test failed so now the output looks like this:

  FAIL: tests/pr/pr-tests
  =======================

  pr: test 1b: mismatch
  pr: test 1b.r: mismatch
  pr: test 1b.p: mismatch
  pr: test 1c: mismatch
  ...
  pr: test 1i failed: exit status mismatch:  expected 0, got 1
  pr: test 1j failed: exit status mismatch:  expected 0, got 1
  ...
  pr: test margin-0 passed.
  ...

This should allow us to work on fixing these issues in the pr tests in the CI while still having the same patch to not overwhelm the logs

@ChrisDryden ChrisDryden changed the title Enable pr-tests.pl with suppressed diff output pr: Enable pr-tests.pl with suppressed diff output Dec 24, 2025
@ChrisDryden ChrisDryden marked this pull request as ready for review December 24, 2025 20:03
@github-actions
Copy link

GNU testsuite comparison:

Note: The gnu test tests/printf/printf-surprise is now being skipped but was previously passing.
Note: The gnu test tests/pr/pr-tests was skipped on 'main' but is now failing.

@ChrisDryden
Copy link
Collaborator Author

Not sure what's going on with printf surprise

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant