Fix CI: widen control_task_factory return to Any

Previous Protocol-based typing went the wrong direction: a Protocol
with send_task(**kwargs) was too loose for ControlTask.send_task
(which has specific positional params) AND too tight to uniformly
cover the test fakes (NoopControlTask only accepts **kwargs,
FakeControlTask takes named positional params). Mypy rejected
ControlTask itself as the default.

Drop the Protocol; type the factory return as Any. The runtime
collaborator is duck-typed — the dispatcher only invokes
send_task(**named_kwargs) — and Any is what the DI seam is actually
for.

Author: mvdbeek

lib/galaxy/managers/sse_dispatch.py

@@ -15,7 +15,6 @@
 from typing import (
     Any,
     Optional,
-    Protocol,
 )
 
 from cachetools import TTLCache
@@ -30,17 +29,6 @@
 from galaxy.web.statsd_client import VanillaGalaxyStatsdClient
 from galaxy.web_stack import ApplicationStack
 
-
-class ControlTaskLike(Protocol):
-    """Structural type for the dispatcher's control-task collaborator.
-
-    The dispatcher only calls ``send_task(**kwargs)``. Typed as a Protocol so
-    tests can pass lightweight fakes (``FakeControlTask``, ``NoopControlTask``)
-    without subclassing ``ControlTask``.
-    """
-
-    def send_task(self, **kwargs: Any) -> Any: ...
-
 log = logging.getLogger(__name__)
 
 
@@ -68,7 +56,10 @@ def __init__(
         application_stack: ApplicationStack,
         statsd_client: Optional[VanillaGalaxyStatsdClient] = None,
         clock: Callable[[], float] = time.monotonic,
-        control_task_factory: Callable[[GalaxyQueueWorker], ControlTaskLike] = ControlTask,
+        # Factory return is typed ``Any`` so ``ControlTask`` itself and test-only
+        # duck-typed doubles (FakeControlTask/BoomControlTask/NoopControlTask)
+        # all satisfy the signature under mypy.
+        control_task_factory: Callable[[GalaxyQueueWorker], Any] = ControlTask,
         queues_provider: Optional[Callable[[], list[Queue]]] = None,
     ) -> None:
         self._queue_worker = queue_worker