Skip to content

Conversation

@sheremet-va
Copy link
Member

@sheremet-va sheremet-va commented Aug 28, 2025

Description

Make the verbose output consistent. This PR:

  • Makes verbose output print only test case right after they are finished
  • Moves the tree view into a --reporter=tree
  • Prints tests as a tree when there is only one test file running (previously it was printing them as a list)
  • Removes inline error messages in default and tree view, they are kept only in verbose reporter

TODO:

  • Tests
  • Document tree reporter

@netlify
Copy link

netlify bot commented Aug 28, 2025

Deploy Preview for vitest-dev ready!

Name Link
🔨 Latest commit 0130da2
🔍 Latest deploy log https://app.netlify.com/projects/vitest-dev/deploys/68bfb0651b397400083f9743
😎 Deploy Preview https://deploy-preview-8500--vitest-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

hi-ogawa
hi-ogawa previously approved these changes Aug 28, 2025
Copy link
Member

@AriPerkkio AriPerkkio left a comment

Choose a reason for hiding this comment

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

Isn't this same as reverting #7503? I kind of liked the tree view 🤷

We could remove stuff like printTestSuite completely. Or revert all changes from base reporter to make it simpler.

@sheremet-va sheremet-va moved this to P2 - 5 in Team Board Aug 29, 2025
@sheremet-va sheremet-va changed the title fix!: always print test cases in verbose reporter even in TTY fix!: always print test cases in verbose reporter the same way Aug 29, 2025
@sheremet-va sheremet-va moved this from P2 - 5 to Approved in Team Board Aug 29, 2025
@sheremet-va sheremet-va marked this pull request as ready for review August 29, 2025 11:54
@sheremet-va sheremet-va changed the title fix!: always print test cases in verbose reporter the same way fix!: verbose reporter prints tests in a list, introduce tree reporter Aug 29, 2025
AriPerkkio
AriPerkkio previously approved these changes Sep 4, 2025
Co-authored-by: Ari Perkkiö <[email protected]>
AriPerkkio
AriPerkkio previously approved these changes Sep 7, 2025
@sheremet-va sheremet-va merged commit 25fd32b into vitest-dev:main Sep 9, 2025
12 of 14 checks passed
@sheremet-va sheremet-va deleted the fix/reporter-verbose-always branch September 9, 2025 04:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants