fix lint error

Author: yvonnep165

firebase_admin/app_check.py

@@ -107,7 +107,9 @@ def verify_token(self, token: str, consume: bool = False) -> Dict[str, Any]:
             except requests.exceptions.RequestException as error:
                 raise _utils.handle_requests_error(error)
 
-            already_consumed = body.get('alreadyConsumed', False) if isinstance(body, dict) else False
+            already_consumed = False
+            if isinstance(body, dict):
+                already_consumed = body.get('alreadyConsumed', False)
             verified_claims['already_consumed'] = bool(already_consumed)
 
         return verified_claims

tests/test_app_check.py

@@ -15,6 +15,7 @@
 """Test cases for the firebase_admin.app_check module."""
 import base64
 import pytest
+import requests
 
 from jwt import PyJWK, InvalidAudienceError, InvalidIssuerError
 from jwt import ExpiredSignatureError, InvalidSignatureError
@@ -280,36 +281,47 @@ def test_verify_token_with_consume_true_not_consumed(self, mocker):
         mocker.patch("jwt.get_unverified_header", return_value=JWT_PAYLOAD_SAMPLE.get("headers"))
         app = firebase_admin.get_app()
         app_check_service = app_check._get_app_check_service(app)
-        mock_body = mocker.patch.object(app_check_service._http_client, "body", return_value={"alreadyConsumed": False})
+        mock_body = mocker.patch.object(
+            app_check_service._http_client, "body", return_value={"alreadyConsumed": False}
+        )
 
         payload = app_check.verify_token("encoded", app=app, consume=True)
         expected = JWT_PAYLOAD_SAMPLE.copy()
         expected["app_id"] = APP_ID
         expected["already_consumed"] = False
         assert payload == expected
 
-        expected_url = f"https://firebaseappcheck.googleapis.com/v1/projects/{PROJECT_ID}:verifyAppCheckToken"
-        mock_body.assert_called_once_with("post", expected_url, json={"app_check_token": "encoded"})
+        expected_url = (
+            f"https://firebaseappcheck.googleapis.com/v1/projects/{PROJECT_ID}:verifyAppCheckToken"
+        )
+        mock_body.assert_called_once_with(
+            "post", expected_url, json={"app_check_token": "encoded"}
+        )
 
     def test_verify_token_with_consume_true_already_consumed(self, mocker):
         mocker.patch("jwt.decode", return_value=JWT_PAYLOAD_SAMPLE)
         mocker.patch("jwt.PyJWKClient.get_signing_key_from_jwt", return_value=PyJWK(signing_key))
         mocker.patch("jwt.get_unverified_header", return_value=JWT_PAYLOAD_SAMPLE.get("headers"))
         app = firebase_admin.get_app()
         app_check_service = app_check._get_app_check_service(app)
-        mock_body = mocker.patch.object(app_check_service._http_client, "body", return_value={"alreadyConsumed": True})
+        mock_body = mocker.patch.object(
+            app_check_service._http_client, "body", return_value={"alreadyConsumed": True}
+        )
 
         payload = app_check.verify_token("encoded", app=app, consume=True)
         expected = JWT_PAYLOAD_SAMPLE.copy()
         expected["app_id"] = APP_ID
         expected["already_consumed"] = True
         assert payload == expected
 
-        expected_url = f"https://firebaseappcheck.googleapis.com/v1/projects/{PROJECT_ID}:verifyAppCheckToken"
-        mock_body.assert_called_once_with("post", expected_url, json={"app_check_token": "encoded"})
+        expected_url = (
+            f"https://firebaseappcheck.googleapis.com/v1/projects/{PROJECT_ID}:verifyAppCheckToken"
+        )
+        mock_body.assert_called_once_with(
+            "post", expected_url, json={"app_check_token": "encoded"}
+        )
 
     def test_verify_token_with_consume_true_backend_error(self, mocker):
-        import requests
         mocker.patch("jwt.decode", return_value=JWT_PAYLOAD_SAMPLE)
         mocker.patch("jwt.PyJWKClient.get_signing_key_from_jwt", return_value=PyJWK(signing_key))
         mocker.patch("jwt.get_unverified_header", return_value=JWT_PAYLOAD_SAMPLE.get("headers"))