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

Reporting: Add current Link to each Step in Allure Report #261

Open
4 of 6 tasks
wurzelkuchen opened this issue Jul 1, 2024 · 2 comments · May be fixed by #276
Open
4 of 6 tasks

Reporting: Add current Link to each Step in Allure Report #261

wurzelkuchen opened this issue Jul 1, 2024 · 2 comments · May be fixed by #276
Assignees
@wurzelkuchen
Labels
codeReadyForMerge The issue was implemented and the branch is ready to be reviewed and merged into develop docsReadyForMerge Documentation was created for this issue in a separate branch which is ready to merge to develop improvement
Milestone
v5.1.0

Comments

@wurzelkuchen
Copy link
Contributor

wurzelkuchen commented Jul 1, 2024 •
edited by georgkunze

Quick Description

To quickly jump into reproducing a test case, it would be good to have the current link available at each step.

User Story

As a report reviewer,
I want to quickly click a link that brings me to the page the test saw when it broke
so that I don't have to work manually to redundant steps while reproducing errors.

Functional Requirements

  • Each Step in the Allure Report should provide a link to the current page
  • The link should be clickable
  • It should be possible to toggle this feature in our configuration
  • The URL should be added to the report if it changes (e.g. after a click, an open or any other site/URL changing action)

Documentation Requirements

  • Document this feature and the according property to disable it.

Test Requirements

  • Smoke Test the feature with all supported formats in an external test suite (e.g. neodymium example)
@wurzelkuchen wurzelkuchen added improvement docsNeeded The documenttion needs to be checked if there are changes needed and if so, adaptions should be made codeImplementationNeeded This issue requires a change in the code of Neodymium labels Jul 1, 2024
@georgkunze georgkunze 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 Jul 30, 2024
@georgkunze
Copy link

As discussed Functional Requirement was changed to not provide links for every step but for every action that changes the URL of the site.

@wurzelkuchen
Copy link
Contributor Author

Thanks, I adapted the original ticket to reflect the new requirements.

@wurzelkuchen wurzelkuchen added this to the v5.1.0 milestone Jul 30, 2024
@georgkunze georgkunze linked a pull request Jul 30, 2024 that will close this issue
#261 add link to step report #276
Open
@georgkunze georgkunze added docsReadyForMerge Documentation was created for this issue in a separate branch which is ready to merge to develop and removed docsNeeded The documenttion needs to be checked if there are changes needed and if so, adaptions should be made labels Jul 30, 2024
oomelianchuk added a commit that referenced this issue Aug 9, 2024
@oomelianchuk
use internal thread-safe map to store last url value #261
d12365e
@georgkunze georgkunze removed their assignment Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wurzelkuchen wurzelkuchen

Labels
codeReadyForMerge The issue was implemented and the branch is ready to be reviewed and merged into develop docsReadyForMerge Documentation was created for this issue in a separate branch which is ready to merge to develop improvement
Projects
None yet
Milestone
v5.1.0
Development

Successfully merging a pull request may close this issue.

#261 add link to step report Xceptance/neodymium-library
2 participants
@wurzelkuchen @georgkunze