Failed Conditions
Pull Request — master (#6749)
by Konstantin
21:03 queued 11:29
created
Tests/ORM/Tools/Console/Command/GenerateRepositoriesCommandTest.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
     {
31 31
         parent::setUp();
32 32
 
33
-        $this->path = \sys_get_temp_dir() . DIRECTORY_SEPARATOR . uniqid('doctrine_');
33
+        $this->path = \sys_get_temp_dir().DIRECTORY_SEPARATOR.uniqid('doctrine_');
34 34
 
35 35
         \mkdir($this->path);
36 36
 
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
 
40 40
         $metadataDriver->addPaths(
41 41
             [
42
-            __DIR__ . '/../../../../Models/DDC3231/'
42
+            __DIR__.'/../../../../Models/DDC3231/'
43 43
             ]
44 44
         );
45 45
 
@@ -86,13 +86,13 @@  discard block
 block discarded – undo
86 86
         $this->generateRepositories('DDC3231User1');
87 87
 
88 88
         $cname = 'Doctrine\Tests\Models\DDC3231\DDC3231User1Repository';
89
-        $fname = str_replace('\\', DIRECTORY_SEPARATOR, $cname) . '.php';
89
+        $fname = str_replace('\\', DIRECTORY_SEPARATOR, $cname).'.php';
90 90
 
91
-        $this->assertFileExists($this->path . DIRECTORY_SEPARATOR . $fname);
92
-        $this->assertFileExists($this->path . DIRECTORY_SEPARATOR . 'DDC3231User1NoNamespaceRepository.php');
91
+        $this->assertFileExists($this->path.DIRECTORY_SEPARATOR.$fname);
92
+        $this->assertFileExists($this->path.DIRECTORY_SEPARATOR.'DDC3231User1NoNamespaceRepository.php');
93 93
 
94
-        require $this->path . DIRECTORY_SEPARATOR . $fname;
95
-        require $this->path . DIRECTORY_SEPARATOR . 'DDC3231User1NoNamespaceRepository.php';
94
+        require $this->path.DIRECTORY_SEPARATOR.$fname;
95
+        require $this->path.DIRECTORY_SEPARATOR.'DDC3231User1NoNamespaceRepository.php';
96 96
 
97 97
         $this->assertTrue(class_exists($cname));
98 98
         $this->assertTrue(class_exists('DDC3231User1NoNamespaceRepository'));
@@ -109,13 +109,13 @@  discard block
 block discarded – undo
109 109
         $this->generateRepositories('DDC3231User2', DDC3231EntityRepository::class);
110 110
 
111 111
         $cname = 'Doctrine\Tests\Models\DDC3231\DDC3231User2Repository';
112
-        $fname = str_replace('\\', DIRECTORY_SEPARATOR, $cname) . '.php';
112
+        $fname = str_replace('\\', DIRECTORY_SEPARATOR, $cname).'.php';
113 113
 
114
-        $this->assertFileExists($this->path . DIRECTORY_SEPARATOR . $fname);
115
-        $this->assertFileExists($this->path . DIRECTORY_SEPARATOR . 'DDC3231User2NoNamespaceRepository.php');
114
+        $this->assertFileExists($this->path.DIRECTORY_SEPARATOR.$fname);
115
+        $this->assertFileExists($this->path.DIRECTORY_SEPARATOR.'DDC3231User2NoNamespaceRepository.php');
116 116
 
117
-        require $this->path . DIRECTORY_SEPARATOR . $fname;
118
-        require $this->path . DIRECTORY_SEPARATOR . 'DDC3231User2NoNamespaceRepository.php';
117
+        require $this->path.DIRECTORY_SEPARATOR.$fname;
118
+        require $this->path.DIRECTORY_SEPARATOR.'DDC3231User2NoNamespaceRepository.php';
119 119
 
120 120
         $this->assertTrue(class_exists($cname));
121 121
         $this->assertTrue(class_exists('DDC3231User2NoNamespaceRepository'));
Please login to merge, or discard this patch.
Tests/ORM/Tools/Console/Command/ConvertDoctrine1SchemaCommandTest.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -17,8 +17,8 @@
 block discarded – undo
17 17
 
18 18
         $output = $this->createMock(OutputInterface::class);
19 19
         $output->expects($this->once())
20
-               ->method('writeln')
21
-               ->with($this->equalTo('No Metadata Classes to process.'));
20
+                ->method('writeln')
21
+                ->with($this->equalTo('No Metadata Classes to process.'));
22 22
 
23 23
         $command->convertDoctrine1Schema([], sys_get_temp_dir(), 'annotation', 4, null, $output);
24 24
     }
Please login to merge, or discard this patch.
Tests/ORM/Tools/Console/Command/ClearCacheQueryRegionCommandTest.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
             ], ['decorated' => false]
