summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorhelenyao <yaohelan@huawei.com>2017-01-12 06:12:04 -0500
committerhelenyao <yaohelan@huawei.com>2017-01-12 06:16:34 -0500
commit650efd9c929e8e53c55e362078aede82260ca071 (patch)
treeda1a63b834f3773bcacf53db485a4ea182ab6a26
parent38db2197d4ae741ff9807152a48cb441fdc2cfa9 (diff)
Bugfix: fix the wrong return code
The bug was introduced by patch[1] [1] https://gerrit.opnfv.org/gerrit/#/c/24619/ Change-Id: Icf6c267aba26fcb5b129a611423dcf2200f95277 Signed-off-by: helenyao <yaohelan@huawei.com>
-rwxr-xr-xfunctest/ci/run_tests.py7
1 files changed, 4 insertions, 3 deletions
diff --git a/functest/ci/run_tests.py b/functest/ci/run_tests.py
index 7aac9d2cf..35345fee5 100755
--- a/functest/ci/run_tests.py
+++ b/functest/ci/run_tests.py
@@ -170,7 +170,7 @@ def run_test(test, tier_name):
if result != 0:
logger.error("The test case '%s' failed. " % test_name)
- OVERALL_RESULT = -1
+ GlobalVariables.OVERALL_RESULT = -1
result_str = "FAIL"
if test.is_blocking():
@@ -180,8 +180,9 @@ def run_test(test, tier_name):
# if it is a single test we don't print the whole results table
update_test_info(test_name, result_str, duration_str)
generate_report.main(GlobalVariables.EXECUTED_TEST_CASES)
- logger.info("Execution exit value: %s" % OVERALL_RESULT)
- sys.exit(OVERALL_RESULT)
+ logger.info("Execution exit value: %s" %
+ GlobalVariables.OVERALL_RESULT)
+ sys.exit(GlobalVariables.OVERALL_RESULT)
update_test_info(test_name, result_str, duration_str)