summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorCédric Ollivier <cedric.ollivier@orange.com>2017-03-26 11:55:10 +0200
committerCédric Ollivier <cedric.ollivier@orange.com>2017-03-26 14:01:27 +0200
commite4d7506a5a21550a89f897dfaea3f0dcae352e5b (patch)
treeb6ff83746d2472684f3e3df0e02577214c2052ad
parent18745da8f335fe3049ffdd5acdafb5848981a1c9 (diff)
Fix pylint warnings in testcase_base
It takes into account the following warnings: - bare-except - unused-argument - broad-except Change-Id: I3a50bf6d07823edc80a7f0f78f46681ace7f4a39 Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
-rw-r--r--functest/core/testcase_base.py5
1 files changed, 3 insertions, 2 deletions
diff --git a/functest/core/testcase_base.py b/functest/core/testcase_base.py
index 838b63983..2848073f7 100644
--- a/functest/core/testcase_base.py
+++ b/functest/core/testcase_base.py
@@ -35,11 +35,12 @@ class TestcaseBase(object):
assert self.criteria
if self.criteria == 'PASS':
return TestcaseBase.EX_OK
- except:
+ except AssertionError:
self.logger.error("Please run test before checking the results")
return TestcaseBase.EX_TESTCASE_FAILED
def run(self, **kwargs):
+ # pylint: disable=unused-argument
self.logger.error("Run must be implemented")
return TestcaseBase.EX_RUN_ERROR
@@ -58,6 +59,6 @@ class TestcaseBase(object):
else:
self.logger.error("The results cannot be pushed to DB")
return TestcaseBase.EX_PUSH_TO_DB_ERROR
- except Exception:
+ except Exception: # pylint: disable=broad-except
self.logger.exception("The results cannot be pushed to DB")
return TestcaseBase.EX_PUSH_TO_DB_ERROR