doctrine / orm

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

Spacing +5 added lines, -5 removed lines

@@ -17,7 +17,7 @@ 
 
     protected function setUp()
     {
-        if (!class_exists('GearmanClient', false)) {
+        if ( ! class_exists('GearmanClient', false)) {
             $this->markTestSkipped('pecl/gearman is required for this test to run.');
         }
 
@@ -133,11 +133,11 @@ 
         $this->gearman->runTasks();
 
         $this->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.");
         $this->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/LockTest.php

Indentation +2 added lines, -2 removed lines

@@ -206,7 +206,7 @@
         $this->expectExceptionMessage('The optimistic lock on an entity failed.');
 
         $this->_em->createQuery($dql)
-                  ->setHint(Query::HINT_LOCK_MODE, LockMode::OPTIMISTIC)
-                  ->getSQL();
+                    ->setHint(Query::HINT_LOCK_MODE, LockMode::OPTIMISTIC)
+                    ->getSQL();
     }
 }

Spacing +6 added lines, -6 removed lines

@@ -87,7 +87,7 @@ 
         $article = new CmsArticle();
 
         $this->expectException(\InvalidArgumentException::class);
-        $this->expectExceptionMessage('Entity ' . CmsArticle::class);
+        $this->expectExceptionMessage('Entity '.CmsArticle::class);
 
         $this->_em->lock($article, LockMode::OPTIMISTIC, $article->version + 1);
     }
@@ -136,7 +136,7 @@ 
     {
         $writeLockSql = $this->_em->getConnection()->getDatabasePlatform()->getWriteLockSQL();
 
-        if (! $writeLockSql) {
+        if ( ! $writeLockSql) {
             $this->markTestSkipped('Database Driver has no Write Lock support.');
         }
 
@@ -156,8 +156,8 @@ 
             throw $e;
         }
 
-        $query = array_pop( $this->_sqlLoggerStack->queries );
-        $query = array_pop( $this->_sqlLoggerStack->queries );
+        $query = array_pop($this->_sqlLoggerStack->queries);
+        $query = array_pop($this->_sqlLoggerStack->queries);
         $this->assertContains($writeLockSql, $query['sql']);
     }
 
