Added tests for broker ctx manager.

Author: s3rius

taskiq/abc/broker.py

@@ -535,7 +535,7 @@ def _register_task(
         self.local_task_registry[task_name] = task
 
     async def __aenter__(self) -> None:
-        """Satarts the broker as ctx manager."""
+        """Starts the broker as ctx manager."""
         await self.startup()
 
     async def __aexit__(self, *args: object, **kwargs: Any) -> None:

tests/abc/test_broker.py

@@ -1,9 +1,13 @@
 from collections.abc import AsyncGenerator
 from copy import copy
 
+import pytest
+
 from taskiq.abc.broker import AsyncBroker
 from taskiq.decor import AsyncTaskiqDecoratedTask
+from taskiq.events import TaskiqEvents
 from taskiq.message import BrokerMessage
+from taskiq.state import TaskiqState
 
 
 class _TestBroker(AsyncBroker):
@@ -76,3 +80,106 @@ async def test_task() -> None: ...
     assert "another_label" in test_kicker.labels
 
     assert test_task.labels == old_labels
+
+
+@pytest.mark.anyio
+async def test_async_context_manager_enter() -> None:
+    """Test that __aenter__ calls startup."""
+    broker = _TestBroker()
+    startup_called = False
+
+    @broker.on_event(TaskiqEvents.CLIENT_STARTUP)
+    async def track_startup(state: TaskiqState) -> None:
+        nonlocal startup_called
+        startup_called = True
+
+    async with broker:
+        assert startup_called is True
+
+
+@pytest.mark.anyio
+async def test_async_context_manager_exit() -> None:
+    """Test that __aexit__ calls shutdown."""
+    broker = _TestBroker()
+    shutdown_called = False
+
+    @broker.on_event(TaskiqEvents.CLIENT_SHUTDOWN)
+    async def track_shutdown(state: TaskiqState) -> None:
+        nonlocal shutdown_called
+        shutdown_called = True
+
+    async with broker:
+        pass
+
+    assert shutdown_called is True
+
+
+@pytest.mark.anyio
+async def test_async_context_manager_enter_worker() -> None:
+    """Test that __aenter__ calls worker startup when is_worker_process is True."""
+    broker = _TestBroker()
+    broker.is_worker_process = True
+    startup_called = False
+
+    @broker.on_event(TaskiqEvents.WORKER_STARTUP)
+    async def track_startup(state: TaskiqState) -> None:
+        nonlocal startup_called
+        startup_called = True
+
+    async with broker:
+        assert startup_called is True
+
+
+@pytest.mark.anyio
+async def test_async_context_manager_exit_worker() -> None:
+    """Test that __aexit__ calls worker shutdown when is_worker_process is True."""
+    broker = _TestBroker()
+    broker.is_worker_process = True
+    shutdown_called = False
+
+    @broker.on_event(TaskiqEvents.WORKER_SHUTDOWN)
+    async def track_shutdown(state: TaskiqState) -> None:
+        nonlocal shutdown_called
+        shutdown_called = True
+
+    async with broker:
+        pass
+
+    assert shutdown_called is True
+
+
+@pytest.mark.anyio
+async def test_async_context_manager_exit_on_exception() -> None:
+    """Test that __aexit__ calls shutdown even if exception is raised."""
+    broker = _TestBroker()
+    shutdown_called = False
+
+    @broker.on_event(TaskiqEvents.CLIENT_SHUTDOWN)
+    async def track_shutdown(state: TaskiqState) -> None:
+        nonlocal shutdown_called
+        shutdown_called = True
+
+    with pytest.raises(ValueError, match="Test exception"):
+        async with broker:
+            raise ValueError("Test exception")
+
+    assert shutdown_called is True
+
+
+@pytest.mark.anyio
+async def test_async_context_manager_exit_worker_on_exception() -> None:
+    """Test that __aexit__ calls worker shutdown even if exception is raised."""
+    broker = _TestBroker()
+    broker.is_worker_process = True
+    shutdown_called = False
+
+    @broker.on_event(TaskiqEvents.WORKER_SHUTDOWN)
+    async def track_shutdown(state: TaskiqState) -> None:
+        nonlocal shutdown_called
+        shutdown_called = True
+
+    with pytest.raises(ValueError, match="Test exception"):
+        async with broker:
+            raise ValueError("Test exception")
+
+    assert shutdown_called is True