Merge pull request #10 from ivome/develop

Add test for ommitted clientMutationId, bugfix
ivome · May 9, 2017 · 219d391 · 219d391
2 parents c464c8c + 0c4cd27
commit 219d391
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 1 deletion.
diff --git a/src/Mutation/Mutation.php b/src/Mutation/Mutation.php
@@ -82,7 +82,7 @@ public static function mutationWithClientMutationId(array $config)
             ],
             'resolve' => function ($query, $args, $context, ResolveInfo $info) use ($mutateAndGetPayload) {
                 $payload = call_user_func($mutateAndGetPayload, $args['input'], $context, $info);
-                $payload['clientMutationId'] = $args['input']['clientMutationId'];
+                $payload['clientMutationId'] = isset($args['input']['clientMutationId']) ? $args['input']['clientMutationId'] : null;
                 return $payload;
             }
         ];

diff --git a/tests/Mutation/MutationTest.php b/tests/Mutation/MutationTest.php
@@ -148,6 +148,24 @@ public function testReturnsTheSameClientMutationID()
         $this->assertValidQuery($query, $expected);
     }
 
+    public function testReturnsNullWithOmittedClientMutationID()
+    {
+        $query = 'mutation M {
+            simpleMutation(input: {}) {
+              result
+              clientMutationId
+            }
+          }';
+
+        $expected = [
+            'simpleMutation' => [
+                'result' => 1,
+                'clientMutationId' => null
+            ]
+        ];
+
+        $this->assertValidQuery($query, $expected);
+    }
 
     public function testSupportsEdgeAsOutputField()
     {