Skip to content

Commit 6276eff

Browse files
martinRenouakrishna1995
authored andcommitted
Fix broken tags
1 parent cf9ce20 commit 6276eff

File tree

6 files changed

+7
-7
lines changed

6 files changed

+7
-7
lines changed

tests/unit/sagemaker/model/test_model.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -361,7 +361,7 @@ def test_create_sagemaker_model_tags(prepare_container_def, sagemaker_session):
361361

362362
model = Model(MODEL_IMAGE, MODEL_DATA, name=MODEL_NAME, sagemaker_session=sagemaker_session)
363363

364-
tags = {"Key": "foo", "Value": "bar"}
364+
tags = [{"Key": "foo", "Value": "bar"}]
365365
model._create_sagemaker_model(INSTANCE_TYPE, tags=tags)
366366

367367
sagemaker_session.create_model.assert_called_with(

tests/unit/sagemaker/model/test_model_package.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,7 @@ def test_create_sagemaker_model_include_tags(sagemaker_session):
201201
env_key = "env_key"
202202
env_value = "env_value"
203203
environment = {env_key: env_value}
204-
tags = {"Key": "foo", "Value": "bar"}
204+
tags = [{"Key": "foo", "Value": "bar"}]
205205

206206
model_package = ModelPackage(
207207
role="role",
@@ -314,7 +314,7 @@ def test_model_package_create_transformer_with_product_id(sagemaker_session):
314314

315315
@patch("sagemaker.model.ModelPackage.update_approval_status")
316316
def test_model_package_auto_approve_on_deploy(update_approval_status, sagemaker_session):
317-
tags = {"Key": "foo", "Value": "bar"}
317+
tags = [{"Key": "foo", "Value": "bar"}]
318318
model_package = ModelPackage(
319319
role="role",
320320
model_package_arn=MODEL_PACKAGE_VERSIONED_ARN,

tests/unit/sagemaker/tensorflow/test_estimator.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -324,7 +324,7 @@ def test_transformer_creation_with_optional_args(
324324
env = {"foo": "bar"}
325325
max_concurrent_transforms = 3
326326
max_payload = 100
327-
tags = {"Key": "foo", "Value": "bar"}
327+
tags = [{"Key": "foo", "Value": "bar"}]
328328
new_role = "role"
329329
vpc_config = {"Subnets": ["1234"], "SecurityGroupIds": ["5678"]}
330330
model_name = "model-name"

tests/unit/test_predictor.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -307,7 +307,7 @@ def test_update_endpoint_all_args(name_from_base, production_variant):
307307
new_instance_type = "ml.c4.xlarge"
308308
new_accelerator_type = "ml.eia1.medium"
309309
new_model_name = "new-model"
310-
new_tags = {"Key": "foo", "Value": "bar"}
310+
new_tags = [{"Key": "foo", "Value": "bar"}]
311311
new_kms_key = "new-key"
312312
new_data_capture_config_dict = {}
313313

tests/unit/test_predictor_async.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -404,7 +404,7 @@ def test_update_endpoint_all_args():
404404
new_instance_type = "ml.c4.xlarge"
405405
new_accelerator_type = "ml.eia1.medium"
406406
new_model_name = "new-model"
407-
new_tags = {"Key": "foo", "Value": "bar"}
407+
new_tags = [{"Key": "foo", "Value": "bar"}]
408408
new_kms_key = "new-key"
409409
new_data_capture_config_dict = {}
410410

tests/unit/test_transformer.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -249,7 +249,7 @@ def test_transformer_init_optional_params(sagemaker_session):
249249
accept = "text/csv"
250250
max_concurrent_transforms = 100
251251
max_payload = 100
252-
tags = {"Key": "foo", "Value": "bar"}
252+
tags = [{"Key": "foo", "Value": "bar"}]
253253
env = {"FOO": "BAR"}
254254

255255
transformer = Transformer(

0 commit comments

Comments
 (0)