diff options
| author |   helenyao <yaohelan@huawei.com> | 2017-01-12 06:12:04 -0500 |
|---|---|---|
| committer | helenyao <yaohelan@huawei.com> | 2017-01-12 06:16:34 -0500 |
| commit | 650efd9c929e8e53c55e362078aede82260ca071 (patch) | |
| tree | da1a63b834f3773bcacf53db485a4ea182ab6a26 | |
| parent | 38db2197d4ae741ff9807152a48cb441fdc2cfa9 (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-x | functest/ci/run_tests.py | 7 |
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) |