Skip to content

Commit 9a5979f

Browse files
committed
Remove unnecessary tests/conftest.py
After further investigation, conftest.py is not needed because: 1. When tox installs the package (even with usedevelop=false), it runs 'pip install .' which installs ALL packages found by find_packages('src') including both 'datapilot' and 'vendor' packages. 2. The existing tests (tests/core/**) import from 'datapilot' and work fine without any conftest.py manipulating sys.path. 3. The actual fix for our import issues was renaming tests/vendor/ to tests/test_vendor/ to avoid the directory naming conflict with src/vendor/, not adding conftest.py. When the package is properly installed, both 'datapilot' and 'vendor' are importable without needing to manually add src/ to sys.path.
1 parent 27240b4 commit 9a5979f

File tree

1 file changed

+0
-33
lines changed

1 file changed

+0
-33
lines changed

tests/conftest.py

Lines changed: 0 additions & 33 deletions
This file was deleted.

0 commit comments

Comments
 (0)