Use merge-base (three dot diff) instead of direct diff for git-diff-b…

…ase (infection#1653)
bdsl · Jan 24, 2022 · 6ffb41a · 6ffb41a
1 parent 4e21783
commit 6ffb41a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/Logger/GitHub/GitDiffFileProvider.php b/src/Logger/GitHub/GitDiffFileProvider.php
@@ -56,7 +56,7 @@ public function provide(string $gitDiffFilter, string $gitDiffBase): string
     {
         $filter = $this->shellCommandLineExecutor->execute(sprintf(
             'git diff %s --diff-filter=%s --name-only | grep src/ | paste -s -d "," -',
-            escapeshellarg($gitDiffBase),
+            escapeshellarg($gitDiffBase . '...HEAD'),
             escapeshellarg($gitDiffFilter)
         ));
 

diff --git a/tests/phpunit/Logger/GitHub/GitDiffFileProviderTest.php b/tests/phpunit/Logger/GitHub/GitDiffFileProviderTest.php
@@ -58,10 +58,10 @@ public function test_it_throws_no_code_to_mutate_exception_when_diff_is_empty():
 
     public function test_it_executes_diff_and_returns_filter_as_a_string(): void
     {
-        $expectedCommandLine = 'git diff \'master\' --diff-filter=\'AM\' --name-only | grep src/ | paste -s -d "," -';
+        $expectedCommandLine = 'git diff \'master...HEAD\' --diff-filter=\'AM\' --name-only | grep src/ | paste -s -d "," -';
 
         if (PHP_OS_FAMILY === 'Windows') {
-            $expectedCommandLine = 'git diff "master" --diff-filter="AM" --name-only | grep src/ | paste -s -d "," -';
+            $expectedCommandLine = 'git diff "master...HEAD" --diff-filter="AM" --name-only | grep src/ | paste -s -d "," -';
         }
 
         $shellCommandLineExecutor = $this->createMock(ShellCommandLineExecutor::class);