Failed Conditions
Pull Request — master (#6593)
by Thomas
16:12
created
tests/Doctrine/Tests/ORM/Tools/SchemaValidatorTest.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -30,8 +30,8 @@
 block discarded – undo
30 30
     public function testCmsModelSet(string $path)
31 31
     {
32 32
         $this->em->getConfiguration()
33
-                 ->getMetadataDriverImpl()
34
-                 ->addPaths([$path]);
33
+                    ->getMetadataDriverImpl()
34
+                    ->addPaths([$path]);
35 35
 
36 36
         self::assertEmpty($this->validator->validateMapping());
37 37
     }
Please login to merge, or discard this patch.
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -39,12 +39,12 @@  discard block
 block discarded – undo
39 39
     public function modelSetProvider(): array
40 40
     {
41 41
         return [
42
-            'cms'        => [__DIR__ . '/../../Models/CMS'],
43
-            'company'    => [__DIR__ . '/../../Models/Company'],
44
-            'ecommerce'  => [__DIR__ . '/../../Models/ECommerce'],
45
-            'forum'      => [__DIR__ . '/../../Models/Forum'],
46
-            'navigation' => [__DIR__ . '/../../Models/Navigation'],
47
-            'routing'    => [__DIR__ . '/../../Models/Routing'],
42
+            'cms'        => [__DIR__.'/../../Models/CMS'],
43
+            'company'    => [__DIR__.'/../../Models/Company'],
44
+            'ecommerce'  => [__DIR__.'/../../Models/ECommerce'],
45
+            'forum'      => [__DIR__.'/../../Models/Forum'],
46
+            'navigation' => [__DIR__.'/../../Models/Navigation'],
47
+            'routing'    => [__DIR__.'/../../Models/Routing'],
48 48
         ];
49 49
     }
50 50
 
@@ -124,8 +124,8 @@  discard block
 block discarded – undo
124 124
 
125 125
         $this->assertEquals(
126 126
             [
127
-                "The field Doctrine\Tests\ORM\Tools\DDC3274One#two is on the inverse side of a bi-directional " .
128
-                "relationship, but the specified mappedBy association on the target-entity " .
127
+                "The field Doctrine\Tests\ORM\Tools\DDC3274One#two is on the inverse side of a bi-directional ".
128
+                "relationship, but the specified mappedBy association on the target-entity ".
129 129
                 "Doctrine\Tests\ORM\Tools\DDC3274Two#one does not contain the required 'inversedBy=\"two\"' attribute."
130 130
             ],
131 131
             $ce
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
 
143 143
         $this->assertEquals(
144 144
             [
145
-                "The association Doctrine\Tests\ORM\Tools\DDC3322One#invalidAssoc is ordered by a foreign field " .
145
+                "The association Doctrine\Tests\ORM\Tools\DDC3322One#invalidAssoc is ordered by a foreign field ".
146 146
                 "invalidField that is not a field on the target entity Doctrine\Tests\ORM\Tools\DDC3322ValidEntity1."
147 147
             ],
148 148
             $ce
@@ -159,7 +159,7 @@  discard block
 block discarded – undo
159 159
 
160 160
         $this->assertEquals(
161 161
             [
162
-                "The association Doctrine\Tests\ORM\Tools\DDC3322Two#invalidAssoc is ordered by a field oneToMany " .
162
+                "The association Doctrine\Tests\ORM\Tools\DDC3322Two#invalidAssoc is ordered by a field oneToMany ".
163 163
                 "on Doctrine\Tests\ORM\Tools\DDC3322ValidEntity1 that is a collection-valued association."
164 164
             ],
165 165
             $ce
@@ -176,7 +176,7 @@  discard block
 block discarded – undo
176 176
 
177 177
         $this->assertEquals(
178 178
             [
179
-                "The association Doctrine\Tests\ORM\Tools\DDC3322Three#invalidAssoc is ordered by a field oneToOneInverse " .
179
+                "The association Doctrine\Tests\ORM\Tools\DDC3322Three#invalidAssoc is ordered by a field oneToOneInverse ".
180 180
                 "on Doctrine\Tests\ORM\Tools\DDC3322ValidEntity1 that is the inverse side of an association."
181 181
             ],
182 182
             $ce
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Tools/SetupTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
         $loaders = spl_autoload_functions();
38 38
         $numberOfLoaders = count($loaders);
39 39
         for ($i = 0; $i < $numberOfLoaders; $i++) {
40
-            if ($i > $this->originalAutoloaderCount+1) {
40
+            if ($i > $this->originalAutoloaderCount + 1) {
41 41
                 spl_autoload_unregister($loaders[$i]);
42 42
             }
43 43
         }
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
 
46 46
     public function testDirectoryAutoload()
47 47
     {
48
-        Setup::registerAutoloadDirectory(__DIR__ . "/../../../../../vendor/doctrine/common/lib");
48
+        Setup::registerAutoloadDirectory(__DIR__."/../../../../../vendor/doctrine/common/lib");
49 49
 
50 50
         $this->assertEquals($this->originalAutoloaderCount + 2, count(spl_autoload_functions()));
51 51
     }
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Tools/EntityRepositoryGeneratorTest.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
     public function setUp()
33 33
     {
34 34
         $this->_namespace   = uniqid('doctrine_');
35
-        $this->_tmpDir      = \sys_get_temp_dir() . DIRECTORY_SEPARATOR . $this->_namespace;
35
+        $this->_tmpDir      = \sys_get_temp_dir().DIRECTORY_SEPARATOR.$this->_namespace;
36 36
         \mkdir($this->_tmpDir);
37 37
 
38 38
         $this->_generator = new EntityGenerator();
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
         $em = $this->_getTestEntityManager();
79 79
         $ns = $this->_namespace;
80 80
 
81
-        $className = $ns . '\DDC3231User1Tmp';
81
+        $className = $ns.'\DDC3231User1Tmp';
82 82
         $this->writeEntityClass(DDC3231User1::class, $className);
83 83
 
84 84
         $rpath = $this->writeRepositoryClass($className);
@@ -90,10 +90,10 @@  discard block
 block discarded – undo
90 90
         $repo = new \ReflectionClass($em->getRepository($className));
91 91
 
92 92
         $this->assertTrue($repo->inNamespace());
93
-        $this->assertSame($className . 'Repository', $repo->getName());
93
+        $this->assertSame($className.'Repository', $repo->getName());
94 94
         $this->assertSame(EntityRepository::class, $repo->getParentClass()->getName());
95 95
 
96
-        require_once __DIR__ . '/../../Models/DDC3231/DDC3231User1NoNamespace.php';
96
+        require_once __DIR__.'/../../Models/DDC3231/DDC3231User1NoNamespace.php';
97 97
 
98 98
         $className2 = 'DDC3231User1NoNamespaceTmp';
99 99
         $this->writeEntityClass(\DDC3231User1NoNamespace::class, $className2);
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
         $repo2 = new \ReflectionClass($em->getRepository($className2));
108 108
 
109 109
         $this->assertFalse($repo2->inNamespace());
110
-        $this->assertSame($className2 . 'Repository', $repo2->getName());
110
+        $this->assertSame($className2.'Repository', $repo2->getName());
111 111
         $this->assertSame(EntityRepository::class, $repo2->getParentClass()->getName());
112 112
     }
113 113
 
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
         $em = $this->_getTestEntityManager();
120 120
         $ns = $this->_namespace;
121 121
 
122
-        $className = $ns . '\DDC3231User2Tmp';
122
+        $className = $ns.'\DDC3231User2Tmp';
123 123
         $this->writeEntityClass(DDC3231User2::class, $className);
124 124
 
125 125
         $rpath = $this->writeRepositoryClass($className, DDC3231EntityRepository::class);
@@ -132,11 +132,11 @@  discard block
 block discarded – undo
132 132
         $repo = new \ReflectionClass($em->getRepository($className));
133 133
 
134 134
         $this->assertTrue($repo->inNamespace());
135
-        $this->assertSame($className . 'Repository', $repo->getName());
135
+        $this->assertSame($className.'Repository', $repo->getName());
136 136
         $this->assertSame(DDC3231EntityRepository::class, $repo->getParentClass()->getName());
137 137
 
138 138
 
139
-        require_once __DIR__ . '/../../Models/DDC3231/DDC3231User2NoNamespace.php';
139
+        require_once __DIR__.'/../../Models/DDC3231/DDC3231User2NoNamespace.php';
140 140
 
141 141
         $className2 = 'DDC3231User2NoNamespaceTmp';
142 142
         $this->writeEntityClass('DDC3231User2NoNamespace', $className2);
@@ -151,7 +151,7 @@  discard block
 block discarded – undo
151 151
         $repo2 = new \ReflectionClass($em->getRepository($className2));
152 152
 
153 153
         $this->assertFalse($repo2->inNamespace());
154
-        $this->assertSame($className2 . 'Repository', $repo2->getName());
154
+        $this->assertSame($className2.'Repository', $repo2->getName());
155 155
         $this->assertSame(DDC3231EntityRepository::class, $repo2->getParentClass()->getName());
156 156
     }
157 157
 
@@ -169,11 +169,11 @@  discard block
 block discarded – undo
169 169
         $metadata               = $cmf->getMetadataFor($className);
170 170
         $metadata->namespace    = $this->_namespace;
171 171
         $metadata->name         = $newClassName;
172
-        $metadata->customRepositoryClassName = $newClassName . "Repository";
172
+        $metadata->customRepositoryClassName = $newClassName."Repository";
173 173
 
174 174
         $this->_generator->writeEntityClass($metadata, $this->_tmpDir);
175 175
 
176
-        require $this->_tmpDir . DIRECTORY_SEPARATOR . str_replace('\\', DIRECTORY_SEPARATOR, $newClassName) . ".php";
176
+        require $this->_tmpDir.DIRECTORY_SEPARATOR.str_replace('\\', DIRECTORY_SEPARATOR, $newClassName).".php";
177 177
     }
178 178
 
179 179
     /**
@@ -185,9 +185,9 @@  discard block
 block discarded – undo
185 185
     {
186 186
         $this->_repositoryGenerator->setDefaultRepositoryName($defaultRepository);
187 187
 
188
-        $this->_repositoryGenerator->writeEntityRepositoryClass($className . 'Repository', $this->_tmpDir);
188
+        $this->_repositoryGenerator->writeEntityRepositoryClass($className.'Repository', $this->_tmpDir);
189 189
 
190
-        return $this->_tmpDir . DIRECTORY_SEPARATOR . str_replace('\\', DIRECTORY_SEPARATOR, $className) . 'Repository.php';
190
+        return $this->_tmpDir.DIRECTORY_SEPARATOR.str_replace('\\', DIRECTORY_SEPARATOR, $className).'Repository.php';
191 191
     }
192 192
 
193 193
 }
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Tools/Export/YamlClassMetadataExporterTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@
 block discarded – undo
16 16
 {
17 17
     protected function _getType()
18 18
     {
19
-        if (!class_exists('Symfony\Component\Yaml\Yaml', true)) {
19
+        if ( ! class_exists('Symfony\Component\Yaml\Yaml', true)) {
20 20
             $this->markTestSkipped('Please install Symfony YAML Component into the include path of your PHP installation.');
21 21
         }
22 22
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Tools/Pagination/LimitSubqueryWalkerTest.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -72,7 +72,7 @@
 block discarded – undo
72 72
     /**
73 73
      * Arbitrary Join
74 74
      */
75
-     public function testLimitSubqueryWithArbitraryJoin()
75
+        public function testLimitSubqueryWithArbitraryJoin()
76 76
     {
77 77
         $dql        = 'SELECT p, c FROM Doctrine\Tests\ORM\Tools\Pagination\MyBlogPost p JOIN Doctrine\Tests\ORM\Tools\Pagination\Category c WITH p.category = c';
78 78
         $query      = $this->entityManager->createQuery($dql);
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Tools/Pagination/LimitSubqueryOutputWalkerTest.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -84,7 +84,7 @@
 block discarded – undo
84 84
         $this->entityManager->getConnection()->setDatabasePlatform(new PostgreSqlPlatform);
85 85
 
86 86
         $query = $this->entityManager->createQuery(
87
-           'SELECT u, g, COUNT(g.id) AS hidden g_quantity FROM Doctrine\Tests\ORM\Tools\Pagination\User u JOIN u.groups g ORDER BY g_quantity, u.id DESC'
87
+            'SELECT u, g, COUNT(g.id) AS hidden g_quantity FROM Doctrine\Tests\ORM\Tools\Pagination\User u JOIN u.groups g ORDER BY g_quantity, u.id DESC'
88 88
         );
89 89
         $limitQuery = clone $query;
90 90
         $limitQuery->setHint(Query::HINT_CUSTOM_OUTPUT_WALKER, LimitSubqueryOutputWalker::class);
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Tools/Pagination/WhereInWalkerTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -127,7 +127,7 @@
 block discarded – undo
127 127
      */
128 128
     public function testWhereInQueryWithArbitraryJoin_NoWhere()
129 129
     {
130
-        $whereInQuery  = $this->entityManager->createQuery(
130
+        $whereInQuery = $this->entityManager->createQuery(
131 131
             'SELECT p FROM Doctrine\Tests\ORM\Tools\Pagination\BlogPost p JOIN Doctrine\Tests\ORM\Tools\Pagination\Category c WITH p.category = c'
132 132
         );
133 133
         $whereInQuery->setHint(Query::HINT_CUSTOM_TREE_WALKERS, [WhereInWalker::class]);
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Cache/DefaultQueryCacheTest.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
             $result[] = $city;
135 135
 
136 136
             $cityClass->setFieldValue($city, 'id', $i);
137
-            $stateClass->setFieldValue($state, 'id', $i*2);
137
+            $stateClass->setFieldValue($state, 'id', $i * 2);
138 138
 
139 139
             $uow->registerManaged($state, ['id' => $state->getId()], ['name' => $city->getName()]);
140 140
             $uow->registerManaged($city, ['id' => $city->getId()], ['name' => $city->getName(), 'state' => $state]);
@@ -179,8 +179,8 @@  discard block
 block discarded – undo
179 179
             $result[] = $city;
180 180
 
181 181
             $cityClass->setFieldValue($city, 'id', $i);
182
-            $stateClass->setFieldValue($state, 'id', $i*2);
183
-            $countryClass->setFieldValue($country, 'id', $i*3);
182
+            $stateClass->setFieldValue($state, 'id', $i * 2);
183
+            $countryClass->setFieldValue($country, 'id', $i * 3);
184 184
 
185 185
             $uow->registerManaged($country, ['id' => $country->getId()], ['name' => $country->getName()]);
186 186
             $uow->registerManaged($state, ['id' => $state->getId()], ['name' => $state->getName(), 'country' => $country]
@@ -364,7 +364,7 @@  discard block
 block discarded – undo
364 364
         $uow->registerManaged($state, ['id' => $state->getId()], ['name' => $city->getName()]);
365 365
         $uow->registerManaged($city, ['id' => $city->getId()], ['name' => $city->getName(), 'state' => $state]);
366 366
 
367
-        $this->region->addReturn('put', true);  // put root entity
367
+        $this->region->addReturn('put', true); // put root entity
368 368
         $this->region->addReturn('put', false); // association fails
369 369
 
370 370
         $this->assertFalse($this->queryCache->put($key, $rsm, $result));
@@ -399,7 +399,7 @@  discard block
 block discarded – undo
399 399
         $uow->registerManaged($state, ['id' => $state->getId()], ['name' => $state->getName(), 'cities' => $state->getCities()]
400 400
         );
401 401
 
402
-        $this->region->addReturn('put', true);  // put root entity
402
+        $this->region->addReturn('put', true); // put root entity
403 403
         $this->region->addReturn('put', false); // collection association fails
404 404
 
405 405
         $this->assertFalse($this->queryCache->put($key, $rsm, $result));
@@ -564,7 +564,7 @@  discard block
 block discarded – undo
564 564
         $attractions = $reflection->invoke($this->queryCache, $rsm, 'a', $bavaria);
565 565
 
566 566
         $this->assertCount(2, $cities);
567
-        $this->assertCount(2,  $attractions);
567
+        $this->assertCount(2, $attractions);
568 568
 
569 569
         $this->assertInstanceOf(Collection::class, $cities);
570 570
         $this->assertInstanceOf(Collection::class, $attractions[0]);
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Cache/FileLockRegionTest.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
 
53 53
     protected function createRegion()
54 54
     {
55
-        $this->directory = sys_get_temp_dir() . '/doctrine_lock_'. uniqid();
55
+        $this->directory = sys_get_temp_dir().'/doctrine_lock_'.uniqid();
56 56
 
57 57
         $region = new DefaultRegion('concurren_region_test', $this->cache);
58 58
 
@@ -100,10 +100,10 @@  discard block
 block discarded – undo
100 100
 
101 101
         file_put_contents($file, 'foo');
102 102
         $this->assertFileExists($file);
103
-        $this->assertEquals('foo' , file_get_contents($file));
103
+        $this->assertEquals('foo', file_get_contents($file));
104 104
 
105 105
         $this->assertNull($this->region->lock($key));
106
-        $this->assertEquals('foo' , file_get_contents($file));
106
+        $this->assertEquals('foo', file_get_contents($file));
107 107
         $this->assertFileExists($file);
108 108
 
109 109
         // should be not available
@@ -128,11 +128,11 @@  discard block
 block discarded – undo
128 128
         // change the lock
129 129
         file_put_contents($file, 'foo');
130 130
         $this->assertFileExists($file);
131
-        $this->assertEquals('foo' , file_get_contents($file));
131
+        $this->assertEquals('foo', file_get_contents($file));
132 132
 
133 133
         //try to unlock
134 134
         $this->assertFalse($this->region->unlock($key, $lock));
135
-        $this->assertEquals('foo' , file_get_contents($file));
135
+        $this->assertEquals('foo', file_get_contents($file));
136 136
         $this->assertFileExists($file);
137 137
 
138 138
         // should be not available
@@ -153,14 +153,14 @@  discard block
 block discarded – undo
153 153
         // create lock
154 154
         file_put_contents($file, 'foo');
155 155
         $this->assertFileExists($file);
156
-        $this->assertEquals('foo' , file_get_contents($file));
156
+        $this->assertEquals('foo', file_get_contents($file));
157 157
 
158 158
         $this->assertFalse($this->region->contains($key));
159 159
         $this->assertFalse($this->region->put($key, $entry));
160 160
         $this->assertFalse($this->region->contains($key));
161 161
 
162 162
         $this->assertFileExists($file);
163
-        $this->assertEquals('foo' , file_get_contents($file));
163
+        $this->assertEquals('foo', file_get_contents($file));
164 164
     }
165 165
 
166 166
     public function testLockedEvict()
@@ -255,7 +255,7 @@  discard block
 block discarded – undo
255 255
         $reflectionDirectory->setAccessible(true);
256 256
         $reflectionDirectory->setValue($region, str_repeat('a', 10000));
257 257
 
258
-        set_error_handler(function () {}, E_WARNING);
258
+        set_error_handler(function() {}, E_WARNING);
259 259
         $this->assertTrue($region->evictAll());
260 260
         restore_error_handler();
261 261
     }
Please login to merge, or discard this patch.