nautobot 2.0.0a2__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
@@ -50,7 +51,7 @@ from nautobot.dcim.models import (
     VirtualChassis,
 )
 from nautobot.extras.models import ConfigContextSchema, Role, SecretsGroup, Status
-from nautobot.ipam.models import IPAddress, VLAN
+from nautobot.ipam.models import IPAddress, VLAN, Namespace, Prefix
 from nautobot.tenancy.models import Tenant
 from nautobot.virtualization.models import Cluster, ClusterType
 
@@ -61,7 +62,6 @@ User = get_user_model()
 
 class AppTest(APITestCase):
     def test_root(self):
-
         url = reverse("dcim-api:api-root")
         response = self.client.get(f"{url}?format=api", **self.header)
 
@@ -83,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}")
@@ -106,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",
         }
@@ -119,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,
@@ -178,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"]
 
@@ -191,18 +198,18 @@ class LocationTest(APIViewTestCases.APIViewTestCase):
         cls.lt3 = LocationType.objects.get(name="Floor")
         cls.lt4 = LocationType.objects.get(name="Room")
 
-        cls.status_active = Status.objects.get(slug="active")
-        tenant = Tenant.objects.create(name="Test Tenant")
+        cls.location_statuses = Status.objects.get_for_model(Location)
+        tenant = Tenant.objects.first()
 
-        cls.loc1 = Location.objects.create(name="RTP", location_type=cls.lt1, status=cls.status_active)
+        cls.loc1 = Location.objects.create(name="RTP", location_type=cls.lt1, status=cls.location_statuses[0])
         cls.loc2 = Location.objects.create(
-            name="RTP4E", location_type=cls.lt2, status=cls.status_active, parent=cls.loc1
+            name="RTP4E", location_type=cls.lt2, status=cls.location_statuses[0], parent=cls.loc1
         )
         cls.loc3 = Location.objects.create(
-            name="RTP4E-3", location_type=cls.lt3, status=cls.status_active, parent=cls.loc2
+            name="RTP4E-3", location_type=cls.lt3, status=cls.location_statuses[0], parent=cls.loc2
         )
         cls.loc4 = Location.objects.create(
-            name="RTP4E-3-0101", location_type=cls.lt4, status=cls.status_active, parent=cls.loc3, tenant=tenant
+            name="RTP4E-3-0101", location_type=cls.lt4, status=cls.location_statuses[1], parent=cls.loc3, tenant=tenant
         )
         for loc in [cls.loc1, cls.loc2, cls.loc3, cls.loc4]:
             loc.validated_save()
@@ -211,35 +218,33 @@ class LocationTest(APIViewTestCases.APIViewTestCase):
             {
                 "name": "Downtown Durham",
                 "location_type": cls.lt1.pk,
-                "status": cls.status_active.pk,
+                "status": cls.location_statuses[0].pk,
             },
             {
                 "name": "RTP12",
                 "slug": "rtp-12",
                 "location_type": cls.lt2.pk,
                 "parent": cls.loc1.pk,
-                "status": cls.status_active.pk,
+                "status": cls.location_statuses[0].pk,
             },
             {
                 "name": "RTP4E-2",
                 "location_type": cls.lt3.pk,
                 "parent": cls.loc2.pk,
-                "status": cls.status_active.pk,
+                "status": cls.location_statuses[0].pk,
                 "description": "Second floor of RTP4E",
                 "tenant": tenant.pk,
             },
         ]
 
-        status_planned = Status.objects.get(slug="planned")
-
         # Changing location_type of an existing instance is not permitted
         cls.update_data = {
             "name": "A revised location",
             "slug": "a-different-slug",
-            "status": status_planned.pk,
+            "status": cls.location_statuses[1].pk,
         }
         cls.bulk_update_data = {
-            "status": status_planned.pk,
+            "status": cls.location_statuses[1].pk,
         }
 
     def test_time_zone_field_post_null(self):
