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

nautobot/core/templatetags/bootstrapped_goodies_tags.py

@@ -16,7 +16,7 @@ def render_with_template_if_exist(context, template, fallback):  # pylint: disab
     text = fallback
     try:
         text = render_to_string(template, context)
-    except Exception:
+    except Exception:  # noqa: S110  # try-except-pass --  generally an antipattern, but it's intentional here
         pass
     return text
 
@@ -36,7 +36,7 @@ def language_selector(context):
         context["i18n_is_set"] = True
         try:
             output = render_to_string(template_, context)
-        except Exception:
+        except Exception:  # noqa: S110  # try-except-pass --  generally an antipattern, but it's intentional here
             pass
     return output
 
@@ -96,7 +96,10 @@ def render_app_label(context, app, fallback=""):
 
 @register.simple_tag(takes_context=True)
 def render_app_description(
-    context, app, fallback="", template="/admin_app_description.html"  # pylint: disable=redefined-outer-name
+    context,
+    app,
+    fallback="",
+    template="/admin_app_description.html",  # pylint: disable=redefined-outer-name
 ):
     """Render the application description using the default template name. If it cannot find a
     template matching the given path, fallback to the fallback argument.

nautobot/core/templatetags/buttons.py

@@ -1,8 +1,8 @@
 from django import template
 from django.urls import NoReverseMatch, reverse
 
-from nautobot.core.views import utils as views_utils
 from nautobot.core.utils import lookup
+from nautobot.core.views import utils as views_utils
 from nautobot.extras import models
 
 register = template.Library()

nautobot/core/templatetags/helpers.py

@@ -6,11 +6,11 @@ import re
 from django import template
 from django.conf import settings
 from django.contrib.staticfiles.finders import find
-from django.templatetags.static import StaticNode, static
+from django.templatetags.static import static, StaticNode
 from django.urls import NoReverseMatch, reverse
 from django.utils.html import format_html, strip_tags
-from django.utils.text import slugify as django_slugify
 from django.utils.safestring import mark_safe
+from django.utils.text import slugify as django_slugify
 from django_jinja import library
 from markdown import markdown
 import yaml
@@ -18,9 +18,9 @@ import yaml
 from nautobot.apps.config import get_app_settings_or_config
 from nautobot.core import forms
 from nautobot.core.utils import color, config, data, lookup
-from nautobot.core.utils.navigation import is_route_new_ui_ready
 from nautobot.core.utils.requests import add_nautobot_version_query_param_to_url
 
+# S308 is suspicious-mark-safe-usage, but these are all using static strings that we know to be safe
 HTML_TRUE = mark_safe('<span class="text-success"><i class="mdi mdi-check-bold" title="Yes"></i></span>')  # noqa: S308
 HTML_FALSE = mark_safe('<span class="text-danger"><i class="mdi mdi-close-thick" title="No"></i></span>')  # noqa: S308
 HTML_NONE = mark_safe('<span class="text-muted">&mdash;</span>')  # noqa: S308
@@ -124,7 +124,7 @@ def add_html_id(element_str, id_str):
     match = re.match(r"^(.*?<\w+) ?(.*)$", element_str, flags=re.DOTALL)
     if not match:
         return element_str
-    return mark_safe(match.group(1) + format_html(' id="{}" ', id_str) + match.group(2))  # noqa: S308
+    return mark_safe(match.group(1) + format_html(' id="{}" ', id_str) + match.group(2))  # noqa: S308  # suspicious-mark-safe-usage
 
 
 @library.filter()
@@ -181,7 +181,7 @@ def render_markdown(value):
     # Render Markdown
     html = markdown(value, extensions=["fenced_code", "tables"])
 
-    return mark_safe(html)  # noqa: S308
+    return mark_safe(html)  # noqa: S308  # suspicious-mark-safe-usage, OK here since we sanitized the string earlier
 
 
 @library.filter()
@@ -779,14 +779,3 @@ def queryset_to_pks(obj):
     result = list(obj.values_list("pk", flat=True)) if obj else []
     result = [str(entry) for entry in result]
     return ",".join(result)
-
-
-#
-# Navigation
-#
-
-
-@register.filter()
-def is_new_ui_ready(url_path):
-    """Return True if url_path is NewUI Ready else False"""
-    return is_route_new_ui_ready(url_path)

nautobot/core/testing/__init__.py

@@ -1,8 +1,7 @@
 import collections
 
 from django.contrib.auth import get_user_model
-from django.test import TransactionTestCase as _TransactionTestCase
-from django.test import tag
+from django.test import tag, TransactionTestCase as _TransactionTestCase
 
 from nautobot.core.testing.api import APITestCase, APIViewTestCases
 from nautobot.core.testing.filters import FilterTestCases

nautobot/core/testing/api.py

@@ -10,8 +10,8 @@ from django.test import override_settings, tag
 from django.urls import reverse
 from django.utils.text import slugify
 from rest_framework import status
-from rest_framework.test import APITransactionTestCase as _APITransactionTestCase
 from rest_framework.relations import ManyRelatedField
+from rest_framework.test import APITransactionTestCase as _APITransactionTestCase
 
 from nautobot.core import testing
 from nautobot.core.api.utils import get_serializer_for_model
@@ -21,9 +21,7 @@ from nautobot.core.templatetags.helpers import bettertitle
 from nautobot.core.testing import mixins, views
 from nautobot.core.utils import lookup
 from nautobot.core.utils.data import is_uuid
-from nautobot.extras import choices as extras_choices
-from nautobot.extras import models as extras_models
-from nautobot.extras import registry
+from nautobot.extras import choices as extras_choices, models as extras_models, registry
 from nautobot.users import models as users_models
 
 __all__ = (

nautobot/core/testing/filters.py

@@ -39,7 +39,7 @@ class FilterTestCases:
             values_with_count = queryset.values(field_name).annotate(count=Count(field_name)).order_by("count")
             for value in values_with_count:
                 # randomly break out of loop after 2 values have been selected
-                if len(test_values) > 1 and random.choice([True, False]):
+                if len(test_values) > 1 and random.choice([True, False]):  # noqa: S311  # suspicious-non-cryptographic-random-usage
                     break
                 if value[field_name] and value["count"] < qs_count:
                     qs_count -= value["count"]

nautobot/core/testing/integration.py

@@ -67,7 +67,7 @@ class SeleniumTestCase(StaticLiveServerTestCase, testing.NautobotTestCaseMixin):
     so there is no need to run `collectstatic` prior to running tests.
     """
 
-    host = "0.0.0.0"  # Always listen publicly
+    host = "0.0.0.0"  # noqa: S104  # hardcoded-bind-all-interfaces -- false positive
     selenium_host = SELENIUM_HOST  # Docker: `nautobot`; else `host.docker.internal`
 
     @classmethod
@@ -86,7 +86,7 @@ class SeleniumTestCase(StaticLiveServerTestCase, testing.NautobotTestCaseMixin):
     def setUp(self):
         super().setUpNautobot(populate_status=True)
 
-        self.password = "testpassword"
+        self.password = "testpassword"  # noqa: S105  # hardcoded-password-string
         self.user.set_password(self.password)
         self.user.save()
 

nautobot/core/testing/migrations.py

@@ -1,4 +1,5 @@
 from unittest import skip
+
 from django.apps import apps
 from django.core.management import call_command
 from django.db import connection

nautobot/core/testing/mixins.py

@@ -13,8 +13,7 @@ from rest_framework.test import APIClient, APIRequestFactory
 from nautobot.core import testing
 from nautobot.core.models import fields as core_fields
 from nautobot.core.utils import permissions
-from nautobot.extras import management
-from nautobot.extras import models as extras_models
+from nautobot.extras import management, models as extras_models
 from nautobot.users import models as users_models
 
 # Use the proper swappable User model

nautobot/core/testing/schema.py

@@ -1,10 +1,10 @@
 from io import StringIO
-import yaml
 
 from django.conf import settings
 from django.core.management import call_command
 from django.test import tag
 from rest_framework.settings import api_settings
+import yaml
 
 from nautobot.core.testing import views
 

nautobot/core/testing/utils.py

