Skip to content

Commit 1f82ba9

Browse files
committed
Add unit test for request.summary
1 parent d1fc482 commit 1f82ba9

File tree

2 files changed

+13
-0
lines changed

2 files changed

+13
-0
lines changed

src/app/api.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import logging
2+
import logging.config
23
import sys
34
import time
45
from datetime import datetime
@@ -47,6 +48,7 @@
4748

4849
summary_logger = logging.getLogger("request.summary")
4950

51+
5052
@app.get("/", include_in_schema=False)
5153
def root(request: Request):
5254
"""GET via root redirects to /docs.

tests/unit/app/test_api.py

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,3 +6,14 @@ def test_read_root(anon_client):
66
prev_resp = resp.history[0]
77
assert prev_resp.status_code == 307 # Temporary Redirect
88
assert prev_resp.headers["location"] == "./docs"
9+
10+
11+
def test_request_summary_is_logged(anon_client, caplog):
12+
anon_client.get("/__lbheartbeat__")
13+
14+
summary = caplog.records[-1]
15+
16+
assert summary.name == "request.summary"
17+
assert summary.method == "GET"
18+
assert summary.path == "/__lbheartbeat__"
19+
assert summary.querystring == {}

0 commit comments

Comments
 (0)