fix: EventBridgeEvent data class

Author: Amin Farjadi

aws_lambda_powertools/utilities/data_classes/event_bridge_event.py

@@ -14,41 +14,41 @@ class EventBridgeEvent(DictWrapper):
     """
 
     @property
-    def get_id(self) -> str:
+    def get_id(self) -> str | None:
         """A unique value is generated for every event. This can be helpful in tracing events as
         they move through rules to targets, and are processed."""
         # Note: this name conflicts with existing python builtins
-        return self["id"]
+        return self.get("id")
 
     @property
-    def version(self) -> str:
+    def version(self) -> str | None:
         """By default, this is set to 0 (zero) in all events."""
-        return self["version"]
+        return self.get("version")
 
     @property
-    def account(self) -> str:
+    def account(self) -> str | None:
         """The 12-digit number identifying an AWS account."""
-        return self["account"]
+        return self.get("account")
 
     @property
-    def time(self) -> str:
+    def time(self) -> str | None:
         """The event timestamp, which can be specified by the service originating the event.
 
         If the event spans a time interval, the service might choose to report the start time, so
         this value can be noticeably before the time the event is actually received.
         """
-        return self["time"]
+        return self.get("time")
 
     @property
-    def region(self) -> str:
+    def region(self) -> str | None:
         """Identifies the AWS region where the event originated."""
-        return self["region"]
+        return self.get("region")
 
     @property
-    def resources(self) -> list[str]:
+    def resources(self) -> list[str] | None:
         """This JSON array contains ARNs that identify resources that are involved in the event.
         Inclusion of these ARNs is at the discretion of the service."""
-        return self["resources"]
+        return self.get("resources")
 
     @property
     def source(self) -> str:
@@ -68,4 +68,4 @@ def detail(self) -> dict[str, Any]:
     @property
     def replay_name(self) -> str | None:
         """Identifies whether the event is being replayed and what is the name of the replay."""
-        return self["replay-name"]
+        return self.get("replay-name")

tests/events/eventBridgeEventBase.json

@@ -0,0 +1,8 @@
+{
+  "detail-type": "EC2 Instance State-change Notification",
+  "source": "aws.ec2",
+  "detail": {
+    "instance_id": "i-1234567890abcdef0",
+    "state": "terminated"
+  }
+}

tests/unit/data_classes/required_dependencies/test_event_bridge_event.py

@@ -18,3 +18,20 @@ def test_event_bridge_event():
     assert parsed_event.detail_type == raw_event["detail-type"]
     assert parsed_event.detail == raw_event["detail"]
     assert parsed_event.replay_name == "replay_archive"
+
+
+def test_event_bridge_event_base():
+    raw_event = load_event("eventBridgeEventBase.json")
+    parsed_event = EventBridgeEvent(raw_event)
+
+    assert parsed_event.source == raw_event["source"]
+    assert parsed_event.detail_type == raw_event["detail-type"]
+    assert parsed_event.detail == raw_event["detail"]
+
+    assert parsed_event.get_id is None
+    assert parsed_event.version is None
+    assert parsed_event.account is None
+    assert parsed_event.time is None
+    assert parsed_event.region is None
+    assert parsed_event.resources is None
+    assert parsed_event.replay_name is None