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.13.z] - Removing the direct Airgun imports from robottelo #13740

Merged

Conversation

omkarkhatavkar
Copy link

Problem Statement

  • Remove the Airgun Imports from the Robottelo and Use target_sat.ui_session or module_target_sat.ui

Solution

  • UI tests are updated to use the modified "target_sat" or "module_target_sat" for efficient management of multiple browser sessions.
  • The streamlined UI tests function correctly and manage browser sessions effectively.
  • Remove the nailgun entities from the UI code considering the Airgun Imports (some more PR needs to be added remove this as not wanted to do everything in single PR)

@omkarkhatavkar omkarkhatavkar changed the title Removing the direct Airgun imports from robottelo [6.13.z] - Removing the direct Airgun imports from robottelo Jan 11, 2024
@omkarkhatavkar omkarkhatavkar force-pushed the 6.13.z-remove-airgun-imports branch from 9717670 to df379b2 Compare January 12, 2024 11:35
@omkarkhatavkar omkarkhatavkar added the No-CherryPick PR doesnt need CherryPick to previous branches label Jan 12, 2024
@omkarkhatavkar
Copy link
Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ldap_authentication.py -k test_positive_delete_external_roles
pod_resources_size: extra_large

@omkarkhatavkar
Copy link
Author

copying the PRT results here for future reference: Robottelo-Runner — Build 5355 has Passed! 2 tests run, 0 skipped, 0 failed.

@omkarkhatavkar omkarkhatavkar force-pushed the 6.13.z-remove-airgun-imports branch from df379b2 to 6c806f1 Compare January 12, 2024 12:14
@omkarkhatavkar
Copy link
Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ldap_authentication.py -k test_positive_delete_external_roles
pod_resources_size: extra_large

@Gauravtalreja1 Gauravtalreja1 merged commit 517b415 into SatelliteQE:6.13.z Jan 12, 2024
6 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