chore(tests): Enhance AccessList API view tests

Refactors and extends AccessList API view test cases. Adds support for
testing virtual machines and improves test data setup by using
`assigned_object` directly instead of separate fields for object type
and ID.

Author: pheus

netbox_acls/tests/api/test_access_lists.py

@@ -1,40 +1,70 @@
 from dcim.models import Device, DeviceRole, DeviceType, Manufacturer, Site
 from django.contrib.contenttypes.models import ContentType
 from utilities.testing import APIViewTestCases
+from virtualization.models import Cluster, ClusterType, VirtualMachine
 
 from netbox_acls.choices import *
 from netbox_acls.models import *
 
 
-class ACLTestCase(
-    APIViewTestCases.APIViewTestCase,
-):
-    """Test the AccessList Test"""
+class AccessListAPIViewTestCase(APIViewTestCases.APIViewTestCase):
+    """
+    API view test case for AccessList.
+    """
 
     model = AccessList
     view_namespace = "plugins-api:netbox_acls"
     brief_fields = ["display", "id", "name", "url"]
+    user_permissions = (
+        "dcim.view_site",
+        "dcim.view_devicetype",
+        "dcim.view_device",
+        "virtualization.view_cluster",
+        "virtualization.view_clustergroup",
+        "virtualization.view_clustertype",
+        "virtualization.view_virtualmachine",
+    )
 
     @classmethod
     def setUpTestData(cls):
-        site = Site.objects.create(name="Site 1", slug="site-1")
+        """Set up Access List for API view testing."""
+        site = Site.objects.create(
+            name="Site 1",
+            slug="site-1",
+        )
+
+        # Device
         manufacturer = Manufacturer.objects.create(
             name="Manufacturer 1",
             slug="manufacturer-1",
         )
-        devicetype = DeviceType.objects.create(
+        device_type = DeviceType.objects.create(
             manufacturer=manufacturer,
             model="Device Type 1",
         )
-        devicerole = DeviceRole.objects.create(
+        device_role = DeviceRole.objects.create(
             name="Device Role 1",
             slug="device-role-1",
         )
         device = Device.objects.create(
             name="Device 1",
             site=site,
-            device_type=devicetype,
-            role=devicerole,
+            device_type=device_type,
+            role=device_role,
+        )
+
+        # Virtual Machine
+        cluster_type = ClusterType.objects.create(
+            name="Cluster Type 1",
+            slug="cluster-type-1",
+        )
+        cluster = Cluster.objects.create(
+            name="Cluster 1",
+            type=cluster_type,
+        )
+        virtual_machine = VirtualMachine.objects.create(
+            name="VM 1",
+            cluster=cluster,
         )
 
         access_lists = (
@@ -47,16 +77,15 @@ def setUpTestData(cls):
             ),
             AccessList(
                 name="testacl2",
-                assigned_object_type=ContentType.objects.get_for_model(Device),
-                assigned_object_id=device.id,
-                type=ACLTypeChoices.TYPE_STANDARD,
-                default_action=ACLActionChoices.ACTION_DENY,
+                assigned_object=device,
+                type=ACLTypeChoices.TYPE_EXTENDED,
+                default_action=ACLActionChoices.ACTION_PERMIT,
             ),
             AccessList(
                 name="testacl3",
-                assigned_object_type=ContentType.objects.get_for_model(Device),
-                assigned_object_id=device.id,
-                type=ACLTypeChoices.TYPE_STANDARD,
+                assigned_object_type=ContentType.objects.get_for_model(VirtualMachine),
+                assigned_object_id=virtual_machine.id,
+                type=ACLTypeChoices.TYPE_EXTENDED,
                 default_action=ACLActionChoices.ACTION_DENY,
             ),
         )
@@ -79,9 +108,12 @@ def setUpTestData(cls):
             },
             {
                 "name": "testacl6",
-                "assigned_object_type": "dcim.device",
-                "assigned_object_id": device.id,
+                "assigned_object_type": "virtualization.virtualmachine",
+                "assigned_object_id": virtual_machine.id,
                 "type": ACLTypeChoices.TYPE_STANDARD,
-                "default_action": ACLActionChoices.ACTION_DENY,
+                "default_action": ACLActionChoices.ACTION_PERMIT,
             },
         ]
+        cls.bulk_update_data = {
+            "default_action": ACLActionChoices.ACTION_PERMIT,
+        }