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

removed (.) which fails to navigate to host #1461

Merged
merged 1 commit into from
Jul 15, 2024

Conversation

amolpati30
Copy link
Contributor

@amolpati30 amolpati30 commented Jul 8, 2024

A dot (.) was added before the locator, causing the host not to navigate on the UI page and resulting in a JavaScript error. I removed this dot (.), and now the navigation to the given host works correctly.

Dependent PR - SatelliteQE/robottelo#15489

@amolpati30 amolpati30 added CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing 6.14.z 6.15.z labels Jul 8, 2024
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py -k test_positive_non_admin_user_access_with_usergroup
airgun: 15489

@omkarkhatavkar
Copy link

@amolpati30 Airgun PRT is failing anything related to this PR ?

@amolpati30
Copy link
Contributor Author

"trigger": "test-robottelo"
"pytest": "-k 'test_positive_non_admin_user_access_with_usergroup' tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt"
"airgun": "15489"

@Satellite-QE
Copy link
Contributor

PRT Result

Build Number: 317
Build Status: SUCCESS
PRT Comment: pytest /opt/app-root/src/robottelo/tests/foreman -v -m build_sanity --external-logging
Test Result : ======= 12 passed, 5517 deselected, 5626 warnings in 1485.29s (0:24:45) ========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jul 15, 2024
@amolpati30
Copy link
Contributor Author

@omkarkhatavkar Airgun PRT passed now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing CherryPick PR needs CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants