Remove outdated xfails

Author: dmtucker

src/pytest_mypy.py

@@ -295,8 +295,7 @@ class MypyWarning(pytest.PytestWarning):
 def pytest_terminal_summary(terminalreporter, config):
     """Report stderr and unrecognized lines from stdout."""
     if not _is_xdist_controller(config):
-        # This isn't hit in pytest 5.0 for some reason.
-        return  # pragma: no cover
+        return
     try:
         with open(config._mypy_results_path, mode="r") as results_f:
             results = MypyResults.load(results_f)

tests/test_pytest_mypy.py

@@ -4,14 +4,12 @@
 
 import mypy.version
 from packaging.version import Version
-import pexpect
 import pytest
 
 import pytest_mypy
 
 
 MYPY_VERSION = Version(mypy.version.__version__)
-PYTEST_VERSION = Version(pytest.__version__)
 PYTHON_VERSION = Version(
     ".".join(
         str(token)
@@ -325,14 +323,7 @@ def pytest_configure(config):
 @pytest.mark.parametrize(
     "module_name",
     [
-        pytest.param(
-            "__init__",
-            marks=pytest.mark.xfail(
-                Version("3.10") <= PYTEST_VERSION < Version("6.2"),
-                raises=AssertionError,
-                reason="https://github.com/pytest-dev/pytest/issues/8016",
-            ),
-        ),
+        "__init__",
         "good",
     ],
 )
@@ -464,14 +455,6 @@ def pyfunc(x: int) -> str:
         expect_timeout=60.0,
     )
 
-    num_tests = 2
-    if module_name == "__init__" and Version("3.10") <= PYTEST_VERSION < Version("6.2"):
-        # https://github.com/pytest-dev/pytest/issues/8016
-        # Pytest had a bug where it assumed only a Package would have a basename of
-        # __init__.py. In this test, Pytest mistakes MypyFile for a Package and
-        # returns after collecting only one object (the MypyFileItem).
-        num_tests = 1
-
     def _expect_session():
         child.expect("==== test session starts ====")
 
@@ -480,11 +463,9 @@ def _expect_failure():
         child.expect("==== FAILURES ====")
         child.expect(pyfile.basename + " ____")
         child.expect("2: error: Incompatible return value")
-        # if num_tests == 2:
-        #     # These only show with mypy>=0.730:
-        #     child.expect("==== mypy ====")
-        #     child.expect("Found 1 error in 1 file (checked 1 source file)")
-        child.expect(str(num_tests) + " failed")
+        child.expect("==== mypy ====")
+        child.expect("Found 1 error in 1 file (checked 1 source file)")
+        child.expect("2 failed")
         child.expect("#### LOOPONFAILING ####")
         _expect_waiting()
 
@@ -503,29 +484,9 @@ def _expect_changed():
     def _expect_success():
         for _ in range(2):
             _expect_session()
-            # if num_tests == 2:
-            #     # These only show with mypy>=0.730:
-            #     child.expect("==== mypy ====")
-            #     child.expect("Success: no issues found in 1 source file")
-            try:
-                child.expect(str(num_tests) + " passed")
-            except pexpect.exceptions.TIMEOUT:
-                if module_name == "__init__" and (
-                    Version("6.0") <= PYTEST_VERSION < Version("6.2")
-                ):
-                    # MypyItems hit the __init__.py bug too when --looponfail
-                    # re-collects them after the failing file is modified.
-                    # Unlike MypyFile, MypyItem is not a Collector, so this used
-                    # to cause an AttributeError until a workaround was added
-                    # (MypyItem.collect was defined to yield itself).
-                    # Mypy probably noticed the __init__.py problem during the
-                    # development of Pytest 6.0, but the error was addressed
-                    # with an isinstance assertion, which broke the workaround.
-                    # Here, we hit that assertion:
-                    child.expect("AssertionError")
-                    child.expect("1 error")
-                    pytest.xfail("https://github.com/pytest-dev/pytest/issues/8016")
-                raise
+            child.expect("==== mypy ====")
+            child.expect("Success: no issues found in 1 source file")
+            child.expect("2 passed")
         _expect_waiting()
 
     def _break():
@@ -550,20 +511,12 @@ def test_mypy_results_from_mypy_with_opts():
 
 def test_mypy_no_output(testdir, xdist_args):
     """No terminal summary is shown if there is no output from mypy."""
-    type_ignore = (
-        "# type: ignore"
-        if (
-            PYTEST_VERSION
-            < Version("6.0")  # Pytest didn't add type annotations until 6.0.
-        )
-        else ""
-    )
     testdir.makepyfile(
         # Mypy prints a success message to stderr by default:
         # "Success: no issues found in 1 source file"
         # Clear stderr and unmatched_stdout to simulate mypy having no output:
-        conftest=f"""
-            import pytest  {type_ignore}
+        conftest="""
+            import pytest
 
             @pytest.hookimpl(hookwrapper=True)
             def pytest_terminal_summary(config):
@@ -578,7 +531,7 @@ def pytest_terminal_summary(config):
                         stdout="",
                         stderr="",
                         status=0,
-                        abspath_errors={{}},
+                        abspath_errors={},
                         unmatched_stdout="",
                     ).dump(results_f)
                 yield

tox.ini

@@ -32,7 +32,6 @@ deps =
     mypy1.x: mypy ~= 1.0
 
     packaging ~= 21.3
-    pexpect ~= 4.8.0
     pytest-cov ~= 4.1.0
     pytest-randomly ~= 3.4
     pytest-xdist ~= 1.34