doctrine / orm

tests/Doctrine/Tests/ORM/Query/SelectSqlGenerationTest.php

Indentation +19 added lines, -19 removed lines

@@ -1657,9 +1657,9 @@ 
         );
     }
 
-     /**
-     * @group DDC-1430
-     */
+        /**
+         * @group DDC-1430
+         */
     public function testGroupByAllFieldsWhenObjectHasForeignKeys()
     {
         $this->assertSqlGeneration(
@@ -1999,9 +1999,9 @@ 
         );
     }
 
-   /**
-    * @group DDC-1845
-    */
+    /**
+     * @group DDC-1845
+     */
     public function testQuotedWalkJoinVariableDeclaration()
     {
         $this->assertSqlGeneration(
@@ -2035,9 +2035,9 @@ 
         );
     }
 
-   /**
-    * @group DDC-2208
-    */
+    /**
+     * @group DDC-2208
+     */
     public function testCaseThenParameterArithmeticExpression()
     {
         $this->assertSqlGeneration(
@@ -2057,8 +2057,8 @@ 
     }
 
     /**
-    * @group DDC-2268
-    */
+     * @group DDC-2268
+     */
     public function testCaseThenFunction()
     {
         $this->assertSqlGeneration(
@@ -2123,17 +2123,17 @@ 
         $this->assertSqlGeneration(
             'SELECT u.id FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE CONCAT(u.name, u.status, \'s\') = ?1',
             'SELECT t0.[id] AS c0 FROM [cms_users] t0 WHERE (t0.[name] + t0.[status] + \'s\') = ?'
-    	);
+        );
 
-    	$this->assertSqlGeneration(
+        $this->assertSqlGeneration(
             'SELECT CONCAT(u.id, u.name, u.status) FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = ?1',
             'SELECT (t0.[id] + t0.[name] + t0.[status]) AS c0 FROM [cms_users] t0 WHERE t0.[id] = ?'
-    	);
+        );
     }
 
-     /**
-     * @group DDC-2188
-     */
+        /**
+         * @group DDC-2188
+         */
     public function testArithmeticPriority()
     {
         $this->assertSqlGeneration(
@@ -2153,8 +2153,8 @@ 
     }
 
     /**
-    * @group DDC-2475
-    */
+     * @group DDC-2475
+     */
     public function testOrderByClauseShouldReplaceOrderByRelationMapping()
     {
         $this->assertSqlGeneration(

Spacing +8 added lines, -8 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Query;
 
@@ -64,7 +64,7 @@ 
 
             $query->free();
         } catch (\Exception $e) {
-            $this->fail($e->getMessage() ."\n".$e->getTraceAsString());
+            $this->fail($e->getMessage()."\n".$e->getTraceAsString());
         }
 
         self::assertEquals($sqlToBeConfirmed, $sqlGenerated);
@@ -1614,7 +1614,7 @@ 
     public function testAliasDoesNotExceedPlatformDefinedLength()
     {
         $this->assertSqlGeneration(
-            'SELECT m FROM ' . __NAMESPACE__ .  '\\DDC1384Model m',
+            'SELECT m FROM '.__NAMESPACE__.'\\DDC1384Model m',
             'SELECT t0."aVeryLongIdentifierThatShouldBeShortenedByTheSQLWalker_fooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo" AS c0 FROM "DDC1384Model" t0'
         );
     }
@@ -1647,12 +1647,12 @@ 
     public function testSelectWithArithmeticExpressionBeforeField()
     {
         $this->assertSqlGeneration(
-            'SELECT - e.value AS value, e.id FROM ' . __NAMESPACE__ . '\DDC1474Entity e',
+            'SELECT - e.value AS value, e.id FROM '.__NAMESPACE__.'\DDC1474Entity e',
             'SELECT -t0."value" AS c0, t0."id" AS c1 FROM "DDC1474Entity" t0'
         );
 
         $this->assertSqlGeneration(
-            'SELECT e.id, + e.value AS value FROM ' . __NAMESPACE__ . '\DDC1474Entity e',
+            'SELECT e.id, + e.value AS value FROM '.__NAMESPACE__.'\DDC1474Entity e',
             'SELECT t0."id" AS c0, +t0."value" AS c1 FROM "DDC1474Entity" t0'
         );
     }
@@ -2328,8 +2328,8 @@ 
     public function testHavingRegressionUsingVariableWithMathOperatorsExpression($operator)
     {
         $this->assertSqlGeneration(
-            'SELECT COUNT(u.name) AS countName FROM Doctrine\Tests\Models\CMS\CmsUser u HAVING 1 ' . $operator . ' countName > 0',
-            'SELECT COUNT(t0."name") AS c0 FROM "cms_users" t0 HAVING 1 ' . $operator . ' c0 > 0'
+            'SELECT COUNT(u.name) AS countName FROM Doctrine\Tests\Models\CMS\CmsUser u HAVING 1 '.$operator.' countName > 0',
+            'SELECT COUNT(t0."name") AS c0 FROM "cms_users" t0 HAVING 1 '.$operator.' c0 > 0'
         );
     }
 
@@ -2351,7 +2351,7 @@ 
      */
     public function getSql(SqlWalker $sqlWalker)
     {
-        return 'ABS(' . $sqlWalker->walkSimpleArithmeticExpression($this->simpleArithmeticExpression) . ')';
+        return 'ABS('.$sqlWalker->walkSimpleArithmeticExpression($this->simpleArithmeticExpression).')';
     }
 
     /**

tests/Doctrine/Tests/ORM/Query/DeleteSqlGenerationTest.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Query;
 use Doctrine\Tests\OrmTestCase;

Doc Comments +4 added lines

@@ -25,6 +25,10 @@
         $this->em = $this->getTestEntityManager();
     }
 
+    /**
+     * @param string $dqlToBeTested
+     * @param string $sqlToBeConfirmed
+     */
     public function assertSqlGeneration($dqlToBeTested, $sqlToBeConfirmed)
     {
         try {

tests/Doctrine/Tests/ORM/Query/FilterCollectionTest.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Query;
 
@@ -88,6 +88,6 @@ 
     public function addFilterConstraint(ClassMetadata $targetEntity, $targetTableAlias)
     {
         // getParameter applies quoting automatically
-        return $targetTableAlias . '.id = ' . $this->getParameter('id');
+        return $targetTableAlias.'.id = '.$this->getParameter('id');
     }
 }

tests/Doctrine/Tests/ORM/Persisters/BasicEntityPersisterCompositeTypeSqlTest.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Persisters;
 

tests/Doctrine/Tests/ORM/Functional/SchemaTool/PostgreSqlSchemaToolTest.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional\SchemaTool;
 
@@ -16,7 +16,7 @@ 
         parent::setUp();
 
         if ($this->em->getConnection()->getDatabasePlatform()->getName() !== 'postgresql') {
-            $this->markTestSkipped('The ' . __CLASS__ .' requires the use of postgresql.');
+            $this->markTestSkipped('The '.__CLASS__.' requires the use of postgresql.');
         }
     }
 

tests/Doctrine/Tests/ORM/Functional/SchemaTool/CompanySchemaTest.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional\SchemaTool;
 
@@ -58,7 +58,7 @@ 
      */
     public function testDropPartSchemaWithForeignKeys()
     {
-        if (!$this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped("Foreign Key test");
         }
 

tests/Doctrine/Tests/ORM/Functional/SchemaTool/DBAL483Test.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional\SchemaTool;
 
@@ -30,7 +30,7 @@ 
 
         $updateSql = $this->schemaTool->getUpdateSchemaSql([$class]);
 
-        $updateSql = array_filter($updateSql, function ($sql) {
+        $updateSql = array_filter($updateSql, function($sql) {
             return strpos($sql, 'DBAL483') !== false;
         });
 

tests/Doctrine/Tests/ORM/Functional/SchemaTool/MySqlSchemaToolTest.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional\SchemaTool;
 
@@ -14,7 +14,7 @@ 
     protected function setUp() {
         parent::setUp();
         if ($this->em->getConnection()->getDatabasePlatform()->getName() !== 'mysql') {
-            $this->markTestSkipped('The ' . __CLASS__ .' requires the use of mysql.');
+            $this->markTestSkipped('The '.__CLASS__.' requires the use of mysql.');
         }
     }
 

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

Spacing +28 added lines, -28 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\Tests\ORM\Functional;
 
@@ -744,9 +744,9 @@ 
         self::assertEquals($this->fixtures[1]->address->country, $result[1][1]->country);
         self::assertEquals($this->fixtures[2]->address->country, $result[2][1]->country);
 
-        self::assertEquals($this->fixtures[0]->status,$result[0]['status']);
-        self::assertEquals($this->fixtures[1]->status,$result[1]['status']);
-        self::assertEquals($this->fixtures[2]->status,$result[2]['status']);
+        self::assertEquals($this->fixtures[0]->status, $result[0]['status']);
+        self::assertEquals($this->fixtures[1]->status, $result[1]['status']);
+        self::assertEquals($this->fixtures[2]->status, $result[2]['status']);
     }
 
     public function testShouldSupportMultipleNewOperatorsAndSingleScalarWithAliases()
@@ -801,9 +801,9 @@ 
         self::assertEquals($this->fixtures[1]->address->country, $result[1]['cmsAddress']->country);
         self::assertEquals($this->fixtures[2]->address->country, $result[2]['cmsAddress']->country);
 
-        self::assertEquals($this->fixtures[0]->status,$result[0]['cmsUserStatus']);
-        self::assertEquals($this->fixtures[1]->status,$result[1]['cmsUserStatus']);
-        self::assertEquals($this->fixtures[2]->status,$result[2]['cmsUserStatus']);
+        self::assertEquals($this->fixtures[0]->status, $result[0]['cmsUserStatus']);
+        self::assertEquals($this->fixtures[1]->status, $result[1]['cmsUserStatus']);
+        self::assertEquals($this->fixtures[2]->status, $result[2]['cmsUserStatus']);
     }
 
     public function testShouldSupportMultipleNewOperatorsAndSingleScalarWithAndWithoutAliases()
@@ -858,9 +858,9 @@ 
         self::assertEquals($this->fixtures[1]->address->country, $result[1][0]->country);
         self::assertEquals($this->fixtures[2]->address->country, $result[2][0]->country);
 
-        self::assertEquals($this->fixtures[0]->status,$result[0]['status']);
-        self::assertEquals($this->fixtures[1]->status,$result[1]['status']);
-        self::assertEquals($this->fixtures[2]->status,$result[2]['status']);
+        self::assertEquals($this->fixtures[0]->status, $result[0]['status']);
+        self::assertEquals($this->fixtures[1]->status, $result[1]['status']);
+        self::assertEquals($this->fixtures[2]->status, $result[2]['status']);
     }
 
     public function testShouldSupportMultipleNewOperatorsAndMultipleScalars()
@@ -916,13 +916,13 @@ 
         self::assertEquals($this->fixtures[1]->address->country, $result[1][1]->country);
         self::assertEquals($this->fixtures[2]->address->country, $result[2][1]->country);
 
-        self::assertEquals($this->fixtures[0]->status,$result[0]['status']);
-        self::assertEquals($this->fixtures[1]->status,$result[1]['status']);
-        self::assertEquals($this->fixtures[2]->status,$result[2]['status']);
+        self::assertEquals($this->fixtures[0]->status, $result[0]['status']);
+        self::assertEquals($this->fixtures[1]->status, $result[1]['status']);
+        self::assertEquals($this->fixtures[2]->status, $result[2]['status']);
 
-        self::assertEquals($this->fixtures[0]->username,$result[0]['username']);
-        self::assertEquals($this->fixtures[1]->username,$result[1]['username']);
-        self::assertEquals($this->fixtures[2]->username,$result[2]['username']);
+        self::assertEquals($this->fixtures[0]->username, $result[0]['username']);
+        self::assertEquals($this->fixtures[1]->username, $result[1]['username']);
+        self::assertEquals($this->fixtures[2]->username, $result[2]['username']);
     }
 
     public function testShouldSupportMultipleNewOperatorsAndMultipleScalarsWithAliases()
@@ -978,13 +978,13 @@ 
         self::assertEquals($this->fixtures[1]->address->country, $result[1]['cmsAddress']->country);
         self::assertEquals($this->fixtures[2]->address->country, $result[2]['cmsAddress']->country);
 
-        self::assertEquals($this->fixtures[0]->status,$result[0]['cmsUserStatus']);
-        self::assertEquals($this->fixtures[1]->status,$result[1]['cmsUserStatus']);
-        self::assertEquals($this->fixtures[2]->status,$result[2]['cmsUserStatus']);
+        self::assertEquals($this->fixtures[0]->status, $result[0]['cmsUserStatus']);
+        self::assertEquals($this->fixtures[1]->status, $result[1]['cmsUserStatus']);
+        self::assertEquals($this->fixtures[2]->status, $result[2]['cmsUserStatus']);
 
-        self::assertEquals($this->fixtures[0]->username,$result[0]['cmsUserUsername']);
-        self::assertEquals($this->fixtures[1]->username,$result[1]['cmsUserUsername']);
-        self::assertEquals($this->fixtures[2]->username,$result[2]['cmsUserUsername']);
+        self::assertEquals($this->fixtures[0]->username, $result[0]['cmsUserUsername']);
+        self::assertEquals($this->fixtures[1]->username, $result[1]['cmsUserUsername']);
+        self::assertEquals($this->fixtures[2]->username, $result[2]['cmsUserUsername']);
     }
 
     public function testShouldSupportMultipleNewOperatorsAndMultipleScalarsWithAndWithoutAliases()
@@ -1040,13 +1040,13 @@ 
         self::assertEquals($this->fixtures[1]->address->country, $result[1][0]->country);
         self::assertEquals($this->fixtures[2]->address->country, $result[2][0]->country);
 
-        self::assertEquals($this->fixtures[0]->status,$result[0]['status']);
-        self::assertEquals($this->fixtures[1]->status,$result[1]['status']);
-        self::assertEquals($this->fixtures[2]->status,$result[2]['status']);
+        self::assertEquals($this->fixtures[0]->status, $result[0]['status']);
+        self::assertEquals($this->fixtures[1]->status, $result[1]['status']);
+        self::assertEquals($this->fixtures[2]->status, $result[2]['status']);
 
-        self::assertEquals($this->fixtures[0]->username,$result[0]['cmsUserUsername']);
-        self::assertEquals($this->fixtures[1]->username,$result[1]['cmsUserUsername']);
-        self::assertEquals($this->fixtures[2]->username,$result[2]['cmsUserUsername']);
+        self::assertEquals($this->fixtures[0]->username, $result[0]['cmsUserUsername']);
+        self::assertEquals($this->fixtures[1]->username, $result[1]['cmsUserUsername']);
+        self::assertEquals($this->fixtures[2]->username, $result[2]['cmsUserUsername']);
     }
 
     /**