Initial commit to support QUnit tests

Signed-off-by: Roberto Rosario <roberto.rosario@mayan-edms.com>

HISTORY.rst

@@ -17,7 +17,6 @@
   of selection of documents.
 - Add parsed content deleted event.
 - Allow scaling of UI on mobile devices.
-- Add Chinese fonts to the Docker image
 
 3.2.7 (2019-08-28)
 ==================

docker/Dockerfile

@@ -23,8 +23,6 @@ RUN set -x \
 apt-get update \
 && apt-get install -y --no-install-recommends \
     exiftool \
-    fonts-arphic-uming \
-    fonts-arphic-ukai \
     ghostscript \
     gpgv \
     gnupg1 \
@@ -62,6 +60,7 @@ echo "save \"\"" >> /etc/redis/redis.conf \
 # Only provision 1 database
 && echo "databases 1" >> /etc/redis/redis.conf
 
+
 ####
 # BUILDER_IMAGE - This image buildS the Python package and is discarded afterwards
 ####

docs/releases/3.2.8.rst

@@ -33,7 +33,6 @@ Other changes
   of selection of documents.
 - Add parsed content deleted event.
 - Allow scaling of UI on mobile devices.
-- Add Chinese fonts to the Docker image
 
 
 Removals
@@ -127,9 +126,7 @@ Backward incompatible changes
 Bugs fixed or issues closed
 ---------------------------
 
-- :gitlab-issue:`666` Chinese document such as .doc can't display well.
 - :gitlab-issue:`668` Permission denied errors with custom uid persist (650 needs re-open)
 - :forum-topic:`1120` Cabinet Presentation
-- :forum-topic:`2202` Cannot display Chinese character and cannot identify Excel files
 
 .. _PyPI: https://pypi.python.org/pypi/mayan-edms/

mayan/apps/appearance/static/appearance/css/base.css

@@ -418,4 +418,5 @@ a i {
     font: 11px Verdana, sans-serif;
     padding: 0px;
     padding-bottom: 10px; /* Padding for scrollbar */
+    border-radius: 5px;
 }

mayan/apps/appearance/static/appearance/js/test/unit/partial_navigation.js

@@ -0,0 +1,31 @@
+'use strict';
+
+QUnit.test('partialNavigation.filterLocation', function (assert) {
+    var testPartialNavigation = new PartialNavigation({
+        initialURL: '/testInitialURL',
+    });
+
+    /*
+     * For an empty newLocation we expect the fragment of the URL minus the
+     * query
+     */
+    var expected = new URI(new URI(location).fragment()).path().toString();
+    assert.strictEqual(
+        testPartialNavigation.filterLocation(''), expected, 'newLocation === ""');
+
+    /*
+     * For an empty root value we expect initialURL passed to the
+     * partialNavigation instance when initialized.
+     */
+    assert.strictEqual(
+        testPartialNavigation.filterLocation('/'), testPartialNavigation.initialURL, 'newLocation === "/"'
+    );
+
+    /*
+     * For an empty root value we expect initialURL passed to the
+     * partialNavigation instance when initialized.
+     */
+    assert.strictEqual(
+        testPartialNavigation.filterLocation('random'), 'random', 'newLocation === "random"'
+    );
+});

mayan/apps/appearance/templates/appearance/base_plain.html

@@ -38,6 +38,7 @@
         <script src="{% static 'appearance/node_modules/jquery/dist/jquery.min.js' %}" type="text/javascript"></script>
         <script src="{% static 'appearance/node_modules/bootstrap/dist/js/bootstrap.min.js' %}" type="text/javascript"></script>
         <script src="{% static 'appearance/node_modules/@fortawesome/fontawesome-free/js/all.min.js' %}" data-auto-replace-svg="nest" type="text/javascript"></script>
+        {% block javascript %}{% endblock %}
     </body>
 </html>
 {% endspaceless %}

mayan/apps/cabinets/templates/cabinets/cabinet_details.html

@@ -13,7 +13,7 @@
 
 {% block content %}
     <div class="row">
