spiral / framework

src/Core/tests/Scope/FibersTest.php

Spacing +10 added lines, -10 removed lines

@@ -63,7 +63,7 @@ 
         );
 
         self::assertCount(5, $result);
-        foreach ($result as $suspendValue) {
+        foreach ($result as $suspendValue){
             self::assertSame(self::TEST_DATA, $suspendValue);
         }
     }
@@ -81,7 +81,7 @@ 
         );
 
         self::assertCount(2, $result);
-        foreach ($result as $suspendValue) {
+        foreach ($result as $suspendValue){
             self::assertSame(self::TEST_DATA, $suspendValue);
         }
     }
@@ -97,7 +97,7 @@ 
                     $result = '';
                     $result .= Fiber::suspend('foo');
                     $result .= Fiber::suspend('bar');
-                    return $result . Fiber::suspend('error');
+                    return $result.Fiber::suspend('error');
                 }
             ),
             static fn(string $suspendValue): string => $suspendValue !== 'error'
@@ -114,12 +114,12 @@ 
                     $result = '';
                     $result .= Fiber::suspend('foo');
                     $result .= Fiber::suspend('bar');
-                    try {
+                    try{
                         $result .= Fiber::suspend('error');
-                    } catch (\Throwable $e) {
+                    }catch (\Throwable $e){
                         $result .= $e->getMessage();
                     }
-                    return $result . Fiber::suspend('baz');
+                    return $result.Fiber::suspend('baz');
                 }
             ),
             static fn(string $suspendValue): string => $suspendValue !== 'error'
@@ -142,7 +142,7 @@ 
         ?callable $load = null,
         ?Container $container = null,
     ): callable {
-        return static function () use ($load, $container): array {
+        return static function () use ($load, $container) : array {
             // The function should be called in a fiber
             self::assertNotNull(Fiber::getCurrent());
 
@@ -163,7 +163,7 @@ 
                         $resource = $c3->get('resource');
                         self::assertInstanceOf(DateTimeImmutable::class, $c3->get('bar'));
                         self::assertInstanceOf(stdClass::class, $resource);
-                        foreach (self::TEST_DATA as $key => $value) {
+                        foreach (self::TEST_DATA as $key => $value){
                             $resource->$key = $value;
                             $load === null or $load();
                             Fiber::suspend($value);
@@ -176,8 +176,8 @@ 
             }, ['foo' => new DateTime()]);
             self::assertFalse($c1->has('foo'));
 
-            self::assertSame(self::TEST_DATA, (array) $result);
-            return (array) $result;
+            self::assertSame(self::TEST_DATA, (array)$result);
+            return (array)$result;
         };
     }
 }

Braces +15 added lines, -7 removed lines

@@ -63,7 +63,8 @@ 
         );
 
         self::assertCount(5, $result);
-        foreach ($result as $suspendValue) {
+        foreach ($result as $suspendValue)
+        {
             self::assertSame(self::TEST_DATA, $suspendValue);
         }
     }
@@ -81,7 +82,8 @@ 
         );
 
         self::assertCount(2, $result);
-        foreach ($result as $suspendValue) {
+        foreach ($result as $suspendValue)
+        {
             self::assertSame(self::TEST_DATA, $suspendValue);
         }
     }
@@ -114,9 +116,12 @@ 
                     $result = '';
                     $result .= Fiber::suspend('foo');
                     $result .= Fiber::suspend('bar');
