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
ec2_snapshot_info/tests: add unit-tests #1211
ec2_snapshot_info/tests: add unit-tests #1211
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
Docs Build 📝Thank you for contribution!✨ This PR has been merged and your docs changes will be incorporated when they are next published. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
def list_ec2_snapshots(connection, module, request_args): | ||
|
||
try: | ||
snapshots = get_snapshots(connection, module, request_args) |
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.
Since exit_json()
actually call exit() syscall, I would advice to just return a dict and call module.exit_json()
from outside of the function. This way, to can test the behaviour of your function just by checking the output.
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.
Fixed.
recheck |
recheck |
This change depends on a change that failed to merge. |
recheck |
recheck |
…s#1211) wafv2_ip_set - fix bugs with changing description SUMMARY updating just the description didn't update the changed state ISSUE TYPE Bugfix Pull Request COMPONENT NAME wafv2_ip_set ADDITIONAL INFORMATION Reviewed-by: Markus Bergholz <git@osuv.de>
…s#1211) wafv2_ip_set - fix bugs with changing description SUMMARY updating just the description didn't update the changed state ISSUE TYPE Bugfix Pull Request COMPONENT NAME wafv2_ip_set ADDITIONAL INFORMATION Reviewed-by: Markus Bergholz <git@osuv.de>
…s#1211) wafv2_ip_set - fix bugs with changing description SUMMARY updating just the description didn't update the changed state ISSUE TYPE Bugfix Pull Request COMPONENT NAME wafv2_ip_set ADDITIONAL INFORMATION Reviewed-by: Markus Bergholz <git@osuv.de>
Depends-On: #1234
Depends-On: #1235
SUMMARY
list_ec2_snapshots
to moveconnection.describe_snapshots
to separate method.COMPONENT NAME
ec2_snapshot_info
ADDITIONAL INFO
latest CI run coverage report plugins_modules_ec2_snapshot_info_py.html