Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature]: Make tags auto-appending configurable #33947

Open
vitalets opened this issue Dec 11, 2024 · 3 comments
Open

[Feature]: Make tags auto-appending configurable #33947

vitalets opened this issue Dec 11, 2024 · 3 comments

Comments

@vitalets
Copy link
Contributor

vitalets commented Dec 11, 2024

🚀 Feature Request

After #32930 tags are automatically appended to test titles in list and line reporters.

Could you still consider making this behavior configurable?
Imho one of the benefits of tags annotations is cleaner output. In case of many tags appended to the test title, it may look cluttered and unexpected to users.

Here is the report I've got for playwright-bdd vitalets/playwright-bdd#251

Example

I'd suggest to keep originally proposed appendTags option (with default to true):

reporter: [
  ['list', { appendTags: false }]
],

Motivation

  • cleaner reporter output
  • no unexpected modifications of user's input
@yury-s
Copy link
Member

yury-s commented Dec 12, 2024

How do you want these tags to be represented if not in the reporter? If you don't want them to be printed maybe don't add them as tags?

@vitalets
Copy link
Contributor Author

How do you want these tags to be represented if not in the reporter? If you don't want them to be printed maybe don't add them as tags?

I think HTML reporter is a great place to show all tags. Visual chips are nice and clickable.
Showing all tags in terminal appended to the title is not very useful for my workflow.

@dimkin-eu
Copy link

+1 to issue. exposing tags in console output in CI clutters test output ( we have 10+ tags for some tests )

tags weren't exposed in version 1.46 for sure and appeared in 1.49 at least

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

No branches or pull requests

3 participants