clean up test mixins

api_tests/draft_registrations/views/test_draft_registration_contributor_detail.py

@@ -2,7 +2,7 @@
 
 from api.base.settings.defaults import API_BASE
 from api_tests.nodes.views.test_node_contributors_detail import (
-    TestContributorDetail,
+    ContributorDetailMixin,
     TestNodeContributorPartialUpdate,
     TestNodeContributorDelete
 )
@@ -20,8 +20,8 @@
 def user():
     return AuthUserFactory(given_name='Dawn')
 
-
-class TestDraftContributorDetail(TestContributorDetail):
+@pytest.mark.django_db
+class TestDraftContributorDetail(ContributorDetailMixin):
     @pytest.fixture()
     def project_public(self, user, title, description, category):
         # Defining "project public" as a draft reg, overriding TestContributorDetail

api_tests/nodes/views/test_node_contributors_detail.py

@@ -14,9 +14,7 @@
 from website.project.signals import contributor_removed
 
 
-@pytest.mark.django_db
-@pytest.mark.enable_implicit_clean
-class TestContributorDetail:
+class ContributorDetailMixin:
 
     @pytest.fixture()
     def user(self):
@@ -68,6 +66,9 @@ def url_public(self, user, project_public):
     def url_private(self, node, user_id):
         return f'/{API_BASE}nodes/{node._id}/contributors/{user_id}/'
 
+@pytest.mark.django_db
+@pytest.mark.enable_implicit_clean
+class TestContributorDetail(ContributorDetailMixin):
     def test_get_public_contributor_detail(self, app, user, project_public, project_private, url_public):
         res = app.get(url_public)
         assert res.status_code == 200

api_tests/preprints/views/test_preprint_contributors_detail.py

@@ -15,9 +15,8 @@
 from api_tests.utils import disconnected_from_listeners
 from website.project.signals import contributor_removed
 
-
 @pytest.fixture()
-def user(self):
+def user():
     return AuthUserFactory()
 
 @pytest.mark.django_db