fix(repo): unquote git check-ignore paths in Repo.ignored

`Repo.ignored` runs `git check-ignore -- <paths>` and recovered the
matched paths from stdout with
`.replace("\\\\", "\\").replace('"', "").split("\n")`. git C-quotes a
path that contains a non-ASCII byte (with the default `core.quotePath`),
a double quote, a backslash or a control character, and that replacement
does not reverse the quoting. With `*.env` in `.gitignore`:

- `données.env` came back as `donn\303\251es.env`
- `q"uote.env` came back as `q\uote.env`
- `tab<TAB>.env` came back as `tab\t.env`
- a name containing a line feed came back with a literal `\n`

A caller that compares the result with the paths it passed in, for
example to leave ignored files out of a commit, an upload or an archive,
therefore treats those ignored files as not ignored.

Read stdout as bytes and unquote each quoted line with
`git.diff._unquote_path`, which already reverses git's C-quoting for
patch headers. Splitting at line feeds stays safe because git always
quotes a path that contains one. `git check-ignore -z` would avoid the
quoting, but git accepts `-z` only together with `--stdin`. Paths git
does not quote are returned unchanged.

Adds `test_ignored_items_reported_verbatim` to `test/test_repo.py`; it
fails before this change and passes after. `ruff check`,
`ruff format --check` and `mypy` pass. The full pytest suite run locally
fails the same 83 tests with and without this change; those failures
come from fixture commits missing in the local clone, not from this
change.

Author: Keerthana-64

git/repo/base.py

@@ -26,6 +26,7 @@
 from git.compat import defenc, safe_decode
 from git.config import GitConfigParser
 from git.db import GitCmdObjectDB
+from git.diff import _unquote_path
 from git.exc import (
     GitCommandError,
     InvalidGitRepositoryError,
@@ -1158,7 +1159,7 @@ def ignored(self, *paths: PathLike) -> List[str]:
             Subset of those paths which are ignored
         """
         try:
-            proc: str = self.git.check_ignore("--", *paths)
+            proc: bytes = self.git.check_ignore("--", *paths, stdout_as_string=False)
         except GitCommandError as err:
             if err.status == 1:
                 # If return code is 1, this means none of the items in *paths are
@@ -1168,7 +1169,12 @@ def ignored(self, *paths: PathLike) -> List[str]:
                 # Raise the exception on all other return codes.
                 raise
 
-        return proc.replace("\\\\", "\\").replace('"', "").split("\n")
+        # Git C-quotes a path containing special characters, including a line feed,
+        # so each path is on its own line.
+        return [
+            safe_decode(_unquote_path(path[1:-1]) if path[:1] == path[-1:] == b'"' else path)
+            for path in proc.split(b"\n")
+        ]
 
     @property
     def active_branch(self) -> Head:

test/test_repo.py

@@ -1657,6 +1657,17 @@ def test_ignored_items_reported(self):
                 ["included_file.txt", "ignored_file.txt", "included_dir/file.txt", "ignored_dir/file.txt"]
             ) == ["ignored_file.txt", "ignored_dir/file.txt"]
 
+    def test_ignored_items_reported_verbatim(self):
+        with tempfile.TemporaryDirectory() as tdir:
+            temp_repo = Repo.init(pathlib.Path(tdir) / "repo")
+
+            with open(pathlib.Path(temp_repo.working_tree_dir) / ".gitignore", "w") as file:
+                file.write("*.env\n")
+
+            # Git C-quotes each of these in its default output format.
+            paths = ["donn\u00e9es.env", 'q"uote.env', "tab\t.env", "nl\nx.env"]
+            assert temp_repo.ignored(["keep.txt", *paths]) == paths
+
     @requires_symlinks
     def test_ignored_raises_error_w_symlink(self):
         with tempfile.TemporaryDirectory() as tdir: