From cc94c8ef5be33e03f98e7c93f566e4a934c116af Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alexis=20M=C3=A9taireau?= Date: Tue, 25 Mar 2025 12:34:37 +0100 Subject: [PATCH] fixup! Use a `Runtime` class to get information about container runtimes --- tests/test_container_utils.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/tests/test_container_utils.py b/tests/test_container_utils.py index aeec210..705acf9 100644 --- a/tests/test_container_utils.py +++ b/tests/test_container_utils.py @@ -32,7 +32,13 @@ def test_get_runtime_name_linux(mocker: MockerFixture, tmp_path: Path) -> None: def test_get_runtime_name_non_linux(mocker: MockerFixture, tmp_path: Path) -> None: mocker.patch("platform.system", return_value="Windows") mocker.patch("dangerzone.settings.get_config_dir", return_value=tmp_path) - assert Runtime().name == "docker" + mocker.patch( + "dangerzone.container_utils.shutil.which", return_value="/usr/bin/docker" + ) + mocker.patch("dangerzone.container_utils.os.path.exists", return_value=True) + runtime = Runtime() + assert runtime.name == "docker" + assert runtime.path == Path("/usr/bin/docker") mocker.patch("platform.system", return_value="Something else") assert Runtime().name == "docker"