fix CI

Author: nguyenk

.travis.yml

@@ -13,24 +13,13 @@ env:
 matrix:
   fast_finish: true
   include:
-      # Test the latest stable release
-    - php: 7.2
-    - php: 8.0
-      env: COVERAGE=true PHPUNIT_FLAGS="-v --coverage-text"
-
       # Test LTS versions.
-    - php: 8.0
-      env: DEPENDENCIES="symfony/lts:^4"
-
-      # Latest commit to master
-    - php: 8.0
-      env: STABILITY="dev"
+    - php: 8.1
+      env: DEPENDENCIES="symfony/^6"
 
   allow_failures:
     # Minimum supported dependencies with the latest and oldest PHP version
-    - php: 8.0
-      env: COMPOSER_FLAGS="--prefer-stable --prefer-lowest" SYMFONY_DEPRECATIONS_HELPER="weak_vendors"
-    - php: 7.2
+    - php: 8.1
       env: COMPOSER_FLAGS="--prefer-stable --prefer-lowest" SYMFONY_DEPRECATIONS_HELPER="weak_vendors"
     # Dev-master is allowed to fail.
     - env: STABILITY="dev"

composer.json

@@ -23,9 +23,9 @@
     "doctrine/annotations": "^1.13"
   },
   "require-dev": {
-    "phpunit/phpunit": "^8.4.1",
+    "phpunit/phpunit": "^9.6.5",
     "mouf/picotainer": "^1.1",
-    "phpstan/phpstan": "^0.12.14",
+    "phpstan/phpstan": "^1.8",
     "php-coveralls/php-coveralls": "^2.1.0",
     "symfony/translation": "^6",
     "doctrine/coding-standard": "^11.1"

phpstan.neon

@@ -1,5 +1,10 @@
 parameters:
     ignoreErrors:
 
+    excludePaths:
+            - vendor
+            - cache
+            - .phpstan-cache
+
 #includes:
 #    - vendor/thecodingmachine/phpstan-strict-rules/phpstan-strict-rules.neon

src/Annotations/Assertion.php

@@ -28,9 +28,7 @@ class Assertion implements ParameterAnnotationInterface
     /** @var Constraint[] */
     private $constraint;
 
-    /**
-     * @param array<string, mixed> $values
-     */
+    /** @param array<string, mixed> $values */
     public function __construct(array $values)
     {
         if (! isset($values['for'])) {
@@ -50,9 +48,7 @@ public function getTarget(): string
         return $this->for;
     }
 
-    /**
-     * @return Constraint[]
-     */
+    /** @return Constraint[] */
     public function getConstraint(): array
     {
         return $this->constraint;

src/ConstraintViolationException.php

@@ -16,6 +16,7 @@ class ConstraintViolationException extends Exception implements GraphQLException
     public function __construct(ConstraintViolationInterface $violation)
     {
         parent::__construct((string) $violation->getMessage(), 400);
+
         $this->violation = $violation;
     }
 

src/Mappers/Parameters/AssertParameterMiddleware.php

@@ -39,7 +39,7 @@ public function __construct(ConstraintValidatorFactoryInterface $constraintValid
         $this->translator = $translator;
     }
 
-    public function mapParameter(ReflectionParameter $refParameter, DocBlock $docBlock, ?Type $paramTagType, ParameterAnnotations $parameterAnnotations, ParameterHandlerInterface $next): ParameterInterface
+    public function mapParameter(ReflectionParameter $refParameter, DocBlock $docBlock, Type|null $paramTagType, ParameterAnnotations $parameterAnnotations, ParameterHandlerInterface $next): ParameterInterface
     {
         /** @var Assertion[] $assertionAnnotations */
         $assertionAnnotations = $parameterAnnotations->getAnnotationsByType(Assertion::class);

src/Mappers/Parameters/ParameterValidator.php

@@ -29,9 +29,7 @@ class ParameterValidator implements InputTypeParameterInterface
     /** @var TranslatorInterface */
     private $translator;
 
-    /**
-     * @param Constraint[] $constraints
-     */
+    /** @param Constraint[] $constraints */
     public function __construct(InputTypeParameterInterface $parameter, string $parameterName, array $constraints, ConstraintValidatorFactoryInterface $constraintValidatorFactory, ValidatorInterface $validator, TranslatorInterface $translator)
     {
         $this->parameter = $parameter;
@@ -48,7 +46,7 @@ public function __construct(InputTypeParameterInterface $parameter, string $para
      *
      * @return mixed
      */
-    public function resolve(?object $source, array $args, $context, ResolveInfo $info)
+    public function resolve(object|null $source, array $args, $context, ResolveInfo $info)
     {
         $value = $this->parameter->resolve($source, $args, $context, $info);
 
@@ -79,9 +77,7 @@ public function hasDefaultValue(): bool
         return $this->parameter->hasDefaultValue();
     }
 
-    /**
-     * @return mixed
-     */
+    /** @return mixed */
     public function getDefaultValue()
     {
         return $this->parameter->getDefaultValue();

src/ValidationFailedException.php

@@ -16,12 +16,11 @@ class ValidationFailedException extends InvalidArgumentException implements Grap
     /** @var ConstraintViolationException[] */
     private $exceptions = [];
 
-    /**
-     * @param ConstraintViolationListInterface<ConstraintViolationInterface> $constraintViolationList
-     */
+    /** @param ConstraintViolationListInterface<ConstraintViolationInterface> $constraintViolationList */
     public function __construct(ConstraintViolationListInterface $constraintViolationList)
     {
         parent::__construct('Validation failed:', 400);
+
         foreach ($constraintViolationList as $constraintViolation) {
             $this->add($constraintViolation);
         }
@@ -33,9 +32,7 @@ private function add(ConstraintViolationInterface $violation): void
         $this->message .= "\n" . $violation->getMessage();
     }
 
-    /**
-     * @return (ClientAware&Throwable)[]
-     */
+    /** @return (ClientAware&Throwable)[] */
     public function getExceptions(): array
     {
         return $this->exceptions;