Drop old Mypy ignores

.mypy.ini

@@ -26,12 +26,6 @@ warn_return_any = True
 #warn_unreachable = True
 warn_unused_ignores = True
 
-[mypy-aiodns]
-ignore_missing_imports = True
-
-[mypy-asynctest]
-ignore_missing_imports = True
-
 [mypy-brotli]
 ignore_missing_imports = True
 
@@ -40,6 +34,3 @@ ignore_missing_imports = True
 
 [mypy-gunicorn.*]
 ignore_missing_imports = True
-
-[mypy-python_on_whales]
-ignore_missing_imports = True

aiohttp/abc.py

@@ -146,7 +146,7 @@ class AbstractResolver(ABC):
 
     @abstractmethod
     async def resolve(
-        self, host: str, port: int = 0, family: int = socket.AF_INET
+        self, host: str, port: int = 0, family: socket.AddressFamily = socket.AF_INET
     ) -> List[ResolveResult]:
         """Return IP address for given hostname"""
 

aiohttp/connector.py

@@ -745,7 +745,7 @@ def __init__(
         *,
         use_dns_cache: bool = True,
         ttl_dns_cache: Optional[int] = 10,
-        family: int = 0,
+        family: socket.AddressFamily = socket.AddressFamily.AF_UNSPEC,
         ssl: Union[bool, Fingerprint, SSLContext] = True,
         local_addr: Optional[Tuple[str, int]] = None,
         resolver: Optional[AbstractResolver] = None,

aiohttp/resolver.py

@@ -12,7 +12,7 @@
 
     # aiodns_default = hasattr(aiodns.DNSResolver, 'getaddrinfo')
 except ImportError:  # pragma: no cover
-    aiodns = None
+    aiodns = None  # type: ignore[assignment]
 
 
 aiodns_default = False
@@ -32,7 +32,7 @@ def __init__(self) -> None:
         self._loop = asyncio.get_running_loop()
 
     async def resolve(
-        self, host: str, port: int = 0, family: int = socket.AF_INET
+        self, host: str, port: int = 0, family: socket.AddressFamily = socket.AF_INET
     ) -> List[ResolveResult]:
         infos = await self._loop.getaddrinfo(
             host,
@@ -86,11 +86,10 @@ def __init__(self, *args: Any, **kwargs: Any) -> None:
         if aiodns is None:
             raise RuntimeError("Resolver requires aiodns library")
 
-        self._loop = asyncio.get_running_loop()
-        self._resolver = aiodns.DNSResolver(*args, loop=self._loop, **kwargs)
+        self._resolver = aiodns.DNSResolver(*args, **kwargs)
 
     async def resolve(
-        self, host: str, port: int = 0, family: int = socket.AF_INET
+        self, host: str, port: int = 0, family: socket.AddressFamily = socket.AF_INET
     ) -> List[ResolveResult]:
         try:
             resp = await self._resolver.getaddrinfo(

examples/fake_server.py

@@ -3,7 +3,7 @@
 import pathlib
 import socket
 import ssl
-from typing import Dict, List, Union
+from typing import Dict, List
 
 from aiohttp import ClientSession, TCPConnector, test_utils, web
 from aiohttp.abc import AbstractResolver, ResolveResult
@@ -22,7 +22,7 @@ async def resolve(
         self,
         host: str,
         port: int = 0,
-        family: Union[socket.AddressFamily, int] = socket.AF_INET,
+        family: socket.AddressFamily = socket.AF_INET,
     ) -> List[ResolveResult]:
         fake_port = self._fakes.get(host)
         if fake_port is not None:

requirements/lint.in

@@ -1,6 +1,8 @@
+aiodns
 aioredis
 mypy; implementation_name == "cpython"
 pre-commit
 pytest
+python-on-whales
 slotscheck
 uvloop; platform_system != "Windows"

tests/test_resolver.py

@@ -20,7 +20,7 @@
 
     getaddrinfo: Any = hasattr(aiodns.DNSResolver, "getaddrinfo")
 except ImportError:
-    aiodns = None
+    aiodns = None  # type: ignore[assignment]
     getaddrinfo = False
 
 
@@ -289,13 +289,6 @@ async def test_default_loop_for_threaded_resolver(loop: Any) -> None:
     assert resolver._loop is loop
 
 
-@pytest.mark.skipif(aiodns is None, reason="aiodns required")
-async def test_default_loop_for_async_resolver(loop: Any) -> None:
-    asyncio.set_event_loop(loop)
-    resolver = AsyncResolver()
-    assert resolver._loop is loop
-
-
 @pytest.mark.skipif(not getaddrinfo, reason="aiodns >=3.2.0 required")
 async def test_async_resolver_ipv6_positive_lookup(loop: Any) -> None:
     with patch("aiodns.DNSResolver") as mock: