nautobot 2.0.0a3__py3-none-any.whl → 2.0.0b1__py3-none-any.whl

nautobot/dcim/tests/test_api.py

@@ -1,4 +1,5 @@
 import json
+from unittest import skip
 
 from django.contrib.auth import get_user_model
 from django.contrib.contenttypes.models import ContentType
@@ -82,12 +83,20 @@ class Mixins:
                 location=Location.objects.filter(location_type=LocationType.objects.get(name="Campus")).first(),
                 device_type=DeviceType.objects.first(),
                 role=Role.objects.get_for_model(Device).first(),
+                status=Status.objects.get_for_model(Device).first(),
                 name="Peer Device",
             )
             if self.peer_termination_type is None:
                 raise NotImplementedError("Test case must set peer_termination_type")
-            peer_obj = self.peer_termination_type.objects.create(device=peer_device, name="Peer Termination")
-            cable = Cable(termination_a=obj, termination_b=peer_obj, label="Cable 1")
+            if self.peer_termination_type is Interface:
+                intf_status = Status.objects.get_for_model(Interface).first()
+                peer_obj = self.peer_termination_type.objects.create(
+                    device=peer_device, name="Peer Termination", status=intf_status
+                )
+            else:
+                peer_obj = self.peer_termination_type.objects.create(device=peer_device, name="Peer Termination")
+            cable_status = Status.objects.get_for_model(Cable).first()
+            cable = Cable(termination_a=obj, termination_b=peer_obj, label="Cable 1", status=cable_status)
             cable.save()
 
             self.add_permissions(f"dcim.view_{self.model._meta.model_name}")
@@ -105,7 +114,6 @@ class Mixins:
         """Mixin class for all `ComponentModel` model class tests."""
 
         model = None
-        brief_fields = ["device", "display", "id", "name", "url"]
         bulk_update_data = {
             "description": "New description",
         }
@@ -118,25 +126,26 @@ class Mixins:
             cls.manufacturer = cls.device_type.manufacturer
             cls.location = Location.objects.filter(location_type=LocationType.objects.get(name="Campus")).first()
             cls.device_role = Role.objects.get_for_model(Device).first()
+            cls.device_status = Status.objects.get_for_model(Device).first()
             cls.device = Device.objects.create(
-                device_type=cls.device_type, role=cls.device_role, name="Device 1", location=cls.location
+                device_type=cls.device_type,
+                role=cls.device_role,
+                name="Device 1",
+                location=cls.location,
+                status=cls.device_status,
             )
 
     class BasePortTestMixin(ComponentTraceMixin, BaseComponentTestMixin):
         """Mixin class for all `FooPort` tests."""
 
         peer_termination_type = None
-        brief_fields = ["cable", "device", "display", "id", "name", "url"]
 
     class BasePortTemplateTestMixin(BaseComponentTestMixin):
         """Mixin class for all `FooPortTemplate` tests."""
 
-        brief_fields = ["display", "id", "name", "url"]
-
 
-class LocationTypeTest(APIViewTestCases.APIViewTestCase):
+class LocationTypeTest(APIViewTestCases.APIViewTestCase, APIViewTestCases.TreeModelAPIViewTestCaseMixin):
     model = LocationType
-    brief_fields = ["display", "id", "name", "slug", "tree_depth", "url"]
     bulk_update_data = {
         "description": "Some generic description of multiple types. Not very useful.",
         "nestable": True,
@@ -177,9 +186,8 @@ class LocationTypeTest(APIViewTestCases.APIViewTestCase):
         ]
 
 
-class LocationTest(APIViewTestCases.APIViewTestCase):
+class LocationTest(APIViewTestCases.APIViewTestCase, APIViewTestCases.TreeModelAPIViewTestCaseMixin):
     model = Location
-    brief_fields = ["display", "id", "name", "slug", "tree_depth", "url"]
     choices_fields = []
     slug_source = ["parent__slug", "name"]
 