-                    try {
+                    try
+                    {
                         $result .= Fiber::suspend('error');
-                    } catch (\Throwable $e) {
+                    }
+                    catch (\Throwable $e)
+                    {
                         $result .= $e->getMessage();
                     }
                     return $result . Fiber::suspend('baz');
@@ -150,20 +155,23 @@ 
             $c1 = $container ?? new Container();
             $c1->bindSingleton('resource', new stdClass());
 
-            $result = $c1->runScoped(static function (Container $c2) use ($load) {
+            $result = $c1->runScoped(static function (Container $c2) use ($load)
+            {
                 // check local binding
                 self::assertTrue($c2->has('foo'));
                 self::assertInstanceOf(DateTime::class, $c2->get('foo'));
 
                 return $c2->runScoped(
-                    static function (ContainerInterface $c3) use ($load) {
+                    static function (ContainerInterface $c3) use ($load)
+                    {
                         // check local binding
                         self::assertTrue($c3->has('bar'));
 
                         $resource = $c3->get('resource');
                         self::assertInstanceOf(DateTimeImmutable::class, $c3->get('bar'));
                         self::assertInstanceOf(stdClass::class, $resource);
-                        foreach (self::TEST_DATA as $key => $value) {
+                        foreach (self::TEST_DATA as $key => $value)
+                        {
                             $resource->$key = $value;
                             $load === null or $load();
                             Fiber::suspend($value);

src/Core/tests/Scope/ProxyTest.php

Spacing +9 added lines, -9 removed lines

@@ -49,7 +49,7 @@ 
                     // from the current `foo` scope
                     self::assertInstanceOf(KVLogger::class, $logger);
 
-                    for ($i = 0; $i < 10; $i++) {
+                    for ($i = 0; $i < 10; $i++){
                         // because of proxy
                         self::assertNotInstanceOf(KVLogger::class, $carrier->getLogger());
                         self::assertSame('kv', $carrier->logger->getName());
@@ -69,7 +69,7 @@ 
                     // from the current `foo` scope
                     self::assertInstanceOf(FileLogger::class, $logger);
 
-                    for ($i = 0; $i < 10; $i++) {
+                    for ($i = 0; $i < 10; $i++){
                         // because of proxy
                         self::assertNotInstanceOf(FileLogger::class, $carrier->getLogger());
                         self::assertSame('file', $carrier->logger->getName());
@@ -137,7 +137,7 @@ 
             ->bind(
                 ContextInterface::class,
                 new \Spiral\Core\Config\Injectable(
-                    new class implements InjectorInterface {
+                    new class implements InjectorInterface{
                         public function createInjection(\ReflectionClass $class, mixed $context = null): Context
                         {
                             return new Context($context);
@@ -178,7 +178,7 @@ 
             ->bind(
                 ContextInterface::class,
                 new \Spiral\Core\Config\Injectable(
-                    new class implements InjectorInterface {
+                    new class implements InjectorInterface{
                         public function createInjection(\ReflectionClass $class, mixed $context = null): Context
                         {
                             return new Context($context);
@@ -189,14 +189,14 @@ 
 
         FiberHelper::runFiberSequence(
             static fn() => $root->runScope(new Scope(name: 'foo'), static function (ContextInterface $ctx): void {
-                for ($i = 0; $i < 10; $i++) {
+                for ($i = 0; $i < 10; $i++){
                     self::assertInstanceOf(ReflectionParameter::class, $ctx->getValue(), 'Context injected');
                     self::assertSame('ctx', $ctx->getValue()->getName());
                     \Fiber::suspend();
                 }
             }),
             static fn() => $root->runScope(new Scope(name: 'foo'), static function (ContextInterface $context): void {
-                for ($i = 0; $i < 10; $i++) {
+                for ($i = 0; $i < 10; $i++){
                     self::assertInstanceOf(ReflectionParameter::class, $context->getValue(), 'Context injected');
                     self::assertSame('context', $context->getValue()->getName());
                     \Fiber::suspend();
@@ -257,10 +257,10 @@ 
     {
         $root = new Container();
         $context = (object)['destroyed' => false];
-        $class = new class($context) implements DestroyableInterface {
+        $class = new class($context) implements DestroyableInterface{
             public function __construct(
                 private readonly \stdClass $context,
-            ) {
+            ){
             }
 
             public function __destruct()
@@ -288,7 +288,7 @@ 
             new Scope('http'),
             static function () use ($root, $proxy): void {
                 self::assertSame('Foo', $proxy->getName());
-                $proxy->setName(new class implements \Stringable {
+                $proxy->setName(new class implements \Stringable{
                     public function __toString(): string
                     {
                         return 'Bar';

Braces +14 added lines, -7 removed lines

@@ -49,7 +49,8 @@ 
                     // from the current `foo` scope
                     self::assertInstanceOf(KVLogger::class, $logger);
 
-                    for ($i = 0; $i < 10; $i++) {
+                    for ($i = 0; $i < 10; $i++)
+                    {
                         // because of proxy
                         self::assertNotInstanceOf(KVLogger::class, $carrier->getLogger());
                         self::assertSame('kv', $carrier->logger->getName());
@@ -69,7 +70,8 @@ 
                     // from the current `foo` scope
                     self::assertInstanceOf(FileLogger::class, $logger);
 
-                    for ($i = 0; $i < 10; $i++) {
+                    for ($i = 0; $i < 10; $i++)
+                    {
                         // because of proxy
                         self::assertNotInstanceOf(FileLogger::class, $carrier->getLogger());
                         self::assertSame('file', $carrier->logger->getName());
@@ -137,7 +139,8 @@ 
             ->bind(
                 ContextInterface::class,
                 new \Spiral\Core\Config\Injectable(
-                    new class implements InjectorInterface {
+                    new class implements InjectorInterface
+                    {
                         public function createInjection(\ReflectionClass $class, mixed $context = null): Context
                         {
                             return new Context($context);
@@ -178,7 +181,8 @@ 
             ->bind(
                 ContextInterface::class,
                 new \Spiral\Core\Config\Injectable(
-                    new class implements InjectorInterface {
+                    new class implements InjectorInterface
+                    {
                         public function createInjection(\ReflectionClass $class, mixed $context = null): Context
                         {
                             return new Context($context);
@@ -189,14 +193,16 @@ 
 
         FiberHelper::runFiberSequence(
             static fn() => $root->runScope(new Scope(name: 'foo'), static function (ContextInterface $ctx): void {
-                for ($i = 0; $i < 10; $i++) {
+                for ($i = 0; $i < 10; $i++)
+                {
                     self::assertInstanceOf(ReflectionParameter::class, $ctx->getValue(), 'Context injected');
                     self::assertSame('ctx', $ctx->getValue()->getName());
                     \Fiber::suspend();
                 }
             }),
             static fn() => $root->runScope(new Scope(name: 'foo'), static function (ContextInterface $context): void {
-                for ($i = 0; $i < 10; $i++) {
+                for ($i = 0; $i < 10; $i++)
+                {
                     self::assertInstanceOf(ReflectionParameter::class, $context->getValue(), 'Context injected');
                     self::assertSame('context', $context->getValue()->getName());
                     \Fiber::suspend();
@@ -257,7 +263,8 @@ 
     {
         $root = new Container();
         $context = (object)['destroyed' => false];
-        $class = new class($context) implements DestroyableInterface {
+        $class = new class($context) implements DestroyableInterface
+        {
             public function __construct(
                 private readonly \stdClass $context,
             ) {

src/Core/tests/Scope/ExceptionsTest.php

Spacing +6 added lines, -6 removed lines

@@ -21,10 +21,10 @@ 
         $container = new Container();
 
         $container->runScoped(static function (Container $c1): void {
-            try {
+            try{
                 $c1->get(ExceptionConstructor::class);
                 self::fail('Exception should be thrown');
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
                 self::assertInstanceOf(\Exception::class, $e);
                 throw $e;
             }
@@ -39,10 +39,10 @@ 
         $container = new Container();
 
         $container->runScoped(static function (Container $c1): void {
-            try {
+            try{
                 $c1->get(ExceptionConstructor::class);
                 self::fail('Exception should be thrown');
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
                 self::assertInstanceOf(\Exception::class, $e);
                 throw $e;
             }
@@ -57,10 +57,10 @@ 
         $container = new Container();
 
         $container->runScoped(static function (Container $c1): void {
-            try {
+            try{
                 $c1->get(DatetimeCarrier::class);
                 self::fail('Exception should be thrown');
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
                 self::assertInstanceOf(NotFoundException::class, $e);
                 self::assertInstanceOf(NotFoundException::class, $e->getPrevious());
                 self::assertStringContainsString(

Braces +15 added lines, -6 removed lines

@@ -21,10 +21,13 @@ 
         $container = new Container();
 
         $container->runScoped(static function (Container $c1): void {
-            try {
+            try
+            {
                 $c1->get(ExceptionConstructor::class);
                 self::fail('Exception should be thrown');
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 self::assertInstanceOf(\Exception::class, $e);
                 throw $e;
             }
@@ -39,10 +42,13 @@ 
         $container = new Container();
 
         $container->runScoped(static function (Container $c1): void {
-            try {
+            try
+            {
                 $c1->get(ExceptionConstructor::class);
                 self::fail('Exception should be thrown');
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 self::assertInstanceOf(\Exception::class, $e);
                 throw $e;
             }
@@ -57,10 +63,13 @@ 
         $container = new Container();
 
         $container->runScoped(static function (Container $c1): void {
-            try {
+            try
+            {
                 $c1->get(DatetimeCarrier::class);
                 self::fail('Exception should be thrown');
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 self::assertInstanceOf(NotFoundException::class, $e);
                 self::assertInstanceOf(NotFoundException::class, $e->getPrevious());
                 self::assertStringContainsString(

src/Core/tests/ScopesTest.php

Spacing +6 added lines, -6 removed lines

@@ -32,11 +32,11 @@ 
 
         $this->assertNull(ContainerScope::getContainer());
 
-        try {
+        try{
             $this->assertTrue(ContainerScope::runScope($container, function () use ($container): void {
                 throw new RuntimeException('exception');
             }));
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
         }
 
         $this->assertInstanceOf(RuntimeException::class, $e);
@@ -54,7 +54,7 @@ 
         $this->assertTrue($c->runScope([
             'bucket' => new Bucket('b'),
             'other'  => new SampleClass()
-        ], function ($c) {
+        ], function ($c){
             $this->assertSame('b', $c->get('bucket')->getName());
             $this->assertTrue($c->has('other'));
 
@@ -76,21 +76,21 @@ 
         $this->assertTrue($c->runScope([
             'bucket' => new Bucket('b'),
             'other'  => new SampleClass()
-        ], function ($c) {
+        ], function ($c){
             $this->assertSame('b', $c->get('bucket')->getName());
             $this->assertTrue($c->has('other'));
 
             return $c->get('bucket')->getName() == 'b' && $c->has('other');
         }));
 
-        try {
+        try{
             $this->assertTrue($c->runScope([
                 'bucket' => new Bucket('b'),
                 'other'  => new SampleClass()
             ], function () use ($c): void {
                 throw new RuntimeException('exception');
             }));
-        } catch (\Throwable) {
+        }catch (\Throwable){
         }
 
         $this->assertSame('a', $c->get('bucket')->getName());

Braces +17 added lines, -7 removed lines

@@ -32,11 +32,14 @@ 
 
         $this->assertNull(ContainerScope::getContainer());
 
-        try {
+        try
+        {
             $this->assertTrue(ContainerScope::runScope($container, function () use ($container): void {
                 throw new RuntimeException('exception');
             }));
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
         }
 
         $this->assertInstanceOf(RuntimeException::class, $e);
@@ -54,7 +57,8 @@ 
         $this->assertTrue($c->runScope([
             'bucket' => new Bucket('b'),
             'other'  => new SampleClass()
-        ], function ($c) {
+        ], function ($c)
+        {
             $this->assertSame('b', $c->get('bucket')->getName());
             $this->assertTrue($c->has('other'));
 
@@ -76,21 +80,25 @@ 
         $this->assertTrue($c->runScope([
             'bucket' => new Bucket('b'),
             'other'  => new SampleClass()
-        ], function ($c) {
+        ], function ($c)
+        {
             $this->assertSame('b', $c->get('bucket')->getName());
             $this->assertTrue($c->has('other'));
 
             return $c->get('bucket')->getName() == 'b' && $c->has('other');
         }));
 
-        try {
+        try
+        {
             $this->assertTrue($c->runScope([
                 'bucket' => new Bucket('b'),
                 'other'  => new SampleClass()
             ], function () use ($c): void {
                 throw new RuntimeException('exception');
             }));
-        } catch (\Throwable) {
+        }
+        catch (\Throwable)
+        {
         }
 
         $this->assertSame('a', $c->get('bucket')->getName());
@@ -138,7 +146,9 @@ 
     public function testHasInstanceAfterMakeWithoutAliasInScope(): void
     {
         $container = new Container();
-        $container->bindSingleton('test', new #[Singleton] class {});
+        $container->bindSingleton('test', new #[Singleton] class
+        {
+});
         $container->make('test');
 
         $container->runScoped(function (Container $container): void {