@@ -168,7 +168,7 @@ 
     {
         $readLockSql = $this->_em->getConnection()->getDatabasePlatform()->getReadLockSQL();
 
-        if (! $readLockSql) {
+        if ( ! $readLockSql) {
             $this->markTestSkipped('Database Driver has no Write Lock support.');
         }
 
@@ -200,7 +200,7 @@ 
      */
     public function testLockOptimisticNonVersionedThrowsExceptionInDQL()
     {
-        $dql = "SELECT u FROM " . CmsUser::class . " u WHERE u.username = 'gblanco'";
+        $dql = "SELECT u FROM ".CmsUser::class." u WHERE u.username = 'gblanco'";
 
         $this->expectException(OptimisticLockException::class);
         $this->expectExceptionMessage('The optimistic lock on an entity failed.');

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

Spacing +2 added lines, -2 removed lines

@@ -115,7 +115,7 @@ 
         $test = new OptimisticStandard();
 
         for ($i = 0; $i < 5; $i++) {
-            $test->name = 'test' . $i;
+            $test->name = 'test'.$i;
 
             $this->_em->persist($test);
             $this->_em->flush();
@@ -249,7 +249,7 @@ 
         $caughtException = null;
 
         try {
-            $expectedVersionExpired = DateTime::createFromFormat('U', $test->version->getTimestamp()-3600);
+            $expectedVersionExpired = DateTime::createFromFormat('U', $test->version->getTimestamp() - 3600);
 
             $this->_em->lock($test, LockMode::OPTIMISTIC, $expectedVersionExpired);
         } catch (OptimisticLockException $e) {

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

Spacing +1 added lines, -1 removed lines

@@ -232,7 +232,7 @@
         $entity     = $this->_em->find(Travel::class, $entitiId);
 
         $this->assertEquals(0, $entity->getVisitedCities()->count());
-        $this->assertEquals($queryCount+2, $this->getCurrentQueryCount());
+        $this->assertEquals($queryCount + 2, $this->getCurrentQueryCount());
 
         $this->_em->clear();
 

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

Indentation +2 added lines, -2 removed lines

@@ -102,8 +102,8 @@
         $routeId = $this->createPersistedRouteWithLegs();
 
         $route = $this->_em->createQuery("SELECT r, l FROM Doctrine\Tests\Models\Routing\RoutingRoute r JOIN r.legs l WHERE r.id = ?1")
-                           ->setParameter(1, $routeId)
-                           ->getSingleResult();
+                            ->setParameter(1, $routeId)
+                            ->getSingleResult();
 
         $this->assertEquals(2, count($route->legs));
         $this->assertEquals("Berlin", $route->legs[0]->fromLocation->getName());

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

Spacing +15 added lines, -15 removed lines

@@ -27,7 +27,7 @@ 
      */
     public function testIssue2059()
     {
-        if (!$this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -49,7 +49,7 @@ 
 
     public function testLoadMetadataFromDatabase()
     {
-        if (!$this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -65,22 +65,22 @@ 
         $this->assertArrayHasKey('DbdriverFoo', $metadatas);
         $metadata = $metadatas['DbdriverFoo'];
 
-        $this->assertArrayHasKey('id',          $metadata->fieldMappings);
-        $this->assertEquals('id',               $metadata->fieldMappings['id']['fieldName']);
-        $this->assertEquals('id',               strtolower($metadata->fieldMappings['id']['columnName']));
-        $this->assertEquals('integer',          (string)$metadata->fieldMappings['id']['type']);
+        $this->assertArrayHasKey('id', $metadata->fieldMappings);
+        $this->assertEquals('id', $metadata->fieldMappings['id']['fieldName']);
+        $this->assertEquals('id', strtolower($metadata->fieldMappings['id']['columnName']));
+        $this->assertEquals('integer', (string) $metadata->fieldMappings['id']['type']);
 
-        $this->assertArrayHasKey('bar',         $metadata->fieldMappings);
-        $this->assertEquals('bar',              $metadata->fieldMappings['bar']['fieldName']);
-        $this->assertEquals('bar',              strtolower($metadata->fieldMappings['bar']['columnName']));
-        $this->assertEquals('string',           (string)$metadata->fieldMappings['bar']['type']);
-        $this->assertEquals(200,                $metadata->fieldMappings['bar']['length']);
+        $this->assertArrayHasKey('bar', $metadata->fieldMappings);
+        $this->assertEquals('bar', $metadata->fieldMappings['bar']['fieldName']);
+        $this->assertEquals('bar', strtolower($metadata->fieldMappings['bar']['columnName']));
+        $this->assertEquals('string', (string) $metadata->fieldMappings['bar']['type']);
+        $this->assertEquals(200, $metadata->fieldMappings['bar']['length']);
         $this->assertTrue($metadata->fieldMappings['bar']['nullable']);
     }
 
     public function testLoadMetadataWithForeignKeyFromDatabase()
     {
-        if (!$this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -114,7 +114,7 @@ 
 
     public function testDetectManyToManyTables()
     {
-        if (!$this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->_em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -169,7 +169,7 @@ 
 
         $table->addColumn('column_index1', 'string');
         $table->addColumn('column_index2', 'string');
-        $table->addIndex(['column_index1','column_index2'], 'index1');
+        $table->addIndex(['column_index1', 'column_index2'], 'index1');
 
         $table->addColumn('column_unique_index1', 'string');
         $table->addColumn('column_unique_index2', 'string');
@@ -208,7 +208,7 @@ 
 
         $this->assertTrue( ! empty($metadata->table['indexes']['index1']['columns']));
         $this->assertEquals(
-            ['column_index1','column_index2'],
+            ['column_index1', 'column_index2'],
             $metadata->table['indexes']['index1']['columns']
         );
 

tests/Doctrine/Tests/ORM/EntityManagerTest.php

Indentation +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
     public function testCreateQueryBuilderAliasValid()
     {
         $q = $this->_em->createQueryBuilder()
-             ->select('u')->from(CmsUser::class, 'u');
+                ->select('u')->from(CmsUser::class, 'u');
         $q2 = clone $q;
 
         $this->assertEquals('SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u', $q->getQuery()->getDql());

Spacing +4 added lines, -4 removed lines

@@ -162,7 +162,7 @@ 
      */
     public function testThrowsExceptionOnNonObjectValues($methodName) {
         $this->expectException(ORMInvalidArgumentException::class);
-        $this->expectExceptionMessage('EntityManager#' . $methodName . '() expects parameter 1 to be an entity object, NULL given.');
+        $this->expectExceptionMessage('EntityManager#'.$methodName.'() expects parameter 1 to be an entity object, NULL given.');
 
         $this->_em->$methodName(null);
     }
@@ -196,7 +196,7 @@ 
      */
     public function testTransactionalAcceptsReturn()
     {
-        $return = $this->_em->transactional(function ($em) {
+        $return = $this->_em->transactional(function($em) {
             return 'foo';
         });
 
@@ -238,8 +238,8 @@ 
     public function testTransactionalReThrowsThrowables()
     {
         try {
-            $this->_em->transactional(function () {
-                (function (array $value) {
+            $this->_em->transactional(function() {
+                (function(array $value) {
                     // this only serves as an IIFE that throws a `TypeError`
                 })(null);
             });

tests/Doctrine/Tests/ORM/Decorator/EntityManagerDecoratorTest.php

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
         $methods = [];
 
         foreach ($class->getMethods() as $method) {
-            if ($method->isConstructor() || $method->isStatic() || !$method->isPublic()) {
+            if ($method->isConstructor() || $method->isStatic() || ! $method->isPublic()) {
                 continue;
             }
 
@@ -72,7 +72,7 @@ 
      */
     public function testAllMethodCallsAreDelegatedToTheWrappedInstance($method, array $parameters)
     {
-        $return = !in_array($method, self::VOID_METHODS) ? 'INNER VALUE FROM ' . $method : null;
+        $return = ! in_array($method, self::VOID_METHODS) ? 'INNER VALUE FROM '.$method : null;
 
         $this->wrapped->expects($this->once())
             ->method($method)

tests/Doctrine/Tests/ORM/Internal/HydrationCompleteHandlerTest.php

Spacing +2 added lines, -2 removed lines

@@ -89,7 +89,7 @@ 
                 $metadata,
                 Events::postLoad,
                 $entity,
-                $this->callback(function (LifecycleEventArgs $args) use ($entityManager, $entity) {
+                $this->callback(function(LifecycleEventArgs $args) use ($entityManager, $entity) {
                     return $entity === $args->getEntity() && $entityManager === $args->getObjectManager();
                 }),
                 $listenersFlag
@@ -157,7 +157,7 @@ 
                 $this->logicalOr($metadata1, $metadata2),
                 Events::postLoad,
                 $this->logicalOr($entity1, $entity2),
-                $this->callback(function (LifecycleEventArgs $args) use ($entityManager, $entity1, $entity2) {
+                $this->callback(function(LifecycleEventArgs $args) use ($entityManager, $entity1, $entity2) {
                     return in_array($args->getEntity(), [$entity1, $entity2], true)
                         && $entityManager === $args->getObjectManager();
                 }),