@@ -252,7 +260,6 @@ class LocationTest(APIViewTestCases.APIViewTestCase):
             "status": self.location_statuses[0].pk,
             "time_zone": None,
             "location_type": self.lt1.pk,
-            "location": self.loc1.pk,
         }
 
         # Attempt to create new location with null time_zone attr.
@@ -273,7 +280,6 @@ class LocationTest(APIViewTestCases.APIViewTestCase):
             "status": self.location_statuses[0].pk,
             "time_zone": "",
             "location_type": self.lt1.pk,
-            "location": self.loc1.pk,
         }
 
         # Attempt to create new location with blank time_zone attr.
@@ -295,7 +301,6 @@ class LocationTest(APIViewTestCases.APIViewTestCase):
             "status": self.location_statuses[0].pk,
             "time_zone": time_zone,
             "location_type": self.lt1.pk,
-            "location": self.loc1.pk,
         }
 
         # Attempt to create new location with valid time_zone attr.
@@ -317,7 +322,6 @@ class LocationTest(APIViewTestCases.APIViewTestCase):
             "status": self.location_statuses[0].pk,
             "time_zone": time_zone,
             "location_type": self.lt1.pk,
-            "location": self.loc1.pk,
         }
 
         # Attempt to create new location with invalid time_zone attr.
@@ -341,9 +345,8 @@ class LocationTest(APIViewTestCases.APIViewTestCase):
         self.assertEqual(response.json()["time_zone"], None)
 
 
-class RackGroupTest(APIViewTestCases.APIViewTestCase):
+class RackGroupTest(APIViewTestCases.APIViewTestCase, APIViewTestCases.TreeModelAPIViewTestCaseMixin):
     model = RackGroup
-    brief_fields = ["display", "id", "name", "rack_count", "slug", "tree_depth", "url"]
     bulk_update_data = {
         "description": "New description",
     }
@@ -459,7 +462,6 @@ class RackGroupTest(APIViewTestCases.APIViewTestCase):
 
 class RackTest(APIViewTestCases.APIViewTestCase):
     model = Rack
-    brief_fields = ["device_count", "display", "id", "name", "url"]
     choices_fields = ["outer_unit", "type", "width"]
 
     @classmethod
@@ -552,7 +554,7 @@ class RackTest(APIViewTestCases.APIViewTestCase):
         rack = Rack.objects.first()
         self.add_permissions("dcim.view_rack")
         url = reverse("dcim-api:rack-elevation", kwargs={"pk": rack.pk})
-        params = {"brief": "true", "face": "front", "exclude": "a85a31aa-094f-4de9-8ba6-16cb088a1b74"}
+        params = {"face": "front", "exclude": "a85a31aa-094f-4de9-8ba6-16cb088a1b74"}
         response = self.client.get(url, params, **self.header)
         self.assertHttpStatus(response, 200)
 
@@ -604,7 +606,6 @@ class RackTest(APIViewTestCases.APIViewTestCase):
 
 class RackReservationTest(APIViewTestCases.APIViewTestCase):
     model = RackReservation
-    brief_fields = ["display", "id", "units", "url", "user"]
     bulk_update_data = {
         "description": "New description",
     }
@@ -613,10 +614,11 @@ class RackReservationTest(APIViewTestCases.APIViewTestCase):
     def setUpTestData(cls):
         user = User.objects.create(username="user1", is_active=True)
         location = Location.objects.filter(location_type=LocationType.objects.get(name="Campus")).first()
+        rack_status = Status.objects.get_for_model(Rack).first()
 
         cls.racks = (
-            Rack.objects.create(location=location, name="Rack 1"),
-            Rack.objects.create(location=location, name="Rack 2"),
+            Rack.objects.create(location=location, name="Rack 1", status=rack_status),
+            Rack.objects.create(location=location, name="Rack 2", status=rack_status),
         )
 
         RackReservation.objects.create(rack=cls.racks[0], units=[1, 2, 3], user=user, description="Reservation #1")
