Update php-cs-fixer issues

denisyukphp · Mar 30, 2023 · 4610cbf · 4610cbf
1 parent bd2f904
commit 4610cbf
Show file tree

Hide file tree

Showing 13 changed files with 16 additions and 94 deletions.
diff --git a/.github/workflows/continuous-integration.yml b/.github/workflows/continuous-integration.yml
diff --git a/LICENSE.txt b/LICENSE.txt
diff --git a/src/Generator/Generator.php b/src/Generator/Generator.php
@@ -6,8 +6,8 @@
 
 use Orangesoft\BackOff\Duration\DurationInterface;
 use Orangesoft\BackOff\Duration\Nanoseconds;
-use Orangesoft\BackOff\Strategy\StrategyInterface;
 use Orangesoft\BackOff\Jitter\JitterInterface;
+use Orangesoft\BackOff\Strategy\StrategyInterface;
 
 final class Generator implements GeneratorInterface
 {

diff --git a/src/Retry/ExceptionClassifier/ExceptionClassifier.php b/src/Retry/ExceptionClassifier/ExceptionClassifier.php
@@ -12,19 +12,15 @@ final class ExceptionClassifier implements ExceptionClassifierInterface
     public function __construct(
         private array $classNames = [],
     ) {
-        if (0 === count($classNames)) {
+        if (0 === \count($classNames)) {
             $this->classNames = [
                 \Error::class,
                 \Exception::class,
             ];
         } else {
             foreach ($classNames as $className) {
                 if (!class_exists($className) || !is_a($className, \Throwable::class, true)) {
-                    throw new \InvalidArgumentException(
-                        vsprintf('Exception class must be a class that exists and can be thrown, "%s" given.', [
-                            get_debug_type($className),
-                        ])
-                    );
+                    throw new \InvalidArgumentException(vsprintf('Exception class must be a class that exists and can be thrown, "%s" given.', [get_debug_type($className)]));
                 }
 
                 $this->classNames[] = $className;

diff --git a/src/Retry/Retry.php b/src/Retry/Retry.php
@@ -30,7 +30,7 @@ public function call(callable $callback, array $args = []): mixed
 
             $this->backOff->backOff($attempt, $throwable);
 
-            $attempt++;
+            ++$attempt;
 
             goto retrying;
         }

diff --git a/tests/ExponentialBackOffTest.php b/tests/ExponentialBackOffTest.php
@@ -4,8 +4,8 @@
 
 namespace Orangesoft\BackOff\Tests;
 
-use Orangesoft\BackOff\ExponentialBackOff;
 use Orangesoft\BackOff\Duration\Nanoseconds;
+use Orangesoft\BackOff\ExponentialBackOff;
 use Orangesoft\BackOff\Jitter\NullJitter;
 use Orangesoft\BackOff\Sleeper\Sleeper;
 use PHPUnit\Framework\TestCase;

diff --git a/tests/Generator/GeneratorTest.php b/tests/Generator/GeneratorTest.php
@@ -4,12 +4,12 @@
 
 namespace Orangesoft\BackOff\Tests\Generator;
 
-use Orangesoft\BackOff\Generator\Generator;
 use Orangesoft\BackOff\Duration\Nanoseconds;
+use Orangesoft\BackOff\Generator\Generator;
+use Orangesoft\BackOff\Jitter\EqualJitter;
+use Orangesoft\BackOff\Jitter\NullJitter;
 use Orangesoft\BackOff\Strategy\ConstantStrategy;
 use Orangesoft\BackOff\Strategy\LinearStrategy;
-use Orangesoft\BackOff\Jitter\NullJitter;
-use Orangesoft\BackOff\Jitter\EqualJitter;
 use PHPUnit\Framework\TestCase;
 
 class GeneratorTest extends TestCase

diff --git a/tests/LinearBackOffTest.php b/tests/LinearBackOffTest.php
@@ -4,9 +4,9 @@
 
 namespace Orangesoft\BackOff\Tests;
 
-use Orangesoft\BackOff\LinearBackOff;
 use Orangesoft\BackOff\Duration\Nanoseconds;
 use Orangesoft\BackOff\Jitter\NullJitter;
+use Orangesoft\BackOff\LinearBackOff;
 use Orangesoft\BackOff\Sleeper\Sleeper;
 use PHPUnit\Framework\TestCase;
 

diff --git a/tests/Retry/RetryTest.php b/tests/Retry/RetryTest.php
@@ -18,7 +18,7 @@ public function testReturnValue(): void
             exceptionClassifier: new ExceptionClassifier(),
         );
 
-        $result = $retry->call(fn(int $a, int $b): int => $a + $b, [5, 10]);
+        $result = $retry->call(fn (int $a, int $b): int => $a + $b, [5, 10]);
 
         $this->assertSame(15, $result);
     }
@@ -34,7 +34,7 @@ public function testThrowException(): void
 
         $this->expectExceptionObject($throwable);
 
-        $retry->call(fn() => throw $throwable);
+        $retry->call(fn () => throw $throwable);
     }
 
     public function testAttemptsCounter(): void
@@ -45,7 +45,7 @@ public function testAttemptsCounter(): void
         );
 
         try {
-            $retry->call(fn() => throw new \Exception());
+            $retry->call(fn () => throw new \Exception());
         } catch (\Exception) {
             $this->assertSame(3, $attemptsCounterBackOff->getLastAttempt());
         }

diff --git a/tests/Strategy/ConstantStrategyTest.php b/tests/Strategy/ConstantStrategyTest.php
@@ -4,8 +4,8 @@
 
 namespace Orangesoft\BackOff\Tests\Strategy;
 
-use Orangesoft\BackOff\Strategy\ConstantStrategy;
 use Orangesoft\BackOff\Duration\Nanoseconds;
+use Orangesoft\BackOff\Strategy\ConstantStrategy;
 use PHPUnit\Framework\TestCase;
 
 class ConstantStrategyTest extends TestCase

diff --git a/tests/Strategy/DecorrelatedStrategyTest.php b/tests/Strategy/DecorrelatedStrategyTest.php
@@ -4,8 +4,8 @@
 
 namespace Orangesoft\BackOff\Tests\Strategy;
 
-use Orangesoft\BackOff\Strategy\DecorrelatedStrategy;
 use Orangesoft\BackOff\Duration\Nanoseconds;
+use Orangesoft\BackOff\Strategy\DecorrelatedStrategy;
 use PHPUnit\Framework\TestCase;
 
 class DecorrelatedStrategyTest extends TestCase

diff --git a/tests/Strategy/ExponentialStrategyTest.php b/tests/Strategy/ExponentialStrategyTest.php
@@ -4,8 +4,8 @@
 
 namespace Orangesoft\BackOff\Tests\Strategy;
 
-use Orangesoft\BackOff\Strategy\ExponentialStrategy;
 use Orangesoft\BackOff\Duration\Nanoseconds;
+use Orangesoft\BackOff\Strategy\ExponentialStrategy;
 use PHPUnit\Framework\TestCase;
 
 class ExponentialStrategyTest extends TestCase

diff --git a/tests/Strategy/LinearStrategyTest.php b/tests/Strategy/LinearStrategyTest.php
@@ -4,8 +4,8 @@
 
 namespace Orangesoft\BackOff\Tests\Strategy;
 
-use Orangesoft\BackOff\Strategy\LinearStrategy;
 use Orangesoft\BackOff\Duration\Nanoseconds;
+use Orangesoft\BackOff\Strategy\LinearStrategy;
 use PHPUnit\Framework\TestCase;
 
 class LinearStrategyTest extends TestCase