feat: upgrade to doctrine/codingstandard:10 (#115)

Author: canvural

composer.json

@@ -18,7 +18,7 @@
         "thecodingmachine/safe": "^2.2"
     },
     "require-dev": {
-        "doctrine/coding-standard": "^9",
+        "doctrine/coding-standard": "^10",
         "ergebnis/composer-normalize": "^2.27",
         "infection/infection": "^0.26",
         "phpstan/extension-installer": "^1.1",

src/Exception/NoResponse.php

@@ -12,7 +12,7 @@ class NoResponse extends Exception
 {
     protected string $method;
     protected string $path;
-    protected ?string $statusCode = null;
+    protected string|null $statusCode = null;
 
     public static function forPathAndMethod(string $path, string $method): self
     {

src/OpenAPIFaker.php

@@ -66,7 +66,7 @@ public static function createFromYaml(string $yaml): self
     public function mockRequest(
         string $path,
         string $method,
-        string $contentType = 'application/json'
+        string $contentType = 'application/json',
     ): mixed {
         $operation = $this->findOperation($path, $method);
 
@@ -93,7 +93,7 @@ public function mockResponse(
         string $path,
         string $method,
         string $statusCode = '200',
-        string $contentType = 'application/json'
+        string $contentType = 'application/json',
     ): mixed {
         $operation = $this->findOperation($path, $method);
 
@@ -119,9 +119,7 @@ public function mockResponse(
         return (new SchemaFaker($content->schema, $this->options))->generate();
     }
 
-    /**
-     * @throws Exception
-     */
+    /** @throws Exception */
     public function mockComponentSchema(string $schemaName): mixed
     {
         if ($this->openAPISchema->components === null) {
@@ -138,9 +136,7 @@ public function mockComponentSchema(string $schemaName): mixed
         return (new SchemaFaker($schema, $this->options))->generate();
     }
 
-    /**
-     * @param array{minItems?:?int, maxItems?:?int, alwaysFakeOptionals?:bool} $options
-     */
+    /** @param array{minItems?:?int, maxItems?:?int, alwaysFakeOptionals?:bool} $options */
     public function setOptions(array $options): self
     {
         foreach ($options as $key => $value) {
@@ -154,15 +150,13 @@ public function setOptions(array $options): self
         return $this;
     }
 
-    /**
-     * @throws NoPath
-     */
+    /** @throws NoPath */
     private function findOperation(string $path, string $method): Operation
     {
         try {
             $operation = (new LeagueOpenAPI\SpecFinder($this->openAPISchema))
                 ->findOperationSpec(new LeagueOpenAPI\OperationAddress($path, strtolower($method)));
-        } catch (LeagueOpenAPI\Exception\NoPath $e) {
+        } catch (LeagueOpenAPI\Exception\NoPath) {
             throw NoPath::forPathAndMethod($path, $method);
         }
 

src/Options.php

@@ -6,8 +6,8 @@
 
 final class Options
 {
-    private ?int $minItems            = null;
-    private ?int $maxItems            = null;
+    private int|null $minItems        = null;
+    private int|null $maxItems        = null;
     private bool $alwaysFakeOptionals = false;
 
     public function setMinItems(int $minItems): Options
@@ -31,12 +31,12 @@ public function setAlwaysFakeOptionals(bool $alwaysFakeOptionals): self
         return $this;
     }
 
-    public function getMinItems(): ?int
+    public function getMinItems(): int|null
     {
         return $this->minItems;
     }
 
-    public function getMaxItems(): ?int
+    public function getMaxItems(): int|null
     {
         return $this->maxItems;
     }

src/SchemaFaker/ArrayFaker.php

@@ -15,14 +15,10 @@
 use const SORT_REGULAR;
 use const SORT_STRING;
 
-/**
- * @internal
- */
+/** @internal */
 final class ArrayFaker
 {
-    /**
-     * @return array<mixed>
-     */
+    /** @return array<mixed> */
     public static function generate(Schema $schema, Options $options): array
     {
         $minimum = $schema->minItems ?? 0;

src/SchemaFaker/BooleanFaker.php

@@ -9,9 +9,7 @@
 
 use function random_int;
 
-/**
- * @internal
- */
+/** @internal */
 final class BooleanFaker
 {
     public static function generate(Schema $schema): bool

src/SchemaFaker/NumberFaker.php

@@ -9,9 +9,7 @@
 
 use function mt_getrandmax;
 
-/**
- * @internal
- */
+/** @internal */
 final class NumberFaker
 {
     public static function generate(Schema $schema): int|float

src/SchemaFaker/ObjectFaker.php

@@ -14,14 +14,10 @@
 use function count;
 use function in_array;
 
-/**
- * @internal
- */
+/** @internal */
 final class ObjectFaker
 {
-    /**
-     * @return array<mixed>
-     */
+    /** @return array<mixed> */
     public static function generate(Schema $schema, Options $options, bool $request = false): array
     {
         $result = [];

src/SchemaFaker/SchemaFaker.php

@@ -16,26 +16,18 @@
 use function Safe\json_decode;
 use function Safe\json_encode;
 
-/**
- * @internal
- */
+/** @internal */
 final class SchemaFaker
 {
     private Schema $schema;
-    private Options $options;
-    private bool $request;
 
-    public function __construct(Schema $schema, Options $options, bool $request = false)
+    public function __construct(Schema $schema, private Options $options, private bool $request = false)
     {
-        $schemaData    = json_decode(json_encode($schema->getSerializableData()), true);
-        $this->schema  = new Schema($this->resolveOfConstraints($schemaData));
-        $this->options = $options;
-        $this->request = $request;
+        $schemaData   = json_decode(json_encode($schema->getSerializableData()), true);
+        $this->schema = new Schema($this->resolveOfConstraints($schemaData));
     }
 
-    /**
-     * @return array<mixed>|string|bool|int|float
-     */
+    /** @return array<mixed>|string|bool|int|float */
     public function generate(): array|string|bool|int|float
     {
         if ($this->schema->type === 'array') {

src/SchemaFaker/StringFaker.php

@@ -18,9 +18,7 @@
 
 use const DATE_RFC3339;
 
-/**
- * @internal
- */
+/** @internal */
 final class StringFaker
 {
     public static function generate(Schema $schema): string