Remove mypy overrides for tests.test_quickstart

Author: adamtheturtle

pyproject.toml

@@ -295,7 +295,6 @@ disallow_untyped_defs = false
 [[tool.mypy.overrides]]
 module = [
     # tests/
-    "tests.test_quickstart",
     "tests.test_search",
     # tests/test_builders
     "tests.test_builders.test_build_latex",

tests/test_quickstart.py

@@ -4,7 +4,7 @@
 
 import time
 from io import StringIO
-from typing import TYPE_CHECKING
+from typing import TYPE_CHECKING, Protocol
 
 import pytest
 
@@ -13,20 +13,26 @@
 from sphinx.testing.util import SphinxTestApp
 
 if TYPE_CHECKING:
-    from collections.abc import Callable
     from pathlib import Path
     from typing import Any
 
 warnfile = StringIO()
 
 
-def setup_module():
+def setup_module() -> None:
     disable_colour()
 
 
+class _MockTermInput(Protocol):
+    """Protocol for mocking term_input in quickstart."""
+
+    def __call__(self, prompt: str) -> str:
+        """Mock input function that returns a string based on the prompt."""
+
+
 def mock_input(
     answers: dict[str, str], needanswer: bool = False
-) -> Callable[[str], str]:
+) -> _MockTermInput:
     called = set()
 
     def input_(prompt: str) -> str:
@@ -45,10 +51,16 @@ def input_(prompt: str) -> str:
     return input_
 
 
-real_input: Callable[[str], str] = input
+def _real_input_but_allow_named_args_in_hint(*args: str, **kwargs: str) -> str:
+    """A wrapper for the real input function to allow named arguments in the type hint."""
+    assert not kwargs, 'Named arguments are not allowed in input()'
+    return input(*args)
+
 
+real_input: _MockTermInput = _real_input_but_allow_named_args_in_hint
 
-def teardown_module():
+
+def teardown_module() -> None:
     qs.term_input = real_input
     enable_colour()
 
@@ -61,7 +73,7 @@ def test_do_prompt() -> None:
         'Q5': 'no',
         'Q6': 'foo',
     }
-    qs.term_input = mock_input(answers)  # type: ignore[assignment]
+    qs.term_input = mock_input(answers)
 
     assert qs.do_prompt('Q1', default='v1') == 'v1'
     assert qs.do_prompt('Q3', default='v3_default') == 'v3'
@@ -79,7 +91,7 @@ def test_do_prompt_inputstrip() -> None:
         'Q3': 'N',
         'Q4': 'N ',
     }
-    qs.term_input = mock_input(answers)  # type: ignore[assignment]
+    qs.term_input = mock_input(answers)
 
     assert qs.do_prompt('Q1') == 'Y'
     assert qs.do_prompt('Q2') == 'Yes'
@@ -91,12 +103,12 @@ def test_do_prompt_with_nonascii() -> None:
     answers = {
         'Q1': '\u30c9\u30a4\u30c4',
     }
-    qs.term_input = mock_input(answers)  # type: ignore[assignment]
+    qs.term_input = mock_input(answers)
     result = qs.do_prompt('Q1', default='\u65e5\u672c')
     assert result == '\u30c9\u30a4\u30c4'
 
 
-def test_quickstart_defaults(tmp_path):
+def test_quickstart_defaults(tmp_path: Path) -> None:
     answers = {
         'Root path': str(tmp_path),
         'Project name': 'Sphinx Test',
@@ -127,7 +139,7 @@ def test_quickstart_defaults(tmp_path):
     assert (tmp_path / 'make.bat').is_file()
 
 
-def test_quickstart_all_answers(tmp_path):
+def test_quickstart_all_answers(tmp_path: Path) -> None:
     answers = {
         'Root path': str(tmp_path),
         'Separate source and build': 'y',
@@ -185,7 +197,7 @@ def test_quickstart_all_answers(tmp_path):
     assert (tmp_path / 'source' / 'contents.txt').is_file()
 
 
-def test_generated_files_eol(tmp_path):
+def test_generated_files_eol(tmp_path: Path) -> None:
     answers = {
         'Root path': str(tmp_path),
         'Project name': 'Sphinx Test',
@@ -205,7 +217,7 @@ def assert_eol(filename: Path, eol: str) -> None:
     assert_eol(tmp_path / 'Makefile', '\n')
 
 
-def test_quickstart_and_build(tmp_path):
+def test_quickstart_and_build(tmp_path: Path) -> None:
     answers = {
         'Root path': str(tmp_path),
         'Project name': 'Fullwidth characters: \u30c9\u30a4\u30c4',
@@ -224,7 +236,7 @@ def test_quickstart_and_build(tmp_path):
     assert not warnings
 
 
-def test_default_filename(tmp_path):
+def test_default_filename(tmp_path: Path) -> None:
     answers = {
         'Root path': str(tmp_path),
         'Project name': '\u30c9\u30a4\u30c4',  # Fullwidth characters only
@@ -242,7 +254,7 @@ def test_default_filename(tmp_path):
     exec(conffile.read_text(encoding='utf8'), ns)  # NoQA: S102
 
 
-def test_extensions(tmp_path):
+def test_extensions(tmp_path: Path) -> None:
     qs.main([
         '-q',
         '-p',
@@ -261,7 +273,7 @@ def test_extensions(tmp_path):
     assert ns['extensions'] == ['foo', 'bar', 'baz']
 
 
-def test_exits_when_existing_confpy(monkeypatch):
+def test_exits_when_existing_confpy(monkeypatch: pytest.MonkeyPatch) -> None:
     # The code detects existing conf.py with path.is_file()
     # so we mock it as True with pytest's monkeypatch
     monkeypatch.setattr('os.path.isfile', lambda path: True)