Skip to content

Commit 921f7d4

Browse files
author
Jonathan Makunga
committed
Fix unit tests
1 parent da2c0a2 commit 921f7d4

File tree

1 file changed

+14
-2
lines changed

1 file changed

+14
-2
lines changed

tests/unit/sagemaker/serve/model_server/tgi/test_tgi_prepare.py

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,11 @@ def test_create_dir_structure_invalid_path(self, mock_path):
6666
self.assertEquals("model_dir is not a valid directory", str(context.exception))
6767

6868
@patch("sagemaker.serve.model_server.tgi.prepare.S3Downloader")
69-
def test_prepare_tgi_js_resources_for_jumpstart_uncompressed_str(self, mock_s3_downloader):
69+
@patch("builtins.open", read_data="data")
70+
@patch("json.load", return_value={})
71+
def test_prepare_tgi_js_resources_for_jumpstart_uncompressed_str(
72+
self, mock_json_load, mock_open, mock_s3_downloader
73+
):
7074
mock_code_dir = Mock()
7175
mock_s3_downloader_obj = Mock()
7276
mock_s3_downloader.return_value = mock_s3_downloader_obj
@@ -80,7 +84,11 @@ def test_prepare_tgi_js_resources_for_jumpstart_uncompressed_str(self, mock_s3_d
8084
)
8185

8286
@patch("sagemaker.serve.model_server.tgi.prepare.S3Downloader")
83-
def test_prepare_tgi_js_resources_for_jumpstart_invalid_model_data(self, mock_s3_downloader):
87+
@patch("builtins.open", read_data="data")
88+
@patch("json.load", return_value={})
89+
def test_prepare_tgi_js_resources_for_jumpstart_invalid_model_data(
90+
self, mock_json_load, mock_open, mock_s3_downloader
91+
):
8492
mock_code_dir = Mock()
8593
mock_s3_downloader_obj = Mock()
8694
mock_s3_downloader.return_value = mock_s3_downloader_obj
@@ -108,8 +116,12 @@ def test_prepare_tgi_js_resources_for_jumpstart_invalid_format(self):
108116
@patch("sagemaker.serve.model_server.tgi.prepare.S3Downloader")
109117
@patch("sagemaker.serve.model_server.tgi.prepare._tmpdir")
110118
@patch("sagemaker.serve.model_server.tgi.prepare._extract_js_resource")
119+
@patch("builtins.open", read_data="data")
120+
@patch("json.load", return_value={})
111121
def test_prepare_tgi_js_resources_for_jumpstart_compressed_str(
112122
self,
123+
mock_open,
124+
mock_json_load,
113125
mock_extract_js_resource,
114126
mock_tmpdir,
115127
mock_s3_downloader,

0 commit comments

Comments
 (0)