chore: recommended updates

Author: Michael Brewer

aws_lambda_powertools/utilities/parser/models/apigw.py

@@ -68,7 +68,7 @@ class APIGatewayEventRequestContext(BaseModel):
     routeKey: Optional[str]
     operationName: Optional[str]
 
-    @root_validator()
+    @root_validator
     def check_message_id(cls, values):
         message_id, event_type = values.get("messageId"), values.get("eventType")
         if message_id is not None and event_type != "MESSAGE":

tests/functional/parser/test_apigw.py

@@ -108,7 +108,27 @@ def test_apigw_event():
 def test_apigw_event_with_invalid_websocket_request():
     # GIVEN an event with an eventType != MESSAGE and has  a messageId
     event = {
+        "resource": "/",
+        "path": "/",
+        "httpMethod": "GET",
+        "headers": {},
+        "multiValueHeaders": {},
+        "isBase64Encoded": False,
+        "body": "Foo!",
         "requestContext": {
+            "accountId": "1234",
+            "apiId": "myApi",
+            "httpMethod": "GET",
+            "identity": {
+                "sourceIp": "127.0.0.1",
+            },
+            "path": "/",
+            "protocol": "Https",
+            "requestId": "1234",
+            "requestTime": "2018-09-07T16:20:46Z",
+            "requestTimeEpoch": 1536992496000,
+            "resourcePath": "/",
+            "stage": "test",
             "eventType": "DISCONNECT",
             "messageId": "messageId",
         },
@@ -119,4 +139,5 @@ def test_apigw_event_with_invalid_websocket_request():
         handle_apigw_event(event, LambdaContext())
 
     # THEN raise TypeError for invalid event
+    assert len(err.value.errors()) == 1
     assert "messageId is available only when the `eventType` is `MESSAGE`" in str(err.value)