Test Failed
Pull Request — master (#6869)
by Michael
63:52
created
tests/Doctrine/Tests/Mocks/DriverConnectionMock.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@
 block discarded – undo
49 49
     /**
50 50
      * {@inheritdoc}
51 51
      */
52
-    public function quote($input, $type=\PDO::PARAM_STR)
52
+    public function quote($input, $type = \PDO::PARAM_STR)
53 53
     {
54 54
     }
55 55
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/Mocks/EntityManagerMock.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@
 block discarded – undo
70 70
     {
71 71
         if (null === $config) {
72 72
             $config = new Configuration();
73
-            $config->setProxyDir(__DIR__ . '/../Proxies');
73
+            $config->setProxyDir(__DIR__.'/../Proxies');
74 74
             $config->setProxyNamespace('Doctrine\Tests\Proxies');
75 75
             $config->setMetadataDriverImpl($config->newDefaultAnnotationDriver([], true));
76 76
         }
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/EntityRepositoryTest.php 2 patches
Indentation   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -308,7 +308,7 @@  discard block
 block discarded – undo
308 308
      */
309 309
     public function testExceptionIsThrownWhenCallingFindByWithoutParameter() {
310 310
         $this->_em->getRepository(CmsUser::class)
311
-                  ->findByStatus();
311
+                    ->findByStatus();
312 312
     }
313 313
 
314 314
     /**
@@ -316,7 +316,7 @@  discard block
 block discarded – undo
316 316
      */
317 317
     public function testExceptionIsThrownWhenUsingInvalidFieldName() {
318 318
         $this->_em->getRepository(CmsUser::class)
319
-                  ->findByThisFieldDoesNotExist('testvalue');
319
+                    ->findByThisFieldDoesNotExist('testvalue');
320 320
     }
321 321
 
322 322
     /**
@@ -328,7 +328,7 @@  discard block
 block discarded – undo
328 328
         $this->expectException(TransactionRequiredException::class);
329 329
 
330 330
         $this->_em->getRepository(CmsUser::class)
331
-                  ->find(1, LockMode::PESSIMISTIC_READ);
331
+                    ->find(1, LockMode::PESSIMISTIC_READ);
332 332
     }
333 333
 
334 334
     /**
@@ -340,7 +340,7 @@  discard block
 block discarded – undo
340 340
         $this->expectException(TransactionRequiredException::class);
341 341
 
342 342
         $this->_em->getRepository(CmsUser::class)
343
-                  ->find(1, LockMode::PESSIMISTIC_WRITE);
343
+                    ->find(1, LockMode::PESSIMISTIC_WRITE);
344 344
     }
345 345
 
346 346
     /**
@@ -352,7 +352,7 @@  discard block
 block discarded – undo
352 352
         $this->expectException(OptimisticLockException::class);
353 353
 
354 354
         $this->_em->getRepository(CmsUser::class)
355
-                  ->find(1, LockMode::OPTIMISTIC);
355
+                    ->find(1, LockMode::OPTIMISTIC);
356 356
     }
357 357
 
358 358
     /**
@@ -433,13 +433,13 @@  discard block
 block discarded – undo
433 433
      */
434 434
     public function testFindOneByOrderBy()
435 435
     {
436
-    	$this->loadFixture();
436
+        $this->loadFixture();
437 437
 
438
-    	$repos = $this->_em->getRepository(CmsUser::class);
439
-    	$userAsc = $repos->findOneBy([], ["username" => "ASC"]);
440
-    	$userDesc = $repos->findOneBy([], ["username" => "DESC"]);
438
+        $repos = $this->_em->getRepository(CmsUser::class);
439
+        $userAsc = $repos->findOneBy([], ["username" => "ASC"]);
440
+        $userDesc = $repos->findOneBy([], ["username" => "DESC"]);
441 441
 
442
-    	$this->assertNotSame($userAsc, $userDesc);
442
+        $this->assertNotSame($userAsc, $userDesc);
443 443
     }
444 444
 
445 445
     /**
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
         $repos = $this->_em->getRepository(CmsUser::class);
181 181
 
182 182
         $user = $repos->find($user1Id);
183
-        $this->assertInstanceOf(CmsUser::class,$user);
183
+        $this->assertInstanceOf(CmsUser::class, $user);
184 184
         $this->assertEquals('Roman', $user->name);
185 185
         $this->assertEquals('freak', $user->status);
186 186
     }
@@ -192,7 +192,7 @@  discard block
 block discarded – undo
192 192
 
193 193
         $users = $repos->findBy(['status' => 'dev']);
194 194
         $this->assertEquals(2, count($users));
195
-        $this->assertInstanceOf(CmsUser::class,$users[0]);
195
+        $this->assertInstanceOf(CmsUser::class, $users[0]);
196 196
         $this->assertEquals('Guilherme', $users[0]->name);
197 197
         $this->assertEquals('dev', $users[0]->status);
198 198
     }
@@ -218,7 +218,7 @@  discard block
 block discarded – undo
218 218
         $addresses  = $repository->findBy(['user' => [$user1->getId(), $user2->getId()]]);
219 219
 
220 220
         $this->assertEquals(2, count($addresses));
221
-        $this->assertInstanceOf(CmsAddress::class,$addresses[0]);
221
+        $this->assertInstanceOf(CmsAddress::class, $addresses[0]);
222 222
     }
223 223
 
224 224
     public function testFindByAssociationWithObjectAsParameter()
@@ -242,7 +242,7 @@  discard block
 block discarded – undo
242 242
         $addresses  = $repository->findBy(['user' => [$user1, $user2]]);
243 243
 
244 244
         $this->assertEquals(2, count($addresses));
245
-        $this->assertInstanceOf(CmsAddress::class,$addresses[0]);
245
+        $this->assertInstanceOf(CmsAddress::class, $addresses[0]);
246 246
     }
247 247
 
248 248
     public function testFindFieldByMagicCall()
@@ -252,7 +252,7 @@  discard block
 block discarded – undo
252 252
 
253 253
         $users = $repos->findByStatus('dev');
254 254
         $this->assertEquals(2, count($users));
255
-        $this->assertInstanceOf(CmsUser::class,$users[0]);
255
+        $this->assertInstanceOf(CmsUser::class, $users[0]);
256 256
         $this->assertEquals('Guilherme', $users[0]->name);
257 257
         $this->assertEquals('dev', $users[0]->status);
258 258
     }
@@ -592,7 +592,7 @@  discard block
 block discarded – undo
592 592
         $this->assertEquals(2, count($usersAsc));
593 593
         $this->assertEquals(2, count($usersDesc));
594 594
 
595
-        $this->assertInstanceOf(CmsUser::class,$usersAsc[0]);
595
+        $this->assertInstanceOf(CmsUser::class, $usersAsc[0]);
596 596
         $this->assertEquals('Alexander', $usersAsc[0]->name);
597 597
         $this->assertEquals('dev', $usersAsc[0]->status);
598 598
 
@@ -673,7 +673,7 @@  discard block
 block discarded – undo
673 673
     public function testCanRetrieveRepositoryFromClassNameWithLeadingBackslash()
674 674
     {
675 675
         $this->assertSame(
676
-            $this->_em->getRepository('\\' . CmsUser::class),
676
+            $this->_em->getRepository('\\'.CmsUser::class),
677 677
             $this->_em->getRepository(CmsUser::class)
678 678
         );
679 679
     }
@@ -711,7 +711,7 @@  discard block
 block discarded – undo
711 711
         $data = $repo->findBy(['user' => [1, 2, 3]]);
712 712
 
713 713
         $query = array_pop($this->_sqlLoggerStack->queries);
714
-        $this->assertEquals([1,2,3], $query['params'][0]);
714
+        $this->assertEquals([1, 2, 3], $query['params'][0]);
715 715
         $this->assertEquals(Connection::PARAM_INT_ARRAY, $query['types'][0]);
716 716
     }
717 717
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/ManyToManyBasicAssociationTest.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -265,8 +265,8 @@
 block discarded – undo
265 265
         $this->_em->clear();
266 266
 
267 267
         $newUser = $this->_em->createQuery('SELECT u, g FROM Doctrine\Tests\Models\CMS\CmsUser u LEFT JOIN u.groups g WHERE u.id = ?1')
268
-                             ->setParameter(1, $user->getId())
269
-                             ->getSingleResult();
268
+                                ->setParameter(1, $user->getId())
269
+                                ->getSingleResult();
270 270
         $this->assertEquals(0, count($newUser->groups));
271 271
         $this->assertInternalType('array', $newUser->groups->getMapping());
272 272
 
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -290,9 +290,9 @@  discard block
 block discarded – undo
290 290
         $user->username = 'gblanco';
291 291
         $user->status = 'developer';
292 292
 
293
-        for ($i=0; $i < $groupCount; ++$i) {
293
+        for ($i = 0; $i < $groupCount; ++$i) {
294 294
             $group = new CmsGroup;
295
-            $group->name = 'Developers_' . $i;
295
+            $group->name = 'Developers_'.$i;
296 296
             $user->addGroup($group);
297 297
         }
298 298
 
@@ -406,7 +406,7 @@  discard block
 block discarded – undo
406 406
             $user
407 407
                 ->getGroups()
408 408
                 ->matching($criteria)
409
-                ->map(function (CmsGroup $group) {
409
+                ->map(function(CmsGroup $group) {
410 410
                     return $group->getName();
411 411
                 })
412 412
                 ->toArray()
@@ -450,7 +450,7 @@  discard block
 block discarded – undo
450 450
             $user
451 451
                 ->getTags()
452 452
                 ->matching($criteria)
453
-                ->map(function (CmsTag $tag) {
453
+                ->map(function(CmsTag $tag) {
454 454
                     return $tag->getName();
455 455
                 })
456 456
                 ->toArray()
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/MergeProxiesTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -237,10 +237,10 @@
 block discarded – undo
237 237
     {
238 238
         $config = new Configuration();
239 239
 
240
-        $config->setProxyDir(realpath(__DIR__ . '/../../Proxies'));
240
+        $config->setProxyDir(realpath(__DIR__.'/../../Proxies'));
241 241
         $config->setProxyNamespace('Doctrine\Tests\Proxies');
242 242
         $config->setMetadataDriverImpl($config->newDefaultAnnotationDriver(
243
-            [realpath(__DIR__ . '/../../Models/Cache')],
243
+            [realpath(__DIR__.'/../../Models/Cache')],
244 244
             true
245 245
         ));
246 246
         $config->setSQLLogger($logger);
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/CascadeRemoveOrderTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -152,10 +152,10 @@
 block discarded – undo
152 152
      */
153 153
     private $ownerO;
154 154
 
155
-    public function __construct(CascadeRemoveOrderEntityO $eO, $position=1)
155
+    public function __construct(CascadeRemoveOrderEntityO $eO, $position = 1)
156 156
     {
157 157
         $this->position = $position;
158
-        $this->ownerO= $eO;
158
+        $this->ownerO = $eO;
159 159
         $this->ownerO->addOneToManyG($this);
160 160
     }
161 161
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/SecondLevelCacheManyToOneTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -214,7 +214,7 @@
 block discarded – undo
214 214
         $this->assertNull($this->cache->getEntityCacheRegion(ComplexAction::class));
215 215
         $this->assertInstanceOf(Region::class, $this->cache->getEntityCacheRegion(Token::class));
216 216
 
217
-        $token  = new Token('token-hash');
217
+        $token = new Token('token-hash');
218 218
 
219 219
         $action1 = new Action('login');
220 220
         $action2 = new Action('logout');
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/PaginationTest.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
         $iter = $paginator->getIterator();
130 130
         $this->assertCount(9, $iter);
131 131
         $result = iterator_to_array($iter);
132
-        $this->assertEquals($checkField . "0", $result[0]->$checkField);
132
+        $this->assertEquals($checkField."0", $result[0]->$checkField);
133 133
     }
134 134
 
135 135
     private function iterateWithOrderAscWithLimit($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
         $iter = $paginator->getIterator();
146 146
         $this->assertCount(3, $iter);
147 147
         $result = iterator_to_array($iter);
148
-        $this->assertEquals($checkField . "0", $result[0]->$checkField);
148
+        $this->assertEquals($checkField."0", $result[0]->$checkField);
149 149
     }
150 150
 
151 151
     private function iterateWithOrderAscWithLimitAndOffset($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
161 161
         $iter = $paginator->getIterator();
162 162
         $this->assertCount(3, $iter);
163 163
         $result = iterator_to_array($iter);
164
-        $this->assertEquals($checkField . "3", $result[0]->$checkField);
164
+        $this->assertEquals($checkField."3", $result[0]->$checkField);
165 165
     }
166 166
 
167 167
     private function iterateWithOrderDesc($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
174 174
         $iter = $paginator->getIterator();
175 175
         $this->assertCount(9, $iter);
176 176
         $result = iterator_to_array($iter);
177
-        $this->assertEquals($checkField . "8", $result[0]->$checkField);
177
+        $this->assertEquals($checkField."8", $result[0]->$checkField);
178 178
     }
179 179
 
180 180
     private function iterateWithOrderDescWithLimit($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -189,7 +189,7 @@  discard block
 block discarded – undo
189 189
         $iter = $paginator->getIterator();
190 190
         $this->assertCount(3, $iter);
191 191
         $result = iterator_to_array($iter);
192
-        $this->assertEquals($checkField . "8", $result[0]->$checkField);
192
+        $this->assertEquals($checkField."8", $result[0]->$checkField);
193 193
     }
194 194
 
195 195
     private function iterateWithOrderDescWithLimitAndOffset($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -204,7 +204,7 @@  discard block
 block discarded – undo
204 204
         $iter = $paginator->getIterator();
205 205
         $this->assertCount(3, $iter);
206 206
         $result = iterator_to_array($iter);
207
-        $this->assertEquals($checkField . "5", $result[0]->$checkField);
207
+        $this->assertEquals($checkField."5", $result[0]->$checkField);
208 208
     }
209 209
 
210 210
     /**
@@ -477,8 +477,8 @@  discard block
 block discarded – undo
477 477
     public function testIterateWithFetchJoinOneToManyWithOrderByColumnFromBoth($useOutputWalkers)
478 478
     {
479 479
         $dql = 'SELECT c, d FROM Doctrine\Tests\Models\Pagination\Company c JOIN c.departments d ORDER BY c.name';
480
-        $dqlAsc = $dql . " ASC, d.name";
481
-        $dqlDesc = $dql . " DESC, d.name";
480
+        $dqlAsc = $dql." ASC, d.name";
481
+        $dqlDesc = $dql." DESC, d.name";
482 482
         $this->iterateWithOrderAsc($useOutputWalkers, true, $dqlAsc, "name");
483 483
         $this->iterateWithOrderDesc($useOutputWalkers, true, $dqlDesc, "name");
484 484
     }
@@ -486,8 +486,8 @@  discard block
 block discarded – undo
486 486
     public function testIterateWithFetchJoinOneToManyWithOrderByColumnFromBothWithLimitWithOutputWalker()
487 487
     {
488 488
         $dql = 'SELECT c, d FROM Doctrine\Tests\Models\Pagination\Company c JOIN c.departments d ORDER BY c.name';
489
-        $dqlAsc = $dql . " ASC, d.name";
490
-        $dqlDesc = $dql . " DESC, d.name";
489
+        $dqlAsc = $dql." ASC, d.name";
490
+        $dqlDesc = $dql." DESC, d.name";
491 491
         $this->iterateWithOrderAscWithLimit(true, true, $dqlAsc, "name");
492 492
         $this->iterateWithOrderDescWithLimit(true, true, $dqlDesc, "name");
493 493
     }
@@ -498,8 +498,8 @@  discard block
 block discarded – undo
498 498
         $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.');
499 499
 
500 500
         $dql = 'SELECT c, d FROM Doctrine\Tests\Models\Pagination\Company c JOIN c.departments d ORDER BY c.name';
501
-        $dqlAsc = $dql . " ASC, d.name";
502
-        $dqlDesc = $dql . " DESC, d.name";
501
+        $dqlAsc = $dql." ASC, d.name";
502
+        $dqlDesc = $dql." DESC, d.name";
503 503
         $this->iterateWithOrderAscWithLimit(false, true, $dqlAsc, "name");
504 504
         $this->iterateWithOrderDescWithLimit(false, true, $dqlDesc, "name");
505 505
     }
@@ -583,7 +583,7 @@  discard block
 block discarded – undo
583 583
         $users = iterator_to_array($paginator->getIterator());
584 584
         $this->assertCount(9, $users);
585 585
         foreach ($users as $i => $user) {
586
-            $this->assertEquals("username" . (8 - $i), $user->username);
586
+            $this->assertEquals("username".(8 - $i), $user->username);
587 587
         }
588 588
     }
589 589
 
@@ -733,7 +733,7 @@  discard block
 block discarded – undo
733 733
             $company->logo->image_width = 100 + $i;
734 734
             $company->logo->image_height = 100 + $i;
735 735
             $company->logo->company = $company;
736
-            for($j=0;$j<3;$j++) {
736
+            for ($j = 0; $j < 3; $j++) {
737 737
                 $department = new Department();
738 738
                 $department->name = "name$i$j";
739 739
                 $department->company = $company;
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/SingleTableInheritanceTest.php 2 patches
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -411,9 +411,9 @@
 block discarded – undo
411 411
 
412 412
         $dql = 'SELECT f FROM Doctrine\Tests\Models\Company\CompanyFixContract f WHERE f.id = ?1';
413 413
         $contract = $this->_em->createQuery($dql)
414
-                              ->setFetchMode(CompanyFixContract::class, 'salesPerson', ClassMetadata::FETCH_EAGER)
415
-                              ->setParameter(1, $this->fix->getId())
416
-                              ->getSingleResult();
414
+                                ->setFetchMode(CompanyFixContract::class, 'salesPerson', ClassMetadata::FETCH_EAGER)
415
+                                ->setParameter(1, $this->fix->getId())
416
+                                ->getSingleResult();
417 417
 
418 418
         $this->assertNotInstanceOf(Proxy::class, $contract->getSalesPerson());
419 419
     }
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -330,19 +330,19 @@
 block discarded – undo
330 330
 
331 331
         $repos = $this->_em->getRepository(CompanyContract::class);
332 332
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
333
-        $this->assertEquals(3, count($contracts), "There should be 3 entities related to " . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyContract'");
333
+        $this->assertEquals(3, count($contracts), "There should be 3 entities related to ".$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyContract'");
334 334
 
335 335
         $repos = $this->_em->getRepository(CompanyFixContract::class);
336 336
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
337
-        $this->assertEquals(1, count($contracts), "There should be 1 entities related to " . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyFixContract'");
337
+        $this->assertEquals(1, count($contracts), "There should be 1 entities related to ".$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyFixContract'");
338 338
 
339 339
         $repos = $this->_em->getRepository(CompanyFlexContract::class);
340 340
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
341
-        $this->assertEquals(2, count($contracts), "There should be 2 entities related to " . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyFlexContract'");
341
+        $this->assertEquals(2, count($contracts), "There should be 2 entities related to ".$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyFlexContract'");
342 342
 
343 343
         $repos = $this->_em->getRepository(CompanyFlexUltraContract::class);
344 344
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
345
-        $this->assertEquals(1, count($contracts), "There should be 1 entities related to " . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyFlexUltraContract'");
345
+        $this->assertEquals(1, count($contracts), "There should be 1 entities related to ".$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyFlexUltraContract'");
346 346
     }
347 347
 
348 348
     /**
Please login to merge, or discard this patch.