@@ -651,7 +653,6 @@ class RackReservationTest(APIViewTestCases.APIViewTestCase):
 
 class ManufacturerTest(APIViewTestCases.APIViewTestCase):
     model = Manufacturer
-    brief_fields = ["device_type_count", "display", "id", "name", "url"]
     create_data = [
         {
             "name": "Test Manufacturer 4",
@@ -681,15 +682,6 @@ class ManufacturerTest(APIViewTestCases.APIViewTestCase):
 
 class DeviceTypeTest(APIViewTestCases.APIViewTestCase):
     model = DeviceType
-    brief_fields = [
-        "device_count",
-        "display",
-        "id",
-        "manufacturer",
-        "model",
-        "slug",
-        "url",
-    ]
     bulk_update_data = {
         "part_number": "ABC123",
     }
@@ -1019,7 +1011,6 @@ class DeviceBayTemplateTest(Mixins.BasePortTemplateTestMixin):
 
 class PlatformTest(APIViewTestCases.APIViewTestCase):
     model = Platform
-    brief_fields = ["device_count", "display", "id", "name", "url", "virtual_machine_count"]
     create_data = [
         {
             "name": "Test Platform 4",
@@ -1041,16 +1032,16 @@ class PlatformTest(APIViewTestCases.APIViewTestCase):
 
 class DeviceTest(APIViewTestCases.APIViewTestCase):
     model = Device
-    brief_fields = ["display", "id", "name", "url"]
     choices_fields = ["face"]
 
     @classmethod
     def setUpTestData(cls):
         locations = Location.objects.filter(location_type=LocationType.objects.get(name="Campus"))[:2]
 
+        rack_status = Status.objects.get_for_model(Rack).first()
         racks = (
-            Rack.objects.create(name="Rack 1", location=locations[0]),
-            Rack.objects.create(name="Rack 2", location=locations[1]),
+            Rack.objects.create(name="Rack 1", location=locations[0], status=rack_status),
+            Rack.objects.create(name="Rack 2", location=locations[1], status=rack_status),
         )
 
         device_statuses = Status.objects.get_for_model(Device)
@@ -1140,27 +1131,28 @@ class DeviceTest(APIViewTestCases.APIViewTestCase):
             "status": device_statuses[1].pk,
         }
 
-    def test_config_context_included_by_default_in_list_view(self):
+    def test_config_context_excluded_by_default_in_list_view(self):
         """
-        Check that config context data is included by default in the devices list.
+        Check that config context data is excluded by default in the devices list.
         """
         self.add_permissions("dcim.view_device")
         url = reverse("dcim-api:device-list")
         response = self.client.get(url, **self.header)
 
         self.assertHttpStatus(response, status.HTTP_200_OK)
-        self.assertEqual(response.data["results"][0].get("config_context", {}).get("A"), 1)
+        self.assertNotIn("config_context", response.data["results"][0])
 
-    def test_config_context_excluded(self):
+    def test_config_context_included(self):
         """
-        Check that config context data can be excluded by passing ?exclude=config_context.
+        Check that config context data can be included by passing ?include=config_context.
         """
         self.add_permissions("dcim.view_device")
-        url = reverse("dcim-api:device-list") + "?exclude=config_context"
+        url = reverse("dcim-api:device-list") + "?include=config_context"
         response = self.client.get(url, **self.header)
 
         self.assertHttpStatus(response, status.HTTP_200_OK)
-        self.assertFalse("config_context" in response.data["results"][0])
+        self.assertIn("config_context", response.data["results"][0])
+        self.assertEqual(response.data["results"][0]["config_context"], {"A": 1})
 
     def test_unique_name_per_location_constraint(self):
         """
@@ -1197,7 +1189,7 @@ class DeviceTest(APIViewTestCases.APIViewTestCase):
             self._get_detail_url(Device.objects.get(name="Device 1")), patch_data, format="json", **self.header
         )
         self.assertHttpStatus(response, status.HTTP_200_OK)
-        self.assertEqual(response.data["local_config_context_schema"]["id"], str(schema.pk))
+        self.assertEqual(str(response.data["local_config_context_schema"]), self.absolute_api_url(schema))
 
     def test_local_config_context_schema_schema_validation_fails(self):
         """
@@ -1226,10 +1218,13 @@ class DeviceTest(APIViewTestCases.APIViewTestCase):
         self.add_permissions("dcim.change_device")
 
         dev = Device.objects.get(name="Device 3")
-        dev_intf = Interface.objects.create(name="Ethernet1", device=dev, type="1000base-t")
+        intf_status = Status.objects.get_for_model(Interface).first()
+        dev_intf = Interface.objects.create(name="Ethernet1", device=dev, type="1000base-t", status=intf_status)
+        ipaddr_status = Status.objects.get_for_model(IPAddress).first()
+        prefix_status = Status.objects.get_for_model(Prefix).first()
         namespace = Namespace.objects.first()
-        Prefix.objects.create(prefix="192.0.2.0/24", namespace=namespace)
-        dev_ip_addr = IPAddress.objects.create(address="192.0.2.1/24", namespace=namespace)
+        Prefix.objects.create(prefix="192.0.2.0/24", namespace=namespace, status=prefix_status)
+        dev_ip_addr = IPAddress.objects.create(address="192.0.2.1/24", namespace=namespace, status=ipaddr_status)
         dev_intf.add_ip_addresses(dev_ip_addr)
 
         patch_data = {"primary_ip4": dev_ip_addr.pk}
@@ -1408,14 +1403,20 @@ class InterfaceTest(Mixins.BasePortTestMixin):
         interface_status = Status.objects.get_for_model(Interface).first()
 
         cls.devices = (
+            cls.device,
             Device.objects.create(
-                device_type=cls.device_type, role=cls.device_role, name="Device 1", location=cls.location
-            ),
-            Device.objects.create(
-                device_type=cls.device_type, role=cls.device_role, name="Device 2", location=cls.location
+                device_type=cls.device_type,
+                role=cls.device_role,
+                status=cls.device_status,
+                name="Device 2",
+                location=cls.location,
             ),
             Device.objects.create(
-                device_type=cls.device_type, role=cls.device_role, name="Device 3", location=cls.location
+                device_type=cls.device_type,
+                role=cls.device_role,
+                status=cls.device_status,
+                name="Device 3",
+                location=cls.location,
             ),
         )
 
@@ -1425,24 +1426,58 @@ class InterfaceTest(Mixins.BasePortTestMixin):
         Device.objects.filter(id=cls.devices[0].id).update(virtual_chassis=cls.virtual_chassis, vc_position=1)
         Device.objects.filter(id=cls.devices[1].id).update(virtual_chassis=cls.virtual_chassis, vc_position=2)
 
+        # Interfaces have special handling around the "Active" status so let's set our interfaces to something else.
+        non_default_status = Status.objects.get_for_model(Interface).exclude(name="Active").first()
         cls.interfaces = (
-            Interface.objects.create(device=cls.devices[0], name="Interface 1", type="1000base-t"),
-            Interface.objects.create(device=cls.devices[0], name="Interface 2", type="1000base-t"),
-            Interface.objects.create(device=cls.devices[0], name="Interface 3", type=InterfaceTypeChoices.TYPE_BRIDGE),
             Interface.objects.create(
-                device=cls.devices[1], name="Interface 4", type=InterfaceTypeChoices.TYPE_1GE_GBIC
+                device=cls.devices[0],
+                name="Interface 1",
+                type="1000base-t",
+                status=non_default_status,
+            ),
+            Interface.objects.create(
+                device=cls.devices[0],
+                name="Interface 2",
+                type="1000base-t",
+                status=non_default_status,
+            ),
+            Interface.objects.create(
+                device=cls.devices[0],
+                name="Interface 3",
+                type=InterfaceTypeChoices.TYPE_BRIDGE,
+                status=non_default_status,
+            ),
+            Interface.objects.create(
+                device=cls.devices[1],
+                name="Interface 4",
+                type=InterfaceTypeChoices.TYPE_1GE_GBIC,
+                status=non_default_status,
             ),
-            Interface.objects.create(device=cls.devices[1], name="Interface 5", type=InterfaceTypeChoices.TYPE_LAG),
-            Interface.objects.create(device=cls.devices[2], name="Interface 6", type=InterfaceTypeChoices.TYPE_LAG),
             Interface.objects.create(
-                device=cls.devices[2], name="Interface 7", type=InterfaceTypeChoices.TYPE_1GE_GBIC
+                device=cls.devices[1],
+                name="Interface 5",
+                type=InterfaceTypeChoices.TYPE_LAG,
+                status=non_default_status,
+            ),
+            Interface.objects.create(
+                device=cls.devices[2],
+                name="Interface 6",
+                type=InterfaceTypeChoices.TYPE_LAG,
+                status=non_default_status,
+            ),
+            Interface.objects.create(
+                device=cls.devices[2],
+                name="Interface 7",
+                type=InterfaceTypeChoices.TYPE_1GE_GBIC,
+                status=non_default_status,
             ),
         )
 
+        vlan_status = Status.objects.get_for_model(VLAN).first()
         cls.vlans = (
-            VLAN.objects.create(name="VLAN 1", vid=1),
-            VLAN.objects.create(name="VLAN 2", vid=2),
-            VLAN.objects.create(name="VLAN 3", vid=3),
+            VLAN.objects.create(name="VLAN 1", vid=1, status=vlan_status),
+            VLAN.objects.create(name="VLAN 2", vid=2, status=vlan_status),
+            VLAN.objects.create(name="VLAN 3", vid=3, status=vlan_status),
         )
 
         cls.create_data = [
@@ -1454,6 +1489,7 @@ class InterfaceTest(Mixins.BasePortTestMixin):
                 "mode": InterfaceModeChoices.MODE_TAGGED,
                 "tagged_vlans": [cls.vlans[0].pk, cls.vlans[1].pk],
                 "untagged_vlan": cls.vlans[2].pk,
+                "mac_address": "00-01-02-03-04-05",
             },
             {
                 "device": cls.devices[0].pk,
@@ -1620,6 +1656,7 @@ class InterfaceTest(Mixins.BasePortTestMixin):
                 name="Tagged Interface",
                 mode=InterfaceModeChoices.MODE_TAGGED,
                 type=InterfaceTypeChoices.TYPE_VIRTUAL,
+                status=Status.objects.get_for_model(Interface).first(),
             )
             interface.tagged_vlans.add(self.vlans[0])
             payload = {"mode": None, "tagged_vlans": [self.vlans[2].pk]}
@@ -1760,28 +1797,33 @@ class DeviceBayTest(Mixins.BaseComponentTestMixin):
         )
 
         devices = (
+            # "Device 1" was already created in super().setUpTestData
             Device.objects.create(
                 device_type=device_types[0],
                 role=cls.device_role,
-                name="Device 1",
+                status=cls.device_status,
+                name="Device 2",
                 location=cls.location,
             ),
             Device.objects.create(
                 device_type=device_types[1],
                 role=cls.device_role,
-                name="Device 2",
+                status=cls.device_status,
+                name="Device 3",
                 location=cls.location,
             ),
             Device.objects.create(
                 device_type=device_types[1],
                 role=cls.device_role,
-                name="Device 3",
+                status=cls.device_status,
+                name="Device 4",
                 location=cls.location,
             ),
             Device.objects.create(
                 device_type=device_types[1],
                 role=cls.device_role,
-                name="Device 4",
+                status=cls.device_status,
+                name="Device 5",
                 location=cls.location,
             ),
         )
@@ -1809,9 +1851,8 @@ class DeviceBayTest(Mixins.BaseComponentTestMixin):
         ]
 
 
-class InventoryItemTest(Mixins.BaseComponentTestMixin):
+class InventoryItemTest(Mixins.BaseComponentTestMixin, APIViewTestCases.TreeModelAPIViewTestCaseMixin):
     model = InventoryItem
-    brief_fields = ["device", "display", "id", "name", "tree_depth", "url"]
     choices_fields = []
 
     @classmethod
@@ -1840,10 +1881,18 @@ class InventoryItemTest(Mixins.BaseComponentTestMixin):
             },
         ]
 
