diff options
author | Kenny Woodson <kwoodson@redhat.com> | 2017-03-24 13:07:24 -0400 |
---|---|---|
committer | Kenny Woodson <kwoodson@redhat.com> | 2017-03-28 15:28:34 -0400 |
commit | aa9e19c20b745f91a47292f8754a8cf52017c3e4 (patch) | |
tree | 4bb2dd64e3adc6483c5e76cad76d9f211ecffd50 /roles/lib_utils/src/ansible | |
parent | a69a84339e1bf187aa970316a4148138cb7f82fb (diff) | |
download | openshift-aa9e19c20b745f91a47292f8754a8cf52017c3e4.tar.gz openshift-aa9e19c20b745f91a47292f8754a8cf52017c3e4.tar.bz2 openshift-aa9e19c20b745f91a47292f8754a8cf52017c3e4.tar.xz openshift-aa9e19c20b745f91a47292f8754a8cf52017c3e4.zip |
Adding a few more test cases. Fixed a bug when key was empty. Safeguard against yedit module being passed an empty key
Diffstat (limited to 'roles/lib_utils/src/ansible')
-rw-r--r-- | roles/lib_utils/src/ansible/yedit.py | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/roles/lib_utils/src/ansible/yedit.py b/roles/lib_utils/src/ansible/yedit.py index ea112ac83..bdb9915d6 100644 --- a/roles/lib_utils/src/ansible/yedit.py +++ b/roles/lib_utils/src/ansible/yedit.py @@ -32,6 +32,9 @@ def main(): required_one_of=[["content", "src"]], ) + if module.params['src'] is not None and module.params['key'] in [None, '']: + module.fail_json(failed=True, msg='Empty value for parameter key not allowed.') + rval = Yedit.run_ansible(module.params) if 'failed' in rval and rval['failed']: module.fail_json(**rval) |