From ba39d1158c1e675e45d6e70c322ccb3f8713bdd1 Mon Sep 17 00:00:00 2001 From: Allan Date: Wed, 1 Feb 2017 06:05:53 +0200 Subject: [PATCH] Update test-module (#20737) * Update test-module Ensuring invoke is assigned Traceback (most recent call last): File "ansible/hacking/test-module", line 267, in main() File "ansible/hacking/test-module", line 263, in main runtest(modfile, argspath, modname, module_style, interpreters) File "ansible/hacking/test-module", line 207, in runtest invoke = "%s%s" % (invoke, modfile) UnboundLocalError: local variable 'invoke' referenced before assignment * Update test-module Made the change to only require a single if, making the function more 'DRY'. --- hacking/test-module | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/hacking/test-module b/hacking/test-module index 0301dd1452..3ef8bd65df 100755 --- a/hacking/test-module +++ b/hacking/test-module @@ -195,12 +195,11 @@ def ansiballz_setup(modfile, modname, interpreters): def runtest(modfile, argspath, modname, module_style, interpreters): """Test run a module, piping it's output for reporting.""" + invoke = "" if module_style == 'ansiballz': modfile, argspath = ansiballz_setup(modfile, modname, interpreters) if 'ansible_python_interpreter' in interpreters: invoke = "%s " % interpreters['ansible_python_interpreter'] - else: - invoke = "" os.system("chmod +x %s" % modfile)