Bump ruff from 0.8.6 to 0.9.1 (#72)

Bumps [ruff](https://github.com/astral-sh/ruff) from 0.8.6 to 0.9.1.
- [Release notes](https://github.com/astral-sh/ruff/releases)
- [Changelog](https://github.com/astral-sh/ruff/blob/main/CHANGELOG.md)
- [Commits](astral-sh/ruff@0.8.6...0.9.1)

---
updated-dependencies:
- dependency-name: ruff
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Tim Walter <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

kcwarden/custom_types/result.py

@@ -102,5 +102,5 @@ def get_severity_by_name(severity_name: str) -> Severity:
         return Severity[severity_name.lower().capitalize()]
     except KeyError:
         raise ValueError(
-            f"Provided severity was \"{severity_name}\" but must be one of {', '.join(s.name for s in Severity)}."
+            f'Provided severity was "{severity_name}" but must be one of {", ".join(s.name for s in Severity)}.'
         )

pyproject.toml

@@ -24,7 +24,7 @@ pyyaml = "^6.0.2"
 pytest = "^8.3.2"
 testcontainers = { extras = ["keycloak"], version = "^4.8.0" }
 python-keycloak = ">=4.3,<6.0"
-ruff = ">=0.6.3,<0.9.0"
+ruff = ">=0.6.3,<0.10.0"
 
 [tool.poetry.group.docs]
 optional = true

tests/auditors/client/test_clients_should_disable_implicit_grant_flow.py

@@ -24,9 +24,9 @@ def auditor(self, database, default_config):
     )
     def test_consider_client_based_on_oidc_status(self, mock_client, auditor, is_oidc, expected):
         mock_client.is_oidc_client.return_value = is_oidc
-        assert (
-            auditor.should_consider_client(mock_client) == expected
-        ), "Client consideration logic failed based on OIDC status"
+        assert auditor.should_consider_client(mock_client) == expected, (
+            "Client consideration logic failed based on OIDC status"
+        )
 
     @pytest.mark.parametrize(
         "has_implicit_flow,expected",
@@ -37,9 +37,9 @@ def test_consider_client_based_on_oidc_status(self, mock_client, auditor, is_oid
     )
     def test_detect_implicit_grant_flow(self, mock_client, auditor, has_implicit_flow, expected):
         mock_client.has_implicit_flow_enabled.return_value = has_implicit_flow
-        assert (
-            auditor.client_uses_implicit_grant_flow(mock_client) == expected
-        ), "Implicit grant flow detection logic failed"
+        assert auditor.client_uses_implicit_grant_flow(mock_client) == expected, (
+            "Implicit grant flow detection logic failed"
+        )
 
     @pytest.mark.parametrize(
         "enable_implicit_flow,expected_count",

tests/auditors/client/test_confidential_client_should_disable_direct_access_grants.py

@@ -27,9 +27,9 @@ def auditor(self, database, default_config):
     def test_should_consider_client(self, mock_client, auditor, is_oidc, is_public, expected):
         mock_client.is_oidc_client.return_value = is_oidc
         mock_client.is_public.return_value = is_public
-        assert (
-            auditor.should_consider_client(mock_client) == expected
-        ), "Client consideration logic failed based on OIDC status and confidentiality"
+        assert auditor.should_consider_client(mock_client) == expected, (
+            "Client consideration logic failed based on OIDC status and confidentiality"
+        )
 
     @pytest.mark.parametrize(
         "has_direct_access_grants,expected",
@@ -40,9 +40,9 @@ def test_should_consider_client(self, mock_client, auditor, is_oidc, is_public,
     )
     def test_client_uses_direct_access_grants(self, confidential_client, auditor, has_direct_access_grants, expected):
         confidential_client.has_direct_access_grants_enabled.return_value = has_direct_access_grants
-        assert (
-            auditor.client_uses_direct_access_grants(confidential_client) == expected
-        ), "Direct access grants detection logic failed"
+        assert auditor.client_uses_direct_access_grants(confidential_client) == expected, (
+            "Direct access grants detection logic failed"
+        )
 
     @pytest.mark.parametrize(
         "enable_direct_access_grants,expected_count",

tests/auditors/client/test_public_client_should_disable_direct_access_grants.py

@@ -27,9 +27,9 @@ def auditor(self, database, default_config):
     def test_client_consideration_logic(self, mock_client, auditor, is_oidc, is_public, expected):
         mock_client.is_oidc_client.return_value = is_oidc
         mock_client.is_public.return_value = is_public
-        assert (
-            auditor.should_consider_client(mock_client) == expected
-        ), "Client consideration logic failed based on type and OIDC status"
+        assert auditor.should_consider_client(mock_client) == expected, (
+            "Client consideration logic failed based on type and OIDC status"
+        )
 
     @pytest.mark.parametrize(
         "has_direct_access_grants,expected",
@@ -40,9 +40,9 @@ def test_client_consideration_logic(self, mock_client, auditor, is_oidc, is_publ
     )
     def test_detect_direct_access_grants(self, public_client, auditor, has_direct_access_grants, expected):
         public_client.has_direct_access_grants_enabled.return_value = has_direct_access_grants
-        assert (
-            auditor.client_uses_direct_access_grants(public_client) == expected
-        ), "Direct access grants detection logic failed"
+        assert auditor.client_uses_direct_access_grants(public_client) == expected, (
+            "Direct access grants detection logic failed"
+        )
 
     @pytest.mark.parametrize(
         "enable_direct_access_grants,expected_count",