refactor(access): Update Test Suite for Person model

ref: #839 #841

app/access/tests/unit/person/conftest.py

@@ -1,14 +1,18 @@
 import pytest
 
-from access.models.person import Person
-
 
 
 @pytest.fixture( scope = 'class')
-def model(request):
+def model(model_person):
 
-    request.cls.model = Person
+    yield model_person
 
-    yield request.cls.model
+@pytest.fixture( scope = 'class')
+def model_kwargs(request, kwargs_person):
 
-    del request.cls.model
+    request.cls.kwargs_create_item = kwargs_person.copy()
+
+    yield kwargs_person.copy()
+
+    if hasattr(request.cls, 'kwargs_create_item'):
+        del request.cls.kwargs_create_item

app/access/tests/unit/person/test_unit_person_api_fields.py

@@ -1,9 +1,12 @@
+import pytest
+
 from access.tests.unit.entity.test_unit_entity_api_fields import (
     EntityAPIInheritedCases
 )
 
 
 
+@pytest.mark.model_person
 class PersonAPITestCases(
     EntityAPIInheritedCases,
 ):
@@ -43,6 +46,7 @@ class PersonAPIInheritedCases(
 
 
 
+@pytest.mark.module_access
 class PersonAPIPyTest(
     PersonAPITestCases,
 ):

app/access/tests/unit/person/test_unit_person_model.py

@@ -9,16 +9,11 @@ from access.tests.unit.entity.test_unit_entity_model import (
 
 
 
+@pytest.mark.model_person
 class PersonModelTestCases(
     EntityModelInheritedCases,
 ):
 
-    kwargs_create_item: dict = {
-        'f_name': 'Ian',
-        'm_name': 'Peter',
-        'l_name': 'Funny',
-        'dob': '2025-04-08',
-    }
 
     sub_model_type = 'person'
     """Sub Model Type
@@ -27,64 +22,66 @@ class PersonModelTestCases(
     """
 
 
-    parameterized_fields: dict = {
-        "f_name": {
-            'field_type': models.fields.CharField,
-            'field_parameter_default_exists': False,
-            'field_parameter_verbose_name_type': str,
-        },
-        "m_name": {
-            'field_type': models.fields.CharField,
-            'field_parameter_default_exists': False,
-            'field_parameter_verbose_name_type': str,
-        },
-        "l_name": {
-            'field_type': models.fields.CharField,
-            'field_parameter_default_exists': False,
-            'field_parameter_verbose_name_type': str,
-        },
-        "dob": {
-            'field_type': models.fields.DateField,
-            'field_parameter_default_exists': False,
-            'field_parameter_verbose_name_type': str,
-        },
-    }
+    @property
+    def parameterized_class_attributes(self):
+
+        return {
+            '_is_submodel': {
+                'value': True
+            },
+            'url_model_name': {
+                'type': str,
+                'value': 'entity'
+            }
+        }
+
+
+    @property
+    def parameterized_fields(self):
+
+        return {
+            'f_name': {
+                'blank': False,
+                'default': models.fields.NOT_PROVIDED,
+                'field_type': models.CharField,
+                'length': 64,
+                'null': False,
+                'unique': False,
+            },
+            'm_name': {
+                'blank': True,
+                'default': models.fields.NOT_PROVIDED,
+                'field_type': models.CharField,
+                'length': 100,
+                'null': True,
+                'unique': False,
+            },
+            'l_name': {
+                'blank': False,
+                'default': models.fields.NOT_PROVIDED,
+                'field_type': models.CharField,
+                'length': 64,
+                'null': False,
+                'unique': False,
+            },
+            'dob': {
+                'blank': True,
+                'default': models.fields.NOT_PROVIDED,
+                'field_type': models.DateField,
+                'null': True,
+                'unique': False,
+            },
+        }
 
 
 
-    def test_class_inherits_person(self):
+    def test_class_inherits_person(self, model):
         """ Class inheritence
 
         TenancyObject must inherit SaveHistory
         """
 
-        assert issubclass(self.model, Person)
-
-
-    def test_attribute_value_history_app_label(self):
-        """Attribute Type
-
-        history_app_label has been set, override this test case with the value
-        of attribute `history_app_label`
-        """
-
-        assert self.model.history_app_label == 'access'
-
-
-    def test_attribute_value_history_model_name(self):
-        """Attribute Type
-
-        history_model_name has been set, override this test case with the value
-        of attribute `history_model_name`
-        """
-
-        assert self.model.history_model_name == 'person'
-
-
-
-    def test_function_value_get_url(self):
-
-        assert self.item.get_url() == '/api/v2/access/entity/person/' + str(self.item.id)
+        assert issubclass(model, Person)
 
 
 
@@ -95,28 +92,20 @@ class PersonModelInheritedCases(
 
     Test Cases for Ticket models that inherit from model Entity
     """
-
-    kwargs_create_item: dict = {}
-
-    model = None
-
-    sub_model_type = None
-    """Ticket Sub Model Type
-    
-    Ticket sub-models must have this attribute defined in `ModelNam.Meta.sub_model_type`
-    """
+    pass
 
 
 
+@pytest.mark.module_access
 class PersonModelPyTest(
     PersonModelTestCases,
 ):
 
 
-    def test_function_value_get_related_model(self):
+    def test_function_value_get_related_model(self, model_instance):
         """Function test
 
         Confirm function `get_related_model` is None for base model
         """
 
-        assert self.item.get_related_model() is None
+        assert model_instance.get_related_model() is None

app/access/tests/unit/person/test_unit_person_viewset.py

@@ -1,3 +1,5 @@
+import pytest
+
 from django.test import TestCase
 
 from access.models.person import Person
@@ -7,6 +9,7 @@ from access.tests.unit.entity.test_unit_entity_viewset import (
 
 
 
+@pytest.mark.model_person
 class ViewsetTestCases(
     EntityViewsetInheritedCases,
 ):
@@ -28,6 +31,7 @@ class PersonViewsetInheritedCases(
 
 
 
+@pytest.mark.module_access
 class PersonViewsetTest(
     ViewsetTestCases,
     TestCase,

app/tests/fixtures/__init__.py

@@ -78,6 +78,11 @@ from .model_configgroupsoftware import (
     model_configgroupsoftware,
 )
 
+from .model_contact import (
+    kwargs_contact,
+    model_contact,
+)
+
 from .model_contenttype import (
     model_contenttype,
 )
@@ -183,6 +188,11 @@ from .model_permission import (
     model_permission,
 )
 
+from .model_person import (
+    kwargs_person,
+    model_person,
+)
+
 from .model_port import (
     kwargs_port,
     model_port,

app/tests/fixtures/model_contact.py

@@ -0,0 +1,27 @@
+import datetime
+import pytest
+
+from access.models.contact import Contact
+
+
+
+@pytest.fixture( scope = 'class')
+def model_contact():
+
+    yield Contact
+
+
+@pytest.fixture( scope = 'class')
+def kwargs_contact( kwargs_person ):
+
+    random_str = str(datetime.datetime.now(tz=datetime.timezone.utc))
+    random_str = str(random_str).replace(
+            ' ', '').replace(':', '').replace('+', '').replace('.', '')
+
+    kwargs = {
+        **kwargs_person.copy(),
+        'entity_type': 'contact',
+        'email': 'p' + random_str + '@domain.tld'
+    }
+
+    yield kwargs.copy()

app/tests/fixtures/model_person.py

@@ -0,0 +1,28 @@
+import datetime
+import pytest
+
+from access.models.person import Person
+
+
+
+@pytest.fixture( scope = 'class')
+def model_person():
+
+    yield Person
+
+
+@pytest.fixture( scope = 'class')
+def kwargs_person( kwargs_entity ):
+
+    random_str = str(datetime.datetime.now(tz=datetime.timezone.utc))
+    random_str = str(random_str).replace(
+            ' ', '').replace(':', '').replace('+', '').replace('.', '')
+
+    kwargs = {
+        **kwargs_entity.copy(),
+        'entity_type': 'person',
+        'f_name': 'p' + random_str,
+        'l_name': 'p' + random_str
+    }
+
+    yield kwargs.copy()

pyproject.toml

@@ -1126,6 +1126,7 @@ markers = [
     "model_manufacturer: Select all manufacturer tests.",
     "model_operatingsystem: Select tests for model Operating System.",
     "model_operatingsystemversion: Select tests for model Operating System Version.",
+    "model_person: Selects test for model Person.",
     "model_port: Selects tests for model port.",
     "model_project: Selects tests for model Project.",
     "model_projectmilestone: Selects tests for model Project Milestone.",