Test Failed
Pull Request — master (#1137)
by f
05:31
created
src/Prototype/tests/Traverse/ConstructorParamsVisitor.php 2 patches
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -24,19 +24,19 @@
 block discarded – undo
24 24
      */
25 25
     public function leaveNode(Node $node): void
26 26
     {
27
-        if ($node instanceof Node\Stmt\ClassMethod && $node->name->name === '__construct') {
28
-            foreach ($node->params as $param) {
29
-                if ($param->type instanceof Node\NullableType) {
30
-                    if ($param->type->type instanceof Node\Identifier) {
27
+        if ($node instanceof Node\Stmt\ClassMethod && $node->name->name === '__construct'){
28
+            foreach ($node->params as $param){
29
+                if ($param->type instanceof Node\NullableType){
30
+                    if ($param->type->type instanceof Node\Identifier){
31 31
                         $type = $param->type->type->name;
32
-                    } else {
32
+                    }else{
33 33
                         $type = implode('\\', $param->type->type->parts);
34 34
                     }
35 35
 
36 36
                     $type = "?$type";
37
-                } elseif ($param->type instanceof Node\Name) {
37
+                } elseif ($param->type instanceof Node\Name){
38 38
                     $type = implode('\\', $param->type->parts);
39
-                } else {
39
+                }else{
40 40
                     $type = $param->type->name ?? null;
41 41
                 }
42 42
 
Please login to merge, or discard this patch.
Braces   +17 added lines, -7 removed lines patch added patch discarded remove patch
@@ -24,19 +24,29 @@
 block discarded – undo
24 24
      */
25 25
     public function leaveNode(Node $node): void
26 26
     {
27
-        if ($node instanceof Node\Stmt\ClassMethod && $node->name->name === '__construct') {
28
-            foreach ($node->params as $param) {
29
-                if ($param->type instanceof Node\NullableType) {
30
-                    if ($param->type->type instanceof Node\Identifier) {
27
+        if ($node instanceof Node\Stmt\ClassMethod && $node->name->name === '__construct')
28
+        {
29
+            foreach ($node->params as $param)
30
+            {
31
+                if ($param->type instanceof Node\NullableType)
32
+                {
33
+                    if ($param->type->type instanceof Node\Identifier)
34
+                    {
31 35
                         $type = $param->type->type->name;
32
-                    } else {
36
+                    }
37
+                    else
38
+                    {
33 39
                         $type = implode('\\', $param->type->type->parts);
34 40
                     }
35 41
 
36 42
                     $type = "?$type";
37
-                } elseif ($param->type instanceof Node\Name) {
43
+                }
44
+                elseif ($param->type instanceof Node\Name)
45
+                {
38 46
                     $type = implode('\\', $param->type->parts);
39
-                } else {
47
+                }
48
+                else
49
+                {
40 50
                     $type = $param->type->name ?? null;
41 51
                 }
42 52
 
Please login to merge, or discard this patch.
monorepo-builder.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -10,8 +10,8 @@  discard block
 block discarded – undo
10 10
  *
11 11
  * @see https://github.com/symplify/symplify/issues/2061
12 12
  */
13
-\register_shutdown_function(static function () {
14
-    $dest = \json_decode(\file_get_contents(__DIR__ . '/composer.json'), true);
13
+\register_shutdown_function(static function (){
14
+    $dest = \json_decode(\file_get_contents(__DIR__.'/composer.json'), true);
15 15
 
16 16
     $result = [
17 17
         'name'              => 'spiral/framework',
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 
47 47
     $json = \json_encode($result, \JSON_PRETTY_PRINT | \JSON_UNESCAPED_SLASHES);
48 48
 
49
-    \file_put_contents(__DIR__ . '/composer.json', $json . "\n");
49
+    \file_put_contents(__DIR__.'/composer.json', $json."\n");
50 50
 });
51 51
 
52 52
 
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -10,7 +10,8 @@
 block discarded – undo
10 10
  *
11 11
  * @see https://github.com/symplify/symplify/issues/2061
12 12
  */
13
-\register_shutdown_function(static function () {
13
+\register_shutdown_function(static function ()
14
+{
14 15
     $dest = \json_decode(\file_get_contents(__DIR__ . '/composer.json'), true);
15 16
 
16 17
     $result = [
Please login to merge, or discard this patch.
src/Streams/tests/StreamsTest.php 2 patches
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 
18 18
 class StreamsTest extends TestCase
19 19
 {
20
-    private const FIXTURE_DIRECTORY = __DIR__ . '/fixtures';
20
+    private const FIXTURE_DIRECTORY = __DIR__.'/fixtures';
21 21
 
22 22
     public function setUp(): void
23 23
     {
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
         $this->assertSame(strlen('sample text'), filesize($filename));
43 43
         $this->assertSame(md5('sample text'), md5_file($filename));
44 44
 
45
-        $newFilename = self::FIXTURE_DIRECTORY . '/test.txt';
45
+        $newFilename = self::FIXTURE_DIRECTORY.'/test.txt';
46 46
         copy($filename, $newFilename);
47 47
 
48 48
         $this->assertFileExists($newFilename);
@@ -83,15 +83,15 @@  discard block
 block discarded – undo
83 83
      */
84 84
     public function testException()
85 85
     {
86
-        try {
86
+        try{
87 87
             fopen('spiral://non-exists', 'rb');
88
-        } catch (\Throwable $e) {
88
+        }catch (\Throwable $e){
89 89
             $this->assertStringContainsString('failed to open stream', $e->getMessage());
90 90
         }
91 91
 
92
-        try {
92
+        try{
93 93
             filemtime('spiral://non-exists');
94
-        } catch (\Throwable $e) {
94
+        }catch (\Throwable $e){
95 95
             $this->assertStringContainsString('stat failed', $e->getMessage());
96 96
         }
97 97
     }
@@ -101,15 +101,15 @@  discard block
 block discarded – undo
101 101
      */
102 102
     public function testExceptionPHP8()
103 103
     {
104
-        try {
104
+        try{
105 105
             fopen('spiral://non-exists', 'rb');
106
-        } catch (\Throwable $e) {
106
+        }catch (\Throwable $e){
107 107
             $this->assertStringContainsString('Failed to open stream', $e->getMessage());
108 108
         }
109 109
 
110
-        try {
110
+        try{
111 111
             filemtime('spiral://non-exists');
112
-        } catch (\Throwable $e) {
112
+        }catch (\Throwable $e){
113 113
             $this->assertStringContainsString('stat failed', $e->getMessage());
114 114
         }
115 115
     }
Please login to merge, or discard this patch.
Braces   +20 added lines, -8 removed lines patch added patch discarded remove patch
@@ -83,15 +83,21 @@  discard block
 block discarded – undo
83 83
      */
84 84
     public function testException()
85 85
     {
86
-        try {
86
+        try
87
+        {
87 88
             fopen('spiral://non-exists', 'rb');
88
-        } catch (\Throwable $e) {
89
+        }
90
+        catch (\Throwable $e)
91
+        {
89 92
             $this->assertStringContainsString('failed to open stream', $e->getMessage());
90 93
         }
91 94
 
92
-        try {
95
+        try
96
+        {
93 97
             filemtime('spiral://non-exists');
94
-        } catch (\Throwable $e) {
98
+        }
99
+        catch (\Throwable $e)
100
+        {
95 101
             $this->assertStringContainsString('stat failed', $e->getMessage());
96 102
         }
97 103
     }
@@ -101,15 +107,21 @@  discard block
 block discarded – undo
101 107
      */
102 108
     public function testExceptionPHP8()
103 109
     {
104
-        try {
110
+        try
111
+        {
105 112
             fopen('spiral://non-exists', 'rb');
106
-        } catch (\Throwable $e) {
113
+        }
114
+        catch (\Throwable $e)
115
+        {
107 116
             $this->assertStringContainsString('Failed to open stream', $e->getMessage());
108 117
         }
109 118
 
110
-        try {
119
+        try
120
+        {
111 121
             filemtime('spiral://non-exists');
112
-        } catch (\Throwable $e) {
122
+        }
123
+        catch (\Throwable $e)
124
+        {
113 125
             $this->assertStringContainsString('stat failed', $e->getMessage());
114 126
         }
115 127
     }
Please login to merge, or discard this patch.
src/SendIt/src/MessageSerializer.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -51,11 +51,11 @@
 block discarded – undo
51 51
     public static function unpack(array $payload): MessageInterface
52 52
     {
53 53
         $message = new Message($payload['subject'], $payload['to'], $payload['data']);
54
-        if ($payload['from'] !== null) {
54
+        if ($payload['from'] !== null){
55 55
             $message->setFrom($payload['from']);
56 56
         }
57 57
 
58
-        if ($payload['replyTo'] !== null) {
58
+        if ($payload['replyTo'] !== null){
59 59
             $message->setReplyTo($payload['replyTo']);
60 60
         }
61 61
 
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -51,11 +51,13 @@
 block discarded – undo
51 51
     public static function unpack(array $payload): MessageInterface
52 52
     {
53 53
         $message = new Message($payload['subject'], $payload['to'], $payload['data']);
54
-        if ($payload['from'] !== null) {
54
+        if ($payload['from'] !== null)
55
+        {
55 56
             $message->setFrom($payload['from']);
56 57
         }
57 58
 
58
-        if ($payload['replyTo'] !== null) {
59
+        if ($payload['replyTo'] !== null)
60
+        {
59 61
             $message->setReplyTo($payload['replyTo']);
60 62
         }
61 63
 
Please login to merge, or discard this patch.
src/Prototype/tests/Commands/InjectCommandTest.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
         $source = file_get_contents($filename);
30 30
         $this->assertStringContainsString('use PrototypeTrait;', $source);
31 31
 
32
-        try {
32
+        try{
33 33
             $this->app->bindApp();
34 34
 
35 35
             $inp = new ArrayInput(['--remove' => true]);
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
             $this->app->get(Console::class)->run('prototype:inject', $inp, $out);
38 38
 
39 39
             $this->assertStringNotContainsString('use PrototypeTrait;', file_get_contents($filename));
40
-        } finally {
40
+        }finally{
41 41
             file_put_contents($filename, $source);
42 42
         }
43 43
     }
Please login to merge, or discard this patch.
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,7 +29,8 @@  discard block
 block discarded – undo
29 29
         $source = file_get_contents($filename);
30 30
         $this->assertStringContainsString('use PrototypeTrait;', $source);
31 31
 
32
-        try {
32
+        try
33
+        {
33 34
             $this->app->bindApp();
34 35
 
35 36
             $inp = new ArrayInput(['--remove' => true]);
@@ -37,7 +38,9 @@  discard block
 block discarded – undo
37 38
             $this->app->get(Console::class)->run('prototype:inject', $inp, $out);
38 39
 
39 40
             $this->assertStringNotContainsString('use PrototypeTrait;', file_get_contents($filename));
40
-        } finally {
41
+        }
42
+        finally
43
+        {
41 44
             file_put_contents($filename, $source);
42 45
         }
43 46
     }
Please login to merge, or discard this patch.
src/SendIt/views/builder.dark.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -7,7 +7,7 @@  discard block
 block discarded – undo
7 7
  */
8 8
 $_msg_->subject(\Spiral\SendIt\Renderer\ViewRenderer::escapeSubject(inject('subject')));
9 9
 
10
-if (injected('from')) {
10
+if (injected('from')){
11 11
     $_msg_->from(\Symfony\Component\Mime\Address::create(inject('from')));
12 12
 }
13 13
 
@@ -19,13 +19,13 @@  discard block
 block discarded – undo
19 19
 
20 20
 ob_start(); ?>${html}<?php $_html_ = ob_get_clean();
21 21
 
22
-if (!empty($_html_)) {
22
+if (!empty($_html_)){
23 23
     $_msg_->html($_html_);
24 24
 }
25 25
 
26 26
 ob_start(); ?>${text}<?php $_text_ = ob_get_clean();
27 27
 
28
-if (!empty($_text_)) {
28
+if (!empty($_text_)){
29 29
     $_msg_->text($_text_);
30 30
 }
31 31
 ?>
Please login to merge, or discard this patch.
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -7,7 +7,8 @@  discard block
 block discarded – undo
7 7
  */
8 8
 $_msg_->subject(\Spiral\SendIt\Renderer\ViewRenderer::escapeSubject(inject('subject')));
9 9
 
10
-if (injected('from')) {
10
+if (injected('from'))
11
+{
11 12
     $_msg_->from(\Symfony\Component\Mime\Address::create(inject('from')));
12 13
 }
13 14
 
@@ -19,13 +20,15 @@  discard block
 block discarded – undo
19 20
 
20 21
 ob_start(); ?>${html}<?php $_html_ = ob_get_clean();
21 22
 
22
-if (!empty($_html_)) {
23
+if (!empty($_html_))
24
+{
23 25
     $_msg_->html($_html_);
24 26
 }
25 27
 
26 28
 ob_start(); ?>${text}<?php $_text_ = ob_get_clean();
27 29
 
28
-if (!empty($_text_)) {
30
+if (!empty($_text_))
31
+{
29 32
     $_msg_->text($_text_);
30 33
 }
31 34
 ?>
Please login to merge, or discard this patch.
src/Bridge/Stempler/src/ExceptionMapper.php 2 patches
Braces   +10 added lines, -5 removed lines patch added patch discarded remove patch
@@ -45,9 +45,12 @@  discard block
 block discarded – undo
45 45
         $classFilename = (new \ReflectionClass($class))->getFileName();
46 46
 
47 47
         // looking for proper view error location
48
-        if ($e->getFile() !== $classFilename) {
49
-            foreach ($e->getTrace() as $trace) {
50
-                if (isset($trace['file']) && $trace['file'] === $classFilename) {
48
+        if ($e->getFile() !== $classFilename)
49
+        {
50
+            foreach ($e->getTrace() as $trace)
51
+            {
52
+                if (isset($trace['file']) && $trace['file'] === $classFilename)
53
+                {
51 54
                     $line = $trace['line'];
52 55
                     break;
53 56
                 }
@@ -55,7 +58,8 @@  discard block
 block discarded – undo
55 58
         }
56 59
 
57 60
         $userStack = [];
58
-        foreach ($this->sourcemap->getStack($line - $this->lineOffset) as $stack) {
61
+        foreach ($this->sourcemap->getStack($line - $this->lineOffset) as $stack)
62
+        {
59 63
             $userStack[] = [
60 64
                 'file'     => $stack['file'],
61 65
                 'line'     => $stack['line'],
@@ -65,7 +69,8 @@  discard block
 block discarded – undo
65 69
                 'args'     => [$data],
66 70
             ];
67 71
 
68
-            if ($stack['file'] === $filename) {
72
+            if ($stack['file'] === $filename)
73
+            {
69 74
                 // no need to jump over root template
70 75
                 break;
71 76
             }
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@  discard block
 block discarded – undo
12 12
     public function __construct(
13 13
         private readonly SourceMap $sourcemap,
14 14
         private readonly int $lineOffset
15
-    ) {
15
+    ){
16 16
     }
17 17
 
18 18
     public function mapException(\Throwable $e, string $class, string $filename, array $data): \Throwable
@@ -21,9 +21,9 @@  discard block
 block discarded – undo
21 21
         $classFilename = (new \ReflectionClass($class))->getFileName();
22 22
 
23 23
         // looking for proper view error location
24
-        if ($e->getFile() !== $classFilename) {
25
-            foreach ($e->getTrace() as $trace) {
26
-                if (isset($trace['file']) && $trace['file'] === $classFilename) {
24
+        if ($e->getFile() !== $classFilename){
25
+            foreach ($e->getTrace() as $trace){
26
+                if (isset($trace['file']) && $trace['file'] === $classFilename){
27 27
                     $line = $trace['line'];
28 28
                     break;
29 29
                 }
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
         }
32 32
 
33 33
         $userStack = [];
34
-        foreach ($this->sourcemap->getStack($line - $this->lineOffset) as $stack) {
34
+        foreach ($this->sourcemap->getStack($line - $this->lineOffset) as $stack){
35 35
             $userStack[] = [
36 36
                 'file'     => $stack['file'],
37 37
                 'line'     => $stack['line'],
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
                 'args'     => [$data],
42 42
             ];
43 43
 
44
-            if ($stack['file'] === $filename) {
44
+            if ($stack['file'] === $filename){
45 45
                 // no need to jump over root template
46 46
                 break;
47 47
             }
Please login to merge, or discard this patch.
src/Bridge/Stempler/src/StemplerLoader.php 2 patches
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -56,7 +56,8 @@
 block discarded – undo
56 56
      */
57 57
     public function load(string $path): Source
58 58
     {
59
-        if ($this->context === null) {
59
+        if ($this->context === null)
60
+        {
60 61
             throw new EngineException('Unable to use StemplerLoader without given context.');
61 62
         }
62 63
 
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
     public function __construct(
24 24
         private readonly LoaderInterface $loader,
25 25
         array $processors
26
-    ) {
26
+    ){
27 27
         $this->processors = $processors;
28 28
     }
29 29
 
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
      */
41 41
     public function load(string $path): Source
42 42
     {
43
-        if ($this->context === null) {
43
+        if ($this->context === null){
44 44
             throw new EngineException('Unable to use StemplerLoader without given context.');
45 45
         }
46 46
 
Please login to merge, or discard this patch.
src/Bridge/Stempler/src/StemplerView.php 2 patches
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -59,7 +59,8 @@
 block discarded – undo
59 59
             $this->context
60 60
         );
61 61
 
62
-        if ($sourcemap === null) {
62
+        if ($sourcemap === null)
63
+        {
63 64
             return $e;
64 65
         }
65 66
 
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
         protected StemplerEngine $engine,
21 21
         protected ViewSource $view,
22 22
         protected ContextInterface $context
23
-    ) {
23
+    ){
24 24
         $this->container = $engine->getContainer();
25 25
     }
26 26
 
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
             $this->context
35 35
         );
36 36
 
37
-        if ($sourcemap === null) {
37
+        if ($sourcemap === null){
38 38
             return $e;
39 39
         }
40 40
 
Please login to merge, or discard this patch.