Skip to content

Commit cbc3bf9

Browse files
alessandrobolognayaythomas
authored andcommitted
fix: use correct key name for ChainedInvokeDetails deserialization
- Change key from snake_case "chained_invoke_details" to PascalCase "ChainedInvokeDetails" in Operation.from_dict() to match API response - Add missing assertions in test_operation_from_dict_complete to verify chained_invoke_details is properly deserialized Fixes #237
1 parent 6202b53 commit cbc3bf9

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

src/aws_durable_execution_sdk_python/lambda_service.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -749,7 +749,7 @@ def from_dict(cls, data: MutableMapping[str, Any]) -> Operation:
749749
callback_details = CallbackDetails.from_dict(callback_details_input)
750750

751751
chained_invoke_details = None
752-
if chained_invoke_details := data.get("chained_invoke_details"):
752+
if chained_invoke_details := data.get("ChainedInvokeDetails"):
753753
chained_invoke_details = ChainedInvokeDetails.from_dict(
754754
chained_invoke_details
755755
)

tests/lambda_service_test.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1481,6 +1481,8 @@ def test_operation_from_dict_complete():
14811481
assert operation.step_details.result == "step_result"
14821482
assert operation.wait_details.scheduled_end_timestamp == start_time
14831483
assert operation.callback_details.callback_id == "cb1"
1484+
assert operation.chained_invoke_details is not None
1485+
assert operation.chained_invoke_details.result == "invoke_result"
14841486

14851487

14861488
def test_operation_to_dict_with_subtype():

0 commit comments

Comments
 (0)