-
Notifications
You must be signed in to change notification settings - Fork 82
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
Fix: new home page issues #1428
base: master
Are you sure you want to change the base?
Fix: new home page issues #1428
Conversation
Robot Results
|
ods_ci/tests/Resources/Page/ODH/ODHDashboard/ODHDashboard.robot
Outdated
Show resolved
Hide resolved
# Detect if we are ein the new (2.10) landing page instead in "Enable" | ||
${first_header}= Set Variable (//h1)[1] | ||
Wait Until Element is Visible xpath:${first_header} | ||
${h1_text} Get Text xpath:${first_header} |
Check warning
Code scanning / Robocop
The assignment sign is not consistent within the file. Expected '{{ expected_sign }}' but got '{{ actual_sign }}' instead Warning test
…ci into fix/new_home_page_issues
[Documentation] Compare the h1 headers of the landing page with the expected ones | ||
Wait Until Element Is Visible xpath:(//h1)[1] | ||
${h1_elements}= Get WebElements xpath://h1 | ||
${h1_texts}= Create List |
Check notice
Code scanning / Robocop
{{ create_keyword }} can be replaced with VAR Note test
Wait Until RHODS Dashboard ${dashboard_app} Is Visible | ||
# Ideally the timeout would be an arg but Robot does not allow "normal" and "embedded" arguments | ||
# Setting timeout to 30seconds since anything beyond that should be flagged as a UI bug | ||
Wait Until Element is Visible xpath://div[contains(@class,'gallery')]/div//div[@class="pf-v5-c-card__title"]//*[text()="${dashboard_app}"] | ||
Wait Until Element Is Visible xpath://div[contains(@class,'gallery')]/div//div[@class="pf-v5-c-card__title"]//*[text()="${dashboard_app}"] |
Check warning
Code scanning / Robocop
Line is too long ({{ line_length }}/{{ allowed_length }}) Warning test
ods_ci/tests/Resources/Page/ODH/ODHDashboard/ODHDashboard.robot
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, just please run with latest 2.10 (landing page should have been removed)
Co-authored-by: Noam Manos <nmanos@redhat.com>
Quality Gate passedIssues Measures |
In the last nightly (autotrigger-smoke/149) we had 37 failed test cases
Afte running the smoke from this branch (/rhods-smoke/5374), we have just 2 failed test cases. None related with the new landing page.
This PR is just a fix in order to enable again the nightlies. Maybe we should rethink the way we evaluate where is the user navigating, and if page by default should be "Enabled"