fix: platform-specific coverage config

Author: CNSeniorious000

pyproject.toml

@@ -161,7 +161,9 @@ exclude_lines = [
     "uvicorn/supervisors/multiprocess.py",
     "tests/supervisors/test_multiprocess.py",
 ]
-"sys_platform != 'win32'" = ["uvicorn/loops/asyncio.py"]
+"sys_platform != 'win32'" = [
+    "uvicorn/loops/winloop.py",
+]
 
 [tool.coverage.coverage_conditional_plugin.rules]
 py-win32 = "sys_platform == 'win32'"

tests/test_auto_detection.py

@@ -11,17 +11,14 @@
 from uvicorn.protocols.websockets.auto import AutoWebSocketsProtocol
 from uvicorn.server import ServerState
 
-# Determine expected loop implementation based on platform
 try:
-    if sys.platform == "win32":
-        # On Windows, check for winloop first, then asyncio
+    if sys.platform == "win32":  # pragma: py-not-win32
         importlib.import_module("winloop")
         expected_loop = "winloop"
-    else:
-        # On Unix, check for uvloop first, then asyncio
+    else:  # pragma: py-win32
         importlib.import_module("uvloop")
-        expected_loop = "uvloop"  # pragma: py-win32
-except ImportError:  # pragma: py-not-win32
+        expected_loop = "uvloop"
+except ImportError:  # pragma: no cover
     expected_loop = "asyncio"
 
 try:

tests/test_config.py

@@ -593,17 +593,3 @@ def test_setup_event_loop_is_removed(caplog: pytest.LogCaptureFixture) -> None:
         AttributeError, match="The `setup_event_loop` method was replaced by `get_loop_factory` in uvicorn 0.36.0."
     ):
         config.setup_event_loop()
-
-
-@pytest.mark.skipif(sys.platform != "win32", reason="winloop is for Windows only")
-def test_winloop_loop_factory() -> None:
-    """Test that winloop can be explicitly configured on Windows."""
-    pytest.importorskip("winloop")
-    config = Config(app=asgi_app, loop="winloop")
-    config.load()
-    loop_factory = config.get_loop_factory()
-    assert loop_factory is not None
-    event_loop = loop_factory()
-    with closing(event_loop):
-        assert event_loop is not None
-        assert type(event_loop).__module__.startswith("winloop")

uvicorn/loops/asyncio.py

@@ -6,6 +6,6 @@
 
 
 def asyncio_loop_factory(use_subprocess: bool = False) -> Callable[[], asyncio.AbstractEventLoop]:
-    if sys.platform == "win32" and not use_subprocess:
+    if sys.platform == "win32" and not use_subprocess:  # pragma: py-not-win32
         return asyncio.ProactorEventLoop
-    return asyncio.SelectorEventLoop
+    return asyncio.SelectorEventLoop  # pragma: py-win32

uvicorn/loops/auto.py

@@ -7,9 +7,9 @@
 
 def auto_loop_factory(use_subprocess: bool = False) -> Callable[[], asyncio.AbstractEventLoop]:
     try:
-        if sys.platform == "win32":
+        if sys.platform == "win32":  # pragma: py-not-win32
             from uvicorn.loops.winloop import winloop_loop_factory as loop_factory
-        else:
+        else:  # pragma: py-win32
             from uvicorn.loops.uvloop import uvloop_loop_factory as loop_factory
     except ImportError:  # pragma: no cover
         from uvicorn.loops.asyncio import asyncio_loop_factory as loop_factory