diff options
author | Morgan Richomme <morgan.richomme@orange.com> | 2017-02-01 07:24:10 +0000 |
---|---|---|
committer | Gerrit Code Review <gerrit@opnfv.org> | 2017-02-01 07:24:10 +0000 |
commit | 1a498ce89ae559e73bc63d2ec3a31188a0d411df (patch) | |
tree | 29578d254d44a221fdba21b03705af6ff71451ef /functest/core/pytest_suite_runner.py | |
parent | 1cc752f55c9565cfd417a50afa7f355d421dd624 (diff) | |
parent | 11a4c6611bd758c82dbb98f426f40204b3bd2817 (diff) |
Merge "Tempest and SNAPS not pushing results to DB in case of FAIL"
Diffstat (limited to 'functest/core/pytest_suite_runner.py')
-rw-r--r-- | functest/core/pytest_suite_runner.py | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/functest/core/pytest_suite_runner.py b/functest/core/pytest_suite_runner.py index 1eed92b5..c168d7d9 100644 --- a/functest/core/pytest_suite_runner.py +++ b/functest/core/pytest_suite_runner.py @@ -41,14 +41,18 @@ class PyTestSuiteRunner(base.TestcaseBase): for test, message in result.failures: self.logger.error(str(test) + " FAILED with " + message) + # a result can be PASS or FAIL + # But in this case it means that the Execution was OK + # we shall distinguish Execution Error from FAIL results + # TestcaseBase.EX_RUN_ERROR means that the test case was not run + # not that it was run but the result was FAIL + exit_code = base.TestcaseBase.EX_OK if ((result.errors and len(result.errors) > 0) or (result.failures and len(result.failures) > 0)): self.logger.info("%s FAILED" % self.case_name) self.criteria = 'FAIL' - exit_code = base.TestcaseBase.EX_RUN_ERROR else: self.logger.info("%s OK" % self.case_name) - exit_code = base.TestcaseBase.EX_OK self.criteria = 'PASS' self.details = {} |