nautobot 2.1.0b1__py3-none-any.whl → 2.1.1__py3-none-any.whl

nautobot/dcim/tests/test_filters.py

@@ -34,8 +34,8 @@ from nautobot.dcim.filters import (
     FrontPortFilterSet,
     FrontPortTemplateFilterSet,
     InterfaceFilterSet,
-    InterfaceRedundancyGroupFilterSet,
     InterfaceRedundancyGroupAssociationFilterSet,
+    InterfaceRedundancyGroupFilterSet,
     InterfaceTemplateFilterSet,
     InventoryItemFilterSet,
     LocationFilterSet,
@@ -43,11 +43,11 @@ from nautobot.dcim.filters import (
     ManufacturerFilterSet,
     PlatformFilterSet,
     PowerFeedFilterSet,
+    PowerOutletFilterSet,
+    PowerOutletTemplateFilterSet,
     PowerPanelFilterSet,
     PowerPortFilterSet,
     PowerPortTemplateFilterSet,
-    PowerOutletFilterSet,
-    PowerOutletTemplateFilterSet,
     RackFilterSet,
     RackGroupFilterSet,
     RackReservationFilterSet,
@@ -55,7 +55,6 @@ from nautobot.dcim.filters import (
     RearPortTemplateFilterSet,
     VirtualChassisFilterSet,
 )
-
 from nautobot.dcim.models import (
     Cable,
     ConsolePort,
@@ -79,11 +78,11 @@ from nautobot.dcim.models import (
     Manufacturer,
     Platform,
     PowerFeed,
+    PowerOutlet,
+    PowerOutletTemplate,
     PowerPanel,
     PowerPort,
     PowerPortTemplate,
-    PowerOutlet,
-    PowerOutletTemplate,
     Rack,
     RackGroup,
     RackReservation,
@@ -92,16 +91,16 @@ from nautobot.dcim.models import (
     VirtualChassis,
 )
 from nautobot.extras.models import Role, SecretsGroup, Status, Tag
-from nautobot.ipam.models import IPAddress, Prefix, Service, VLAN, VLANGroup, Namespace
+from nautobot.ipam.models import IPAddress, Namespace, Prefix, Service, VLAN, VLANGroup
 from nautobot.tenancy.models import Tenant
 from nautobot.virtualization.models import Cluster, ClusterType, VirtualMachine
 
-
 # Use the proper swappable User model
 User = get_user_model()
 
 
 def common_test_data(cls):
+    Device.objects.all().delete()
     tenants = Tenant.objects.filter(tenant_group__isnull=False)
     cls.tenants = tenants
 
@@ -1089,7 +1088,8 @@ class ConsoleServerPortTemplateTestCase(Mixins.ComponentTemplateMixin):
 class PowerPortTemplateTestCase(Mixins.ComponentTemplateMixin):
     queryset = PowerPortTemplate.objects.all()
     filterset = PowerPortTemplateFilterSet
-    generic_filter_tests = Mixins.ComponentTemplateMixin.generic_filter_tests + [
+    generic_filter_tests = [
+        *Mixins.ComponentTemplateMixin.generic_filter_tests,
         ("allocated_draw",),
         ("maximum_draw",),
         ("power_outlet_templates", "power_outlet_templates__id"),
@@ -1114,7 +1114,8 @@ class PowerPortTemplateTestCase(Mixins.ComponentTemplateMixin):
 class PowerOutletTemplateTestCase(Mixins.ComponentTemplateMixin):
     queryset = PowerOutletTemplate.objects.all()
     filterset = PowerOutletTemplateFilterSet
-    generic_filter_tests = Mixins.ComponentTemplateMixin.generic_filter_tests + [
+    generic_filter_tests = [
+        *Mixins.ComponentTemplateMixin.generic_filter_tests,
         ("power_port_template", "power_port_template__id"),
         ("power_port_template", "power_port_template__name"),
     ]
@@ -1172,7 +1173,8 @@ class InterfaceTemplateTestCase(Mixins.ComponentTemplateMixin):
 class FrontPortTemplateTestCase(Mixins.ComponentTemplateMixin):
     queryset = FrontPortTemplate.objects.all()
     filterset = FrontPortTemplateFilterSet
-    generic_filter_tests = Mixins.ComponentTemplateMixin.generic_filter_tests + [
+    generic_filter_tests = [
+        *Mixins.ComponentTemplateMixin.generic_filter_tests,
         ("rear_port_position",),
         ("rear_port_template", "rear_port_template__id"),
     ]
@@ -1189,7 +1191,8 @@ class FrontPortTemplateTestCase(Mixins.ComponentTemplateMixin):
 class RearPortTemplateTestCase(Mixins.ComponentTemplateMixin):
     queryset = RearPortTemplate.objects.all()
     filterset = RearPortTemplateFilterSet
-    generic_filter_tests = Mixins.ComponentTemplateMixin.generic_filter_tests + [
+    generic_filter_tests = [
+        *Mixins.ComponentTemplateMixin.generic_filter_tests,
         ("front_port_templates", "front_port_templates__id"),
     ]
 
@@ -1261,7 +1264,7 @@ class PlatformTestCase(FilterTestCases.NameOnlyFilterTestCase):
         )
 
     def test_devices(self):
-        devices = [Device.objects.first(), Device.objects.last()]
+        devices = Device.objects.filter(platform__isnull=False)[:2]
         params = {"devices": [devices[0].pk, devices[1].pk]}
         self.assertEqual(self.filterset(params, self.queryset).qs.count(), len(devices))
 

nautobot/dcim/tests/test_forms.py

@@ -1,8 +1,7 @@
 from django.test import TestCase
 
-from nautobot.dcim.forms import DeviceForm, DeviceFilterForm, InterfaceCreateForm, RackForm
 from nautobot.dcim.choices import DeviceFaceChoices, InterfaceTypeChoices, RackWidthChoices
-
+from nautobot.dcim.forms import DeviceFilterForm, DeviceForm, InterfaceCreateForm, RackForm
 from nautobot.dcim.models import (
     Device,
     DeviceType,
@@ -13,8 +12,8 @@ from nautobot.dcim.models import (
     Platform,
     Rack,
 )
-from nautobot.tenancy.models import Tenant
 from nautobot.extras.models import Role, SecretsGroup, Status
+from nautobot.tenancy.models import Tenant
 from nautobot.virtualization.models import Cluster, ClusterGroup, ClusterType
 
 

nautobot/dcim/tests/test_graphql.py

@@ -10,6 +10,7 @@ from nautobot.extras.models import DynamicGroup, Role, Status
 
 class GraphQLTestCase(TestCase):
     def setUp(self):
+        Device.objects.all().delete()
         self.user = create_test_user("graphql_testuser")
         self.location = Location.objects.filter(location_type=LocationType.objects.get(name="Campus")).first()
         self.device_role = Role.objects.get_for_model(Device).first()

nautobot/dcim/tests/test_migrations.py

@@ -1,11 +1,11 @@
-import uuid
 from unittest import skip, skipIf
+import uuid
 
 from django.db import connection
 
+from nautobot.circuits.choices import CircuitTerminationSideChoices
 from nautobot.core.models.fields import TagsField
 from nautobot.core.testing.migrations import NautobotDataMigrationTest
-from nautobot.circuits.choices import CircuitTerminationSideChoices
 from nautobot.extras.choices import CustomFieldTypeChoices, ObjectChangeActionChoices, RelationshipTypeChoices
 
 

nautobot/dcim/tests/test_models.py

@@ -37,11 +37,11 @@ from nautobot.dcim.models import (
     LocationType,
     Manufacturer,
     Platform,
-    PowerPort,
-    PowerPortTemplate,
     PowerOutlet,
     PowerOutletTemplate,
     PowerPanel,
+    PowerPort,
+    PowerPortTemplate,
     Rack,
     RackGroup,
     RearPort,

nautobot/dcim/tests/test_views.py

@@ -1,19 +1,18 @@
 from decimal import Decimal
 import unittest
 
-import pytz
-import yaml
-
 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
 from django.urls import reverse
 from netaddr import EUI
+import pytz
+import yaml
 
 from nautobot.circuits.choices import CircuitTerminationSideChoices
 from nautobot.circuits.models import Circuit, CircuitTermination, CircuitType, Provider
-from nautobot.core.testing import ViewTestCases, extract_page_body, ModelViewTestCase, post_data
+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,
@@ -52,19 +51,20 @@ from nautobot.dcim.models import (
     FrontPort,
     FrontPortTemplate,
     Interface,
-    InterfaceTemplate,
     InterfaceRedundancyGroup,
-    Manufacturer,
+    InterfaceRedundancyGroupAssociation,
+    InterfaceTemplate,
     InventoryItem,
     Location,
     LocationType,
+    Manufacturer,
     Platform,
     PowerFeed,
-    PowerPort,
-    PowerPortTemplate,
     PowerOutlet,
     PowerOutletTemplate,
     PowerPanel,
+    PowerPort,
+    PowerPortTemplate,
     Rack,
     RackGroup,
     RackReservation,
@@ -85,7 +85,7 @@ from nautobot.extras.models import (
     Tag,
 )
 from nautobot.ipam.choices import IPAddressTypeChoices
-from nautobot.ipam.models import IPAddress, Namespace, Prefix, VLAN, VLANGroup
+from nautobot.ipam.models import IPAddress, Namespace, Prefix, VLAN, VLANGroup, VRF
 from nautobot.tenancy.models import Tenant
 from nautobot.users.models import ObjectPermission
 
@@ -570,6 +570,7 @@ class ManufacturerTestCase(ViewTestCases.OrganizationalObjectViewTestCase):
         # 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.
+        Device.objects.all().delete()
         DeviceType.objects.all().delete()
         Platform.objects.all().delete()
 
@@ -602,6 +603,7 @@ class DeviceTypeTestCase(
 
     @classmethod
     def setUpTestData(cls):
+        Device.objects.all().delete()
         manufacturers = Manufacturer.objects.all()[:2]
 
         DeviceType.objects.create(model="Test Device Type 1", manufacturer=manufacturers[0])
@@ -1180,6 +1182,7 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
 
     @classmethod
     def setUpTestData(cls):
+        Device.objects.all().delete()
         locations = Location.objects.filter(location_type=LocationType.objects.get(name="Campus"))[:2]
 
         rack_group = RackGroup.objects.create(location=locations[0], name="Rack Group 1")
@@ -1387,7 +1390,7 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
 
     @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
     def test_device_interfaces(self):
-        device = Device.objects.first()
+        device = Device.objects.filter(interfaces__isnull=False).first()
         self.add_permissions("ipam.add_ipaddress", "dcim.change_interface")
 
         url = reverse("dcim:device_interfaces", kwargs={"pk": device.pk})
@@ -1539,8 +1542,8 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         self.add_permissions("dcim.change_device")
 
         # Create an interface and assign an IP to it.
-        device = Device.objects.first()
-        interface = Interface.objects.first()
+        device = Device.objects.filter(interfaces__isnull=False).first()
+        interface = device.interfaces.first()
         namespace = Namespace.objects.first()
         Prefix.objects.create(prefix="1.2.3.0/24", namespace=namespace, status=self.prefix_status)
         ip_address = IPAddress.objects.create(address="1.2.3.4/32", namespace=namespace, status=self.ipaddr_status)
@@ -1821,6 +1824,9 @@ class InterfaceTestCase(ViewTestCases.DeviceComponentViewTestCase):
     @classmethod
     def setUpTestData(cls):
         device = create_test_device("Device 1")
+        vrfs = list(VRF.objects.all()[:3])
+        for vrf in vrfs:
+            vrf.add_device(device)
 
         statuses = Status.objects.get_for_model(Interface)
         status_active = statuses[0]
@@ -1892,6 +1898,7 @@ class InterfaceTestCase(ViewTestCases.DeviceComponentViewTestCase):
             "tagged_vlans": [v.pk for v in vlans[1:4]],
             "tags": [t.pk for t in Tag.objects.get_for_model(Interface)],
             "status": status_active.pk,
+            "vrf": vrfs[0].pk,
         }
 
         cls.bulk_add_data = {
@@ -1906,6 +1913,7 @@ class InterfaceTestCase(ViewTestCases.DeviceComponentViewTestCase):
             "description": "An Interface",
             "mode": InterfaceModeChoices.MODE_TAGGED,
             "tags": [],
+            "vrf": vrfs[1].pk,
         }
 
         cls.bulk_edit_data = {
@@ -1920,6 +1928,7 @@ class InterfaceTestCase(ViewTestCases.DeviceComponentViewTestCase):
             "untagged_vlan": vlans[0].pk,
             "tagged_vlans": [v.pk for v in vlans[1:4]],
             "status": status_active.pk,
+            "vrf": vrfs[2].pk,
         }
 
         cls.csv_data = (
@@ -2904,7 +2913,7 @@ class InterfaceRedundancyGroupTestCase(ViewTestCases.PrimaryObjectViewTestCase):
             SecretsGroup.objects.create(name="Secrets Group 3"),
         )
 
-        interface_redundancy_groups = (
+        cls.interface_redundancy_groups = (
             InterfaceRedundancyGroup(
                 name="Interface Redundancy Group 1",
                 protocol="hsrp",
@@ -2938,14 +2947,42 @@ class InterfaceRedundancyGroupTestCase(ViewTestCases.PrimaryObjectViewTestCase):
             ),
         )
 
-        for group in interface_redundancy_groups:
+        for group in cls.interface_redundancy_groups:
             group.validated_save()
 
+        locations = Location.objects.filter(location_type=LocationType.objects.get(name="Campus"))[:2]
+
+        devicetypes = DeviceType.objects.all()[:2]
+
+        deviceroles = Role.objects.get_for_model(Device)[:2]
+
+        device_statuses = Status.objects.get_for_model(Device)
+        status_active = device_statuses[0]
+        device = Device.objects.create(
+            name="Device 1",
+            location=locations[0],
+            device_type=devicetypes[0],
+            role=deviceroles[0],
+            status=status_active,
+        )
+        intf_status = Status.objects.get_for_model(Interface).first()
+
+        cls.interfaces = (
+            Interface.objects.create(device=device, name="Interface 1", status=intf_status),
+            Interface.objects.create(device=device, name="Interface 2", status=intf_status),
+            Interface.objects.create(device=device, name="Interface 3", status=intf_status),
+        )
+
         cls.form_data = {
             "name": "IRG χ",
             "protocol": InterfaceRedundancyGroupProtocolChoices.GLBP,
             "status": statuses[3].pk,
         }
+        cls.interface_add_form_data = {
+            "interface_redundancy_group": cls.interface_redundancy_groups[0].pk,
+            "interface": cls.interfaces[0].pk,
+            "priority": 100,
+        }
 
         cls.csv_data = (
             "name,protocol,status",
@@ -2961,3 +2998,29 @@ class InterfaceRedundancyGroupTestCase(ViewTestCases.PrimaryObjectViewTestCase):
             "virtual_ip": cls.ips[0].pk,
             "secrets_group": cls.secrets_groups[1].pk,
         }
+
+    @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
+    def test_adding_interfaces_to_group(self):
+        initial_count = InterfaceRedundancyGroupAssociation.objects.all().count()
+
+        # Assign unconstrained permission
+        self.add_permissions("dcim.add_interfaceredundancygroupassociation")
+        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)
+
+        # Try POST with model-level permission
+        request = {
+            "path": url,
+            "data": post_data(self.interface_add_form_data),
+        }
+        self.assertHttpStatus(self.client.post(**request), 302)
+        self.assertEqual(initial_count + 1, InterfaceRedundancyGroupAssociation.objects.all().count())
+        self.interface_add_form_data["interface"] = self.interfaces[1]
+        request = {
+            "path": url,
+            "data": post_data(self.interface_add_form_data),
+        }
+        self.assertHttpStatus(self.client.post(**request), 302)
+        self.assertEqual(initial_count + 2, InterfaceRedundancyGroupAssociation.objects.all().count())

nautobot/dcim/urls.py

@@ -1,8 +1,9 @@
 from django.urls import path
 
 from nautobot.core.views.routers import NautobotUIViewSetRouter
-from nautobot.extras.views import ObjectChangeLogView, ObjectDynamicGroupsView, ObjectNotesView, ImageAttachmentEditView
+from nautobot.extras.views import ImageAttachmentEditView, ObjectChangeLogView, ObjectDynamicGroupsView, ObjectNotesView
 from nautobot.ipam.views import ServiceEditView
+
 from . import views
 from .models import (
     Cable,
@@ -19,9 +20,9 @@ from .models import (
     Manufacturer,
     Platform,
     PowerFeed,
+    PowerOutlet,
     PowerPanel,
     PowerPort,
-    PowerOutlet,
     Rack,
     RackGroup,
     RackReservation,

nautobot/dcim/utils.py

@@ -14,8 +14,8 @@ from netutils.lib_mapper import (
     SCRAPLI_LIB_MAPPER_REVERSE,
 )
 
-from nautobot.core.utils.config import get_settings_or_config
 from nautobot.core.utils.color import hex_to_rgb, lighten_color, rgb_to_hex
+from nautobot.core.utils.config import get_settings_or_config
 from nautobot.dcim.choices import InterfaceModeChoices
 from nautobot.dcim.constants import NETUTILS_NETWORK_DRIVER_MAPPING_NAMES
 

nautobot/dcim/views.py

@@ -1,5 +1,5 @@
-import uuid
 from collections import OrderedDict
+import uuid
 
 from django.contrib import messages
 from django.contrib.contenttypes.models import ContentType
@@ -7,9 +7,9 @@ from django.core.paginator import EmptyPage, PageNotAnInteger
 from django.db import transaction
 from django.db.models import F, Prefetch
 from django.forms import (
+    modelformset_factory,
     ModelMultipleChoiceField,
     MultipleHiddenInput,
-    modelformset_factory,
 )
 from django.shortcuts import get_object_or_404, redirect, render
 from django.utils.functional import cached_property
@@ -30,11 +30,12 @@ from nautobot.core.views.mixins import (
 )
 from nautobot.core.views.paginator import EnhancedPaginator, get_paginate_count
 from nautobot.core.views.viewsets import NautobotUIViewSet
-from nautobot.dcim.utils import get_network_driver_mapping_tool_names, get_all_network_driver_mappings
+from nautobot.dcim.utils import get_all_network_driver_mappings, get_network_driver_mapping_tool_names
 from nautobot.extras.views import ObjectChangeLogView, ObjectConfigContextView, ObjectDynamicGroupsView
 from nautobot.ipam.models import IPAddress, Prefix, Service, VLAN
 from nautobot.ipam.tables import InterfaceIPAddressTable, InterfaceVLANTable, VRFDeviceAssignmentTable
 from nautobot.virtualization.models import VirtualMachine
+
 from . import filters, forms, tables
 from .api import serializers
 from .choices import DeviceFaceChoices

nautobot/extras/admin.py

@@ -1,6 +1,7 @@
 from db_file_storage.form_widgets import DBAdminClearableFileInput
 from django import forms
 from django.contrib import admin
+
 from nautobot.core.admin import NautobotModelAdmin
 
 from .models import FileProxy, JobResult
@@ -22,7 +23,7 @@ def order_content_types(field):
 class FileProxyForm(forms.ModelForm):
     class Meta:
         model = FileProxy
-        exclude = []
+        fields = ["name", "file"]
         widgets = {
             "file": DBAdminClearableFileInput,
         }

nautobot/extras/api/customfields.py

@@ -1,12 +1,10 @@
 from django.contrib.contenttypes.models import ContentType
-
 from drf_spectacular.types import OpenApiTypes
 from drf_spectacular.utils import extend_schema_field
 from rest_framework.fields import Field
 
 from nautobot.extras.models import CustomField
 
-
 #
 # Custom fields
 #

nautobot/extras/api/mixins.py

@@ -1,6 +1,5 @@
 import logging
 
-
 from nautobot.core.api import (
     BaseModelSerializer,
 )

nautobot/extras/api/serializers.py

@@ -30,6 +30,9 @@ from nautobot.dcim.api.serializers import (
     RackSerializer,
 )
 from nautobot.extras import choices, models
+from nautobot.extras.api.mixins import (
+    TaggedModelSerializerMixin,
+)
 from nautobot.extras.choices import (
     CustomFieldFilterLogicChoices,
     CustomFieldTypeChoices,
@@ -37,9 +40,6 @@ from nautobot.extras.choices import (
     JobResultStatusChoices,
     ObjectChangeActionChoices,
 )
-from nautobot.extras.api.mixins import (
-    TaggedModelSerializerMixin,
-)
 from nautobot.extras.datasources import get_datasource_content_choices
 from nautobot.extras.models import (
     ComputedField,
@@ -342,7 +342,7 @@ class GitRepositorySerializer(NautobotModelSerializer):
 
 
 class GraphQLQuerySerializer(ValidatedModelSerializer, NotesSerializerMixin):
-    variables = serializers.DictField(required=False, allow_null=True, default={})
+    variables = serializers.DictField(read_only=True)
 
     class Meta:
         model = GraphQLQuery

nautobot/extras/api/urls.py

@@ -1,6 +1,6 @@
 from nautobot.core.api.routers import OrderedDefaultRouter
-from . import views
 
+from . import views
 
 router = OrderedDefaultRouter()
 router.APIRootView = views.ExtrasRootView

nautobot/extras/api/views.py

@@ -7,7 +7,7 @@ from django.http import FileResponse, Http404
 from django.shortcuts import get_object_or_404
 from django.utils import timezone
 from drf_spectacular.types import OpenApiTypes
-from drf_spectacular.utils import OpenApiParameter, extend_schema, extend_schema_view
+from drf_spectacular.utils import extend_schema, extend_schema_view, OpenApiParameter
 from graphene_django.views import GraphQLView
 from graphql import GraphQLError
 from rest_framework import mixins, status, viewsets

nautobot/extras/apps.py

@@ -1,14 +1,12 @@
 import logging
 
 from django.db.utils import ProgrammingError
-
 import graphene
 from health_check.plugins import plugin_dir
 
 from nautobot.core.apps import NautobotConfig
 from nautobot.core.signals import nautobot_database_ready
 
-
 logger = logging.getLogger(__name__)
 
 
@@ -17,12 +15,13 @@ class ExtrasConfig(NautobotConfig):
 
     def ready(self):
         super().ready()
-        import nautobot.extras.signals  # noqa
+        import nautobot.extras.signals  # noqa: F401  # unused-import -- but this import installs the signals
         from nautobot.extras.signals import refresh_job_models
 
         nautobot_database_ready.connect(refresh_job_models, sender=self)
 
         from graphene_django.converter import convert_django_field
+
         from nautobot.core.models.fields import TagsField
         from nautobot.extras.graphql.types import TagType
 
@@ -51,8 +50,8 @@ class ExtrasConfig(NautobotConfig):
         plugin_dir.register(RedisBackend)
 
         # Register built-in SecretsProvider classes
-        from nautobot.extras.secrets.providers import EnvironmentVariableSecretsProvider, TextFileSecretsProvider
         from nautobot.extras.secrets import register_secrets_provider
+        from nautobot.extras.secrets.providers import EnvironmentVariableSecretsProvider, TextFileSecretsProvider
 
         register_secrets_provider(EnvironmentVariableSecretsProvider)
         register_secrets_provider(TextFileSecretsProvider)

nautobot/extras/choices.py

@@ -3,7 +3,6 @@ from celery import states
 from nautobot.core.choices import ChoiceSet
 from nautobot.core.utils.deprecation import class_deprecated_in_favor_of
 
-
 #
 # Banners (currently plugin-specific)
 #
@@ -130,20 +129,6 @@ class DynamicGroupOperatorChoices(ChoiceSet):
 #
 
 
-class JobSourceChoices(ChoiceSet):
-    SOURCE_LOCAL = "local"
-    SOURCE_GIT = "git"
-    SOURCE_PLUGIN = "plugins"
-    SOURCE_SYSTEM = "system"
-
-    CHOICES = (
-        (SOURCE_LOCAL, "Installed in $JOBS_ROOT"),
-        (SOURCE_GIT, "Provided by a Git repository"),
-        (SOURCE_PLUGIN, "Part of a plugin"),
-        (SOURCE_SYSTEM, "Provided by Nautobot"),
-    )
-
-
 class JobExecutionType(ChoiceSet):
     TYPE_IMMEDIATELY = "immediately"
     TYPE_FUTURE = "future"
@@ -389,9 +374,9 @@ class SecretsGroupAccessTypeChoices(ChoiceSet):
 
 class SecretsGroupSecretTypeChoices(ChoiceSet):
     TYPE_KEY = "key"
-    TYPE_PASSWORD = "password"
-    TYPE_SECRET = "secret"
-    TYPE_TOKEN = "token"
+    TYPE_PASSWORD = "password"  # noqa: S105  # hardcoded-password-string -- false positive
+    TYPE_SECRET = "secret"  # noqa: S105  # hardcoded-password-string -- false positive
+    TYPE_TOKEN = "token"  # noqa: S105  # hardcoded-password-string -- false positive
     TYPE_USERNAME = "username"
 
     CHOICES = (

nautobot/extras/context_managers.py

@@ -1,5 +1,5 @@
-import uuid
 from contextlib import contextmanager
+import uuid
 
 from django.contrib.auth import get_user_model
 from django.contrib.auth.models import AnonymousUser

nautobot/extras/datasources/__init__.py

@@ -2,11 +2,12 @@ from .git import (
     enqueue_git_repository_diff_origin_and_local,
     enqueue_pull_git_repository_and_refresh_data,
     ensure_git_repository,
+    get_repo_access_url,
     git_repository_dry_run,
 )
 from .registry import (
-    get_datasource_contents,
     get_datasource_content_choices,
+    get_datasource_contents,
     refresh_datasource_content,
 )
 
@@ -16,6 +17,7 @@ __all__ = (
     "ensure_git_repository",
     "get_datasource_content_choices",
     "get_datasource_contents",
+    "get_repo_access_url",
     "git_repository_dry_run",
     "refresh_datasource_content",
 )