53 53
         );
54 54
 
55
-        $this->assertEquals('Clearing all second-level cache query regions' . PHP_EOL, $tester->getDisplay());
55
+        $this->assertEquals('Clearing all second-level cache query regions'.PHP_EOL, $tester->getDisplay());
56 56
     }
57 57
 
58 58
     public function testClearDefaultRegionName()
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
             ], ['decorated' => false]
67 67
         );
68 68
 
69
-        $this->assertEquals('Clearing second-level cache query region named "query_cache_region"' . PHP_EOL, $tester->getDisplay());
69
+        $this->assertEquals('Clearing second-level cache query region named "query_cache_region"'.PHP_EOL, $tester->getDisplay());
70 70
     }
71 71
 
72 72
     public function testClearByRegionName()
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
             ], ['decorated' => false]
81 81
         );
82 82
 
83
-        $this->assertEquals('Clearing second-level cache query region named "my_region"' . PHP_EOL, $tester->getDisplay());
83
+        $this->assertEquals('Clearing second-level cache query region named "my_region"'.PHP_EOL, $tester->getDisplay());
84 84
     }
85 85
 
86 86
     public function testFlushRegionName()
@@ -95,6 +95,6 @@  discard block
 block discarded – undo
95 95
             ], ['decorated' => false]
96 96
         );
97 97
 
98
-        $this->assertEquals('Flushing cache provider configured for second-level cache query region named "my_region"' . PHP_EOL, $tester->getDisplay());
98
+        $this->assertEquals('Flushing cache provider configured for second-level cache query region named "my_region"'.PHP_EOL, $tester->getDisplay());
99 99
     }
100 100
 }
Please login to merge, or discard this patch.
Tests/ORM/Tools/Console/Command/ClearCacheEntityRegionCommandTest.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
             ], ['decorated' => false]
54 54
         );
55 55
 
56
-        $this->assertEquals('Clearing all second-level cache entity regions' . PHP_EOL, $tester->getDisplay());
56
+        $this->assertEquals('Clearing all second-level cache entity regions'.PHP_EOL, $tester->getDisplay());
57 57
     }
58 58
 
59 59
     public function testClearByEntityClassName()
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
             ], ['decorated' => false]
68 68
         );
69 69
 
70
-        $this->assertEquals('Clearing second-level cache for entity "Doctrine\Tests\Models\Cache\Country"' . PHP_EOL, $tester->getDisplay());
70
+        $this->assertEquals('Clearing second-level cache for entity "Doctrine\Tests\Models\Cache\Country"'.PHP_EOL, $tester->getDisplay());
71 71
     }
72 72
 
73 73
     public function testClearCacheEntryName()
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
             ], ['decorated' => false]
83 83
         );
84 84
 
85
-        $this->assertEquals('Clearing second-level cache entry for entity "Doctrine\Tests\Models\Cache\Country" identified by "1"' . PHP_EOL, $tester->getDisplay());
85
+        $this->assertEquals('Clearing second-level cache entry for entity "Doctrine\Tests\Models\Cache\Country" identified by "1"'.PHP_EOL, $tester->getDisplay());
86 86
     }
87 87
 
88 88
     public function testFlushRegionName()
@@ -97,6 +97,6 @@  discard block
 block discarded – undo
97 97
             ], ['decorated' => false]
98 98
         );
99 99
 
100
-        $this->assertEquals('Flushing cache provider configured for entity named "Doctrine\Tests\Models\Cache\Country"' . PHP_EOL, $tester->getDisplay());
100
+        $this->assertEquals('Flushing cache provider configured for entity named "Doctrine\Tests\Models\Cache\Country"'.PHP_EOL, $tester->getDisplay());
101 101
     }
102 102
 }
Please login to merge, or discard this patch.
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.