diff options
| author | Daniƫl van Noord <13665637+DanielNoord@users.noreply.github.com> | 2021-12-21 19:42:24 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-12-21 19:42:24 +0100 |
| commit | 996da9ef9b6ce3cdefaa35afbb2550c5f795c635 (patch) | |
| tree | 7d67ba2416ade928ae464d19be3ae90ba87bd43e | |
| parent | 797d99b5e85549aa60b3edea039d79f5a4040662 (diff) | |
| download | pylint-git-996da9ef9b6ce3cdefaa35afbb2550c5f795c635.tar.gz | |
Rename ``init_linter`` fixture to ``initialized_linter``, add filename (#5581)
| -rw-r--r-- | tests/lint/unittest_lint.py | 40 |
1 files changed, 21 insertions, 19 deletions
diff --git a/tests/lint/unittest_lint.py b/tests/lint/unittest_lint.py index f504efcf6..582e9faa8 100644 --- a/tests/lint/unittest_lint.py +++ b/tests/lint/unittest_lint.py @@ -249,9 +249,9 @@ def reporter(): @pytest.fixture -def init_linter(linter: PyLinter) -> PyLinter: +def initialized_linter(linter: PyLinter) -> PyLinter: linter.open() - linter.set_current_module("toto") + linter.set_current_module("toto", "mydir/toto") linter.file_state = FileState("toto") return linter @@ -273,8 +273,8 @@ def test_pylint_visit_method_taken_in_account(linter: PyLinter) -> None: linter.check(["abc"]) -def test_enable_message(init_linter: PyLinter) -> None: - linter = init_linter +def test_enable_message(initialized_linter: PyLinter) -> None: + linter = initialized_linter assert linter.is_message_enabled("W0101") assert linter.is_message_enabled("W0102") linter.disable("W0101", scope="package") @@ -290,8 +290,8 @@ def test_enable_message(init_linter: PyLinter) -> None: assert linter.is_message_enabled("W0102", 1) -def test_enable_message_category(init_linter: PyLinter) -> None: - linter = init_linter +def test_enable_message_category(initialized_linter: PyLinter) -> None: + linter = initialized_linter assert linter.is_message_enabled("W0101") assert linter.is_message_enabled("C0202") linter.disable("W", scope="package") @@ -309,11 +309,11 @@ def test_enable_message_category(init_linter: PyLinter) -> None: assert linter.is_message_enabled("C0202", line=1) -def test_message_state_scope(init_linter: PyLinter) -> None: +def test_message_state_scope(initialized_linter: PyLinter) -> None: class FakeConfig: confidence = ["HIGH"] - linter = init_linter + linter = initialized_linter linter.disable("C0202") assert MSG_STATE_SCOPE_CONFIG == linter._get_message_state_scope("C0202") linter.disable("W0101", scope="module", line=3) @@ -327,8 +327,8 @@ def test_message_state_scope(init_linter: PyLinter) -> None: ) -def test_enable_message_block(init_linter: PyLinter) -> None: - linter = init_linter +def test_enable_message_block(initialized_linter: PyLinter) -> None: + linter = initialized_linter linter.open() filepath = join(REGRTEST_DATA_DIR, "func_block_disable_msg.py") linter.set_current_module("func_block_disable_msg") @@ -385,12 +385,12 @@ def test_enable_message_block(init_linter: PyLinter) -> None: assert fs._suppression_mapping["E1101", 110] == 109 -def test_enable_by_symbol(init_linter: PyLinter) -> None: +def test_enable_by_symbol(initialized_linter: PyLinter) -> None: """messages can be controlled by symbolic names. The state is consistent across symbols and numbers. """ - linter = init_linter + linter = initialized_linter assert linter.is_message_enabled("W0101") assert linter.is_message_enabled("unreachable") assert linter.is_message_enabled("W0102") @@ -652,8 +652,10 @@ def test_full_documentation(linter: PyLinter) -> None: assert re.search(regexp, output) -def test_list_msgs_enabled(init_linter: PyLinter, capsys: CaptureFixture) -> None: - linter = init_linter +def test_list_msgs_enabled( + initialized_linter: PyLinter, capsys: CaptureFixture +) -> None: + linter = initialized_linter linter.enable("W0101", scope="package") linter.disable("W0102", scope="package") linter.list_messages_enabled() @@ -865,12 +867,12 @@ def test_multiprocessing(jobs: int) -> None: assert len(messages) == len(set(messages)) -def test_filename_with__init__(init_linter: PyLinter) -> None: +def test_filename_with__init__(initialized_linter: PyLinter) -> None: # This tracks a regression where a file whose name ends in __init__.py, # such as flycheck__init__.py, would accidentally lead to linting the # entire containing directory. reporter = testutils.GenericTestReporter() - linter = init_linter + linter = initialized_linter linter.open() linter.set_reporter(reporter) filepath = join(INPUT_DIR, "not__init__.py") @@ -879,15 +881,15 @@ def test_filename_with__init__(init_linter: PyLinter) -> None: assert len(messages) == 0 -def test_by_module_statement_value(init_linter: PyLinter) -> None: +def test_by_module_statement_value(initialized_linter: PyLinter) -> None: """Test "statement" for each module analyzed of computed correctly.""" - linter = init_linter + linter = initialized_linter linter.check([os.path.join(os.path.dirname(__file__), "data")]) by_module_stats = linter.stats.by_module for module, module_stats in by_module_stats.items(): - linter2 = init_linter + linter2 = initialized_linter if module == "data": linter2.check([os.path.join(os.path.dirname(__file__), "data/__init__.py")]) else: |
