Add tests for error count

Author: Ph0tonic

src/BaselinePerIdentifierFormatter.php

@@ -74,6 +74,7 @@ public function formatErrors(
         foreach ($fileErrors as $identifier => $errors) {
             $errorsToOutput = [];
 
+            $errorsCount = 0;
             foreach ($errors as $file => $errorMessages) {
                 $fileErrorsCounts = [];
 
@@ -94,12 +95,12 @@ public function formatErrors(
                         'count' => $count,
                         'path' => NeonHelper::escape($file),
                     ];
+                    $errorsCount += $count;
                 }
             }
 
             $includes[] = $identifier . '.neon';
             $baselineFilePath = $this->baselinesDir . '/' . $identifier . '.neon';
-            $errorsCount = count($errorsToOutput);
 
             $output->writeLineFormatted(sprintf('Writing baseline file %s with %d errors', $baselineFilePath, $errorsCount));
 

tests/Rule/BaselinePerIdentifierFormatterTest.php

@@ -1,4 +1,4 @@
-<?php declare(strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace ShipMonk\PHPStan\Baseline;
 
@@ -24,6 +24,7 @@ public function testFormat(): void
             $this->createAnalysisResult(
                 [
                     (new Error('Error simple', $fakeRoot . '/app/file.php', 1))->withIdentifier('sample.identifier'), // @phpstan-ignore phpstanApi.constructor
+                    (new Error('Error simple', $fakeRoot . '/app/file.php', 5))->withIdentifier('sample.identifier'), // @phpstan-ignore phpstanApi.constructor
                     (new Error('Error to escape \'#', $fakeRoot . '/app/config.php', 2))->withIdentifier('another.identifier'), // @phpstan-ignore phpstanApi.constructor
                     (new Error('Error 3', $fakeRoot . '/app/index.php', 3)), // @phpstan-ignore phpstanApi.constructor
                 ],

tests/Rule/data/baselines-neon/sample.identifier.neon

@@ -1,8 +1,8 @@
-# total 1 error
+# total 2 errors
 
 parameters:
     ignoreErrors:
         -
             message: '#^Error simple$#'
-            count: 1
+            count: 2
             path: ../app/file.php

tests/Rule/data/baselines-php/sample.identifier.php

@@ -1,11 +1,11 @@
 <?php declare(strict_types = 1);
 
-// total 1 error
+// total 2 errors
 
 $ignoreErrors = [];
 $ignoreErrors[] = [
     'message' => '#^Error simple$#',
-    'count' => 1,
+    'count' => 2,
     'path' => __DIR__ . '/../app/file.php',
 ];
 

tests/SplitterTest.php

@@ -61,7 +61,7 @@ public function testSplitter(): void
         self::assertSame([
             $folder . '/baselines/another.identifier.neon' => 1,
             $folder . '/baselines/missing-identifier.neon' => 1,
-            $folder . '/baselines/sample.identifier.neon' => 1,
+            $folder . '/baselines/sample.identifier.neon' => 2,
             $folder . '/baselines/loader.neon' => null,
         ], $written);
     }
@@ -84,7 +84,7 @@ private function getSampleErrors(): array
                 'ignoreErrors' => [
                     [
                         'message' => '#^Error simple$#',
-                        'count' => 1,
+                        'count' => 2,
                         'path' => '../app/file.php',
                         'identifier' => 'sample.identifier',
                     ],