From fecaf174b188d2001d12846f2a7e1fd7631eb295 Mon Sep 17 00:00:00 2001 From: Trenton H <797416+stumpylog@users.noreply.github.com> Date: Wed, 1 Apr 2026 07:35:23 -0700 Subject: [PATCH] Try removing DirectoriesMixin from tests which probably don't actually need dirs, just DB --- src/documents/tests/test_api_bulk_edit.py | 3 +-- src/documents/tests/test_api_custom_fields.py | 3 +-- src/documents/tests/test_api_objects.py | 5 ++--- src/documents/tests/test_api_permissions.py | 7 +++---- src/documents/tests/test_api_profile.py | 4 +--- src/documents/tests/test_api_uisettings.py | 3 +-- src/documents/tests/test_api_workflows.py | 3 +-- 7 files changed, 10 insertions(+), 18 deletions(-) diff --git a/src/documents/tests/test_api_bulk_edit.py b/src/documents/tests/test_api_bulk_edit.py index 8a0ed5196..8393e1024 100644 --- a/src/documents/tests/test_api_bulk_edit.py +++ b/src/documents/tests/test_api_bulk_edit.py @@ -15,10 +15,9 @@ from documents.models import Document from documents.models import DocumentType from documents.models import StoragePath from documents.models import Tag -from documents.tests.utils import DirectoriesMixin -class TestBulkEditAPI(DirectoriesMixin, APITestCase): +class TestBulkEditAPI(APITestCase): @classmethod def setUpTestData(cls) -> None: super().setUpTestData() diff --git a/src/documents/tests/test_api_custom_fields.py b/src/documents/tests/test_api_custom_fields.py index c02a05e8f..59169f6dd 100644 --- a/src/documents/tests/test_api_custom_fields.py +++ b/src/documents/tests/test_api_custom_fields.py @@ -13,10 +13,9 @@ from rest_framework.test import APITestCase from documents.models import CustomField from documents.models import CustomFieldInstance from documents.models import Document -from documents.tests.utils import DirectoriesMixin -class TestCustomFieldsAPI(DirectoriesMixin, APITestCase): +class TestCustomFieldsAPI(APITestCase): ENDPOINT = "/api/custom_fields/" def setUp(self) -> None: diff --git a/src/documents/tests/test_api_objects.py b/src/documents/tests/test_api_objects.py index bf1ac4d9c..59fb7dce4 100644 --- a/src/documents/tests/test_api_objects.py +++ b/src/documents/tests/test_api_objects.py @@ -16,10 +16,9 @@ from documents.models import Document from documents.models import DocumentType from documents.models import StoragePath from documents.models import Tag -from documents.tests.utils import DirectoriesMixin -class TestApiObjects(DirectoriesMixin, APITestCase): +class TestApiObjects(APITestCase): def setUp(self) -> None: super().setUp() @@ -162,7 +161,7 @@ class TestApiObjects(DirectoriesMixin, APITestCase): ) -class TestApiStoragePaths(DirectoriesMixin, APITestCase): +class TestApiStoragePaths(APITestCase): ENDPOINT = "/api/storage_paths/" def setUp(self) -> None: diff --git a/src/documents/tests/test_api_permissions.py b/src/documents/tests/test_api_permissions.py index da001297f..9e473ff93 100644 --- a/src/documents/tests/test_api_permissions.py +++ b/src/documents/tests/test_api_permissions.py @@ -19,10 +19,9 @@ from documents.models import DocumentType from documents.models import MatchingModel from documents.models import StoragePath from documents.models import Tag -from documents.tests.utils import DirectoriesMixin -class TestApiAuth(DirectoriesMixin, APITestCase): +class TestApiAuth(APITestCase): def test_auth_required(self) -> None: d = Document.objects.create(title="Test") @@ -654,7 +653,7 @@ class TestApiAuth(DirectoriesMixin, APITestCase): self.assertEqual(response.status_code, status.HTTP_200_OK) -class TestApiUser(DirectoriesMixin, APITestCase): +class TestApiUser(APITestCase): ENDPOINT = "/api/users/" @classmethod @@ -996,7 +995,7 @@ class TestApiUser(DirectoriesMixin, APITestCase): self.assertEqual(returned_user1.is_staff, True) -class TestApiGroup(DirectoriesMixin, APITestCase): +class TestApiGroup(APITestCase): ENDPOINT = "/api/groups/" @classmethod diff --git a/src/documents/tests/test_api_profile.py b/src/documents/tests/test_api_profile.py index d642f52d8..05a372d9b 100644 --- a/src/documents/tests/test_api_profile.py +++ b/src/documents/tests/test_api_profile.py @@ -8,8 +8,6 @@ from rest_framework import status from rest_framework.authtoken.models import Token from rest_framework.test import APITestCase -from documents.tests.utils import DirectoriesMixin - # see allauth.socialaccount.providers.openid.provider.OpenIDProvider class MockOpenIDProvider: @@ -49,7 +47,7 @@ class MockOpenIDConnectProvider: return f"{self.app.provider_id}/login/?process=connect" -class TestApiProfile(DirectoriesMixin, APITestCase): +class TestApiProfile(APITestCase): ENDPOINT = "/api/profile/" def setUp(self) -> None: diff --git a/src/documents/tests/test_api_uisettings.py b/src/documents/tests/test_api_uisettings.py index 811b77286..ec8775500 100644 --- a/src/documents/tests/test_api_uisettings.py +++ b/src/documents/tests/test_api_uisettings.py @@ -6,11 +6,10 @@ from django.test import override_settings from rest_framework import status from rest_framework.test import APITestCase -from documents.tests.utils import DirectoriesMixin from paperless.version import __full_version_str__ -class TestApiUiSettings(DirectoriesMixin, APITestCase): +class TestApiUiSettings(APITestCase): ENDPOINT = "/api/ui_settings/" def setUp(self) -> None: diff --git a/src/documents/tests/test_api_workflows.py b/src/documents/tests/test_api_workflows.py index d23a2dc47..ecbf50e01 100644 --- a/src/documents/tests/test_api_workflows.py +++ b/src/documents/tests/test_api_workflows.py @@ -14,10 +14,9 @@ from documents.models import Tag from documents.models import Workflow from documents.models import WorkflowAction from documents.models import WorkflowTrigger -from documents.tests.utils import DirectoriesMixin -class TestApiWorkflows(DirectoriesMixin, APITestCase): +class TestApiWorkflows(APITestCase): ENDPOINT = "/api/workflows/" ENDPOINT_TRIGGERS = "/api/workflow_triggers/" ENDPOINT_ACTIONS = "/api/workflow_actions/"