chore: ruff findings

src/firefighter/incidents/models/incident.py

@@ -540,7 +540,7 @@ def update_roles(
 
         return incident_update
 
-    def create_incident_update(  # noqa: PLR0913
+    def create_incident_update(
         self: Incident,
         message: str | None = None,
         status: int | None = None,

src/firefighter/slack/admin.py

@@ -209,8 +209,7 @@ def save_model(
                     group_slack_id=obj.usergroup_id
                 )
             elif obj.handle:
-                if obj.handle.startswith("@"):
-                    obj.handle = obj.handle[1:]
+                obj.handle = obj.handle.removeprefix("@")
                 obj.handle.strip()
                 fetch_obj = UserGroup.objects.fetch_usergroup(group_handle=obj.handle)
             if fetch_obj:

src/firefighter/slack/signals/incident_updated.py

@@ -53,7 +53,7 @@ def incident_updated_update_status_handler(
         incident=incident,
         incident_update=incident_update,
         status_changed=bool("_status" in updated_fields),
-        old_priority=kwargs.get("old_priority", None),
+        old_priority=kwargs.get("old_priority"),
     )
 
 

tests/conftest.py

@@ -40,14 +40,14 @@ def _debug(settings: SettingsWrapper) -> None:
         template["OPTIONS"]["debug"] = True
 
 
-@pytest.fixture()
+@pytest.fixture
 def main_heading() -> str:
     """An example fixture containing some html fragment."""
     return '<p class="mt-2 text-sm text-base-content text-opacity-80">Report, manage, escalate!</p>'
 
 
 @pytest.fixture(scope="session")
-def django_db_setup(  # noqa: PT004
+def django_db_setup(
     django_db_setup: Any, django_db_blocker: DjangoDbBlocker
 ) -> None:
     # XXX Allow override of fixtures path
@@ -76,13 +76,13 @@ def django_db_setup(  # noqa: PT004
     django_db_blocker.restore()
 
 
-@pytest.fixture()
+@pytest.fixture
 def incident() -> Incident:
     return IncidentFactory.build()
 
 
-@pytest.fixture()
-@pytest.mark.django_db()
+@pytest.fixture
+@pytest.mark.django_db
 def incident_saved() -> Incident:
     incident: Incident = IncidentFactory.build()
     incident.component.group.save()

tests/test_api/test_api_urls.py

@@ -22,7 +22,7 @@ def test_api_schema(client: Client) -> None:
     assert response.status_code == 200
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_incidents_cost_api(admin_client: Client, admin_user: User) -> None:
     """This test ensures that the incidents costs API endpoint is accessible for an admin"""
     admin_client.force_login(admin_user)
@@ -31,7 +31,7 @@ def test_incidents_cost_api(admin_client: Client, admin_user: User) -> None:
     assert response.status_code == 200
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_incidents_cost_api_unauthorized(client: Client) -> None:
     """This test ensures that the incidents costs API endpoint is not accessible for a guest user"""
     user = UserFactory.create()
@@ -41,7 +41,7 @@ def test_incidents_cost_api_unauthorized(client: Client) -> None:
     assert response.status_code == 403
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_incidents_cost_type_api(admin_client: Client, admin_user: User) -> None:
     """This test ensures that the incidents cost types API endpoint is accessible for an admin"""
     admin_client.force_login(admin_user)
@@ -50,7 +50,7 @@ def test_incidents_cost_type_api(admin_client: Client, admin_user: User) -> None
     assert response.status_code == 200
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_incidents_cost_api_type_unauthorized(client: Client) -> None:
     """This test ensures that the incidents costs API endpoint is not accessible for a guest user"""
     user = UserFactory.create()

tests/test_firefighter/test_urls.py

@@ -17,23 +17,23 @@
 logger = logging.getLogger(__name__)
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_health_check(client: Client) -> None:
     """This test ensures that health check is accessible."""
     response = client.get("/api/v2/firefighter/monitoring/healthcheck")
 
     assert response.status_code == 200
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_admin_unauthorized(client: Client) -> None:
     """This test ensures that admin panel requires auth."""
     response = client.get("/admin/")
 
     assert response.status_code == 302
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 @pytest.mark.usefixtures("_debug")
 def test_admin_authorized(admin_client: Client, admin_user: User) -> None:
     """This test ensures that admin panel is accessible."""
@@ -44,7 +44,7 @@ def test_admin_authorized(admin_client: Client, admin_user: User) -> None:
     assert response.status_code == 200
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_admin_docs_unauthorized(client: Client) -> None:
     """This test ensures that admin panel docs requires auth."""
     response = client.get("/admin/doc/")
@@ -54,7 +54,7 @@ def test_admin_docs_unauthorized(client: Client) -> None:
     assert response.url == "/admin/login/?next=/admin/doc/"
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_admin_docs_authorized(admin_client: Client) -> None:
     """This test ensures that admin panel docs are accessible."""
     response = admin_client.get("/admin/doc/")
@@ -63,7 +63,7 @@ def test_admin_docs_authorized(admin_client: Client) -> None:
     assert b"docutils" not in response.content
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_robotstxt_present(client: Client) -> None:
     """This test ensures that robots.txt is present."""
     response = client.get("/robots.txt")
@@ -72,7 +72,7 @@ def test_robotstxt_present(client: Client) -> None:
     assert b"Disallow: /" in response.content
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_login_sso_button_admin_login(client: Client) -> None:
     """This test ensures that login page contains the SSO button."""
     response = client.get("/admin/login/")
@@ -81,7 +81,7 @@ def test_login_sso_button_admin_login(client: Client) -> None:
     assert b"Log in with SSO" in response.content
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_error_500_pretty(client: Client) -> None:
     """This test ensures that 500 error page is pretty."""
     with pytest.raises(Exception, match="Test exception for 500"):  # noqa: PT012
@@ -91,7 +91,7 @@ def test_error_500_pretty(client: Client) -> None:
         assert b"Please try again later, and report it" in response.content
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_error_404_pretty(client: Client) -> None:
     """This test ensures that 404 error page is pretty."""
     response = client.get("/err/404/")
@@ -100,7 +100,7 @@ def test_error_404_pretty(client: Client) -> None:
     assert b"Please check the URL in the address bar and try again" in response.content
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_error_403_pretty(client: Client) -> None:
     """This test ensures that 403 error page is pretty."""
     response = client.get("/err/403/")
@@ -109,7 +109,7 @@ def test_error_403_pretty(client: Client) -> None:
     assert b"You do not have permission to access this page." in response.content
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_error_400_pretty(client: Client) -> None:
     """This test ensures that 400 error page is pretty."""
     response = client.get("/err/400/")
@@ -129,7 +129,7 @@ def test_errors_json(client: Client) -> None:
         assert json.loads(response.content).get("error") is not None
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 @pytest.mark.timeout(30)
 @pytest.mark.usefixtures("_debug")
 def test_all_admin_list_no_error(admin_client: Client, admin_user: User) -> None:

tests/test_incidents/test_forms/test_form_select_impact.py

@@ -5,7 +5,7 @@
 from firefighter.incidents.forms.select_impact import SelectImpactForm
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 @pytest.mark.parametrize(
     ("form_data", "expected_priority"),
     [
@@ -63,7 +63,7 @@ def test_suggest_priority_from_impact(form_data, expected_priority):
     assert priority == expected_priority
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 @pytest.mark.parametrize(
     ("form_data", "expected_priority"),
     [
@@ -96,7 +96,7 @@ def test_suggest_priority_from_impact(form_data, expected_priority):
         ),
     ],
 )
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_suggest_priority_from_impact_with_invalid_form(form_data, expected_priority):
     form = SelectImpactForm(data=form_data)
     assert not form.is_valid()

tests/test_incidents/test_forms/test_form_utils.py

@@ -20,12 +20,12 @@ class GroupedModelChoiceFieldForm(Form):
     )
 
 
-@pytest.fixture()
+@pytest.fixture
 def group() -> Group:
     return Group.objects.create(name="Group 1")
 
 
-@pytest.fixture()
+@pytest.fixture
 def components(group: Group):
     return [
         Component.objects.create(name="Component 1", group=group, order=1),
@@ -46,7 +46,7 @@ def test_enum_choice_field_invalid() -> None:
     assert "status" in form.errors
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_grouped_model_choice_field_valid(components: list[Component]):
     form = GroupedModelChoiceFieldForm({"component": components[0].id})
     assert form.is_valid()
@@ -60,7 +60,7 @@ def test_grouped_model_choice_field_invalid() -> None:
 
 
 @pytest.fixture(scope="module")
-def test_grouped_model_choice_field_grouping(  # noqa: PT004
+def test_grouped_model_choice_field_grouping(
     components: list[Component],
 ):
     form = GroupedModelChoiceFieldForm()

tests/test_incidents/test_forms/test_update_key_events.py

@@ -10,12 +10,12 @@
 from firefighter.incidents.models.milestone_type import MilestoneType
 
 
-@pytest.fixture()
+@pytest.fixture
 def user() -> User:
     return User.objects.create_user(username="testuser")
 
 
-@pytest.fixture()
+@pytest.fixture
 def milestone_type() -> MilestoneType:
     return MilestoneType(
         event_type="detected",
@@ -26,7 +26,7 @@ def milestone_type() -> MilestoneType:
     )
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_incident_update_key_events_form(
     user: User, incident_saved: Incident, milestone_type: MilestoneType
 ):

tests/test_incidents/test_incident_urls.py

@@ -16,23 +16,23 @@
 logger = logging.getLogger(__name__)
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_incidents_dashboard_unauthorized(client: Client) -> None:
     """This test ensures that the incidents dashboard is not accessible, and that we are redirect to auth."""
     response = client.get(reverse("incidents:dashboard"))
 
     assert response.status_code == 302
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_incidents_list_unauthorized(client: Client) -> None:
     """This test ensures that the incidents page list is accessible."""
     response = client.get(reverse("incidents:incident-list"))
 
     assert response.status_code == 302
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 @pytest.mark.usefixtures("_debug")
 def test_incidents_statistics_unauthorized(client: Client) -> None:
     """This test ensures that the incidents statistics page is accessible."""
@@ -41,7 +41,7 @@ def test_incidents_statistics_unauthorized(client: Client) -> None:
     assert response.status_code == 302
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 @pytest.mark.usefixtures("_debug")
 def test_incidents_dashboard_authorized(admin_client: Client, admin_user: User) -> None:
     """This test ensures that the incidents dashboard is accessible for an admin."""
@@ -51,7 +51,7 @@ def test_incidents_dashboard_authorized(admin_client: Client, admin_user: User)
     assert response.status_code == 200
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 @pytest.mark.usefixtures("_debug")
 def test_incidents_details(client: Client, admin_user: User) -> None:
     """This test ensures that the incidents dashboard is accessible for an admin."""
@@ -65,7 +65,7 @@ def test_incidents_details(client: Client, admin_user: User) -> None:
     assert response.status_code == 200
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 @pytest.mark.usefixtures("_debug")
 def test_incidents_details_unauthorized_404_redirect(client: Client) -> None:
     """This test ensures that the incidents dashboard will return a 404."""
@@ -76,7 +76,7 @@ def test_incidents_details_unauthorized_404_redirect(client: Client) -> None:
     assert response.status_code == 302
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 @pytest.mark.usefixtures("_debug")
 def test_incidents_details_authorized_404(client: Client, admin_user: User) -> None:
     """This test ensures that the incidents dashboard will return a 404."""
@@ -88,7 +88,7 @@ def test_incidents_details_authorized_404(client: Client, admin_user: User) -> N
     assert response.status_code == 404
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 @pytest.mark.usefixtures("_debug")
 def test_incident_create_unauthorized(client: Client) -> None:
     """This test ensures that the incident create page is accessible."""
@@ -98,7 +98,7 @@ def test_incident_create_unauthorized(client: Client) -> None:
     assert response.status_code == 302
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 @pytest.mark.usefixtures("_debug")
 def test_component_list(client: Client) -> None:
     """This test ensures that the component list is accessible."""
@@ -107,7 +107,7 @@ def test_component_list(client: Client) -> None:
     assert response.status_code == 302
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 @pytest.mark.usefixtures("_debug")
 def test_incident_list(client: Client) -> None:
     """This test ensures that the incident list is accessible."""

tests/test_incidents/test_models/test_incident_model.py

@@ -13,7 +13,7 @@
     from firefighter.incidents.models import Incident
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 class TestIncident(django.TestCase):
     """This is a property-based test that ensures model correctness."""
 

tests/test_incidents/test_views/test_incident_detail_view.py

@@ -8,7 +8,7 @@
 from firefighter.incidents.models.user import User
 
 
-@pytest.mark.django_db()
+@pytest.mark.django_db
 def test_incident_detail_view(
     client: Client, main_heading: str, incident_saved: Incident, admin_user: User
 ) -> None: