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.

Sign up for GitHub

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

Jump to bottom

Improvement: Reduce Allure INFO entries #264

Closed
3 tasks done
wurzelkuchen opened this issue Jul 9, 2024 · 0 comments · Fixed by #313
Closed
3 tasks done

Improvement: Reduce Allure INFO entries #264

wurzelkuchen opened this issue Jul 9, 2024 · 0 comments · Fixed by #313
Assignees
@oomelianchuk
Labels
codeDone The issue is rechecked for development branch improvement
Milestone
v5.1.0

Comments

@wurzelkuchen
Copy link
Contributor

wurzelkuchen commented Jul 9, 2024 •
edited by oomelianchuk
Loading

Quick Description

Currently the time to render the videos is always logged to the allure report, giving information not necessarily needed in daily test automation business.

User Story

As a Report reviewer,
I want to see the additional information like the time video took to render only if I specifically asked for it
so that important information are not overshadowed by the big amount of data presented in the report.

Functional Requirements

  • Make the appearance of the INFO optional configured by properties (gif/video.properties)
  • there is a typo in the word "avarage", which should be fixed with this issue as well.

Test Requirements

  • Smoke test this with external test suites like the neodymium example.
@wurzelkuchen wurzelkuchen added improvement codeImplementationNeeded This issue requires a change in the code of Neodymium labels Jul 9, 2024
@wurzelkuchen wurzelkuchen added this to the v5.1.0 milestone Jul 9, 2024
oomelianchuk added a commit that referenced this issue Nov 26, 2024
@oomelianchuk
[#264] Improvement: Reduce Allure INFO entries
1966841
@oomelianchuk oomelianchuk added codeReadyForMerge The issue was implemented and the branch is ready to be reviewed and merged into develop and removed codeImplementationNeeded This issue requires a change in the code of Neodymium labels Nov 26, 2024
@oomelianchuk oomelianchuk linked a pull request Nov 26, 2024 that will close this issue
[#264] Improvement: Reduce Allure INFO entries #313
Merged
wurzelkuchen added a commit that referenced this issue Nov 26, 2024
@wurzelkuchen
Merge pull request #313 from Xceptance/#264-improvement-reduce-allure…
7ebc0e1 
…-info-entries

[#264] Improvement: Reduce Allure INFO entries
@wurzelkuchen wurzelkuchen added codeReadyForRecheck The issue is fixed/implemented AND merged into development, but needs to be rechecked there and removed codeReadyForMerge The issue was implemented and the branch is ready to be reviewed and merged into develop labels Nov 29, 2024
@wurzelkuchen wurzelkuchen added codeDone The issue is rechecked for development branch and removed codeReadyForRecheck The issue is fixed/implemented AND merged into development, but needs to be rechecked there labels Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@oomelianchuk oomelianchuk

Labels
codeDone The issue is rechecked for development branch improvement
Projects
None yet
Milestone
v5.1.0
Development

Successfully merging a pull request may close this issue.

[#264] Improvement: Reduce Allure INFO entries Xceptance/neodymium
2 participants
@wurzelkuchen @oomelianchuk