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.15.z-fix subscription search box xpath #1405

Merged

Conversation

vijaysawant
Copy link
Contributor

@vijaysawant vijaysawant commented May 30, 2024

Problem
Content - Subscription page search box xpath unable to locate due to wrong xpath

Solution
This updated xpath will work as expected.

Note
Robottelo changes from the PR [Link] are not dependant these changes as I have updated some part of test case ( test_positive_access_with_non_admin_user_with_manifest) which won't search for any Subscription

@vijaysawant vijaysawant added Automation failure Makes Robottelo automation fail No-CherryPick PR doesnt need CherryPick to previous branches 6.15.z labels May 30, 2024
@vijaysawant vijaysawant force-pushed the 6.15.z-subscription-search-box-fix branch from c0000ae to 4af9e05 Compare May 31, 2024 13:55
@vijaysawant vijaysawant marked this pull request as ready for review May 31, 2024 14:36
Copy link
Contributor

@sambible sambible left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@omkarkhatavkar can you review?

Copy link
Collaborator

@omkarkhatavkar omkarkhatavkar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor change

airgun/entities/subscription.py Outdated Show resolved Hide resolved
@vijaysawant vijaysawant force-pushed the 6.15.z-subscription-search-box-fix branch from 4af9e05 to 7500fa0 Compare June 3, 2024 09:12
@vijaysawant vijaysawant added 6.14.z AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing and removed No-CherryPick PR doesnt need CherryPick to previous branches labels Jun 3, 2024
@vijaysawant vijaysawant force-pushed the 6.15.z-subscription-search-box-fix branch from 7500fa0 to e1394bd Compare June 3, 2024 09:49
@vijaysawant vijaysawant added the CherryPick PR needs CherryPick to previous branches label Jun 3, 2024
@vsedmik vsedmik merged commit 4e395a8 into SatelliteQE:6.15.z Jun 3, 2024
7 of 8 checks passed
github-actions bot pushed a commit that referenced this pull request Jun 3, 2024
* fix subscription search box xpath

* add sleep to test changes in CI

(cherry picked from commit 4e395a8)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.14.z 6.15.z Automation failure Makes Robottelo automation fail AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing CherryPick PR needs CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants