diff --git a/mayan/apps/common/tests/test_compressed_files.py b/mayan/apps/common/tests/test_compressed_files.py index e004514c28..cc78d7f7ac 100644 --- a/mayan/apps/common/tests/test_compressed_files.py +++ b/mayan/apps/common/tests/test_compressed_files.py @@ -73,5 +73,3 @@ class TarGzArchiveClassTestCase(TarArchiveClassTestCase): class TarBz2ArchiveClassTestCase(TarArchiveClassTestCase): archive_path = TEST_TAR_BZ2_FILE_PATH cls = TarArchive - - diff --git a/mayan/apps/document_states/widgets.py b/mayan/apps/document_states/widgets.py index 0bd9dbaf57..87ae0a5f4b 100644 --- a/mayan/apps/document_states/widgets.py +++ b/mayan/apps/document_states/widgets.py @@ -3,6 +3,7 @@ from __future__ import unicode_literals from django import forms from django.urls import reverse from django.utils.html import format_html_join, mark_safe +from django.utils.translation import ugettext_lazy as _ def widget_transition_events(transition): diff --git a/mayan/apps/linking/tests/mixins.py b/mayan/apps/linking/tests/mixins.py index 5d482d6731..47af982c3b 100644 --- a/mayan/apps/linking/tests/mixins.py +++ b/mayan/apps/linking/tests/mixins.py @@ -10,6 +10,14 @@ from .literals import ( ) +class SmartLinkDocumentViewTestMixin(object): + def _request_test_smart_link_document_instances_view(self): + return self.get( + viewname='linking:smart_link_instances_for_document', + kwargs={'pk': self.test_document.pk} + ) + + class SmartLinkTestMixin(object): def _create_test_smart_link(self, add_test_document_type=False): self.test_smart_link = SmartLink.objects.create( diff --git a/mayan/apps/linking/tests/test_api.py b/mayan/apps/linking/tests/test_api.py index 5e78646e23..b7c0139d0b 100644 --- a/mayan/apps/linking/tests/test_api.py +++ b/mayan/apps/linking/tests/test_api.py @@ -22,19 +22,64 @@ from .literals import ( from .mixins import SmartLinkTestMixin -class SmartLinkAPITestCase(DocumentTestMixin, SmartLinkTestMixin, BaseAPITestCase): - auto_create_document_type = False - auto_upload_document = False - - def _request_smartlink_create_view(self): +class SmartLinkAPIViewTestMixin(object): + def _request_test_smart_link_create_api_view(self): return self.post( viewname='rest_api:smartlink-list', data={ 'label': TEST_SMART_LINK_LABEL } ) + def _request_test_smart_link_create_with_document_type_api_view(self): + return self.post( + viewname='rest_api:smartlink-list', data={ + 'label': TEST_SMART_LINK_LABEL, + 'document_types_pk_list': self.test_document_type.pk + }, + ) + + def _request_test_smart_link_delete_api_view(self): + return self.delete( + viewname='rest_api:smartlink-detail', kwargs={ + 'pk': self.test_smart_link.pk + } + ) + + def _request_test_smart_link_detail_api_view(self): + return self.get( + viewname='rest_api:smartlink-detail', kwargs={ + 'pk': self.test_smart_link.pk + } + ) + + def _request_test_smart_link_edit_patch_api_view(self): + return self.patch( + viewname='rest_api:smartlink-detail', + kwargs={'pk': self.test_smart_link.pk}, data={ + 'label': TEST_SMART_LINK_LABEL_EDITED, + 'document_types_pk_list': self.test_document_type.pk + } + ) + + def _request_test_smart_link_edit_put_api_view(self): + return self.put( + viewname='rest_api:smartlink-detail', + kwargs={'pk': self.test_smart_link.pk}, data={ + 'label': TEST_SMART_LINK_LABEL_EDITED, + 'document_types_pk_list': self.test_document_type.pk + } + ) + + +class SmartLinkAPIViewTestCase( + DocumentTestMixin, SmartLinkTestMixin, SmartLinkAPIViewTestMixin, + BaseAPITestCase +): + auto_create_document_type = False + auto_upload_document = False + def test_smart_link_create_view_no_permission(self): - response = self._request_smartlink_create_view() + response = self._request_test_smart_link_create_api_view() self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) self.assertEqual(SmartLink.objects.count(), 0) @@ -42,7 +87,7 @@ class SmartLinkAPITestCase(DocumentTestMixin, SmartLinkTestMixin, BaseAPITestCas def test_smart_link_create_view_with_permission(self): self.grant_permission(permission=permission_smart_link_create) - response = self._request_smartlink_create_view() + response = self._request_test_smart_link_create_api_view() self.assertEqual(response.status_code, status.HTTP_201_CREATED) smart_link = SmartLink.objects.first() @@ -52,18 +97,10 @@ class SmartLinkAPITestCase(DocumentTestMixin, SmartLinkTestMixin, BaseAPITestCas self.assertEqual(SmartLink.objects.count(), 1) self.assertEqual(smart_link.label, TEST_SMART_LINK_LABEL) - def _request_smart_link_create_with_document_type_view(self): - return self.post( - viewname='rest_api:smartlink-list', data={ - 'label': TEST_SMART_LINK_LABEL, - 'document_types_pk_list': self.test_document_type.pk - }, - ) - def test_smart_link_create_with_document_types_view_no_permission(self): self._create_document_type() - response = self._request_smart_link_create_with_document_type_view() + response = self._request_test_smart_link_create_with_document_type_api_view() self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) self.assertEqual(SmartLink.objects.count(), 0) @@ -72,7 +109,7 @@ class SmartLinkAPITestCase(DocumentTestMixin, SmartLinkTestMixin, BaseAPITestCas self._create_document_type() self.grant_permission(permission=permission_smart_link_create) - response = self._request_smart_link_create_with_document_type_view() + response = self._request_test_smart_link_create_with_document_type_api_view() self.assertEqual(response.status_code, status.HTTP_201_CREATED) smart_link = SmartLink.objects.first() @@ -81,21 +118,14 @@ class SmartLinkAPITestCase(DocumentTestMixin, SmartLinkTestMixin, BaseAPITestCas self.assertEqual(SmartLink.objects.count(), 1) self.assertEqual(smart_link.label, TEST_SMART_LINK_LABEL) - self.assertQuerysetEqual( - smart_link.document_types.all(), (repr(self.test_document_type),) - ) - - def _request_smart_link_delete_view(self): - return self.delete( - viewname='rest_api:smartlink-detail', kwargs={ - 'pk': self.test_smart_link.pk - } + self.assertTrue( + self.test_document_type in smart_link.document_types.all() ) def test_smart_link_delete_view_no_permission(self): self._create_test_smart_link() - response = self._request_smart_link_delete_view() + response = self._request_test_smart_link_delete_api_view() self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND) self.assertEqual(SmartLink.objects.count(), 1) @@ -106,22 +136,15 @@ class SmartLinkAPITestCase(DocumentTestMixin, SmartLinkTestMixin, BaseAPITestCas obj=self.test_smart_link, permission=permission_smart_link_delete ) - response = self._request_smart_link_delete_view() + response = self._request_test_smart_link_delete_api_view() self.assertEqual(response.status_code, status.HTTP_204_NO_CONTENT) self.assertEqual(SmartLink.objects.count(), 0) - def _request_smart_link_detail_view(self): - return self.get( - viewname='rest_api:smartlink-detail', kwargs={ - 'pk': self.test_smart_link.pk - } - ) - def test_smart_link_detail_view_no_permission(self): self._create_test_smart_link() - response = self._request_smart_link_detail_view() + response = self._request_test_smart_link_detail_api_view() self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND) self.assertFalse('label' in response.data) @@ -132,22 +155,13 @@ class SmartLinkAPITestCase(DocumentTestMixin, SmartLinkTestMixin, BaseAPITestCas obj=self.test_smart_link, permission=permission_smart_link_view ) - response = self._request_smart_link_detail_view() + response = self._request_test_smart_link_detail_api_view() self.assertEqual(response.status_code, status.HTTP_200_OK) self.assertEqual( response.data['label'], TEST_SMART_LINK_LABEL ) - def _request_test_smart_link_edit_patch_api_view(self): - return self.patch( - viewname='rest_api:smartlink-detail', - kwargs={'pk': self.test_smart_link.pk}, data={ - 'label': TEST_SMART_LINK_LABEL_EDITED, - 'document_types_pk_list': self.test_document_type.pk - } - ) - def test_smart_link_edit_view_via_patch_no_permission(self): self._create_document_type() self._create_test_smart_link() @@ -169,15 +183,8 @@ class SmartLinkAPITestCase(DocumentTestMixin, SmartLinkTestMixin, BaseAPITestCas self.assertEqual(response.status_code, status.HTTP_200_OK) self.test_smart_link.refresh_from_db() - self.assertEqual(self.test_smart_link.label, TEST_SMART_LINK_LABEL_EDITED) - - def _request_test_smart_link_edit_put_api_view(self): - return self.put( - viewname='rest_api:smartlink-detail', - kwargs={'pk': self.test_smart_link.pk}, data={ - 'label': TEST_SMART_LINK_LABEL_EDITED, - 'document_types_pk_list': self.test_document_type.pk - } + self.assertEqual( + self.test_smart_link.label, TEST_SMART_LINK_LABEL_EDITED ) def test_smart_link_edit_view_via_put_no_permission(self): @@ -201,14 +208,67 @@ class SmartLinkAPITestCase(DocumentTestMixin, SmartLinkTestMixin, BaseAPITestCas self.assertEqual(response.status_code, status.HTTP_200_OK) self.test_smart_link.refresh_from_db() - self.assertEqual(self.test_smart_link.label, TEST_SMART_LINK_LABEL_EDITED) + self.assertEqual( + self.test_smart_link.label, TEST_SMART_LINK_LABEL_EDITED + ) -class SmartLinkConditionAPITestCase(DocumentTestMixin, SmartLinkTestMixin, BaseAPITestCase): - def setUp(self): - super(SmartLinkConditionAPITestCase, self).setUp() - self._create_test_smart_link(add_test_document_type=True) +class SmartLinkConditionAPIViewTestMixin(object): + def _request_smart_link_condition_create_view(self): + return self.post( + viewname='rest_api:smartlinkcondition-list', + kwargs={'pk': self.test_smart_link.pk}, data={ + 'foreign_document_data': TEST_SMART_LINK_CONDITION_FOREIGN_DOCUMENT_DATA, + 'expression': TEST_SMART_LINK_CONDITION_EXPRESSION, + 'operator': TEST_SMART_LINK_CONDITION_OPERATOR + } + ) + def _request_smart_link_condition_delete_view(self): + return self.delete( + viewname='rest_api:smartlinkcondition-detail', + kwargs={ + 'pk': self.test_smart_link.pk, + 'condition_pk': self.test_smart_link_condition.pk + } + ) + + def _request_smart_link_condition_detail_view(self): + return self.get( + viewname='rest_api:smartlinkcondition-detail', + kwargs={ + 'pk': self.test_smart_link.pk, + 'condition_pk': self.test_smart_link_condition.pk + } + ) + + def _request_smart_link_condition_edit_view_via_patch(self): + return self.patch( + viewname='rest_api:smartlinkcondition-detail', + kwargs={ + 'pk': self.test_smart_link.pk, + 'condition_pk': self.test_smart_link_condition.pk + }, data={ + 'expression': TEST_SMART_LINK_CONDITION_EXPRESSION_EDITED, + } + ) + + def _request_smart_link_condition_edit_view_via_put(self): + return self.put( + viewname='rest_api:smartlinkcondition-detail', + kwargs={ + 'pk': self.test_smart_link.pk, + 'condition_pk': self.test_smart_link_condition.pk + }, data={ + 'expression': TEST_SMART_LINK_CONDITION_EXPRESSION_EDITED, + 'foreign_document_data': TEST_SMART_LINK_CONDITION_FOREIGN_DOCUMENT_DATA, + 'operator': TEST_SMART_LINK_CONDITION_OPERATOR + + } + ) + + +class ResolvedSmartLinkAPIViewTestMixin(object): def _request_resolved_smart_link_detail_view(self): return self.get( viewname='rest_api:resolvedsmartlink-detail', @@ -219,6 +279,163 @@ class SmartLinkConditionAPITestCase(DocumentTestMixin, SmartLinkTestMixin, BaseA ) self._create_test_smart_link(add_test_document_type=True) + def _request_resolved_smart_link_list_view(self): + return self.get( + viewname='rest_api:resolvedsmartlink-list', kwargs={ + 'pk': self.test_document.pk + } + ) + + def _request_resolved_smart_link_document_list_view(self): + return self.get( + viewname='rest_api:resolvedsmartlinkdocument-list', + kwargs={ + 'pk': self.test_document.pk, + 'smart_link_pk': self.test_smart_link.pk + } + ) + + +class SmartLinkConditionAPIViewTestCase( + DocumentTestMixin, SmartLinkTestMixin, + SmartLinkConditionAPIViewTestMixin, BaseAPITestCase +): + auto_upload_document = False + + def setUp(self): + super(SmartLinkConditionAPIViewTestCase, self).setUp() + self._create_test_smart_link(add_test_document_type=True) + + def test_smart_link_condition_create_view_no_permission(self): + response = self._request_smart_link_condition_create_view() + self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) + self.assertFalse('id' in response.data) + + def test_smart_link_condition_create_view_with_access(self): + self.grant_access( + obj=self.test_smart_link, permission=permission_smart_link_edit + ) + + response = self._request_smart_link_condition_create_view() + self.assertEqual(response.status_code, status.HTTP_201_CREATED) + + smart_link_condition = SmartLinkCondition.objects.first() + self.assertEqual(response.data['id'], smart_link_condition.pk) + self.assertEqual( + response.data['operator'], TEST_SMART_LINK_CONDITION_OPERATOR + ) + + self.assertEqual(SmartLinkCondition.objects.count(), 1) + self.assertEqual( + smart_link_condition.operator, TEST_SMART_LINK_CONDITION_OPERATOR + ) + + def test_smart_link_condition_delete_view_no_permission(self): + self._create_test_smart_link_condition() + + response = self._request_smart_link_condition_delete_view() + self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) + + self.assertEqual(SmartLinkCondition.objects.count(), 1) + + def test_smart_link_condition_delete_view_with_access(self): + self._create_test_smart_link_condition() + + self.grant_access( + obj=self.test_smart_link, permission=permission_smart_link_edit + ) + + response = self._request_smart_link_condition_delete_view() + self.assertEqual(response.status_code, status.HTTP_204_NO_CONTENT) + + self.assertEqual(SmartLinkCondition.objects.count(), 0) + + def test_smart_link_condition_detail_view_no_permission(self): + self._create_test_smart_link_condition() + + response = self._request_smart_link_condition_detail_view() + self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) + + self.assertFalse('id' in response.data) + + def test_smart_link_condition_detail_view_with_access(self): + self._create_test_smart_link_condition() + + self.grant_access( + obj=self.test_smart_link, permission=permission_smart_link_view + ) + + response = self._request_smart_link_condition_detail_view() + self.assertEqual(response.status_code, status.HTTP_200_OK) + self.assertEqual( + response.data['operator'], TEST_SMART_LINK_CONDITION_OPERATOR + ) + + def test_smart_link_condition_patch_view_no_permission(self): + self._create_test_smart_link_condition() + + response = self._request_smart_link_condition_edit_view_via_patch() + self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) + + self.test_smart_link_condition.refresh_from_db() + self.assertEqual( + self.test_smart_link_condition.expression, + TEST_SMART_LINK_CONDITION_EXPRESSION + ) + + def test_smart_link_condition_patch_view_with_access(self): + self._create_test_smart_link_condition() + + self.grant_access( + obj=self.test_smart_link, permission=permission_smart_link_edit + ) + + response = self._request_smart_link_condition_edit_view_via_patch() + self.assertEqual(response.status_code, status.HTTP_200_OK) + + self.test_smart_link_condition.refresh_from_db() + self.assertEqual( + self.test_smart_link_condition.expression, + TEST_SMART_LINK_CONDITION_EXPRESSION_EDITED + ) + + def test_smart_link_condition_put_view_no_permission(self): + self._create_test_smart_link_condition() + + response = self._request_smart_link_condition_edit_view_via_put() + self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) + + self.test_smart_link_condition.refresh_from_db() + self.assertEqual( + self.test_smart_link_condition.expression, + TEST_SMART_LINK_CONDITION_EXPRESSION + ) + + def test_smart_link_condition_put_view_with_access(self): + self._create_test_smart_link_condition() + + self.grant_access( + obj=self.test_smart_link, permission=permission_smart_link_edit + ) + + response = self._request_smart_link_condition_edit_view_via_put() + self.assertEqual(response.status_code, status.HTTP_200_OK) + + self.test_smart_link_condition.refresh_from_db() + self.assertEqual( + self.test_smart_link_condition.expression, + TEST_SMART_LINK_CONDITION_EXPRESSION_EDITED + ) + + +class ResolvedSmartLinkAPIViewTestCase( + DocumentTestMixin, SmartLinkTestMixin, + ResolvedSmartLinkAPIViewTestMixin, BaseAPITestCase +): + def setUp(self): + super(ResolvedSmartLinkAPIViewTestCase, self).setUp() + self._create_test_smart_link(add_test_document_type=True) + def test_resolved_smart_link_detail_view_no_permission(self): self._create_test_smart_link_condition() @@ -262,13 +479,6 @@ class SmartLinkConditionAPITestCase(DocumentTestMixin, SmartLinkTestMixin, BaseA response.data['label'], TEST_SMART_LINK_LABEL ) - def _request_resolved_smart_link_list_view(self): - return self.get( - viewname='rest_api:resolvedsmartlink-list', kwargs={ - 'pk': self.test_document.pk - } - ) - def test_resolved_smart_link_list_view_no_permission(self): self._create_test_smart_link_condition() @@ -316,15 +526,6 @@ class SmartLinkConditionAPITestCase(DocumentTestMixin, SmartLinkTestMixin, BaseA response.data['results'][0]['label'], TEST_SMART_LINK_LABEL ) - def _request_resolved_smart_link_document_list_view(self): - return self.get( - viewname='rest_api:resolvedsmartlinkdocument-list', - kwargs={ - 'pk': self.test_document.pk, - 'smart_link_pk': self.test_smart_link.pk - } - ) - def test_resolved_smart_link_document_list_view_no_permission(self): self._create_test_smart_link_condition() @@ -366,178 +567,3 @@ class SmartLinkConditionAPITestCase(DocumentTestMixin, SmartLinkTestMixin, BaseA self.assertEqual( response.data['results'][0]['label'], self.test_document.label ) - - def _request_smart_link_condition_create_view(self): - return self.post( - viewname='rest_api:smartlinkcondition-list', - kwargs={'pk': self.test_smart_link.pk}, data={ - 'foreign_document_data': TEST_SMART_LINK_CONDITION_FOREIGN_DOCUMENT_DATA, - 'expression': TEST_SMART_LINK_CONDITION_EXPRESSION, - 'operator': TEST_SMART_LINK_CONDITION_OPERATOR - } - ) - - def test_smart_link_condition_create_view_no_permission(self): - - response = self._request_smart_link_condition_create_view() - self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) - self.assertFalse('id' in response.data) - - def test_smart_link_condition_create_view_with_access(self): - self.grant_access( - obj=self.test_smart_link, permission=permission_smart_link_edit - ) - - response = self._request_smart_link_condition_create_view() - self.assertEqual(response.status_code, status.HTTP_201_CREATED) - - smart_link_condition = SmartLinkCondition.objects.first() - self.assertEqual(response.data['id'], smart_link_condition.pk) - self.assertEqual( - response.data['operator'], TEST_SMART_LINK_CONDITION_OPERATOR - ) - - self.assertEqual(SmartLinkCondition.objects.count(), 1) - self.assertEqual( - smart_link_condition.operator, TEST_SMART_LINK_CONDITION_OPERATOR - ) - - def _request_smart_link_condition_delete_view(self): - return self.delete( - viewname='rest_api:smartlinkcondition-detail', - kwargs={ - 'pk': self.test_smart_link.pk, - 'condition_pk': self.test_smart_link_condition.pk - } - ) - - def test_smart_link_condition_delete_view_no_permission(self): - self._create_test_smart_link_condition() - - response = self._request_smart_link_condition_delete_view() - self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) - - self.assertEqual(SmartLinkCondition.objects.count(), 1) - - def test_smart_link_condition_delete_view_with_access(self): - self._create_test_smart_link_condition() - - self.grant_access( - obj=self.test_smart_link, permission=permission_smart_link_edit - ) - - response = self._request_smart_link_condition_delete_view() - self.assertEqual(response.status_code, status.HTTP_204_NO_CONTENT) - - self.assertEqual(SmartLinkCondition.objects.count(), 0) - - def _request_smart_link_condition_detail_view(self): - return self.get( - viewname='rest_api:smartlinkcondition-detail', - kwargs={ - 'pk': self.test_smart_link.pk, - 'condition_pk': self.test_smart_link_condition.pk - } - ) - - def test_smart_link_condition_detail_view_no_permission(self): - self._create_test_smart_link_condition() - - response = self._request_smart_link_condition_detail_view() - self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) - - self.assertFalse('id' in response.data) - - def test_smart_link_condition_detail_view_with_access(self): - self._create_test_smart_link_condition() - - self.grant_access( - obj=self.test_smart_link, permission=permission_smart_link_view - ) - - response = self._request_smart_link_condition_detail_view() - self.assertEqual(response.status_code, status.HTTP_200_OK) - self.assertEqual( - response.data['operator'], TEST_SMART_LINK_CONDITION_OPERATOR - ) - - def _request_smart_link_condition_edit_view_via_patch(self): - return self.patch( - viewname='rest_api:smartlinkcondition-detail', - kwargs={ - 'pk': self.test_smart_link.pk, - 'condition_pk': self.test_smart_link_condition.pk - }, data={ - 'expression': TEST_SMART_LINK_CONDITION_EXPRESSION_EDITED, - } - ) - - def test_smart_link_condition_patch_view_no_permission(self): - self._create_test_smart_link_condition() - - response = self._request_smart_link_condition_edit_view_via_patch() - self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) - - self.test_smart_link_condition.refresh_from_db() - self.assertEqual( - self.test_smart_link_condition.expression, - TEST_SMART_LINK_CONDITION_EXPRESSION - ) - - def test_smart_link_condition_patch_view_with_access(self): - self._create_test_smart_link_condition() - - self.grant_access( - obj=self.test_smart_link, permission=permission_smart_link_edit - ) - - response = self._request_smart_link_condition_edit_view_via_patch() - self.assertEqual(response.status_code, status.HTTP_200_OK) - - self.test_smart_link_condition.refresh_from_db() - self.assertEqual( - self.test_smart_link_condition.expression, - TEST_SMART_LINK_CONDITION_EXPRESSION_EDITED - ) - - def _request_smart_link_condition_edit_view_via_put(self): - return self.put( - viewname='rest_api:smartlinkcondition-detail', - kwargs={ - 'pk': self.test_smart_link.pk, - 'condition_pk': self.test_smart_link_condition.pk - }, data={ - 'expression': TEST_SMART_LINK_CONDITION_EXPRESSION_EDITED, - 'foreign_document_data': TEST_SMART_LINK_CONDITION_FOREIGN_DOCUMENT_DATA, - 'operator': TEST_SMART_LINK_CONDITION_OPERATOR - - } - ) - - def test_smart_link_condition_put_view_no_permission(self): - self._create_test_smart_link_condition() - - response = self._request_smart_link_condition_edit_view_via_put() - self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN) - - self.test_smart_link_condition.refresh_from_db() - self.assertEqual( - self.test_smart_link_condition.expression, - TEST_SMART_LINK_CONDITION_EXPRESSION - ) - - def test_smart_link_condition_put_view_with_access(self): - self._create_test_smart_link_condition() - - self.grant_access( - obj=self.test_smart_link, permission=permission_smart_link_edit - ) - - response = self._request_smart_link_condition_edit_view_via_put() - self.assertEqual(response.status_code, status.HTTP_200_OK) - - self.test_smart_link_condition.refresh_from_db() - self.assertEqual( - self.test_smart_link_condition.expression, - TEST_SMART_LINK_CONDITION_EXPRESSION_EDITED - ) diff --git a/mayan/apps/linking/tests/test_views.py b/mayan/apps/linking/tests/test_views.py index 1fa62f79a7..3b446b4ef4 100644 --- a/mayan/apps/linking/tests/test_views.py +++ b/mayan/apps/linking/tests/test_views.py @@ -14,10 +14,15 @@ from .literals import ( TEST_SMART_LINK_DYNAMIC_LABEL, TEST_SMART_LINK_LABEL_EDITED, TEST_SMART_LINK_LABEL ) -from .mixins import SmartLinkTestMixin, SmartLinkViewTestMixin +from .mixins import ( + SmartLinkDocumentViewTestMixin, SmartLinkTestMixin, + SmartLinkViewTestMixin +) -class SmartLinkViewTestCase(SmartLinkTestMixin, SmartLinkViewTestMixin, GenericViewTestCase): +class SmartLinkViewTestCase( + SmartLinkTestMixin, SmartLinkViewTestMixin, GenericViewTestCase +): def test_smart_link_create_view_no_permission(self): response = self._request_test_smart_link_create_view() self.assertEqual(response.status_code, 403) @@ -74,10 +79,15 @@ class SmartLinkViewTestCase(SmartLinkTestMixin, SmartLinkViewTestMixin, GenericV self.assertEqual(response.status_code, 302) self.test_smart_link.refresh_from_db() - self.assertEqual(self.test_smart_link.label, TEST_SMART_LINK_LABEL_EDITED) + self.assertEqual( + self.test_smart_link.label, TEST_SMART_LINK_LABEL_EDITED + ) -class SmartLinkDocumentViewTestCase(SmartLinkTestMixin, GenericDocumentViewTestCase): +class SmartLinkDocumentViewTestCase( + SmartLinkTestMixin, SmartLinkDocumentViewTestMixin, + GenericDocumentViewTestCase +): def setUp(self): super(SmartLinkDocumentViewTestCase, self).setUp() self._create_test_smart_link() @@ -89,12 +99,6 @@ class SmartLinkDocumentViewTestCase(SmartLinkTestMixin, GenericDocumentViewTestC ) self.test_smart_link_2.document_types.add(self.test_document_type) - def _request_test_smart_link_document_instances_view(self): - return self.get( - viewname='linking:smart_link_instances_for_document', - kwargs={'pk': self.test_document.pk} - ) - def test_document_smart_link_list_view_no_permission(self): self.grant_access( obj=self.test_document, permission=permission_document_view