-
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
Fix long failing reporting API tests #17189
base: master
Are you sure you want to change the base?
Fix long failing reporting API tests #17189
Conversation
|
All tests failed with |
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.
Codewisely looks good to me.
We may need to wait until a fix PR for the --force
issue is up and kill two flies with one stone (merge two PRs with one PRT).
Katello/hammer-cli-katello#979 sorry about that ;) |
|
This fixture sets up an activation key with LCE that has CV promoted to it. | ||
|
||
:return: activation_key, cv, function_lce, function_org |
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.
This fixture sets up an activation key with LCE that has CV promoted to it. | |
:return: activation_key, cv, function_lce, function_org | |
This function scope fixture creates CV and publish / promote it to LCE. It aslo creates AK which is associate with LCE & CV. | |
:return: AK, CV, LCE and Org |
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 am suggesting some non blocking changes related to doc string.
Problem Statement
Some API reporting tests did not work in the stream due to multi-CV changes.
The main goal is to get rid of
Environment ID and content view ID must be provided together
errors which appear due to outdated test composition.Solution
This PR fixes these issues by creating a fixture that promotes CV that is used with LCE which is needed now.
Related Issues
It is expected for the PRT to fail with an error related to the usage of
--force
option.see: (SAT-30043)
PRT test Cases example