Skip to content

Commit

Permalink
fix test
Browse files Browse the repository at this point in the history
  • Loading branch information
pintaoz-aws committed Sep 13, 2024
1 parent 06dc458 commit 0110a52
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 7 deletions.
6 changes: 5 additions & 1 deletion integ/test_codegen.py
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,11 @@ def test_training_and_inference(self):
content_type="text/csv",
accept="application/csv",
)
assert invoke_result.body.payload_part

def deserialise(response):
return [res_part for res_part in response.body]

assert deserialise(invoke_result)

def test_intelligent_defaults(self):
os.environ["SAGEMAKER_CORE_ADMIN_CONFIG_OVERRIDE"] = (
Expand Down
9 changes: 3 additions & 6 deletions src/sagemaker_core/main/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -500,8 +500,7 @@ def _serialize_dict(value: Dict) -> dict:
"""
serialized_dict = {}
for k, v in value.items():
serialize_result = serialize(v)
if serialize_result is not None:
if (serialize_result := serialize(v)) is not None:
serialized_dict.update({k: serialize_result})
return serialized_dict

Expand All @@ -518,8 +517,7 @@ def _serialize_list(value: List) -> list:
"""
serialized_list = []
for v in value:
serialize_result = serialize(v)
if serialize_result is not None:
if (serialize_result := serialize(v)) is not None:
serialized_list.append(serialize_result)
return serialized_list

Expand All @@ -536,8 +534,7 @@ def _serialize_shape(value: Any) -> dict:
"""
serialized_dict = {}
for k, v in vars(value).items():
serialize_result = serialize(v)
if serialize_result is not None:
if (serialize_result := serialize(v)) is not None:
key = snake_to_pascal(k) if is_snake_case(k) else k
serialized_dict.update({key[0].upper() + key[1:]: serialize_result})
return serialized_dict

0 comments on commit 0110a52

Please sign in to comment.