Skip to content

Change tests output format to testname#61

Merged
carole-lavillonniere merged 1 commit intomainfrom
fix-hanging-tests
Mar 3, 2026
Merged

Change tests output format to testname#61
carole-lavillonniere merged 1 commit intomainfrom
fix-hanging-tests

Conversation

@carole-lavillonniere
Copy link
Collaborator

testdox format waits for each package to finish before flushing its output, which means the integration tests is not printing any progress during the ~100sec of its duration and often looks like it's hanging.

Switch to --format testname, which prints each test result line as soon as the event arrives.

@coderabbitai
Copy link

coderabbitai bot commented Mar 3, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Repository UI (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a84a8c7 and dba6eea.

📒 Files selected for processing (1)
  • Makefile

📝 Walkthrough

Walkthrough

Updates test output format from testdox to testname in the Makefile test targets. Changes affect three command configurations: the general test target and two test-integration platform-specific paths (Darwin and non-Darwin). No other modifications made.

Changes

Cohort / File(s) Summary
Test Output Format Configuration
Makefile
Updated --format flag from testdox to testname in three test target definitions: main test target and dual platform variants of test-integration target.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change—switching test output format from testdox to testname.
Description check ✅ Passed The description is directly related to the changeset, explaining the motivation for switching output formats and the benefits of using testname.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-hanging-tests

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Member

@silv-io silv-io left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Arguably this one is more useful either way in the logs :)

@carole-lavillonniere carole-lavillonniere merged commit 3fcc399 into main Mar 3, 2026
8 checks passed
@carole-lavillonniere carole-lavillonniere deleted the fix-hanging-tests branch March 3, 2026 09:34
@coderabbitai coderabbitai bot mentioned this pull request Mar 9, 2026
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.

2 participants