vcs: remove unused git code

Author: abn

src/poetry/core/vcs/git.py

@@ -244,67 +244,6 @@ def version(self) -> tuple[int, int, int]:
             return (0, 0, 0)
         return int(version.group(1)), int(version.group(2)), int(version.group(3))
 
-    def clone(self, repository: str, dest: Path) -> str:
-        self._check_parameter(repository)
-        cmd = [
-            "clone",
-            "--filter=blob:none",
-            "--recurse-submodules",
-            "--",
-            repository,
-            str(dest),
-        ]
-        # Blobless clones introduced in Git 2.17
-        if self.version < (2, 17):
-            cmd.remove("--filter=blob:none")
-        return self.run(*cmd)
-
-    def checkout(self, rev: str, folder: Path | None = None) -> str:
-        args = []
-        if folder is None and self._work_dir:
-            folder = self._work_dir
-
-        if folder:
-            args += [
-                "--git-dir",
-                (folder / ".git").as_posix(),
-                "--work-tree",
-                folder.as_posix(),
-            ]
-
-        self._check_parameter(rev)
-
-        args += ["checkout", "--recurse-submodules", rev]
-
-        return self.run(*args)
-
-    def rev_parse(self, rev: str, folder: Path | None = None) -> str:
-        args = []
-        if folder is None and self._work_dir:
-            folder = self._work_dir
-
-        self._check_parameter(rev)
-
-        # We need "^0" (an alternative to "^{commit}") to ensure that the
-        # commit SHA of the commit the tag points to is returned, even in
-        # the case of annotated tags.
-        #
-        # We deliberately avoid the "^{commit}" syntax itself as on some
-        # platforms (cygwin/msys to be specific), the braces are interpreted
-        # as special characters and would require escaping, while on others
-        # they should not be escaped.
-        args += ["rev-parse", rev + "^0"]
-
-        return self.run(*args, folder=folder)
-
-    def get_current_branch(self, folder: Path | None = None) -> str:
-        if folder is None and self._work_dir:
-            folder = self._work_dir
-
-        output = self.run("symbolic-ref", "--short", "HEAD", folder=folder)
-
-        return output.strip()
-
     def get_ignored_files(self, folder: Path | None = None) -> list[str]:
         args = []
         if folder is None and self._work_dir:
@@ -323,23 +262,6 @@ def get_ignored_files(self, folder: Path | None = None) -> list[str]:
 
         return output.strip().split("\n")
 
-    def remote_urls(self, folder: Path | None = None) -> dict[str, str]:
-        output = self.run(
-            "config", "--get-regexp", r"remote\..*\.url", folder=folder
-        ).strip()
-
-        urls = {}
-        for url in output.splitlines():
-            name, url = url.split(" ", 1)
-            urls[name.strip()] = url.strip()
-
-        return urls
-
-    def remote_url(self, folder: Path | None = None) -> str:
-        urls = self.remote_urls(folder=folder)
-
-        return urls.get("remote.origin.url", urls[next(iter(urls.keys()))])
-
     def run(self, *args: Any, **kwargs: Any) -> str:
         folder = kwargs.pop("folder", None)
         if folder:
@@ -358,10 +280,3 @@ def run(self, *args: Any, **kwargs: Any) -> str:
             .decode()
             .strip()
         )
-
-    def _check_parameter(self, parameter: str) -> None:
-        """
-        Checks a git parameter to avoid unwanted code execution.
-        """
-        if parameter.strip().startswith("-"):
-            raise GitError(f"Invalid Git parameter: {parameter}")

tests/vcs/test_vcs.py

@@ -12,7 +12,6 @@
 from poetry.core.utils._compat import WINDOWS
 from poetry.core.vcs import get_vcs
 from poetry.core.vcs.git import Git
-from poetry.core.vcs.git import GitError
 from poetry.core.vcs.git import GitUrl
 from poetry.core.vcs.git import ParsedUrl
 from poetry.core.vcs.git import _reset_executable
@@ -440,21 +439,6 @@ def test_parse_url_should_fail() -> None:
         ParsedUrl.parse(url)
 
 
-def test_git_clone_raises_error_on_invalid_repository() -> None:
-    with pytest.raises(GitError):
-        Git().clone("-u./payload", Path("foo"))
-
-
-def test_git_checkout_raises_error_on_invalid_repository() -> None:
-    with pytest.raises(GitError):
-        Git().checkout("-u./payload")
-
-
-def test_git_rev_parse_raises_error_on_invalid_repository() -> None:
-    with pytest.raises(GitError):
-        Git().rev_parse("-u./payload")
-
-
 @pytest.mark.skipif(
     not WINDOWS,
     reason=(