Parser: 0 is valid range

Author: dg

src/Parser.php

@@ -77,9 +77,9 @@ private function parseFileDiff(Diff $diff, array $lines): void
             if (preg_match('/^@@\s+-(?P<start>\d+)(?:,\s*(?P<startrange>\d+))?\s+\+(?P<end>\d+)(?:,\s*(?P<endrange>\d+))?\s+@@/', $line, $match)) {
                 $chunk = new Chunk(
                     (int) $match['start'],
-                    isset($match['startrange']) ? max(1, (int) $match['startrange']) : 1,
+                    isset($match['startrange']) ? max(0, (int) $match['startrange']) : 1,
                     (int) $match['end'],
-                    isset($match['endrange']) ? max(1, (int) $match['endrange']) : 1
+                    isset($match['endrange']) ? max(0, (int) $match['endrange']) : 1
                 );
 
                 $chunks[]  = $chunk;

tests/ParserTest.php

@@ -193,6 +193,31 @@ public function testParseDiffForMulitpleFiles(): void
         $this->assertCount(1, $diff->getChunks());
     }
 
+    public function testParseWithRange(): void
+    {
+        $content = <<<'END'
+diff --git a/Test.txt b/Test.txt
+index abcdefg..abcdefh 100644
+--- a/Test.txt
++++ b/Test.txt
+@@ -49,0 +49,0 @@
+END;
+        $diffs = $this->parser->parse($content);
+        $this->assertContainsOnlyInstancesOf(Diff::class, $diffs);
+        $this->assertCount(1, $diffs);
+
+        $chunks = $diffs[0]->getChunks();
+
+        $this->assertContainsOnlyInstancesOf(Chunk::class, $chunks);
+        $this->assertCount(1, $chunks);
+
+        $chunk = $chunks[0];
+        $this->assertSame(49, $chunk->getStart());
+        $this->assertSame(49, $chunk->getEnd());
+        $this->assertSame(0, $chunk->getStartRange());
+        $this->assertSame(0, $chunk->getEndRange());
+    }
+
     /**
      * @psalm-param list<Diff> $expected
      */