-
Notifications
You must be signed in to change notification settings - Fork 115
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
[stream] fixes for combined session failures, Errata UI #14822
Conversation
trigger: test-robottelo |
PRT Result
|
trigger: test-robottelo |
PRT Result
|
trigger: test-robottelo |
trigger: test-robottelo |
trigger: test-robottelo |
PRT Result
|
PRT Result
|
trigger: test-robottelo |
PRT Result
|
PRT Result
|
PRT Result
|
PRT Result
|
trigger: test-robottelo |
PRT Result
|
PRT Result
|
PRT Result
|
trigger: test-robottelo |
PRT Result
|
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.
One small fix and a question
9cb3ca3
to
20a7391
Compare
trigger: test-robottelo |
PRT Result
|
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.
ack pending griffin's comments.
* Refactor and fix of some Errata UI tests * UI fixture and other updates for combined sessions --------- Co-authored-by: Ladislav Vasina <[email protected]> (cherry picked from commit 9448d4b)
…14822) * Refactor and fix of some Errata UI tests * UI fixture and other updates for combined sessions --------- Co-authored-by: Ladislav Vasina <[email protected]>
Problem Statement
Initial PR #14773 raised by @LadislavVasina1, for 3 errata UI test fixes/updates.
Error in setup for
test_positive_check_errata_counts_by_type_on_host_details_page
:Locally have intermittently reproduced issue with the
registered_contenthost
UI fixture with adding repos to cv, and promotion of cv to lce, in between parameters of combined session. Issues with some associated activation keys not being deleted in cleanup.^^ Issues fixed with 2nd commit.
PRT Case
along with
test_positive_list
, which fails in CI and is fixed by the changes.We exclude the otherwise collected
test_positive_list_permissions
, it is not impacted by these changes.