chore(backend): make ruff 0.8.0 happy

adfinis · Nov 25, 2024 · a2b43e4 · a2b43e4
1 parent 994108e
commit a2b43e4
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 16 deletions.
diff --git a/backend/ruff.toml b/backend/ruff.toml
@@ -62,7 +62,6 @@ select = [
 	"W605",    # invalid escape sequence
 ]
 ignore = [
-	"ANN101",  # this is deprecated and annotating self is unnecessary
 	"D203",    # we prefer blank-line-before-class (D211) for black compat
 	"D213",    # we prefer multi-line-summary-first-line (D212)
 	"COM812",  # ignore due to conflict with formatter

diff --git a/backend/timed/employment/models.py b/backend/timed/employment/models.py
@@ -264,7 +264,7 @@ def calculate_worktime(
         # converting workdays as db expects 1 (Sunday) to 7 (Saturday)
         workdays_db = [
             # special case for Sunday
-            int(day) == 7 and 1 or int(day) + 1  # noqa: PLR2004
+            (int(day) == 7 and 1) or int(day) + 1  # noqa: PLR2004
             for day in self.location.workdays
         ]
         holidays = PublicHoliday.objects.filter(

diff --git a/backend/timed/notifications/tests/test_notify_reviewers_unverified.py b/backend/timed/notifications/tests/test_notify_reviewers_unverified.py
@@ -57,10 +57,7 @@ def test_notify_reviewers_with_cc_and_message(mailoutbox, cc, message):
     assert len(mailoutbox) == 1
     mail = mailoutbox[0]
     assert mail.to == [reviewer_work.email]
-    url = (
-        "http://localhost:4200/analysis?fromDate=2017-07-01&"
-        "toDate=2017-07-31&reviewer=%d&editable=1"
-    ) % reviewer_work.id
+    url = f"http://localhost:4200/analysis?fromDate=2017-07-01&toDate=2017-07-31&reviewer={reviewer_work.id}&editable=1"
     assert url in mail.body
     assert message in mail.body
     assert mail.cc[0] == cc
@@ -85,10 +82,7 @@ def test_notify_reviewers(mailoutbox):
     assert len(mailoutbox) == 1
     mail = mailoutbox[0]
     assert mail.to == [reviewer_work.email]
-    url = (
-        "http://localhost:4200/analysis?fromDate=2017-07-01&"
-        "toDate=2017-07-31&reviewer=%d&editable=1"
-    ) % reviewer_work.id
+    url = f"http://localhost:4200/analysis?fromDate=2017-07-01&toDate=2017-07-31&reviewer={reviewer_work.id}&editable=1"
     assert url in mail.body
     assert Notification.objects.count() == 1
 
@@ -118,8 +112,5 @@ def test_notify_reviewers_reviewer_hierarchy(mailoutbox):
     assert len(mailoutbox) == 1
     mail = mailoutbox[0]
     assert mail.to == [task_reviewer.email]
-    url = (
-        "http://localhost:4200/analysis?fromDate=2017-07-01&"
-        "toDate=2017-07-31&reviewer=%d&editable=1"
-    ) % task_reviewer.id
+    url = f"http://localhost:4200/analysis?fromDate=2017-07-01&toDate=2017-07-31&reviewer={task_reviewer.id}&editable=1"
     assert url in mail.body
diff --git a/backend/timed/reports/views.py b/backend/timed/reports/views.py
@@ -28,7 +28,7 @@
 from . import filters
 
 if TYPE_CHECKING:
-    from typing import Iterable
+    from collections.abc import Iterable
 
     from ezodf.document import FlatXMLDocument, PackagedDocument
 

diff --git a/backend/timed/tracking/views.py b/backend/timed/tracking/views.py
@@ -249,7 +249,7 @@ def bulk(self, request):
                     _("Reviewer filter needs to be set to verifying user")
                 )
 
-            fields["verified_by"] = verified and user or None
+            fields["verified_by"] = (verified and user) or None
 
             if fields.get("review") or any(queryset.values_list("review", flat=True)):
                 raise exceptions.ParseError(