investigate failing tests

conftest.py

@@ -44,8 +44,8 @@ def django_db_setup(django_db_setup, django_db_blocker, fixtures): # noqa: PT004
 @pytest.fixture
 def files(settings, tmp_path):
     """Create a temporary MEDIA_ROOT directory and copy test data into it."""
-    media_path = Path(__file__).parent.joinpath("testing").joinpath("media")
-    settings.MEDIA_ROOT = tmp_path.joinpath("media")
+    media_path = Path(__file__).parent / "testing" / "media"
+    settings.MEDIA_ROOT = tmp_path / "media"
     shutil.copytree(media_path, settings.MEDIA_ROOT)
     return settings.MEDIA_ROOT
 

pyproject.toml

@@ -101,7 +101,7 @@ pytest = [
   "pytest-mock~=3.11",
   "pytest-playwright>=0.4.3,<0.6.0",
   "pytest-randomly~=3.15",
-  "pytest-xdist~=3.3",
+  "pytest-xdist[psutil]~=3.3",
 ]
 
 [project.urls]
@@ -199,6 +199,7 @@ pythonpath = [".", "testing"]
 addopts = '-p no:randomly -m "not e2e"'
 markers = [
   "e2e: marks tests as end-to-end tests using playwright (deselect with '-m \"not e2e\"')",
+  "files: marks tests that use the 'files' fixture",
 ]
 filterwarnings = [
   # fail on RemovedInDjango50Warning exception

rdmo/projects/tests/test_view_issue.py

@@ -176,6 +176,7 @@ def test_issue_send_post_email(db, client, username, password, project_id, issue
         assert response.status_code == 404
 
 
+@pytest.mark.files()
 @pytest.mark.parametrize('username,password', users)
 @pytest.mark.parametrize('issue_id', issues)
 @pytest.mark.parametrize('project_id', projects)

rdmo/projects/tests/test_view_project.py

@@ -602,6 +602,7 @@ def test_project_delete_post(db, client, username, password, project_id):
         assert Project.objects.filter(pk=project_id).first()
 
 
+@pytest.mark.files()
 @pytest.mark.parametrize('username,password', users)
 @pytest.mark.parametrize('project_id', projects)
 def test_project_export_xml(db, client, files, username, password, project_id):

rdmo/projects/tests/test_view_project_create_import.py

@@ -183,6 +183,7 @@ def test_project_create_import_post_upload_file_forbidden(db, client, settings):
     assert response.status_code == 403
 
 
+@pytest.mark.files()
 @pytest.mark.parametrize('username,password', users)
 def test_project_create_import_post_import_file(db, settings, client, files, username, password):
     client.login(username=username, password=password)
@@ -238,6 +239,7 @@ def test_project_create_import_post_import_file(db, settings, client, files, use
         assert response.url.startswith('/account/login/')
 
 
+@pytest.mark.files()
 @pytest.mark.parametrize('username,password', users)
 def test_project_create_import_post_import_file_cancel(db, settings, client, files, username, password):
     client.login(username=username, password=password)

rdmo/projects/tests/test_view_project_update_import.py

@@ -151,6 +151,7 @@ def test_project_update_import_post_upload_file_empty(db, client, username, pass
         assert response.url.startswith('/account/login/')
 
 
+@pytest.mark.files()
 @pytest.mark.parametrize('username,password', users)
 @pytest.mark.parametrize('project_id', projects)
 def test_project_update_import_post_import_file(db, settings, client, files, username, password, project_id):
@@ -223,6 +224,7 @@ def test_project_update_import_post_import_file(db, settings, client, files, use
         assert response.url.startswith('/account/login/')
 
 
+@pytest.mark.files()
 @pytest.mark.parametrize('username,password', users)
 @pytest.mark.parametrize('project_id', projects)
 def test_project_update_import_post_import_file_cancel(db, settings, client, files, username, password, project_id):

rdmo/projects/tests/test_view_snapshot.py

@@ -57,6 +57,7 @@ def test_snapshot_create_get(db, client, username, password, project_id):
         assert response.status_code == 302
 
 
+@pytest.mark.files()
 @pytest.mark.parametrize('username,password', users)
 @pytest.mark.parametrize('project_id', projects)
 def test_snapshot_create_post(db, client, files, username, password, project_id):
@@ -182,6 +183,7 @@ def test_snapshot_rollback_get(db, client, username, password, project_id, snaps
         assert response.status_code == 404
 
 
+@pytest.mark.files()
 @pytest.mark.parametrize('username,password', users)
 @pytest.mark.parametrize('project_id', projects)
 @pytest.mark.parametrize('snapshot_id', snapshots)

rdmo/projects/tests/test_viewset_project_snapshot.py

@@ -86,6 +86,7 @@ def test_detail(db, client, username, password, project_id, snapshot_id):
         assert response.status_code == 404
 
 
+@pytest.mark.files()
 @pytest.mark.parametrize('username,password', users)
 @pytest.mark.parametrize('project_id', projects)
 def test_create(db, client, files, username, password, project_id):

rdmo/projects/tests/test_viewset_project_value.py

@@ -266,6 +266,7 @@ def test_set(db, client, username, password, project_id, value_id, set_values_co
         assert Value.objects.count() == values_count
 
 
+@pytest.mark.files()
 @pytest.mark.parametrize('username,password', users)
 @pytest.mark.parametrize('project_id', projects)
 @pytest.mark.parametrize('value_id', values)
@@ -285,6 +286,7 @@ def test_file_get(db, client, files, username, password, project_id, value_id):
         assert response.status_code == 404
 
 
+@pytest.mark.files()
 @pytest.mark.parametrize('username,password', users)
 @pytest.mark.parametrize('project_id', projects)
 @pytest.mark.parametrize('value_id', values)

rdmo/projects/tests/test_viewset_value.py

@@ -142,6 +142,7 @@ def test_delete(db, client, username, password, value_id):
         assert response.status_code == 401
 
 
+@pytest.mark.files()
 @pytest.mark.parametrize('username,password', users)
 @pytest.mark.parametrize('value_id', values)
 def test_file(db, client, files, username, password, value_id):