fix linting

Author: Stephen Bracken

providers/keycloak/docs/auth-manager/index.rst

@@ -60,4 +60,4 @@ When using the Keycloak auth manager with a multi-team deployment, you will need
 
 If a team has dags added to Airflow but they are not setup in the keycloak client, keycloak will return a 500 error on the ``/dags`` screen:
 ```Resource with id [Dag:<team name>] does not exist.```
-The keycloak auth manager will handle this error by registering it as a 'deny' response to preserve access to the ``/dags`` screen for other teams.
+The keycloak auth manager will handle this error by registering it as a 'deny' response to preserve access to the ``/dags`` screen for other teams.

providers/keycloak/src/airflow/providers/keycloak/auth_manager/keycloak_auth_manager.py

@@ -82,7 +82,11 @@
 
 
 def _is_missing_keycloak_resource_response(resource_name: str, status_code: int, text: Any) -> bool:
-    return status_code == 500 and isinstance(text, str) and KEYCLOAK_RESOURCE_NOT_FOUND_ERROR.format(resource_name=resource_name).lower() in text.lower()
+    return (
+        status_code == 500
+        and isinstance(text, str)
+        and KEYCLOAK_RESOURCE_NOT_FOUND_ERROR.format(resource_name=resource_name).lower() in text.lower()
+    )
 
 
 TEAM_SCOPED_RESOURCES = frozenset(
@@ -444,7 +448,9 @@ def _is_authorized(
             raise AirflowException(
                 f"Request not recognized by Keycloak. {error.get('error')}. {error.get('error_description')}"
             )
-        if is_team_resource and _is_missing_keycloak_resource_response(resource_name, resp.status_code, resp.text):
+        if is_team_resource and _is_missing_keycloak_resource_response(
+            resource_name, resp.status_code, resp.text
+        ):
             log.warning("Keycloak authorization resource is missing; denying access. Response: %s", resp.text)
             return False
         raise AirflowException(f"Unexpected error: {resp.status_code} - {resp.text}")

providers/keycloak/tests/unit/keycloak/auth_manager/test_keycloak_auth_manager.py

@@ -38,8 +38,8 @@
     VariableDetails,
 )
 
-from tests_common.test_utils.version_compat import AIRFLOW_V_3_1_7_PLUS, AIRFLOW_V_3_2_PLUS
 from tests_common.test_utils.config import conf_vars
+from tests_common.test_utils.version_compat import AIRFLOW_V_3_1_7_PLUS, AIRFLOW_V_3_2_PLUS
 
 if AIRFLOW_V_3_2_PLUS:
     from airflow.api_fastapi.auth.managers.models.resource_details import TeamDetails
@@ -66,8 +66,6 @@
 )
 from airflow.providers.keycloak.auth_manager.user import KeycloakAuthManagerUser
 
-from tests_common.test_utils.config import conf_vars
-
 
 def _build_access_token(payload: dict[str, object]) -> str:
     header = {"alg": "none", "typ": "JWT"}
@@ -396,7 +394,9 @@ def test_is_authorized_missing_keycloak_resource(self, auth_manager, user, caplo
         auth_manager.http_session.post = Mock(return_value=resp)
         caplog.set_level("WARNING", logger="airflow.providers.keycloak.auth_manager.keycloak_auth_manager")
 
-        result = auth_manager.is_authorized_dag(method="GET", details=DagDetails(id="dag_0", team_name="team-a"), user=user)
+        result = auth_manager.is_authorized_dag(
+            method="GET", details=DagDetails(id="dag_0", team_name="team-a"), user=user
+        )
 
         assert result is False
         assert "Keycloak authorization resource is missing; denying access" in caplog.text