fix(graphql): resolver before validation (#6363)

fixes #6354

Author: soyuka

features/graphql/mutation.feature

@@ -1034,3 +1034,20 @@ Feature: GraphQL mutation support
     """
     Then the response status code should be 200
     And the JSON node "data.uploadMultipleMediaObject.mediaObject.contentUrl" should be equal to "test.gif"
+
+  @!mongodb
+  Scenario: Mutation should run before validation
+    When I send the following GraphQL request:
+    """
+    mutation {
+      createActivityLog(input: {name: ""}) {
+        activityLog {
+          name
+        }
+      }
+    }
+    """
+    Then the response status code should be 200
+    And the response should be in JSON
+    And the header "Content-Type" should be equal to "application/json"
+    And the JSON node "data.createActivityLog.activityLog.name" should be equal to "hi"

src/Symfony/Bundle/Resources/config/graphql.xml

@@ -158,8 +158,8 @@
             <argument type="tagged_locator" tag="api_platform.parameter_provider" index-by="key" />
         </service>
 
-        <!-- Validation occurs at 200, we want validation to be over when we reach custom resolvers -->
-        <service id="api_platform.graphql.state_provider.resolver" class="ApiPlatform\GraphQl\State\Provider\ResolverProvider" decorates="api_platform.graphql.state_provider" decoration-priority="190">
+        <!-- Validation occurs at 200, we want validation to be after we reach custom resolvers -->
+        <service id="api_platform.graphql.state_provider.resolver" class="ApiPlatform\GraphQl\State\Provider\ResolverProvider" decorates="api_platform.graphql.state_provider" decoration-priority="220">
             <argument type="service" id="api_platform.graphql.state_provider.resolver.inner" />
             <argument type="service" id="api_platform.graphql.resolver_locator" />
         </service>

tests/Fixtures/TestBundle/ApiResource/Issue6354/ActivityLog.php

@@ -0,0 +1,33 @@
+<?php
+
+/*
+ * This file is part of the API Platform project.
+ *
+ * (c) Kévin Dunglas <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace ApiPlatform\Tests\Fixtures\TestBundle\ApiResource\Issue6354;
+
+use ApiPlatform\Metadata\ApiResource;
+use ApiPlatform\Metadata\GraphQl\Mutation;
+use Symfony\Component\Validator\Constraints\NotBlank;
+
+#[ApiResource(
+    graphQlOperations: [
+        new Mutation(
+            resolver: 'app.graphql.mutation_resolver.activity_log',
+            name: 'create'
+        ),
+    ]
+)]
+class ActivityLog
+{
+    public function __construct(#[NotBlank()] public ?string $name = null)
+    {
+    }
+}

tests/Fixtures/TestBundle/ApiResource/Issue6354/CreateActivityLogResolver.php

@@ -0,0 +1,34 @@
+<?php
+
+/*
+ * This file is part of the API Platform project.
+ *
+ * (c) Kévin Dunglas <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace ApiPlatform\Tests\Fixtures\TestBundle\ApiResource\Issue6354;
+
+use ApiPlatform\GraphQl\Resolver\MutationResolverInterface;
+
+class CreateActivityLogResolver implements MutationResolverInterface
+{
+    /**
+     * @param object|null $item
+     * @param mixed[]     $context
+     */
+    public function __invoke($item, array $context): ActivityLog
+    {
+        if (!$item instanceof ActivityLog) {
+            throw new \InvalidArgumentException('Missing input of type ActivityLog');
+        }
+
+        $item->name = 'hi';
+
+        return $item;
+    }
+}

tests/Fixtures/app/config/config_common.yml

@@ -455,3 +455,8 @@ services:
         tags:
             - name: 'api_platform.parameter_provider'
               key: 'ApiPlatform\Tests\Fixtures\TestBundle\Parameter\CustomGroupParameterProvider'
+
+    app.graphql.mutation_resolver.activity_log:
+        class: 'ApiPlatform\Tests\Fixtures\TestBundle\ApiResource\Issue6354\CreateActivityLogResolver'
+        tags:
+            - name: 'api_platform.graphql.resolver'