Enable coverage reporting via Codecov and cleanup nits remaining changes

Author: namidanam

.github/workflows/ci.yml

@@ -9,7 +9,7 @@ on:
 jobs:
   test:
     runs-on: ubuntu-latest
-    services:              
+    services:
       redis:
         image: redis:7
         ports:

api-server/pyproject.toml

@@ -15,11 +15,13 @@ dependencies = [
     "python-dotenv>=1.1.1",
     "structlog>=25.4.0",
     "uvicorn>=0.35.0",
-    "redis[hiredis]>=6.0.0"]
+    "redis[hiredis]>=6.0.0"
+    ]
 
 [dependency-groups]
 dev = [
     "ruff>=0.12.2",
     "pytest>=8.0.0",
     "httpx>=0.27.0",
-     "pytest-cov>=5.0.0"]
+     "pytest-cov>=5.0.0"
+     ]

api-server/tests/test_unhandled_exceptions_middleware.py

@@ -4,21 +4,17 @@
 from fastapi.testclient import TestClient
 from app.middlewares.unhandled_exceptions_middleware import UnhandledExceptionsMiddleware
 
-# Handlers for various exception types
+# Exception handlers to trigger different exceptions
 
-# 1. RuntimeError
 async def fail(request: Request):
     raise RuntimeError("boom")
 
-# 2. ValueError
 async def fail_value_error(request: Request):
     raise ValueError("Invalid value test")
 
-# 3. KeyError
 async def fail_key_error(request: Request):
     raise KeyError("Missing key test")
 
-# 4. Healthy endpoint
 async def ok(request: Request):
     return JSONResponse({"ok": True})
 
@@ -32,29 +28,25 @@ def client():
     app.add_api_route("/ok", ok, methods=["GET"])
     return TestClient(app)
 
+ERROR_RESPONSE = {
+    "success": False,
+    "detail": "internal server error, please reach out to support team at nivedit@exosphere.host"
+}
+
 def test_runtime_error_returns_expected_json(client):
     resp = client.get("/fail")
     assert resp.status_code == 500
-    response_json = resp.json()
-    assert response_json.get("success") is False
-    assert "detail" in response_json
-    assert "server error" in response_json["detail"].lower()
+    assert resp.json() == ERROR_RESPONSE
 
 def test_value_error_returns_expected_json(client):
     resp = client.get("/fail_value_error")
     assert resp.status_code == 500
-    response_json = resp.json()
-    assert response_json.get("success") is False
-    assert "detail" in response_json
-    assert "server error" in response_json["detail"].lower()
+    assert resp.json() == ERROR_RESPONSE
 
 def test_key_error_returns_expected_json(client):
     resp = client.get("/fail_key_error")
     assert resp.status_code == 500
-    response_json = resp.json()
-    assert response_json.get("success") is False
-    assert "detail" in response_json
-    assert "server error" in response_json["detail"].lower()
+    assert resp.json() == ERROR_RESPONSE
 
 def test_normal_request_passes_through(client):
     resp = client.get("/ok")