fix(root): isolate api test sqlite database per process

Use a process-local temporary SQLite path for API tests so
concurrent test runs and pre-push hooks do not collide on the same
file-backed database.

Co-Authored-By: First Fluke <our.first.fluke@gmail.com>

Author: gracefullight

apps/api/tests/conftest.py

@@ -1,19 +1,25 @@
 import asyncio
 import os
+import tempfile
 from collections.abc import Iterator
 from pathlib import Path
 
 import pytest
 from fastapi.testclient import TestClient
 
-TEST_DB_PATH = Path(__file__).parent / "test.db"
+TEST_DB_PATH = (
+    Path(tempfile.gettempdir()) / f"fullstack-starter-api-tests-{os.getpid()}.db"
+)
 os.environ["PROJECT_ENV"] = "staging"
 os.environ["DATABASE_URL"] = f"sqlite+aiosqlite:///{TEST_DB_PATH}"
 os.environ["DATABASE_URL_SYNC"] = f"sqlite:///{TEST_DB_PATH}"
 
 from src.lib.database import Base, engine  # noqa: E402
 from src.main import app  # noqa: E402
 
+if TEST_DB_PATH.exists():
+    TEST_DB_PATH.unlink()
+
 
 @pytest.fixture(autouse=True)
 def reset_database() -> Iterator[None]:
@@ -33,3 +39,11 @@ def client() -> Iterator[TestClient]:
     """Test client fixture."""
     with TestClient(app) as test_client:
         yield test_client
+
+
+@pytest.fixture(scope="session", autouse=True)
+def cleanup_test_database() -> Iterator[None]:
+    """Remove the process-local SQLite file after the test session."""
+    yield
+    if TEST_DB_PATH.exists():
+        TEST_DB_PATH.unlink()