Test Failed
Pull Request — master (#925)
by butschster
16:57 queued 09:14
created
src/Core/tests/Internal/Factory/CommonCasesTest.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -85,7 +85,7 @@
 block discarded – undo
85 85
 
86 86
     public function testClosureFactory(): void
87 87
     {
88
-        $this->bind(Bucket::class, function ($data) {
88
+        $this->bind(Bucket::class, function ($data){
89 89
             return new Bucket('via-closure', $data);
90 90
         });
91 91
 
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -85,7 +85,8 @@  discard block
 block discarded – undo
85 85
 
86 86
     public function testClosureFactory(): void
87 87
     {
88
-        $this->bind(Bucket::class, function ($data) {
88
+        $this->bind(Bucket::class, function ($data)
89
+        {
89 90
             return new Bucket('via-closure', $data);
90 91
         });
91 92
 
@@ -125,7 +126,8 @@  discard block
 block discarded – undo
125 126
         $sample = new SampleClass();
126 127
 
127 128
         $this->bind(Bucket::class, [self::class, 'makeBucketWithSample']);
128
-        $this->bind(SampleClass::class, function () use ($sample) {
129
+        $this->bind(SampleClass::class, function () use ($sample)
130
+        {
129 131
             return $sample;
130 132
         });
131 133
 
Please login to merge, or discard this patch.
src/Core/tests/Exception/ClosureRendererTraitTest.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
     public function testClassWithNamespace(): void
94 94
     {
95 95
         $this->assertSame(
96
-            'function (?' . Container::class . ' ...$v)',
96
+            'function (?'.Container::class.' ...$v)',
97 97
             $this->renderClosureForTesting(fn (?Container ...$v) => null),
98 98
         );
99 99
     }
@@ -102,14 +102,14 @@  discard block
 block discarded – undo
102 102
     {
103 103
         $this->assertSame(
104 104
             'function (self|string|int|null $v)',
105
-            $this->renderClosureForTesting(fn (self|string|int|null $v) => null),
105
+            $this->renderClosureForTesting(fn (self | string | int | null $v) => null),
106 106
         );
107 107
     }
108 108
 
109 109
     public function testTypeIntersection(): void
110 110
     {
111 111
         $this->assertSame(
112
-            'function (' . ContainerInterface::class . '&' . ContainerExceptionInterface::class . ' $v)',
112
+            'function ('.ContainerInterface::class.'&'.ContainerExceptionInterface::class.' $v)',
113 113
             $this->renderClosureForTesting(fn (ContainerInterface&ContainerExceptionInterface $v) => null),
114 114
         );
115 115
     }
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
     {
128 128
         $this->assertSame(
129 129
             'function (Foo|Bar $v)',
130
-            $this->renderClosureForTesting(fn (\Foo|\Bar $v) => null),
130
+            $this->renderClosureForTesting(fn (\Foo | \Bar $v) => null),
131 131
         );
132 132
     }
133 133
 
Please login to merge, or discard this patch.
src/Core/tests/Fixtures/TypedClass.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,6 +13,6 @@
 block discarded – undo
13 13
         public bool $bool,
14 14
         public array $array = [],
15 15
         public ?string $pong = null
16
-    ) {
16
+    ){
17 17
     }
18 18
 }
Please login to merge, or discard this patch.
src/Core/tests/InjectableTest.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 
32 32
     public function testInvalidInjector(): void
33 33
     {
34
-        $excepted = "Class 'Spiral\Tests\Core\Fixtures\InvalidInjector' must be an " .
34
+        $excepted = "Class 'Spiral\Tests\Core\Fixtures\InvalidInjector' must be an ".
35 35
                     "instance of InjectorInterface for 'Spiral\Tests\Core\Fixtures\TestConfig'";
36 36
         $this->expectException(InjectionException::class);
37 37
         $this->expectExceptionMessage($excepted);
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
 
56 56
     public function testInvalidRuntimeInjector(): void
57 57
     {
58
-        $excepted = "Class 'Spiral\Tests\Core\Fixtures\InvalidInjector' must be an " .
58
+        $excepted = "Class 'Spiral\Tests\Core\Fixtures\InvalidInjector' must be an ".
59 59
             "instance of InjectorInterface for 'Spiral\Tests\Core\Fixtures\TestConfig'";
60 60
         $this->expectException(InjectionException::class);
61 61
         $this->expectExceptionMessage($excepted);
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
         $container->bind(ConfigsInterface::class, $configurator);
103 103
 
104 104
         $configurator->shouldReceive('createInjection')
105
-            ->with(m::on(static function (ReflectionClass $r) {
105
+            ->with(m::on(static function (ReflectionClass $r){
106 106
                 return $r->getName() === TestConfig::class;
107 107
             }), null)
108 108
             ->andReturn($expected)
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
         $container->bind(ConfigsInterface::class, $configurator);
121 121
 
122 122
         $configurator->shouldReceive('createInjection')
123
-            ->with(m::on(static function (ReflectionClass $r) {
123
+            ->with(m::on(static function (ReflectionClass $r){
124 124
                 return $r->getName() === TestConfig::class;
125 125
             }), 'context')
126 126
             ->andReturn($expected)
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -102,7 +102,8 @@  discard block
 block discarded – undo
102 102
         $container->bind(ConfigsInterface::class, $configurator);
103 103
 
104 104
         $configurator->shouldReceive('createInjection')
105
-            ->with(m::on(static function (ReflectionClass $r) {
105
+            ->with(m::on(static function (ReflectionClass $r)
106
+            {
106 107
                 return $r->getName() === TestConfig::class;
107 108
             }), null)
108 109
             ->andReturn($expected)
@@ -120,7 +121,8 @@  discard block
 block discarded – undo
120 121
         $container->bind(ConfigsInterface::class, $configurator);
121 122
 
122 123
         $configurator->shouldReceive('createInjection')
123
-            ->with(m::on(static function (ReflectionClass $r) {
124
+            ->with(m::on(static function (ReflectionClass $r)
125
+            {
124 126
                 return $r->getName() === TestConfig::class;
125 127
             }), 'context')
126 128
             ->andReturn($expected)
Please login to merge, or discard this patch.
src/Core/tests/Stub/LightEngineDecorator.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@
 block discarded – undo
8 8
 {
9 9
     public function __construct(
10 10
         public LightEngine $engine
11
-    ) {
11
+    ){
12 12
     }
13 13
 
14 14
     public function __call(string $name, array $arguments)
Please login to merge, or discard this patch.
src/Core/tests/Stub/NewObjectInParam.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -10,6 +10,6 @@
 block discarded – undo
10 10
 {
11 11
     public function __construct(
12 12
         private object $object = new stdClass()
13
-    ) {
13
+    ){
14 14
     }
15 15
 }
Please login to merge, or discard this patch.
src/Core/tests/InvokerTest.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
         $this->container->bindSingleton(Bucket::class, $bucket = new Bucket('foo'));
96 96
 
97 97
         $result = $this->container->invoke(
98
-            static function (Bucket $bucket, SampleClass $class, string $name, string $path = 'baz') {
98
+            static function (Bucket $bucket, SampleClass $class, string $name, string $path = 'baz'){
99 99
                 return \compact('bucket', 'class', 'name', 'path');
100 100
             },
101 101
             ['name' => 'bar']
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
         $this->expectErrorMessage('Unable to resolve required argument `name` when resolving');
114 114
 
115 115
         $this->container->invoke(
116
-            static function (Bucket $bucket, SampleClass $class, string $name, string $path = 'baz') {
116
+            static function (Bucket $bucket, SampleClass $class, string $name, string $path = 'baz'){
117 117
                 return \compact('bucket', 'class', 'name', 'path');
118 118
             },
119 119
             ['name' => 'bar']
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -95,7 +95,8 @@  discard block
 block discarded – undo
95 95
         $this->container->bindSingleton(Bucket::class, $bucket = new Bucket('foo'));
96 96
 
97 97
         $result = $this->container->invoke(
98
-            static function (Bucket $bucket, SampleClass $class, string $name, string $path = 'baz') {
98
+            static function (Bucket $bucket, SampleClass $class, string $name, string $path = 'baz')
99
+            {
99 100
                 return \compact('bucket', 'class', 'name', 'path');
100 101
             },
101 102
             ['name' => 'bar']
@@ -113,7 +114,8 @@  discard block
 block discarded – undo
113 114
         $this->expectErrorMessage('Unable to resolve required argument `name` when resolving');
114 115
 
115 116
         $this->container->invoke(
116
-            static function (Bucket $bucket, SampleClass $class, string $name, string $path = 'baz') {
117
+            static function (Bucket $bucket, SampleClass $class, string $name, string $path = 'baz')
118
+            {
117 119
                 return \compact('bucket', 'class', 'name', 'path');
118 120
             },
119 121
             ['name' => 'bar']
Please login to merge, or discard this patch.
src/Core/src/Internal/Invoker.php 2 patches
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -35,18 +35,18 @@  discard block
 block discarded – undo
35 35
      */
36 36
     public function invoke(mixed $target, array $parameters = []): mixed
37 37
     {
38
-        if (\is_array($target) && isset($target[1])) {
38
+        if (\is_array($target) && isset($target[1])){
39 39
             // In a form of resolver and method
40 40
             [$resolver, $method] = $target;
41 41
 
42 42
             // Resolver instance (i.e. [ClassName::class, 'method'])
43
-            if (\is_string($resolver)) {
43
+            if (\is_string($resolver)){
44 44
                 $resolver = $this->container->get($resolver);
45 45
             }
46 46
 
47
-            try {
47
+            try{
48 48
                 $method = new \ReflectionMethod($resolver, $method);
49
-            } catch (\ReflectionException $e) {
49
+            }catch (\ReflectionException $e){
50 50
                 throw new ContainerException($e->getMessage(), $e->getCode(), $e);
51 51
             }
52 52
 
@@ -57,14 +57,14 @@  discard block
 block discarded – undo
57 57
             );
58 58
         }
59 59
 
60
-        if (\is_string($target) && \is_callable($target)) {
60
+        if (\is_string($target) && \is_callable($target)){
61 61
             $target = $target(...);
62 62
         }
63 63
 
64
-        if ($target instanceof \Closure) {
65
-            try {
64
+        if ($target instanceof \Closure){
65
+            try{
66 66
                 $reflection = new \ReflectionFunction($target);
67
-            } catch (\ReflectionException $e) {
67
+            }catch (\ReflectionException $e){
68 68
                 throw new ContainerException($e->getMessage(), $e->getCode(), $e);
69 69
             }
70 70
 
Please login to merge, or discard this patch.
Braces   +18 added lines, -8 removed lines patch added patch discarded remove patch
@@ -35,18 +35,23 @@  discard block
 block discarded – undo
35 35
      */
36 36
     public function invoke(mixed $target, array $parameters = []): mixed
37 37
     {
38
-        if (\is_array($target) && isset($target[1])) {
38
+        if (\is_array($target) && isset($target[1]))
39
+        {
39 40
             // In a form of resolver and method
40 41
             [$resolver, $method] = $target;
41 42
 
42 43
             // Resolver instance (i.e. [ClassName::class, 'method'])
43
-            if (\is_string($resolver)) {
44
+            if (\is_string($resolver))
45
+            {
44 46
                 $resolver = $this->container->get($resolver);
45 47
             }
46 48
 
47
-            try {
49
+            try
50
+            {
48 51
                 $method = new \ReflectionMethod($resolver, $method);
49
-            } catch (\ReflectionException $e) {
52
+            }
53
+            catch (\ReflectionException $e)
54
+            {
50 55
                 throw new ContainerException($e->getMessage(), $e->getCode(), $e);
51 56
             }
52 57
 
@@ -57,14 +62,19 @@  discard block
 block discarded – undo
57 62
             );
58 63
         }
59 64
 
60
-        if (\is_string($target) && \is_callable($target)) {
65
+        if (\is_string($target) && \is_callable($target))
66
+        {
61 67
             $target = $target(...);
62 68
         }
63 69
 
64
-        if ($target instanceof \Closure) {
65
-            try {
70
+        if ($target instanceof \Closure)
71
+        {
72
+            try
73
+            {
66 74
                 $reflection = new \ReflectionFunction($target);
67
-            } catch (\ReflectionException $e) {
75
+            }
76
+            catch (\ReflectionException $e)
77
+            {
68 78
                 throw new ContainerException($e->getMessage(), $e->getCode(), $e);
69 79
             }
70 80
 
Please login to merge, or discard this patch.
src/Core/src/Internal/Container.php 2 patches
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -50,7 +50,8 @@
 block discarded – undo
50 50
      */
51 51
     public function get(string|Autowire $id, string $context = null): mixed
52 52
     {
53
-        if ($id instanceof Autowire) {
53
+        if ($id instanceof Autowire)
54
+        {
54 55
             return $id->resolve($this->factory);
55 56
         }
56 57
 
Please login to merge, or discard this 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
     use DestructorTrait;
18 18
 
19 19
     private State $state;
20
-    private FactoryInterface|Factory $factory;
20
+    private FactoryInterface | Factory $factory;
21 21
 
22 22
     public function __construct(Registry $constructor)
23 23
     {
@@ -46,9 +46,9 @@  discard block
 block discarded – undo
46 46
      * @throws ContainerException
47 47
      * @throws \Throwable
48 48
      */
49
-    public function get(string|Autowire $id, string $context = null): mixed
49
+    public function get(string | Autowire $id, string $context = null): mixed
50 50
     {
51
-        if ($id instanceof Autowire) {
51
+        if ($id instanceof Autowire){
52 52
             return $id->resolve($this->factory);
53 53
         }
54 54
 
Please login to merge, or discard this patch.