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 · 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 · Fixed by #276
Assignees
@wurzelkuchen
Labels
codeReadyForRecheck The issue is fixed/implemented AND merged into development, but needs to be rechecked there docsDone The documentation for this issue was rechecked and merged to the develop branch improvement
Milestone
v5.1.0

Comments

@wurzelkuchen
Copy link
Contributor

wurzelkuchen commented Jul 1, 2024 •
edited by georgkunze
Loading

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
Merged
@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
wurzelkuchen added a commit that referenced this issue Oct 22, 2024
@wurzelkuchen
Merge pull request #276 from Xceptance/#261_Add_Link_to_Step_Report
cc8837b 
#261 add link to step report
@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 Oct 23, 2024
@wurzelkuchen wurzelkuchen added docsDone The documentation for this issue was rechecked and merged to the develop branch and removed docsReadyForMerge Documentation was created for this issue in a separate branch which is ready to merge to develop labels Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wurzelkuchen wurzelkuchen

Labels
codeReadyForRecheck The issue is fixed/implemented AND merged into development, but needs to be rechecked there docsDone The documentation for this issue was rechecked and merged to the develop branch 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
2 participants
@wurzelkuchen @georgkunze