doctrine / orm

tests/Doctrine/Tests/ORM/Tools/SetupTest.php

Spacing +4 added lines, -4 removed lines

@@ -35,7 +35,7 @@ 
 
     public function tearDown() : void
     {
-        if (! $this->originalIncludePath) {
+        if ( ! $this->originalIncludePath) {
             return;
         }
 
@@ -74,7 +74,7 @@ 
         $config = Setup::createConfiguration(false, __DIR__);
         $cache  = $config->getMetadataCacheImpl();
 
-        self::assertSame('dc2_' . \md5(__DIR__) . '_', $cache->getNamespace());
+        self::assertSame('dc2_'.\md5(__DIR__).'_', $cache->getNamespace());
     }
 
     /**
@@ -85,7 +85,7 @@ 
         $config = Setup::createConfiguration(false, __DIR__, new ArrayCache());
         $cache  = $config->getMetadataCacheImpl();
 
-        self::assertSame('dc2_' . \md5(__DIR__) . '_', $cache->getNamespace());
+        self::assertSame('dc2_'.\md5(__DIR__).'_', $cache->getNamespace());
     }
 
     /**
@@ -100,7 +100,7 @@ 
         $cache  = $config->getMetadataCacheImpl();
 
         self::assertSame($originalCache, $cache);
-        self::assertSame('foo:dc2_' . \md5(__DIR__) . '_', $cache->getNamespace());
+        self::assertSame('foo:dc2_'.\md5(__DIR__).'_', $cache->getNamespace());
     }
 
     /**

tests/Doctrine/Tests/ORM/Functional/DatabaseDriverTestCase.php

Spacing +2 added lines, -2 removed lines

@@ -64,7 +64,7 @@ 
         $driver = new DatabaseDriver($sm);
 
         foreach ($driver->getAllClassNames() as $className) {
-            if (! \in_array(\strtolower($className), $classNames, true)) {
+            if ( ! \in_array(\strtolower($className), $classNames, true)) {
                 continue;
             }
 
@@ -74,7 +74,7 @@ 
         }
 
         if (\count($metadataList) !== \count($classNames)) {
-            $this->fail("Have not found all classes matching the names '" . \implode(', ', $classNames) . "' only tables " . \implode(', ', \array_keys($metadataList)));
+            $this->fail("Have not found all classes matching the names '".\implode(', ', $classNames)."' only tables ".\implode(', ', \array_keys($metadataList)));
         }
 
         return $metadataList;

tests/Doctrine/Tests/ORM/Functional/Locking/GearmanLockTest.php

Spacing +5 added lines, -5 removed lines

@@ -24,7 +24,7 @@ 
 
     protected function setUp() : void
     {
-        if (! \class_exists('GearmanClient', false)) {
+        if ( ! \class_exists('GearmanClient', false)) {
             $this->markTestSkipped('pecl/gearman is required for this test to run.');
         }
 
@@ -141,13 +141,13 @@ 
 
         self::assertTrue(
             $this->maxRunTime > $forTime,
-            'Because of locking this tests should have run at least ' . $forTime . ' seconds, ' .
-            'but only did for ' . $this->maxRunTime . ' seconds.'
+            'Because of locking this tests should have run at least '.$forTime.' seconds, '.
+            'but only did for '.$this->maxRunTime.' seconds.'
         );
         self::assertTrue(
             $this->maxRunTime < $notLongerThan,
-            'The longest task should not run longer than ' . $notLongerThan . ' seconds, ' .
-            'but did for ' . $this->maxRunTime . ' seconds.'
+            'The longest task should not run longer than '.$notLongerThan.' seconds, '.
+            'but did for '.$this->maxRunTime.' seconds.'
         );
     }
 

tests/Doctrine/Tests/ORM/Functional/Locking/LockAgentWorker.php

Spacing +9 added lines, -9 removed lines

@@ -36,7 +36,7 @@ 
 
         while ($worker->work()) {
             if ($worker->returnCode() !== GEARMAN_SUCCESS) {
-                echo 'return_code: ' . $worker->returnCode() . "\n";
+                echo 'return_code: '.$worker->returnCode()."\n";
                 break;
             }
         }
@@ -61,14 +61,14 @@ 
 
     public function findWithLock($job)
     {
-        return $this->process($job, static function ($fixture, $em) {
+        return $this->process($job, static function($fixture, $em) {
             $entity = $em->find($fixture['entityName'], $fixture['entityId'], $fixture['lockMode']);
         });
     }
 
     public function dqlWithLock($job)
     {
-        return $this->process($job, static function ($fixture, $em) {
+        return $this->process($job, static function($fixture, $em) {
             /** @var Doctrine\ORM\Query $query */
             $query = $em->createQuery($fixture['dql']);
 
@@ -80,7 +80,7 @@ 
 
     public function lock($job)
     {
-        return $this->process($job, static function ($fixture, $em) {
+        return $this->process($job, static function($fixture, $em) {
             $entity = $em->find($fixture['entityName'], $fixture['entityId']);
 
             $em->lock($entity, $fixture['lockMode']);
@@ -89,18 +89,18 @@ 
 
     protected function processWorkload($job)
     {
-        echo 'Received job: ' . $job->handle() . ' for function ' . $job->functionName() . "\n";
+        echo 'Received job: '.$job->handle().' for function '.$job->functionName()."\n";
 
         $workload = $job->workload();
         $workload = \unserialize($workload);
 
-        if (! isset($workload['conn']) || ! \is_array($workload['conn'])) {
+        if ( ! isset($workload['conn']) || ! \is_array($workload['conn'])) {
             throw new InvalidArgumentException('Missing Database parameters');
         }
 
         $this->em = $this->createEntityManager($workload['conn']);
 
-        if (! isset($workload['fixture'])) {
+        if ( ! isset($workload['fixture'])) {
             throw new InvalidArgumentException('Missing Fixture parameters');
         }
 
@@ -110,11 +110,11 @@ 
     protected function createEntityManager($conn)
     {
         $config = new Configuration();
-        $config->setProxyDir(__DIR__ . '/../../../Proxies');
+        $config->setProxyDir(__DIR__.'/../../../Proxies');
         $config->setProxyNamespace('MyProject\Proxies');
         $config->setAutoGenerateProxyClasses(true);
 
-        $annotDriver = $config->newDefaultAnnotationDriver([__DIR__ . '/../../../Models/']);
+        $annotDriver = $config->newDefaultAnnotationDriver([__DIR__.'/../../../Models/']);
         $config->setMetadataDriverImpl($annotDriver);
 
         $cache = new ArrayCache();

tests/Doctrine/Tests/ORM/Functional/SQLFilterTest.php

Spacing +11 added lines, -11 removed lines

@@ -335,8 +335,8 @@ 
             'locale' => ['value' => 'en', 'type' => DBALType::STRING],
         ];
 
-        self::assertEquals(\serialize($parameters), '' . $filter);
-        self::assertEquals('' . $filter, '' . $filter2);
+        self::assertEquals(\serialize($parameters), ''.$filter);
+        self::assertEquals(''.$filter, ''.$filter2);
     }
 
     public function testQueryCacheDependsOnFilters() : void
@@ -1105,7 +1105,7 @@ 
             return '';
         }
 
-        return $targetTableAlias . '.deleted = 0';
+        return $targetTableAlias.'.deleted = 0';
     }
 }
 
@@ -1113,11 +1113,11 @@ 
 {
     public function addFilterConstraint(ClassMetadata $targetEntity, $targetTableAlias)
     {
-        if (! \in_array('LocaleAware', $targetEntity->getReflectionClass()->getInterfaceNames(), true)) {
+        if ( ! \in_array('LocaleAware', $targetEntity->getReflectionClass()->getInterfaceNames(), true)) {
             return '';
         }
 
-        return $targetTableAlias . '.locale = ' . $this->getParameter('locale'); // getParam uses connection to quote the value.
+        return $targetTableAlias.'.locale = '.$this->getParameter('locale'); // getParam uses connection to quote the value.
     }
 }
 
@@ -1129,7 +1129,7 @@ 
             return '';
         }
 
-        return $targetTableAlias . '.country = ' . $this->getParameter('country'); // getParam uses connection to quote the value.
+        return $targetTableAlias.'.country = '.$this->getParameter('country'); // getParam uses connection to quote the value.
     }
 }
 
@@ -1141,7 +1141,7 @@ 
             return '';
         }
 
-        return $targetTableAlias . '.name LIKE ' . $this->getParameter('prefix'); // getParam uses connection to quote the value.
+        return $targetTableAlias.'.name LIKE '.$this->getParameter('prefix'); // getParam uses connection to quote the value.
     }
 }
 