-        <div class="col-xs-12 col-sm-12 col-md-12 col-lg-2" style="margin-right: -25px;">{# Remove gutter #}
+        <div class="col-xs-12 col-sm-12 col-md-12 col-lg-2">
             <h4>{% trans 'Navigation:' %}</h4>
             <div id="jstree"></div>
         </div>

mayan/apps/common/compressed_files.py

@@ -11,7 +11,6 @@ except ImportError:
     COMPRESSION = zipfile.ZIP_STORED
 
 from django.core.files.uploadedfile import SimpleUploadedFile
-from django.utils.encoding import force_text
 
 from mayan.apps.mimetype.api import get_mimetype
 
@@ -137,32 +136,9 @@ class ZipArchive(Archive):
         return self._archive.read(filename)
 
     def members(self):
-        results = []
+        return [
-        from django.utils.encoding import force_str
+            filename for filename in self._archive.namelist() if not filename.endswith('/')
-
+        ]
-        for filename in self._archive.namelist():
-            # Zip files only support UTF-8 and CP437 encodings.
-            # Attempt to decode CP437 to be able to check if it ends
-            # with a slash.
-            # Future improvement that violates the Zip format:
-            # Add chardet.detect to detect the most likely encoding
-            # if other than CP437.
-            try:
-                filename = filename.decode('CP437')
-                is_unicode = False
-            except UnicodeEncodeError:
-                is_unicode = True
-
-            if not filename.endswith('/'):
-                # Re encode in the original encoding
-                if not is_unicode:
-                    filename = filename.encode(
-                        encoding='CP437', errors='strict'
-                    )
-
-                results.append(filename)
-
-        return results
 
     def open_member(self, filename):
         return self._archive.open(filename)

mayan/apps/common/dependencies.py

@@ -4,7 +4,12 @@ from django.utils.translation import ugettext_lazy as _
 
 from mayan.apps.dependencies.classes import (
     environment_build, environment_development, environment_testing,
-    PythonDependency
+    JavaScriptDependency, PythonDependency
+)
+
+JavaScriptDependency(
+    environment=environment_testing, label=_('QUnit'), module=__name__,
+    name='qunit', version_string='=2.9.2'
 )
 
 PythonDependency(

mayan/apps/common/templates/common/qunit.html

@@ -0,0 +1,22 @@
+{% extends 'appearance/base.html' %}
+
+{% load i18n %}
+{% load static %}
+
+{% block base_title %}{% trans 'Qunit' %}{% endblock %}
+
+{% block project_name %}{% endblock %}
+
+{% block stylesheets %}
+    <link href="{% static 'common/node_modules/qunit/qunit/qunit.css' %}" media="screen" rel="stylesheet" type="text/css" />
+{% endblock %}
+
+{% block content %}
+    <div id="qunit"></div>
+    <div id="qunit-fixture"></div>
+{% endblock %}
+
+{% block javascript %}
+    <script src="{% static 'common/node_modules/qunit/qunit/qunit.js' %}" type="text/javascript"></script>
+    <script src="{% static 'appearance/js/test/unit/partial_navigation.js' %}" type="text/javascript"></script>
+{% endblock %}

mayan/apps/common/tests/literals.py

@@ -12,7 +12,6 @@ TEST_VIEW_URL = 'test-view-url'
 
 # Filenames
 TEST_ARCHIVE_ZIP_SPECIAL_CHARACTERS_FILENAME_MEMBER = 'test_archvive_with_special_characters_filename_member.zip'
-TEST_ARCHIVE_ZIP_CP437_MEMBER = 'test_archvive_with_cp437_member.zip'
 TEST_FILENAME1 = 'test_file1.txt'
 TEST_FILENAME2 = 'test_file2.txt'
 TEST_FILENAME3 = 'test_file3.txt'
@@ -29,10 +28,6 @@ TEST_ARCHIVE_ZIP_SPECIAL_CHARACTERS_FILENAME_MEMBER_PATH = os.path.join(
     settings.BASE_DIR, 'apps', 'common', 'tests', 'contrib',
     TEST_ARCHIVE_ZIP_SPECIAL_CHARACTERS_FILENAME_MEMBER
 )
-TEST_ARCHIVE_ZIP_CP437_MEMBER_PATH = os.path.join(
-    settings.BASE_DIR, 'apps', 'common', 'tests', 'contrib',
-    TEST_ARCHIVE_ZIP_CP437_MEMBER
-)
 TEST_FILE3_PATH = os.path.join(
     settings.BASE_DIR, 'apps', 'common', 'tests', 'contrib', TEST_FILENAME3
 )

mayan/apps/common/tests/test_compressed_files.py

@@ -5,7 +5,6 @@ from mayan.apps.common.tests import BaseTestCase
 from ..compressed_files import Archive, TarArchive, ZipArchive
 
 from .literals import (
-    TEST_ARCHIVE_ZIP_CP437_MEMBER_PATH,
     TEST_ARCHIVE_ZIP_SPECIAL_CHARACTERS_FILENAME_MEMBER_PATH,
     TEST_COMPRESSED_FILE_CONTENTS, TEST_FILE_CONTENTS_1, TEST_FILE3_PATH,
     TEST_FILENAME1, TEST_FILENAME3, TEST_TAR_BZ2_FILE_PATH,
@@ -65,11 +64,6 @@ class ZipArchiveClassTestCase(TarArchiveClassTestCase):
             archive = Archive.open(file_object=file_object)
             list(archive.get_members())
 
-    def test_open_cp437_member(self):
-        with open(TEST_ARCHIVE_ZIP_CP437_MEMBER_PATH, mode='rb') as file_object:
-            archive = Archive.open(file_object=file_object)
-            list(archive.get_members())
-
 
 class TarGzArchiveClassTestCase(TarArchiveClassTestCase):
     archive_path = TEST_TAR_GZ_FILE_PATH

mayan/apps/common/urls.py

@@ -10,7 +10,8 @@ from .views import (
     AboutView, CurrentUserLocaleProfileDetailsView,
     CurrentUserLocaleProfileEditView, FaviconRedirectView, HomeView,
     LicenseView, ObjectErrorLogEntryListClearView, ObjectErrorLogEntryListView,
-    RootView, SetupListView, ToolsListView, multi_object_action_view
+    RootView, SetupListView, ToolsListView, QUnitView,
+    multi_object_action_view
 )
 
 urlpatterns = [
@@ -43,6 +44,9 @@ urlpatterns = [
         view=ObjectErrorLogEntryListClearView.as_view(),
         name='object_error_list_clear'
     ),
+    url(
+        regex=r'^qunit/', view=QUnitView.as_view(), name='qunit'
+    )
 ]
 
 urlpatterns += [

mayan/apps/common/views.py

@@ -276,3 +276,8 @@ def multi_object_action_view(request):
                 action, urlencode({'id_list': id_list, 'next': next})
             )
         )
+
+
+class QUnitView(SimpleView):
+    extra_context = {'title': _('QUnit tests')}
+    template_name = 'common/qunit.html'

mayan/apps/django_gpg/tests/test_api.py

@@ -13,28 +13,19 @@ from .literals import TEST_KEY_DATA, TEST_KEY_FINGERPRINT
 from .mixins import KeyTestMixin
 
 
-class KeyAPIViewTestMixin(object):
+class KeyAPITestCase(KeyTestMixin, BaseAPITestCase):
-    def _request_test_key_create_view(self):
+
+    # Key creation by upload
+
+    def _request_key_create_view(self):
         return self.post(
             viewname='rest_api:key-list', data={
                 'key_data': TEST_KEY_DATA
             }
         )
 
-    def _request_test_key_delete_view(self):
-        return self.delete(
-            viewname='rest_api:key-detail', kwargs={'pk': self.test_key.pk}
-        )
-
-    def _request_test_key_detail_view(self):
-        return self.get(
-            viewname='rest_api:key-detail', kwargs={'pk': self.test_key.pk}
-        )
-
-
-class KeyAPITestCase(KeyTestMixin, KeyAPIViewTestMixin, BaseAPITestCase):
     def test_key_create_view_no_permission(self):
-        response = self._request_test_key_create_view()
+        response = self._request_key_create_view()
         self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
 
         self.assertEqual(Key.objects.all().count(), 0)
@@ -42,7 +33,7 @@ class KeyAPITestCase(KeyTestMixin, KeyAPIViewTestMixin, BaseAPITestCase):
     def test_key_create_view_with_permission(self):
         self.grant_permission(permission=permission_key_upload)
 
-        response = self._request_test_key_create_view()
+        response = self._request_key_create_view()
         self.assertEqual(response.status_code, status.HTTP_201_CREATED)
         self.assertEqual(response.data['fingerprint'], TEST_KEY_FINGERPRINT)
 
@@ -50,10 +41,17 @@ class KeyAPITestCase(KeyTestMixin, KeyAPIViewTestMixin, BaseAPITestCase):
         self.assertEqual(Key.objects.count(), 1)
         self.assertEqual(key.fingerprint, TEST_KEY_FINGERPRINT)
 
+    # Key deletion
+
+    def _request_key_delete_view(self):
+        return self.delete(
+            viewname='rest_api:key-detail', kwargs={'pk': self.test_key.pk}
+        )
+
     def test_key_delete_view_no_access(self):
         self._create_test_key()
 
-        response = self._request_test_key_delete_view()
+        response = self._request_key_delete_view()
         self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND)
 
         self.assertEqual(Key.objects.count(), 1)
@@ -64,15 +62,22 @@ class KeyAPITestCase(KeyTestMixin, KeyAPIViewTestMixin, BaseAPITestCase):
             obj=self.test_key, permission=permission_key_delete
         )
 
-        response = self._request_test_key_delete_view()
+        response = self._request_key_delete_view()
         self.assertEqual(response.status_code, status.HTTP_204_NO_CONTENT)
 
         self.assertEqual(Key.objects.count(), 0)
 