+    # TODO: Unskip after resolving #2908, #2909
+    @skip("DRF's built-in InventoryItem nautral_key is infinitely recursive")
+    def test_list_objects_ascending_ordered(self):
+        pass
+
+    @skip("DRF's built-in InventoryItem nautral_key is infinitely recursive")
+    def test_list_objects_descending_ordered(self):
+        pass
+
 
 class CableTest(Mixins.BaseComponentTestMixin):
     model = Cable
-    brief_fields = ["display", "id", "label", "url"]
     bulk_update_data = {
         "length": 100,
         "length_unit": "m",
@@ -1860,18 +1909,21 @@ class CableTest(Mixins.BaseComponentTestMixin):
             Device.objects.create(
                 device_type=cls.device_type,
                 role=cls.device_role,
+                status=cls.device_status,
                 name="Device 2",
                 location=cls.location,
             ),
             Device.objects.create(
                 device_type=cls.device_type,
                 role=cls.device_role,
+                status=cls.device_status,
                 name="Device 3",
                 location=cls.location,
             ),
         )
 
         interfaces = []
+        interface_status = Status.objects.get_for_model(Interface).first()
         for device in devices:
             for i in range(0, 10):
                 interfaces.append(
@@ -1879,6 +1931,7 @@ class CableTest(Mixins.BaseComponentTestMixin):
                         device=device,
                         type=InterfaceTypeChoices.TYPE_1GE_FIXED,
                         name=f"eth{i}",
+                        status=interface_status,
                     )
                 )
 
