diff options
author | Cédric Ollivier <cedric.ollivier@orange.com> | 2019-04-23 16:49:55 +0200 |
---|---|---|
committer | Cédric Ollivier <cedric.ollivier@orange.com> | 2019-04-24 20:25:14 +0200 |
commit | 9dcc30874194382a25c66baf359b863c6e013caf (patch) | |
tree | 421fa4ee80964f775fc1d0c0008e0afaf1ebe8ea /functest/opnfv_tests/vnf/ims/clearwater.py | |
parent | b9048a25355e1d6c29c84ca10a3d859841468f50 (diff) |
Refactor modules to avoid duplicated code
It also increases number of lines to warm about duplicated codes.
Change-Id: I3914da8a90cbfe8fd024e9944c3adc8a617330d3
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
Diffstat (limited to 'functest/opnfv_tests/vnf/ims/clearwater.py')
-rw-r--r-- | functest/opnfv_tests/vnf/ims/clearwater.py | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/functest/opnfv_tests/vnf/ims/clearwater.py b/functest/opnfv_tests/vnf/ims/clearwater.py index 605ba8a2a..b05f8e7d6 100644 --- a/functest/opnfv_tests/vnf/ims/clearwater.py +++ b/functest/opnfv_tests/vnf/ims/clearwater.py @@ -205,7 +205,12 @@ class ClearwaterTesting(object): vims_test_result["skipped"] = int(grp.group(3)) vims_test_result['passed'] = ( int(grp.group(2)) - int(grp.group(3)) - int(grp.group(1))) + if vims_test_result['total'] - vims_test_result['skipped'] > 0: + vnf_test_rate = vims_test_result['passed'] / ( + vims_test_result['total'] - vims_test_result['skipped']) + else: + vnf_test_rate = 0 except Exception: # pylint: disable=broad-except self.logger.exception("Cannot parse live tests results") - return None - return vims_test_result + return None, 0 + return vims_test_result, vnf_test_rate |