Fix UnboundLocalError in basic.py

* Fix for UnboundLocalError while accessing deprecations
  in result
* Add Unit test

Fixes #24592

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
This commit is contained in:
Abhijeet Kasurde 2017-05-14 17:36:32 +05:30 committed by Brian Coca
parent 9f3a405706
commit 764b4b20ec
2 changed files with 19 additions and 1 deletions

View file

@ -1999,7 +1999,7 @@ class AnsibleModule(object):
else: else:
self.deprecate(d) self.deprecate(d)
else: else:
self.deprecate(d) self.deprecate(kwargs['deprecations'])
if self._deprecations: if self._deprecations:
kwargs['deprecations'] = self._deprecations kwargs['deprecations'] = self._deprecations

View file

@ -69,3 +69,21 @@ class TestAnsibleModuleWarnDeprecate(unittest.TestCase):
{u'msg': u'deprecation4', u'version': '2.4'}, {u'msg': u'deprecation4', u'version': '2.4'},
]) ])
def test_deprecate_without_list(self):
args = json.dumps(dict(ANSIBLE_MODULE_ARGS={}))
with swap_stdin_and_argv(stdin_data=args):
with swap_stdout():
ansible.module_utils.basic._ANSIBLE_ARGS = None
am = ansible.module_utils.basic.AnsibleModule(
argument_spec = dict(),
)
am._name = 'unittest'
with self.assertRaises(SystemExit):
am.exit_json(deprecations='Simple deprecation warning')
output = json.loads(sys.stdout.getvalue())
self.assertTrue('warnings' not in output or output['warnings'] == [])
self.assertEquals(output['deprecations'], [
{u'msg': u'Simple deprecation warning', u'version': None},
])