diff options
| author |   Cédric Ollivier <cedric.ollivier@orange.com> | 2017-03-26 11:55:10 +0200 |
|---|---|---|
| committer |   Morgan Richomme <morgan.richomme@orange.com> | 2017-03-27 06:48:04 +0000 |
| commit | 3e3d3ed62d87eb5ff8bc4860c328f218b0f67315 (patch) | |
| tree | fd2c7d40fbb25b33c9aef9e73b2784ebf594d582 | |
| parent | 609ecc2f8f326eea24107fc89fdd0ae73a480ab3 (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>
(cherry picked from commit e4d7506a5a21550a89f897dfaea3f0dcae352e5b)
| -rw-r--r-- | functest/core/testcase_base.py | 5 |
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 |