@@ -252,10 +257,9 @@ class LocationTest(APIViewTestCases.APIViewTestCase):
         location = {
             "name": "foo",
             "slug": "foo",
-            "status": self.status_active.pk,
+            "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,10 +277,9 @@ class LocationTest(APIViewTestCases.APIViewTestCase):
         location = {
             "name": "foo",
             "slug": "foo",
-            "status": self.status_active.pk,
+            "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,10 +298,9 @@ class LocationTest(APIViewTestCases.APIViewTestCase):
         location = {
             "name": "foo",
             "slug": "foo",
-            "status": self.status_active.pk,
+            "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,10 +319,9 @@ class LocationTest(APIViewTestCases.APIViewTestCase):
         location = {
             "name": "foo",
             "slug": "foo",
-            "status": self.status_active.pk,
+            "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.
@@ -344,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",
     }
@@ -354,11 +354,11 @@ class RackGroupTest(APIViewTestCases.APIViewTestCase):
 
     @classmethod
     def setUpTestData(cls):
-        cls.active = Status.objects.get(slug="active")
+        cls.status = Status.objects.get_for_model(Location).first()
         location_type = LocationType.objects.create(name="Location Type 1")
         cls.locations = (
-            Location.objects.create(name="Location 1", location_type=location_type, status=cls.active),
-            Location.objects.create(name="Location 2", location_type=location_type, status=cls.active),
+            Location.objects.create(name="Location 1", location_type=location_type, status=cls.status),
+            Location.objects.create(name="Location 2", location_type=location_type, status=cls.status),
         )
         cls.parent_rack_groups = (
             RackGroup.objects.create(location=cls.locations[0], name="Parent Rack Group 1", slug="parent-rack-group-1"),
@@ -423,7 +423,7 @@ class RackGroupTest(APIViewTestCases.APIViewTestCase):
         )
         child_location_type.content_types.add(ContentType.objects.get_for_model(RackGroup))
         child_location = Location.objects.create(
-            name="Child Location", location_type=child_location_type, parent=self.locations[0], status=self.active
+            name="Child Location", location_type=child_location_type, parent=self.locations[0], status=self.status
         )
 
         data = {
@@ -442,7 +442,7 @@ class RackGroupTest(APIViewTestCases.APIViewTestCase):
         parent_group = RackGroup.objects.filter(location=self.locations[0]).first()
         # A sibling of locations[0], not a child of it.
         sibling_location = Location.objects.create(
-            name="Location 1B", location_type=self.locations[0].location_type, status=self.active
+            name="Location 1B", location_type=self.locations[0].location_type, status=self.status
         )
 
         data = {
@@ -462,12 +462,10 @@ 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
     def setUpTestData(cls):
-
         locations = Location.objects.all()[:2]
 
         rack_groups = (
@@ -556,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)
 
@@ -608,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",
     }
@@ -617,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")
@@ -655,19 +653,15 @@ class RackReservationTest(APIViewTestCases.APIViewTestCase):
 
 class ManufacturerTest(APIViewTestCases.APIViewTestCase):
     model = Manufacturer
-    brief_fields = ["device_type_count", "display", "id", "name", "slug", "url"]
     create_data = [
         {
             "name": "Test Manufacturer 4",
-            "slug": "test-manufacturer-4",
         },
         {
             "name": "Test Manufacturer 5",
-            "slug": "test-manufacturer-5",
         },
         {
             "name": "Test Manufacturer 6",
-            "slug": "test-manufacturer-6",
         },
         {
             "name": "Test Manufacturer 7",
@@ -676,11 +670,9 @@ class ManufacturerTest(APIViewTestCases.APIViewTestCase):
     bulk_update_data = {
         "description": "New description",
     }
-    slug_source = "name"
 
     @classmethod
     def setUpTestData(cls):
-
         # FIXME(jathan): This has to be replaced with# `get_deletable_object` and
         # `get_deletable_object_pks` but this is a workaround just so all of these objects are
         # deletable for now.
@@ -690,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",
     }
@@ -1028,19 +1011,15 @@ class DeviceBayTemplateTest(Mixins.BasePortTemplateTestMixin):
 
 class PlatformTest(APIViewTestCases.APIViewTestCase):
     model = Platform
-    brief_fields = ["device_count", "display", "id", "name", "slug", "url", "virtual_machine_count"]
     create_data = [
         {
             "name": "Test Platform 4",
-            "slug": "test-platform-4",
         },
         {
             "name": "Test Platform 5",
-            "slug": "test-platform-5",
         },
         {
             "name": "Test Platform 6",
-            "slug": "test-platform-6",
         },
         {
             "name": "Test Platform 7",
@@ -1049,27 +1028,25 @@ class PlatformTest(APIViewTestCases.APIViewTestCase):
     bulk_update_data = {
         "description": "New description",
     }
-    slug_source = "name"
 
 
 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)
 
-        cluster_type = ClusterType.objects.create(name="Cluster Type 1", slug="cluster-type-1")
+        cluster_type = ClusterType.objects.create(name="Cluster Type 1")
 
         clusters = (
             Cluster.objects.create(name="Cluster 1", cluster_type=cluster_type),
@@ -1077,8 +1054,8 @@ class DeviceTest(APIViewTestCases.APIViewTestCase):
         )
 
         secrets_groups = (
-            SecretsGroup.objects.create(name="Secrets Group 1", slug="secrets-group-1"),
-            SecretsGroup.objects.create(name="Secrets Group 2", slug="secrets-group-2"),
+            SecretsGroup.objects.create(name="Secrets Group 1"),
+            SecretsGroup.objects.create(name="Secrets Group 2"),
         )
 
         device_type = DeviceType.objects.first()
@@ -1154,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):
         """
@@ -1211,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):
         """
@@ -1240,8 +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")
-        dev_ip_addr = IPAddress.objects.create(address="192.0.2.1/24")
+        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, 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}
@@ -1417,17 +1400,23 @@ class InterfaceTest(Mixins.BasePortTestMixin):
     @classmethod
     def setUpTestData(cls):
         super().setUpTestData()
-        status_active = Status.objects.get(slug="active")
+        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,
             ),
         )
 
@@ -1437,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,
+                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[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 = [
@@ -1462,16 +1485,17 @@ class InterfaceTest(Mixins.BasePortTestMixin):
                 "device": cls.devices[0].pk,
                 "name": "Interface 8",
                 "type": "1000base-t",
-                "status": status_active.pk,
+                "status": interface_status.pk,
                 "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,
                 "name": "Interface 9",
                 "type": "1000base-t",
-                "status": status_active.pk,
+                "status": interface_status.pk,
                 "mode": InterfaceModeChoices.MODE_TAGGED,
                 "bridge": cls.interfaces[3].pk,
                 "tagged_vlans": [cls.vlans[0].pk, cls.vlans[1].pk],
@@ -1481,7 +1505,7 @@ class InterfaceTest(Mixins.BasePortTestMixin):
                 "device": cls.devices[0].pk,
                 "name": "Interface 10",
                 "type": "virtual",
-                "status": status_active.pk,
+                "status": interface_status.pk,
                 "mode": InterfaceModeChoices.MODE_TAGGED,
                 "parent_interface": cls.interfaces[1].pk,
                 "tagged_vlans": [cls.vlans[0].pk, cls.vlans[1].pk],
@@ -1493,7 +1517,7 @@ class InterfaceTest(Mixins.BasePortTestMixin):
             "device": cls.devices[0].pk,
             "name": "expected-to-fail",
             "type": InterfaceTypeChoices.TYPE_VIRTUAL,
-            "status": status_active.pk,
+            "status": interface_status.pk,
             "untagged_vlan": cls.vlans[0].pk,
         }
 
@@ -1502,7 +1526,7 @@ class InterfaceTest(Mixins.BasePortTestMixin):
                 "device": cls.devices[0].pk,
                 "name": "interface test 1",
                 "type": InterfaceTypeChoices.TYPE_VIRTUAL,
-                "status": status_active.pk,
+                "status": interface_status.pk,
                 "parent_interface": cls.interfaces[3].id,  # belongs to different device but same vc
                 "bridge": cls.interfaces[2].id,  # belongs to different device but same vc
             },
@@ -1510,7 +1534,7 @@ class InterfaceTest(Mixins.BasePortTestMixin):
                 "device": cls.devices[0].pk,
                 "name": "interface test 2",
                 "type": InterfaceTypeChoices.TYPE_1GE_GBIC,
-                "status": status_active.pk,
+                "status": interface_status.pk,
                 "lag": cls.interfaces[4].id,  # belongs to different device but same vc
             },
         ]
