View GitHub Repository GraphQLBundle

Schedule Inspection
Subscribe

Scrutinizer GitHub App not installed

We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.

Install GitHub App

Completed
Pull Request — master (#73)
by Jérémiah
04:58
created
Status
Category
Toggle Patches
Tests/Request/ExecutorTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -31,7 +31,7 @@  discard block
 
 block discarded – undo
31 31 
             'fields' => [
32 32 
                 'myField' => [
33 33 
                     'type' => Type::boolean(),
34 
-                    'resolve' => function () {
34 
+                    'resolve' => function() {
35 35 
                         return false;
36 36 
                     },
37 37 
                 ],
 
@@ -46,7 +46,7 @@  discard block
 
 block discarded – undo
46 46 
      */
47 47 
     public function testInvalidExecutorReturnNotObject()
48 48 
     {
49 
-        $this->executor->setExecutor(function () {
49 
+        $this->executor->setExecutor(function() {
50 50 
             return false;
51 51 
         });
52 52 
         $this->executor->execute($this->request);
 
@@ -58,7 +58,7 @@  discard block
 
 block discarded – undo
58 58 
      */
59 59 
     public function testInvalidExecutorReturnInvalidObject()
60 60 
     {
61 
-        $this->executor->setExecutor(function () {
61 
+        $this->executor->setExecutor(function() {
62 62 
             return new \stdClass();
63 63 
         });
64 64 
         $this->executor->execute($this->request);
Please login to merge, or discard this patch.
Resolver/AccessResolver.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -39,11 +39,11 @@  discard block
 
 block discarded – undo
39 39 
     {
40 40 
         $result = call_user_func_array($resolveCallback, $resolveArgs);
41 41
42 
-        $onFulFilled = function ($result) use ($accessChecker, $resolveArgs) {
42 
+        $onFulFilled = function($result) use ($accessChecker, $resolveArgs) {
43 43 
             switch (true) {
44 44 
                 case is_array($result):
45 45 
                     $result = array_map(
46 
-                        function ($object) use ($accessChecker, $resolveArgs) {
46 
+                        function($object) use ($accessChecker, $resolveArgs) {
47 47 
                             return $this->hasAccess($accessChecker, $object, $resolveArgs) ? $object : null;
48 48 
                         },
49 49 
                         $result
 
@@ -51,7 +51,7 @@  discard block
 
 block discarded – undo
51 51 
                     break;
52 52 
                 case $result instanceof Connection:
53 53 
                     $result->edges = array_map(
54 
-                        function (Edge $edge) use ($accessChecker, $resolveArgs) {
54 
+                        function(Edge $edge) use ($accessChecker, $resolveArgs) {
55 55 
                             $edge->node = $this->hasAccess($accessChecker, $edge->node, $resolveArgs) ? $edge->node : null;
56 56
57 57 
                             return $edge;
Please login to merge, or discard this patch.