Skip to content

Commit e4fb3bc

Browse files
committed
fix: unit tests
1 parent 41a2113 commit e4fb3bc

File tree

1 file changed

+18
-3
lines changed

1 file changed

+18
-3
lines changed

tests/unit/sagemaker/model/test_deploy.py

Lines changed: 18 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,12 @@ def test_deploy_accelerator_type(
159159
accelerator_type=ACCELERATOR_TYPE,
160160
)
161161

162-
create_sagemaker_model.assert_called_with(INSTANCE_TYPE, ACCELERATOR_TYPE, None, None)
162+
create_sagemaker_model.assert_called_with(
163+
instance_type=INSTANCE_TYPE,
164+
accelerator_type=ACCELERATOR_TYPE,
165+
tags=None,
166+
serverless_inference_config=None,
167+
)
163168
production_variant.assert_called_with(
164169
MODEL_NAME,
165170
INSTANCE_TYPE,
@@ -271,7 +276,12 @@ def test_deploy_tags(create_sagemaker_model, production_variant, name_from_base,
271276
tags = [{"Key": "ModelName", "Value": "TestModel"}]
272277
model.deploy(instance_type=INSTANCE_TYPE, initial_instance_count=INSTANCE_COUNT, tags=tags)
273278

274-
create_sagemaker_model.assert_called_with(INSTANCE_TYPE, None, tags, None)
279+
create_sagemaker_model.assert_called_with(
280+
instance_type=INSTANCE_TYPE,
281+
accelerator_type=None,
282+
tags=tags,
283+
serverless_inference_config=None,
284+
)
275285
sagemaker_session.endpoint_from_production_variants.assert_called_with(
276286
name=ENDPOINT_NAME,
277287
production_variants=[BASE_PRODUCTION_VARIANT],
@@ -463,7 +473,12 @@ def test_deploy_serverless_inference(production_variant, create_sagemaker_model,
463473
serverless_inference_config=serverless_inference_config,
464474
)
465475

466-
create_sagemaker_model.assert_called_with(None, None, None, serverless_inference_config)
476+
create_sagemaker_model.assert_called_with(
477+
instance_type=None,
478+
accelerator_type=None,
479+
tags=None,
480+
serverless_inference_config=serverless_inference_config,
481+
)
467482
production_variant.assert_called_with(
468483
MODEL_NAME,
469484
None,

0 commit comments

Comments
 (0)