Apply suggestions from code review

Co-authored-by: Sam Bull <[email protected]>

Author: Dreamsorcerer

tests/conftest.py

@@ -37,7 +37,7 @@
 from aiohttp.compression_utils import ZLibBackend, ZLibBackendProtocol, set_zlib_backend
 from aiohttp.helpers import TimerNoop
 from aiohttp.http import WS_KEY, HttpVersion11
-from aiohttp.test_utils import REUSE_ADDRESS, get_unused_port_socket, loop_context
+from aiohttp.test_utils import REUSE_ADDRESS, loop_context
 
 
 def pytest_configure(config: pytest.Config) -> None:

tests/test_client_request.py

@@ -2120,7 +2120,6 @@ async def test_content_length_with_async_iterable(
     make_client_request: _RequestMaker,
 ) -> None:
     """Test that async iterables use chunked encoding, not Content-Length."""
-
     loop = asyncio.get_running_loop()
 
     async def data_gen() -> AsyncIterator[bytes]:

tests/test_client_ws_functional.py

@@ -1377,7 +1377,6 @@ async def handler(request: web.Request) -> NoReturn:
 
 async def test_websocket_connection_cancellation(aiohttp_client: AiohttpClient) -> None:
     """Test canceling the WebSocket connection task does not raise an exception in __del__."""
-
     loop = asyncio.get_running_loop()
 
     async def handler(request: web.Request) -> NoReturn:

tests/test_connector.py

@@ -2835,7 +2835,6 @@ async def test_multiple_dns_resolution_requests_success(
     make_client_request: _RequestMaker,
 ) -> None:
     """Verify that multiple DNS resolution requests are handled correctly."""
-
     loop = asyncio.get_running_loop()
 
     async def delay_resolve(*args: object, **kwargs: object) -> list[ResolveResult]:
@@ -2899,7 +2898,6 @@ async def test_multiple_dns_resolution_requests_failure(
     make_client_request: _RequestMaker,
 ) -> None:
     """Verify that DNS resolution failure for multiple requests is handled correctly."""
-
     loop = asyncio.get_running_loop()
 
     async def delay_resolve(*args: object, **kwargs: object) -> list[ResolveResult]:
@@ -2954,7 +2952,6 @@ async def test_multiple_dns_resolution_requests_cancelled(
     make_client_request: _RequestMaker,
 ) -> None:
     """Verify that DNS resolution cancellation does not affect other tasks."""
-
     loop = asyncio.get_running_loop()
 
     async def delay_resolve(*args: object, **kwargs: object) -> list[ResolveResult]:
@@ -3008,7 +3005,6 @@ async def test_multiple_dns_resolution_requests_first_cancelled(
     make_client_request: _RequestMaker,
 ) -> None:
     """Verify that first DNS resolution cancellation does not make other resolutions fail."""
-
     loop = asyncio.get_running_loop()
 
     async def delay_resolve(*args: object, **kwargs: object) -> list[ResolveResult]:

tests/test_web_middleware.py

@@ -1,4 +1,3 @@
-import asyncio
 from collections.abc import Awaitable, Callable, Iterable
 from typing import NoReturn