@@ -99,7 +99,7 @@ def disable_warnings(logger_name):
 
 def get_deletable_objects(model, queryset):
     """
-    Returns a queryset of objects in the supplied queryset that have no protected relationships that would prevent deletion.
+    Returns a subset of objects in the given queryset that have no protected relationships that would prevent deletion.
     """
     q = Q()
     for field in model._meta.get_fields(include_parents=True):
@@ -113,7 +113,7 @@ def get_deletable_objects(model, queryset):
 
 def generate_random_device_asset_tag_of_specified_size(size):
     """
-    This function is for testing purposes only and it will return a random string of size 100 consists of letters and numbers.
+    For testing purposes only; it returns a random string of size 100 consisting of letters and numbers.
     """
-    asset_tag = "".join(random.choices(string.ascii_letters + string.digits, k=size))
+    asset_tag = "".join(random.choices(string.ascii_letters + string.digits, k=size))  # noqa: S311  # suspicious-non-cryptographic-random-usage
     return asset_tag

nautobot/core/testing/views.py

@@ -9,11 +9,10 @@ from django.contrib.contenttypes.models import ContentType
 from django.core.exceptions import ObjectDoesNotExist, ValidationError
 from django.core.files.uploadedfile import SimpleUploadedFile
 from django.core.validators import URLValidator
-from django.test import TestCase as _TestCase
-from django.test import override_settings, tag
+from django.test import override_settings, tag, TestCase as _TestCase
 from django.urls import NoReverseMatch, reverse
-from django.utils.http import urlencode
 from django.utils.html import escape
+from django.utils.http import urlencode
 from django.utils.text import slugify
 from tree_queries.models import TreeNode
 
@@ -21,9 +20,7 @@ from nautobot.core import testing
 from nautobot.core.templatetags import helpers
 from nautobot.core.testing import mixins
 from nautobot.core.utils import lookup
-from nautobot.extras import choices as extras_choices
-from nautobot.extras import models as extras_models
-from nautobot.extras import querysets as extras_querysets
+from nautobot.extras import choices as extras_choices, models as extras_models, querysets as extras_querysets
 from nautobot.users import models as users_models
 
 __all__ = (
@@ -1155,7 +1152,7 @@ class ViewTestCases:
         def test_bulk_edit_objects_with_constrained_permission(self):
             # Select some objects that are *not* already set to match the first value in self.bulk_edit_data or null.
             # We have to exclude null cases because Django filter()/exclude() doesn't like `__in=[None]` as a case.
-            attr_name = list(self.bulk_edit_data.keys())[0]
+            attr_name = next(iter(self.bulk_edit_data.keys()))
             objects = (
                 self._get_queryset()
                 .exclude(**{attr_name: self.bulk_edit_data[attr_name]})

nautobot/core/tests/integration/test_plugin_home.py

@@ -1,6 +1,7 @@
-from django.conf import settings
 from unittest import skipIf
 
+from django.conf import settings
+
 from nautobot.circuits.models import Circuit, Provider
 from nautobot.core.testing.integration import SeleniumTestCase
 from nautobot.dcim.models import Location, PowerFeed, PowerPanel

nautobot/core/tests/integration/test_plugin_navbar.py

@@ -1,8 +1,8 @@
 from unittest import skipIf
 
 from django.conf import settings
-from nautobot.core.choices import ButtonActionColorChoices, ButtonActionIconChoices
 
+from nautobot.core.choices import ButtonActionColorChoices, ButtonActionIconChoices
 from nautobot.core.testing.integration import SeleniumTestCase
 
 

nautobot/core/tests/nautobot_config.py

@@ -5,7 +5,7 @@
 
 import os
 
-from nautobot.core.settings import *  # noqa: F401,F403
+from nautobot.core.settings import *  # noqa: F403  # undefined-local-with-import-star
 from nautobot.core.settings_funcs import parse_redis_connection
 
 ALLOWED_HOSTS = ["nautobot.example.com"]
@@ -22,12 +22,12 @@ PLUGINS = [
 ]
 
 # Hard-code the SECRET_KEY for simplicity
-SECRET_KEY = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789"
+SECRET_KEY = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789"  # noqa: S105  # hardcoded-password-string
 
 # Redis variables
 
 # Use *different* redis_databases than the ones (0 and 1) used during non-automated-testing operations.
-CACHES["default"]["LOCATION"] = parse_redis_connection(redis_database=2)  # noqa: F405
+CACHES["default"]["LOCATION"] = parse_redis_connection(redis_database=2)  # noqa: F405  # undefined-local-with-import-star-usage
 
 # Testing storages within cli.py
 STORAGE_CONFIG = {

nautobot/core/tests/runner.py

@@ -1,8 +1,7 @@
-import yaml
-
-from django.core.management import call_command
 from django.conf import settings
+from django.core.management import call_command
 from django.test.runner import DiscoverRunner
+import yaml
 
 from nautobot.core.celery import app, setup_nautobot_job_logging
 
@@ -67,7 +66,7 @@ class NautobotTestRunner(DiscoverRunner):
             for connection in result:
                 db_name = connection[0].alias
                 print(f'Pre-populating test database "{db_name}" with factory data...')
-                db_command = command + ["--database", db_name]
+                db_command = [*command, "--database", db_name]
                 call_command(*db_command)
 
         return result

nautobot/core/tests/test_api.py

@@ -3,14 +3,13 @@ from io import BytesIO, StringIO
 import json
 from unittest import skip
 
-from django.contrib.contenttypes.models import ContentType
+from constance import config
+from constance.test import override_config
 from django.conf import settings
 from django.contrib.auth import get_user_model
-from django.test import RequestFactory, override_settings, TestCase
+from django.contrib.contenttypes.models import ContentType
+from django.test import override_settings, RequestFactory, TestCase
 from django.urls import reverse
-
-from constance import config
-from constance.test import override_config
 from rest_framework import status
 from rest_framework.exceptions import ParseError
 from rest_framework.settings import api_settings
@@ -23,12 +22,10 @@ from nautobot.core.api.versioning import NautobotAPIVersioning
 from nautobot.core.constants import COMPOSITE_KEY_SEPARATOR
 from nautobot.dcim import models as dcim_models
 from nautobot.dcim.api import serializers as dcim_serializers
-from nautobot.extras import choices
-from nautobot.extras import models as extras_models
+from nautobot.extras import choices, models as extras_models
 from nautobot.ipam import models as ipam_models
 from nautobot.ipam.api import serializers as ipam_serializers
 
-
 User = get_user_model()
 
 
@@ -495,7 +492,7 @@ class NautobotCSVRendererTest(TestCase):
 
         # Make sure a) it's well-constructed parsable CSV and b) it contains what we expect it to, within reason
         reader = csv.DictReader(StringIO(csv_text))
-        read_data = list(reader)[0]
+        read_data = next(iter(reader))
         self.assertIn("id", read_data)
         self.assertEqual(read_data["id"], str(location_type.id))
         self.assertIn("display", read_data)
@@ -925,45 +922,3 @@ class NewUIGetMenuAPIViewTestCase(testing.APITestCase):
 
         self.assertEqual(response.status_code, 200)
         self.assertEqual(response.data, expected_response)
-
-
-class GetSettingsAPIViewTestCase(testing.APITestCase):
-    def test_get_settings(self):
-        """Assert get settings value"""
-        url = reverse("ui-api:settings")
-
-        with self.subTest("Test get allowed settings"):
-            url_with_filters = f"{url}?name=FEEDBACK_BUTTON_ENABLED"
-            response = self.client.get(url_with_filters, **self.header)
-            self.assertEqual(response.status_code, 200)
-            expected_response = {"FEEDBACK_BUTTON_ENABLED": True}
-            self.assertEqual(response.data, expected_response)
-        with self.subTest("Test get not-allowed settings"):
-            url_with_filters = f"{url}?name=NAPALM_PASSWORD"
-            response = self.client.get(url_with_filters, **self.header)
-            self.assertEqual(response.status_code, 400)
-            expected_response = {"error": "Invalid settings names specified: NAPALM_PASSWORD."}
-            self.assertEqual(response.data, expected_response)
-
-
-class NewUIReadyRoutesAPIViewTestCase(testing.APITestCase):
-    def test_new_ui_ready_routes(self):
-        """Assert get settings value"""
-        url = reverse("ui-api:new-ui-ready-routes")
-        response = self.client.get(url, **self.header)
-        self.assertEqual(response.status_code, 200)
-        expected_response = [
-            "^$",
-            "^login\\/$",
-            "^dcim\\/device\\-types\\/$",
-            "^dcim\\/device\\-types\\/([0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12})\\/$",
-            "^dcim\\/devices\\/$",
-            "^dcim\\/devices\\/([0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12})\\/$",
-            "^dcim\\/locations\\/$",
-            "^dcim\\/locations\\/([0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12})\\/$",
-            "^ipam\\/ip\\-addresses\\/$",
-            "^ipam\\/ip\\-addresses\\/([0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12})\\/$",
-            "^ipam\\/prefixes\\/$",
-            "^ipam\\/prefixes\\/([0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12})\\/$",
-        ]
-        self.assertEqual(sorted(response.data), sorted(expected_response))

nautobot/core/tests/test_authentication.py

@@ -12,10 +12,9 @@ from nautobot.core.settings_funcs import sso_auth_enabled
 from nautobot.core.testing import NautobotTestClient, TestCase
 from nautobot.dcim.models import Location, LocationType
 from nautobot.extras.models import Status
-from nautobot.ipam.models import Prefix, Namespace
+from nautobot.ipam.models import Namespace, Prefix
 from nautobot.users.models import ObjectPermission, Token
 
-
 # Use the proper swappable User model
 User = get_user_model()
 
@@ -51,7 +50,7 @@ class ExternalAuthenticationTestCase(TestCase):
         self.assertEqual(settings.REMOTE_AUTH_HEADER, "HTTP_REMOTE_USER")
 
         # Client should not be authenticated
-        self.client.get(reverse("home"), follow=True, **headers)  # noqa
+        self.client.get(reverse("home"), follow=True, **headers)
         self.assertNotIn("_auth_user_id", self.client.session)
 
     @override_settings(AUTHENTICATION_BACKENDS=TEST_AUTHENTICATION_BACKENDS)

nautobot/core/tests/test_checks.py

@@ -1,5 +1,4 @@
-from django.test import TestCase
-from django.test import override_settings
+from django.test import override_settings, TestCase
 
 from nautobot.core import checks
 

nautobot/core/tests/test_config.py

@@ -1,7 +1,7 @@
 """Test cases for nautobot.core.config module."""
 
 from constance.test import override_config
-from django.test import TestCase, override_settings
+from django.test import override_settings, TestCase
 
 from nautobot.apps import config as app_config
 from nautobot.core.utils import config

nautobot/core/tests/test_csv.py

@@ -1,8 +1,8 @@
-from django.test import RequestFactory, TestCase, override_settings
-from django.urls import reverse
-from nautobot.core.constants import CSV_NULL_TYPE, CSV_NO_OBJECT, VARBINARY_IP_FIELD_REPR_OF_CSV_NO_OBJECT
 from django.contrib.contenttypes.models import ContentType
+from django.test import override_settings, RequestFactory, TestCase
+from django.urls import reverse
 
+from nautobot.core.constants import CSV_NO_OBJECT, CSV_NULL_TYPE, VARBINARY_IP_FIELD_REPR_OF_CSV_NO_OBJECT
 from nautobot.dcim.api.serializers import DeviceSerializer
 from nautobot.dcim.models.devices import Device, DeviceType
 from nautobot.dcim.models.locations import Location

nautobot/core/tests/test_factory.py

@@ -2,9 +2,7 @@ import itertools
 
 import factory as factoryboy
 
-from nautobot.core import constants
-from nautobot.core import factory
-from nautobot.core import testing
+from nautobot.core import constants, factory, testing
 
 
 class FactoryTestCase(testing.TestCase):

nautobot/core/tests/test_filters.py

@@ -7,7 +7,6 @@ from django.db import models as django_models
 from django.shortcuts import reverse
 from django.test import TestCase
 from django.test.utils import isolate_apps
-
 import django_filters
 from tree_queries.models import TreeNodeForeignKey
 
@@ -16,14 +15,11 @@ from nautobot.circuits.models import Circuit, CircuitType, Provider
 from nautobot.core import filters, testing
 from nautobot.core.models import fields as core_fields
 from nautobot.core.utils import lookup
-from nautobot.dcim import choices as dcim_choices
-from nautobot.dcim import filters as dcim_filters
-from nautobot.dcim import models as dcim_models
+from nautobot.dcim import choices as dcim_choices, filters as dcim_filters, models as dcim_models
 from nautobot.dcim.models import Device
 from nautobot.extras import models as extras_models
 from nautobot.extras.utils import FeatureQuery
-from nautobot.ipam import factory as ipam_factory
-from nautobot.ipam import models as ipam_models
+from nautobot.ipam import factory as ipam_factory, models as ipam_models
 
 
 class TreeNodeMultipleChoiceFilterTest(TestCase):
@@ -416,7 +412,7 @@ class BaseFilterSetTest(TestCase):
     class TestFilterSet(filters.BaseFilterSet):
         """Filterset for testing various fields."""
 
-        class TestModel(django_models.Model):
+        class TestModel(django_models.Model):  # noqa: DJ008  # django-model-without-dunder-str -- fine since this isn't a "real" model
             """
             Test model used by BaseFilterSetTest for filter validation. Should never appear in a schema migration.
             """

nautobot/core/tests/test_forms.py

@@ -9,13 +9,10 @@ from netaddr import IPNetwork
 
 from nautobot.core import filters, forms, testing
 from nautobot.core.utils import requests
-from nautobot.dcim import filters as dcim_filters, forms as dcim_forms
-from nautobot.dcim import models as dcim_models
+from nautobot.dcim import filters as dcim_filters, forms as dcim_forms, models as dcim_models
 from nautobot.dcim.tests import test_views
-from nautobot.extras import filters as extras_filters
-from nautobot.extras import models as extras_models
-from nautobot.ipam import forms as ipam_forms
-from nautobot.ipam import models as ipam_models
+from nautobot.extras import filters as extras_filters, models as extras_models
+from nautobot.ipam import forms as ipam_forms, models as ipam_models
 
 
 class SearchFormTestCase(TestCase):

nautobot/core/tests/test_graphql.py

@@ -8,18 +8,17 @@ from django.contrib.auth import get_user_model
 from django.contrib.auth.models import Group
 from django.contrib.contenttypes.models import ContentType
 from django.db.models import Q
-from django.test import TestCase, override_settings
+from django.test import override_settings, TestCase
 from django.test.client import RequestFactory
 from django.urls import reverse
-from graphql import GraphQLError
 import graphene.types
-from graphene_django.settings import graphene_settings
 from graphene_django.registry import get_global_registry
+from graphene_django.settings import graphene_settings
+from graphql import get_default_backend, GraphQLError
 from graphql.error.located_error import GraphQLLocatedError
-from graphql import get_default_backend
 from rest_framework import status
 
-from nautobot.circuits.models import Provider, CircuitTermination
+from nautobot.circuits.models import CircuitTermination, Provider
 from nautobot.core.graphql import execute_query, execute_saved_query
 from nautobot.core.graphql.generators import (
     generate_list_search_parameters,
@@ -27,16 +26,16 @@ from nautobot.core.graphql.generators import (
 )
 from nautobot.core.graphql.schema import (
     extend_schema_type,
-    extend_schema_type_custom_field,
-    extend_schema_type_tags,
     extend_schema_type_config_context,
-    extend_schema_type_relationships,
+    extend_schema_type_custom_field,
     extend_schema_type_null_field_choice,
+    extend_schema_type_relationships,
+    extend_schema_type_tags,
 )
 from nautobot.core.graphql.types import OptimizedNautobotObjectType
 from nautobot.core.graphql.utils import str_to_var_name
-from nautobot.core.testing import NautobotTestClient, create_test_user
-from nautobot.dcim.choices import InterfaceTypeChoices, InterfaceModeChoices, PortTypeChoices, ConsolePortTypeChoices
+from nautobot.core.testing import create_test_user, NautobotTestClient
+from nautobot.dcim.choices import ConsolePortTypeChoices, InterfaceModeChoices, InterfaceTypeChoices, PortTypeChoices
 from nautobot.dcim.filters import DeviceFilterSet, LocationFilterSet
 from nautobot.dcim.graphql.types import DeviceType as DeviceTypeGraphQL
 from nautobot.dcim.models import (
@@ -52,17 +51,17 @@ from nautobot.dcim.models import (
     Location,
     LocationType,
     PowerFeed,
-    PowerPort,
     PowerOutlet,
     PowerPanel,
+    PowerPort,
     Rack,
     RearPort,
 )
 from nautobot.extras.choices import CustomFieldTypeChoices
 from nautobot.extras.models import (
     ChangeLoggedModel,
-    CustomField,
     ConfigContext,
+    CustomField,
     GraphQLQuery,
     Relationship,
     RelationshipAssociation,
@@ -82,8 +81,8 @@ from nautobot.ipam.models import (
     VRFDeviceAssignment,
     VRFPrefixAssignment,
 )
-from nautobot.users.models import ObjectPermission, Token
 from nautobot.tenancy.models import Tenant
+from nautobot.users.models import ObjectPermission, Token
 from nautobot.virtualization.factory import ClusterTypeFactory
 from nautobot.virtualization.models import Cluster, VirtualMachine, VMInterface
 
@@ -708,7 +707,7 @@ class GraphQLQueryTest(TestCase):
         roles = Role.objects.get_for_model(Device)
         cls.device_role1 = roles[0]
         cls.device_role2 = roles[1]
-        cls.device_role3 = random.choice(roles)
+        cls.device_role3 = random.choice(roles)  # noqa: S311  # suspicious-non-cryptographic-random-usage
         cls.location_statuses = list(Status.objects.get_for_model(Location))[:2]
         cls.location_type = LocationType.objects.get(name="Campus")
         cls.location1 = Location.objects.filter(location_type=cls.location_type).first()

nautobot/core/tests/test_jinja_filters.py

@@ -47,17 +47,17 @@ class NautobotJinjaFilterTest(TestCase):
             "slugify",
         ]
 
-        # For each helper, try to render a jinja template with render_jinja2 and fail if TemplateAssertionError is raised
+        # For each helper, try to render jinja template with render_jinja2 and fail if TemplateAssertionError is raised
         for helper in helpers_to_validate:
             try:
                 data.render_jinja2("{{ data | " + helper + " }}", {"data": None})
             except TemplateAssertionError:
                 raise
-            except Exception:
+            except Exception:  # noqa: S110  # try-except-pass -- an antipattern in general, but OK here
                 pass
 
     def test_netutils_filters_in_jinja(self):
-        """Import the list of all Jinja filters from Netutils and validate that all of them have been properly loaded in Django Jinja"""
+        """Import all Jinja filters from Netutils and validate that all have been properly loaded in Django Jinja."""
         filters = jinja2_convenience_function()
 
         for filter_ in filters.keys():
@@ -65,7 +65,7 @@ class NautobotJinjaFilterTest(TestCase):
                 data.render_jinja2("{{ data | " + filter_ + " }}", {"data": None})
             except TemplateAssertionError:
                 raise
-            except Exception:
+            except Exception:  # noqa: S110  # try-except-pass -- an antipattern in general, but OK here
                 pass
 
     def test_sandboxed_render(self):

nautobot/core/tests/test_jobs.py

@@ -3,7 +3,7 @@ from pathlib import Path
 from django.contrib.contenttypes.models import ContentType
 import yaml
 
-from nautobot.core.testing import TransactionTestCase, create_job_result_and_run_job
+from nautobot.core.testing import create_job_result_and_run_job, TransactionTestCase
 from nautobot.dcim.models import DeviceType, Manufacturer
 from nautobot.extras.choices import JobResultStatusChoices, LogLevelChoices
 from nautobot.extras.models import ExportTemplate, JobLogEntry, Status

nautobot/core/tests/test_models.py

@@ -1,8 +1,7 @@
 import time
-import uuid
-
 from unittest import skip
 from unittest.mock import patch
+import uuid
 
 from django.core.cache import cache
 from django.core.exceptions import ValidationError