View GitHub Repository framework

Schedule Inspection
Subscribe
Test Failed
Pull Request — master (#1190)
by butschster
10:27
created
Status
Category
Toggle Patches
src/Files/tests/DirectoriesTest.php 2 patches
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch 
@@ -25,7 +25,7 @@  discard block
 
 block discarded – undo
25 25 
     public function testEnsureDirectory(): void
26 26 
     {
27 27 
         $files = new Files();
28 
-        $directory = self::FIXTURE_DIRECTORY . 'directory/';
28 
+        $directory = self::FIXTURE_DIRECTORY.'directory/';
29 29
30 30 
         self::assertFalse($files->exists($directory));
31 31 
         self::assertFalse($files->isDirectory($directory));
 
@@ -39,7 +39,7 @@  discard block
 
 block discarded – undo
39 39 
     public function testEnsureExistedDirectory(): void
40 40 
     {
41 41 
         $files = new Files();
42 
-        $directory = self::FIXTURE_DIRECTORY . 'directory/';
42 
+        $directory = self::FIXTURE_DIRECTORY.'directory/';
43 43
44 44 
         self::assertFalse($files->exists($directory));
45 45 
         self::assertFalse($files->isDirectory($directory));
 
@@ -58,7 +58,7 @@  discard block
 
 block discarded – undo
58 58 
     public function testEnsureNestedDirectory(): void
59 59 
     {
60 60 
         $files = new Files();
61 
-        $directory = self::FIXTURE_DIRECTORY . 'directory/sub/other';
61 
+        $directory = self::FIXTURE_DIRECTORY.'directory/sub/other';
62 62
63 63 
         self::assertFalse($files->exists($directory));
64 64 
         self::assertFalse($files->isDirectory($directory));
 
@@ -72,7 +72,7 @@  discard block
 
 block discarded – undo
72 72 
     public function testEnsureNestedDirectoryNoRecursivePermissions(): void
73 73 
     {
74 74 
         $files = new Files();
75 
-        $directory = self::FIXTURE_DIRECTORY . 'directory/sub/other';
75 
+        $directory = self::FIXTURE_DIRECTORY.'directory/sub/other';
76 76
77 77 
         self::assertFalse($files->exists($directory));
78 78 
         self::assertFalse($files->isDirectory($directory));
 
@@ -86,14 +86,14 @@  discard block
 
 block discarded – undo
86 86 
     public function testEnsureExistedNestedDirectory(): void
87 87 
     {
88 88 
         $files = new Files();
89 
-        $directory = self::FIXTURE_DIRECTORY . 'directory/sub/other';
89 
+        $directory = self::FIXTURE_DIRECTORY.'directory/sub/other';
90 90
91 91 
         self::assertFalse($files->exists($directory));
92 92 
         self::assertFalse($files->isDirectory($directory));
93 93
94 
-        mkdir(self::FIXTURE_DIRECTORY . 'directory');
95 
-        mkdir(self::FIXTURE_DIRECTORY . 'directory/sub');
96 
-        mkdir(self::FIXTURE_DIRECTORY . 'directory/sub/other');
94 
+        mkdir(self::FIXTURE_DIRECTORY.'directory');
95 
+        mkdir(self::FIXTURE_DIRECTORY.'directory/sub');
96 
+        mkdir(self::FIXTURE_DIRECTORY.'directory/sub/other');
97 97
98 98 
         self::assertTrue($files->exists($directory));
99 99 
         self::assertTrue($files->isDirectory($directory));
 
@@ -107,13 +107,13 @@  discard block
 
 block discarded – undo
107 107 
     public function testDeleteDirectoryContent(): void
108 108 
     {
109 109 
         $files = new Files();
110 
-        $baseDirectory = self::FIXTURE_DIRECTORY . 'directory/';
111 
-        $directory = $baseDirectory . 'sub/other';
110 
+        $baseDirectory = self::FIXTURE_DIRECTORY.'directory/';
111 
+        $directory = $baseDirectory.'sub/other';
112 112
113 113 
         $filenames = [
114 
-            $baseDirectory . 'test.file',
115 
-            $directory . 'other.file',
116 
-            $directory . '.sample',
114 
+            $baseDirectory.'test.file',
115 
+            $directory.'other.file',
116 
+            $directory.'.sample',
117 117 
         ];
118 118
119 119 
         self::assertFalse($files->exists($baseDirectory));
 
@@ -130,7 +130,7 @@  discard block
 
 block discarded – undo
130 130 
         self::assertTrue($files->exists($directory));
131 131 
         self::assertTrue($files->isDirectory($directory));
132 132
133 
-        foreach ($filenames as $filename) {
133 
+        foreach ($filenames as $filename){
134 134 
             self::assertFalse($files->exists($filename));
135 135 
             $files->write($filename, 'random-data');
136 136 
             self::assertTrue($files->exists($filename));
 
@@ -144,7 +144,7 @@  discard block
 
 block discarded – undo
144 144 
         self::assertFalse($files->exists($directory));
145 145 
         self::assertFalse($files->isDirectory($directory));
146 146
147 
-        foreach ($filenames as $filename) {
147 
+        foreach ($filenames as $filename){
148 148 
             self::assertFalse($files->exists($filename));
149 149 
         }
150 150 
     }
 
@@ -152,13 +152,13 @@  discard block
 
 block discarded – undo
152 152 
     public function testDeleteDirectory(): void
153 153 
     {
154 154 
         $files = new Files();
155 
-        $baseDirectory = self::FIXTURE_DIRECTORY . 'directory/';
156 
-        $directory = $baseDirectory . 'sub/other';
155 
+        $baseDirectory = self::FIXTURE_DIRECTORY.'directory/';
156 
+        $directory = $baseDirectory.'sub/other';
157 157
158 158 
         $filenames = [
159 
-            $baseDirectory . 'test.file',
160 
-            $directory . 'other.file',
161 
-            $directory . '.sample',
159 
+            $baseDirectory.'test.file',
160 
+            $directory.'other.file',
161 
+            $directory.'.sample',
162 162 
         ];
163 163
164 164 
         self::assertFalse($files->exists($baseDirectory));
 
@@ -175,7 +175,7 @@  discard block
 
 block discarded – undo
175 175 
         self::assertTrue($files->exists($directory));
176 176 
         self::assertTrue($files->isDirectory($directory));
177 177
178 
-        foreach ($filenames as $filename) {
178 
+        foreach ($filenames as $filename){
179 179 
             self::assertFalse($files->exists($filename));
180 180 
             $files->write($filename, 'random-data');
181 181 
             self::assertTrue($files->exists($filename));
 
@@ -189,7 +189,7 @@  discard block
 
 block discarded – undo
189 189 
         self::assertFalse($files->exists($directory));
190 190 
         self::assertFalse($files->isDirectory($directory));
191 191
192 
-        foreach ($filenames as $filename) {
192 
+        foreach ($filenames as $filename){
193 193 
             self::assertFalse($files->exists($filename));
194 194 
         }
195 195 
     }
 
@@ -200,8 +200,8 @@  discard block
 
 block discarded – undo
200 200
201 201 
         $files = new Files();
202 202
203 
-        $files->write(self::FIXTURE_DIRECTORY . 'test', 'data');
204 
-        $files->deleteDirectory(self::FIXTURE_DIRECTORY . 'test');
203 
+        $files->write(self::FIXTURE_DIRECTORY.'test', 'data');
204 
+        $files->deleteDirectory(self::FIXTURE_DIRECTORY.'test');
205 205 
     }
206 206
207 207 
     public function testGetFiles(): void
Please login to merge, or discard this patch.
Braces   +8 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -130,7 +130,8 @@  discard block
 
 block discarded – undo
130 130 
         self::assertTrue($files->exists($directory));
131 131 
         self::assertTrue($files->isDirectory($directory));
132 132
133 
-        foreach ($filenames as $filename) {
133 
+        foreach ($filenames as $filename)
134 
+        {
134 135 
             self::assertFalse($files->exists($filename));
135 136 
             $files->write($filename, 'random-data');
136 137 
             self::assertTrue($files->exists($filename));
 
@@ -144,7 +145,8 @@  discard block
 
 block discarded – undo
144 145 
         self::assertFalse($files->exists($directory));
145 146 
         self::assertFalse($files->isDirectory($directory));
146 147
147 
-        foreach ($filenames as $filename) {
148 
+        foreach ($filenames as $filename)
149 
+        {
148 150 
             self::assertFalse($files->exists($filename));
149 151 
         }
150 152 
     }
 
@@ -175,7 +177,8 @@  discard block
 
 block discarded – undo
175 177 
         self::assertTrue($files->exists($directory));
176 178 
         self::assertTrue($files->isDirectory($directory));
177 179
178 
-        foreach ($filenames as $filename) {
180 
+        foreach ($filenames as $filename)
181 
+        {
179 182 
             self::assertFalse($files->exists($filename));
180 183 
             $files->write($filename, 'random-data');
181 184 
             self::assertTrue($files->exists($filename));
 
@@ -189,7 +192,8 @@  discard block
 
 block discarded – undo
189 192 
         self::assertFalse($files->exists($directory));
190 193 
         self::assertFalse($files->isDirectory($directory));
191 194
192 
-        foreach ($filenames as $filename) {
195 
+        foreach ($filenames as $filename)
196 
+        {
193 197 
             self::assertFalse($files->exists($filename));
194 198 
         }
195 199 
     }
Please login to merge, or discard this patch.
src/Files/tests/IOTest.php 2 patches
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch 
@@ -27,7 +27,7 @@  discard block
 
 block discarded – undo
27 27 
     {
28 28 
         $files = new Files();
29 29
30 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
30 
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
31 31 
         self::assertFalse($files->exists($filename));
32 32
33 33 
         $files->write($filename, 'some-data');
 
@@ -40,8 +40,8 @@  discard block
 
 block discarded – undo
40 40 
     {
41 41 
         $files = new Files();
42 42
43 
-        $directory = self::FIXTURE_DIRECTORY . '/directory/abc/';
44 
-        $filename = $directory . 'test.txt';
43 
+        $directory = self::FIXTURE_DIRECTORY.'/directory/abc/';
44 
+        $filename = $directory.'test.txt';
45 45
46 46 
         self::assertFalse($files->exists($directory));
47 47 
         self::assertFalse($files->exists($filename));
 
@@ -59,7 +59,7 @@  discard block
 
 block discarded – undo
59 59 
     {
60 60 
         $files = new Files();
61 61
62 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
62 
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
63 63 
         self::assertFalse($files->exists($filename));
64 64
65 65 
         $files->write($filename, 'some-data');
 
@@ -75,7 +75,7 @@  discard block
 
 block discarded – undo
75 75
76 76 
         $files = new Files();
77 77
78 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
78 
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
79 79 
         self::assertFalse($files->exists($filename));
80 80
81 81 
         $files->read($filename);
 
@@ -94,14 +94,14 @@  discard block
 
 block discarded – undo
94 94 
         $this->expectException(FileNotFoundException::class);
95 95
96 96 
         $files = new Files();
97 
-        $files->getPermissions(self::FIXTURE_DIRECTORY . '/missing');
97 
+        $files->getPermissions(self::FIXTURE_DIRECTORY.'/missing');
98 98 
     }
99 99
100 100 
     public function testAppend(): void
101 101 
     {
102 102 
         $files = new Files();
103 103
104 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
104 
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
105 105 
         self::assertFalse($files->exists($filename));
106 106
107 107 
         $files->append($filename, 'some-data');
 
@@ -117,8 +117,8 @@  discard block
 
 block discarded – undo
117 117 
     {
118 118 
         $files = new Files();
119 119
120 
-        $directory = self::FIXTURE_DIRECTORY . '/directory/abc/';
121 
-        $filename = $directory . 'test.txt';
120 
+        $directory = self::FIXTURE_DIRECTORY.'/directory/abc/';
121 
+        $filename = $directory.'test.txt';
122 122
123 123 
         self::assertFalse($files->exists($directory));
124 124 
         self::assertFalse($files->exists($filename));
 
@@ -139,7 +139,7 @@  discard block
 
 block discarded – undo
139 139 
     {
140 140 
         $files = new Files();
141 141
142 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
142 
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
143 143
144 144 
         self::assertFalse($files->exists($filename));
145 145 
         $files->touch($filename);
 
@@ -149,7 +149,7 @@  discard block
 
 block discarded – undo
149 149 
     public function testDelete(): void
150 150 
     {
151 151 
         $files = new Files();
152 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
152 
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
153 153
154 154 
         self::assertFalse($files->exists($filename));
155 155 
 
@@ -163,7 +163,7 @@  discard block
 
 block discarded – undo
163 163 
     public function testDeleteMissingFile(): void
164 164 
     {
165 165 
         $files = new Files();
166 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
166 
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
167 167
168 168 
         self::assertFalse($files->exists($filename));
169 169 
         $files->delete($filename);
 
@@ -172,8 +172,8 @@  discard block
 
 block discarded – undo
172 172 
     public function testCopy(): void
173 173 
     {
174 174 
         $files = new Files();
175 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
176 
-        $destination = self::FIXTURE_DIRECTORY . '/new.txt';
175 
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
176 
+        $destination = self::FIXTURE_DIRECTORY.'/new.txt';
177 177
178 178 
         self::assertFalse($files->exists($filename));
179 179 
         $files->write($filename, 'some-data');
 
@@ -196,8 +196,8 @@  discard block
 
 block discarded – undo
196 196 
         $this->expectException(FileNotFoundException::class);
197 197
198 198 
         $files = new Files();
199 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
200 
-        $destination = self::FIXTURE_DIRECTORY . '/new.txt';
199 
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
200 
+        $destination = self::FIXTURE_DIRECTORY.'/new.txt';
201 201
202 202 
         self::assertFalse($files->exists($filename));
203 203 
         $files->copy($filename, $destination);
 
@@ -206,8 +206,8 @@  discard block
 
 block discarded – undo
206 206 
     public function testMove(): void
207 207 
     {
208 208 
         $files = new Files();
209 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
210 
-        $destination = self::FIXTURE_DIRECTORY . '/new.txt';
209 
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
210 
+        $destination = self::FIXTURE_DIRECTORY.'/new.txt';
211 211
212 212 
         self::assertFalse($files->exists($filename));
213 213 
         $files->write($filename, 'some-data');
 
@@ -230,8 +230,8 @@  discard block
 
 block discarded – undo
230 230 
         $this->expectException(FileNotFoundException::class);
231 231
232 232 
         $files = new Files();
233 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
234 
-        $destination = self::FIXTURE_DIRECTORY . '/new.txt';
233 
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
234 
+        $destination = self::FIXTURE_DIRECTORY.'/new.txt';
235 235
236 236 
         self::assertFalse($files->exists($filename));
237 237 
         $files->move($filename, $destination);
Please login to merge, or discard this patch.
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -48,7 +48,7 @@
 
 block discarded – undo
48 48
49 49 
         self::assertFalse($files->isDirectory($directory));
50 50
51 
-        $files->write($filename, 'some-data', FilesInterface::READONLY, true);
51 
+        $files->write($filename, 'some-data', FilesInterface::readonly, true);
52 52
53 53 
         self::assertTrue($files->isDirectory($directory));
54 54 
         self::assertTrue($files->exists($filename));
Please login to merge, or discard this patch.
src/Router/tests/Loader/Configurator/RoutingConfiguratorTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -17,7 +17,7 @@  discard block
 
 block discarded – undo
17 17
18 18 
         self::assertCount(0, $routes->getCollection());
19 19
20 
-        $routes->import(\dirname(__DIR__, 2) . '/Fixtures/file.php');
20 
+        $routes->import(\dirname(__DIR__, 2).'/Fixtures/file.php');
21 21
22 22 
         self::assertCount(3, $routes->getCollection());
23 23 
     }
 
@@ -28,7 +28,7 @@  discard block
 
 block discarded – undo
28 28
29 29 
         self::assertCount(0, $routes->getCollection());
30 30
31 
-        $routes->import(\dirname(__DIR__, 2) . '/Fixtures/file.php', 'php');
31 
+        $routes->import(\dirname(__DIR__, 2).'/Fixtures/file.php', 'php');
32 32
33 33 
         self::assertCount(3, $routes->getCollection());
34 34 
     }
 
@@ -39,7 +39,7 @@  discard block
 
 block discarded – undo
39 39
40 40 
         self::assertCount(0, $routes->getCollection());
41 41
42 
-        $routes->import(\dirname(__DIR__, 2) . '/Fixtures/file.php', 'yaml');
42 
+        $routes->import(\dirname(__DIR__, 2).'/Fixtures/file.php', 'yaml');
43 43
44 44 
         self::assertCount(0, $routes->getCollection());
45 45 
     }
Please login to merge, or discard this patch.
src/Router/tests/Loader/PhpFileLoaderTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -29,12 +29,12 @@
 
 block discarded – undo
29 29 
     {
30 30 
         $loader = new PhpFileLoader($this->container, $this->container);
31 31
32 
-        $routes = $loader->load(\dirname(__DIR__) . '/Fixtures/file.php');
32 
+        $routes = $loader->load(\dirname(__DIR__).'/Fixtures/file.php');
33 33 
         self::assertInstanceOf(RouteCollection::class, $routes);
34 34 
         self::assertCount(3, $routes);
35 35
36 36 
         $this->expectException(LoaderLoadException::class);
37 
-        $loader->load(\dirname(__DIR__) . '/Fixtures/unknown.php');
37 
+        $loader->load(\dirname(__DIR__).'/Fixtures/unknown.php');
38 38 
     }
39 39
40 40 
     public function testSupports(): void
Please login to merge, or discard this patch.
src/Router/tests/RouterTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -54,7 +54,7 @@  discard block
 
 block discarded – undo
54 54 
         $dispatcher
55 55 
             ->expects(self::exactly(2))
56 56 
             ->method('dispatch')
57 
-            ->with($this->callback(static fn (Routing|RouteMatched $event): bool => $event->request instanceof ServerRequest));
57 
+            ->with($this->callback(static fn (Routing | RouteMatched $event): bool => $event->request instanceof ServerRequest));
58 58
59 59 
         $router = $this->makeRouter('', $dispatcher);
60 60 
         $router->setDefault($route);
 
@@ -90,7 +90,7 @@  discard block
 
 block discarded – undo
90 90 
         $router->import($configurator);
91 91 
         $this->getContainer()->get(GroupRegistry::class)->registerRoutes($router);
92 92
93 
-        $uri = (string) $router->uri('foo', ['host' => 'some']);
93 
+        $uri = (string)$router->uri('foo', ['host' => 'some']);
94 94 
         self::assertSame('some/register', $uri);
95 95 
         self::assertFalse(\str_contains('https://host.com', $uri));
96 96 
     }
Please login to merge, or discard this patch.
src/Router/tests/SingleActionTest.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -96,9 +96,9 @@
 
 block discarded – undo
96 96 
         self::assertSame('echoed', (string)$response->getBody());
97 97
98 98 
         $e = null;
99 
-        try {
99 
+        try{
100 100 
             $router->handle(new ServerRequest('GET', new Uri('/test')));
101 
-        } catch (UndefinedRouteException $e) {
101 
+        }catch (UndefinedRouteException $e){
102 102 
         }
103 103
104 104 
         self::assertNotNull($e, 'Autofill not fired');
Please login to merge, or discard this patch.
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -96,9 +96,12 @@
 
 block discarded – undo
96 96 
         self::assertSame('echoed', (string)$response->getBody());
97 97
98 98 
         $e = null;
99 
-        try {
99 
+        try
100 
+        {
100 101 
             $router->handle(new ServerRequest('GET', new Uri('/test')));
101 
-        } catch (UndefinedRouteException $e) {
102 
+        }
103 
+        catch (UndefinedRouteException $e)
104 
+        {
102 105 
         }
103 106
104 107 
         self::assertNotNull($e, 'Autofill not fired');
Please login to merge, or discard this patch.
src/Router/tests/RouteGroupTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -150,7 +150,7 @@
 
 block discarded – undo
150 150 
         $route = $this->router->getRoute('name');
151 151 
         self::assertNotNull($route->match(new ServerRequest('GET', '/api/blog')));
152 152
153 
-        self::assertSame('/api/blog', (string) $route->uri());
153 
+        self::assertSame('/api/blog', (string)$route->uri());
154 154 
     }
155 155
156 156 
     public static function routePrefixDataProvider(): iterable
Please login to merge, or discard this patch.
src/Stempler/tests/TraverserTest.php 2 patches
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch 
@@ -32,10 +32,10 @@  discard block
 
 block discarded – undo
32 32 
         $doc = $this->parse('<a href="url"></a>');
33 33
34 34 
         $t = new Traverser();
35 
-        $t->addVisitor(new class() implements VisitorInterface {
35 
+        $t->addVisitor(new class() implements VisitorInterface{
36 36 
             public function enterNode(mixed $node, VisitorContext $ctx): mixed
37 37 
             {
38 
-                if ($node instanceof Tag && $node->name === 'a') {
38 
+                if ($node instanceof Tag && $node->name === 'a'){
39 39 
                     $node->name = 'b';
40 40 
                 }
41 41 
 
@@ -58,7 +58,7 @@  discard block
 
 block discarded – undo
58 58 
         $doc = $this->parse('<a href="url"></a>');
59 59
60 60 
         $t = new Traverser();
61 
-        $t->addVisitor(new class() implements VisitorInterface {
61 
+        $t->addVisitor(new class() implements VisitorInterface{
62 62 
             public function enterNode(mixed $node, VisitorContext $ctx): mixed
63 63 
             {
64 64 
                 return null;
 
@@ -66,7 +66,7 @@  discard block
 
 block discarded – undo
66 66
67 67 
             public function leaveNode(mixed $node, VisitorContext $ctx): mixed
68 68 
             {
69 
-                if ($node instanceof Tag && $node->name === 'a') {
69 
+                if ($node instanceof Tag && $node->name === 'a'){
70 70 
                     $new = new Tag();
71 71 
                     $new->name = 'link';
72 72 
                     $new->void = true;
 
@@ -88,7 +88,7 @@  discard block
 
 block discarded – undo
88 88 
         $doc = $this->parse('<a href="url"><b></b>hello</a>');
89 89
90 90 
         $t = new Traverser();
91 
-        $t->addVisitor(new class() implements VisitorInterface {
91 
+        $t->addVisitor(new class() implements VisitorInterface{
92 92 
             public function enterNode(mixed $node, VisitorContext $ctx): mixed
93 93 
             {
94 94 
                 return null;
 
@@ -96,7 +96,7 @@  discard block
 
 block discarded – undo
96 96
97 97 
             public function leaveNode(mixed $node, VisitorContext $ctx): mixed
98 98 
             {
99 
-                if ($node instanceof Tag && $node->name === 'b') {
99 
+                if ($node instanceof Tag && $node->name === 'b'){
100 100 
                     return VisitorInterface::REMOVE_NODE;
101 101 
                 }
102 102 
 
@@ -123,7 +123,7 @@  discard block
 
 block discarded – undo
123 123
124 124 
     public function enterNode(mixed $node, VisitorContext $ctx): mixed
125 125 
     {
126 
-        if ($ctx->getCurrentNode() instanceof Raw) {
126 
+        if ($ctx->getCurrentNode() instanceof Raw){
127 127 
             self::assertInstanceOf(Tag::class, $ctx->getParentNode());
128 128 
             self::assertSame('b', $ctx->getParentNode()->name);
129 129
Please login to merge, or discard this patch.
Braces   +14 added lines, -7 removed lines patch added patch discarded remove patch 
@@ -32,10 +32,12 @@  discard block
 
 block discarded – undo
32 32 
         $doc = $this->parse('<a href="url"></a>');
33 33
34 34 
         $t = new Traverser();
35 
-        $t->addVisitor(new class() implements VisitorInterface {
35 
+        $t->addVisitor(new class() implements VisitorInterface
36 
+        {
36 37 
             public function enterNode(mixed $node, VisitorContext $ctx): mixed
37 38 
             {
38 
-                if ($node instanceof Tag && $node->name === 'a') {
39 
+                if ($node instanceof Tag && $node->name === 'a')
40 
+                {
39 41 
                     $node->name = 'b';
40 42 
                 }
41 43 
 
@@ -58,7 +60,8 @@  discard block
 
 block discarded – undo
58 60 
         $doc = $this->parse('<a href="url"></a>');
59 61
60 62 
         $t = new Traverser();
61 
-        $t->addVisitor(new class() implements VisitorInterface {
63 
+        $t->addVisitor(new class() implements VisitorInterface
64 
+        {
62 65 
             public function enterNode(mixed $node, VisitorContext $ctx): mixed
63 66 
             {
64 67 
                 return null;
 
@@ -66,7 +69,8 @@  discard block
 
 block discarded – undo
66 69
67 70 
             public function leaveNode(mixed $node, VisitorContext $ctx): mixed
68 71 
             {
69 
-                if ($node instanceof Tag && $node->name === 'a') {
72 
+                if ($node instanceof Tag && $node->name === 'a')
73 
+                {
70 74 
                     $new = new Tag();
71 75 
                     $new->name = 'link';
72 76 
                     $new->void = true;
 
@@ -88,7 +92,8 @@  discard block
 
 block discarded – undo
88 92 
         $doc = $this->parse('<a href="url"><b></b>hello</a>');
89 93
90 94 
         $t = new Traverser();
91 
-        $t->addVisitor(new class() implements VisitorInterface {
95 
+        $t->addVisitor(new class() implements VisitorInterface
96 
+        {
92 97 
             public function enterNode(mixed $node, VisitorContext $ctx): mixed
93 98 
             {
94 99 
                 return null;
 
@@ -96,7 +101,8 @@  discard block
 
 block discarded – undo
96 101
97 102 
             public function leaveNode(mixed $node, VisitorContext $ctx): mixed
98 103 
             {
99 
-                if ($node instanceof Tag && $node->name === 'b') {
104 
+                if ($node instanceof Tag && $node->name === 'b')
105 
+                {
100 106 
                     return VisitorInterface::REMOVE_NODE;
101 107 
                 }
102 108 
 
@@ -123,7 +129,8 @@  discard block
 
 block discarded – undo
123 129
124 130 
     public function enterNode(mixed $node, VisitorContext $ctx): mixed
125 131 
     {
126 
-        if ($ctx->getCurrentNode() instanceof Raw) {
132 
+        if ($ctx->getCurrentNode() instanceof Raw)
133 
+        {
127 134 
             self::assertInstanceOf(Tag::class, $ctx->getParentNode());
128 135 
             self::assertSame('b', $ctx->getParentNode()->name);
129 136
Please login to merge, or discard this patch.
src/Stempler/tests/ExceptionTest.php 2 patches
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch 
@@ -33,9 +33,9 @@  discard block
 
 block discarded – undo
33 33 
     {
34 34 
         $builder = $this->getBuilder($this->getFixtureLoader());
35 35
36 
-        try {
36 
+        try{
37 37 
             $builder->compile('broken')->getContent();
38 
-        } catch (ParserException $e) {
38 
+        }catch (ParserException $e){
39 39 
             self::assertInstanceOf(SyntaxException::class, $e->getPrevious());
40 40 
             self::assertStringContainsString('broken.dark.php', $e->getFile());
41 41 
             self::assertSame(3, $e->getLine());
 
@@ -46,9 +46,9 @@  discard block
 
 block discarded – undo
46 46 
     {
47 47 
         $builder = $this->getBuilder($this->getFixtureLoader());
48 48
49 
-        try {
49 
+        try{
50 50 
             $builder->compile('import/bad-element');
51 
-        } catch (ImportException $e) {
51 
+        }catch (ImportException $e){
52 52 
             self::assertStringContainsString('bad-element.dark.php', $e->getFile());
53 53 
             self::assertSame(1, $e->getLine());
54 54 
         }
 
@@ -58,9 +58,9 @@  discard block
 
 block discarded – undo
58 58 
     {
59 59 
         $builder = $this->getBuilder($this->getFixtureLoader());
60 60
61 
-        try {
61 
+        try{
62 62 
             $builder->compile('import/bad-element-3');
63 
-        } catch (ImportException $e) {
63 
+        }catch (ImportException $e){
64 64 
             self::assertStringContainsString('bad-element-3.dark.php', $e->getFile());
65 65 
             self::assertSame(3, $e->getLine());
66 66 
         }
 
@@ -70,9 +70,9 @@  discard block
 
 block discarded – undo
70 70 
     {
71 71 
         $builder = $this->getBuilder($this->getFixtureLoader());
72 72
73 
-        try {
73 
+        try{
74 74 
             $builder->compile('import/bad-dir');
75 
-        } catch (ImportException $e) {
75 
+        }catch (ImportException $e){
76 76 
             self::assertStringContainsString('bad-dir.dark.php', $e->getFile());
77 77 
             self::assertSame(1, $e->getLine());
78 78 
         }
 
@@ -82,9 +82,9 @@  discard block
 
 block discarded – undo
82 82 
     {
83 83 
         $builder = $this->getBuilder($this->getFixtureLoader());
84 84
85 
-        try {
85 
+        try{
86 86 
             $builder->compile('import/bad-dir-2');
87 
-        } catch (ImportException $e) {
87 
+        }catch (ImportException $e){
88 88 
             self::assertStringContainsString('bad-dir-2.dark.php', $e->getFile());
89 89 
             self::assertSame(2, $e->getLine());
90 90 
         }
 
@@ -94,9 +94,9 @@  discard block
 
 block discarded – undo
94 94 
     {
95 95 
         $builder = $this->getBuilder($this->getFixtureLoader());
96 96
97 
-        try {
97 
+        try{
98 98 
             $builder->compile('bad-directive');
99 
-        } catch (DirectiveException $e) {
99 
+        }catch (DirectiveException $e){
100 100 
             self::assertStringContainsString('bad-directive.dark.php', $e->getFile());
101 101 
             self::assertSame(2, $e->getLine());
102 102 
         }
 
@@ -106,9 +106,9 @@  discard block
 
 block discarded – undo
106 106 
     {
107 107 
         $builder = $this->getBuilder($this->getFixtureLoader());
108 108
109 
-        try {
109 
+        try{
110 110 
             $builder->compile('exception-in-import');
111 
-        } catch (ImportException $e) {
111 
+        }catch (ImportException $e){
112 112 
             self::assertStringContainsString('exception-in-import.dark.php', $e->getFile());
113 113 
             self::assertSame(3, $e->getLine());
114 114 
 
@@ -123,9 +123,9 @@  discard block
 
 block discarded – undo
123 123 
     {
124 124 
         $builder = $this->getBuilder($this->getFixtureLoader());
125 125
126 
-        try {
126 
+        try{
127 127 
             $builder->compile('exception-in-import-2');
128 
-        } catch (ImportException $e) {
128 
+        }catch (ImportException $e){
129 129 
             self::assertInstanceOf(ImportException::class, $e);
130 130 
             self::assertStringContainsString('exception-in-import-2.dark.php', $e->getFile());
131 131 
             self::assertSame(3, $e->getLine());
 
@@ -141,9 +141,9 @@  discard block
 
 block discarded – undo
141 141 
     {
142 142 
         $builder = $this->getBuilder($this->getFixtureLoader());
143 143
144 
-        try {
144 
+        try{
145 145 
             $builder->compile('bad-extends');
146 
-        } catch (ExtendsException $e) {
146 
+        }catch (ExtendsException $e){
147 147 
             self::assertStringContainsString('bad-extends.dark.php', $e->getFile());
148 148 
             self::assertSame(1, $e->getLine());
149 149 
 
@@ -155,9 +155,9 @@  discard block
 
 block discarded – undo
155 155 
     {
156 156 
         $builder = $this->getBuilder($this->getFixtureLoader());
157 157
158 
-        try {
158 
+        try{
159 159 
             $builder->compile('bad-extends-2');
160 
-        } catch (ExtendsException $e) {
160 
+        }catch (ExtendsException $e){
161 161 
             self::assertInstanceOf(ExtendsException::class, $e);
162 162 
             self::assertStringContainsString('bad-extends-2.dark.php', $e->getFile());
163 163 
             self::assertSame(1, $e->getLine());
Please login to merge, or discard this patch.
Braces   +50 added lines, -20 removed lines patch added patch discarded remove patch 
@@ -33,9 +33,12 @@  discard block
 
 block discarded – undo
33 33 
     {
34 34 
         $builder = $this->getBuilder($this->getFixtureLoader());
35 35
36 
-        try {
36 
+        try
37 
+        {
37 38 
             $builder->compile('broken')->getContent();
38 
-        } catch (ParserException $e) {
39 
+        }
40 
+        catch (ParserException $e)
41 
+        {
39 42 
             self::assertInstanceOf(SyntaxException::class, $e->getPrevious());
40 43 
             self::assertStringContainsString('broken.dark.php', $e->getFile());
41 44 
             self::assertSame(3, $e->getLine());
 
@@ -46,9 +49,12 @@  discard block
 
 block discarded – undo
46 49 
     {
47 50 
         $builder = $this->getBuilder($this->getFixtureLoader());
48 51
49 
-        try {
52 
+        try
53 
+        {
50 54 
             $builder->compile('import/bad-element');
51 
-        } catch (ImportException $e) {
55 
+        }
56 
+        catch (ImportException $e)
57 
+        {
52 58 
             self::assertStringContainsString('bad-element.dark.php', $e->getFile());
53 59 
             self::assertSame(1, $e->getLine());
54 60 
         }
 
@@ -58,9 +64,12 @@  discard block
 
 block discarded – undo
58 64 
     {
59 65 
         $builder = $this->getBuilder($this->getFixtureLoader());
60 66
61 
-        try {
67 
+        try
68 
+        {
62 69 
             $builder->compile('import/bad-element-3');
63 
-        } catch (ImportException $e) {
70 
+        }
71 
+        catch (ImportException $e)
72 
+        {
64 73 
             self::assertStringContainsString('bad-element-3.dark.php', $e->getFile());
65 74 
             self::assertSame(3, $e->getLine());
66 75 
         }
 
@@ -70,9 +79,12 @@  discard block
 
 block discarded – undo
70 79 
     {
71 80 
         $builder = $this->getBuilder($this->getFixtureLoader());
72 81
73 
-        try {
82 
+        try
83 
+        {
74 84 
             $builder->compile('import/bad-dir');
75 
-        } catch (ImportException $e) {
85 
+        }
86 
+        catch (ImportException $e)
87 
+        {
76 88 
             self::assertStringContainsString('bad-dir.dark.php', $e->getFile());
77 89 
             self::assertSame(1, $e->getLine());
78 90 
         }
 
@@ -82,9 +94,12 @@  discard block
 
 block discarded – undo
82 94 
     {
83 95 
         $builder = $this->getBuilder($this->getFixtureLoader());
84 96
85 
-        try {
97 
+        try
98 
+        {
86 99 
             $builder->compile('import/bad-dir-2');
87 
-        } catch (ImportException $e) {
100 
+        }
101 
+        catch (ImportException $e)
102 
+        {
88 103 
             self::assertStringContainsString('bad-dir-2.dark.php', $e->getFile());
89 104 
             self::assertSame(2, $e->getLine());
90 105 
         }
 
@@ -94,9 +109,12 @@  discard block
 
 block discarded – undo
94 109 
     {
95 110 
         $builder = $this->getBuilder($this->getFixtureLoader());
96 111
97 
-        try {
112 
+        try
113 
+        {
98 114 
             $builder->compile('bad-directive');
99 
-        } catch (DirectiveException $e) {
115 
+        }
116 
+        catch (DirectiveException $e)
117 
+        {
100 118 
             self::assertStringContainsString('bad-directive.dark.php', $e->getFile());
101 119 
             self::assertSame(2, $e->getLine());
102 120 
         }
 
@@ -106,9 +124,12 @@  discard block
 
 block discarded – undo
106 124 
     {
107 125 
         $builder = $this->getBuilder($this->getFixtureLoader());
108 126
109 
-        try {
127 
+        try
128 
+        {
110 129 
             $builder->compile('exception-in-import');
111 
-        } catch (ImportException $e) {
130 
+        }
131 
+        catch (ImportException $e)
132 
+        {
112 133 
             self::assertStringContainsString('exception-in-import.dark.php', $e->getFile());
113 134 
             self::assertSame(3, $e->getLine());
114 135 
 
@@ -123,9 +144,12 @@  discard block
 
 block discarded – undo
123 144 
     {
124 145 
         $builder = $this->getBuilder($this->getFixtureLoader());
125 146
126 
-        try {
147 
+        try
148 
+        {
127 149 
             $builder->compile('exception-in-import-2');
128 
-        } catch (ImportException $e) {
150 
+        }
151 
+        catch (ImportException $e)
152 
+        {
129 153 
             self::assertInstanceOf(ImportException::class, $e);
130 154 
             self::assertStringContainsString('exception-in-import-2.dark.php', $e->getFile());
131 155 
             self::assertSame(3, $e->getLine());
 
@@ -141,9 +165,12 @@  discard block
 
 block discarded – undo
141 165 
     {
142 166 
         $builder = $this->getBuilder($this->getFixtureLoader());
143 167
144 
-        try {
168 
+        try
169 
+        {
145 170 
             $builder->compile('bad-extends');
146 
-        } catch (ExtendsException $e) {
171 
+        }
172 
+        catch (ExtendsException $e)
173 
+        {
147 174 
             self::assertStringContainsString('bad-extends.dark.php', $e->getFile());
148 175 
             self::assertSame(1, $e->getLine());
149 176 
 
@@ -155,9 +182,12 @@  discard block
 
 block discarded – undo
155 182 
     {
156 183 
         $builder = $this->getBuilder($this->getFixtureLoader());
157 184
158 
-        try {
185 
+        try
186 
+        {
159 187 
             $builder->compile('bad-extends-2');
160 
-        } catch (ExtendsException $e) {
188 
+        }
189 
+        catch (ExtendsException $e)
190 
+        {
161 191 
             self::assertInstanceOf(ExtendsException::class, $e);
162 192 
             self::assertStringContainsString('bad-extends-2.dark.php', $e->getFile());
163 193 
             self::assertSame(1, $e->getLine());
Please login to merge, or discard this patch.