@@ -1938,23 +1991,27 @@ class ConnectedDeviceTest(APITestCase):
         location = Location.objects.filter(location_type=LocationType.objects.get(name="Campus")).first()
         device_type = DeviceType.objects.exclude(manufacturer__isnull=True).first()
         device_role = Role.objects.get_for_model(Device).first()
+        device_status = Status.objects.get_for_model(Device).first()
 
         cable_status = Status.objects.get_for_model(Cable).get(name="Connected")
 
         self.device1 = Device.objects.create(
             device_type=device_type,
             role=device_role,
+            status=device_status,
             name="TestDevice1",
             location=location,
         )
         device2 = Device.objects.create(
             device_type=device_type,
             role=device_role,
+            status=device_status,
             name="TestDevice2",
             location=location,
         )
-        interface1 = Interface.objects.create(device=self.device1, name="eth0")
-        interface2 = Interface.objects.create(device=device2, name="eth0")
+        interface_status = Status.objects.get_for_model(Interface).first()
+        interface1 = Interface.objects.create(device=self.device1, name="eth0", status=interface_status)
+        interface2 = Interface.objects.create(device=device2, name="eth0", status=interface_status)
 
         cable = Cable(termination_a=interface1, termination_b=interface2, status=cable_status)
         cable.validated_save()
