diff options
author | MatthewLi <matthew.lijun@huawei.com> | 2017-03-29 23:24:05 -0400 |
---|---|---|
committer | Cédric Ollivier <cedric.ollivier@orange.com> | 2017-05-13 09:05:15 +0200 |
commit | 66331529e5e897873ff157c461259c3186be36e3 (patch) | |
tree | 28a6b571762c2b1dabcc764f188f05ae4a2de63b /functest/opnfv_tests/openstack/refstack_client/tempest_conf.py | |
parent | 4c8649d84e861412888f625d006d88929ab868d3 (diff) |
refstack client: improvement and fix of TestCase usage
JIRA: FUNCTEST-772
1, RefstackClientParser should not be a subclass of TestCase
2, tempest_conf.py used for generating reference tempest.conf file
not related to functest cmd, use logger is enough
Conflicts:
functest/opnfv_tests/openstack/refstack_client/refstack_client.py
Change-Id: If9c37331a5a7687e5704e546efcdea8175b1529c
Signed-off-by: MatthewLi <matthew.lijun@huawei.com>
(cherry picked from commit 2f0b1281308cb494d83240eaa5137ca0524718b7)
Diffstat (limited to 'functest/opnfv_tests/openstack/refstack_client/tempest_conf.py')
-rwxr-xr-x | functest/opnfv_tests/openstack/refstack_client/tempest_conf.py | 19 |
1 files changed, 9 insertions, 10 deletions
diff --git a/functest/opnfv_tests/openstack/refstack_client/tempest_conf.py b/functest/opnfv_tests/openstack/refstack_client/tempest_conf.py index 5624ed79b..d01f08727 100755 --- a/functest/opnfv_tests/openstack/refstack_client/tempest_conf.py +++ b/functest/opnfv_tests/openstack/refstack_client/tempest_conf.py @@ -5,9 +5,8 @@ # are made available under the terms of the Apache License, Version 2.0 # which accompanies this distribution, and is available at # http://www.apache.org/licenses/LICENSE-2.0 -import sys +import os -from functest.core import testcase from functest.opnfv_tests.openstack.tempest import conf_utils from functest.utils import openstack_utils from functest.utils.constants import CONST @@ -25,6 +24,8 @@ class TempestConf(object): self.DEPLOYMENT_ID = conf_utils.get_verifier_deployment_id() self.DEPLOYMENT_DIR = conf_utils.get_verifier_deployment_dir( self.VERIFIER_ID, self.DEPLOYMENT_ID) + self.confpath = os.path.join(CONST.dir_functest_test, + CONST.refstack_tempest_conf_path) def generate_tempestconf(self): try: @@ -33,21 +34,19 @@ class TempestConf(object): use_custom_images=True, use_custom_flavors=True) conf_utils.configure_tempest_defcore( self.DEPLOYMENT_DIR, img_flavor_dict) - except KeyError as e: - logger.error("defcore prepare env error with: %s", e) + except Exception as e: + logger.error("error with generating refstack client " + "reference tempest conf file: %s", e) def main(self): try: self.generate_tempestconf() - res = testcase.TestCase.EX_OK + logger.info("a reference tempest conf file generated " + "at %s", self.confpath) except Exception as e: logger.error('Error with run: %s', e) - res = testcase.TestCase.EX_RUN_ERROR - return res if __name__ == '__main__': tempestconf = TempestConf() - result = tempestconf.main() - if result != testcase.TestCase.EX_OK: - sys.exit(result) + tempestconf.main() |