@@ -1153,7 +1153,7 @@ 
             return '';
         }
 
-        return $targetTableAlias . '.topic = ' . $this->getParameter('topic'); // getParam uses connection to quote the value.
+        return $targetTableAlias.'.topic = '.$this->getParameter('topic'); // getParam uses connection to quote the value.
     }
 }
 
@@ -1165,7 +1165,7 @@ 
             return '';
         }
 
-        return $targetTableAlias . '.name LIKE ' . $this->getParameter('name');
+        return $targetTableAlias.'.name LIKE '.$this->getParameter('name');
     }
 }
 
@@ -1177,7 +1177,7 @@ 
             return '';
         }
 
-        return $targetTableAlias . '."completed" = ' . $this->getParameter('completed');
+        return $targetTableAlias.'."completed" = '.$this->getParameter('completed');
     }
 }
 
@@ -1189,6 +1189,6 @@ 
             return '';
         }
 
-        return $targetTableAlias . '.id = ' . $this->getParameter('id');
+        return $targetTableAlias.'.id = '.$this->getParameter('id');
     }
 }

tests/Doctrine/Tests/ORM/Functional/PaginationTest.php

Spacing +14 added lines, -14 removed lines

@@ -135,7 +135,7 @@ 
         $iter = $paginator->getIterator();
         self::assertCount(9, $iter);
         $result = \iterator_to_array($iter);
