Addressing Heitor's feedback

Author: leandrodamascena

aws_lambda_powertools/event_handler/util.py

@@ -36,7 +36,10 @@ class _FrozenListDict(List[Dict[str, List[str]]]):
     """
 
     def __hash__(self):
-        return hash(frozenset({_FrozenDict({key: frozenset(self) for key, self in item.items()}) for item in self}))
+        hashable_items = []
+        for item in self:
+            hashable_items.extend((key, frozenset(value)) for key, value in item.items())
+        return hash(frozenset(hashable_items))
 
 
 def extract_origin_header(resolver_headers: Dict[str, Any]):

tests/functional/event_handler/test_openapi_security.py

@@ -32,6 +32,7 @@ def handler():
 def test_openapi_top_level_security_missing():
     # GIVEN an APIGatewayRestResolver instance
     app = APIGatewayRestResolver()
+    app.enable_swagger()
 
     @app.get("/")
     def handler():
@@ -48,17 +49,14 @@ def handler():
 def test_openapi_operation_security():
     # GIVEN an APIGatewayRestResolver instance
     app = APIGatewayRestResolver()
+    security_schemes = {"apiKey": APIKey(name="X-API-KEY", description="API Key", in_=APIKeyIn.header)}
 
     @app.get("/", security=[{"apiKey": []}])
     def handler():
         raise NotImplementedError()
 
     # WHEN the get_openapi_schema method is called with security defined at the operation level
-    schema = app.get_openapi_schema(
-        security_schemes={
-            "apiKey": APIKey(name="X-API-KEY", description="API Key", in_=APIKeyIn.header),
-        },
-    )
+    schema = app.get_openapi_schema(security_schemes=security_schemes)
 
     # THEN the resulting schema should have security defined at the operation level, not the top level
     top_level_security = schema.security
@@ -71,6 +69,7 @@ def test_openapi_operation_security_with_router():
     # GIVEN an APIGatewayRestResolver instance with a Router
     app = APIGatewayRestResolver()
     router = Router()
+    security_schemes = {"apiKey": APIKey(name="X-API-KEY", description="API Key", in_=APIKeyIn.header)}
 
     @router.get("/", security=[{"apiKey": []}])
     def handler():
@@ -79,11 +78,7 @@ def handler():
     app.include_router(router)
 
     # WHEN the get_openapi_schema method is called with security defined at the operation level in the Router
-    schema = app.get_openapi_schema(
-        security_schemes={
-            "apiKey": APIKey(name="X-API-KEY", description="API Key", in_=APIKeyIn.header),
-        },
-    )
+    schema = app.get_openapi_schema(security_schemes=security_schemes)
 
     # THEN the resulting schema should have security defined at the operation level
     top_level_security = schema.security