Repository navigation
test: cover request discovery, exception text and fastapi/quart extractors - #145
Conversation
…ctors Add tests for call-stack request discovery, cached exception text, config_root_logger, and the FastAPI and Quart request info extractors. Refs #144 Signed-off-by: Srinivas Chippagiri <cvas22@gmail.com>
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 30 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing |
![llamapreview[bot]](https://avatars.githubusercontent.com/in/1023541?s=60&v=4){kind=small}
There was a problem hiding this comment.
LlamaPReview — Review skipped
This repository has used its free review capacity for today (3 reviews per UTC day). Capacity resets at 2026-10-25T00:00:00Z.
LlamaPReview is free for public repositories and funded personally, so a daily bound keeps one high-volume repository from consuming the shared budget. Nothing about this pull request was judged. You can also run LlamaPReview yourself with your own provider key — see Self-hosting.
No model-driven code review was run; this result does not assess whether the PR is ready to merge.
Closes #144. Also relates to #137.
Adds
tests/test_request_discovery.py,tests/test_fastapi_quart_extractors.pyand oneconfig_root_loggertest intests/test_init_validation.py.Coverage (local, Python 3.14.6, Windows): 92% -> 94%;
util.py96% -> 99%,fastapi/implementation.py94% -> 98%,quart/__init__.py88% -> 96%,formatters.py98% -> 99%.Tested: unit suite 157 passed, 1 skipped (gunicorn is not installable on Windows); strict flake8 selection is clean. Smoke tests for all four backends also pass locally (they need the package importable). Not run on other Python versions.
Not addressed: the Sanic test client's
Task was destroyed but it is pendinglog at teardown, which does not fail a test; it is listed in the issue for follow-up.