Merge pull request #40 from igoose1/39-explicit-url

taskiq-python · Sep 1, 2023 · c02b4f5 · c02b4f5
2 parents 419d620 + 43da3cb
commit c02b4f5
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/taskiq_redis/redis_broker.py b/taskiq_redis/redis_broker.py
@@ -16,7 +16,7 @@ class BaseRedisBroker(AsyncBroker):
 
     def __init__(
         self,
-        url: Optional[str] = None,
+        url: str,
         task_id_generator: Optional[Callable[[], str]] = None,
         result_backend: Optional[AsyncResultBackend[_T]] = None,
         queue_name: str = "taskiq",

diff --git a/tests/test_broker.py b/tests/test_broker.py
@@ -8,6 +8,12 @@
 from taskiq_redis import ListQueueBroker, PubSubBroker
 
 
+def test_no_url_should_raise_typeerror() -> None:
+    """Test that url is expected."""
+    with pytest.raises(TypeError):
+        ListQueueBroker()  # type: ignore
+
+
 async def get_message(  # type: ignore
     broker: AsyncBroker,
 ) -> Union[bytes, AckableMessage]: