Update: package, php version, ci, everything !

.github/dependabot.yml

@@ -5,8 +5,7 @@
 
 version: 2
 updates:
-  - package-ecosystem: "" # See documentation for possible values
+  - package-ecosystem: "composer" # See documentation for possible values
     directory: "/" # Location of package manifests
     schedule:
       interval: "weekly"
-

.github/workflows/continuous-integration.yml

@@ -21,9 +21,10 @@ jobs:
           - "highest"
           - "locked"
         php-version:
-          - "8.1"
           - "8.2"
           - "8.3"
+          - "8.4"
+          - "8.5"
         operating-system:
           - "ubuntu-latest"
           - "windows-latest"
@@ -57,9 +58,10 @@ jobs:
         dependencies:
           - "locked"
         php-version:
-          - "8.1"
           - "8.2"
           - "8.3"
+          - "8.4"
+          - "8.5"
         operating-system:
           - "ubuntu-latest"
 
@@ -92,7 +94,7 @@ jobs:
         dependencies:
           - "locked"
         php-version:
-          - "8.3"
+          - "8.4"
         operating-system:
           - "ubuntu-latest"
 

.github/workflows/test-coverage.yml

@@ -18,7 +18,7 @@ jobs:
         uses: "shivammathur/setup-php@v2"
         with:
           coverage: "pcov"
-          php-version: "8.3"
+          php-version: "8.4"
           ini-values: memory_limit=-1
 
       - name: "Install dependencies"

composer.json

@@ -20,18 +20,18 @@
         }
     ],
     "require": {
-        "php": "^8.1.0",
+        "php": "^8.2.0",
         "league/openapi-psr7-validator": "0.*",
         "devizzent/cebe-php-openapi": "^1.0",
-        "psr/http-message": "^1.0",
+        "psr/http-message": "^2.0",
         "psr/http-client": "^1.0",
         "psr/http-factory": "^1.0",
         "psr/log": "^1.0|^2.0|^3.0",
         "fakerphp/faker": "^1.20"
     },
     "require-dev": {
         "friendsofphp/php-cs-fixer": "^3.11",
-        "phpstan/phpstan": "^1.8",
-        "phpunit/phpunit": "^9.5"
+        "phpstan/phpstan": "^2.1",
+        "phpunit/phpunit": "^11.0 || ^12.0"
     }
 }

src/Bridge/LeagueOpenAPIValidation/Exception/InvalidOpenApiDefinitionException.php

