Skip to content

Commit cf38f47

Browse files
committed
Fix unit test failures
1 parent 1abf0e3 commit cf38f47

File tree

1 file changed

+7
-9
lines changed

1 file changed

+7
-9
lines changed

test/unit/test_model_server.py

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -205,32 +205,30 @@ def test_create_torchserve_config_file(write_file, generate_ts_config_props):
205205
@patch("sagemaker_inference.utils.read_file", return_value=DEFAULT_CONFIGURATION)
206206
@patch("sagemaker_inference.environment.Environment")
207207
def test_generate_ts_config_properties(env, read_file):
208-
model_sever_timeout = "torchserve_timeout"
209-
torchserve_workers = "torchserve_workers"
208+
model_server_timeout = "torchserve_timeout"
209+
model_server_workers = "torchserve_workers"
210210
http_port = "http_port"
211211

212-
env.return_value.model_sever_timeout = torchserve_timeout
213-
env.return_value.torchserve_workers = torchserve_workers
212+
env.return_value.model_server_timeout = model_server_timeout
213+
env.return_value.model_sever_workerse = model_server_workers
214214
env.return_value.inference_http_port = http_port
215215

216216
ts_config_properties = torchserve._generate_ts_config_properties()
217217

218218
inference_address = "inference_address=http://0.0.0.0:{}\n".format(http_port)
219-
server_timeout = "default_response_timeout={}\n".format(model_sever_timeout)
220-
workers = "default_workers_per_model={}\n".format(torchserve_workers)
219+
server_timeout = "default_response_timeout={}\n".format(model_server_timeout)
221220

222221
read_file.assert_called_once_with(torchserve.DEFAULT_TS_CONFIG_FILE)
223222

224223
assert ts_config_properties.startswith(DEFAULT_CONFIGURATION)
225224
assert inference_address in ts_config_properties
226225
assert server_timeout in ts_config_properties
227-
assert workers in ts_config_properties
228226

229227

230228
@patch("sagemaker_inference.utils.read_file", return_value=DEFAULT_CONFIGURATION)
231229
@patch("sagemaker_inference.environment.Environment")
232230
def test_generate_ts_config_properties_default_workers(env, read_file):
233-
env.return_value.torchserve_workers = None
231+
env.return_value.model_server_workers = None
234232

235233
ts_config_properties = torchserve._generate_ts_config_properties()
236234

@@ -245,7 +243,7 @@ def test_generate_ts_config_properties_default_workers(env, read_file):
245243
@patch("sagemaker_inference.utils.read_file", return_value=DEFAULT_CONFIGURATION)
246244
@patch("sagemaker_inference.environment.Environment")
247245
def test_generate_ts_config_properties_multi_model(env, read_file):
248-
env.return_value.torchserve_workers = None
246+
env.return_value.model_server_workers = None
249247

250248
torchserve.ENABLE_MULTI_MODEL = True
251249
ts_config_properties = torchserve._generate_ts_config_properties()

0 commit comments

Comments
 (0)