Add pytest-xdist for parallel test execution

.gitlab-ci.yml

@@ -20,10 +20,7 @@ run-tox-pipeline:
   # Change pip's cache directory to be inside the project directory since we can
   # only cache local items.
   variables:
-    POSTGRES_DB: gso-test-db
-    POSTGRES_USER: nwa
-    POSTGRES_PASSWORD: nwa
-    DATABASE_URI_TEST: 'postgresql://nwa:nwa@postgres:5432/gso-test-db'
+    DATABASE_HOST: 'postgres'
     PIP_CACHE_DIR: "$CI_PROJECT_DIR/.cache/pip"
 
   cache:

requirements.txt

@@ -23,3 +23,4 @@ sphinx-rtd-theme==2.0.0
 urllib3_mock==0.3.3
 pytest-asyncio==0.23.6
 pre-commit~=3.7.0
+pytest-xdist==3.6.1
\ No newline at end of file

test/conftest.py

@@ -142,8 +142,14 @@ def data_config_filename() -> str:
 
 @pytest.fixture(scope="session")
 def db_uri():
-    """Provide the database uri configuration to run the migration on."""
-    return os.environ.get("DATABASE_URI_TEST", "postgresql://nwa:nwa@localhost/gso-test-db")
+    """Provide a unique database URI for each pytest-xdist worker, or a default URI if running without xdist."""
+    worker_id = os.getenv("PYTEST_XDIST_WORKER")
+    db_host = os.getenv("DATABASE_HOST", "localhost")
+
+    if worker_id:
+        return f"postgresql://nwa:nwa@{db_host}/gso-test-db_{worker_id}"
+
+    return f"postgresql://nwa:nwa@{db_host}/gso-test-db"
 
 
 def run_migrations(db_uri: str) -> None:
@@ -281,7 +287,7 @@ def partner_factory():
     return _create_partner
 
 
-@pytest.fixture(scope="session")
+@pytest.fixture()
 def geant_partner(partner_factory):
     return partner_factory(name="GEANT-TEST", email="goat-test@geant.org")
 

tox.ini

@@ -6,7 +6,7 @@ markers = "workflow,noautofixt"
 filterwarnings = "ignore,default:::gso"
 
 [testenv]
-passenv = DATABASE_URI_TEST,SKIP_ALL_TESTS,ENVIRONMENT_IGNORE_MUTATION_DISABLED
+passenv = DATABASE_HOST,SKIP_ALL_TESTS,ENVIRONMENT_IGNORE_MUTATION_DISABLED
 setenv =
     OAUTH2_ACTIVE = False
     TRANSLATIONS_DIR = ./gso/translations
@@ -17,14 +17,7 @@ deps =
     -r requirements.txt
 
 commands =
-    ruff check --respect-gitignore --preview .
-    ruff format --respect-gitignore --preview --check .
-    mypy .
-    coverage erase
-    coverage run --source gso --omit="gso/migrations/*" -m pytest {posargs}
-    coverage xml
-    coverage html
-    sh -c "if [ $SKIP_ALL_TESTS -eq 1 ]; then echo 'Skipping coverage report'; else coverage report --fail-under 80; fi"
+    pytest -n auto {posargs}
 
 allowlist_externals =
     sh