Use path more consistent

Author: spoorcc

dfetch/project/superproject.py

@@ -41,12 +41,12 @@ def __init__(self) -> None:
 
         logger.debug(f"Using manifest {manifest_path}")
         self._manifest = parse(manifest_path)
-        self._root_directory = str(
-            resolve_absolute_path(os.path.dirname(self._manifest.path))
+        self._root_directory = resolve_absolute_path(
+            os.path.dirname(self._manifest.path)
         )
 
     @property
-    def root_directory(self) -> str:
+    def root_directory(self) -> pathlib.Path:
         """Return the directory that contains the manifest file."""
         return self._root_directory
 
@@ -67,11 +67,10 @@ def get_sub_project(self, project: ProjectEntry) -> SubProject | None:
     def ignored_files(self, path: str) -> Sequence[str]:
         """Return a list of files that can be ignored in a given path."""
         resolved_path = resolve_absolute_path(path)
-        root_path = pathlib.Path(self.root_directory)
 
-        if not resolved_path.is_relative_to(root_path):
+        if not resolved_path.is_relative_to(self.root_directory):
             raise RuntimeError(
-                f"{resolved_path} not in superproject {self.root_directory}! "
+                f"{resolved_path} not in superproject {self.root_directory}!"
             )
 
         if GitLocalRepo(self.root_directory).is_git():

dfetch/util/util.py

@@ -63,14 +63,14 @@ def safe_rmtree(path: str) -> None:
 
 
 @contextmanager
-def in_directory(path: str) -> Generator[str, None, None]:
+def in_directory(path: Union[str, Path]) -> Generator[str, None, None]:
     """Work temporarily in a given directory."""
     pwd = os.getcwd()
     if not os.path.isdir(path):
         path = os.path.dirname(path)
     os.chdir(path)
     try:
-        yield path
+        yield str(path)
     finally:
         os.chdir(pwd)
 

dfetch/vcs/git.py

@@ -7,7 +7,7 @@
 import tempfile
 from collections.abc import Generator, Sequence
 from pathlib import Path, PurePath
-from typing import NamedTuple, Optional
+from typing import NamedTuple, Optional, Union
 
 from dfetch.log import get_logger
 from dfetch.util.cmdline import SubprocessCommandError, run_on_cmdline
@@ -234,9 +234,9 @@ class GitLocalRepo:
 
     METADATA_DIR = ".git"
 
-    def __init__(self, path: str = ".") -> None:
+    def __init__(self, path: Union[str, Path] = ".") -> None:
         """Create a local git repo."""
-        self._path = path
+        self._path = str(path)
 
     def is_git(self) -> bool:
         """Check if is git."""

dfetch/vcs/svn.py

@@ -4,7 +4,7 @@
 import pathlib
 import re
 from collections.abc import Sequence
-from typing import NamedTuple, Optional
+from typing import NamedTuple, Optional, Union
 
 from dfetch.log import get_logger
 from dfetch.util.cmdline import SubprocessCommandError, run_on_cmdline
@@ -76,10 +76,10 @@ class SvnRepo:
 
     def __init__(
         self,
-        path: str = ".",
+        path: Union[str, pathlib.Path] = ".",
     ) -> None:
         """Create a svn repo."""
-        self._path = path
+        self._path = str(path)
 
     def is_svn(self) -> bool:
         """Check if is SVN."""

tests/test_check.py

@@ -4,6 +4,7 @@
 # flake8: noqa
 
 import argparse
+from pathlib import Path
 from unittest.mock import Mock, patch
 
 import pytest
@@ -30,7 +31,7 @@ def test_check(name, projects):
 
     fake_superproject = Mock()
     fake_superproject.manifest = mock_manifest(projects)
-    fake_superproject.root_directory = "/tmp"
+    fake_superproject.root_directory = Path("/tmp")
 
     with patch("dfetch.commands.check.SuperProject", return_value=fake_superproject):
         with patch(

tests/test_report.py

@@ -4,6 +4,7 @@
 # flake8: noqa
 
 import argparse
+from pathlib import Path
 from unittest.mock import Mock, patch
 
 import pytest
@@ -30,7 +31,7 @@ def test_report(name, projects):
 
     fake_superproject = Mock()
     fake_superproject.manifest = mock_manifest(projects)
-    fake_superproject.root_directory = "/tmp"
+    fake_superproject.root_directory = Path("/tmp")
 
     with patch("dfetch.commands.report.SuperProject", return_value=fake_superproject):
         with patch("dfetch.log.DLogger.print_info_line") as mocked_print_info_line:

tests/test_update.py

@@ -4,6 +4,7 @@
 # flake8: noqa
 
 import argparse
+from pathlib import Path
 from unittest.mock import Mock, patch
 
 import pytest
@@ -30,7 +31,7 @@ def test_update(name, projects):
 
     fake_superproject = Mock()
     fake_superproject.manifest = mock_manifest(projects)
-    fake_superproject.root_directory = "/tmp"
+    fake_superproject.root_directory = Path("/tmp")
 
     with patch("dfetch.commands.update.SuperProject", return_value=fake_superproject):
         with patch(
@@ -53,7 +54,7 @@ def test_forced_update():
 
     fake_superproject = Mock()
     fake_superproject.manifest = mock_manifest([{"name": "some_project"}])
-    fake_superproject.root_directory = "/tmp"
+    fake_superproject.root_directory = Path("/tmp")
     fake_superproject.ignored_files.return_value = []
 
     with patch("dfetch.commands.update.SuperProject", return_value=fake_superproject):