matthias/mayan-edms
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix failing tests

Browse Source 
Signed-off-by: Roberto Rosario <Roberto.Rosario@mayan-edms.com>
This commit is contained in:
Roberto Rosario
2018-12-29 04:47:40 -04:00
parent c40e0c136a
commit 23a4a56aae
9 changed files with 517 additions and 326 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
mayan/apps/common/tests/mixins.py
View File

@@ -130,22 +130,27 @@ class TempfileCheckMixin(object):
class UserMixin(object): class UserMixin(object):
auto_create_group = True
auto_create_users = True
def setUp(self): def setUp(self):
super(UserMixin, self).setUp() super(UserMixin, self).setUp()
self.admin_user = get_user_model().objects.create_superuser( if self.auto_create_users:
username=TEST_ADMIN_USERNAME, email=TEST_ADMIN_EMAIL, self.admin_user = get_user_model().objects.create_superuser(
password=TEST_ADMIN_PASSWORD username=TEST_ADMIN_USERNAME, email=TEST_ADMIN_EMAIL,
) password=TEST_ADMIN_PASSWORD
)
self.user = get_user_model().objects.create_user( self.user = get_user_model().objects.create_user(
username=TEST_USER_USERNAME, email=TEST_USER_EMAIL, username=TEST_USER_USERNAME, email=TEST_USER_EMAIL,
password=TEST_USER_PASSWORD password=TEST_USER_PASSWORD
) )
self.group = Group.objects.create(name=TEST_GROUP_NAME) if self.auto_create_group:
self.role = Role.objects.create(label=TEST_ROLE_LABEL) self.group = Group.objects.create(name=TEST_GROUP_NAME)
self.group.user_set.add(self.user) self.role = Role.objects.create(label=TEST_ROLE_LABEL)
self.role.groups.add(self.group) self.group.user_set.add(self.user)
self.role.groups.add(self.group)
def grant_access(self, obj, permission): def grant_access(self, obj, permission):
AccessControlList.objects.grant( AccessControlList.objects.grant(

285
mayan/apps/mailer/tests/test_views.py
View File

@@ -2,6 +2,7 @@ from __future__ import unicode_literals
from django.core import mail from django.core import mail
from mayan.apps.common.tests import GenericViewTestCase
from mayan.apps.documents.tests import GenericDocumentViewTestCase from mayan.apps.documents.tests import GenericDocumentViewTestCase
from ..models import UserMailer from ..models import UserMailer
@@ -23,103 +24,37 @@ from .mailers import TestBackend
from .mixins import MailerTestMixin from .mixins import MailerTestMixin
class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase): class MailerViewsTestCase(MailerTestMixin, GenericViewTestCase):
def _request_document_link_send(self): def setUp(self):
return self.post( super(MailerViewsTestCase, self).setUp()
'mailer:send_document_link', args=(self.document.pk,), self.login_user()
data={
'email': getattr(
self, 'test_email_address', TEST_EMAIL_ADDRESS
),
'user_mailer': self.user_mailer.pk
},
)
def _request_document_send(self):
return self.post(
'mailer:send_document', args=(self.document.pk,),
data={
'email': getattr(
self, 'test_email_address', TEST_EMAIL_ADDRESS
),
'user_mailer': self.user_mailer.pk
},
)
def _request_user_mailer_create(self): def _request_user_mailer_create(self):
return self.post( return self.post(
'mailer:user_mailer_create', args=( viewname='mailer:user_mailer_create', args=(
TEST_USER_MAILER_BACKEND_PATH, TEST_USER_MAILER_BACKEND_PATH,
), data={ ), data={
'default': True, 'default': True, 'enabled': True,
'enabled': True,
'label': TEST_USER_MAILER_LABEL, 'label': TEST_USER_MAILER_LABEL,
}, follow=True }
) )
def _request_user_mailer_delete(self): def _request_user_mailer_delete(self):
return self.post( return self.post(
'mailer:user_mailer_delete', args=(self.user_mailer.pk,) viewname='mailer:user_mailer_delete', args=(self.user_mailer.pk,)
) )
def _request_user_mailer_test(self): def _request_user_mailer_test(self):
return self.post( return self.post(
'mailer:user_mailer_test', args=(self.user_mailer.pk,), data={ viewname='mailer:user_mailer_test', args=(self.user_mailer.pk,),
data={
'email': getattr( 'email': getattr(
self, 'test_email_address', TEST_EMAIL_ADDRESS self, 'test_email_address', TEST_EMAIL_ADDRESS
) )
}, follow=True }
) )
def test_mail_link_view_no_permissions(self):
self._create_user_mailer()
self.login_user()
response = self._request_document_link_send()
self.assertContains(
response, 'Select a valid choice', status_code=200
)
def test_mail_link_view_with_permission(self):
self._create_user_mailer()
self.login_user()
self.grant_permission(permission=permission_mailing_link)
self.grant_permission(permission=permission_user_mailer_use)
self._request_document_link_send()
self.assertEqual(len(mail.outbox), 1)
self.assertEqual(mail.outbox[0].from_email, TEST_EMAIL_FROM_ADDRESS)
self.assertEqual(mail.outbox[0].to, [TEST_EMAIL_ADDRESS])
def test_mail_document_view_no_permissions(self):
self._create_user_mailer()
self.login_user()
response = self._request_document_send()
self.assertContains(
response, 'Select a valid choice', status_code=200
)
def test_mail_document_view_with_permission(self):
self._create_user_mailer()
self.login_user()
self.grant_permission(permission=permission_mailing_send_document)
self.grant_permission(permission=permission_user_mailer_use)
self._request_document_send()
self.assertEqual(len(mail.outbox), 1)
self.assertEqual(mail.outbox[0].from_email, TEST_EMAIL_FROM_ADDRESS)
self.assertEqual(mail.outbox[0].to, [TEST_EMAIL_ADDRESS])
def test_user_mailer_create_view_no_permissions(self): def test_user_mailer_create_view_no_permissions(self):
self.login_user()
self.grant_permission(permission=permission_user_mailer_view)
response = self._request_user_mailer_create() response = self._request_user_mailer_create()
self.assertNotContains( self.assertNotContains(
@@ -128,21 +63,15 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
self.assertEqual(UserMailer.objects.count(), 0) self.assertEqual(UserMailer.objects.count(), 0)
def test_user_mailer_create_view_with_permissions(self): def test_user_mailer_create_view_with_permissions(self):
self.login_user()
self.grant_permission(permission=permission_user_mailer_create) self.grant_permission(permission=permission_user_mailer_create)
self.grant_permission(permission=permission_user_mailer_view)
response = self._request_user_mailer_create() response = self._request_user_mailer_create()
self.assertContains( self.assertEqual(response.status_code, 302)
response, text=TestBackend.label, status_code=200
)
self.assertEqual(UserMailer.objects.count(), 1) self.assertEqual(UserMailer.objects.count(), 1)
def test_user_mailer_delete_view_no_permissions(self): def test_user_mailer_delete_view_no_permissions(self):
self._create_user_mailer() self._create_user_mailer()
self.login_user()
self._request_user_mailer_delete() self._request_user_mailer_delete()
@@ -152,7 +81,6 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
def test_user_mailer_delete_view_with_access(self): def test_user_mailer_delete_view_with_access(self):
self._create_user_mailer() self._create_user_mailer()
self.login_user()
self.grant_access( self.grant_access(
obj=self.user_mailer, permission=permission_user_mailer_delete obj=self.user_mailer, permission=permission_user_mailer_delete
@@ -162,26 +90,24 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
self.assertEqual(UserMailer.objects.count(), 0) self.assertEqual(UserMailer.objects.count(), 0)
def _request_user_mailer_list_view(self):
return self.get(viewname='mailer:user_mailer_list')
def test_user_mailer_list_view_no_permissions(self): def test_user_mailer_list_view_no_permissions(self):
self._create_user_mailer() self._create_user_mailer()
self.login_user()
response = self.get( response = self._request_user_mailer_list_view()
'mailer:user_mailer_list',
)
self.assertNotContains( self.assertNotContains(
response, text=self.user_mailer.label, status_code=200 response, text=self.user_mailer.label, status_code=200
) )
def test_user_mailer_list_view_with_permissions(self): def test_user_mailer_list_view_with_access(self):
self._create_user_mailer() self._create_user_mailer()
self.login_user()
self.grant_permission(permission=permission_user_mailer_view) self.grant_access(
permission=permission_user_mailer_view, obj=self.user_mailer
response = self.get(
'mailer:user_mailer_list',
) )
response = self._request_user_mailer_list_view()
self.assertContains( self.assertContains(
response, text=self.user_mailer.label, status_code=200 response, text=self.user_mailer.label, status_code=200
@@ -189,7 +115,6 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
def test_user_mailer_test_view_no_permissions(self): def test_user_mailer_test_view_no_permissions(self):
self._create_user_mailer() self._create_user_mailer()
self.login_user()
response = self._request_user_mailer_test() response = self._request_user_mailer_test()
@@ -199,7 +124,6 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
def test_user_mailer_test_view_with_access(self): def test_user_mailer_test_view_with_access(self):
self._create_user_mailer() self._create_user_mailer()
self.login_user()
self.grant_access( self.grant_access(
obj=self.user_mailer, permission=permission_user_mailer_use obj=self.user_mailer, permission=permission_user_mailer_use
@@ -207,14 +131,13 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
response = self._request_user_mailer_test() response = self._request_user_mailer_test()
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 302)
self.assertEqual(len(mail.outbox), 1) self.assertEqual(len(mail.outbox), 1)
self.assertEqual(mail.outbox[0].from_email, TEST_EMAIL_FROM_ADDRESS) self.assertEqual(mail.outbox[0].from_email, TEST_EMAIL_FROM_ADDRESS)
self.assertEqual(mail.outbox[0].to, [TEST_EMAIL_ADDRESS]) self.assertEqual(mail.outbox[0].to, [TEST_EMAIL_ADDRESS])
def test_send_multiple_recipients_comma(self): def test_send_multiple_recipients_comma(self):
self._create_user_mailer() self._create_user_mailer()
self.login_user()
self.grant_access( self.grant_access(
obj=self.user_mailer, permission=permission_user_mailer_use obj=self.user_mailer, permission=permission_user_mailer_use
@@ -222,7 +145,7 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
self.test_email_address = TEST_RECIPIENTS_MULTIPLE_COMMA self.test_email_address = TEST_RECIPIENTS_MULTIPLE_COMMA
response = self._request_user_mailer_test() response = self._request_user_mailer_test()
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 302)
self.assertEqual(len(mail.outbox), 1) self.assertEqual(len(mail.outbox), 1)
self.assertEqual(mail.outbox[0].from_email, TEST_EMAIL_FROM_ADDRESS) self.assertEqual(mail.outbox[0].from_email, TEST_EMAIL_FROM_ADDRESS)
self.assertEqual( self.assertEqual(
@@ -231,7 +154,6 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
def test_send_multiple_recipients_mixed(self): def test_send_multiple_recipients_mixed(self):
self._create_user_mailer() self._create_user_mailer()
self.login_user()
self.grant_access( self.grant_access(
obj=self.user_mailer, permission=permission_user_mailer_use obj=self.user_mailer, permission=permission_user_mailer_use
@@ -239,7 +161,7 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
self.test_email_address = TEST_RECIPIENTS_MULTIPLE_MIXED self.test_email_address = TEST_RECIPIENTS_MULTIPLE_MIXED
response = self._request_user_mailer_test() response = self._request_user_mailer_test()
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 302)
self.assertEqual(len(mail.outbox), 1) self.assertEqual(len(mail.outbox), 1)
self.assertEqual(mail.outbox[0].from_email, TEST_EMAIL_FROM_ADDRESS) self.assertEqual(mail.outbox[0].from_email, TEST_EMAIL_FROM_ADDRESS)
self.assertEqual( self.assertEqual(
@@ -248,7 +170,6 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
def test_send_multiple_recipients_semicolon(self): def test_send_multiple_recipients_semicolon(self):
self._create_user_mailer() self._create_user_mailer()
self.login_user()
self.grant_access( self.grant_access(
obj=self.user_mailer, permission=permission_user_mailer_use obj=self.user_mailer, permission=permission_user_mailer_use
@@ -256,16 +177,165 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
self.test_email_address = TEST_RECIPIENTS_MULTIPLE_SEMICOLON self.test_email_address = TEST_RECIPIENTS_MULTIPLE_SEMICOLON
response = self._request_user_mailer_test() response = self._request_user_mailer_test()
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 302)
self.assertEqual(len(mail.outbox), 1) self.assertEqual(len(mail.outbox), 1)
self.assertEqual(mail.outbox[0].from_email, TEST_EMAIL_FROM_ADDRESS) self.assertEqual(mail.outbox[0].from_email, TEST_EMAIL_FROM_ADDRESS)
self.assertEqual( self.assertEqual(
mail.outbox[0].to, TEST_RECIPIENTS_MULTIPLE_SEMICOLON_RESULT mail.outbox[0].to, TEST_RECIPIENTS_MULTIPLE_SEMICOLON_RESULT
) )
class DocumentViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
def setUp(self):
super(DocumentViewsTestCase, self).setUp()
self.login_user()
def _request_document_link_send(self):
return self.post(
viewname='mailer:document_send_link', args=(self.document.pk,),
data={
'email': getattr(
self, 'test_email_address', TEST_EMAIL_ADDRESS
),
'user_mailer': self.user_mailer.pk
}
)
def _request_document_send(self):
return self.post(
viewname='mailer:document_send', args=(self.document.pk,),
data={
'email': getattr(
self, 'test_email_address', TEST_EMAIL_ADDRESS
), 'user_mailer': self.user_mailer.pk
}
)
def test_mail_link_view_no_permissions(self):
self._create_user_mailer()
response = self._request_document_link_send()
self.assertNotContains(
response=response, text=self.user_mailer.label, status_code=200
)
self.assertNotContains(
response=response, text=self.document.label, status_code=200
)
self.assertEqual(len(mail.outbox), 0)
def test_mail_link_view_with_document_access(self):
self._create_user_mailer()
self.grant_access(
permission=permission_mailing_link, obj=self.document
)
response = self._request_document_link_send()
self.assertNotContains(
response=response, text=self.user_mailer.label, status_code=200
)
self.assertContains(
response=response, text=self.document.label, status_code=200
)
self.assertEqual(len(mail.outbox), 0)
def test_mail_link_view_with_mailer_access(self):
self._create_user_mailer()
self.grant_access(
permission=permission_user_mailer_use, obj=self.user_mailer
)
response = self._request_document_link_send()
self.assertNotContains(
response=response, text=self.user_mailer.label, status_code=302
)
self.assertNotContains(
response=response, text=self.document.label, status_code=302
)
self.assertEqual(len(mail.outbox), 0)
def test_mail_link_view_with_full_access(self):
self._create_user_mailer()
self.grant_access(
permission=permission_mailing_link, obj=self.document
)
self.grant_access(
permission=permission_user_mailer_use, obj=self.user_mailer
)
self._request_document_link_send()
self.assertEqual(len(mail.outbox), 1)
self.assertEqual(mail.outbox[0].from_email, TEST_EMAIL_FROM_ADDRESS)
self.assertEqual(mail.outbox[0].to, [TEST_EMAIL_ADDRESS])
def test_mail_document_view_no_permissions(self):
self._create_user_mailer()
response = self._request_document_send()
self.assertNotContains(
response=response, text=self.user_mailer.label, status_code=200
)
self.assertNotContains(
response=response, text=self.document.label, status_code=200
)
self.assertEqual(len(mail.outbox), 0)
def test_mail_document_view_with_document_access(self):
self._create_user_mailer()
self.grant_access(
permission=permission_mailing_send_document, obj=self.document
)
response = self._request_document_send()
self.assertNotContains(
response=response, text=self.user_mailer.label, status_code=200
)
self.assertContains(
response=response, text=self.document.label, status_code=200
)
self.assertEqual(len(mail.outbox), 0)
def test_mail_document_view_with_mailer_access(self):
self._create_user_mailer()
self.grant_access(
permission=permission_user_mailer_use, obj=self.user_mailer
)
response = self._request_document_send()
self.assertNotContains(
response=response, text=self.document.label, status_code=302
)
self.assertEqual(len(mail.outbox), 0)
def test_mail_document_view_with_full_access(self):
self._create_user_mailer()
self.grant_access(
permission=permission_user_mailer_use, obj=self.user_mailer
)
self.grant_access(
permission=permission_mailing_send_document, obj=self.document
)
response = self._request_document_send()
self.assertEqual(len(mail.outbox), 1)
self.assertEqual(mail.outbox[0].from_email, TEST_EMAIL_FROM_ADDRESS)
self.assertEqual(mail.outbox[0].to, [TEST_EMAIL_ADDRESS])
def test_mail_link_view_recipients_comma(self): def test_mail_link_view_recipients_comma(self):
self._create_user_mailer() self._create_user_mailer()
self.login_user()
self.grant_permission(permission=permission_mailing_link) self.grant_permission(permission=permission_mailing_link)
self.grant_permission(permission=permission_user_mailer_use) self.grant_permission(permission=permission_user_mailer_use)
@@ -281,7 +351,6 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
def test_mail_link_view_recipients_mixed(self): def test_mail_link_view_recipients_mixed(self):
self._create_user_mailer() self._create_user_mailer()
self.login_user()
self.grant_permission(permission=permission_mailing_link) self.grant_permission(permission=permission_mailing_link)
self.grant_permission(permission=permission_user_mailer_use) self.grant_permission(permission=permission_user_mailer_use)
@@ -297,7 +366,6 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
def test_mail_link_view_recipients_semicolon(self): def test_mail_link_view_recipients_semicolon(self):
self._create_user_mailer() self._create_user_mailer()
self.login_user()
self.grant_permission(permission=permission_mailing_link) self.grant_permission(permission=permission_mailing_link)
self.grant_permission(permission=permission_user_mailer_use) self.grant_permission(permission=permission_user_mailer_use)
@@ -313,7 +381,6 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
def test_mail_document_view_recipients_comma(self): def test_mail_document_view_recipients_comma(self):
self._create_user_mailer() self._create_user_mailer()
self.login_user()
self.grant_permission(permission=permission_mailing_send_document) self.grant_permission(permission=permission_mailing_send_document)
self.grant_permission(permission=permission_user_mailer_use) self.grant_permission(permission=permission_user_mailer_use)
@@ -329,7 +396,6 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
def test_mail_document_view_recipients_mixed(self): def test_mail_document_view_recipients_mixed(self):
self._create_user_mailer() self._create_user_mailer()
self.login_user()
self.grant_permission(permission=permission_mailing_send_document) self.grant_permission(permission=permission_mailing_send_document)
self.grant_permission(permission=permission_user_mailer_use) self.grant_permission(permission=permission_user_mailer_use)
@@ -345,7 +411,6 @@ class MailerViewsTestCase(MailerTestMixin, GenericDocumentViewTestCase):
def test_mail_document_view_recipients_semicolon(self): def test_mail_document_view_recipients_semicolon(self):
self._create_user_mailer() self._create_user_mailer()
self.login_user()
self.grant_permission(permission=permission_mailing_send_document) self.grant_permission(permission=permission_mailing_send_document)
self.grant_permission(permission=permission_user_mailer_use) self.grant_permission(permission=permission_user_mailer_use)

20
mayan/apps/ocr/tests/test_views.py
View File

@@ -21,7 +21,8 @@ class OCRViewsTestCase(GenericDocumentViewTestCase):
def _request_document_content_view(self): def _request_document_content_view(self):
return self.get( return self.get(
viewname='ocr:document_ocr_content', args=(self.document.pk,) viewname='ocr:document_content',
kwargs={'pk': self.document.pk}
) )
def test_document_content_view_no_permissions(self): def test_document_content_view_no_permissions(self):
@@ -44,9 +45,8 @@ class OCRViewsTestCase(GenericDocumentViewTestCase):
def _request_document_page_content_view(self): def _request_document_page_content_view(self):
return self.get( return self.get(
viewname='ocr:document_page_ocr_content', args=( viewname='ocr:document_page_content',
self.document.pages.first().pk, kwargs={'pk': self.document.pages.first().pk}
)
) )
def test_document_page_content_view_no_permissions(self): def test_document_page_content_view_no_permissions(self):
@@ -69,7 +69,8 @@ class OCRViewsTestCase(GenericDocumentViewTestCase):
def _request_document_submit_view(self): def _request_document_submit_view(self):
return self.post( return self.post(
viewname='ocr:document_submit', args=(self.document.pk,) viewname='ocr:document_submit',
kwargs={'pk': self.document.pk}
) )
def test_document_submit_view_no_permission(self): def test_document_submit_view_no_permission(self):
@@ -87,7 +88,7 @@ class OCRViewsTestCase(GenericDocumentViewTestCase):
def _request_multiple_document_submit_view(self): def _request_multiple_document_submit_view(self):
return self.post( return self.post(
viewname='ocr:document_submit_multiple', viewname='ocr:document_multiple_submit',
data={ data={
'id_list': self.document.pk, 'id_list': self.document.pk,
} }
@@ -108,7 +109,8 @@ class OCRViewsTestCase(GenericDocumentViewTestCase):
def _request_document_ocr_download_view(self): def _request_document_ocr_download_view(self):
return self.get( return self.get(
viewname='ocr:document_ocr_download', args=(self.document.pk,) viewname='ocr:document_download',
kwargs={'pk': self.document.pk}
) )
def test_document_ocr_download_view_no_permission(self): def test_document_ocr_download_view_no_permission(self):
@@ -141,8 +143,8 @@ class DocumentTypeViewsTestCase(GenericDocumentViewTestCase):
def _request_document_type_ocr_settings_view(self): def _request_document_type_ocr_settings_view(self):
return self.get( return self.get(
viewname='ocr:document_type_ocr_settings', viewname='ocr:document_type_settings',
args=(self.document.document_type.pk,) kwargs={'pk': self.document.document_type.pk}
) )
def test_document_type_ocr_settings_view_no_permission(self): def test_document_type_ocr_settings_view_no_permission(self):

2
mayan/apps/ocr/urls.py
View File

@@ -44,7 +44,7 @@ urlpatterns = [
url( url(
r'^document_types/(?P<pk>\d+)/ocr/settings/$', r'^document_types/(?P<pk>\d+)/ocr/settings/$',
DocumentTypeSettingsEditView.as_view(), DocumentTypeSettingsEditView.as_view(),
name='document_type_ocr_settings' name='document_type_settings'
), ),
url(r'^errors/$', EntryListView.as_view(), name='entry_list'), url(r'^errors/$', EntryListView.as_view(), name='entry_list'),
] ]

8
mayan/apps/user_management/events.py
View File

@@ -9,15 +9,15 @@ namespace = EventTypeNamespace(
) )
event_group_created = namespace.add_event_type( event_group_created = namespace.add_event_type(
label=_('Group created'), name='created' label=_('Group created'), name='group_created'
) )
event_group_edited = namespace.add_event_type( event_group_edited = namespace.add_event_type(
label=_('Group edited'), name='edited' label=_('Group edited'), name='group_edited'
) )
event_user_created = namespace.add_event_type( event_user_created = namespace.add_event_type(
label=_('User created'), name='created' label=_('User created'), name='user_created'
) )
event_user_edited = namespace.add_event_type( event_user_edited = namespace.add_event_type(
label=_('User edited'), name='edited' label=_('User edited'), name='user_edited'
) )

55
mayan/apps/user_management/tests/mixins.py
View File

@@ -1,20 +1,63 @@
from __future__ import unicode_literals from __future__ import unicode_literals
from django.contrib.auth import get_user_model from django.contrib.auth import get_user_model
from django.contrib.auth.models import Group
from .literals import ( from .literals import (
TEST_USER_2_EMAIL, TEST_USER_2_PASSWORD, TEST_USER_2_USERNAME TEST_GROUP_2_NAME, TEST_GROUP_2_NAME_EDITED, TEST_USER_2_EMAIL,
TEST_USER_2_PASSWORD, TEST_USER_2_USERNAME, TEST_USER_2_USERNAME_EDITED
) )
class UserTestMixin(object): class UserTestMixin(object):
def _create_user(self): def _create_test_group(self):
return get_user_model().objects.create_user( self.test_group = Group.objects.create(name=TEST_GROUP_2_NAME)
def _edit_test_group(self):
self.test_group.name = TEST_GROUP_2_NAME_EDITED
self.test_group.save()
def _create_test_user(self):
self.test_user = get_user_model().objects.create(
username=TEST_USER_2_USERNAME, email=TEST_USER_2_EMAIL, username=TEST_USER_2_USERNAME, email=TEST_USER_2_EMAIL,
password=TEST_USER_2_PASSWORD password=TEST_USER_2_PASSWORD
) )
def _create_test_user_2(self): def _request_test_group_create_view(self):
self.user_2 = get_user_model().objects.create( reponse = self.post(
username=TEST_USER_2_USERNAME viewname='user_management:group_create', data={
'name': TEST_GROUP_2_NAME
}
)
self.test_group = Group.objects.filter(name=TEST_GROUP_2_NAME).first()
return reponse
def _request_test_group_edit_view(self):
return self.post(
viewname='user_management:group_edit', kwargs={
'pk': self.test_group.pk
}, data={
'name': TEST_GROUP_2_NAME_EDITED
}
)
def _request_test_user_create_view(self):
reponse = self.post(
viewname='user_management:user_create', data={
'username': TEST_USER_2_USERNAME
}
)
self.test_user = get_user_model().objects.filter(
username=TEST_USER_2_USERNAME
).first()
return reponse
def _request_test_user_edit_view(self):
return self.post(
viewname='user_management:user_edit', kwargs={
'pk': self.test_user.pk
}, data={
'username': TEST_USER_2_USERNAME_EDITED
}
) )

322
mayan/apps/user_management/tests/test_api.py
View File

File diff suppressed because it is too large Load Diff

50
mayan/apps/user_management/tests/test_events.py Normal file
View File

@@ -0,0 +1,50 @@
from __future__ import unicode_literals
from actstream.models import Action
from mayan.apps.common.tests import GenericViewTestCase
from ..events import (
event_group_created, event_group_edited, event_user_created,
event_user_edited
)
from .mixins import UserTestMixin
class GroupEventsTestCase(UserTestMixin, GenericViewTestCase):
auto_create_group = False
def test_group_create_event(self):
self.login_admin_user()
Action.objects.all().delete()
self._request_test_group_create_view()
self.assertEqual(Action.objects.last().target, self.test_group)
self.assertEqual(Action.objects.last().verb, event_group_created.id)
def test_group_edit_event(self):
self.login_admin_user()
self._create_test_group()
Action.objects.all().delete()
self._request_test_group_edit_view()
self.assertEqual(Action.objects.last().target, self.test_group)
self.assertEqual(Action.objects.last().verb, event_group_edited.id)
class UserEventsTestCase(UserTestMixin, GenericViewTestCase):
auto_create_group = False
def test_user_create_event(self):
self.login_admin_user()
Action.objects.all().delete()
self._request_test_user_create_view()
self.assertEqual(Action.objects.last().target, self.test_user)
self.assertEqual(Action.objects.last().verb, event_user_created.id)
def test_user_edit_event(self):
self.login_admin_user()
self._create_test_user()
Action.objects.all().delete()
self._request_test_user_edit_view()
self.assertEqual(Action.objects.last().target, self.test_user)
self.assertEqual(Action.objects.last().verb, event_user_edited.id)

72
mayan/apps/user_management/tests/test_views.py
View File

@@ -28,55 +28,55 @@ class UserManagementViewTestCase(UserTestMixin, GenericViewTestCase):
super(UserManagementViewTestCase, self).setUp() super(UserManagementViewTestCase, self).setUp()
self.login_user() self.login_user()
def _request_user_create_view(self): #def _request_test_user_create_view(self):
return self.post( # return self.post(
viewname='user_management:user_create', data={ # viewname='user_management:user_create', data={
'username': TEST_USER_2_USERNAME # 'username': TEST_USER_2_USERNAME
} # }
) # )
def test_user_create_view_no_permission(self): def test_user_create_view_no_permission(self):
response = self._request_user_create_view() response = self._request_test_user_create_view()
self.assertEqual(response.status_code, 403) self.assertEqual(response.status_code, 403)
self.assertEqual(get_user_model().objects.count(), 2) self.assertEqual(get_user_model().objects.count(), 2)
self.assertFalse(TEST_USER_2_USERNAME in get_user_model().objects.values_list('username', flat=True)) self.assertFalse(TEST_USER_2_USERNAME in get_user_model().objects.values_list('username', flat=True))
def test_user_create_view_with_permission(self): def test_user_create_view_with_permission(self):
self.grant_permission(permission=permission_user_create) self.grant_permission(permission=permission_user_create)
response = self._request_user_create_view() response = self._request_test_user_create_view()
self.assertEqual(response.status_code, 302) self.assertEqual(response.status_code, 302)
self.assertEqual(get_user_model().objects.count(), 3) self.assertEqual(get_user_model().objects.count(), 3)
self.assertTrue(TEST_USER_2_USERNAME in get_user_model().objects.values_list('username', flat=True)) self.assertTrue(TEST_USER_2_USERNAME in get_user_model().objects.values_list('username', flat=True))
def _request_user_groups_view(self): def _request_user_groups_view(self):
return self.post( return self.post(
viewname='user_management:user_groups', args=(self.user_2.pk,) viewname='user_management:user_groups', args=(self.test_user.pk,)
) )
def test_user_groups_view_no_permission(self): def test_user_groups_view_no_permission(self):
self._create_test_user_2() self._create_test_user()
response = self._request_user_groups_view() response = self._request_user_groups_view()
self.assertEqual(response.status_code, 403) self.assertEqual(response.status_code, 403)
def test_user_groups_view_with_access(self): def test_user_groups_view_with_access(self):
self._create_test_user_2() self._create_test_user()
self.grant_access(permission=permission_user_edit, obj=self.user_2) self.grant_access(permission=permission_user_edit, obj=self.test_user)
response = self._request_user_groups_view() response = self._request_user_groups_view()
self.assertContains( self.assertContains(
response=response, text=self.user_2.username, status_code=200 response=response, text=self.test_user.username, status_code=200
) )
def _request_set_password_view(self, password): def _request_set_password_view(self, password):
return self.post( return self.post(
viewname='user_management:user_set_password', args=(self.user_2.pk,), viewname='user_management:user_set_password', args=(self.test_user.pk,),
data={ data={
'new_password1': password, 'new_password2': password 'new_password1': password, 'new_password2': password
} }
) )
def test_user_set_password_view_no_access(self): def test_user_set_password_view_no_access(self):
self._create_test_user_2() self._create_test_user()
response = self._request_set_password_view( response = self._request_set_password_view(
password=TEST_USER_PASSWORD_EDITED password=TEST_USER_PASSWORD_EDITED
) )
@@ -90,13 +90,13 @@ class UserManagementViewTestCase(UserTestMixin, GenericViewTestCase):
username=TEST_USER_2_USERNAME, password=TEST_USER_PASSWORD_EDITED username=TEST_USER_2_USERNAME, password=TEST_USER_PASSWORD_EDITED
) )
response = self.get('common:current_user_details') response = self.get(viewname='user_management:current_user_details')
self.assertEqual(response.status_code, 302) self.assertEqual(response.status_code, 302)
def test_user_set_password_view_with_access(self): def test_user_set_password_view_with_access(self):
self._create_test_user_2() self._create_test_user()
self.grant_access(permission=permission_user_edit, obj=self.user_2) self.grant_access(permission=permission_user_edit, obj=self.test_user)
response = self._request_set_password_view( response = self._request_set_password_view(
password=TEST_USER_PASSWORD_EDITED password=TEST_USER_PASSWORD_EDITED
@@ -108,7 +108,7 @@ class UserManagementViewTestCase(UserTestMixin, GenericViewTestCase):
self.login( self.login(
username=TEST_USER_2_USERNAME, password=TEST_USER_PASSWORD_EDITED username=TEST_USER_2_USERNAME, password=TEST_USER_PASSWORD_EDITED
) )
response = self.get('common:current_user_details') response = self.get(viewname='user_management:current_user_details')
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 200)
@@ -116,14 +116,14 @@ class UserManagementViewTestCase(UserTestMixin, GenericViewTestCase):
return self.post( return self.post(
viewname='user_management:user_multiple_set_password', viewname='user_management:user_multiple_set_password',
data={ data={
'id_list': self.user_2.pk, 'id_list': self.test_user.pk,
'new_password1': password, 'new_password1': password,
'new_password2': password 'new_password2': password
} }
) )
def test_user_multiple_set_password_view_no_access(self): def test_user_multiple_set_password_view_no_access(self):
self._create_test_user_2() self._create_test_user()
response = self._request_multiple_user_set_password_view( response = self._request_multiple_user_set_password_view(
password=TEST_USER_PASSWORD_EDITED password=TEST_USER_PASSWORD_EDITED
) )
@@ -137,12 +137,12 @@ class UserManagementViewTestCase(UserTestMixin, GenericViewTestCase):
username=TEST_USER_2_USERNAME, password=TEST_USER_PASSWORD_EDITED username=TEST_USER_2_USERNAME, password=TEST_USER_PASSWORD_EDITED
) )
response = self.get('common:current_user_details') response = self.get(viewname='user_management:current_user_details')
self.assertEqual(response.status_code, 302) self.assertEqual(response.status_code, 302)
def test_user_multiple_set_password_view_with_access(self): def test_user_multiple_set_password_view_with_access(self):
self._create_test_user_2() self._create_test_user()
self.grant_access(permission=permission_user_edit, obj=self.user_2) self.grant_access(permission=permission_user_edit, obj=self.test_user)
response = self._request_multiple_user_set_password_view( response = self._request_multiple_user_set_password_view(
password=TEST_USER_PASSWORD_EDITED password=TEST_USER_PASSWORD_EDITED
@@ -154,24 +154,24 @@ class UserManagementViewTestCase(UserTestMixin, GenericViewTestCase):
self.login( self.login(
username=TEST_USER_2_USERNAME, password=TEST_USER_PASSWORD_EDITED username=TEST_USER_2_USERNAME, password=TEST_USER_PASSWORD_EDITED
) )
response = self.get('common:current_user_details') response = self.get(viewname='user_management:current_user_details')
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 200)
def _request_user_delete_view(self): def _request_user_delete_view(self):
return self.post( return self.post(
viewname='user_management:user_delete', args=(self.user_2.pk,) viewname='user_management:user_delete', args=(self.test_user.pk,)
) )
def test_user_delete_view_no_access(self): def test_user_delete_view_no_access(self):
self._create_test_user_2() self._create_test_user()
response = self._request_user_delete_view() response = self._request_user_delete_view()
self.assertEqual(response.status_code, 302) self.assertEqual(response.status_code, 302)
self.assertEqual(get_user_model().objects.count(), 3) self.assertEqual(get_user_model().objects.count(), 3)
def test_user_delete_view_with_access(self): def test_user_delete_view_with_access(self):
self._create_test_user_2() self._create_test_user()
self.grant_access(permission=permission_user_delete, obj=self.user_2) self.grant_access(permission=permission_user_delete, obj=self.test_user)
response = self._request_user_delete_view() response = self._request_user_delete_view()
self.assertEqual(response.status_code, 302) self.assertEqual(response.status_code, 302)
self.assertEqual(get_user_model().objects.count(), 2) self.assertEqual(get_user_model().objects.count(), 2)
@@ -179,19 +179,19 @@ class UserManagementViewTestCase(UserTestMixin, GenericViewTestCase):
def _request_user_multiple_delete_view(self): def _request_user_multiple_delete_view(self):
return self.post( return self.post(
viewname='user_management:user_multiple_delete', data={ viewname='user_management:user_multiple_delete', data={
'id_list': self.user_2.pk 'id_list': self.test_user.pk
} }
) )
def test_user_multiple_delete_view_no_access(self): def test_user_multiple_delete_view_no_access(self):
self._create_test_user_2() self._create_test_user()
response = self._request_user_multiple_delete_view() response = self._request_user_multiple_delete_view()
self.assertEqual(response.status_code, 302) self.assertEqual(response.status_code, 302)
self.assertEqual(get_user_model().objects.count(), 3) self.assertEqual(get_user_model().objects.count(), 3)
def test_user_multiple_delete_view_with_access(self): def test_user_multiple_delete_view_with_access(self):
self._create_test_user_2() self._create_test_user()
self.grant_access(permission=permission_user_delete, obj=self.user_2) self.grant_access(permission=permission_user_delete, obj=self.test_user)
response = self._request_user_multiple_delete_view() response = self._request_user_multiple_delete_view()
self.assertEqual(response.status_code, 302) self.assertEqual(response.status_code, 302)
self.assertEqual(get_user_model().objects.count(), 2) self.assertEqual(get_user_model().objects.count(), 2)
@@ -217,7 +217,8 @@ class MetadataLookupIntegrationTestCase(GenericDocumentViewTestCase):
) )
response = self.get( response = self.get(
viewname='metadata:metadata_edit', args=(self.document.pk,) viewname='metadata:document_metadata_edit',
kwargs={'pk': self.document.pk}
) )
self.assertContains( self.assertContains(
response=response, text='<option value="{}">{}</option>'.format( response=response, text='<option value="{}">{}</option>'.format(
@@ -234,7 +235,8 @@ class MetadataLookupIntegrationTestCase(GenericDocumentViewTestCase):
) )
response = self.get( response = self.get(
viewname='metadata:metadata_edit', args=(self.document.pk,) viewname='metadata:document_metadata_edit',
kwargs={'pk': self.document.pk}
) )
self.assertContains( self.assertContains(