Skip to content

Commit 9478c18

Browse files
committed
update tests
1 parent fea9add commit 9478c18

File tree

1 file changed

+7
-32
lines changed

1 file changed

+7
-32
lines changed

edc_locator/tests/test_view_mixins.py

Lines changed: 7 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,13 @@
11
from unittest.case import skip
22

3-
from django.conf import settings
4-
from django.contrib.auth import get_user_model
5-
from django.contrib.auth.models import Permission
6-
from django.contrib.messages.storage.fallback import FallbackStorage
7-
from django.http.request import HttpRequest
83
from django.test import TestCase
94
from django.views.generic.base import ContextMixin
105
from edc_action_item import site_action_items
116
from edc_action_item.models import ActionItem
127
from edc_registration.models import RegisteredSubject
13-
from edc_sites.utils import get_site_model_cls
148
from edc_sites.view_mixins import SiteViewMixin
9+
from edc_test_utils.get_httprequest_for_tests import get_request_object_for_tests
10+
from edc_test_utils.get_user_for_tests import get_user_for_tests
1511

1612
from ..action_items import SUBJECT_LOCATOR_ACTION
1713
from ..view_mixins import SubjectLocatorViewMixin, SubjectLocatorViewMixinError
@@ -24,30 +20,9 @@ def __init__(self, **kwargs):
2420

2521
class TestViewMixins(TestCase):
2622
def setUp(self):
23+
self.user = get_user_for_tests()
2724
self.subject_identifier = "12345"
2825
RegisteredSubject.objects.create(subject_identifier=self.subject_identifier)
29-
self.user = get_user_model().objects.create_superuser(
30-
"user_login", "[email protected]", "pass"
31-
)
32-
self.user.is_active = True
33-
self.user.is_staff = True
34-
self.user.save()
35-
self.user.refresh_from_db()
36-
self.user.userprofile.sites.add(get_site_model_cls().objects.get(id=settings.SITE_ID))
37-
self.user.user_permissions.add(
38-
Permission.objects.get(
39-
codename="view_appointment", content_type__app_label="edc_appointment"
40-
)
41-
)
42-
43-
def get_request_object(self) -> HttpRequest:
44-
request = HttpRequest()
45-
setattr(request, "session", "session")
46-
messages = FallbackStorage(request)
47-
setattr(request, "_messages", messages)
48-
setattr(request, "user", self.user)
49-
setattr(request, "site", get_site_model_cls().objects.get(id=settings.SITE_ID))
50-
return request
5126

5227
def test_subject_locator_raises_on_bad_model(self):
5328
class MySubjectLocatorViewMixin(SiteViewMixin, SubjectLocatorViewMixin, ContextMixin):
@@ -56,7 +31,7 @@ class MySubjectLocatorViewMixin(SiteViewMixin, SubjectLocatorViewMixin, ContextM
5631

5732
mixin = MySubjectLocatorViewMixin()
5833
mixin.kwargs = {"subject_identifier": self.subject_identifier}
59-
mixin.request = self.get_request_object()
34+
mixin.request = get_request_object_for_tests(self.user)
6035
self.assertRaises(SubjectLocatorViewMixinError, mixin.get_context_data)
6136

6237
def test_subject_locator_raisesmissing_wrapper_cls(self):
@@ -73,7 +48,7 @@ class MySubjectLocatorViewMixin(SiteViewMixin, SubjectLocatorViewMixin, ContextM
7348

7449
mixin = MySubjectLocatorViewMixin()
7550
mixin.kwargs = {"subject_identifier": self.subject_identifier}
76-
mixin.request = self.get_request_object()
51+
mixin.request = get_request_object_for_tests(self.user)
7752
self.assertGreater(len(mixin.request._messages._queued_messages), 0)
7853

7954
def test_subject_locator_view_ok(self):
@@ -82,7 +57,7 @@ class MySubjectLocatorViewMixin(SiteViewMixin, SubjectLocatorViewMixin, ContextM
8257
subject_locator_model = "edc_locator.subjectlocator"
8358

8459
mixin = MySubjectLocatorViewMixin()
85-
mixin.request = self.get_request_object()
60+
mixin.request = get_request_object_for_tests(self.user)
8661
mixin.kwargs = {"subject_identifier": self.subject_identifier}
8762
try:
8863
mixin.get_context_data()
@@ -95,7 +70,7 @@ class MySubjectLocatorViewMixin(SiteViewMixin, SubjectLocatorViewMixin, ContextM
9570
subject_locator_model = "edc_locator.subjectlocator"
9671

9772
mixin = MySubjectLocatorViewMixin()
98-
mixin.request = self.get_request_object()
73+
mixin.request = get_request_object_for_tests(self.user)
9974
mixin.kwargs = {"subject_identifier": self.subject_identifier}
10075
try:
10176
mixin.get_context_data()

0 commit comments

Comments
 (0)