@@ -1969,90 +2026,103 @@ class ConnectedDeviceTest(APITestCase):
 
 class VirtualChassisTest(APIViewTestCases.APIViewTestCase):
     model = VirtualChassis
-    brief_fields = ["display", "id", "master", "member_count", "name", "url"]
 
     @classmethod
     def setUpTestData(cls):
         location = Location.objects.filter(location_type=LocationType.objects.get(name="Campus")).first()
         device_type = DeviceType.objects.exclude(manufacturer__isnull=True).first()
         device_role = Role.objects.get_for_model(Device).first()
+        device_status = Status.objects.get_for_model(Device).first()
 
         devices = (
             Device.objects.create(
                 name="Device 1",
                 device_type=device_type,
                 role=device_role,
+                status=device_status,
                 location=location,
             ),
             Device.objects.create(
                 name="Device 2",
                 device_type=device_type,
                 role=device_role,
+                status=device_status,
                 location=location,
             ),
             Device.objects.create(
                 name="Device 3",
                 device_type=device_type,
                 role=device_role,
+                status=device_status,
                 location=location,
             ),
             Device.objects.create(
                 name="Device 4",
                 device_type=device_type,
                 role=device_role,
+                status=device_status,
                 location=location,
             ),
             Device.objects.create(
                 name="Device 5",
                 device_type=device_type,
                 role=device_role,
+                status=device_status,
                 location=location,
             ),
             Device.objects.create(
                 name="Device 6",
                 device_type=device_type,
                 role=device_role,
+                status=device_status,
                 location=location,
             ),
             Device.objects.create(
                 name="Device 7",
                 device_type=device_type,
                 role=device_role,
+                status=device_status,
                 location=location,
             ),
             Device.objects.create(
                 name="Device 8",
                 device_type=device_type,
                 role=device_role,
+                status=device_status,
                 location=location,
             ),
             Device.objects.create(
                 name="Device 9",
                 device_type=device_type,
                 role=device_role,
+                status=device_status,
                 location=location,
             ),
             Device.objects.create(
                 name="Device 10",
                 device_type=device_type,
                 role=device_role,
+                status=device_status,
                 location=location,
             ),
             Device.objects.create(
                 name="Device 11",
                 device_type=device_type,
                 role=device_role,
+                status=device_status,
                 location=location,
             ),
             Device.objects.create(
                 name="Device 12",
                 device_type=device_type,
                 role=device_role,
+                status=device_status,
                 location=location,
             ),
         )
 
         # Create 12 interfaces per device