@@ -1522,7 +1546,7 @@ class InterfaceTest(Mixins.BasePortTestMixin):
                     "device": cls.devices[0].pk,
                     "name": "interface test 1",
                     "type": InterfaceTypeChoices.TYPE_VIRTUAL,
-                    "status": status_active.pk,
+                    "status": interface_status.pk,
                     "parent_interface": cls.interfaces[6].id,  # do not belong to same device or vc
                 },
             ],
@@ -1532,7 +1556,7 @@ class InterfaceTest(Mixins.BasePortTestMixin):
                     "device": cls.devices[0].pk,
                     "name": "interface test 2",
                     "type": InterfaceTypeChoices.TYPE_1GE_GBIC,
-                    "status": status_active.pk,
+                    "status": interface_status.pk,
                     "bridge": cls.interfaces[6].id,  # does not belong to same device or vc
                 },
             ],
@@ -1542,7 +1566,7 @@ class InterfaceTest(Mixins.BasePortTestMixin):
                     "device": cls.devices[0].pk,
                     "name": "interface test 3",
                     "type": InterfaceTypeChoices.TYPE_1GE_GBIC,
-                    "status": status_active.pk,
+                    "status": interface_status.pk,
                     "lag": cls.interfaces[6].id,  # does not belong to same device or vc
                 },
             ],
