nautobot 2.2.2__py3-none-any.whl → 2.2.4__py3-none-any.whl

nautobot/dcim/tests/test_views.py

@@ -14,8 +14,15 @@ import yaml
 from nautobot.circuits.choices import CircuitTerminationSideChoices
 from nautobot.circuits.models import Circuit, CircuitTermination, CircuitType, Provider
 from nautobot.core.templatetags.buttons import job_export_url, job_import_url
-from nautobot.core.testing import extract_page_body, ModelViewTestCase, post_data, ViewTestCases
-from nautobot.core.testing.utils import generate_random_device_asset_tag_of_specified_size
+from nautobot.core.testing import (
+    extract_page_body,
+    ModelViewTestCase,
+    post_data,
+    ViewTestCases,
+)
+from nautobot.core.testing.utils import (
+    generate_random_device_asset_tag_of_specified_size,
+)
 from nautobot.dcim.choices import (
     CableLengthUnitChoices,
     CableTypeChoices,
@@ -25,6 +32,7 @@ from nautobot.dcim.choices import (
     InterfaceModeChoices,
     InterfaceRedundancyGroupProtocolChoices,
     InterfaceTypeChoices,
+    LocationDataToContactActionChoices,
     PortTypeChoices,
     PowerFeedPhaseChoices,
     PowerFeedSupplyChoices,
@@ -89,10 +97,16 @@ from nautobot.dcim.models import (
     SoftwareVersion,
     VirtualChassis,
 )
-from nautobot.dcim.views import ConsoleConnectionsListView, InterfaceConnectionsListView, PowerConnectionsListView
+from nautobot.dcim.views import (
+    ConsoleConnectionsListView,
+    InterfaceConnectionsListView,
+    PowerConnectionsListView,
+)
 from nautobot.extras.choices import CustomFieldTypeChoices, RelationshipTypeChoices
 from nautobot.extras.models import (
     ConfigContextSchema,
+    Contact,
+    ContactAssociation,
     CustomField,
     CustomFieldChoice,
     ExternalIntegration,
@@ -102,6 +116,7 @@ from nautobot.extras.models import (
     SecretsGroup,
     Status,
     Tag,
+    Team,
 )
 from nautobot.ipam.choices import IPAddressTypeChoices
 from nautobot.ipam.models import IPAddress, Namespace, Prefix, VLAN, VLANGroup, VRF
@@ -128,7 +143,11 @@ def create_test_device(name):
     devicerole.content_types.add(device_ct)
     devicestatus = Status.objects.get_for_model(Device).first()
     device = Device.objects.create(
-        name=name, location=location, device_type=devicetype, role=devicerole, status=devicestatus
+        name=name,
+        location=location,
+        device_type=devicetype,
+        role=devicerole,
+        status=devicestatus,
     )
 
     return device
@@ -161,7 +180,10 @@ class LocationTypeTestCase(ViewTestCases.OrganizationalObjectViewTestCase):
             "name": "Intermediate 2",
             # "parent": lt1.pk, # TODO: Either overload how EditObjectViewTestCase finds an editable object or write a specific test case for this.
             "description": "Another intermediate type",
-            "content_types": [ContentType.objects.get_for_model(Rack).pk, ContentType.objects.get_for_model(Device).pk],
+            "content_types": [
+                ContentType.objects.get_for_model(Rack).pk,
+                ContentType.objects.get_for_model(Device).pk,
+            ],
             "nestable": True,
         }
 
@@ -174,6 +196,8 @@ class LocationTestCase(ViewTestCases.PrimaryObjectViewTestCase):
 
     @classmethod
     def setUpTestData(cls):
+        cls.contact_statuses = Status.objects.get_for_model(ContactAssociation)
+        cls.contact_roles = Role.objects.get_for_model(ContactAssociation)
         lt1 = LocationType.objects.get(name="Campus")
         lt2 = LocationType.objects.get(name="Building")
         lt3 = LocationType.objects.get(name="Floor")
@@ -186,7 +210,13 @@ class LocationTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         loc1 = Location.objects.create(name="Root 1", location_type=lt1, status=status)
         loc2 = Location.objects.create(name="Root 2", location_type=lt1, status=status, tenant=tenant)
         loc3 = Location.objects.create(name="Intermediate 1", location_type=lt2, parent=loc2, status=status)
-        loc4 = Location.objects.create(name="Leaf 1", location_type=lt3, parent=loc3, status=status, description="Hi!")
+        loc4 = Location.objects.create(
+            name="Leaf 1",
+            location_type=lt3,
+            parent=loc3,
+            status=status,
+            description="Hi!",
+        )
         for loc in [loc1, loc2, loc3, loc4]:
             loc.validated_save()
 
@@ -222,7 +252,9 @@ class LocationTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         }
 
     @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
-    def test_create_child_location_under_a_non_globally_unique_named_parent_location(self):
+    def test_create_child_location_under_a_non_globally_unique_named_parent_location(
+        self,
+    ):
         self.add_permissions("dcim.add_location")
         status = Status.objects.get_for_model(Location).first()
         region_type = LocationType.objects.create(name="Region")
@@ -252,6 +284,137 @@ class LocationTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         self.assertHttpStatus(self.client.post(**request), 302)
         self.assertEqual(Location.objects.get(name="Root 3").parent.pk, site_1.pk)
 
+    @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
+    def test_migrate_location_data_from_location_assign(self):
+        self.add_permissions("dcim.change_location")
+        location = Location.objects.first()
+        location.contact_name = "Should be unique Contact Name"
+        location.contact_phone = "123123123"
+        location.contact_email = "helloword@example.com"
+        location.physical_address = "418 Brown Locks Barrettchester, NM 85792"
+        location.shipping_address = "53 blue Locks manchester, NY 12124"
+        similar_contact = Contact.objects.first()
+        role = self.contact_roles.first().pk
+        status = self.contact_statuses.first().pk
+        form_data = {
+            "action": LocationDataToContactActionChoices.ASSOCIATE_EXISTING_CONTACT,
+            "contact": similar_contact.pk,
+            "role": role,
+            "status": status,
+        }
+        request = {
+            "path": reverse("dcim:location_migrate_data_to_contact", kwargs={"pk": location.pk}),
+            "data": post_data(form_data),
+        }
+        # Assert permission checks are triggered
+        self.assertHttpStatus(self.client.post(**request), 200)
+        self.add_permissions("extras.add_contactassociation")
+        self.assertHttpStatus(self.client.post(**request), 302)
+        # assert ContactAssociation is created correctly
+        created_contact_association = ContactAssociation.objects.order_by("created").last()
+        self.assertEqual(created_contact_association.associated_object_id, location.pk)
+        self.assertEqual(created_contact_association.contact.pk, similar_contact.pk)
+        self.assertEqual(created_contact_association.role.pk, role)
+        self.assertEqual(created_contact_association.status.pk, status)
+
+        # assert location data is cleared out
+        location.refresh_from_db()
+        self.assertEqual(location.contact_name, "")
+        self.assertEqual(location.contact_phone, "")
+        self.assertEqual(location.contact_email, "")
+
+    @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
+    def test_migrate_location_data_from_location_new_contact(self):
+        self.add_permissions("dcim.change_location")
+        location = Location.objects.first()
+        location.contact_name = "Should be unique Contact Name"
+        location.contact_phone = "123123123"
+        location.contact_email = "helloword@example.com"
+        location.physical_address = "418 Brown Locks Barrettchester, NM 85792"
+        location.shipping_address = "53 blue Locks manchester, NY 12124"
+        role = self.contact_roles.first().pk
+        status = self.contact_statuses.first().pk
+        form_data = {
+            "action": LocationDataToContactActionChoices.CREATE_AND_ASSIGN_NEW_CONTACT,
+            "name": "Should be unique Contact Name",
+            "phone": "123123123",
+            "email": "helloword@example.com",
+            "role": role,
+            "status": status,
+        }
+        request = {
+            "path": reverse("dcim:location_migrate_data_to_contact", kwargs={"pk": location.pk}),
+            "data": post_data(form_data),
+        }
+        # Assert permission checks are triggered
+        self.assertHttpStatus(self.client.post(**request), 200)
+        self.add_permissions("extras.add_contactassociation")
+        self.add_permissions("extras.add_contact")
+        self.assertHttpStatus(self.client.post(**request), 302)
+        # assert a new contact is created successfully
+        contact = Contact.objects.get(name="Should be unique Contact Name")
+        self.assertEqual(contact.name, form_data["name"])
+        self.assertEqual(contact.phone, form_data["phone"])
+        self.assertEqual(contact.email, form_data["email"])
+        # assert ContactAssociation is created correctly
+        created_contact_association = ContactAssociation.objects.order_by("created").last()
+        self.assertEqual(created_contact_association.associated_object_id, location.pk)
+        self.assertEqual(created_contact_association.contact.pk, contact.pk)
+        self.assertEqual(created_contact_association.role.pk, role)
+        self.assertEqual(created_contact_association.status.pk, status)
+
+        # assert location data is cleared out
+        location.refresh_from_db()
+        self.assertEqual(location.contact_name, "")
+        self.assertEqual(location.contact_phone, "")
+        self.assertEqual(location.contact_email, "")
+
+    @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
+    def test_migrate_location_data_from_location_new_team(self):
+        self.add_permissions("dcim.change_location")
+        location = Location.objects.first()
+        location.contact_name = "Should be unique Team Name"
+        location.contact_phone = "123123123"
+        location.contact_email = "helloword@example.com"
+        location.physical_address = "418 Brown Locks Barrettchester, NM 85792"
+        location.shipping_address = "53 blue Locks manchester, NY 12124"
+        role = self.contact_roles.first().pk
+        status = self.contact_statuses.first().pk
+        form_data = {
+            "action": LocationDataToContactActionChoices.CREATE_AND_ASSIGN_NEW_TEAM,
+            "name": "Should be unique Team Name",
+            "phone": "123123123",
+            "email": "helloword@example.com",
+            "role": role,
+            "status": status,
+        }
+        request = {
+            "path": reverse("dcim:location_migrate_data_to_contact", kwargs={"pk": location.pk}),
+            "data": post_data(form_data),
+        }
+        # Assert permission checks are triggered
+        self.assertHttpStatus(self.client.post(**request), 200)
+        self.add_permissions("extras.add_contactassociation")
+        self.add_permissions("extras.add_team")
+        self.assertHttpStatus(self.client.post(**request), 302)
+        # assert a new team is created successfully
+        team = Team.objects.get(name="Should be unique Team Name")
+        self.assertEqual(team.name, form_data["name"])
+        self.assertEqual(team.phone, form_data["phone"])
+        self.assertEqual(team.email, form_data["email"])
+        # assert ContactAssociation is created correctly
+        created_contact_association = ContactAssociation.objects.order_by("created").last()
+        self.assertEqual(created_contact_association.associated_object_id, location.pk)
+        self.assertEqual(created_contact_association.team.pk, team.pk)
+        self.assertEqual(created_contact_association.role.pk, role)
+        self.assertEqual(created_contact_association.status.pk, status)
+
+        # assert location data is cleared out
+        location.refresh_from_db()
+        self.assertEqual(location.contact_name, "")
+        self.assertEqual(location.contact_phone, "")
+        self.assertEqual(location.contact_email, "")
+
 
 class RackGroupTestCase(ViewTestCases.OrganizationalObjectViewTestCase):
     model = RackGroup
@@ -340,7 +503,11 @@ class RackTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         cls.cable_connected = cable_statuses.get(name="Connected")
 
         cls.custom_fields = (
-            CustomField.objects.create(type=CustomFieldTypeChoices.TYPE_MULTISELECT, label="Rack Colors", default=[]),
+            CustomField.objects.create(
+                type=CustomFieldTypeChoices.TYPE_MULTISELECT,
+                label="Rack Colors",
+                default=[],
+            ),
         )
 
         CustomFieldChoice.objects.create(custom_field=cls.custom_fields[0], value="red")
@@ -389,7 +556,9 @@ class RackTestCase(ViewTestCases.PrimaryObjectViewTestCase):
 
         for rack in racks:
             RelationshipAssociation(
-                relationship=cls.relationships[0], source=rack, destination=cls.locations[1]
+                relationship=cls.relationships[0],
+                source=rack,
+                destination=cls.locations[1],
             ).validated_save()
 
         cls.form_data = {
@@ -498,19 +667,31 @@ class RackTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         poweroutlet1 = PowerOutlet.objects.create(device=devices[0], name="Power Outlet 11", power_port=powerport1)
 
         # connect power port to power feed (single-phase)
-        cable1 = Cable(termination_a=powerfeed1, termination_b=powerport1, status=self.cable_connected)
+        cable1 = Cable(
+            termination_a=powerfeed1,
+            termination_b=powerport1,
+            status=self.cable_connected,
+        )
         cable1.save()
 
         # Create power port for 2nd device
         powerport2 = PowerPort.objects.create(device=devices[1], name="Power Port 12", allocated_draw=1200)
 
         # Connect power port to power outlet (dev1)
-        cable2 = Cable(termination_a=powerport2, termination_b=poweroutlet1, status=self.cable_connected)
+        cable2 = Cable(
+            termination_a=powerport2,
+            termination_b=poweroutlet1,
+            status=self.cable_connected,
+        )
         cable2.save()
 
         # Create another power port for 2nd device and directly connect to the second PowerFeed.
         powerport3 = PowerPort.objects.create(device=devices[1], name="Power Port 13", allocated_draw=2400)
-        cable3 = Cable(termination_a=powerfeed2, termination_b=powerport3, status=self.cable_connected)
+        cable3 = Cable(
+            termination_a=powerfeed2,
+            termination_b=powerport3,
+            status=self.cable_connected,
+        )
         cable3.save()
 
         # Test the view
@@ -654,7 +835,10 @@ class DeviceTypeTestCase(
             content,
         )
         self.assertInHTML('<input type="hidden" name="export_format" value="yaml">', content)
-        self.assertInHTML('<button type="submit" class="btn btn-link">YAML format</button>', content)
+        self.assertInHTML(
+            '<button type="submit" class="btn btn-link" form="export_default">YAML format</button>',
+            content,
+        )
         self.assertInHTML('<button type="submit" class="btn btn-link">CSV format</button>', content)
 
     @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
@@ -1294,7 +1478,12 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
 
         rack_status = Status.objects.get_for_model(Rack).first()
         racks = (
-            Rack.objects.create(name="Rack 1", location=locations[0], rack_group=rack_group, status=rack_status),
+            Rack.objects.create(
+                name="Rack 1",
+                location=locations[0],
+                rack_group=rack_group,
+                status=rack_status,
+            ),
             Rack.objects.create(name="Rack 2", location=locations[1], status=rack_status),
         )
 
@@ -1333,7 +1522,11 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         devicetypes[1].software_image_files.set(software_image_files[2:])
 
         cls.custom_fields = (
-            CustomField.objects.create(type=CustomFieldTypeChoices.TYPE_INTEGER, label="Crash Counter", default=0),
+            CustomField.objects.create(
+                type=CustomFieldTypeChoices.TYPE_INTEGER,
+                label="Crash Counter",
+                default=0,
+            ),
         )
         cls.custom_fields[0].content_types.set([ContentType.objects.get_for_model(Device)])
 
@@ -1518,7 +1711,11 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
     def test_device_interface_assign_ipaddress(self):
         device = Device.objects.first()
         self.add_permissions(
-            "ipam.add_ipaddress", "extras.view_status", "ipam.view_namespace", "dcim.view_device", "dcim.view_interface"
+            "ipam.add_ipaddress",
+            "extras.view_status",
+            "ipam.view_namespace",
+            "dcim.view_device",
+            "dcim.view_interface",
         )
         device_list_url = reverse("dcim:device_interfaces", args=(device.pk,))
         namespace = Namespace.objects.first()
@@ -1554,7 +1751,10 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
             self.assertHttpStatus(response, 200)
             self.interfaces[0].refresh_from_db()
             self.assertEqual(self.interfaces[0].ip_addresses.all().count(), 0)
-            self.assertIn(f"Interface with id &quot;{self.interfaces[0].pk}&quot; not found", response_body)
+            self.assertIn(
+                f"Interface with id &quot;{self.interfaces[0].pk}&quot; not found",
+                response_body,
+            )
 
         with self.subTest("Assert Cannnot assign IPAddress(Exsisting IP) without permission"):
             # Assert Assign Exsisting IPAddress
@@ -1563,7 +1763,10 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
             self.assertHttpStatus(response, 200)
             self.interfaces[1].refresh_from_db()
             self.assertEqual(self.interfaces[1].ip_addresses.all().count(), 0)
-            self.assertIn(f"Interface with id &quot;{self.interfaces[1].pk}&quot; not found", response_body)
+            self.assertIn(
+                f"Interface with id &quot;{self.interfaces[1].pk}&quot; not found",
+                response_body,
+            )
 
         self.add_permissions("dcim.change_interface", "ipam.view_ipaddress")
 
@@ -1571,7 +1774,8 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
             self.assertHttpStatus(self.client.post(**add_new_ip_request), 302)
             self.interfaces[0].refresh_from_db()
             self.assertEqual(
-                str(self.interfaces[0].ip_addresses.all().first().address), add_new_ip_form_data["address"]
+                str(self.interfaces[0].ip_addresses.all().first().address),
+                add_new_ip_form_data["address"],
             )
 
         with self.subTest("Assert Assign IPAddress"):
@@ -1681,7 +1885,8 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         Assert that the local context passes schema validation via full_clean()
         """
         schema = ConfigContextSchema.objects.create(
-            name="Schema 1", data_schema={"type": "object", "properties": {"foo": {"type": "string"}}}
+            name="Schema 1",
+            data_schema={"type": "object", "properties": {"foo": {"type": "string"}}},
         )
         self.add_permissions("dcim.add_device")
 
@@ -1695,7 +1900,10 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
             "data": post_data(form_data),
         }
         self.assertHttpStatus(self.client.post(**request), 302)
-        self.assertEqual(self._get_queryset().get(name="Device X").local_config_context_schema.pk, schema.pk)
+        self.assertEqual(
+            self._get_queryset().get(name="Device X").local_config_context_schema.pk,
+            schema.pk,
+        )
 
     @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
     def test_local_config_context_schema_validation_fails(self):
@@ -1705,7 +1913,8 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         Assert that the local context fails schema validation via full_clean()
         """
         schema = ConfigContextSchema.objects.create(
-            name="Schema 1", data_schema={"type": "object", "properties": {"foo": {"type": "integer"}}}
+            name="Schema 1",
+            data_schema={"type": "object", "properties": {"foo": {"type": "integer"}}},
         )
         self.add_permissions("dcim.add_device")
 
@@ -1921,10 +2130,16 @@ class InterfaceTestCase(ViewTestCases.DeviceComponentViewTestCase):
             Interface.objects.create(device=device, name="Interface 2", status=status_active),
             Interface.objects.create(device=device, name="Interface 3", status=status_active),
             Interface.objects.create(
-                device=device, name="LAG", status=status_active, type=InterfaceTypeChoices.TYPE_LAG
+                device=device,
+                name="LAG",
+                status=status_active,
+                type=InterfaceTypeChoices.TYPE_LAG,
             ),
             Interface.objects.create(
-                device=device, name="BRIDGE", status=status_active, type=InterfaceTypeChoices.TYPE_BRIDGE
+                device=device,
+                name="BRIDGE",
+                status=status_active,
+                type=InterfaceTypeChoices.TYPE_BRIDGE,
             ),
         )
         cls.lag_interface = interfaces[3]
@@ -1936,16 +2151,32 @@ class InterfaceTestCase(ViewTestCases.DeviceComponentViewTestCase):
         vlan_group = VLANGroup.objects.first()
         vlans = (
             VLAN.objects.create(
-                vid=1, name="VLAN1", location=device.location, status=vlan_status, vlan_group=vlan_group
+                vid=1,
+                name="VLAN1",
+                location=device.location,
+                status=vlan_status,
+                vlan_group=vlan_group,
             ),
             VLAN.objects.create(
-                vid=101, name="VLAN101", location=device.location, status=vlan_status, vlan_group=vlan_group
+                vid=101,
+                name="VLAN101",
+                location=device.location,
+                status=vlan_status,
+                vlan_group=vlan_group,
             ),
             VLAN.objects.create(
-                vid=102, name="VLAN102", location=device.location, status=vlan_status, vlan_group=vlan_group
+                vid=102,
+                name="VLAN102",
+                location=device.location,
+                status=vlan_status,
+                vlan_group=vlan_group,
             ),
             VLAN.objects.create(
-                vid=103, name="VLAN103", location=device.location, status=vlan_status, vlan_group=vlan_group
+                vid=103,
+                name="VLAN103",
+                location=device.location,
+                status=vlan_status,
+                vlan_group=vlan_group,
             ),
         )
 
@@ -2432,22 +2663,37 @@ class CableTestCase(
         circuittype = CircuitType.objects.first()
         circuit_status = Status.objects.get_for_model(Circuit).first()
         circuit = Circuit.objects.create(
-            cid="Circuit 1", provider=provider, circuit_type=circuittype, status=circuit_status
+            cid="Circuit 1",
+            provider=provider,
+            circuit_type=circuittype,
+            status=circuit_status,
         )
 
         circuit_terminations = [
             CircuitTermination.objects.create(
-                circuit=circuit, term_side=CircuitTerminationSideChoices.SIDE_A, location=location
+                circuit=circuit,
+                term_side=CircuitTerminationSideChoices.SIDE_A,
+                location=location,
             ),
             CircuitTermination.objects.create(
-                circuit=circuit, term_side=CircuitTerminationSideChoices.SIDE_Z, location=location
+                circuit=circuit,
+                term_side=CircuitTerminationSideChoices.SIDE_Z,
+                location=location,
             ),
         ]
 
         status = Status.objects.get_for_model(Cable).get(name="Connected")
         cables = [
-            Cable.objects.create(termination_a=circuit_terminations[0], termination_b=interfaces[0], status=status),
-            Cable.objects.create(termination_a=circuit_terminations[1], termination_b=interfaces[1], status=status),
+            Cable.objects.create(
+                termination_a=circuit_terminations[0],
+                termination_b=interfaces[0],
+                status=status,
+            ),
+            Cable.objects.create(
+                termination_a=circuit_terminations[1],
+                termination_b=interfaces[1],
+                status=status,
+            ),
         ]
 
         request = {
@@ -2467,7 +2713,10 @@ class CableTestCase(
         cable_path_1 = CablePath.objects.filter(
             Q(origin_type=termination_ct, origin_id=circuit_terminations[0].pk)
             | Q(origin_type=interface_ct, origin_id=interfaces[0].pk)
-            | Q(destination_type=termination_ct, destination_id=circuit_terminations[0].pk)
+            | Q(
+                destination_type=termination_ct,
+                destination_id=circuit_terminations[0].pk,
+            )
             | Q(destination_type=interface_ct, destination_id=interfaces[0].pk)
         )
         # pylint: enable=unsupported-binary-operation
@@ -2478,7 +2727,10 @@ class CableTestCase(
         cable_path_2 = CablePath.objects.filter(
             Q(origin_type=termination_ct, origin_id=circuit_terminations[1].pk)
             | Q(origin_type=interface_ct, origin_id=interfaces[1].pk)
-            | Q(destination_type=termination_ct, destination_id=circuit_terminations[1].pk)
+            | Q(
+                destination_type=termination_ct,
+                destination_id=circuit_terminations[1].pk,
+            )
             | Q(destination_type=interface_ct, destination_id=interfaces[1].pk)
         )
         # pylint: enable=unsupported-binary-operation
@@ -2523,9 +2775,21 @@ class ConsoleConnectionsTestCase(ViewTestCases.ListObjectsViewTestCase):
         )
         status_connected = Status.objects.get(name="Connected")
 
-        Cable.objects.create(termination_a=consoleports[0], termination_b=serverports[0], status=status_connected)
-        Cable.objects.create(termination_a=consoleports[1], termination_b=serverports[1], status=status_connected)
-        Cable.objects.create(termination_a=consoleports[2], termination_b=rearport, status=status_connected)
+        Cable.objects.create(
+            termination_a=consoleports[0],
+            termination_b=serverports[0],
+            status=status_connected,
+        )
+        Cable.objects.create(
+            termination_a=consoleports[1],
+            termination_b=serverports[1],
+            status=status_connected,
+        )
+        Cable.objects.create(
+            termination_a=consoleports[2],
+            termination_b=rearport,
+            status=status_connected,
+        )
 
 
 class PowerConnectionsTestCase(ViewTestCases.ListObjectsViewTestCase):
@@ -2572,10 +2836,22 @@ class PowerConnectionsTestCase(ViewTestCases.ListObjectsViewTestCase):
 
         status_connected = Status.objects.get(name="Connected")
 
-        Cable.objects.create(termination_a=powerports[2], termination_b=powerfeed, status=status_connected)
+        Cable.objects.create(
+            termination_a=powerports[2],
+            termination_b=powerfeed,
+            status=status_connected,
+        )
         # Creating a PowerOutlet with a PowerPort via the ORM does *not* automatically cable the two together. Bug?
-        Cable.objects.create(termination_a=powerports[0], termination_b=poweroutlets[0], status=status_connected)
-        Cable.objects.create(termination_a=powerports[1], termination_b=poweroutlets[1], status=status_connected)
+        Cable.objects.create(
+            termination_a=powerports[0],
+            termination_b=poweroutlets[0],
+            status=status_connected,
+        )
+        Cable.objects.create(
+            termination_a=powerports[1],
+            termination_b=poweroutlets[1],
+            status=status_connected,
+        )
 
 
 class InterfaceConnectionsTestCase(ViewTestCases.ListObjectsViewTestCase):
@@ -2608,18 +2884,30 @@ class InterfaceConnectionsTestCase(ViewTestCases.ListObjectsViewTestCase):
         interface_status = Status.objects.get_for_model(Interface).first()
         cls.interfaces = (
             Interface.objects.create(
-                device=device_1, name="Interface 1", type=InterfaceTypeChoices.TYPE_1GE_SFP, status=interface_status
+                device=device_1,
+                name="Interface 1",
+                type=InterfaceTypeChoices.TYPE_1GE_SFP,
+                status=interface_status,
             ),
             Interface.objects.create(
-                device=device_1, name="Interface 2", type=InterfaceTypeChoices.TYPE_1GE_SFP, status=interface_status
+                device=device_1,
+                name="Interface 2",
+                type=InterfaceTypeChoices.TYPE_1GE_SFP,
+                status=interface_status,
             ),
             Interface.objects.create(
-                device=device_1, name="Interface 3", type=InterfaceTypeChoices.TYPE_1GE_SFP, status=interface_status
+                device=device_1,
+                name="Interface 3",
+                type=InterfaceTypeChoices.TYPE_1GE_SFP,
+                status=interface_status,
             ),
         )
 
         cls.device_2_interface = Interface.objects.create(
-            device=device_2, name="Interface 1", type=InterfaceTypeChoices.TYPE_1GE_SFP, status=interface_status
+            device=device_2,
+            name="Interface 1",
+            type=InterfaceTypeChoices.TYPE_1GE_SFP,
+            status=interface_status,
         )
         rearport = RearPort.objects.create(device=device_2, type=PortTypeChoices.TYPE_8P8C)
 
@@ -2627,16 +2915,29 @@ class InterfaceConnectionsTestCase(ViewTestCases.ListObjectsViewTestCase):
         circuittype = CircuitType.objects.first()
         circuit_status = Status.objects.get_for_model(Circuit).first()
         circuit = Circuit.objects.create(
-            cid="Circuit 1", provider=provider, circuit_type=circuittype, status=circuit_status
+            cid="Circuit 1",
+            provider=provider,
+            circuit_type=circuittype,
+            status=circuit_status,
         )
         circuittermination = CircuitTermination.objects.create(
-            circuit=circuit, term_side=CircuitTerminationSideChoices.SIDE_A, location=location
+            circuit=circuit,
+            term_side=CircuitTerminationSideChoices.SIDE_A,
+            location=location,
         )
 
         connected = Status.objects.get(name="Connected")
 
-        Cable.objects.create(termination_a=cls.interfaces[0], termination_b=cls.device_2_interface, status=connected)
-        Cable.objects.create(termination_a=cls.interfaces[1], termination_b=circuittermination, status=connected)
+        Cable.objects.create(
+            termination_a=cls.interfaces[0],
+            termination_b=cls.device_2_interface,
+            status=connected,
+        )
+        Cable.objects.create(
+            termination_a=cls.interfaces[1],
+            termination_b=circuittermination,
+            status=connected,
+        )
         Cable.objects.create(termination_a=cls.interfaces[2], termination_b=rearport, status=connected)
 
     @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
@@ -2814,12 +3115,23 @@ class PowerFeedTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         status_planned = statuses[0]
 
         powerfeed_1 = PowerFeed.objects.create(
-            name="Power Feed 1", power_panel=powerpanels[0], rack=racks[0], status=status_planned
+            name="Power Feed 1",
+            power_panel=powerpanels[0],
+            rack=racks[0],
+            status=status_planned,
         )
         powerfeed_2 = PowerFeed.objects.create(
-            name="Power Feed 2", power_panel=powerpanels[0], rack=racks[0], status=status_planned
+            name="Power Feed 2",
+            power_panel=powerpanels[0],
+            rack=racks[0],
+            status=status_planned,
+        )
+        PowerFeed.objects.create(
+            name="Power Feed 3",
+            power_panel=powerpanels[0],
+            rack=racks[0],
+            status=status_planned,
         )
-        PowerFeed.objects.create(name="Power Feed 3", power_panel=powerpanels[0], rack=racks[0], status=status_planned)
 
         # Assign power feeds for the tests later
         cls.powerfeeds = (powerfeed_1, powerfeed_2)
@@ -2872,7 +3184,9 @@ class PowerFeedTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         powerfeed = self.powerfeeds[0]
 
         Cable.objects.create(
-            termination_a=powerport, termination_b=powerfeed, status=Status.objects.get(name="Connected")
+            termination_a=powerport,
+            termination_b=powerfeed,
+            status=Status.objects.get(name="Connected"),
         )
 
         url = reverse("dcim:powerfeed", kwargs={"pk": powerfeed.pk})
@@ -2895,7 +3209,11 @@ class PathTraceViewTestCase(ModelViewTestCase):
         location_type = LocationType.objects.get(name="Campus")
         location = Location.objects.create(location_type=location_type, name="Location 1", status=active)
         device = Device.objects.create(
-            device_type=devicetype, role=devicerole, name="Device 1", location=location, status=active
+            device_type=devicetype,
+            role=devicerole,
+            name="Device 1",
+            location=location,
+            status=active,
         )
         obj = RearPort.objects.create(device=device, name="Rear Port 1", type=PortTypeChoices.TYPE_8P8C)
         peer_obj = Interface.objects.create(device=device, name="eth0", status=active)
@@ -3026,7 +3344,10 @@ class InterfaceRedundancyGroupTestCase(ViewTestCases.PrimaryObjectViewTestCase):
 
         # Assign unconstrained permission
         self.add_permissions("dcim.add_interfaceredundancygroupassociation")
-        return_url = reverse("dcim:interfaceredundancygroup", kwargs={"pk": self.interface_redundancy_groups[0].pk})
+        return_url = reverse(
+            "dcim:interfaceredundancygroup",
+            kwargs={"pk": self.interface_redundancy_groups[0].pk},
+        )
         url = reverse("dcim:interfaceredundancygroupassociation_add")
         url = url + f"?interface_redundancy_group={self.interface_redundancy_groups[0].pk}&return_url={return_url}"
         self.assertHttpStatus(self.client.get(url), 200)