Update CLI test suite to use subprocesses

.coveragerc-memory

@@ -0,0 +1,10 @@
+# Coverage configuration for memory backend testing
+# CLI tests are skipped with memory:// URLs, so exclude CLI from coverage
+
+[run]
+branch = true
+parallel = true
+omit =
+    src/docket/__main__.py
+    src/docket/cli.py
+    tests/cli/test_*.py

.github/workflows/ci.yml

@@ -31,21 +31,29 @@ jobs:
           # Python 3.10 hangs when we include the CLI tests, so ignore it and
           # use a lower threshold to compensate
           - python-version: "3.10"
-            cov-threshold: 75
-            pytest-ignore: "--ignore tests/cli"
+            cov-threshold: 100
+            pytest-args: ""
           # Python 3.11 coverage reporting is unstable, so use 98% threshold
           - python-version: "3.11"
             cov-threshold: 98
-            pytest-ignore: ""
+            pytest-args: ""
           - python-version: "3.12"
             cov-threshold: 100
-            pytest-ignore: ""
+            pytest-args: ""
           - python-version: "3.13"
             cov-threshold: 100
-            pytest-ignore: ""
+            pytest-args: ""
           - python-version: "3.14"
             cov-threshold: 100
-            pytest-ignore: ""
+            pytest-args: ""
+          # Memory backend: CLI tests are skipped via pytest skip markers because
+          # CLI rejects memory:// URLs. Use separate coverage config to exclude CLI.
+          - backend:
+              name: "Memory (in-memory backend)"
+              redis-version: "memory"
+              redis-py-version: ">=5"
+            cov-threshold: 98 # CLI tests are excluded from coverage and some lines are only covered by CLI tests
+            pytest-args: "--cov-config=.coveragerc-memory"
 
     steps:
       - uses: actions/checkout@v4
@@ -63,7 +71,7 @@ jobs:
       - name: Run tests
         env:
           REDIS_VERSION: ${{ matrix.backend.redis-version }}
-        run: uv run pytest --cov-branch --cov-fail-under=${{ matrix.cov-threshold }} --cov-report=xml --cov-report=term-missing:skip-covered ${{ matrix.pytest-ignore }}
+        run: uv run pytest --cov-branch --cov-fail-under=${{ matrix.cov-threshold }} --cov-report=xml --cov-report=term-missing:skip-covered ${{ matrix.pytest-args }}
 
       - name: Upload coverage reports to Codecov
         uses: codecov/codecov-action@v5

pyproject.toml

@@ -67,11 +67,7 @@ docs = [
     "mkdocstrings>=0.24.1",
     "mkdocstrings-python>=1.8.0",
 ]
-examples = [
-    "fastapi>=0.120.0",
-    "pydantic>=2.11.10",
-    "uvicorn>=0.38.0",
-]
+examples = ["fastapi>=0.120.0", "pydantic>=2.11.10", "uvicorn>=0.38.0"]
 
 [project.scripts]
 docket = "docket.__main__:app"
@@ -109,6 +105,8 @@ filterwarnings = ["error"]
 
 [tool.coverage.run]
 omit = ["src/docket/__main__.py"]
+branch = true
+parallel = true
 
 [tool.pyright]
 include = ["src", "tests"]

sitecustomize.py

@@ -0,0 +1,7 @@
+# This file ensures that we can collect coverage data for the CLI when it's running in a subprocess
+import os
+
+if os.getenv("COVERAGE_PROCESS_START"):
+    import coverage
+
+    coverage.process_startup()

src/docket/cli.py

@@ -114,6 +114,22 @@ def set_logging_level(level: LogLevel) -> None:
     logging.getLogger().setLevel(level.value)
 
 
+def validate_url(url: str) -> str:
+    """
+    Validate that the provided URL is compatible with the CLI.
+
+    The memory:// backend is not compatible with the CLI as it doesn't persist
+    across processes.
+    """
+    if url.startswith("memory://"):
+        raise typer.BadParameter(
+            "The memory:// URL scheme is not supported by the CLI.\n"
+            "The memory backend does not persist across processes.\n"
+            "Please use a persistent backend like Redis or Valkey."
+        )
+    return url
+
+
 def handle_strike_wildcard(value: str) -> str | None:
     if value in ("", "*"):
         return None
@@ -178,6 +194,7 @@ def worker(
         typer.Option(
             help="The URL of the Redis server",
             envvar="DOCKET_URL",
+            callback=validate_url,
         ),
     ] = "redis://localhost:6379/0",
     name: Annotated[
@@ -336,6 +353,7 @@ def strike(
         typer.Option(
             help="The URL of the Redis server",
             envvar="DOCKET_URL",
+            callback=validate_url,
         ),
     ] = "redis://localhost:6379/0",
 ) -> None:
@@ -373,6 +391,7 @@ def clear(
         typer.Option(
             help="The URL of the Redis server",
             envvar="DOCKET_URL",
+            callback=validate_url,
         ),
     ] = "redis://localhost:6379/0",
 ) -> None:
@@ -425,6 +444,7 @@ def restore(
         typer.Option(
             help="The URL of the Redis server",
             envvar="DOCKET_URL",
+            callback=validate_url,
         ),
     ] = "redis://localhost:6379/0",
 ) -> None:
@@ -468,6 +488,7 @@ def trace(
         typer.Option(
             help="The URL of the Redis server",
             envvar="DOCKET_URL",
+            callback=validate_url,
         ),
     ] = "redis://localhost:6379/0",
     message: Annotated[
@@ -511,6 +532,7 @@ def fail(
         typer.Option(
             help="The URL of the Redis server",
             envvar="DOCKET_URL",
+            callback=validate_url,
         ),
     ] = "redis://localhost:6379/0",
     message: Annotated[
@@ -554,6 +576,7 @@ def sleep(
         typer.Option(
             help="The URL of the Redis server",
             envvar="DOCKET_URL",
+            callback=validate_url,
         ),
     ] = "redis://localhost:6379/0",
     seconds: Annotated[
@@ -688,6 +711,7 @@ def snapshot(
         typer.Option(
             help="The URL of the Redis server",
             envvar="DOCKET_URL",
+            callback=validate_url,
         ),
     ] = "redis://localhost:6379/0",
     stats: Annotated[
@@ -840,6 +864,7 @@ def list_workers(
         typer.Option(
             help="The URL of the Redis server",
             envvar="DOCKET_URL",
+            callback=validate_url,
         ),
     ] = "redis://localhost:6379/0",
 ) -> None:
@@ -876,6 +901,7 @@ def workers_for_task(
         typer.Option(
             help="The URL of the Redis server",
             envvar="DOCKET_URL",
+            callback=validate_url,
         ),
     ] = "redis://localhost:6379/0",
 ) -> None: