chore: Move cli tests into correct directory#1406
Conversation
|
Not the quick win I envisioned. The fixture for reloading opentelemetry tracing is causing havok. |
00e81e9 to
4399979
Compare
|
The import reloading is breaking the
When the The same effect can be seen in a repl >>> from pathlib import Path
>>> import pathlib, importlib
>>> importlib.reload(pathlib)
<module 'pathlib' from '/home/qan22331/.local/share/uv/python/cpython-3.14.0-linux-x86_64-gnu/lib/python3.14/pathlib/__init__.py'>
>>> isinstance(pathlib.Path('/tmp'), Path)
FalseI not sure the $ uv run pytest tests/unit_tests/test_cli.py tests/unit_tests/client/test_client.pyRemoving the fixture and |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1406 +/- ##
=======================================
Coverage 95.17% 95.17%
=======================================
Files 43 43
Lines 3111 3111
=======================================
Hits 2961 2961
Misses 150 150 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
4399979 to
683e8b9
Compare
The cli tests were in the root unit test dir, when the cli now has its own module. This makes it match the src layout.