New files not part of dfetch diff generated patch

Fixes #886

Author: spoorcc

dfetch/project/svn.py

@@ -17,7 +17,7 @@
     in_directory,
     safe_rm,
 )
-from dfetch.vcs.patch import filter_patch
+from dfetch.vcs.patch import combine_patches, filter_patch, unified_diff_new_file
 
 logger = get_logger(__name__)
 
@@ -379,8 +379,40 @@ def get_diff(
         with in_directory(self.local_path):
             patch_text = run_on_cmdline(logger, cmd).stdout
 
-        return filter_patch(patch_text, ignore)
+        filtered = filter_patch(patch_text, ignore)
+
+        if new_revision:
+            return filtered
+
+        patches: list[bytes] = [filtered.encode("utf-8")] if filtered else []
+        with in_directory(self.local_path):
+            for file_path in self._untracked_not_ignored("."):
+                diff = unified_diff_new_file(pathlib.Path(file_path))
+                if diff:
+                    patch = [f"Index: {file_path}", "=" * 67] + diff
+
+                    print(patch)
+                    patches.append("\n".join(patch).encode("utf-8"))
+
+        return combine_patches(patches)
 
     def get_default_branch(self) -> str:
         """Get the default branch of this repository."""
         return self.DEFAULT_BRANCH
+
+    @staticmethod
+    def _untracked_not_ignored(path: str) -> list[str]:
+        result = (
+            run_on_cmdline(
+                logger,
+                ["svn", "status", "--no-ignore", path],
+            )
+            .stdout.decode()
+            .splitlines()
+        )
+
+        files = []
+        for line in result:
+            if line.startswith("?"):
+                files.append(line[1:].strip())
+        return files

dfetch/vcs/patch.py

@@ -1,6 +1,8 @@
 """Various patch utilities for VCS systems."""
 
+import difflib
 from collections.abc import Sequence
+from pathlib import Path
 
 import patch_ng
 
@@ -35,3 +37,28 @@ def dump_patch(patch_set: patch_ng.PatchSet) -> str:
             for line in h.text:
                 patch_lines.append(line.rstrip(b"\n").decode("utf-8"))
     return "\n".join(patch_lines)
+
+
+def unified_diff_new_file(path: Path) -> list[str]:
+    """Create a unified diff for a new file."""
+    with path.open("r", encoding="utf-8", errors="replace") as new_file:
+        lines = new_file.readlines()
+
+    return list(
+        difflib.unified_diff(
+            [], lines, fromfile="/dev/null", tofile=str(path), lineterm=""
+        )
+    )
+
+
+def combine_patches(patches: Sequence[bytes]) -> str:
+    """Combine multiple patches into a single patch."""
+    if not patches:
+        return ""
+
+    final_patchset = patch_ng.PatchSet()
+    for patch in patches:
+        for patch_obj in patch_ng.fromstring(patch) or []:
+            final_patchset.items += [patch_obj]
+
+    return dump_patch(final_patchset)

features/diff-in-svn.feature

@@ -21,7 +21,7 @@ Feature: Diff in svn
             """
             An important sentence for the README!
             """
-        When I run "dfetch -v diff SomeProject" in MySvnProject
+        When I run "dfetch diff SomeProject" in MySvnProject
         Then the patch file 'MySvnProject/SomeProject.patch' is generated
             """
             Index: README.md
@@ -39,6 +39,7 @@ Feature: Diff in svn
             """
             A completely new untracked file.
             """
+        And "SomeProject/NEW_UNCOMMITTED_FILE.md" in MySvnProject is created
         And "SomeProject/IGNORE_ME.tmp" in MySvnProject is created
         When I run "dfetch diff SomeProject" in MySvnProject
         Then the patch file 'MySvnProject/SomeProject.patch' is generated
@@ -49,6 +50,12 @@ Feature: Diff in svn
             +++ NEWFILE.md
             @@ -0,0 +1,1 @@
             +A completely new untracked file.
+            Index: NEW_UNCOMMITTED_FILE.md
+            ===================================================================
+            --- /dev/null
+            +++ NEW_UNCOMMITTED_FILE.md
+            @@ -0,0 +1,1 @@
+            +Some content
             """
 
     Scenario: No change is present
@@ -78,7 +85,7 @@ Feature: Diff in svn
 
     Scenario: Metadata is not part of diff
         Given the metadata file ".dfetch_data.yaml" of "MySvnProject/SomeProject" is changed
-        When I run "dfetch diff SomeProject"
+        When I run "dfetch diff SomeProject" in MySvnProject
         Then the output shows
         """
         Dfetch (0.10.0)