+        interface_status = Status.objects.get_for_model(Interface).first()
         interfaces = []
         for i, device in enumerate(devices):
             for j in range(0, 13):
@@ -2062,6 +2132,7 @@ class VirtualChassisTest(APIViewTestCases.APIViewTestCase):
                         device=device,
                         name=f"{i%3+1}/{j}",
                         type=InterfaceTypeChoices.TYPE_1GE_FIXED,
+                        status=interface_status,
                     )
                 )
 
@@ -2137,9 +2208,10 @@ class VirtualChassisTest(APIViewTestCases.APIViewTestCase):
         virtual_chassis_1 = response.json()["results"][0]
 
         # Make sure the master is set
-        self.assertNotEqual(virtual_chassis_1["master"], None)
+        self.assertIsNotNone(virtual_chassis_1["master"])
 
-        master_device = Device.objects.get(pk=virtual_chassis_1["master"]["id"])
+        # The `master` key will be a URL now, but it contains the PK
+        master_device = Device.objects.get(pk=virtual_chassis_1["master"].split("/")[-2])
 
         # Set the virtual_chassis of the master device to null
         url = reverse("dcim-api:device-detail", kwargs={"pk": master_device.id})
@@ -2159,7 +2231,6 @@ class VirtualChassisTest(APIViewTestCases.APIViewTestCase):
 
 class PowerPanelTest(APIViewTestCases.APIViewTestCase):
     model = PowerPanel
