Use content instead of line number for the fingerprint

Author: micheh

phpcs.xml.dist

@@ -15,4 +15,5 @@
 
     <file>src</file>
     <file>tests</file>
+    <exclude-pattern>tests/_files/*</exclude-pattern>
 </ruleset>

phpstan.neon.dist

@@ -4,5 +4,7 @@ parameters:
   paths:
     - src
     - tests
+  excludePaths:
+    - tests/_files/
   scanDirectories:
     - vendor/squizlabs/php_codesniffer

phpunit.xml.dist

@@ -7,6 +7,12 @@
          colors="true"
          forceCoversAnnotation="true"
 >
+    <php>
+        <const name="PHP_CODESNIFFER_IN_TESTS" value="true"/>
+        <const name="PHP_CODESNIFFER_CBF" value="false"/>
+        <const name="PHP_CODESNIFFER_VERBOSITY" value="0"/>
+    </php>
+
     <testsuites>
         <testsuite name="Project Test Suite">
             <directory>tests</directory>

src/Report/Gitlab.php

@@ -11,8 +11,11 @@
 
 use PHP_CodeSniffer\Files\File;
 use PHP_CodeSniffer\Reports\Report;
+use SplFileObject;
 
+use function is_string;
 use function md5;
+use function preg_replace;
 use function rtrim;
 use function str_replace;
 
@@ -26,15 +29,25 @@ class Gitlab implements Report
     public function generateFileReport($report, File $phpcsFile, $showSources = false, $width = 80)
     {
         $hasOutput = false;
+        $fingerprints = [];
 
         foreach ($report['messages'] as $line => $lineErrors) {
-            foreach ($lineErrors as $column => $colErrors) {
+            $lineContent = $this->getContentOfLine($phpcsFile->getFilename(), $line);
+
+            foreach ($lineErrors as $colErrors) {
                 foreach ($colErrors as $error) {
+                    $fingerprint = md5($report['filename'] . $lineContent . $error['source']);
+                    if (isset($fingerprints[$fingerprint])) {
+                        ++$fingerprints[$fingerprint];
+                    } else {
+                        $fingerprints[$fingerprint] = 1;
+                    }
+
                     $issue = [
                         'type' => 'issue',
                         'categories' => ['Style'],
                         'check_name' => $error['source'],
-                        'fingerprint' => md5($report['filename'] . $error['message'] . $line . $column),
+                        'fingerprint' => $fingerprint . '-' . $fingerprints[$fingerprint],
                         'severity' => $error['type'] === 'ERROR' ? 'major' : 'minor',
                         'description' => str_replace(["\n", "\r", "\t"], ['\n', '\r', '\t'], $error['message']),
                         'location' => [
@@ -68,4 +81,25 @@ public function generate(
     ) {
         echo '[' . rtrim($cachedData, ',') . ']' . PHP_EOL;
     }
+
+    /**
+     * @param string $filename
+     * @param int $line
+     * @return string
+     */
+    private function getContentOfLine($filename, $line)
+    {
+        $file = new SplFileObject($filename);
+
+        if (!$file->eof()) {
+            $file->seek($line - 1);
+            $contents = $file->current();
+
+            if (is_string($contents)) {
+                return (string) preg_replace('/\s+/', '', $contents);
+            }
+        }
+
+        return '';
+    }
 }

tests/Fixtures/MixedViolations.php

@@ -10,23 +10,39 @@
 namespace MichehTest\PhpCodeSniffer\Report;
 
 use Micheh\PhpCodeSniffer\Report\Gitlab;
-use MichehTest\PhpCodeSniffer\Fixtures\MixedViolations;
-use MichehTest\PhpCodeSniffer\Fixtures\ReportFixture;
-use MichehTest\PhpCodeSniffer\Fixtures\SingleViolation;
+use PHP_CodeSniffer\Config;
 use PHP_CodeSniffer\Files\File;
