Switch to src layout, move tests to root (#151)

* Switch to src layout, move tests to root

* Add tests to MANIFEST.in

* chore: Configure coverage

* Only track coverage for lazy-loader, not tests

* Review MANIFEST.in

* Try setting coverage to package name

---------

Co-authored-by: Stefan van der Walt <stefanv@berkeley.edu>

Author: effigies

.github/workflows/coverage.yml

@@ -31,7 +31,7 @@ jobs:
         run: |
           python -m pytest --cov=lazy_loader --durations=10
           # Tests fail if using `--doctest-modules`. I.e.,
-          # python -m pytest --cov=lazy_loader --doctest-modules --durations=20
+          # python -m pytest --cov=src --durations=10 --doctest-modules
 
       - name: Upload coverage to Codecov
         uses: codecov/codecov-action@v5

MANIFEST.in

@@ -1 +1,9 @@
 include CHANGELOG.md
+include .spin/cmds.py
+recursive-include tests *
+
+global-exclude *~
+global-exclude *.pyc
+
+prune */.pytest_cache
+prune */__pycache__

pyproject.toml

@@ -25,7 +25,7 @@ dependencies = [
 ]
 
 [project.optional-dependencies]
-test = ["pytest >= 8.0", "pytest-cov >= 5.0"]
+test = ["pytest >= 8.0", "pytest-cov >= 5.0", "coverage[toml] >= 7.2"]
 lint = ["pre-commit == 4.2.0"]
 dev = ["changelist == 0.5"]
 
@@ -48,7 +48,7 @@ log_cli_level = "info"
 
 [tool.ruff]
 exclude = [
-  "lazy_loader/tests/fake_pkg/__init__.pyi",
+  "tests/fake_pkg/__init__.pyi",
 ]
 
 [tool.ruff.lint]
@@ -92,3 +92,13 @@ ignore = [
 
 [tool.ruff.format]
 docstring-code-format = true
+
+[tool.coverage.run]
+branch = true
+source = ["lazy_loader", "tests"]
+
+[tool.coverage.paths]
+source = [
+  "src/lazy_loader",
+  "*/site-packages/lazy_loader",
+]

lazy_loader/__init__.py renamed to src/lazy_loader/__init__.py

@@ -128,15 +128,15 @@ def test_lazy_attach_returns_copies():
 
 
 def test_attach_same_module_and_attr_name():
-    from lazy_loader.tests import fake_pkg
+    from tests import fake_pkg
 
     # Grab attribute twice, to ensure that importing it does not
     # override function by module
     assert isinstance(fake_pkg.some_func, types.FunctionType)
     assert isinstance(fake_pkg.some_func, types.FunctionType)
 
     # Ensure imports from submodule still work
-    from lazy_loader.tests.fake_pkg.some_func import some_func
+    from tests.fake_pkg.some_func import some_func
 
     assert isinstance(some_func, types.FunctionType)
 
@@ -157,7 +157,7 @@ def test_stub_loading(tmp_path):
 
 
 def test_stub_loading_parity():
-    from lazy_loader.tests import fake_pkg
+    from tests import fake_pkg
 
     from_stub = lazy.attach_stub(fake_pkg.__name__, fake_pkg.__file__)
     stub_getter, stub_dir, stub_all = from_stub