Skip to content

Commit fbda61d

Browse files
martinRenouakrishna1995
authored andcommitted
Fix tests
1 parent ce4ac2d commit fbda61d

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

src/sagemaker/workflow/notebook_job_step.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -358,7 +358,7 @@ def _prepare_tags(self):
358358
This function converts the custom tags into training API required format and also
359359
attach the system tags.
360360
"""
361-
custom_tags = format_tags(self.tags)
361+
custom_tags = format_tags(self.tags) or []
362362
system_tags = [
363363
{"Key": "sagemaker:name", "Value": self.notebook_job_name},
364364
{"Key": "sagemaker:notebook-name", "Value": os.path.basename(self.input_notebook)},

tests/unit/test_session.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3608,7 +3608,7 @@ def test_create_endpoint_config_with_tags_list(sagemaker_session):
36083608

36093609

36103610
def test_create_endpoint_config_with_tags_dict(sagemaker_session):
3611-
tags = {"TagtestKey", "TagtestValue"}
3611+
tags = {"TagtestKey": "TagtestValue"}
36123612
call_tags = [{"Key": "TagtestKey", "Value": "TagtestValue"}]
36133613

36143614
sagemaker_session.create_endpoint_config(

tests/unit/test_transformer.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -573,7 +573,7 @@ def test_start_new(prepare_data_processing, load_config, sagemaker_session):
573573
strategy = "MultiRecord"
574574
max_concurrent_transforms = 100
575575
max_payload = 100
576-
tags = {"Key": "foo", "Value": "bar"}
576+
tags = [{"Key": "foo", "Value": "bar"}]
577577
env = {"FOO": "BAR"}
578578

579579
transformer = Transformer(

0 commit comments

Comments
 (0)