Update ruff config

Author: RealOrangeOne

django_tasks/backends/database/management/commands/db_worker.py

@@ -94,7 +94,7 @@ def run(self) -> None:
 
         if self.startup_delay and self.interval:
             # Add a random small delay before starting to avoid a thundering herd
-            time.sleep(random.random())
+            time.sleep(random.random())  # noqa: S311
 
         while self.running:
             tasks = DBTaskResult.objects.ready().filter(backend_name=self.backend_name)

pyproject.toml

@@ -69,12 +69,13 @@ rq = [
 ]
 
 [tool.ruff.lint]
-select = ["E", "F", "I", "W", "N", "B", "A", "C4", "T20", "DJ"]
+select = ["E", "F", "I", "W", "N", "B", "A", "C4", "T20", "DJ", "S", "UP"]
 ignore = ["E501", "DJ008"]
 
 [tool.ruff.lint.per-file-ignores]
 "tests/db_worker_test_settings.py" = ["F403", "F405"]
 "tests/settings_fast.py" = ["F403", "F405"]
+"tests/**.py" = ["S101", "S603", "S105"]
 
 [tool.mypy]
 plugins = ["mypy_django_plugin.main"]

tests/tests/test_database_backend.py

@@ -9,11 +9,12 @@
 import uuid
 import warnings
 from collections import Counter
+from collections.abc import Sequence
 from contextlib import redirect_stderr
 from datetime import datetime, timedelta
 from functools import partial
 from io import StringIO
-from typing import Any, List, Optional, Sequence, Union, cast
+from typing import Any, cast
 from unittest import mock, skipIf
 
 import django
@@ -944,7 +945,7 @@ def test_context(self) -> None:
     }
 )
 class DatabaseTaskResultTestCase(TransactionTestCase):
-    def execute_in_new_connection(self, sql: Union[str, QuerySet]) -> Sequence:
+    def execute_in_new_connection(self, sql: str | QuerySet) -> Sequence:
         if isinstance(sql, QuerySet):
             sql = str(sql.query)
         new_connection = connections.create_connection("default")
@@ -1423,7 +1424,7 @@ class DatabaseWorkerProcessTestCase(TransactionTestCase):
     WORKER_STARTUP_TIME = 1
 
     def setUp(self) -> None:
-        self.processes: List[subprocess.Popen] = []
+        self.processes: list[subprocess.Popen] = []
 
     def tearDown(self) -> None:
         # Try n times to kill any remaining child processes
@@ -1437,10 +1438,10 @@ def tearDown(self) -> None:
 
     def start_worker(
         self,
-        args: Optional[List[str]] = None,
+        args: list[str] | None = None,
         *,
         debug: bool = False,
-        worker_id: Optional[str] = None,
+        worker_id: str | None = None,
     ) -> subprocess.Popen:
         if args is None:
             args = []

tests/tests/test_rq_backend.py

@@ -1,7 +1,7 @@
 import json
 import os
 import uuid
-from typing import Union, cast
+from typing import cast
 from unittest.mock import patch
 
 import django_rq
@@ -25,9 +25,7 @@
 # Set up the connection before RQ Django reads the settings.
 # The connection must be the same because in fakeredis connections
 # do not share the state. Therefore, we define a singleton object to reuse it.
-def get_fake_connection(
-    config: dict, strict: bool
-) -> Union[FakeRedis, FakeStrictRedis]:
+def get_fake_connection(config: dict, strict: bool) -> FakeRedis | FakeStrictRedis:
     redis_cls = FakeStrictRedis if strict else FakeRedis
     if "URL" in config:
         return redis_cls.from_url(