-    brief_fields = ["display", "id", "name", "power_feed_count", "url"]
 
     @classmethod
     def setUpTestData(cls):
@@ -2198,7 +2269,6 @@ class PowerPanelTest(APIViewTestCases.APIViewTestCase):
 
 class PowerFeedTest(APIViewTestCases.APIViewTestCase):
     model = PowerFeed
-    brief_fields = ["cable", "display", "id", "name", "url"]
     choices_fields = ["phase", "supply", "type"]
 
     @classmethod
@@ -2206,12 +2276,21 @@ class PowerFeedTest(APIViewTestCases.APIViewTestCase):
         location = Location.objects.filter(location_type=LocationType.objects.get(name="Campus")).first()
         rackgroup = RackGroup.objects.create(location=location, name="Rack Group 1", slug="rack-group-1")
         rackrole = Role.objects.get_for_model(Rack).first()
+        rackstatus = Status.objects.get_for_model(Rack).first()
 
         racks = (
-            Rack.objects.create(location=location, rack_group=rackgroup, role=rackrole, name="Rack 1"),
-            Rack.objects.create(location=location, rack_group=rackgroup, role=rackrole, name="Rack 2"),
-            Rack.objects.create(location=location, rack_group=rackgroup, role=rackrole, name="Rack 3"),
-            Rack.objects.create(location=location, rack_group=rackgroup, role=rackrole, name="Rack 4"),
+            Rack.objects.create(
+                location=location, rack_group=rackgroup, role=rackrole, name="Rack 1", status=rackstatus
+            ),
+            Rack.objects.create(
+                location=location, rack_group=rackgroup, role=rackrole, name="Rack 2", status=rackstatus
+            ),
+            Rack.objects.create(
+                location=location, rack_group=rackgroup, role=rackrole, name="Rack 3", status=rackstatus
+            ),
+            Rack.objects.create(
+                location=location, rack_group=rackgroup, role=rackrole, name="Rack 4", status=rackstatus
+            ),
         )
 
         power_panels = (
@@ -2221,40 +2300,47 @@ class PowerFeedTest(APIViewTestCases.APIViewTestCase):
 
         PRIMARY = PowerFeedTypeChoices.TYPE_PRIMARY
         REDUNDANT = PowerFeedTypeChoices.TYPE_REDUNDANT
+        pf_status = Status.objects.get_for_model(PowerFeed).first()
         PowerFeed.objects.create(
             power_panel=power_panels[0],
             rack=racks[0],
             name="Power Feed 1A",
+            status=pf_status,
             type=PRIMARY,
         )
         PowerFeed.objects.create(
             power_panel=power_panels[1],
             rack=racks[0],
             name="Power Feed 1B",
+            status=pf_status,
             type=REDUNDANT,
         )
         PowerFeed.objects.create(
             power_panel=power_panels[0],
             rack=racks[1],
             name="Power Feed 2A",
+            status=pf_status,
             type=PRIMARY,
         )
         PowerFeed.objects.create(
             power_panel=power_panels[1],
             rack=racks[1],
             name="Power Feed 2B",
+            status=pf_status,
             type=REDUNDANT,
         )
         PowerFeed.objects.create(
             power_panel=power_panels[0],
             rack=racks[2],
             name="Power Feed 3A",
+            status=pf_status,
             type=PRIMARY,
         )
         PowerFeed.objects.create(
             power_panel=power_panels[1],
             rack=racks[2],
             name="Power Feed 3B",
+            status=pf_status,
             type=REDUNDANT,
         )
 
@@ -2283,7 +2369,6 @@ class PowerFeedTest(APIViewTestCases.APIViewTestCase):
 
 class DeviceRedundancyGroupTest(APIViewTestCases.APIViewTestCase):
     model = DeviceRedundancyGroup
-    brief_fields = ["display", "failover_strategy", "id", "name", "url"]
     choices_fields = ["failover_strategy"]
 
     @classmethod