+use PHP_CodeSniffer\Files\LocalFile;
+use PHP_CodeSniffer\Reporter;
+use PHP_CodeSniffer\Ruleset;
+use PHP_CodeSniffer\Util\Tokens;
 use PHPUnit\Framework\TestCase;
 
+use function file_get_contents;
+
 class GitlabTest extends TestCase
 {
     /**
      * @var Gitlab
      */
     private $report;
 
+    /**
+     * @var Config
+     */
+    private $config;
+
 
     protected function setUp(): void
     {
         $this->report = new Gitlab();
+
+        $this->config = new Config();
+        $this->config->basepath = __DIR__ . '/../'; // @phpstan-ignore property.notFound
+        $this->config->standards = ['PSR12']; // @phpstan-ignore property.notFound
+
+        // Autoload PHP_CodeSniffer-specific token type constants
+        $tokens = new Tokens();
     }
 
     /**
@@ -48,27 +64,49 @@ public function testGenerateWithEmpty(): void
     }
 
     /**
-     * @return array<string, array<class-string<ReportFixture>>>
+     * @return array<string, array<string>>
      */
     public function violations(): array
     {
         return [
-            'single' => [SingleViolation::class],
-            'mixed' => [MixedViolations::class],
+            'single' => ['Single'],
+            'mixed' => ['Mixed'],
+            'multiple' => ['Multiple'],
         ];
     }
 
     /**
-     * @param class-string<ReportFixture> $class
      * @covers \Micheh\PhpCodeSniffer\Report\Gitlab::generateFileReport
      * @dataProvider violations
      */
-    public function testGenerateFileReport(string $class): void
+    public function testGenerateFileReport(string $fileName): void
     {
-        $fixture = new $class();
-        self::assertInstanceOf(ReportFixture::class, $fixture);
+        $phpPath = __DIR__ . '/../_files/' . $fileName . '.php';
+        self::assertFileExists($phpPath);
+
+        $outputPath = __DIR__ . '/../_files/' . $fileName . '.json';
+        self::assertFileExists($outputPath);
 
-        $this->expectOutputString($fixture->getExpectedOutput());
-        $this->report->generateFileReport($fixture->getReportData(), $this->createMock(File::class));
+        $file = $this->createFile($phpPath);
+
+        $this->expectOutputString((string) file_get_contents($outputPath));
+        $this->report->generateFileReport($this->getReportData($file), $file);
+    }
+
+    private function createFile(string $path): File
+    {
+        $file = new LocalFile($path, new Ruleset($this->config), $this->config);
+        $file->process();
+
+        return $file;
+    }
+
+    /**
+     * @return array{filename: string, errors: int, warnings: int, fixable: int, messages: array<mixed>}
+     */
+    private function getReportData(File $file): array
+    {
+        $reporter = new Reporter($this->config);
+        return $reporter->prepareFileReport($file); // @phpstan-ignore return.type
     }
 }

tests/Fixtures/ReportFixture.php

@@ -0,0 +1 @@
+{"type":"issue","categories":["Style"],"check_name":"Squiz.Functions.MultiLineFunctionDeclaration.BraceOnSameLine","fingerprint":"0da49d5a2f6b30f151e4abef2fccd4e2-1","severity":"major","description":"Opening brace should be on a new line","location":{"path":"_files/Mixed.php","lines":{"begin":7,"end":7}}},{"type":"issue","categories":["Style"],"check_name":"Generic.Files.LineLength.TooLong","fingerprint":"0be4b9620fcab36dcef7ba50bd3ac097-1","severity":"minor","description":"Line exceeds 120 characters; contains 143 characters","location":{"path":"_files/Mixed.php","lines":{"begin":8,"end":8}}},

tests/Fixtures/SingleViolation.php

@@ -0,0 +1,10 @@
+<?php
+
+namespace DoNotUse;
+
+class Mixed
+{
+    public function __invoke(): string {
+        return 'mixed mixed mixed mixed mixed mixed mixed mixed mixed mixed mixed mixed mixed mixed mixed mixed mixed mixed mixed mixed mixed';
+    }
+}