Skip to content

Commit 5535469

Browse files
amin-farjadiAmin Farjadileandrodamascena
authored
fix: EventBridgeEvent data class (#8301)
* fix: EventBridgeEvent data class * refactor: improve namings, add docstring * address comment on issue 8300 * fix small things --------- Co-authored-by: Amin Farjadi <amin.farjadi@eonnext.com> Co-authored-by: Leandro Damascena <lcdama@amazon.pt>
1 parent f8c7faf commit 5535469

4 files changed

Lines changed: 6 additions & 9 deletions

File tree

aws_lambda_powertools/utilities/data_classes/event_bridge_event.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,4 +68,4 @@ def detail(self) -> dict[str, Any]:
6868
@property
6969
def replay_name(self) -> str | None:
7070
"""Identifies whether the event is being replayed and what is the name of the replay."""
71-
return self["replay-name"]
71+
return self.get("replay-name")

tests/events/eventBridgeEvent.json

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,9 @@
66
"account": "111122223333",
77
"time": "2017-12-22T18:43:48Z",
88
"region": "us-west-1",
9-
"resources": [
10-
"arn:aws:ec2:us-west-1:123456789012:instance/i-1234567890abcdef0"
11-
],
9+
"resources": [],
1210
"detail": {
1311
"instance_id": "i-1234567890abcdef0",
1412
"state": "terminated"
15-
},
16-
"replay-name": "replay_archive"
17-
}
13+
}
14+
}

tests/unit/data_classes/required_dependencies/test_event_bridge_event.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,4 +17,4 @@ def test_event_bridge_event():
1717
assert parsed_event.source == raw_event["source"]
1818
assert parsed_event.detail_type == raw_event["detail-type"]
1919
assert parsed_event.detail == raw_event["detail"]
20-
assert parsed_event.replay_name == "replay_archive"
20+
assert parsed_event.replay_name is None

tests/unit/parser/_pydantic/test_eventbridge.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ def test_handle_eventbridge_trigger_event_no_envelope():
4545
assert parsed_event.resources == raw_event["resources"]
4646
assert parsed_event.source == raw_event["source"]
4747
assert parsed_event.detail_type == raw_event["detail-type"]
48-
assert parsed_event.replay_name == raw_event["replay-name"]
48+
assert parsed_event.replay_name is None
4949

5050

5151
def test_handle_invalid_event_with_eventbridge_envelope():

0 commit comments

Comments
 (0)