Fix test state leaks: reset caches between tests, unique delivery IDs

Add autouse fixture to clear processed_deliveries/jobs/concurrent_jobs
between tests. Use incrementing delivery IDs to prevent dedup hits.

Author: manascb1344

tests/conftest.py

@@ -29,3 +29,12 @@ def _mock_modal_decorators():
 def set_env(monkeypatch):
     monkeypatch.setenv("WEBHOOK_SECRET", "test-secret")
     monkeypatch.setenv("GITHUB_TOKEN", "ghp_test_token")
+
+
+@pytest.fixture(autouse=True)
+def reset_caches():
+    import app
+
+    app._processed_deliveries.clear()
+    app._processed_jobs.clear()
+    app._concurrent_jobs.clear()

tests/test_app.py

@@ -6,12 +6,22 @@
 import httpx
 import pytest
 
+_delivery_counter = 0
+
+
+def _next_delivery_id():
+    global _delivery_counter
+    _delivery_counter += 1
+    return f"test-delivery-{_delivery_counter}"
+
 
 def _sign(body: bytes, secret: str = "test-secret") -> str:
     return "sha256=" + hmac.new(secret.encode(), body, digestmod=hashlib.sha256).hexdigest()
 
 
-def _make_request(body: bytes, secret: str = "test-secret", delivery_id: str = "test-delivery-123"):
+def _make_request(body: bytes, secret: str = "test-secret", delivery_id: str | None = None):
+    if delivery_id is None:
+        delivery_id = _next_delivery_id()
     request = MagicMock()
     request.headers = {
         "Content-Type": "application/json",