Add tests cases for when uv subprocess fails

nathanjmcdougall · nathanjmcdougall · commit 9d40953e11e4 · 2025-03-04T22:55:32.000+13:00
diff --git a/tests/usethis/_core/test_core_tool.py b/tests/usethis/_core/test_core_tool.py
@@ -198,11 +198,7 @@ def test_no_pyproject_toml(
             # Set python version
             (tmp_path / ".python-version").write_text(get_python_version())
 
-            with (
-                change_cwd(tmp_path),
-                PyprojectTOMLManager(),
-                usethis_config.set(subprocess_verbose=True),
-            ):
+            with change_cwd(tmp_path), PyprojectTOMLManager():
                 # Act
                 use_coverage()
 
diff --git a/tests/usethis/_integrations/uv/test_deps.py b/tests/usethis/_integrations/uv/test_deps.py
@@ -2,6 +2,10 @@
 
 import pytest
 
+import usethis
+import usethis._integrations
+import usethis._integrations.uv
+import usethis._integrations.uv.deps
 from usethis._config import usethis_config
 from usethis._integrations.pyproject_toml.core import (
     get_pyproject_value,
@@ -18,6 +22,7 @@
     register_default_group,
     remove_deps_from_group,
 )
+from usethis._integrations.uv.errors import UVDepGroupError, UVSubprocessFailedError
 from usethis._test import change_cwd
 
 
@@ -272,6 +277,30 @@ def test_dev_group_not_registered(self, uv_init_dir: Path):
             # Tool section shouldn't even exist in pyproject.toml
             assert "tool" not in (uv_init_dir / "pyproject.toml").read_text()
 
+    def test_uv_subprocess_error(
+        self,
+        uv_init_dir: Path,
+        monkeypatch: pytest.MonkeyPatch,
+        capfd: pytest.CaptureFixture[str],
+    ):
+        def mock_call_uv_subprocess(*_, **__):
+            raise UVSubprocessFailedError
+
+        monkeypatch.setattr(
+            usethis._integrations.uv.deps, "call_uv_subprocess", mock_call_uv_subprocess
+        )
+
+        with (
+            change_cwd(uv_init_dir),
+            PyprojectTOMLManager(),
+            pytest.raises(
+                UVDepGroupError,
+                match="Failed to add 'pytest' to the 'test' dependency group",
+            ),
+        ):
+            # Act
+            add_deps_to_group([Dependency(name="pytest")], "test")
+
 
 class TestRemoveDepsFromGroup:
     @pytest.mark.usefixtures("_vary_network_conn")
@@ -395,6 +424,35 @@ def test_group_not_in_dependency_groups(
             assert not err
             assert not out
 
+    def test_uv_subprocess_error(
+        self,
+        uv_init_dir: Path,
+        monkeypatch: pytest.MonkeyPatch,
+        capfd: pytest.CaptureFixture[str],
+    ):
+        with (
+            change_cwd(uv_init_dir),
+            PyprojectTOMLManager(),
+        ):
+            # Arrange
+            add_deps_to_group([Dependency(name="pytest")], "test")
+
+            def mock_call_uv_subprocess(*_, **__):
+                raise UVSubprocessFailedError
+
+            monkeypatch.setattr(
+                usethis._integrations.uv.deps,
+                "call_uv_subprocess",
+                mock_call_uv_subprocess,
+            )
+
+            # Act
+            with pytest.raises(
+                UVDepGroupError,
+                match="Failed to remove 'pytest' from the 'test' dependency group",
+            ):
+                remove_deps_from_group([Dependency(name="pytest")], "test")
+
 
 class TestIsDepInAnyGroup:
     def test_no_group(self, uv_init_dir: Path):
