From aa94247c3dfcb0528adf7dc7d9ccd7c7ede34fe3 Mon Sep 17 00:00:00 2001
From: Roberto Rosario <Roberto.Rosario@mayan-edms.com>
Date: Fri, 10 May 2019 19:54:01 -0400
Subject: [PATCH] Add test for required metadata remove issue.

Signed-off-by: Roberto Rosario <Roberto.Rosario@mayan-edms.com>
---
 mayan/apps/metadata/tests/test_views.py | 65 +++++++++++++++++++++++--
 1 file changed, 61 insertions(+), 4 deletions(-)

diff --git a/mayan/apps/metadata/tests/test_views.py b/mayan/apps/metadata/tests/test_views.py
index e159f05623..9529dbb5f2 100644
--- a/mayan/apps/metadata/tests/test_views.py
+++ b/mayan/apps/metadata/tests/test_views.py
@@ -30,10 +30,10 @@ class DocumentMetadataTestCase(GenericDocumentViewTestCase):
     def setUp(self):
         super(DocumentMetadataTestCase, self).setUp()
 
-        self.test_metadata_type = MetadataType.objects.create(
-            name=TEST_METADATA_TYPE_NAME, label=TEST_METADATA_TYPE_LABEL
-        )
-
+        #self.test_metadata_type = MetadataType.objects.create(
+        #    name=TEST_METADATA_TYPE_NAME, label=TEST_METADATA_TYPE_LABEL
+        #)
+        self._create_test_metadata_type()
         self.test_document_type.metadata.create(
             metadata_type=self.test_metadata_type
         )
@@ -332,6 +332,63 @@ class DocumentMetadataTestCase(GenericDocumentViewTestCase):
         )
 
 
+
+class DocumentMetadataRequiredTestCase(
+    MetadataTestMixin, MetadataTypeTestMixin, GenericDocumentViewTestCase
+):
+    def setUp(self):
+        super(DocumentMetadataRequiredTestCase, self).setUp()
+        self._create_test_metadata_type()
+        self._create_test_metadata_type()
+        self.test_document_type.metadata.create(
+            metadata_type=self.test_metadata_types[0]
+        )
+        self.test_document_type.metadata.create(
+            metadata_type=self.test_metadata_types[1], required=True
+        )
+
+    def _create_test_document_metadata(self):
+        self.test_document_metadatas = []
+        self.test_document_metadatas.append(
+            self.test_document.metadata.get_or_create(
+                metadata_type=self.test_metadata_types[0], #defaults={'value': 'a'}
+            )
+        )
+        self.test_document_metadatas.append(
+            self.test_document.metadata.get_or_create(
+                metadata_type=self.test_metadata_types[1], #defaults={'value': 'b'}
+            )
+        )
+
+    def _request_document_metadata_remove_post_view(self):
+        return self.post(
+            viewname='metadata:metadata_remove',
+            kwargs={'pk': self.test_document.pk}, data={
+                'form-0-id': self.test_metadata_types[0].pk,
+                'form-0-update': True,
+                'form-TOTAL_FORMS': '1',
+                'form-INITIAL_FORMS': '0',
+                'form-MAX_NUM_FORMS': '',
+            }
+        )
+
+    def test_document_metadata_remove_post_view_with_access(self):
+        self._create_test_document_metadata()
+
+        self.grant_access(
+            obj=self.test_document,
+            permission=permission_document_metadata_remove
+        )
+        response = self._request_document_metadata_remove_post_view()
+        self.assertEqual(response.status_code, 302)
+
+        self.assertTrue(
+            self.test_document_metadata not in self.test_document.metadata.all()
+        )
+
+
+
+
 class MetadataTypeViewTestCase(
     DocumentTestMixin, MetadataTestMixin, MetadataTypeTestMixin,
     GenericViewTestCase