test coverage, profiling tests, and rearranging uv lock#206
Conversation
✅ Deploy Preview for mistql canceled.
|
There was a problem hiding this comment.
Pull Request Overview
This PR enhances the test infrastructure by adding code coverage reporting and improving profiling output in tests. The changes configure pytest with coverage reporting and update the profiling test to output more useful statistics while saving profile data for analysis.
- Adds pytest-cov dependency and enables coverage reporting in CI
- Updates profiling test to output top 20 functions and save profile stats
- Migrates from optional-dependencies to dependency-groups in pyproject.toml
Reviewed Changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| py/tests/test_performance.py | Improves profiling output by removing string buffer and showing top 20 stats |
| py/pyproject.toml | Adds pytest-cov dependency and migrates to dependency-groups format |
| .github/workflows/python.yml | Enables pytest coverage reporting and adds PROFILE environment variable |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| @@ -35,7 +36,7 @@ jobs: | |||
| - name: Install deps | |||
| run: uv pip install -e ".[dev]" | |||
There was a problem hiding this comment.
The installation command still references the old [dev] optional dependency format, but pyproject.toml has been updated to use dependency-groups. This should be changed to uv pip install -e . --group dev to match the new dependency-groups format.
| run: uv pip install -e ".[dev]" | |
| run: uv pip install -e . --group dev |
No description provided.