💚 Fix organization of curator tests to avoid collisions

laminlabs · Jan 2, 2025 · e5778b3 · e5778b3
1 parent df35be0
commit e5778b3
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 2 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -36,7 +36,7 @@ jobs:
           filters: |
             curator:
               - 'lamindb/curators/**'
-              - 'tests/core/test_curator.py'
+              - 'tests/curators/test_curator.py'
 
       - uses: actions/setup-python@v5
         if: ${{ matrix.group != 'curator' || steps.changes.outputs.curator == 'true' }}

diff --git a/noxfile.py b/noxfile.py
@@ -212,7 +212,7 @@ def build(session, group):
     elif group == "curator":
         run(
             session,
-            f"pytest {coverage_args} tests/core/test_curator.py --durations=50",
+            f"pytest {coverage_args} tests/curators/test_curator.py --durations=50",
         )
     elif group == "cli":
         run(session, f"pytest {coverage_args} ./sub/lamin-cli/tests --durations=50")

diff --git a/tests/curators/conftest.py b/tests/curators/conftest.py
@@ -0,0 +1,10 @@
+import lamindb_setup as ln_setup
+import pytest
+
+
+def pytest_sessionstart():
+    ln_setup.init(storage="./testdb", schema="bionty")
+
+
+def pytest_sessionfinish(session: pytest.Session):
+    ln_setup.delete("testdb", force=True)
diff --git a/tests/core/test_curator.py → tests/curators/test_curator.py b/tests/core/test_curator.py → tests/curators/test_curator.py