Repository navigation
Show only failing tests in CI output - #169
Merged
Merged
Conversation
Run the suite with --compact so passing tests print as dots and only failures get the full details. Widen the runner's terminal to 200 columns too, because at 80 the names of failed tests were cut off. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Tests job printed a line for every passing test, about 1,900 of them, so you had to scroll a long way to find a failure. It now runs
php artisan test --compact. Passing tests show as dots, which fits on a few lines, and only failures get the full message and trace.The runner's terminal is 80 columns, which cut the names of failed tests off at something like
ConversationTest >…. SettingCOLUMNS: 200on the step gives them room. I checked both changes locally with a throwaway failing test and the output piped, the same as on the runner.Main is currently red for an unrelated reason. Those 8 failures need the newer mobile-ui and mobile-air, and that update isn't committed yet. Expect this PR's run to show the same 8, just in the shorter format.
Screenshots
🤖 Generated with Claude Code