removes security dependency from IsGranted function constructor

Author: sgehrig

src/ExpressionLanguage/ExpressionFunction/Security/IsGranted.php

@@ -5,19 +5,18 @@
 namespace Overblog\GraphQLBundle\ExpressionLanguage\ExpressionFunction\Security;
 
 use Overblog\GraphQLBundle\ExpressionLanguage\ExpressionFunction;
-use Overblog\GraphQLBundle\Security\Security;
 
 final class IsGranted extends ExpressionFunction
 {
-    public function __construct(Security $security)
+    public function __construct()
     {
         parent::__construct(
             'isGranted',
             static function ($attributes, $object = 'null'): string {
                 return \sprintf('$globalVariable->get(\'security\')->isGranted(%s, %s)', $attributes, $object);
             },
-            static function ($_, $attributes, $object = null) use ($security): bool {
-                return $security->isGranted($attributes, $object);
+            static function ($arguments, $attributes, $object = null): bool {
+                return $arguments['globalVariable']->get('security')->isGranted($attributes, $object);
             }
         );
     }

tests/ExpressionLanguage/ExpressionFunction/Security/IsGrantedTest.php

@@ -4,24 +4,28 @@
 
 namespace Overblog\GraphQLBundle\Tests\ExpressionLanguage\ExpressionFunction\Security;
 
+use Overblog\GraphQLBundle\Definition\GlobalVariables;
 use Overblog\GraphQLBundle\ExpressionLanguage\ExpressionFunction\Security\IsGranted;
 use Overblog\GraphQLBundle\Tests\ExpressionLanguage\TestCase;
 
 class IsGrantedTest extends TestCase
 {
     protected function getFunctions()
     {
-        $security = $this->getSecurityIsGrantedWithExpectation(
-            $this->matchesRegularExpression('/^ROLE_(USER|ADMIN)$/'),
-            $this->any()
-        );
-
-        return [new IsGranted($security)];
+        return [new IsGranted()];
     }
 
     public function testEvaluator(): void
     {
-        $this->assertTrue($this->expressionLanguage->evaluate('isGranted("ROLE_USER")'));
+        $security       = $this->getSecurityIsGrantedWithExpectation(
+            $this->matchesRegularExpression('/^ROLE_(USER|ADMIN)$/'),
+            $this->any()
+        );
+        $globalVariable = new GlobalVariables(['security' => $security]);
+
+        $this->assertTrue(
+            $this->expressionLanguage->evaluate('isGranted("ROLE_USER")', ['globalVariable' => $globalVariable])
+        );
     }
 
     public function testIsGranted(): void