diff options
| author |   Morgan Richomme <morgan.richomme@orange.com> | 2016-05-27 10:24:31 +0200 |
|---|---|---|
| committer | Morgan Richomme <morgan.richomme@orange.com> | 2016-05-27 10:24:31 +0200 |
| commit | 9290fed4a7503367dc1eda024d2cdeabd50157d4 (patch) | |
| tree | 2ccd0cae4ba5238d573ae276662feddf454eca33 /testcases/features/bgpvpn.py | |
| parent | d00870df2d5d6c3be03799623372ee6801b380e6 (diff) | |
bug fix: if all tests passed failed could be empty
consider failed only if regex match
JIRA: FUNCTEST-234
Change-Id: Ie9d29c7bc1eb2f051600d689133b47bf276bc856
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>
Diffstat (limited to 'testcases/features/bgpvpn.py')
| -rw-r--r-- | testcases/features/bgpvpn.py | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/testcases/features/bgpvpn.py b/testcases/features/bgpvpn.py index 03aecbb32..11554ab1a 100644 --- a/testcases/features/bgpvpn.py +++ b/testcases/features/bgpvpn.py @@ -82,7 +82,8 @@ def main(): tests = m.group(1) # Look for tests failed m = re.search('Failed:(.*)', output) - failed = m.group(1) + if m is not None: + failed = m.group(1) except: logger.error("Impossible to parse the result file") @@ -93,9 +94,7 @@ def main(): "errors": error_logs} logger.info("Results: " + str(json_results)) - criteria = "failed" - if int(tests) > 0 and int(failed) < 1: - criteria = "passed" + # Push results in payload of testcase if args.report: logger.debug("Push result into DB") @@ -104,6 +103,9 @@ def main(): version = ft_utils.get_version(logger) pod_name = ft_utils.get_pod_name(logger) build_tag = ft_utils.get_build_tag(logger) + criteria = "failed" + if int(tests) > 0 and int(failed) < 1: + criteria = "passed" ft_utils.push_results_to_db(url, "sdnvpn", "bgpvpn_api", logger, pod_name, version, scenario, criteria, |