+    # Key detail
+
+    def _request_key_detail_view(self):
+        return self.get(
+            viewname='rest_api:key-detail', kwargs={'pk': self.test_key.pk}
+        )
+
     def test_key_detail_view_no_access(self):
         self._create_test_key()
 
-        response = self._request_test_key_detail_view()
+        response = self._request_key_detail_view()
         self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND)
 
     def test_key_detail_view_with_access(self):
@@ -81,7 +86,7 @@ class KeyAPITestCase(KeyTestMixin, KeyAPIViewTestMixin, BaseAPITestCase):
             obj=self.test_key, permission=permission_key_view
         )
 
-        response = self._request_test_key_detail_view()
+        response = self._request_key_detail_view()
         self.assertEqual(response.status_code, status.HTTP_200_OK)
         self.assertEqual(
             response.data['fingerprint'], self.test_key.fingerprint

mayan/apps/django_gpg/tests/test_views.py

@@ -11,23 +11,13 @@ from .literals import TEST_KEY_DATA, TEST_KEY_FINGERPRINT
 from .mixins import KeyTestMixin
 
 
-class KeyViewTestMixin(object):
+class KeyViewTestCase(KeyTestMixin, GenericViewTestCase):
-    def _request_test_key_download_view(self):
-        return self.get(
-            viewname='django_gpg:key_download', kwargs={'pk': self.test_key.pk}
-        )
-
-    def _request_test_key_upload_view(self):
-        return self.post(
-            viewname='django_gpg:key_upload', data={'key_data': TEST_KEY_DATA}
-        )
-
-
-class KeyViewTestCase(KeyTestMixin, KeyViewTestMixin, GenericViewTestCase):
     def test_key_download_view_no_permission(self):
         self._create_test_key()
 
-        response = self._request_test_key_download_view()
+        response = self.get(
+            viewname='django_gpg:key_download', kwargs={'pk': self.test_key.pk}
+        )
         self.assertEqual(response.status_code, 403)
 
     def test_key_download_view_with_permission(self):
@@ -35,18 +25,20 @@ class KeyViewTestCase(KeyTestMixin, KeyViewTestMixin, GenericViewTestCase):
 
         self._create_test_key()
 
-        self.grant_access(
+        self.grant_access(obj=self.test_key, permission=permission_key_download)
-            obj=self.test_key, permission=permission_key_download
-        )
 
-        response = self._request_test_key_download_view()
+        response = self.get(
+            viewname='django_gpg:key_download', kwargs={'pk': self.test_key.pk}
+        )
         assert_download_response(
             self, response=response, content=self.test_key.key_data,
             basename=self.test_key.key_id,
         )
 
     def test_key_upload_view_no_permission(self):
-        response = self._request_test_key_upload_view()
+        response = self.post(
+            viewname='django_gpg:key_upload', data={'key_data': TEST_KEY_DATA}
+        )
         self.assertEqual(response.status_code, 403)
 
         self.assertEqual(Key.objects.count(), 0)
@@ -54,10 +46,12 @@ class KeyViewTestCase(KeyTestMixin, KeyViewTestMixin, GenericViewTestCase):
     def test_key_upload_view_with_permission(self):
         self.grant_permission(permission=permission_key_upload)
 
-        response = self._request_test_key_upload_view()
+        response = self.post(
-        self.assertEqual(response.status_code, 302)
+            viewname='django_gpg:key_upload', data={'key_data': TEST_KEY_DATA},
+            follow=True
+        )
+
+        self.assertContains(response=response, text='created', status_code=200)
 
         self.assertEqual(Key.objects.count(), 1)
-        self.assertEqual(
+        self.assertEqual(Key.objects.first().fingerprint, TEST_KEY_FINGERPRINT)
-            Key.objects.first().fingerprint, TEST_KEY_FINGERPRINT
-        )

mayan/apps/document_comments/tests/test_api.py

@@ -15,7 +15,9 @@ from .literals import TEST_COMMENT_TEXT, TEST_COMMENT_TEXT_EDITED
 from .mixins import DocumentCommentTestMixin
 
 
-class CommentAPIViewTestMixin(object):
+class CommentAPITestCase(
+    DocumentCommentTestMixin, DocumentTestMixin, BaseAPITestCase
+):
     def _request_test_comment_create_api_view(self):
         return self.post(
             viewname='rest_api:comment-list', kwargs={
@@ -25,42 +27,6 @@ class CommentAPIViewTestMixin(object):
             }
         )
 
-    def _request_test_comment_delete_api_view(self):
-        return self.delete(
-            viewname='rest_api:comment-detail', kwargs={
-                'document_pk': self.test_document.pk,
-                'comment_pk': self.test_document_comment.pk,
-            }
-        )
-
-    def _request_test_comment_detail_api_view(self):
-        return self.get(
-            viewname='rest_api:comment-detail', kwargs={
-                'document_pk': self.test_document.pk,
-                'comment_pk': self.test_document_comment.pk
-            }
-        )
-
-    def _request_test_comment_edit_patch_api_view(self):
-        return self.patch(
-            viewname='rest_api:comment-detail', kwargs={
-                'document_pk': self.test_document.pk,
-                'comment_pk': self.test_document_comment.pk,
-            }, data={'comment': TEST_COMMENT_TEXT_EDITED}
-        )
-
-    def _request_test_comment_list_api_view(self):
-        return self.get(
-            viewname='rest_api:comment-list', kwargs={
-                'document_pk': self.test_document.pk
-            }
-        )
-
-
-class CommentAPIViewTestCase(
-    CommentAPIViewTestMixin, DocumentCommentTestMixin, DocumentTestMixin,
-    BaseAPITestCase
-):
     def test_comment_create_view_no_access(self):
         response = self._request_test_comment_create_api_view()
         self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
@@ -79,6 +45,14 @@ class CommentAPIViewTestCase(
         self.assertEqual(Comment.objects.count(), 1)
         self.assertEqual(response.data['id'], comment.pk)
 
+    def _request_test_comment_delete_api_view(self):
+        return self.delete(
+            viewname='rest_api:comment-detail', kwargs={
+                'document_pk': self.test_document.pk,
+                'comment_pk': self.test_document_comment.pk,
+            }
+        )
+
     def test_comment_delete_view_no_access(self):
         self._create_test_comment()
 
@@ -98,11 +72,19 @@ class CommentAPIViewTestCase(
 
         self.assertFalse(self.test_document_comment in Comment.objects.all())
 
+    def _request_comment_edit_patch_api_view(self):
+        return self.patch(
+            viewname='rest_api:comment-detail', kwargs={
+                'document_pk': self.test_document.pk,
+                'comment_pk': self.test_document_comment.pk,
+            }, data={'comment': TEST_COMMENT_TEXT_EDITED}
+        )
+
     def test_comment_edit_view_no_access(self):
         self._create_test_comment()
         comment_text = self.test_document_comment.comment
 
-        response = self._request_test_comment_edit_patch_api_view()
+        response = self._request_comment_edit_patch_api_view()
         self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
 
         self.test_document_comment.refresh_from_db()
@@ -115,16 +97,24 @@ class CommentAPIViewTestCase(
         )
         comment_text = self.test_document_comment.comment
 
-        response = self._request_test_comment_edit_patch_api_view()
+        response = self._request_comment_edit_patch_api_view()
         self.assertEqual(response.status_code, status.HTTP_200_OK)
 
         self.test_document_comment.refresh_from_db()
         self.assertNotEqual(self.test_document_comment.comment, comment_text)
 
+    def _request_test_comment_api_view(self):
+        return self.get(
+            viewname='rest_api:comment-detail', kwargs={
+                'document_pk': self.test_document.pk,
+                'comment_pk': self.test_document_comment.pk
+            }
+        )
+
     def test_comment_detail_view_no_access(self):
         self._create_test_comment()
 
-        response = self._request_test_comment_detail_api_view()
+        response = self._request_test_comment_api_view()
         self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
 
     def test_comment_detail_view_with_access(self):
@@ -133,11 +123,18 @@ class CommentAPIViewTestCase(
             obj=self.test_document, permission=permission_document_comment_view
         )
 
-        response = self._request_test_comment_detail_api_view()
+        response = self._request_test_comment_api_view()
         self.assertEqual(response.status_code, status.HTTP_200_OK)
 
         self.assertEqual(response.data['comment'], self.test_document_comment.comment)
 
+    def _request_test_comment_list_api_view(self):
+        return self.get(
+            viewname='rest_api:comment-list', kwargs={
+                'document_pk': self.test_document.pk
+            }
+        )
+
     def test_comment_list_view_no_access(self):
         self._create_test_comment()
 

mayan/apps/document_indexing/tests/mixins.py

@@ -13,8 +13,7 @@ class IndexTestMixin(object):
         self.test_index = Index.objects.create(label=TEST_INDEX_LABEL)
 
         # Add our document type to the new index
-        if hasattr(self, 'test_document_type'):
+        self.test_index.document_types.add(self.test_document_type)
-            self.test_index.document_types.add(self.test_document_type)
 
         # Rebuild indexes
         if rebuild:

mayan/apps/document_indexing/tests/test_api.py

@@ -15,8 +15,10 @@ from .literals import TEST_INDEX_LABEL, TEST_INDEX_SLUG
 from .mixins import IndexTestMixin
 
 
-class DocumentIndexingAPIViewTestMixin(object):
+class DocumentIndexingAPITestCase(IndexTestMixin, DocumentTestMixin, BaseAPITestCase):
-    def _request_test_index_create_api_view(self):
+    auto_upload_document = False
+
+    def _request_index_create_api_view(self):
         return self.post(
             viewname='rest_api:index-list', data={
                 'label': TEST_INDEX_LABEL, 'slug': TEST_INDEX_SLUG,
@@ -24,29 +26,8 @@ class DocumentIndexingAPIViewTestMixin(object):
             }
         )
 
-    def _request_test_index_delete_api_view(self):
-        return self.delete(
-            viewname='rest_api:index-detail', kwargs={
-                'pk': self.test_index.pk
-            }
-        )
-
-    def _request_test_index_detail_api_view(self):
-        return self.get(
-            viewname='rest_api:index-detail', kwargs={
-                'pk': self.test_index.pk
-            }
-        )
-
-
-class DocumentIndexingAPITestCase(
-    IndexTestMixin, DocumentIndexingAPIViewTestMixin, DocumentTestMixin,
-    BaseAPITestCase
-):
-    auto_upload_document = False
-
     def test_index_create_api_view_no_permission(self):
-        response = self._request_test_index_create_api_view()
+        response = self._request_index_create_api_view()
         self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
 
         self.assertEqual(Index.objects.count(), 0)
@@ -54,7 +35,7 @@ class DocumentIndexingAPITestCase(
     def test_index_create_api_view_with_permission(self):
         self.grant_permission(permission=permission_document_indexing_create)
 
-        response = self._request_test_index_create_api_view()
+        response = self._request_index_create_api_view()
         self.assertEqual(response.status_code, status.HTTP_201_CREATED)
 
         index = Index.objects.first()
@@ -65,10 +46,17 @@ class DocumentIndexingAPITestCase(
         self.assertEqual(Index.objects.count(), 1)
         self.assertEqual(index.label, TEST_INDEX_LABEL)
 
+    def _request_index_delete_api_view(self):
+        return self.delete(
+            viewname='rest_api:index-detail', kwargs={
+                'pk': self.test_index.pk
+            }
+        )
+
     def test_index_delete_api_view_no_permission(self):
         self._create_test_index()
 
-        response = self._request_test_index_delete_api_view()
+        response = self._request_index_delete_api_view()
         self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
 
         self.assertTrue(self.test_index in Index.objects.all())
@@ -80,15 +68,22 @@ class DocumentIndexingAPITestCase(
             obj=self.test_index, permission=permission_document_indexing_delete
         )
 
-        response = self._request_test_index_delete_api_view()
+        response = self._request_index_delete_api_view()
         self.assertEqual(response.status_code, status.HTTP_204_NO_CONTENT)
 
         self.assertTrue(self.test_index not in Index.objects.all())
 
+    def _request_index_detail_api_view(self):
+        return self.get(
+            viewname='rest_api:index-detail', kwargs={
+                'pk': self.test_index.pk
+            }
+        )
+
     def test_index_detail_api_view_no_access(self):
         self._create_test_index()
 
-        response = self._request_test_index_detail_api_view()
+        response = self._request_index_detail_api_view()
 
         self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
         self.assertTrue('id' not in response.data)
@@ -100,7 +95,7 @@ class DocumentIndexingAPITestCase(
             obj=self.test_index, permission=permission_document_indexing_view
         )
 
-        response = self._request_test_index_detail_api_view()
+        response = self._request_index_detail_api_view()
 
         self.assertEqual(response.status_code, status.HTTP_200_OK)
         self.assertEqual(

mayan/apps/document_indexing/tests/test_views.py

@@ -1,6 +1,5 @@
 from __future__ import absolute_import, unicode_literals
 
-from mayan.apps.common.tests import GenericViewTestCase
 from mayan.apps.documents.tests import GenericDocumentViewTestCase
 
 from ..models import Index, IndexInstanceNode
@@ -19,8 +18,10 @@ from .mixins import IndexTestMixin, IndexViewTestMixin
 
 
 class IndexViewTestCase(
-    IndexTestMixin, IndexViewTestMixin, GenericViewTestCase
+    IndexTestMixin, IndexViewTestMixin, GenericDocumentViewTestCase
 ):
+    auto_upload_document = False
+
     def test_index_create_view_no_permission(self):
         response = self._request_test_index_create_view()
         self.assertEqual(response.status_code, 403)
@@ -79,7 +80,9 @@ class IndexViewTestCase(
         self.assertEqual(self.test_index.label, TEST_INDEX_LABEL_EDITED)
 
 
-class IndexInstaceViewTestMixin(object):
+class IndexInstaceViewTestCase(
+    IndexTestMixin, IndexViewTestMixin, GenericDocumentViewTestCase
+):
     def _create_index_template_node(self):
         self.test_index.node_templates.create(
             parent=self.test_index.template_root,
@@ -87,18 +90,6 @@ class IndexInstaceViewTestMixin(object):
             link_documents=True
         )
 
-    def _request_test_index_instance_node_view(self, index_instance_node):
-        return self.get(
-            viewname='indexing:index_instance_node_view', kwargs={
-                'pk': index_instance_node.pk
-            }
-        )
-
-
-class IndexInstaceViewTestCase(
-    IndexTestMixin, IndexViewTestMixin, IndexInstaceViewTestMixin,
-    GenericDocumentViewTestCase
-):
     def test_index_rebuild_view_no_permission(self):
         self.upload_document()
         self._create_test_index()
@@ -124,10 +115,17 @@ class IndexInstaceViewTestCase(
 
         self.assertNotEqual(IndexInstanceNode.objects.count(), 0)
 
+    def _request_index_instance_node_view(self, index_instance_node):
+        return self.get(
+            viewname='indexing:index_instance_node_view', kwargs={
+                'pk': index_instance_node.pk
+            }
+        )
+
     def test_index_instance_node_view_no_permission(self):
         self._create_test_index()
 
-        response = self._request_test_index_instance_node_view(
+        response = self._request_index_instance_node_view(
             index_instance_node=self.test_index.instance_root
         )
         self.assertEqual(response.status_code, 403)
@@ -140,13 +138,15 @@ class IndexInstaceViewTestCase(
             permission=permission_document_indexing_instance_view
         )
 
-        response = self._request_test_index_instance_node_view(
+        response = self._request_index_instance_node_view(
             index_instance_node=self.test_index.instance_root
         )
         self.assertContains(response, text=TEST_INDEX_LABEL, status_code=200)
 
 
-class IndexToolsViewTestMixin(object):
+class IndexToolsViewTestCase(
+    IndexTestMixin, IndexViewTestMixin, GenericDocumentViewTestCase
+):
     def _request_indexes_rebuild_get_view(self):
         return self.get(
             viewname='indexing:rebuild_index_instances'
@@ -159,11 +159,6 @@ class IndexToolsViewTestMixin(object):
             }
         )
 
-
-class IndexToolsViewTestCase(
-    IndexTestMixin, IndexViewTestMixin, IndexToolsViewTestMixin,
-    GenericDocumentViewTestCase
-):
     def test_indexes_rebuild_no_permission(self):
         self._create_test_index(rebuild=False)
 

mayan/apps/document_signatures/tests/test_views.py

@@ -23,32 +23,84 @@ TEST_UNSIGNED_DOCUMENT_COUNT = 4
 TEST_SIGNED_DOCUMENT_COUNT = 2
 
 
-class SignaturesViewTestMixin(object):
+class SignaturesViewTestCase(SignaturesTestMixin, GenericDocumentViewTestCase):
-    def _request_test_document_version_signature_delete_view(self):
+    auto_upload_document = False
-        return self.post(
-            viewname='signatures:document_version_signature_delete',
-            kwargs={'pk': self.test_signature.pk}
-        )
 
-    def _request_test_document_version_signature_details_view(self):
+    def _request_document_version_signature_list_view(self, document):
-        return self.get(
-            viewname='signatures:document_version_signature_details',
-            kwargs={'pk': self.test_signature.pk}
-        )
-
-    def _request_test_document_version_signature_download_view(self):
-        return self.get(
-            viewname='signatures:document_version_signature_download',
-            kwargs={'pk': self.test_signature.pk}
-        )
-
-    def _request_test_document_version_signature_list_view(self, document):
         return self.get(
             viewname='signatures:document_version_signature_list',
             kwargs={'pk': self.test_document.latest_version.pk}
         )
 
-    def _request_test_document_version_signature_upload_view(self):
+    def test_signature_list_view_no_permission(self):
+        self._create_test_key()
+
+        self.test_document_path = TEST_DOCUMENT_PATH
+        self.upload_document()
+
+        self._create_test_detached_signature()
+
+        response = self._request_document_version_signature_list_view(
+            document=self.test_document
+        )
+        self.assertEqual(response.status_code, 403)
+
+    def test_signature_list_view_with_access(self):
+        self._create_test_key()
+
+        self.test_document_path = TEST_DOCUMENT_PATH
+        self.upload_document()
+
+        self._create_test_detached_signature()
+
+        self.grant_access(
+            obj=self.test_document,
+            permission=permission_document_version_signature_view
+        )
+
+        response = self._request_document_version_signature_list_view(
+            document=self.test_document
+        )
+        self.assertEqual(response.status_code, 200)
+        self.assertEqual(response.context['object_list'].count(), 1)
+
+    def _request_document_version_signature_details_view(self):
+        return self.get(
+            viewname='signatures:document_version_signature_details',
+            kwargs={'pk': self.test_signature.pk}
+        )
+
+    def test_signature_detail_view_no_permission(self):
+        self._create_test_key()
+
+        self.test_document_path = TEST_DOCUMENT_PATH
+        self.upload_document()
+
+        self._create_test_detached_signature()
+
+        response = self._request_document_version_signature_details_view()
+        self.assertEqual(response.status_code, 404)
+
+    def test_signature_detail_view_with_access(self):
+        self._create_test_key()
+
+        self.test_document_path = TEST_DOCUMENT_PATH
+        self.upload_document()
+
+        self._create_test_detached_signature()
+
+        self.grant_access(
+            obj=self.test_document,
+            permission=permission_document_version_signature_view
+        )
+
+        response = self._request_document_version_signature_details_view()
+        self.assertContains(
+            response=response, text=self.test_signature.signature_id,
+            status_code=200
+        )
+
+    def _request_document_version_signature_upload_view(self):
         with open(TEST_SIGNATURE_FILE_PATH, mode='rb') as file_object:
             return self.post(
                 viewname='signatures:document_version_signature_upload',
@@ -56,16 +108,69 @@ class SignaturesViewTestMixin(object):
                 data={'signature_file': file_object}
             )
 
-    def _request_all_test_document_version_signature_verify_view(self):
+    def test_signature_upload_view_no_permission(self):
-        return self.post(
+        self.test_document_path = TEST_DOCUMENT_PATH
-            viewname='signatures:all_document_version_signature_verify'
+        self.upload_document()
+
+        response = self._request_document_version_signature_upload_view()
+        self.assertEqual(response.status_code, 403)
+
+        self.assertEqual(DetachedSignature.objects.count(), 0)
+
+    def test_signature_upload_view_with_access(self):
+        self.test_document_path = TEST_DOCUMENT_PATH
+        self.upload_document()
+
+        self.grant_access(
+            obj=self.test_document,
+            permission=permission_document_version_signature_upload
         )
 
+        response = self._request_document_version_signature_upload_view()
+        self.assertEqual(response.status_code, 302)
 
-class SignaturesViewTestCase(
+        self.assertEqual(DetachedSignature.objects.count(), 1)
-    SignaturesTestMixin, SignaturesViewTestMixin, GenericDocumentViewTestCase
+
-):
+    def _request_document_version_signature_download_view(self):
-    auto_upload_document = False
+        return self.get(
+            viewname='signatures:document_version_signature_download',
+            kwargs={'pk': self.test_signature.pk}
+        )
+
+    def test_signature_download_view_no_permission(self):
+        self.test_document_path = TEST_DOCUMENT_PATH
+        self.upload_document()
+
+        self._create_test_detached_signature()
+
+        response = self._request_document_version_signature_download_view()
+        self.assertEqual(response.status_code, 403)
+
+    def test_signature_download_view_with_access(self):
+        self.test_document_path = TEST_DOCUMENT_PATH
+        self.upload_document()
+
+        self._create_test_detached_signature()
+
+        self.grant_access(
+            obj=self.test_document,
+            permission=permission_document_version_signature_download
+        )
+
+        self.expected_content_type = 'application/octet-stream; charset=utf-8'
+
+        response = self._request_document_version_signature_download_view()
+
+        with self.test_signature.signature_file as file_object:
+            assert_download_response(
+                self, response=response, content=file_object.read(),
+            )
+
+    def _request_document_version_signature_delete_view(self):
+        return self.post(
+            viewname='signatures:document_version_signature_delete',
+            kwargs={'pk': self.test_signature.pk}
+        )
 
     def test_signature_delete_view_no_permission(self):
         self._create_test_key()
@@ -80,7 +185,7 @@ class SignaturesViewTestCase(
             permission=permission_document_version_signature_view
         )
 
-        response = self._request_test_document_version_signature_delete_view()
+        response = self._request_document_version_signature_delete_view()
         self.assertEqual(response.status_code, 404)
         self.assertEqual(DetachedSignature.objects.count(), 1)
 
@@ -101,124 +206,15 @@ class SignaturesViewTestCase(
             permission=permission_document_version_signature_view
         )
 
-        response = self._request_test_document_version_signature_delete_view()
+        response = self._request_document_version_signature_delete_view()
         self.assertEqual(response.status_code, 302)
         self.assertEqual(DetachedSignature.objects.count(), 0)
 
-    def test_signature_detail_view_no_permission(self):
+    def _request_all_document_version_signature_verify_view(self):
-        self._create_test_key()
+        return self.post(
-
+            viewname='signatures:all_document_version_signature_verify'
-        self.test_document_path = TEST_DOCUMENT_PATH
-        self.upload_document()
-
-        self._create_test_detached_signature()
-
-        response = self._request_test_document_version_signature_details_view()
-        self.assertEqual(response.status_code, 404)
-
-    def test_signature_detail_view_with_access(self):
-        self._create_test_key()
-
-        self.test_document_path = TEST_DOCUMENT_PATH
-        self.upload_document()
-
-        self._create_test_detached_signature()
-
-        self.grant_access(
-            obj=self.test_document,
-            permission=permission_document_version_signature_view
         )
 
-        response = self._request_test_document_version_signature_details_view()
-        self.assertContains(
-            response=response, text=self.test_signature.signature_id,
-            status_code=200
-        )
-
-    def test_signature_download_view_no_permission(self):
-        self.test_document_path = TEST_DOCUMENT_PATH
-        self.upload_document()
-
-        self._create_test_detached_signature()
-
-        response = self._request_test_document_version_signature_download_view()
-        self.assertEqual(response.status_code, 403)
-
-    def test_signature_download_view_with_access(self):
-        self.test_document_path = TEST_DOCUMENT_PATH
-        self.upload_document()
-
-        self._create_test_detached_signature()
-
-        self.grant_access(
-            obj=self.test_document,
-            permission=permission_document_version_signature_download
-        )
-
-        self.expected_content_type = 'application/octet-stream; charset=utf-8'
-
-        response = self._request_test_document_version_signature_download_view()
-
-        with self.test_signature.signature_file as file_object:
-            assert_download_response(
-                self, response=response, content=file_object.read(),
-            )
-
-    def test_signature_list_view_no_permission(self):
-        self._create_test_key()
-
-        self.test_document_path = TEST_DOCUMENT_PATH
-        self.upload_document()
-
-        self._create_test_detached_signature()
-
-        response = self._request_test_document_version_signature_list_view(
-            document=self.test_document
-        )
-        self.assertEqual(response.status_code, 403)
-
-    def test_signature_list_view_with_access(self):
-        self._create_test_key()
-
-        self.test_document_path = TEST_DOCUMENT_PATH
-        self.upload_document()
-
-        self._create_test_detached_signature()
-
-        self.grant_access(
-            obj=self.test_document,
-            permission=permission_document_version_signature_view
-        )
-
-        response = self._request_test_document_version_signature_list_view(
-            document=self.test_document
-        )
-        self.assertEqual(response.status_code, 200)
-        self.assertEqual(response.context['object_list'].count(), 1)
-
-    def test_signature_upload_view_no_permission(self):
-        self.test_document_path = TEST_DOCUMENT_PATH
-        self.upload_document()
-
-        response = self._request_test_document_version_signature_upload_view()
-        self.assertEqual(response.status_code, 403)
-
-        self.assertEqual(DetachedSignature.objects.count(), 0)
-
-    def test_signature_upload_view_with_access(self):
-        self.test_document_path = TEST_DOCUMENT_PATH
-        self.upload_document()
-
-        self.grant_access(
-            obj=self.test_document,
-            permission=permission_document_version_signature_upload
-        )
-
-        response = self._request_test_document_version_signature_upload_view()
-        self.assertEqual(response.status_code, 302)
-
-        self.assertEqual(DetachedSignature.objects.count(), 1)
-
     def test_missing_signature_verify_view_no_permission(self):
         # Silence converter logging
         self._silence_logger(name='mayan.apps.converter.backends')
@@ -244,7 +240,7 @@ class SignaturesViewTestCase(
 
         DocumentVersion._post_save_hooks = old_hooks
 
-        response = self._request_all_test_document_version_signature_verify_view()
+        response = self._request_all_document_version_signature_verify_view()
         self.assertEqual(response.status_code, 403)
 
         self.assertEqual(
@@ -281,7 +277,7 @@ class SignaturesViewTestCase(
             permission=permission_document_version_signature_verify
         )
 
-        response = self._request_all_test_document_version_signature_verify_view()
+        response = self._request_all_document_version_signature_verify_view()
         self.assertEqual(response.status_code, 302)
 
         self.assertEqual(

mayan/apps/dynamic_search/tests/test_api.py

@@ -10,7 +10,7 @@ from mayan.apps.rest_api.tests import BaseAPITestCase
 from ..classes import SearchModel
 
 
-class SearchModelAPIViewTestCase(BaseAPITestCase):
+class SearchModelAPITestCase(BaseAPITestCase):
     def test_search_models_view(self):
         response = self.get(
             viewname='rest_api:searchmodel-list'
@@ -23,7 +23,7 @@ class SearchModelAPIViewTestCase(BaseAPITestCase):
         )
 
 
-class SearchAPIViewTestMixin(object):
+class SearchAPITestCase(DocumentTestMixin, BaseAPITestCase):
     def _request_search_view(self):
         query = {'q': self.test_document.label}
         return self.get(
@@ -32,19 +32,6 @@ class SearchAPIViewTestMixin(object):
             }, query=query
         )
 
-    def _request_advanced_search_view(self):
-        query = {'document_type__label': self.test_document.document_type.label}
-
-        return self.get(
-            viewname='rest_api:advanced-search-view', kwargs={
-                'search_model': document_search.get_full_name()
-            }, query=query
-        )
-
-
-class SearchAPIViewTestCase(
-    SearchAPIViewTestMixin, DocumentTestMixin, BaseAPITestCase
-):
     def test_search_no_permission(self):
         response = self._request_search_view()
         self.assertEqual(response.status_code, status.HTTP_200_OK)
@@ -62,6 +49,15 @@ class SearchAPIViewTestCase(
         )
         self.assertEqual(response.data['count'], 1)
 
+    def _request_advanced_search_view(self):
+        query = {'document_type__label': self.test_document.document_type.label}
+
+        return self.get(
+            viewname='rest_api:advanced-search-view', kwargs={
+                'search_model': document_search.get_full_name()
+            }, query=query
+        )
+
     def test_advanced_search_api_view_no_permission(self):
         response = self._request_advanced_search_view()
         self.assertEqual(response.status_code, status.HTTP_200_OK)

mayan/apps/sources/tests/mixins.py

@@ -1,34 +0,0 @@
-from __future__ import unicode_literals
-
-from ..literals import SOURCE_CHOICE_WEB_FORM
-from ..models import WebFormSource
-
-from .literals import TEST_SOURCE_LABEL, TEST_SOURCE_UNCOMPRESS_N
-
-
-class SourceTestMixin(object):
-    def _create_test_source(self):
-        self.test_source = WebFormSource.objects.create(
-            enabled=True, label=TEST_SOURCE_LABEL,
-            uncompress=TEST_SOURCE_UNCOMPRESS_N
-        )
-
-
-class SourceViewTestMixin(object):
-    def _request_setup_source_list_view(self):
-        return self.get(viewname='sources:setup_source_list')
-
-    def _request_setup_source_create_view(self):
-        return self.post(
-            kwargs={'source_type': SOURCE_CHOICE_WEB_FORM},
-            viewname='sources:setup_source_create', data={
-                'enabled': True, 'label': TEST_SOURCE_LABEL,
-                'uncompress': TEST_SOURCE_UNCOMPRESS_N
-            }
-        )
-
-    def _request_setup_source_delete_view(self):
-        return self.post(
-            viewname='sources:setup_source_delete',
-            kwargs={'pk': self.test_source.pk}
-        )

mayan/apps/sources/tests/test_models.py

@@ -28,6 +28,7 @@ from mayan.apps.storage.utils import mkdtemp
 from ..literals import SOURCE_UNCOMPRESS_CHOICE_Y
 from ..models.email_sources import EmailBaseModel, IMAPEmail, POP3Email
 from ..models.watch_folder_sources import WatchFolderSource
+from ..models.webform_sources import WebFormSource
 
 from .literals import (
     TEST_EMAIL_ATTACHMENT_AND_INLINE, TEST_EMAIL_BASE64_FILENAME,
@@ -36,24 +37,21 @@ from .literals import (
     TEST_EMAIL_NO_CONTENT_TYPE_STRING, TEST_EMAIL_ZERO_LENGTH_ATTACHMENT,
     TEST_WATCHFOLDER_SUBFOLDER
 )
-from .mixins import SourceTestMixin
 
 
-class CompressedUploadsTestCase(SourceTestMixin, GenericDocumentTestCase):
+class CompressedUploadsTestCase(GenericDocumentTestCase):
     auto_upload_document = False
 
     def test_upload_compressed_file(self):
-        self._create_test_source()
+        source = WebFormSource(
-        self.test_source.uncompress = SOURCE_UNCOMPRESS_CHOICE_Y
+            label='test source', uncompress=SOURCE_UNCOMPRESS_CHOICE_Y
-        self.test_source.save()
+        )
 
         with open(TEST_COMPRESSED_DOCUMENT_PATH, mode='rb') as file_object:
-            self.test_source.handle_upload(
+            source.handle_upload(
                 document_type=self.test_document_type,
                 file_object=file_object,
-                expand=(
+                expand=(source.uncompress == SOURCE_UNCOMPRESS_CHOICE_Y)
-                    self.test_source.uncompress == SOURCE_UNCOMPRESS_CHOICE_Y
-                )
             )
 
         self.assertEqual(Document.objects.count(), 2)

mayan/apps/sources/tests/test_views.py

@@ -25,42 +25,32 @@ from ..permissions import (
 from .literals import (
     TEST_SOURCE_LABEL, TEST_SOURCE_UNCOMPRESS_N, TEST_STAGING_PREVIEW_WIDTH
 )
-from .mixins import SourceTestMixin, SourceViewTestMixin
 
 
-class DocumentUploadWizardViewTestMixin(object):
+class DocumentUploadTestCase(GenericDocumentViewTestCase):
+    auto_upload_document = False
+
+    def setUp(self):
+        super(DocumentUploadTestCase, self).setUp()
+        self.source = WebFormSource.objects.create(
+            enabled=True, label=TEST_SOURCE_LABEL,
+            uncompress=TEST_SOURCE_UNCOMPRESS_N
+        )
+
     def _request_upload_wizard_view(self, document_path=TEST_SMALL_DOCUMENT_PATH):
         with open(document_path, mode='rb') as file_object:
             return self.post(
                 viewname='sources:upload_interactive', kwargs={
-                    'source_id': self.test_source.pk
+                    'source_id': self.source.pk
                 }, data={
                     'source-file': file_object,
                     'document_type_id': self.test_document_type.pk,
                 }
             )
 
-    def _request_upload_interactive_view(self):
-        return self.get(
-            viewname='sources:upload_interactive', data={
-                'document_type_id': self.test_document_type.pk,
-            }
-        )
-
-
-class DocumentUploadWizardViewTestCase(
-    SourceTestMixin, DocumentUploadWizardViewTestMixin,
-    GenericDocumentViewTestCase
-):
-    auto_upload_document = False
-
-    def setUp(self):
-        super(DocumentUploadWizardViewTestCase, self).setUp()
-        self._create_test_source()
-
     def test_upload_compressed_file(self):
-        self.test_source.uncompress = SOURCE_UNCOMPRESS_CHOICE_Y
+        self.source.uncompress = SOURCE_UNCOMPRESS_CHOICE_Y
-        self.test_source.save()
+        self.source.save()
 
         self.grant_access(
             obj=self.test_document_type, permission=permission_document_create
@@ -114,7 +104,7 @@ class DocumentUploadWizardViewTestCase(
         with open(TEST_SMALL_DOCUMENT_PATH, mode='rb') as file_object:
             response = self.post(
                 viewname='sources:upload_interactive', kwargs={
-                    'source_id': self.test_source.pk
+                    'source_id': self.source.pk
                 }, data={
                     'source-file': file_object,
                     'document_type_id': self.test_document_type.pk,
@@ -124,6 +114,13 @@ class DocumentUploadWizardViewTestCase(
 
         self.assertEqual(Document.objects.count(), 1)
 
+    def _request_upload_interactive_view(self):
+        return self.get(
+            viewname='sources:upload_interactive', data={
+                'document_type_id': self.test_document_type.pk,
+            }
+        )
+
     def test_upload_interactive_view_no_permission(self):
         response = self._request_upload_interactive_view()
         self.assertEqual(response.status_code, 403)
@@ -134,7 +131,7 @@ class DocumentUploadWizardViewTestCase(
         )
         response = self._request_upload_interactive_view()
         self.assertContains(
-            response=response, text=self.test_source.label, status_code=200
+            response=response, text=self.source.label, status_code=200
         )
 
 
@@ -232,19 +229,7 @@ class NewDocumentVersionViewTestCase(GenericDocumentViewTestCase):
         self.assertEqual(resolved_link, None)
 
 
-class StagingFolderViewTestMixin(object):
+class StagingFolderViewTestCase(GenericViewTestCase):
-    def _request_staging_file_delete_view(self, staging_folder, staging_file):
-        return self.post(
-            viewname='sources:staging_file_delete', kwargs={
-                'pk': staging_folder.pk,
-                'encoded_filename': staging_file.encoded_filename
-            }
-        )
-
-
-class StagingFolderViewTestCase(
-    StagingFolderViewTestMixin, GenericViewTestCase
-):
     def setUp(self):
         super(StagingFolderViewTestCase, self).setUp()
         self.temporary_directory = mkdtemp()
@@ -256,6 +241,14 @@ class StagingFolderViewTestCase(
         fs_cleanup(self.temporary_directory)
         super(StagingFolderViewTestCase, self).tearDown()
 
+    def _request_staging_file_delete_view(self, staging_folder, staging_file):
+        return self.post(
+            viewname='sources:staging_file_delete', kwargs={
+                'pk': staging_folder.pk,
+                'encoded_filename': staging_file.encoded_filename
+            }
+        )
+
     def test_staging_file_delete_no_permission(self):
         staging_folder = StagingFolderSource.objects.create(
             label=TEST_SOURCE_LABEL,
@@ -297,10 +290,44 @@ class StagingFolderViewTestCase(
         self.assertEqual(len(list(staging_folder.get_files())), 0)
 
 
-class SourcesViewTestCase(
+class SourcesTestCase(GenericDocumentViewTestCase):
-    SourceTestMixin, SourceViewTestMixin, GenericViewTestCase
+    def _create_web_source(self):
-):
+        self.source = WebFormSource.objects.create(
+            enabled=True, label=TEST_SOURCE_LABEL,
+            uncompress=TEST_SOURCE_UNCOMPRESS_N
+        )
+
+    def _request_setup_source_list_view(self):
+        return self.get(viewname='sources:setup_source_list')
+
+    def test_source_list_view_no_permission(self):
+        self._create_web_source()
+
+        response = self._request_setup_source_list_view()
+        self.assertEqual(response.status_code, 403)
+
+    def test_source_list_view_with_permission(self):
+        self._create_web_source()
+
+        self.grant_permission(permission=permission_sources_setup_view)
+
+        response = self._request_setup_source_list_view()
+        self.assertContains(
+            response=response, text=self.source.label, status_code=200
+        )
+
+    def _request_setup_source_create_view(self):
+        return self.post(
+            kwargs={'source_type': SOURCE_CHOICE_WEB_FORM},
+            viewname='sources:setup_source_create', data={
+                'enabled': True, 'label': TEST_SOURCE_LABEL,
+                'uncompress': TEST_SOURCE_UNCOMPRESS_N
+            }
+        )
+
     def test_source_create_view_no_permission(self):
+        self.grant_permission(permission=permission_sources_setup_view)
+
         response = self._request_setup_source_create_view()
         self.assertEqual(response.status_code, 403)
 
@@ -308,6 +335,7 @@ class SourcesViewTestCase(
 
     def test_source_create_view_with_permission(self):
         self.grant_permission(permission=permission_sources_setup_create)
+        self.grant_permission(permission=permission_sources_setup_view)
 
         response = self._request_setup_source_create_view()
         self.assertEqual(response.status_code, 302)
@@ -316,10 +344,17 @@ class SourcesViewTestCase(
         self.assertEqual(webform_source.label, TEST_SOURCE_LABEL)
         self.assertEqual(webform_source.uncompress, TEST_SOURCE_UNCOMPRESS_N)
 
+    def _request_setup_source_delete_view(self):
+        return self.post(
+            viewname='sources:setup_source_delete',
+            kwargs={'pk': self.source.pk}
+        )
+
     def test_source_delete_view_with_permission(self):
-        self._create_test_source()
+        self._create_web_source()
 
         self.grant_permission(permission=permission_sources_setup_delete)
+        self.grant_permission(permission=permission_sources_setup_view)
 
         response = self._request_setup_source_delete_view()
         self.assertEqual(response.status_code, 302)
@@ -327,25 +362,11 @@ class SourcesViewTestCase(
         self.assertEqual(WebFormSource.objects.count(), 0)
 
     def test_source_delete_view_no_permission(self):
-        self._create_test_source()
+        self._create_web_source()
+
+        self.grant_permission(permission=permission_sources_setup_view)
 
         response = self._request_setup_source_delete_view()
         self.assertEqual(response.status_code, 403)
 
         self.assertEqual(WebFormSource.objects.count(), 1)
-
-    def test_source_list_view_no_permission(self):
-        self._create_test_source()
-
-        response = self._request_setup_source_list_view()
-        self.assertEqual(response.status_code, 403)
-
-    def test_source_list_view_with_permission(self):
-        self._create_test_source()
-
-        self.grant_permission(permission=permission_sources_setup_view)
-
-        response = self._request_setup_source_list_view()
-        self.assertContains(
-            response=response, text=self.test_source.label, status_code=200
-        )

mayan/apps/sources/views.py

@@ -215,7 +215,6 @@ class UploadInteractiveView(UploadBaseView):
                 UploadInteractiveView, self
             ).dispatch(request, *args, **kwargs)
         except Exception as exception:
-            raise
             if request.is_ajax():
                 return JsonResponse(
                     data={'error': force_text(exception)}, status=500

mayan/apps/tags/tests/test_workflow_actions.py

@@ -20,7 +20,7 @@ class TagActionTestCase(TagTestMixin, ActionTestCase):
         action.execute(context={'document': self.test_document})
 
         self.assertEqual(self.test_tag.documents.count(), 1)
-        self.assertTrue(self.test_document in self.test_tag.documents.all())
+        self.assertEqual(self.test_document in self.test_tag.documents.all())
 
     def test_tag_remove_action(self):
         self.test_tag.attach_to(document=self.test_document)