-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
human_to_bytes: Unit tests for isbits arg of human_to_bytes function, adding explanation of isbits to the docstring #58623
Conversation
5c4f120
to
3c12fea
Compare
@samdoran , hey, as I promised yesterday there are tests for isbits. This PR also contains explanation about how the function handles isbits value. |
test/units/module_utils/common/text/formatters/test_human_to_bytes.py
Outdated
Show resolved
Hide resolved
There are unrelated problems with azure tests that looks like because of #58576, so ready_for_review |
This comment has been minimized.
This comment has been minimized.
ready_for_review |
test/units/module_utils/common/text/formatters/test_human_to_bytes.py
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.
Let's wait for @samdoran to approve too. |
…ytes.py Co-Authored-By: Sviatoslav Sydorenko <wk.cvs.github@sydorenko.org.ua>
@webknjaz thank you for reviewing and approving , the best experience (as always) |
Co-authored-by: Sviatoslav Sydorenko webknjaz@redhat.com
SUMMARY
Unit tests for isbits arg of human_to_bytes function from lib/ansible/module_utils/common/text/formatters.py
Add explanation about isbits to formatters.py and note to validation.py to clarify how the function works.
ISSUE TYPE