Fix CS with PHP-CS-Fixer

Author: mpdude

tests/Doctrine/TranslatableClassMetadataTest.php

@@ -2,14 +2,10 @@
 
 namespace Webfactory\Bundle\PolyglotBundle\Tests\Doctrine;
 
-use Doctrine\ORM\Mapping\Driver\AttributeDriver;
 use Doctrine\Persistence\Mapping\RuntimeReflectionService;
-use PHPUnit\Framework\TestCase;
 use Webfactory\Bundle\PolyglotBundle\Doctrine\TranslatableClassMetadata;
 use Webfactory\Bundle\PolyglotBundle\Tests\Fixtures\Entity\TestEntity;
-use Webfactory\Bundle\PolyglotBundle\Tests\Fixtures\Entity\TestEntityTranslation;
 use Webfactory\Bundle\PolyglotBundle\Tests\Functional\DatabaseFunctionalTestCase;
-use Webfactory\Doctrine\ORMTestInfrastructure\ORMInfrastructure;
 
 class TranslatableClassMetadataTest extends DatabaseFunctionalTestCase
 {

tests/Fixtures/Query.php

@@ -19,20 +19,20 @@ class Query
      *
      * @var string
      */
-    protected $sql = null;
+    protected $sql;
 
     /**
      * The assigned parameters.
      *
      * @var mixed[]
      */
-    protected $params = null;
+    protected $params;
 
     /**
      * Currently not used:
      * - types
      *
-     * @param string $sql - sql
+     * @param string  $sql    - sql
      * @param mixed[] $params - params
      */
     public function __construct($sql, array $params)
@@ -69,6 +69,7 @@ public function getParams()
     public function __toString()
     {
         $template = '"%s" with parameters [%s]';
-        return sprintf($template, $this->getSql(), implode(', ', $this->getParams()));
+
+        return \sprintf($template, $this->getSql(), implode(', ', $this->getParams()));
     }
 }

tests/Fixtures/QueryLogger.php

@@ -3,25 +3,26 @@
 namespace Webfactory\Bundle\PolyglotBundle\Tests\Fixtures;
 
 use Psr\Log\AbstractLogger;
+use Stringable;
 
 class QueryLogger extends AbstractLogger
 {
     public bool $enabled = true;
     private array $queries = [];
 
-    public function log($level, \Stringable|string $message, array $context = []): void
+    public function log($level, Stringable|string $message, array $context = []): void
     {
         if (!$this->enabled) {
             return;
         }
 
         if (str_starts_with($message, 'Executing')) {
             $this->queries[] = new Query($context['sql'], $context['params'] ?? []);
-        } else if ('Beginning transaction' === $message) {
+        } elseif ('Beginning transaction' === $message) {
             $this->queries[] = new Query('"START TRANSACTION"', []);
-        } else if ('Committing transaction' === $message) {
+        } elseif ('Committing transaction' === $message) {
             $this->queries[] = new Query('"COMMIT"', []);
-        } else if ('Rolling back transaction' === $message) {
+        } elseif ('Rolling back transaction' === $message) {
             $this->queries[] = new Query('"ROLLBACK"', []);
         }
     }

tests/Functional/CascadePersistTranslationsTest.php

@@ -25,8 +25,8 @@ public function adding_and_persisting_translations(): void
         $entity->addTranslation('de_DE', 'text de_DE');
 
         self::import([$entity]);
-//        $this->entityManager->persist($entity);
-//        $this->entityManager->flush();
+        //        $this->entityManager->persist($entity);
+        //        $this->entityManager->flush();
 
         // note the absent call to $this->entityManager->persist() for the translation entity
 

tests/Functional/DatabaseFunctionalTestCase.php

@@ -4,9 +4,7 @@
 
 use Doctrine\ORM\EntityManager;
 use Doctrine\ORM\Tools\SchemaTool;
-use PHPUnit\Framework\Attributes\Test;
 use Symfony\Bundle\FrameworkBundle\Test\KernelTestCase;
-use Webfactory\Bundle\PolyglotBundle\Tests\Fixtures\Entity\TestEntity;
 use Webfactory\Bundle\PolyglotBundle\Tests\Fixtures\Query;
 use Webfactory\Bundle\PolyglotBundle\Tests\Fixtures\QueryLogger;
 
@@ -34,7 +32,7 @@ protected static function setupSchema(array $classes): void
 
     protected static function import(object|array $objects): void
     {
-        if (is_object($objects)) {
+        if (\is_object($objects)) {
             $objects = [$objects];
         }
         $container = static::getContainer();
@@ -43,10 +41,10 @@ protected static function import(object|array $objects): void
         $classes = [];
         foreach ($objects as $object) {
             $entityManager->persist($object);
-            #$classes[get_class($object)] = true;
+            // $classes[get_class($object)] = true;
         }
 
-        #self::setupSchema(array_keys($classes));
+        // self::setupSchema(array_keys($classes));
         $entityManager->flush();
     }
 

tests/Functional/IntegrationTest.php

@@ -15,7 +15,7 @@ class IntegrationTest extends DatabaseFunctionalTestCase
     protected function setUp(): void
     {
         parent::setUp();
-        
+
         self::setupSchema([TestEntity::class, TestEntityTranslation::class]);
         $this->defaultLocaleProvider = self::getContainer()->get(DefaultLocaleProvider::class);
     }