test: refactor all

Author: Nytelife26

tests/_test_version.py

@@ -2,6 +2,7 @@
 
 import importlib
 from itertools import chain
+from os import walk
 from pathlib import Path
 from types import ModuleType
 
@@ -12,17 +13,17 @@
 def assert_pass(check: Check, examples: tuple[str, ...]) -> None:
     """Assert that the `check` does not produce results against `examples`."""
     for example in examples:
-        assert (
-            check.check_with_flags(example) == []
-        ), f"False positive against {check.path}: '{example}'"
+        assert check.check_with_flags(example) == [], (
+            f"False positive against {check.path}: '{example}'"
+        )
 
 
 def assert_fail(check: Check, examples: tuple[str, ...]) -> None:
     """Assert that the `check` produces results against `examples`."""
     for example in examples:
-        assert (
-            check.check_with_flags(example) != []
-        ), f"False negative against {check.path}: '{example}'"
+        assert check.check_with_flags(example) != [], (
+            f"False negative against {check.path}: '{example}'"
+        )
 
 
 def is_check(path: Path) -> bool:
@@ -34,8 +35,8 @@ def get_check_paths() -> list[Path]:
     """Traverse through subdirectories and select check paths."""
     return [
         file
-        for root, _, files in (proselint_path / "checks").walk()
-        for file in map(root.__truediv__, files)
+        for root, _, files in walk(proselint_path / "checks")
+        for file in map(Path(root).__truediv__, files)
         if is_check(file) and (file.parent / "__init__.py").exists()
     ]
 

tests/check.py

@@ -45,8 +45,8 @@ def test_check_examples(
         result = chain.from_iterable(
             map(Check.check_with_flags, checks, repeat(example))
         )
-        assert (
-            result != []
-            ), f"False negative against {module_name}: '{example}'"
+        assert result != [], (
+            f"False negative against {module_name}: '{example}'"
+        )
     for check in checks:
         assert_pass(check, should_pass)

tests/common.py

@@ -45,7 +45,7 @@ def test_consistency_smoke(
     term_pair: tuple[str, str], path: str, noise: str
 ) -> None:
     """Return no matches when no elements are present."""
-    assume(term_pair[0] not in noise and term_pair[1] not in noise)
+    assume(all(term not in noise.lower() for term in term_pair))
     check_type = types.Consistency(term_pairs=(term_pair,))
     check = Check(check_type=check_type, path=path, message="{} || {}")
     assert check.check(noise) == []

tests/test_annotations.py

@@ -0,0 +1,46 @@
+"""Verify that the CLI behaves correctly."""
+
+from pathlib import Path
+
+from click.testing import CliRunner
+from pytest import fixture
+
+from proselint.command_line import proselint
+from proselint.version import __version__
+
+CHAR_FILE = Path(__file__, "../invalid-chars.txt").resolve()
+
+
+@fixture
+def _runner() -> CliRunner:  # pyright: ignore[reportUnusedFunction]
+    return CliRunner()
+
+
+def test_exit_code_demo(_runner: CliRunner) -> None:
+    """Ensure that linting the demo returns an exit code of 1."""
+    output = _runner.invoke(proselint, "--demo")
+    assert output.exit_code == 1
+
+
+def test_exit_code_version(_runner: CliRunner) -> None:
+    """Ensure that getting the version returns an exit code of 0."""
+    output = _runner.invoke(proselint, "--version")
+    assert output.exit_code == 0
+
+
+def test_invalid_characters(_runner: CliRunner) -> None:
+    """Ensure that invalid characters do not break proselint."""
+    runner = CliRunner()
+
+    output = runner.invoke(proselint, CHAR_FILE.as_posix())
+
+    assert "UnicodeDecodeError" not in output.stdout
+    assert "FileNotFoundError" not in output.stdout
+
+
+def test_version(_runner: CliRunner) -> None:
+    """Ensure that the version number is correct."""
+    runner = CliRunner()
+
+    output = runner.invoke(proselint, "--version")
+    assert __version__ in output.stdout