diff options
| author |   Juha Kosonen <juha.kosonen@nokia.com> | 2018-02-02 13:27:33 +0200 |
|---|---|---|
| committer | Juha Kosonen <juha.kosonen@nokia.com> | 2018-02-02 13:27:33 +0200 |
| commit | 0925daced02fd7555d43cd96e5b944d0f8edb748 (patch) | |
| tree | 48f3632038e0b6cd98348c513a2753ec004f4c29 /functest/tests/unit/openstack/tempest/test_tempest.py | |
| parent | 2db621767d66b632fab4e2594e4524348484ad56 (diff) | |
Do not use shell to execute commands in tempest
Change-Id: I4d699481fdc63058c12ca9cfe18138e87025e9b3
Signed-off-by: Juha Kosonen <juha.kosonen@nokia.com>
Diffstat (limited to 'functest/tests/unit/openstack/tempest/test_tempest.py')
| -rw-r--r-- | functest/tests/unit/openstack/tempest/test_tempest.py | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/functest/tests/unit/openstack/tempest/test_tempest.py b/functest/tests/unit/openstack/tempest/test_tempest.py index 292eb75b..37b61f3b 100644 --- a/functest/tests/unit/openstack/tempest/test_tempest.py +++ b/functest/tests/unit/openstack/tempest/test_tempest.py @@ -145,12 +145,11 @@ class OSTempestTesting(unittest.TestCase): 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) |