fix ecs

Author: jenschude

composer.json

@@ -63,6 +63,6 @@
     "monolog/monolog": "^1.3",
     "phpunit/phpunit": "^9.0 | ^8.0",
     "cache/array-adapter": "^1.0",
-    "symplify/easy-coding-standard": "^7.2"
+    "symplify/easy-coding-standard": "^9.0"
   }
 }

ecs.php

@@ -0,0 +1,20 @@
+<?php
+
+declare(strict_types=1);
+
+use Commercetools\Tools\SummaryOnlyOutputFormatter;
+use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
+
+return static function (ContainerConfigurator $containerConfigurator): void {
+    $parameters = $containerConfigurator->parameters();
+
+    $parameters->set('cache_directory', __DIR__ . '/.ecs_cache');
+
+    $parameters->set('paths', [__DIR__ . '/src', __DIR__ . '/test', __DIR__ . '/lib/commercetools-api/src', __DIR__ . '/lib/commercetools-api-tests/test', __DIR__ . '/lib/commercetools-base/src', __DIR__ . '/lib/commercetools-base/test', __DIR__ . '/lib/commercetools-import/src', __DIR__ . '/lib/commercetools-import-tests/test', __DIR__ . '/lib/commercetools-ml/src', __DIR__ . '/lib/commercetools-ml-tests/test']);
+
+    $parameters->set('sets', ['psr12']);
+
+    $services = $containerConfigurator->services();
+
+    $services->set(SummaryOnlyOutputFormatter::class);
+};

ecs.yaml

@@ -8,6 +8,7 @@
 use Symplify\EasyCodingStandard\Console\Style\EasyCodingStandardStyle;
 use Symplify\EasyCodingStandard\Contract\Console\Output\OutputFormatterInterface;
 use Symplify\EasyCodingStandard\Error\ErrorAndDiffCollector;
+use Symplify\EasyCodingStandard\ValueObject\Error\ErrorAndDiffResult;
 use Symplify\PackageBuilder\Console\ShellCode;
 
 final class SummaryOnlyOutputFormatter implements OutputFormatterInterface
@@ -39,13 +40,13 @@ public function __construct(
     ) {
         $this->easyCodingStandardStyle = $easyCodingStandardStyle;
         $this->configuration = $configuration;
-        $this->errorAndDiffCollector = $errorAndDiffCollector;
+        $errorAndDiffResult = $errorAndDiffCollector;
     }
 
-    public function report(int $processedFilesCount): int
+    public function report(ErrorAndDiffResult $errorAndDiffResult, int $processedFilesCount): int
     {
-        if ($this->errorAndDiffCollector->getErrorCount() === 0
-            && $this->errorAndDiffCollector->getFileDiffsCount() === 0
+        if ($errorAndDiffResult->getErrorCount() === 0
+            && $errorAndDiffResult->getFileDiffsCount() === 0
         ) {
             if ($processedFilesCount !== 0) {
                 $this->easyCodingStandardStyle->newLine();
@@ -58,53 +59,53 @@ public function report(int $processedFilesCount): int
 
         $this->easyCodingStandardStyle->newLine();
 
-        return $this->configuration->isFixer() ? $this->printAfterFixerStatus() : $this->printNoFixerStatus();
+        return $this->configuration->isFixer() ? $this->printAfterFixerStatus($errorAndDiffResult) : $this->printNoFixerStatus($errorAndDiffResult);
     }
 
     public function getName(): string
     {
         return self::NAME;
     }
 
-    private function printAfterFixerStatus(): int
+    private function printAfterFixerStatus(ErrorAndDiffResult $errorAndDiffResult): int
     {
         if ($this->configuration->shouldShowErrorTable()) {
-            $this->easyCodingStandardStyle->printErrors($this->errorAndDiffCollector->getErrors());
+            $this->easyCodingStandardStyle->printErrors($errorAndDiffResult->getErrors());
         }
 
-        if ($this->errorAndDiffCollector->getErrorCount() === 0) {
+        if ($errorAndDiffResult->getErrorCount() === 0) {
             $this->easyCodingStandardStyle->success(
                 sprintf(
                     '%d error%s successfully fixed and no other found!',
-                    $this->errorAndDiffCollector->getFileDiffsCount(),
-                    $this->errorAndDiffCollector->getFileDiffsCount() === 1 ? '' : 's'
+                    $errorAndDiffResult->getFileDiffsCount(),
+                    $errorAndDiffResult->getFileDiffsCount() === 1 ? '' : 's'
                 )
             );
 
             return ShellCode::SUCCESS;
         }
 
         $this->printErrorMessageFromErrorCounts(
-            $this->errorAndDiffCollector->getErrorCount(),
-            $this->errorAndDiffCollector->getFileDiffsCount()
+            $errorAndDiffResult->getErrorCount(),
+            $errorAndDiffResult->getFileDiffsCount()
         );
 
         return ShellCode::ERROR;
     }
 
-    private function printNoFixerStatus(): int
+    private function printNoFixerStatus(ErrorAndDiffResult $errorAndDiffResult): int
     {
         if ($this->configuration->shouldShowErrorTable()) {
-            $errors = $this->errorAndDiffCollector->getErrors();
+            $errors = $errorAndDiffResult->getErrors();
             if (count($errors) > 0) {
                 $this->easyCodingStandardStyle->newLine();
                 $this->easyCodingStandardStyle->printErrors($errors);
             }
         }
 
         $this->printErrorMessageFromErrorCounts(
-            $this->errorAndDiffCollector->getErrorCount(),
-            $this->errorAndDiffCollector->getFileDiffsCount()
+            $errorAndDiffResult->getErrorCount(),
+            $errorAndDiffResult->getFileDiffsCount()
         );
 
         return ShellCode::ERROR;