-        self::assertEquals($checkField . '0', $result[0]->{$checkField});
+        self::assertEquals($checkField.'0', $result[0]->{$checkField});
     }
 
     private function iterateWithOrderAscWithLimit($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -151,7 +151,7 @@ 
         $iter = $paginator->getIterator();
         self::assertCount(3, $iter);
         $result = \iterator_to_array($iter);
-        self::assertEquals($checkField . '0', $result[0]->{$checkField});
+        self::assertEquals($checkField.'0', $result[0]->{$checkField});
     }
 
     private function iterateWithOrderAscWithLimitAndOffset($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -167,7 +167,7 @@ 
         $iter = $paginator->getIterator();
         self::assertCount(3, $iter);
         $result = \iterator_to_array($iter);
-        self::assertEquals($checkField . '3', $result[0]->{$checkField});
+        self::assertEquals($checkField.'3', $result[0]->{$checkField});
     }
 
     private function iterateWithOrderDesc($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -180,7 +180,7 @@ 
         $iter = $paginator->getIterator();
         self::assertCount(9, $iter);
         $result = \iterator_to_array($iter);
-        self::assertEquals($checkField . '8', $result[0]->{$checkField});
+        self::assertEquals($checkField.'8', $result[0]->{$checkField});
     }
 
     private function iterateWithOrderDescWithLimit($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -195,7 +195,7 @@ 
         $iter = $paginator->getIterator();
         self::assertCount(3, $iter);
         $result = \iterator_to_array($iter);
-        self::assertEquals($checkField . '8', $result[0]->{$checkField});
+        self::assertEquals($checkField.'8', $result[0]->{$checkField});
     }
 
     private function iterateWithOrderDescWithLimitAndOffset($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -210,7 +210,7 @@ 
         $iter = $paginator->getIterator();
         self::assertCount(3, $iter);
         $result = \iterator_to_array($iter);
-        self::assertEquals($checkField . '5', $result[0]->{$checkField});
+        self::assertEquals($checkField.'5', $result[0]->{$checkField});
     }
 
     /**
@@ -483,8 +483,8 @@ 
     public function testIterateWithFetchJoinOneToManyWithOrderByColumnFromBoth($useOutputWalkers) : void
     {
         $dql     = 'SELECT c, d FROM Doctrine\Tests\Models\Pagination\Company c JOIN c.departments d ORDER BY c.name';
-        $dqlAsc  = $dql . ' ASC, d.name';
-        $dqlDesc = $dql . ' DESC, d.name';
+        $dqlAsc  = $dql.' ASC, d.name';
+        $dqlDesc = $dql.' DESC, d.name';
         $this->iterateWithOrderAsc($useOutputWalkers, true, $dqlAsc, 'name');
         $this->iterateWithOrderDesc($useOutputWalkers, true, $dqlDesc, 'name');
     }
@@ -492,8 +492,8 @@ 
     public function testIterateWithFetchJoinOneToManyWithOrderByColumnFromBothWithLimitWithOutputWalker() : void
     {
         $dql     = 'SELECT c, d FROM Doctrine\Tests\Models\Pagination\Company c JOIN c.departments d ORDER BY c.name';
-        $dqlAsc  = $dql . ' ASC, d.name';
-        $dqlDesc = $dql . ' DESC, d.name';
+        $dqlAsc  = $dql.' ASC, d.name';
+        $dqlDesc = $dql.' DESC, d.name';
         $this->iterateWithOrderAscWithLimit(true, true, $dqlAsc, 'name');
         $this->iterateWithOrderDescWithLimit(true, true, $dqlDesc, 'name');
     }
@@ -504,8 +504,8 @@ 
         $this->expectExceptionMessage('Cannot select distinct identifiers from query with LIMIT and ORDER BY on a column from a fetch joined to-many association. Use output walkers.');
 
         $dql     = 'SELECT c, d FROM Doctrine\Tests\Models\Pagination\Company c JOIN c.departments d ORDER BY c.name';
-        $dqlAsc  = $dql . ' ASC, d.name';
-        $dqlDesc = $dql . ' DESC, d.name';
+        $dqlAsc  = $dql.' ASC, d.name';
+        $dqlDesc = $dql.' DESC, d.name';
         $this->iterateWithOrderAscWithLimit(false, true, $dqlAsc, 'name');
         $this->iterateWithOrderDescWithLimit(false, true, $dqlDesc, 'name');
     }
@@ -589,7 +589,7 @@ 
         $users = \iterator_to_array($paginator->getIterator());
         self::assertCount(9, $users);
         foreach ($users as $i => $user) {
-            self::assertEquals('username' . (8 - $i), $user->username);
+            self::assertEquals('username'.(8 - $i), $user->username);
         }
     }
 
@@ -739,7 +739,7 @@ 
             $company->logo->image_width  = 100 + $i;
             $company->logo->image_height = 100 + $i;
             $company->logo->company      = $company;
-            for ($j=0; $j<3; $j++) {
+            for ($j = 0; $j < 3; $j++) {
                 $department             = new Department();
                 $department->name       = \sprintf('name%d%d', $i, $j);
                 $department->company    = $company;

tests/Doctrine/Tests/ORM/Functional/SingleTableInheritanceTest.php

Spacing +5 added lines, -5 removed lines

@@ -233,7 +233,7 @@ 
 
         $contracts = $this->em->createQuery('SELECT c FROM Doctrine\Tests\Models\Company\CompanyContract c ORDER BY c.id')->getScalarResult();
 
-        $discrValues = \array_map(static function ($a) {
+        $discrValues = \array_map(static function($a) {
             return $a['c_discr'];
         }, $contracts);
 
@@ -335,19 +335,19 @@ 
 
         $repos     = $this->em->getRepository(CompanyContract::class);
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
-        self::assertCount(3, $contracts, 'There should be 3 entities related to ' . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyContract'");
+        self::assertCount(3, $contracts, 'There should be 3 entities related to '.$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyContract'");
 
         $repos     = $this->em->getRepository(CompanyFixContract::class);
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
-        self::assertCount(1, $contracts, 'There should be 1 entities related to ' . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyFixContract'");
+        self::assertCount(1, $contracts, 'There should be 1 entities related to '.$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyFixContract'");
 
         $repos     = $this->em->getRepository(CompanyFlexContract::class);
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
-        self::assertCount(2, $contracts, 'There should be 2 entities related to ' . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyFlexContract'");
+        self::assertCount(2, $contracts, 'There should be 2 entities related to '.$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyFlexContract'");
 
         $repos     = $this->em->getRepository(CompanyFlexUltraContract::class);
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
-        self::assertCount(1, $contracts, 'There should be 1 entities related to ' . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyFlexUltraContract'");
+        self::assertCount(1, $contracts, 'There should be 1 entities related to '.$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyFlexUltraContract'");
     }
 
     /**

tests/Doctrine/Tests/ORM/Functional/SecondLevelCacheTest.php

Spacing +4 added lines, -4 removed lines

@@ -203,7 +203,7 @@ 
     {
         $listener = new ListenerSecondLevelCacheTest(
             [
-                Events::postFlush => static function () {
+                Events::postFlush => static function() {
                     throw new RuntimeException('post flush failure');
                 },
             ]
@@ -235,7 +235,7 @@ 
 
         $listener = new ListenerSecondLevelCacheTest(
             [
-                Events::postUpdate => static function () {
+                Events::postUpdate => static function() {
                     throw new RuntimeException('post update failure');
                 },
             ]
@@ -254,7 +254,7 @@ 
         self::assertInstanceOf(State::class, $state);
         self::assertEquals($stateName, $state->getName());
 
-        $state->setName($stateName . \uniqid());
+        $state->setName($stateName.\uniqid());
 
         $this->em->persist($state);
 
@@ -281,7 +281,7 @@ 
         $this->em->clear();
 
         $listener = new ListenerSecondLevelCacheTest([
-            Events::postRemove => static function () {
+            Events::postRemove => static function() {
                 throw new RuntimeException('post remove failure');
             },
         ]);

tests/Doctrine/Tests/ORM/Functional/SchemaValidatorTest.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
      */
     private function registerType(string $className)
     {
-        $type = \constant($className . '::NAME');
+        $type = \constant($className.'::NAME');
 
         if (DBALType::hasType($type)) {
             DBALType::overrideType($type, $className);
@@ -73,7 +73,7 @@ 
         foreach ($classes as $class) {
             $ce = $validator->validateClass($class);
 
-            self::assertEmpty($ce, 'Invalid Modelset: ' . $modelSet . ' class ' . $class->getClassName() . ': ' . \implode("\n", $ce));
+            self::assertEmpty($ce, 'Invalid Modelset: '.$modelSet.' class '.$class->getClassName().': '.\implode("\n", $ce));
         }
     }
 }