Skip to content

Commit d7a2bf6

Browse files
author
Chuyang Deng
committed
Fix unit test.
1 parent 1ae3b34 commit d7a2bf6

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

src/sagemaker/predictor.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -138,8 +138,8 @@ def delete_model(self):
138138
def _get_model_names(self):
139139
endpoint_desc = self.sagemaker_session.sagemaker_client.describe_endpoint(EndpointName=self.endpoint)
140140
endpoint_config_name = endpoint_desc['EndpointConfigName']
141-
endpoint_config = self.sagemaker_session.sagemaker_client.describe_endpoint_config(EndpointConfigName=
142-
endpoint_config_name)
141+
endpoint_config = self.sagemaker_session.sagemaker_client.describe_endpoint_config(
142+
EndpointConfigName=endpoint_config_name)
143143
production_variants = endpoint_config['ProductionVariants']
144144
return map(lambda d: d['ModelName'], production_variants)
145145

tests/unit/test_predictor.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -470,14 +470,17 @@ def test_delete_endpoint_only():
470470

471471
def test_delete_model():
472472
endpoint_desc = {
473+
'EndpointConfigName': 'my-endpoint-config'
474+
}
475+
endpoint_config_desc = {
473476
'ProductionVariants': [{
474-
'VariantName': 'my-model'
477+
'ModelName': 'my-model'
475478
}]
476479
}
477480
sagemaker_session = empty_sagemaker_session()
478481
sagemaker_session.sagemaker_client.describe_endpoint = Mock(return_value=endpoint_desc)
482+
sagemaker_session.sagemaker_client.describe_endpoint_config = Mock(return_value=endpoint_config_desc)
479483
predictor = RealTimePredictor(ENDPOINT, sagemaker_session=sagemaker_session)
480484

481485
predictor.delete_model()
482486
sagemaker_session.delete_model.assert_called_with('my-model')
483-

0 commit comments

Comments
 (0)