Autofix

Author: autofix-ci[bot]

docs/class-reference.md

@@ -70,7 +70,8 @@ static function executeQuery(
     ?array $variableValues = null,
     ?string $operationName = null,
     ?callable $fieldResolver = null,
-    ?array $validationRules = null
+    ?array $validationRules = null,
+    ?GraphQL\Validator\ValidationCache $cache = null
 ): GraphQL\Executor\ExecutionResult
 ```
 
@@ -98,7 +99,8 @@ static function promiseToExecute(
     ?array $variableValues = null,
     ?string $operationName = null,
     ?callable $fieldResolver = null,
-    ?array $validationRules = null
+    ?array $validationRules = null,
+    ?GraphQL\Validator\ValidationCache $cache = null
 ): GraphQL\Executor\Promise\Promise
 ```
 
@@ -1811,7 +1813,8 @@ static function validate(
     GraphQL\Type\Schema $schema,
     GraphQL\Language\AST\DocumentNode $ast,
     ?array $rules = null,
-    ?GraphQL\Utils\TypeInfo $typeInfo = null
+    ?GraphQL\Utils\TypeInfo $typeInfo = null,
+    ?GraphQL\Validator\ValidationCache $cache = null
 ): array
 ```
 

src/Validator/ValidationCache.php

@@ -26,7 +26,7 @@ interface ValidationCache
      * for example, when using persisted queries that are known to be valid ahead of time. In such cases, you
      * can implement this method to always return true.
      *
-     * @return bool True if validation for the given schema + AST is already known to be valid; false otherwise.
+     * @return bool true if validation for the given schema + AST is already known to be valid; false otherwise
      */
     public function isValidated(Schema $schema, DocumentNode $ast): bool;
 
@@ -37,4 +37,4 @@ public function isValidated(Schema $schema, DocumentNode $ast): bool;
      * You should store enough information to recognize this combination on future requests.
      */
     public function markValidated(Schema $schema, DocumentNode $ast): void;
-}
+}

tests/Executor/TestClasses/SpyValidationCacheAdapter.php

@@ -1,5 +1,4 @@
-<?php
-
+<?php declare(strict_types=1);
 
 namespace GraphQL\Tests\Executor\TestClasses;
 
@@ -10,19 +9,20 @@
 final class SpyValidationCacheAdapter extends PsrValidationCacheAdapter
 {
     public int $isValidatedCalls = 0;
+
     public int $markValidatedCalls = 0;
 
     public function isValidated(Schema $schema, DocumentNode $ast): bool
     {
-        $this->isValidatedCalls++;
+        ++$this->isValidatedCalls;
 
         return parent::isValidated($schema, $ast);
     }
 
     public function markValidated(Schema $schema, DocumentNode $ast): void
     {
-        $this->markValidatedCalls++;
+        ++$this->markValidatedCalls;
 
         parent::markValidated($schema, $ast);
     }
-}
+}

tests/Executor/ValidationWithCacheTest.php

@@ -79,9 +79,8 @@ public function testIsValidationCachedWithAdapter(): void
         }';
 
         // make the same call twice in a row. We'll then inspect the cache object to count calls
-        $resultA = GraphQL::executeQuery( $schema, $query, null, null, null, null, null, null, $cache)->toArray();
-        $resultB = GraphQL::executeQuery( $schema, $query, null, null, null, null, null, null, $cache)->toArray();
-
+        $resultA = GraphQL::executeQuery($schema, $query, null, null, null, null, null, null, $cache)->toArray();
+        $resultB = GraphQL::executeQuery($schema, $query, null, null, null, null, null, null, $cache)->toArray();
 
         // ✅ Assert that validation only happened once
         self::assertEquals(2, $cache->isValidatedCalls, 'Should check cache twice');

tests/PsrValidationCacheAdapter.php

@@ -12,34 +12,33 @@
 class PsrValidationCacheAdapter implements ValidationCache
 {
     private const KEY_PREFIX = 'gql_validation_';
+
     private int $ttl;
+
     private CacheInterface $cache;
 
     public function __construct(
-        CacheInterface  $cache,
+        CacheInterface $cache,
         int $ttlSeconds = 300
     ) {
         $this->ttl = $ttlSeconds;
         $this->cache = $cache;
     }
 
-    /**
-     * @throws InvalidArgumentException
-     */
+    /** @throws InvalidArgumentException */
     public function isValidated(Schema $schema, DocumentNode $ast): bool
     {
         try {
             $key = $this->buildKey($schema, $ast);
+
             /** @phpstan-ignore missingType.checkedException */
             return $this->cache->has($key);
         } catch (\Throwable $e) {
             return false;
         }
     }
 
-    /**
-     * @throws InvalidArgumentException
-     */
+    /** @throws InvalidArgumentException */
     public function markValidated(Schema $schema, DocumentNode $ast): void
     {
         try {