Skip to content

Commit 4cdeb8d

Browse files
martinRenouakrishna1995
authored andcommitted
Fix tests
1 parent e6dd02d commit 4cdeb8d

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
@@ -3670,7 +3670,7 @@ def test_create_endpoint_config_with_tags_list(sagemaker_session):
36703670

36713671

36723672
def test_create_endpoint_config_with_tags_dict(sagemaker_session):
3673-
tags = {"TagtestKey", "TagtestValue"}
3673+
tags = {"TagtestKey": "TagtestValue"}
36743674
call_tags = [{"Key": "TagtestKey", "Value": "TagtestValue"}]
36753675

36763676
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)