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] hosts table: Name locator fix #1454

Open
wants to merge 1 commit into
base: 6.15.z
Choose a base branch
from

Conversation

Satellite-QE
Copy link
Contributor

Cherrypick of PR: #1453

Problem Statement

The HostsView Name table column XPath was missing leading '.' character,
which caused that all host rows that were returned, had the same Name value,
equal to the first table row host Name.

The `HostsView` Name table column XPath was missing leading '.' character,
which caused that all host rows returned had the same Name value,
equal to the first table row host Name.

(cherry picked from commit 886a1a8)
@Satellite-QE Satellite-QE added 6.15.z Auto_Cherry_Picked GHA has automatically cherrypicked this PR No-CherryPick PR doesnt need CherryPick to previous branches labels Jul 3, 2024
@Satellite-QE Satellite-QE added the AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing label Jul 3, 2024
@@ -226,7 +226,7 @@ class HostsView(BaseLoggedInView, SearchableViewMixinPF4):
column_widgets={
0: Checkbox(locator=".//input[@class='host_select_boxes']"),
'Name': Text(
"//a[contains(@href, '/new/hosts/') and not(contains(@href, 'Insights'))]"
".//a[contains(@href, '/new/hosts/') and not(contains(@href, 'Insights'))]"
Copy link
Collaborator

Choose a reason for hiding this comment

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

@pnovotny if I remember correctly, this PR can be closed in favor of #1468 ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

@pnovotny Can you confirm?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.15.z Auto_Cherry_Picked GHA has automatically cherrypicked this PR AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing No-CherryPick PR doesnt need CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants