nautobot 2.1.0b1__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()
@@ -123,6 +123,7 @@ def export_button(context, content_type=None):
     else:
         export_templates = []
         export_url = None
+        include_yaml_option = False
 
     return {
         "export_url": export_url,

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,8 +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
@@ -123,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()
@@ -180,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()
@@ -261,6 +262,34 @@ def validated_viewname(model, action):
         return None
 
 
+@library.filter()
+@register.filter()
+def validated_api_viewname(model, action):
+    """
+    Return the API view name for the given model and action if valid, or None if invalid.
+
+    Args:
+        model (models.Model): Class or Instance of a Django Model
+        action (str): name of the action in the viewname
+
+    Returns:
+        (Union[str, None]): return the name of the API view for the model/action provided if valid, or None if invalid.
+    """
+    viewname_str = lookup.get_route_for_model(model, action, api=True)
+
+    try:
+        # Validate and return the view name. We don't return the actual URL yet because many of the templates
+        # are written to pass a name to {% url %}.
+        if action == "detail":
+            # Detail views require an argument, so we'll pass a dummy value just for validation
+            reverse(viewname_str, args=["00000000-0000-0000-0000-000000000000"])
+        else:
+            reverse(viewname_str)
+        return viewname_str
+    except NoReverseMatch:
+        return None
+
+
 @library.filter()
 @register.filter()
 def bettertitle(value):
@@ -707,8 +736,10 @@ def custom_branding_or_static(branding_asset, static_asset):
     branding has been configured in settings, else it returns stock branding via static.
     """
     if settings.BRANDING_FILEPATHS.get(branding_asset):
-        return f"{ settings.MEDIA_URL }{ settings.BRANDING_FILEPATHS.get(branding_asset) }"
-    return StaticNode.handle_simple(static_asset)
+        url = f"{ settings.MEDIA_URL }{ settings.BRANDING_FILEPATHS.get(branding_asset) }"
+    else:
+        url = StaticNode.handle_simple(static_asset)
+    return add_nautobot_version_query_param_to_url(url)
 
 
 @register.simple_tag
@@ -725,6 +756,13 @@ def support_message():
     return render_markdown(message)
 
 
+@register.simple_tag
+def versioned_static(file_path):
+    """Returns a versioned static file URL with a query parameter containing the version number."""
+    url = static(file_path)
+    return add_nautobot_version_query_param_to_url(url)
+
+
 @library.filter()
 @register.filter()
 def hyperlinked_object_with_color(obj):
@@ -741,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

@@ -3,16 +3,16 @@ from typing import Optional, Sequence
 from unittest import skipIf
 import uuid
 
+from django.apps import apps
 from django.conf import settings
 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
 
@@ -20,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__ = (
@@ -90,7 +88,9 @@ class ModelViewTestCase(ModelTestCase):
         Override this if needed for testing of views that don't correspond directly to self.model,
         for example the DCIM "interface-connections" and "console-connections" view tests.
         """
-        if self.model._meta.app_label in settings.PLUGINS:
+        app_name = apps.get_app_config(app_label=self.model._meta.app_label).name
+        # AppConfig.name accounts for NautobotApps that are not built at the root of the package
+        if app_name in settings.PLUGINS:
             return f"plugins:{self.model._meta.app_label}:{self.model._meta.model_name}_{{}}"
         return f"{self.model._meta.app_label}:{self.model._meta.model_name}_{{}}"
 
@@ -1152,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_home.py

@@ -101,7 +101,7 @@ class HomeTestCase(SeleniumTestCase):
                         f"/div[@class='panel-heading']"
                         f"/strong[contains(text(), '{panel_name}')]"
                         f"/../.."
-                        f"/div[@class='list-group']"
+                        f"/div[@class='list-group collapse in collapsible-div']"
                     )
                     links = panel_element_to_search.links.find_by_text(item_name)
                     if item_details["permission"] in user_permissions:

nautobot/core/tests/integration/test_navbar.py

@@ -57,7 +57,7 @@ class NavBarTestCase(SeleniumTestCase):
 
         for tab_name, groups in self.navbar.items():
             # XPath to find tabs using the tab name
-            tab_xpath = f"//*[@id='navbar']//*[normalize-space()='{tab_name}']"
+            tab_xpath = f"//*[@id='navbar']//span[normalize-space()='{tab_name}']/.."
             tab = self.browser.find_by_xpath(tab_xpath)
             tab.click()
             self.assertTrue(bool(tab["aria-expanded"]))
@@ -92,7 +92,7 @@ class NavBarTestCase(SeleniumTestCase):
                         tab_flag = True
 
             # XPath to find tabs using the tab name
-            tabs = self.browser.find_by_xpath(f"//*[@id='navbar']//*[normalize-space()='{tab_name}']")
+            tabs = self.browser.find_by_xpath(f"//*[@id='navbar']//span[normalize-space()='{tab_name}']/..")
             if tab_flag:
                 self.assertEqual(len(tabs), 1)
             else:

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
 
 
@@ -80,7 +80,7 @@ class PluginNavBarTestCase(SeleniumTestCase):
         # Retrieve home page
         self.browser.visit(self.live_server_url)
 
-        tab_xpath = "//*[@id='navbar']//*[normalize-space()='Example Menu']"
+        tab_xpath = "//*[@id='navbar']//span[normalize-space()='Example Menu']/.."
         tab = self.browser.find_by_xpath(tab_xpath)
         tab.click()
         self.assertTrue(bool(tab["aria-expanded"]))
@@ -113,11 +113,11 @@ class PluginNavBarTestCase(SeleniumTestCase):
                 item = group.find_by_xpath(item_xpath)
 
                 for button_name in item_details["buttons"]:
-                    button = item.find_by_xpath(f"{item_xpath}/div//a[@title='{button_name}']")
+                    button = item.find_by_xpath(f"{item_xpath}/div//a[@data-original-title='{button_name}']")
                     if button_class := getattr(ButtonActionColorChoices, button_name.upper(), None):
                         self.assertIn(button_class, button["class"])
                     if button_icon := getattr(ButtonActionIconChoices, button_name.upper(), None):
-                        icon = button.find_by_xpath(f"{item_xpath}/div//a[@title='{button_name}']/i")
+                        icon = button.find_by_xpath(f"{item_xpath}/div//a[@data-original-title='{button_name}']/i")
                         self.assertIn(button_icon, icon["class"])
 
         tab.click()
@@ -145,11 +145,11 @@ class PluginNavBarTestCase(SeleniumTestCase):
                 item = group.find_by_xpath(item_xpath)
 
                 for button_name in item_details["buttons"]:
-                    button = item.find_by_xpath(f"{item_xpath}/div//a[@title='{button_name}']")
+                    button = item.find_by_xpath(f"{item_xpath}/div//a[@data-original-title='{button_name}']")
                     if button_class := getattr(ButtonActionColorChoices, button_name.upper(), None):
                         self.assertIn(button_class, button.get_attribute("class"))
                     if button_icon := getattr(ButtonActionIconChoices, button_name.upper(), None):
-                        icon = button.find_by_xpath(f"{item_xpath}/div//a[@title='{button_name}']/i")
+                        icon = button.find_by_xpath(f"{item_xpath}/div//a[@data-original-title='{button_name}']/i")
                         self.assertIn(button_icon, icon["class"])
 
         tab.click()

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
@@ -25,7 +25,7 @@ class CSVParsingRelatedTestCase(TestCase):
         devicerole = Role.objects.get_for_model(Device).first()
         device_status = Status.objects.get_for_model(Device).first()
         tags = Tag.objects.get_for_model(Device).all()[:3]
-
+        Device.objects.all().delete()
         self.device = Device.objects.create(
             device_type=devicetype,
             role=devicerole,

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):