diff options
| author |   Linda Wang <wangwulin@huawei.com> | 2017-12-04 08:33:59 +0000 |
|---|---|---|
| committer | Linda Wang <wangwulin@huawei.com> | 2017-12-14 07:23:00 +0000 |
| commit | 3f29dd2f11e3bb847fce5ac56060758d6076e8e7 (patch) | |
| tree | d8fce3daf5820ed65082a2451700e2cc9742c858 /functest/ci/tier_handler.py | |
| parent | 3306da5522f2576f2cd8431aac7fd4f3f4b32ca3 (diff) | |
Improve the pylint score of functest-core
It also modifies the file list to ensure the rating of ci modules.
Change-Id: Ia1e414be5364cb3da3d54882db428024ed6bd99f
Signed-off-by: Linda Wang <wangwulin@huawei.com>
Diffstat (limited to 'functest/ci/tier_handler.py')
| -rw-r--r-- | functest/ci/tier_handler.py | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/functest/ci/tier_handler.py b/functest/ci/tier_handler.py index dd3e77ce..9fc3f24d 100644 --- a/functest/ci/tier_handler.py +++ b/functest/ci/tier_handler.py @@ -7,6 +7,8 @@ # which accompanies this distribution, and is available at # http://www.apache.org/licenses/LICENSE-2.0 +"""Tier and TestCase classes to wrap the testcases config file""" +# pylint: disable=missing-docstring import re import textwrap @@ -100,13 +102,9 @@ class Tier(object): class TestCase(object): - def __init__(self, name, - enabled, - dependency, - criteria, - blocking, - description="", - project=""): + def __init__(self, name, enabled, dependency, criteria, blocking, + description="", project=""): + # pylint: disable=too-many-arguments self.name = name self.enabled = enabled self.dependency = dependency @@ -117,7 +115,7 @@ class TestCase(object): @staticmethod def is_none(item): - return item is None or item is "" + return item is None or item == "" def is_compatible(self, ci_installer, ci_scenario): try: |