Skip to content

Commit f381475

Browse files
author
Chuyang Deng
committed
Fix unit test.
1 parent 6e09223 commit f381475

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
@@ -137,8 +137,8 @@ def delete_model(self):
137137
def _get_model_names(self):
138138
endpoint_desc = self.sagemaker_session.sagemaker_client.describe_endpoint(EndpointName=self.endpoint)
139139
endpoint_config_name = endpoint_desc['EndpointConfigName']
140-
endpoint_config = self.sagemaker_session.sagemaker_client.describe_endpoint_config(EndpointConfigName=
141-
endpoint_config_name)
140+
endpoint_config = self.sagemaker_session.sagemaker_client.describe_endpoint_config(
141+
EndpointConfigName=endpoint_config_name)
142142
production_variants = endpoint_config['ProductionVariants']
143143
return map(lambda d: d['ModelName'], production_variants)
144144

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)