diff options
Diffstat (limited to 'xtesting')
-rw-r--r-- | xtesting/ci/run_tests.py | 5 | ||||
-rw-r--r-- | xtesting/ci/tier_builder.py | 2 | ||||
-rw-r--r-- | xtesting/ci/tier_handler.py | 4 | ||||
-rw-r--r-- | xtesting/core/testcase.py | 2 | ||||
-rw-r--r-- | xtesting/tests/unit/core/test_behaveframework.py | 7 | ||||
-rw-r--r-- | xtesting/tests/unit/core/test_feature.py | 1 | ||||
-rw-r--r-- | xtesting/tests/unit/core/test_testcase.py | 1 |
7 files changed, 11 insertions, 11 deletions
diff --git a/xtesting/ci/run_tests.py b/xtesting/ci/run_tests.py index 94bf2787..5e2b49e5 100644 --- a/xtesting/ci/run_tests.py +++ b/xtesting/ci/run_tests.py @@ -46,10 +46,9 @@ class Result(enum.Enum): class BlockingTestFailed(Exception): """Exception when the blocking test fails""" - pass -class RunTestsParser(object): +class RunTestsParser(): """Parser to run tests""" # pylint: disable=too-few-public-methods @@ -78,7 +77,7 @@ class RunTestsParser(object): return vars(self.parser.parse_args(argv)) -class Runner(object): +class Runner(): """Runner class""" def __init__(self): diff --git a/xtesting/ci/tier_builder.py b/xtesting/ci/tier_builder.py index b0050b13..27afc7d1 100644 --- a/xtesting/ci/tier_builder.py +++ b/xtesting/ci/tier_builder.py @@ -16,7 +16,7 @@ from xtesting.ci import tier_handler from xtesting.utils import env -class TierBuilder(object): +class TierBuilder(): # pylint: disable=missing-docstring def __init__(self, testcases_file): diff --git a/xtesting/ci/tier_handler.py b/xtesting/ci/tier_handler.py index 4b74e37c..8359d91c 100644 --- a/xtesting/ci/tier_handler.py +++ b/xtesting/ci/tier_handler.py @@ -34,7 +34,7 @@ def split_text(text, max_len): return lines -class Tier(object): +class Tier(): def __init__(self, name, order, description=""): self.tests_array = [] @@ -96,7 +96,7 @@ class Tier(object): return msg.get_string() -class TestCase(object): +class TestCase(): def __init__(self, name, enabled, skipped, criteria, blocking, description="", project=""): diff --git a/xtesting/core/testcase.py b/xtesting/core/testcase.py index c548a2a8..c89e4c88 100644 --- a/xtesting/core/testcase.py +++ b/xtesting/core/testcase.py @@ -27,7 +27,7 @@ __author__ = "Cedric Ollivier <cedric.ollivier@orange.com>" @six.add_metaclass(abc.ABCMeta) -class TestCase(object): +class TestCase(): # pylint: disable=too-many-instance-attributes """Base model for single test case.""" diff --git a/xtesting/tests/unit/core/test_behaveframework.py b/xtesting/tests/unit/core/test_behaveframework.py index f18cac03..70ca9738 100644 --- a/xtesting/tests/unit/core/test_behaveframework.py +++ b/xtesting/tests/unit/core/test_behaveframework.py @@ -66,11 +66,10 @@ class ParseResultTesting(unittest.TestCase): data = [{'status': 'passed'}, {'status': 'passed'}] self._test_result(data, 100) - def test_count(self): + @mock.patch('six.moves.builtins.open', mock.mock_open()) + def test_count(self, *args): # pylint: disable=unused-argument self._response.extend([{'status': 'failed'}, {'status': 'skipped'}]) - with mock.patch('six.moves.builtins.open', mock.mock_open()), \ - mock.patch('json.load', mock.Mock( - return_value=self._response)): + with mock.patch('json.load', mock.Mock(return_value=self._response)): self.test.parse_results() self.assertEqual(self.test.details['pass_tests'], 1) self.assertEqual(self.test.details['fail_tests'], 1) diff --git a/xtesting/tests/unit/core/test_feature.py b/xtesting/tests/unit/core/test_feature.py index 30678798..785f6e16 100644 --- a/xtesting/tests/unit/core/test_feature.py +++ b/xtesting/tests/unit/core/test_feature.py @@ -29,6 +29,7 @@ class FakeTestCase(feature.Feature): class AbstractFeatureTesting(unittest.TestCase): def test_run_unimplemented(self): + # pylint: disable=abstract-class-instantiated with self.assertRaises(TypeError): feature.Feature(case_name="feature", project_name="xtesting") diff --git a/xtesting/tests/unit/core/test_testcase.py b/xtesting/tests/unit/core/test_testcase.py index 51ea6f35..fc612973 100644 --- a/xtesting/tests/unit/core/test_testcase.py +++ b/xtesting/tests/unit/core/test_testcase.py @@ -35,6 +35,7 @@ class FakeTestCase(testcase.TestCase): class AbstractTestCaseTesting(unittest.TestCase): def test_run_unimplemented(self): + # pylint: disable=abstract-class-instantiated with self.assertRaises(TypeError): testcase.TestCase(case_name="base", project_name="xtesting") |