-
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
Add a test case for capsule content counts granularity #16697
base: master
Are you sure you want to change the base?
Conversation
|
PRT Result
|
Turning into draft until we resolve https://issues.redhat.com/browse/SAT-28743 and discuss the expected behaviour on step |
36fcd95
to
d2f11a5
Compare
d2f11a5
to
557dd89
Compare
|
PRT Result
|
Copilot
AI
left a comment
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.
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 suggestion.
Comments skipped due to low confidence (1)
tests/foreman/cli/test_capsulecontent.py:128
- [nitpick] The function name _dictionarize_counts is ambiguous. Consider renaming it to something more descriptive like convert_counts_to_named_dict.
def _dictionarize_counts(data):
Problem Statement
In 6.17 capsule content counts will be updated separately for each CV when it is synced to the capsule and we should have a test case to cover that.
Solution
This PR just adds such a test case.
PRT test Cases example
Requires Katello/katello#11176