Skip to content

Commit a0f1481

Browse files
committed
tests: do not share CHECKER_CLASS
1 parent 65cdca2 commit a0f1481

File tree

5 files changed

+19
-12
lines changed

5 files changed

+19
-12
lines changed

tests/base_tester.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,12 @@
11
import sys
22
import os
33
from pylint.testutils import CheckerTestCase
4-
from pylint.checkers.variables import VariablesChecker
54
import astroid
65
from pylint_pytest import register, unregister
76

87

9-
class BasePytestFixtureChecker(CheckerTestCase):
10-
CHECKER_CLASS = VariablesChecker
8+
class BasePytestChecker(CheckerTestCase):
9+
CHECKER_CLASS = None
1110
MSG_ID = None
1211
MESSAGES = None
1312

@@ -28,7 +27,7 @@ def run_linter(self, enable_plugin, file_path=None):
2827

2928
try:
3029
self.walk(module) # run all checkers
31-
self.MESSAGES = self.linter.release_messages()
30+
BasePytestChecker.MESSAGES = self.linter.release_messages()
3231
finally:
3332
if enable_plugin:
3433
unregister()

tests/test_redefined_outer_name.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,10 @@
11
import pytest
2-
from base_tester import BasePytestFixtureChecker
2+
from pylint.checkers.variables import VariablesChecker
3+
from base_tester import BasePytestChecker
34

45

5-
class TestRedefinedOuterName(BasePytestFixtureChecker):
6+
class TestRedefinedOuterName(BasePytestChecker):
7+
CHECKER_CLASS = VariablesChecker
68
MSG_ID = 'redefined-outer-name'
79

810
@pytest.mark.parametrize('enable_plugin', [True, False])

tests/test_regression.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
11
import pylint
22
import pytest
3-
from base_tester import BasePytestFixtureChecker
3+
from pylint.checkers.variables import VariablesChecker
4+
from base_tester import BasePytestChecker
45

56

6-
class TestRegression(BasePytestFixtureChecker):
7+
class TestRegression(BasePytestChecker):
78
'''Covering some behaviors that shouldn't get impacted by the plugin'''
9+
CHECKER_CLASS = VariablesChecker
810
MSG_ID = 'regression'
911

1012
@pytest.mark.parametrize('enable_plugin', [True, False])

tests/test_unused_argument.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,10 @@
11
import pytest
2-
from base_tester import BasePytestFixtureChecker
2+
from pylint.checkers.variables import VariablesChecker
3+
from base_tester import BasePytestChecker
34

45

5-
class TestUnusedArgument(BasePytestFixtureChecker):
6+
class TestUnusedArgument(BasePytestChecker):
7+
CHECKER_CLASS = VariablesChecker
68
MSG_ID = 'unused-argument'
79

810
@pytest.mark.parametrize('enable_plugin', [True, False])

tests/test_unused_import.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,10 @@
11
import pytest
2-
from base_tester import BasePytestFixtureChecker
2+
from pylint.checkers.variables import VariablesChecker
3+
from base_tester import BasePytestChecker
34

45

5-
class TestUnusedImport(BasePytestFixtureChecker):
6+
class TestUnusedImport(BasePytestChecker):
7+
CHECKER_CLASS = VariablesChecker
68
MSG_ID = 'unused-import'
79

810
@pytest.mark.parametrize('enable_plugin', [True, False])

0 commit comments

Comments
 (0)