Skip to content

Commit d3e998c

Browse files
feat: add coverage for add_extra_session_options (#187)
* test: add coverage for add_extra_session_options Co-authored-by: n24q02m <135627235+n24q02m@users.noreply.github.com> * style: format tests with ruff Co-authored-by: n24q02m <135627235+n24q02m@users.noreply.github.com> --------- Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
1 parent 3731db9 commit d3e998c

1 file changed

Lines changed: 19 additions & 0 deletions

File tree

tests/test_onnx_model_loading.py

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -218,3 +218,22 @@ def test_load_cuda_warning(model: ConcreteOnnxModel, mock_ort):
218218

219219
with pytest.warns(RuntimeWarning, match="Attempt to set CUDAExecutionProvider failed"):
220220
model._load_onnx_model(Path("dummy"), "model.onnx", threads=None, cuda=True)
221+
222+
223+
def test_add_extra_session_options():
224+
session_options = MagicMock()
225+
226+
# Test valid options
227+
ConcreteOnnxModel.add_extra_session_options(session_options, {"enable_cpu_mem_arena": False})
228+
assert session_options.enable_cpu_mem_arena is False
229+
230+
# Test valid options (True)
231+
ConcreteOnnxModel.add_extra_session_options(session_options, {"enable_cpu_mem_arena": True})
232+
assert session_options.enable_cpu_mem_arena is True
233+
234+
# Test invalid option
235+
with pytest.raises(
236+
ValueError,
237+
match="invalid_option is unknown or not exposed \\(exposed options: \\('enable_cpu_mem_arena',\\)\\)",
238+
):
239+
ConcreteOnnxModel.add_extra_session_options(session_options, {"invalid_option": True})

0 commit comments

Comments
 (0)