aboutsummaryrefslogtreecommitdiffstats
path: root/moonclient
diff options
context:
space:
mode:
authorasteroide <thomas.duval@orange.com>2015-11-24 15:03:21 +0100
committerasteroide <thomas.duval@orange.com>2015-11-24 15:03:21 +0100
commit3f49e552962b719b1c6fa1486d4e2372849da16c (patch)
tree4257fb4bb17e867ec6e39f3ae740a195b0043708 /moonclient
parentfca74d4bc3569506a6659880a89aa009dc11f552 (diff)
Delete the unused shell_command.
Change-Id: I238ab36e06da03b7fd15772e2fdf70254405fd41
Diffstat (limited to 'moonclient')
-rw-r--r--moonclient/moonclient/tests.py8
1 files changed, 2 insertions, 6 deletions
diff --git a/moonclient/moonclient/tests.py b/moonclient/moonclient/tests.py
index 6484009f..e85c276b 100644
--- a/moonclient/moonclient/tests.py
+++ b/moonclient/moonclient/tests.py
@@ -110,15 +110,11 @@ class TestsLaunch(Lister):
if "external_command" in test:
ext_command = test["external_command"]
else:
- ext_command = test["shell_command"]
+ continue
ext_command = self.__replace_var_in_str(ext_command)
self.logfile.write("-----> {}\n".format(ext_command))
self.log.info(" \\-executing external \"{}\"".format(ext_command))
- if "external_command" in test:
- pipe = subprocess.Popen(shlex.split(ext_command), stdout=subprocess.PIPE, stderr=subprocess.PIPE)
- else:
- # Note (asteroide): security hazard! Must reduce the possible commands here.
- pipe = subprocess.Popen(ext_command, shell=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
+ pipe = subprocess.Popen(shlex.split(ext_command), stdout=subprocess.PIPE, stderr=subprocess.PIPE)
com = pipe.communicate()
result_str = com[0]
error_str = com[1]