Implement Dependency.to_requirements_string (#861)

nathanjmcdougall · web-flow · commit daff557eae27 · 2025-07-16T19:47:46.000+12:00
* Implement `Dependency.to_requirements_string`

* Use `Dependency.to_requirements_string()` in `Dependency.__str__`
diff --git a/src/usethis/_integrations/uv/deps.py b/src/usethis/_integrations/uv/deps.py
@@ -17,12 +17,16 @@ class Dependency(BaseModel):
     extras: frozenset[str] = frozenset()
 
     def __str__(self) -> str:
-        extras = sorted(self.extras or set())
-        return self.name + "".join(f"[{extra}]" for extra in extras)
+        return self.to_requirements_string()
 
     def __hash__(self) -> int:
         return hash((self.__class__.__name__, self.name, self.extras))
 
+    def to_requirements_string(self) -> str:
+        """Convert the dependency to a requirements string."""
+        extras_str = f"[{','.join(sorted(self.extras))}]" if self.extras else ""
+        return f"{self.name}{extras_str}"
+
 
 def get_project_deps() -> list[Dependency]:
     """Get all project dependencies.
diff --git a/tests/usethis/_integrations/uv/test_deps.py b/tests/usethis/_integrations/uv/test_deps.py
@@ -28,6 +28,21 @@
 from usethis._test import change_cwd
 
 
+class TestDependency:
+    class TestToRequirementsString:
+        def test_no_extras(self):
+            dep = Dependency(name="requests")
+            assert dep.to_requirements_string() == "requests"
+
+        def test_single_extra(self):
+            dep = Dependency(name="requests", extras=frozenset({"security"}))
+            assert dep.to_requirements_string() == "requests[security]"
+
+        def test_multiple_extras(self):
+            dep = Dependency(name="requests", extras=frozenset({"security", "socks"}))
+            assert dep.to_requirements_string() == "requests[security,socks]"
+
+
 class TestGetProjectDeps:
     def test_no_pyproject(self, tmp_path: Path):
         # Arrange - No pyproject.toml file exists