@@ -9,7 +9,7 @@ class InvalidOpenApiDefinitionException extends InvalidArgumentException
 {
     public function __construct(Throwable $previous)
     {
-        parent::__construct(sprintf(
+        parent::__construct(\sprintf(
             "The given OpenApi definition can't be loaded:\n%s -> %s",
             $previous::class,
             $previous->getMessage()

src/Bridge/LeagueOpenAPIValidation/Factory.php

@@ -14,7 +14,7 @@
 
     public function __construct(
         private readonly ValidatorBuilder $validator,
         ValidationExceptionMapper $mapper = null
     ) {
         $this->mapper = $mapper ?? new ValidationExceptionMapper();
     }
@@ -23,7 +23,7 @@
     {
         if (!is_readable($path)) {
             throw new InvalidArgumentException(
-                sprintf('Filename given isn\'t readable: %s', $path)
+                \sprintf('Filename given isn\'t readable: %s', $path)
             );
         }
 
@@ -36,7 +36,7 @@
     {
         if (!is_readable($path)) {
             throw new InvalidArgumentException(
-                sprintf('Filename given isn\'t readable: %s', $path)
+                \sprintf('Filename given isn\'t readable: %s', $path)
             );
         }
 

src/Http/Factory/Headers.php

@@ -25,7 +25,18 @@ public function __construct(mixed $headers = [])
         }
 
         foreach ($headers as $name => $values) {
-            $this->set($name, $values);
+            if (\is_string($values)) {
+                $this->set($name, $values);
+            } elseif (\is_array($values)) {
+                foreach ($values as $value) {
+                    if (!\is_string($value)) {
+                        throw new InvalidArgumentException('Header values must be string[]|string.');
+                    }
+                    $this->set($name, $value);
+                }
+            } else {
+                throw new InvalidArgumentException('Header values must be string[]|string.');
+            }
         }
     }
 
@@ -40,7 +51,7 @@ public function __toString(): string
         foreach ($this->headers as $name => $values) {
             $name = ucwords($name, '-');
             foreach ($values as $value) {
-                $asString .= sprintf('%s: %s', $name, $value).PHP_EOL;
+                $asString .= \sprintf('%s: %s', $name, $value).PHP_EOL;
             }
         }
 

src/Http/Factory/Resolver/FakerValueResolver.php

@@ -38,7 +38,7 @@ private function resolveFakerValue(array $matches): mixed
             );
         } catch (InvalidArgumentException) {
         } catch (JsonException $error) {
-            $message = sprintf(
+            $message = \sprintf(
                 "Can't extract the arguments to call method %s: [%s]",
                 $matches[1],
                 $matches[2]

src/Http/Factory/Uri.php

@@ -12,18 +12,42 @@ class Uri implements Stringable
     public function __construct(mixed $value)
     {
         if (\is_array($value)) {
-            if (!isset($value['base'])) {
+            if (!\is_string($value['base'])) {
                 throw new InvalidArgumentException(
                     'If you want to build an URI from an array, use the schema: [ base => string, ?parameters => [string => mixed]'
                 );
             }
 
+            // Check if parameters is a valid string[]
             $parameters = $value['parameters'] ?? [];
-            $value = str_replace(array_keys($parameters), $parameters, (string) $value['base']);
-        } elseif (!\is_string($value)) {
+            if (!\is_array($parameters)) {
+                throw new InvalidArgumentException('value parameters must be an array.');
+            }
+
+            $search = [];
+            $replace = [];
+            foreach ($parameters as $name => $parameter) {
+                if (!\is_string($name) || !\is_string($parameter)) {
+                    throw new InvalidArgumentException(\sprintf(
+                        'Invalid parameter given {name: %s, parameter: %s}',
+                        var_export($name, true),
+                        var_export($parameter, true)
+                    ));
+                }
+                $search[] = $name;
+                $replace[] = $parameter;
+            }
+
+            $this->uri = str_replace(
+                $search,
+                $replace,
+                (string) $value['base']
+            );
+        } elseif (\is_string($value)) {
+            $this->uri = $value;
+        } else {
             throw new InvalidArgumentException('$value must be a string or an array.');
         }
-        $this->uri = $value;
     }
 
     public function __toString(): string

src/Validator/Exception/ApiSchemaException.php

@@ -10,7 +10,7 @@ class ApiSchemaException extends RuntimeException implements ValidationException
     public function __construct(
         Throwable $previous
     ) {
-        $message = sprintf(
+        $message = \sprintf(
             'Something went wrong with the API schema: %s.',
             $previous->getMessage()
         );

src/Validator/Exception/DataSchemaException.php

@@ -12,7 +12,7 @@ public function __construct(
         public readonly string $path,
         Throwable $previous
     ) {
-        $message = sprintf(
+        $message = \sprintf(
             "Data validation failed for property \"%s\", invalid value: %s\n\n> %s",
             $path,
             var_export($value, true),

src/Validator/Exception/GenericException.php

@@ -10,7 +10,7 @@ class GenericException extends RuntimeException implements ValidationException
     public function __construct(
         Throwable $previous
     ) {
-        $message = sprintf(
+        $message = \sprintf(
             'Something went wrong: %s.',
             $previous->getMessage()
         );

src/Validator/Exception/OperationNotFoundException.php

@@ -10,9 +10,9 @@
 {
     public function __construct(
         public readonly RequestInterface $request,
         Throwable $previous = null
     ) {
-        $message = sprintf(
+        $message = \sprintf(
             'API operation for request [%s] %s hasn\'t been found.',
             $request->getMethod(),
             $request->getUri()

src/Validator/Exception/ResponseNotExpectedException.php

@@ -14,7 +14,7 @@ public function __construct(
         public readonly ResponseInterface $response,
         Throwable $previous
     ) {
-        $message = sprintf(
+        $message = \sprintf(
             'API response with status code %d isn\'t defined in the spec for request [%s] %s.',
             $response->getStatusCode(),
             $request->getMethod(),

src/Validator/Expectation/ExpectationCollection.php

@@ -22,7 +22,7 @@ public function __construct(
     }
 
     /**
-     * @return ArrayIterator<int, ResponseExpectationInterface>
+     * @return ArrayIterator<int|string, ResponseExpectationInterface>
      */
     public function getIterator(): ArrayIterator
     {

src/Validator/Expectation/StatusCode.php

@@ -16,7 +16,7 @@ public function verify(ResponseInterface $message): ?ExpectationFailedException
         return $message->getStatusCode() === $this->statusCode
             ? null
             : new ExpectationFailedException(
-                sprintf('Unexpected status code %s, expected %s', $message->getStatusCode(), $this->statusCode),
+                \sprintf('Unexpected status code %s, expected %s', $message->getStatusCode(), $this->statusCode),
                 $this
             );
     }

src/Validator/LoggedRequestValidator.php

@@ -15,13 +15,13 @@ public function __construct(
 
     public function validate(RequestInterface $request): void
     {
-        $this->logger->debug(sprintf(
+        $this->logger->debug(\sprintf(
             "Start testing Request: [%s] %s",
             $request->getMethod(),
             $request->getUri()
         ));
         $this->decorated->validate($request);
-        $this->logger->debug(sprintf(
+        $this->logger->debug(\sprintf(
             'Finish testing Request: [%s] %s',
             $request->getMethod(),
             $request->getUri()

src/Validator/LoggedResponseValidator.php

@@ -16,15 +16,15 @@ public function __construct(
 
     public function validate(ResponseInterface $response, RequestInterface $request): void
     {
-        $this->logger->debug(sprintf(
+        $this->logger->debug(\sprintf(
             'Start testing Response: [%d] %s',
             $response->getStatusCode(),
             $response->getReasonPhrase()
         ));
 
         $this->decorated->validate($response, $request);
 
-        $this->logger->debug(sprintf(
+        $this->logger->debug(\sprintf(
             'Finish testing Response: [%d] %s',
             $response->getStatusCode(),
             $response->getReasonPhrase()