-
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] the tests for custom image import #1920
[Fix] the tests for custom image import #1920
Conversation
Robot Results
|
14ba5a2
to
758d176
Compare
This fixes multiple tests in the custom image import and fixes some common keywords from the ODHDashboard.robot file.
758d176
to
ef24e66
Compare
|
# Wait for the image to disappear from the list | ||
Wait Until Page Does Not Contain Element xpath:${custom_image_kebab_btn} timeout=10s | ||
# Assure that the actual ImageStream is also removed | ||
${rc} ${out}= Run And Return Rc And Output |
Check notice
Code scanning / Robocop
Variable '{{ name }}' is assigned but not used Note test
Wait Until Page Contains Unable to add notebook image: HTTP request failed | ||
Log Unable to add second image with error message "Unable to add notebook image: HTTP request failed" due to RHOAIENG-1192 level=WARN # robocop: disable | ||
# Assure that the expected error message is shown in the modal window | ||
${image_name_id}= Replace String ${IMG_NAME} ${SPACE} - |
Check warning
Code scanning / Robocop
The assignment sign is not consistent within the file. Expected '{{ expected_sign }}' but got '{{ actual_sign }}' instead Warning test
Log Unable to add second image with error message "Unable to add notebook image: HTTP request failed" due to RHOAIENG-1192 level=WARN # robocop: disable | ||
# Assure that the expected error message is shown in the modal window | ||
${image_name_id}= Replace String ${IMG_NAME} ${SPACE} - | ||
Wait Until Page Contains Unable to add notebook image: imagestreams.image.openshift.io "custom-${image_name_id}" already exists |
Check warning
Code scanning / Robocop
Line is too long ({{ line_length }}/{{ allowed_length }}) Warning test
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.
I think you could set some of the xpath variables as globals and reuse them across keywords, but this works too
[Tags] Tier1 | ||
... ODS-2470 | ||
... ExcludeOnDisconnected # Since we don't have internal image registry enabled there usually | ||
... InternalImageRegistry # Requires internal image registry enabled | ||
... ProductBug # https://issues.redhat.com/browse/RHOAIENG-1193 |
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.
If the test has ProductBug
, it will be run only in the known-issues quality gate. Beware RHOAIENG-1193 is closed as "Won't Do"
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.
Yeah, TBH, I was wondering whether we shouldn't remove the test at all... will consider it and maybe raise a separate issue.
[Tags] Tier1 | ||
... ODS-2470 | ||
... ExcludeOnDisconnected # Since we don't have internal image registry enabled there usually |
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.
Hi @jstourac this is the matter of an option in our automation where the internal registry is disabled default. Moreover, the internal image registry can be enabled/disabled outside of disconnected automation (using oc
command).
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.
I know, Andrej. But the test isn't passing now anyway. Also, since the linked issue won't be fixed probably, the test will probably go away at all.
You are right, Luca - I was considering it for the |
https://issues.redhat.com/browse/RHOAIENG-1192 has been resolved, so we can update that test accordingly.
Plust this fixes another one test in the custom image import and fixes some common keywords from the ODHDashboard.robot file.
CI: