Merge pull request #54 from moufmouf/null_unlogged_me

Unlogged "me" query now returns null.
thecodingmachine · Feb 21, 2020 · 16f1bf3 · 16f1bf3
2 parents 3f85584 + 6b15dd1
commit 16f1bf3
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 128 deletions.
diff --git a/Controller/GraphQL/MeController.php b/Controller/GraphQL/MeController.php
@@ -41,9 +41,8 @@ public function me(): ?UserInterface
         $user = $token->getUser();
 
         if (!$user instanceof UserInterface) {
-            // getUser() can be an object with a toString or a string
-            $userName = (string) $user;
-            $user = new BasicUser($userName);
+            // getUser() can be the "anon." string.
+            return null;
         }
 
         return $user;

diff --git a/Tests/FunctionalTest.php b/Tests/FunctionalTest.php
@@ -290,10 +290,7 @@ public function testMeQuery(): void
 
         $this->assertSame([
             'data' => [
-                'me' => [
-                    'userName' => 'anon.',
-                    'roles' => [],
-                ]
+                'me' => null
             ]
         ], $result);
 

diff --git a/Tests/Types/BasicUserTest.php b/Tests/Types/BasicUserTest.php
diff --git a/Types/BasicUser.php b/Types/BasicUser.php