From 16214e3a57856e685e000b526414926d29990f54 Mon Sep 17 00:00:00 2001
From: Patrick-Beuks <code@beuks.net>
Date: Mon, 27 Jan 2025 10:58:38 +0100
Subject: [PATCH] Fix incorrect file name on new or deleted empty files (#226)

* Fix incorrect file name on new or deleted empty files

* Add test for empty file diff
---
 src/Gitonomy/Git/Parser/DiffParser.php |  2 ++
 tests/Gitonomy/Git/Tests/DiffTest.php  | 22 ++++++++++++++++++++++
 2 files changed, 24 insertions(+)

diff --git a/src/Gitonomy/Git/Parser/DiffParser.php b/src/Gitonomy/Git/Parser/DiffParser.php
index ad22bea..eec47ca 100644
--- a/src/Gitonomy/Git/Parser/DiffParser.php
+++ b/src/Gitonomy/Git/Parser/DiffParser.php
@@ -38,6 +38,7 @@ protected function doParse()
                 $newMode = $this->consumeTo("\n");
                 $this->consumeNewLine();
                 $oldMode = null;
+                $oldName = '/dev/null';
             }
             if ($this->expects('old mode ')) {
                 $oldMode = $this->consumeTo("\n");
@@ -49,6 +50,7 @@ protected function doParse()
             if ($this->expects('deleted file mode ')) {
                 $oldMode = $this->consumeTo("\n");
                 $newMode = null;
+                $newName = '/dev/null';
                 $this->consumeNewLine();
             }
 
diff --git a/tests/Gitonomy/Git/Tests/DiffTest.php b/tests/Gitonomy/Git/Tests/DiffTest.php
index 2232345..c82061b 100644
--- a/tests/Gitonomy/Git/Tests/DiffTest.php
+++ b/tests/Gitonomy/Git/Tests/DiffTest.php
@@ -150,4 +150,26 @@ public function testWorksWithUmlauts($repository)
         $files = $repository->getCommit(self::FILE_WITH_UMLAUTS_COMMIT)->getDiff()->getFiles();
         $this->assertSame('file_with_umlauts_\303\244\303\266\303\274', $files[0]->getNewName());
     }
+
+    public function testEmptyNewFile()
+    {
+        $diff = Diff::parse("diff --git a/test b/test\nnew file mode 100644\nindex 0000000000000000000000000000000000000000..e69de29bb2d1d6434b8b29ae775ad8c2e48c5391\n");
+        $firstFile = $diff->getFiles()[0];
+
+        $this->assertTrue($firstFile->isCreation());
+        $this->assertFalse($firstFile->isDeletion());
+        $this->assertSame('test', $firstFile->getNewName());
+        $this->assertNull($firstFile->getOldName());
+    }
+
+    public function testEmptyOldFile()
+    {
+        $diff = Diff::parse("diff --git a/test b/test\ndeleted file mode 100644\nindex e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..0000000000000000000000000000000000000000\n");
+        $firstFile = $diff->getFiles()[0];
+
+        $this->assertFalse($firstFile->isCreation());
+        $this->assertTrue($firstFile->isDeletion());
+        $this->assertNull($firstFile->getNewName());
+        $this->assertSame('test', $firstFile->getOldName());
+    }
 }