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

[6.12.z] recording ui-session-id for report portal logging #13188

Merged
merged 1 commit into from
Nov 29, 2023

Conversation

omkarkhatavkar
Copy link

Cherrypick of PR: #13055

Problem Statement

Currently, We are unable to retrieve the Selenium UI session ID from the Airgun GitHub repository and integrate it into our automation testing tool, Robottelo. This session ID is crucial because during the test recording process, videos are generated by the Selenium Airgun repository, and these videos are associated with specific UI session IDs. To improve our testing workflow and enhance the usability of our test results, we need to establish a mechanism that allows us to access this session ID within Robottelo and incorporate it into the JUnit XML results. By doing so, we can leverage the power of the Report Portal logging system, enabling other team members to easily locate and access the video links associated with each test result for comprehensive test result review.

Solution

  • The ui_session_record_property fixture is now function-scoped to ensure it is created once per test function.
  • The fixture uses record_property to load properties like video_link and session_id.
  • The ui_session_record_property fixture returns a Satellite instance with an associated AirgunSession instance, including the updated ui_session_id attribute.

Airgun PR

SatelliteQE/airgun#1038

* recording ui-session-id for report portal logging

* moving the fixture to common location and adding fspath check
@Gauravtalreja1 Gauravtalreja1 merged commit 3a668e5 into SatelliteQE:6.12.z Nov 29, 2023
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No-CherryPick PR doesnt need CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants