nautobot 3.0.0a2__py3-none-any.whl → 3.0.0rc1__py3-none-any.whl

nautobot/dcim/tests/test_tables.py

@@ -0,0 +1,160 @@
+from django.test import TestCase
+
+from nautobot.dcim.choices import InterfaceDuplexChoices, InterfaceSpeedChoices, InterfaceTypeChoices
+from nautobot.dcim.models import Device, DeviceType, Interface, InterfaceTemplate, Location, LocationType, Manufacturer
+from nautobot.dcim.tables.devices import DeviceModuleInterfaceTable, InterfaceTable
+from nautobot.dcim.tables.devicetypes import InterfaceTemplateTable
+from nautobot.extras.models import Role, Status
+
+
+class InterfaceTableRenderMixin:
+    """Mixin for testing render_speed methods on interface tables."""
+
+    table_class = None
+
+    @classmethod
+    def setUpTestData(cls):
+        manufacturer = Manufacturer.objects.create(name="Test Manufacturer")
+        device_type = DeviceType.objects.create(manufacturer=manufacturer, model="Test Device Type")
+        device_role = Role.objects.get_for_model(Device).first()
+        location_type = LocationType.objects.get(name="Campus")
+        location = Location.objects.filter(location_type=location_type).first()
+        device_status = Status.objects.get_for_model(Device).first()
+        cls.interface_status = Status.objects.get_for_model(Interface).first()
+
+        cls.device = Device.objects.create(
+            name="Test Device",
+            device_type=device_type,
+            role=device_role,
+            location=location,
+            status=device_status,
+        )
+
+    def test_render_speed_duplex_with_value(self):
+        """Test that the table renders humanized speed values."""
+        interface = Interface.objects.create(
+            device=self.device,
+            name="eth0",
+            type=InterfaceTypeChoices.TYPE_1GE_FIXED,
+            status=self.interface_status,
+            speed=InterfaceSpeedChoices.SPEED_1G,
+            duplex=InterfaceDuplexChoices.DUPLEX_FULL,
+        )
+
+        queryset = Interface.objects.filter(pk=interface.pk)
+        table = self.table_class(queryset)  # pylint: disable=not-callable
+        bound_row = table.rows[0]
+        rendered_speed = bound_row.get_cell("speed")
+        rendered_duplex = bound_row.get_cell("duplex")
+
+        self.assertEqual(rendered_speed, "1 Gbps")
+        self.assertEqual(rendered_duplex, "Full")
+
+    def test_render_speed_duplex_with_none(self):
+        """Test that the table handles None speed value and renders an emdash."""
+        emdash = "\u2014"
+        interface = Interface.objects.create(
+            device=self.device,
+            name="eth1",
+            type=InterfaceTypeChoices.TYPE_1GE_FIXED,
+            status=self.interface_status,
+            speed=None,
+        )
+
+        queryset = Interface.objects.filter(pk=interface.pk)
+        table = self.table_class(queryset)  # pylint: disable=not-callable
+        bound_row = table.rows[0]
+        rendered_speed = bound_row.get_cell("speed")
+        rendered_duplex = bound_row.get_cell("duplex")
+
+        self.assertEqual(rendered_speed, emdash)
+        self.assertEqual(rendered_duplex, emdash)
+
+    def test_render_speed_various(self):
+        """Test that the table correctly humanizes various speed values."""
+        # Test all speed choices defined in InterfaceSpeedChoices
+        for speed_value, expected_output in InterfaceSpeedChoices.CHOICES:
+            with self.subTest(speed_value=speed_value, expected=expected_output):
+                interface = Interface.objects.create(
+                    device=self.device,
+                    name=f"eth-{speed_value}",
+                    type=InterfaceTypeChoices.TYPE_1GE_FIXED,
+                    status=self.interface_status,
+                    speed=speed_value,
+                )
+
+                queryset = Interface.objects.filter(pk=interface.pk)
+                table = self.table_class(queryset)  # pylint: disable=not-callable
+                bound_row = table.rows[0]
+                rendered_speed = bound_row.get_cell("speed")
+
+                self.assertEqual(rendered_speed, expected_output)
+
+    def test_render_duplex_various(self):
+        """Test that the table correctly renders various duplex values."""
+        for duplex_value, expected_output in InterfaceDuplexChoices.CHOICES:
+            with self.subTest(duplex_value=duplex_value, expected=expected_output):
+                interface = Interface.objects.create(
+                    device=self.device,
+                    name=f"eth-{duplex_value}",
+                    type=InterfaceTypeChoices.TYPE_1GE_FIXED,
+                    status=self.interface_status,
+                    duplex=duplex_value,
+                )
+
+                queryset = Interface.objects.filter(pk=interface.pk)
+                table = self.table_class(queryset)  # pylint: disable=not-callable
+                bound_row = table.rows[0]
+                rendered_duplex = bound_row.get_cell("duplex")
+
+                self.assertEqual(rendered_duplex, expected_output)
+
+
+class InterfaceTableTestCase(InterfaceTableRenderMixin, TestCase):
+    """Test cases for InterfaceTable."""
+
+    table_class = InterfaceTable
+
+
+class DeviceModuleInterfaceTableTestCase(InterfaceTableRenderMixin, TestCase):
+    """Test cases for DeviceModuleInterfaceTable."""
+
+    table_class = DeviceModuleInterfaceTable
+
+
+class InterfaceTemplateTableTestCase(TestCase):
+    """Render tests for InterfaceTemplateTable speed/duplex columns."""
+
+    @classmethod
+    def setUpTestData(cls):
+        manufacturer = Manufacturer.objects.create(name="Test Manuf Tmpl")
+        cls.device_type = DeviceType.objects.create(manufacturer=manufacturer, model="DT-Tmpl")
+
+    def test_render_speed_duplex_with_value(self):
+        interface_template = InterfaceTemplate.objects.create(
+            device_type=self.device_type,
+            name="tmpl-eth0",
+            type=InterfaceTypeChoices.TYPE_1GE_FIXED,
+            speed=InterfaceSpeedChoices.SPEED_1G,
+            duplex=InterfaceDuplexChoices.DUPLEX_FULL,
+        )
+        table = InterfaceTemplateTable(InterfaceTemplate.objects.filter(pk=interface_template.pk))
+        bound_row = table.rows[0]
+        rendered_speed = bound_row.get_cell("speed")  # pylint: disable=no-member
+        rendered_duplex = bound_row.get_cell("duplex")  # pylint: disable=no-member
+        self.assertEqual(rendered_speed, "1 Gbps")
+        self.assertEqual(rendered_duplex, "Full")
+
+    def test_render_speed_duplex_with_none(self):
+        emdash = "\u2014"
+        interface_template = InterfaceTemplate.objects.create(
+            device_type=self.device_type,
+            name="tmpl-eth1",
+            type=InterfaceTypeChoices.TYPE_1GE_FIXED,
+        )
+        table = InterfaceTemplateTable(InterfaceTemplate.objects.filter(pk=interface_template.pk))
+        bound_row = table.rows[0]
+        rendered_speed = bound_row.get_cell("speed")  # pylint: disable=no-member
+        rendered_duplex = bound_row.get_cell("duplex")  # pylint: disable=no-member
+        self.assertEqual(rendered_speed, emdash)
+        self.assertEqual(rendered_duplex, emdash)

nautobot/dcim/tests/test_views.py

@@ -6,8 +6,7 @@ import zoneinfo
 from django.contrib.auth import get_user_model
 from django.contrib.contenttypes.models import ContentType
 from django.db.models import Q
-from django.test import override_settings, tag
-from django.test.client import RequestFactory
+from django.test import override_settings
 from django.urls import reverse
 from django.utils.html import strip_spaces_between_tags
 from netaddr import EUI
@@ -31,6 +30,7 @@ from nautobot.dcim.choices import (
     ConsolePortTypeChoices,
     DeviceFaceChoices,
     DeviceRedundancyGroupFailoverStrategyChoices,
+    InterfaceDuplexChoices,
     InterfaceModeChoices,
     InterfaceRedundancyGroupProtocolChoices,
     InterfaceTypeChoices,
@@ -109,7 +109,6 @@ from nautobot.dcim.models import (
 from nautobot.dcim.views import (
     ConsoleConnectionsListView,
     InterfaceConnectionsListView,
-    LocationUIViewSet,
     PowerConnectionsListView,
 )
 from nautobot.extras.choices import CustomFieldTypeChoices, RelationshipTypeChoices
@@ -211,8 +210,10 @@ class LocationTypeTestCase(ViewTestCases.OrganizationalObjectViewTestCase, ViewT
 
 class LocationTestCase(ViewTestCases.PrimaryObjectViewTestCase):
     model = Location
-    # One query for the natural slug, one for `LocationViewSet.get_extra_context`
-    allowed_number_of_tree_queries_per_view_type = {"retrieve": 2}
+    # One query for the natural slug, one for `LocationViewSet.get_extra_context`, and additional for distinct (which may be fixable?)
+    # See: https://github.com/nautobot/nautobot/pull/7530#discussion_r2432836062
+    # and https://github.com/nautobot/nautobot/pull/7530#discussion_r2432620239 for additional context
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 3}
 
     @classmethod
     def setUpTestData(cls):
@@ -442,7 +443,6 @@ class LocationTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         self.assertEqual(location.contact_email, "")
 
     def test_get_extra_context(self):
-        view_set = LocationUIViewSet()
         child_1, child_2 = Location.objects.filter(name__startswith="Leaf ")
         parent_location = child_1.parent
         child_1.location_type.content_types.add(ContentType.objects.get_for_model(Prefix))
@@ -451,12 +451,24 @@ class LocationTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         prefix_2 = Prefix.objects.create(network="192.0.2.128", prefix_length=25, status=status)
         prefix_1.locations.set([child_1, child_2])
         prefix_2.locations.set([child_1, child_2])
-        request = RequestFactory().get(parent_location.get_absolute_url())
-        request.user = self.user
+
         self.add_permissions("dcim.view_location")
         self.add_permissions("ipam.view_prefix")
-        context = view_set.get_extra_context(request=request, instance=parent_location)
-        self.assertEqual(context["stats"]["prefix_count"], 2)
+
+        url = parent_location.get_absolute_url()
+        context = self.client.get(url).context
+        prefix_count = (
+            Prefix.objects.filter(location__in=parent_location.descendants(include_self=True)).distinct().count()
+        )
+
+        # Ensure that the context contains "stats" and the expected stat for prefix_list
+        self.assertIn("stats", context)
+        found = False
+        for stat in context["stats"].values():
+            if stat[0] == "ipam:prefix_list":
+                self.assertEqual(stat[1], prefix_count)
+                found = True
+        self.assertTrue(found, "ipam:prefix_list stat not found in context['stats']")
 
 
 class RackGroupTestCase(ViewTestCases.OrganizationalObjectViewTestCase, ViewTestCases.BulkEditObjectsViewTestCase):
@@ -1282,7 +1294,6 @@ class ModuleTypeTestCase(
             "comments": "changed comment",
         }
 
-    @tag("fix_in_v3")
     def test_list_has_correct_links(self):
         """Assert that the ModuleType list view has import/export buttons for both CSV and YAML/JSON formats."""
         self.add_permissions("dcim.add_moduletype", "dcim.view_moduletype")
@@ -1294,11 +1305,11 @@ class ModuleTypeTestCase(
         csv_import_url = job_import_url(ContentType.objects.get_for_model(ModuleType))
         # Dropdown provides both YAML/JSON and CSV import as options
         self.assertInHTML(
-            f'<a href="{yaml_import_url}"><span class="mdi mdi-database-import text-secondary" aria-hidden="true"></span> Import from JSON/YAML (single record)</a>',
+            f'<a class="dropdown-item" href="{yaml_import_url}"><span class="mdi mdi-database-import text-secondary" aria-hidden="true"></span> Import from JSON/YAML (single record)</a>',
             content,
         )
         self.assertInHTML(
-            f'<a href="{csv_import_url}"><span class="mdi mdi-database-import text-secondary" aria-hidden="true"></span> Import from CSV (multiple records)</a>',
+            f'<a class="dropdown-item" href="{csv_import_url}"><span class="mdi mdi-database-import text-secondary" aria-hidden="true"></span> Import from CSV (multiple records)</a>',
             content,
         )
 
@@ -1311,11 +1322,11 @@ class ModuleTypeTestCase(
         )
         self.assertInHTML('<input type="hidden" name="export_format" value="yaml">', content)
         self.assertInHTML(
-            '<button type="submit"><span class="mdi mdi-database-export text-secondary" aria-hidden="true"></span> Export as YAML</button>',
+            '<button class="dropdown-item" type="submit"><span class="mdi mdi-database-export text-secondary" aria-hidden="true"></span> Export as YAML</button>',
             content,
         )
         self.assertInHTML(
-            '<button type="submit"><span class="mdi mdi-database-export text-secondary" aria-hidden="true"></span> Export as CSV</button>',
+            '<button class="dropdown-item" type="submit"><span class="mdi mdi-database-export text-secondary" aria-hidden="true"></span> Export as CSV</button>',
             content,
         )
 
@@ -1827,6 +1838,68 @@ class InterfaceTemplateTestCase(ViewTestCases.DeviceComponentTemplateViewTestCas
             "description": "new test description",
         }
 
+    @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
+    def test_create_base_t_with_speed_and_duplex(self):
+        self.add_permissions("dcim.add_interfacetemplate", "dcim.view_devicetype")
+        url = reverse("dcim:interfacetemplate_add")
+        dt = DeviceType.objects.first()
+        data = {
+            "device_type": dt.pk,
+            "name_pattern": "Eth-View-1",
+            "type": InterfaceTypeChoices.TYPE_1GE_FIXED,
+            "mgmt_only": False,
+            "speed": 1_000_000,
+            "duplex": InterfaceDuplexChoices.DUPLEX_FULL,
+            "_create": True,
+        }
+        response = self.client.post(url, data)
+
+        # Successful create redirects (JobResult or object list)
+        self.assertIn(response.status_code, (302, 303))
+        interface_template = InterfaceTemplate.objects.get(name="Eth-View-1")
+        self.assertEqual(interface_template.speed, 1_000_000)
+        self.assertEqual(interface_template.duplex, InterfaceDuplexChoices.DUPLEX_FULL)
+
+    @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
+    def test_create_sfp_with_duplex_rejected(self):
+        self.add_permissions("dcim.add_interfacetemplate", "dcim.view_devicetype")
+        url = reverse("dcim:interfacetemplate_add")
+        dt = DeviceType.objects.first()
+        data = {
+            "device_type": dt.pk,
+            "name_pattern": "SFP-View-1",
+            "type": InterfaceTypeChoices.TYPE_1GE_SFP,
+            "duplex": InterfaceDuplexChoices.DUPLEX_FULL,
+            "_create": True,
+        }
+        response = self.client.post(url, data)
+        # Form error returns 200 with field error displayed
+        self.assertEqual(response.status_code, 200)
+        content = response.content.decode(response.charset)
+        self.assertIn("Duplex is only applicable to copper twisted-pair interfaces.", content)
+
+    @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
+    def test_bulk_create_with_speed_and_duplex(self):
+        self.add_permissions("dcim.add_interfacetemplate", "dcim.view_devicetype")
+        url = reverse("dcim:interfacetemplate_add")
+        dt = DeviceType.objects.first()
+        data = {
+            "device_type": dt.pk,
+            "name_pattern": "Et[1-2]",
+            "type": InterfaceTypeChoices.TYPE_1GE_FIXED,
+            "mgmt_only": False,
+            "speed": 1_000_000,
+            "duplex": InterfaceDuplexChoices.DUPLEX_FULL,
+            "_apply": True,
+        }
+        response = self.client.post(url, data)
+        self.assertIn(response.status_code, (302, 303))
+        objs = InterfaceTemplate.objects.filter(name__in=["Et1", "Et2"]).order_by("name")
+        self.assertEqual(objs.count(), 2)
+        for obj in objs:
+            self.assertEqual(obj.speed, 1_000_000)
+            self.assertEqual(obj.duplex, InterfaceDuplexChoices.DUPLEX_FULL)
+
 
 class FrontPortTemplateTestCase(ViewTestCases.DeviceComponentTemplateViewTestCase):
     model = FrontPortTemplate
@@ -2186,7 +2259,7 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         )
         cls.custom_fields[0].content_types.set([ContentType.objects.get_for_model(Device)])
 
-        devices = (
+        cls.devices = (
             Device.objects.create(
                 name="Device 1",
                 location=locations[0],
@@ -2251,16 +2324,40 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         intf_status = Status.objects.get_for_model(Interface).first()
         intf_role = Role.objects.get_for_model(Interface).first()
         cls.interfaces = (
-            Interface.objects.create(device=devices[0], name="Interface A1", status=intf_status, role=intf_role),
-            Interface.objects.create(device=devices[0], name="Interface A2", status=intf_status),
-            Interface.objects.create(device=devices[0], name="Interface A3", status=intf_status, role=intf_role),
+            Interface.objects.create(device=cls.devices[0], name="Interface A1", status=intf_status, role=intf_role),
+            Interface.objects.create(device=cls.devices[0], name="Interface A2", status=intf_status),
+            Interface.objects.create(device=cls.devices[0], name="Interface A3", status=intf_status, role=intf_role),
         )
 
-        for device, ipaddress in zip(devices, ipaddresses):
+        for device, ipaddress in zip(cls.devices, ipaddresses):
             RelationshipAssociation(
                 relationship=cls.relationships[0], source=device, destination=ipaddress
             ).validated_save()
 
+        powerports = (
+            PowerPort.objects.create(device=cls.devices[0], name="Power Port 1"),
+            PowerPort.objects.create(device=cls.devices[0], name="Power Port 2"),
+            PowerPort.objects.create(device=cls.devices[0], name="Power Port 3"),
+        )
+        PowerOutlet.objects.create(device=cls.devices[0], name="Power Outlet A")
+
+        powerpanel = PowerPanel.objects.create(location=locations[0], name="Power Panel 1")
+        pf_status = Status.objects.get_for_model(PowerFeed).first()
+        powerfeed = PowerFeed.objects.create(
+            power_panel=powerpanel,
+            name="Power Feed 1",
+            status=pf_status,
+            available_power=1000,
+            phase=PowerFeedPhaseChoices.PHASE_3PHASE,
+        )
+
+        status_connected = Status.objects.get(name="Connected")
+
+        Cable.objects.create(termination_a=powerports[0], termination_b=powerfeed, status=status_connected)
+
+        poweroutlet = PowerOutlet.objects.create(device=cls.devices[1], name="Power Outlet 1")
+        Cable.objects.create(termination_a=powerports[1], termination_b=poweroutlet, status=status_connected)
+
         cls.form_data = {
             "device_type": devicetypes[1].pk,
             "role": deviceroles[1].pk,
@@ -2408,11 +2505,7 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
 
     @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
     def test_device_powerports(self):
-        device = Device.objects.first()
-
-        PowerPort.objects.create(device=device, name="Power Port 1")
-        PowerPort.objects.create(device=device, name="Power Port 2")
-        PowerPort.objects.create(device=device, name="Power Port 3")
+        device = self.devices[0]
 
         url = reverse("dcim:device_powerports", kwargs={"pk": device.pk})
         self.assertHttpStatus(self.client.get(url), 200)
@@ -2596,6 +2689,15 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         response = self.client.get(url)
         self.assertHttpStatus(response, 200)
 
+    @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
+    def test_device_vpnendpoints(self):
+        self.add_permissions("vpn.view_vpntunnelendpoint")
+        device = Device.objects.first()
+
+        url = reverse("dcim:device_vpnendpoints", kwargs={"pk": device.pk})
+        response = self.client.get(url)
+        self.assertHttpStatus(response, 200)
+
     @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
     def test_device_napalm_tabs(self):
         self.add_permissions("dcim.napalm_read_device")
@@ -4076,8 +4178,8 @@ class PowerConnectionsTestCase(ViewTestCases.ListObjectsViewTestCase):
         )
 
         poweroutlets = (
-            PowerOutlet.objects.create(device=device_2, name="Power Outlet 1", power_port=powerports[0]),
-            PowerOutlet.objects.create(device=device_2, name="Power Outlet 2", power_port=powerports[1]),
+            PowerOutlet.objects.create(device=device_2, name="Power Outlet 1"),
+            PowerOutlet.objects.create(device=device_2, name="Power Outlet 2"),
         )
 
         powerpanel = PowerPanel.objects.create(location=location, name="Power Panel 1")
@@ -4552,7 +4654,8 @@ class PathTraceViewTestCase(ModelViewTestCase):
         url = reverse("dcim:rearport_trace", args=[obj.pk])
         cablepath_id = CablePath.objects.first().id
         response = self.client.get(url + f"?cablepath_id={cablepath_id}")
-        self.assertBodyContains(response, "<h1>Cable Trace for Rear Port Rear Port 1</h1>", html=True)
+        self.assertBodyContains(response, "Rear Port 1")
+        self.assertBodyContains(response, "eth0")
 
 
 class DeviceRedundancyGroupTestCase(ViewTestCases.PrimaryObjectViewTestCase):
@@ -4781,7 +4884,7 @@ class ControllerTestCase(ViewTestCases.PrimaryObjectViewTestCase):
     model = Controller
     filterset = ControllerFilterSet
     custom_action_required_permissions = {
-        "dcim:controller_wirelessnetworks": [
+        "dcim:controller_wireless_networks": [
             "dcim.view_controller",
             "wireless.view_controllermanageddevicegroupwirelessnetworkassignment",
         ],

nautobot/dcim/urls.py

@@ -7,7 +7,6 @@ from nautobot.ipam.views import ServiceEditView
 
 from . import views
 from .models import (
-    Cable,
     ConsolePort,
     ConsoleServerPort,
     Device,
@@ -26,6 +25,7 @@ from .models import (
 app_name = "dcim"
 
 router = NautobotUIViewSetRouter()
+router.register("cables", views.CableUIViewSet)
 router.register("controller-managed-device-groups", views.ControllerManagedDeviceGroupUIViewSet)
 router.register("controllers", views.ControllerUIViewSet)
 router.register("device-families", views.DeviceFamilyUIViewSet)
@@ -84,6 +84,69 @@ urlpatterns = [
         views.DeviceTypeImportView.as_view(),
         name="devicetype_import",
     ),
+    path(
+        "device-types/<uuid:pk>/console-port-templates/add/",
+        RedirectView.as_view(
+            url="/dcim/console-port-templates/add/?device_type=%(pk)s&return_url=/dcim/device-types/%(pk)s/console-ports/"
+        ),
+        name="devicetype_consoleporttemplate_add",
+    ),
+    path(
+        "devices-types/<uuid:pk>/console-server-port-templates/add/",
+        RedirectView.as_view(
+            url="/dcim/console-server-port-templates/add/?device_type=%(pk)s&return_url=/dcim/device-types/%(pk)s/console-server-ports/"
+        ),
+        name="devicetype_consoleserverporttemplate_add",
+    ),
+    path(
+        "device-types/<uuid:pk>/power-port-templates/add/",
+        RedirectView.as_view(
+            url="/dcim/power-port-templates/add/?device_type=%(pk)s&return_url=/dcim/device-types/%(pk)s/power-ports/"
+        ),
+        name="devicetype_powerporttemplate_add",
+    ),
+    path(
+        "device-types/<uuid:pk>/power-outlet-templates/add/",
+        RedirectView.as_view(
+            url="/dcim/power-outlet-templates/add/?device_type=%(pk)s&return_url=/dcim/device-types/%(pk)s/power-outlets/"
+        ),
+        name="devicetype_poweroutlettemplate_add",
+    ),
+    path(
+        "device-types/<uuid:pk>/interface-templates/add/",
+        RedirectView.as_view(
+            url="/dcim/interface-templates/add/?device_type=%(pk)s&return_url=/dcim/device-types/%(pk)s/interfaces/"
+        ),
+        name="devicetype_interfacetemplate_add",
+    ),
+    path(
+        "device-types/<uuid:pk>/front-port-templates/add/",
+        RedirectView.as_view(
+            url="/dcim/front-port-templates/add/?device_type=%(pk)s&return_url=/dcim/device-types/%(pk)s/front-ports/"
+        ),
+        name="devicetype_frontporttemplate_add",
+    ),
+    path(
+        "device-types/<uuid:pk>/rear-port-templates/add/",
+        RedirectView.as_view(
+            url="/dcim/rear-port-templates/add/?device_type=%(pk)s&return_url=/dcim/device-types/%(pk)s/rear-ports/"
+        ),
+        name="devicetype_rearporttemplate_add",
+    ),
+    path(
+        "device-types/<uuid:pk>/device-bay-templates/add/",
+        RedirectView.as_view(
+            url="/dcim/device-bay-templates/add/?device_type=%(pk)s&return_url=/dcim/device-types/%(pk)s/device-bays/"
+        ),
+        name="devicetype_devicebaytemplate_add",
+    ),
+    path(
+        "device-types/<uuid:pk>/module-bay-templates/add/",
+        RedirectView.as_view(
+            url="/dcim/module-bay-templates/add/?device_type=%(pk)s&return_url=/dcim/device-types/%(pk)s/module-bays/"
+        ),
+        name="devicetype_modulebaytemplate_add",
+    ),
     # Console port templates
     path(
         "console-port-templates/add/",
@@ -1024,26 +1087,6 @@ urlpatterns = [
         RedirectView.as_view(url="/dcim/inventory-items/add/?device=%(pk)s&return_url=/dcim/devices/%(pk)s/inventory/"),
         name="device_inventoryitems_add",
     ),
-    # Cables
-    path("cables/", views.CableListView.as_view(), name="cable_list"),
-    path("cables/import/", views.CableBulkImportView.as_view(), name="cable_import"),  # 3.0 TODO: remove, unused
-    path("cables/edit/", views.CableBulkEditView.as_view(), name="cable_bulk_edit"),
-    path("cables/delete/", views.CableBulkDeleteView.as_view(), name="cable_bulk_delete"),
-    path("cables/<uuid:pk>/", views.CableView.as_view(), name="cable"),
-    path("cables/<uuid:pk>/edit/", views.CableEditView.as_view(), name="cable_edit"),
-    path("cables/<uuid:pk>/delete/", views.CableDeleteView.as_view(), name="cable_delete"),
-    path(
-        "cables/<uuid:pk>/changelog/",
-        ObjectChangeLogView.as_view(),
-        name="cable_changelog",
-        kwargs={"model": Cable},
-    ),
-    path(
-        "cables/<uuid:pk>/notes/",
-        ObjectNotesView.as_view(),
-        name="cable_notes",
-        kwargs={"model": Cable},
-    ),
     # Console/power/interface connections (read-only)
     path(
         "console-connections/",

nautobot/dcim/utils.py

@@ -49,9 +49,9 @@ def cable_status_color_css(record):
         return ""
     else:
         CABLE_STATUS_TO_CSS_CLASS = {
-            ColorChoices.COLOR_GREEN: "success",
-            ColorChoices.COLOR_AMBER: "warning",
-            ColorChoices.COLOR_CYAN: "info",
+            ColorChoices.COLOR_GREEN: "table-success",
+            ColorChoices.COLOR_AMBER: "table-warning",
+            ColorChoices.COLOR_CYAN: "table-info",
         }
         status_color = record.cable.get_status_color().strip("#")
         return CABLE_STATUS_TO_CSS_CLASS.get(status_color, "")