Fix some typing issues in src/conftest.py

These errors went unnoticed in the past because the file wasn't being
checked by `mypy`.

We also need to ignore type-checking for sybil, as it doesn't have
typing information.

Signed-off-by: Leandro Lucarella <[email protected]>

Author: llucax

pyproject.toml

@@ -142,6 +142,8 @@ module = [
   # There is a stubs package available, but it's not working:
   # https://github.com/eggplants/networkx-stubs/issues/1
   "networkx",
+  "sybil",
+  "sybil.*",
 ]
 ignore_missing_imports = true
 

src/conftest.py

@@ -42,11 +42,12 @@ def get_import_statements(code: str) -> list[str]:
         A list of import statements.
     """
     tree = ast.parse(code)
-    import_statements = []
+    import_statements: list[str] = []
 
     for node in ast.walk(tree):
         if isinstance(node, (ast.Import, ast.ImportFrom)):
             import_statement = ast.get_source_segment(code, node)
+            assert import_statement is not None
             import_statements.append(import_statement)
 
     return import_statements
@@ -85,7 +86,9 @@ def path_to_import_statement(path: Path) -> str:
     return import_statement
 
 
-class CustomPythonCodeBlockParser(CodeBlockParser):
+# We need to add the type ignore comment here because the Sybil library does not
+# have type annotations.
+class CustomPythonCodeBlockParser(CodeBlockParser):  # type: ignore[misc]
     """Code block parser that validates extracted code examples using pylint.
 
     This parser is a modified version of the default Python code block parser
@@ -103,7 +106,7 @@ class CustomPythonCodeBlockParser(CodeBlockParser):
     Pylint warnings which are unimportant for code examples are disabled.
     """
 
-    def __init__(self):
+    def __init__(self) -> None:
         """Initialize the parser."""
         super().__init__("python")
 
@@ -200,7 +203,9 @@ def validate_with_pylint(
             check=True,
         )
     except subprocess.CalledProcessError as exception:
-        return exception.output.splitlines()
+        output = exception.output
+        assert isinstance(output, str)
+        return output.splitlines()
 
     return []