@@ -1632,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]}
@@ -1772,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,
             ),
         )
@@ -1821,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
@@ -1852,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",
@@ -1872,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(
@@ -1891,6 +1931,7 @@ class CableTest(Mixins.BaseComponentTestMixin):
                         device=device,
                         type=InterfaceTypeChoices.TYPE_1GE_FIXED,
                         name=f"eth{i}",
+                        status=interface_status,
                     )
                 )
 
@@ -1945,29 +1986,32 @@ class CableTest(Mixins.BaseComponentTestMixin):
 
 class ConnectedDeviceTest(APITestCase):
     def setUp(self):
-
         super().setUp()
 
         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(slug="connected")
+        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()
@@ -1982,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):
@@ -2075,6 +2132,7 @@ class VirtualChassisTest(APIViewTestCases.APIViewTestCase):
                         device=device,
                         name=f"{i%3+1}/{j}",
                         type=InterfaceTypeChoices.TYPE_1GE_FIXED,
+                        status=interface_status,
                     )
                 )
 
@@ -2150,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})
@@ -2172,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):
@@ -2211,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
@@ -2219,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 = (
@@ -2234,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,
         )
 
@@ -2296,7 +2369,6 @@ class PowerFeedTest(APIViewTestCases.APIViewTestCase):
 
 class DeviceRedundancyGroupTest(APIViewTestCases.APIViewTestCase):
     model = DeviceRedundancyGroup
-    brief_fields = ["display", "failover_strategy", "id", "name", "slug", "url"]
     choices_fields = ["failover_strategy"]
 
     @classmethod