Merge pull request #7907 from bluetech/mypy-790

Update mypy 0.782 -> 0.790, iniconfig typing

Author: bluetech

.pre-commit-config.yaml

@@ -49,11 +49,13 @@ repos:
     hooks:
     -   id: python-use-type-annotations
 -   repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v0.782  # NOTE: keep this in sync with setup.cfg.
+    rev: v0.790  # NOTE: keep this in sync with setup.cfg.
     hooks:
     -   id: mypy
         files: ^(src/|testing/)
         args: []
+        additional_dependencies:
+          - iniconfig>=1.1.0
 -   repo: local
     hooks:
     -   id: rst

setup.cfg

@@ -63,7 +63,7 @@ console_scripts =
 
 [options.extras_require]
 checkqa-mypy =
-    mypy==0.780
+    mypy==0.790
 testing =
     argcomplete
     hypothesis>=3.56

src/_pytest/cacheprovider.py

@@ -382,7 +382,7 @@ def pytest_collection_modifyitems(
             self.cached_nodeids.update(item.nodeid for item in items)
 
     def _get_increasing_order(self, items: Iterable[nodes.Item]) -> List[nodes.Item]:
-        return sorted(items, key=lambda item: item.fspath.mtime(), reverse=True)
+        return sorted(items, key=lambda item: item.fspath.mtime(), reverse=True)  # type: ignore[no-any-return]
 
     def pytest_sessionfinish(self) -> None:
         config = self.config

src/_pytest/capture.py

@@ -380,8 +380,7 @@ def __init__(self, targetfd: int) -> None:
             self.syscapture = SysCapture(targetfd)
         else:
             self.tmpfile = EncodedFile(
-                # TODO: Remove type ignore, fixed in next mypy release.
-                TemporaryFile(buffering=0),  # type: ignore[arg-type]
+                TemporaryFile(buffering=0),
                 encoding="utf-8",
                 errors="replace",
                 newline="",

src/_pytest/compat.py

@@ -150,9 +150,8 @@ def getfuncargnames(
         p.name
         for p in parameters.values()
         if (
-            # TODO: Remove type ignore after https://github.com/python/typeshed/pull/4383
-            p.kind is Parameter.POSITIONAL_OR_KEYWORD  # type: ignore[unreachable]
-            or p.kind is Parameter.KEYWORD_ONLY  # type: ignore[unreachable]
+            p.kind is Parameter.POSITIONAL_OR_KEYWORD
+            or p.kind is Parameter.KEYWORD_ONLY
         )
         and p.default is Parameter.empty
     )

src/_pytest/config/findpaths.py

@@ -27,7 +27,7 @@ def _parse_ini_config(path: Path) -> iniconfig.IniConfig:
     Raise UsageError if the file cannot be parsed.
     """
     try:
-        return iniconfig.IniConfig(path)
+        return iniconfig.IniConfig(str(path))
     except iniconfig.ParseError as exc:
         raise UsageError(str(exc)) from exc
 

src/_pytest/pytester.py

@@ -32,6 +32,7 @@
 import attr
 import py
 from iniconfig import IniConfig
+from iniconfig import SectionWrapper
 
 import pytest
 from _pytest import timing
@@ -785,10 +786,10 @@ def makeini(self, source: str) -> Path:
         """Write a tox.ini file with 'source' as contents."""
         return self.makefile(".ini", tox=source)
 
-    def getinicfg(self, source: str) -> IniConfig:
+    def getinicfg(self, source: str) -> SectionWrapper:
         """Return the pytest section from the tox.ini config file."""
         p = self.makeini(source)
-        return IniConfig(p)["pytest"]
+        return IniConfig(str(p))["pytest"]
 
     def makepyprojecttoml(self, source: str) -> Path:
         """Write a pyproject.toml file with 'source' as contents.
@@ -1321,8 +1322,10 @@ def run(
         """
         __tracebackhide__ = True
 
+        # TODO: Remove type ignore in next mypy release.
+        #       https://github.com/python/typeshed/pull/4582
         cmdargs = tuple(
-            os.fspath(arg) if isinstance(arg, os.PathLike) else arg for arg in cmdargs
+            os.fspath(arg) if isinstance(arg, os.PathLike) else arg for arg in cmdargs  # type: ignore[misc]
         )
         p1 = self.path.joinpath("stdout")
         p2 = self.path.joinpath("stderr")
@@ -1541,9 +1544,9 @@ def makeini(self, source) -> py.path.local:
         """See :meth:`Pytester.makeini`."""
         return py.path.local(str(self._pytester.makeini(source)))
 
-    def getinicfg(self, source) -> py.path.local:
+    def getinicfg(self, source: str) -> SectionWrapper:
         """See :meth:`Pytester.getinicfg`."""
-        return py.path.local(str(self._pytester.getinicfg(source)))
+        return self._pytester.getinicfg(source)
 
     def makepyprojecttoml(self, source) -> py.path.local:
         """See :meth:`Pytester.makepyprojecttoml`."""

src/_pytest/python_api.py

@@ -210,7 +210,7 @@ def __repr__(self) -> str:
         # tolerances, i.e. non-numerics and infinities. Need to call abs to
         # handle complex numbers, e.g. (inf + 1j).
         if (not isinstance(self.expected, (Complex, Decimal))) or math.isinf(
-            abs(self.expected)
+            abs(self.expected)  # type: ignore[arg-type]
         ):
             return str(self.expected)
 
@@ -253,16 +253,16 @@ def __eq__(self, actual) -> bool:
         # Allow the user to control whether NaNs are considered equal to each
         # other or not.  The abs() calls are for compatibility with complex
         # numbers.
-        if math.isnan(abs(self.expected)):
-            return self.nan_ok and math.isnan(abs(actual))
+        if math.isnan(abs(self.expected)):  # type: ignore[arg-type]
+            return self.nan_ok and math.isnan(abs(actual))  # type: ignore[arg-type]
 
         # Infinity shouldn't be approximately equal to anything but itself, but
         # if there's a relative tolerance, it will be infinite and infinity
         # will seem approximately equal to everything.  The equal-to-itself
         # case would have been short circuited above, so here we can just
         # return false if the expected value is infinite.  The abs() call is
         # for compatibility with complex numbers.
-        if math.isinf(abs(self.expected)):
+        if math.isinf(abs(self.expected)):  # type: ignore[arg-type]
             return False
 
         # Return true if the two numbers are within the tolerance.

src/_pytest/runner.py

@@ -77,8 +77,7 @@ def pytest_terminal_summary(terminalreporter: "TerminalReporter") -> None:
                 dlist.append(rep)
     if not dlist:
         return
-    dlist.sort(key=lambda x: x.duration)
-    dlist.reverse()
+    dlist.sort(key=lambda x: x.duration, reverse=True)  # type: ignore[no-any-return]
     if not durations:
         tr.write_sep("=", "slowest durations")
     else:

testing/test_assertrewrite.py

@@ -1595,7 +1595,7 @@ def test_get_cache_dir(self, monkeypatch, prefix, source, expected):
         if prefix:
             if sys.version_info < (3, 8):
                 pytest.skip("pycache_prefix not available in py<38")
-            monkeypatch.setattr(sys, "pycache_prefix", prefix)  # type:ignore
+            monkeypatch.setattr(sys, "pycache_prefix", prefix)
 
         assert get_cache_dir(Path(source)) == Path(expected)