super(TempestCommon, self).__init__(**kwargs)
self.resources = TempestResourcesManager(**kwargs)
self.MODE = ""
- self.OPTION = ""
+ self.OPTION = []
self.VERIFIER_ID = conf_utils.get_verifier_id()
self.VERIFIER_REPO_DIR = conf_utils.get_verifier_repo_dir(
self.VERIFIER_ID)
result_file.close()
def run_verifier_tests(self):
- self.OPTION += (" --load-list {} --detailed"
- .format(conf_utils.TEMPEST_LIST))
-
- cmd_line = "rally verify start " + self.OPTION
- logger.info("Starting Tempest test suite: '%s'." % cmd_line)
+ cmd = ["rally", "verify", "start", "--load-list",
+ conf_utils.TEMPEST_LIST]
+ cmd.extend(self.OPTION)
+ logger.info("Starting Tempest test suite: '%s'." % cmd)
header = ("Tempest environment:\n"
" SUT: %s\n Scenario: %s\n Node: %s\n Date: %s\n" %
f_env.write(header)
p = subprocess.Popen(
- cmd_line, shell=True,
+ cmd,
stdout=subprocess.PIPE,
stderr=f_stderr,
bufsize=1)
if self.VERIFICATION_ID is None:
raise Exception('Verification UUID not found')
- cmd_line = "rally verify show --uuid {}".format(self.VERIFICATION_ID)
- logger.info("Showing result for a verification: '%s'." % cmd_line)
- p = subprocess.Popen(cmd_line,
- shell=True,
+ cmd = ["rally", "verify", "show", "--uuid", self.VERIFICATION_ID]
+ logger.info("Showing result for a verification: '%s'." % cmd)
+ p = subprocess.Popen(cmd,
stdout=subprocess.PIPE,
stderr=subprocess.STDOUT)
for line in p.stdout:
kwargs["case_name"] = 'tempest_smoke_serial'
TempestCommon.__init__(self, **kwargs)
self.MODE = "smoke"
- self.OPTION = "--concurrency 1"
+ self.OPTION = ["--concurrency", "1"]
class TempestSmokeParallel(TempestCommon):
kwargs["case_name"] = 'tempest_smoke_parallel'
TempestCommon.__init__(self, **kwargs)
self.MODE = "smoke"
- self.OPTION = ""
class TempestFullParallel(TempestCommon):
kwargs["case_name"] = 'tempest_custom'
TempestCommon.__init__(self, **kwargs)
self.MODE = "custom"
- self.OPTION = "--concurrency 1"
+ self.OPTION = ["--concurrency", "1"]
class TempestDefcore(TempestCommon):
kwargs["case_name"] = 'tempest_defcore'
TempestCommon.__init__(self, **kwargs)
self.MODE = "defcore"
- self.OPTION = "--concurrency 1"
+ self.OPTION = ["--concurrency", "1"]
class TempestResourcesManager(object):
mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
'subprocess.Popen'):
conf_utils.TEMPEST_LIST = 'test_tempest_list'
- cmd_line = ("rally verify start --load-list "
- "test_tempest_list --detailed")
+ cmd = ["rally", "verify", "start", "--load-list",
+ conf_utils.TEMPEST_LIST]
self.tempestcommon.run_verifier_tests()
mock_logger_info. \
- assert_any_call("Starting Tempest test suite: '%s'."
- % cmd_line)
+ assert_any_call("Starting Tempest test suite: '%s'." % cmd)
@mock.patch('functest.opnfv_tests.openstack.tempest.tempest.'
'os.path.exists', return_value=False)