Corrected CS style and applied suggestions

Author: reypm

src/GraphQl/Resolver/Stage/SecurityPostDenormalizeStage.php

@@ -45,7 +45,7 @@ public function __invoke(string $resourceClass, string $operationName, array $co
 
         $isGranted = $resourceMetadata->getGraphqlAttribute($operationName, 'security_post_denormalize', null, true);
 
-        if (null === $this->resourceAccessChecker && $isGranted) {
+        if (null !== $isGranted && null === $this->resourceAccessChecker) {
             throw new \LogicException('Cannot check security expression when SecurityBundle is not installed. Try running "composer require symfony/security-bundle".');
         }
 

src/GraphQl/Resolver/Stage/SecurityStage.php

@@ -44,7 +44,8 @@ public function __invoke(string $resourceClass, string $operationName, array $co
         $resourceMetadata = $this->resourceMetadataFactory->create($resourceClass);
 
         $isGranted = $resourceMetadata->getGraphqlAttribute($operationName, 'security', null, true);
-        if (null === $this->resourceAccessChecker && $isGranted) {
+
+        if (null !== $isGranted && null === $this->resourceAccessChecker) {
             throw new \LogicException('Cannot check security expression when SecurityBundle is not installed. Try running "composer require symfony/security-bundle".');
         }
 

tests/GraphQl/Resolver/Stage/SecurityPostDenormalizeStageTest.php

@@ -129,20 +129,14 @@ public function testNoSecurityBundleInstalled(): void
         );
         $this->resourceMetadataFactoryProphecy->create($resourceClass)->willReturn($resourceMetadata);
 
-        $this->securityPostDenormalizeStage = new SecurityPostDenormalizeStage(
-            $this->resourceMetadataFactoryProphecy->reveal(), null
-        );
+        $this->securityPostDenormalizeStage = new SecurityPostDenormalizeStage($this->resourceMetadataFactoryProphecy->reveal(), null);
 
         $info = $this->prophesize(ResolveInfo::class)->reveal();
         $this->expectException(\LogicException::class);
 
-        ($this->securityPostDenormalizeStage)(
-            $resourceClass,
-            'item_query',
-            [
-                'info' => $info,
-                'extra_variables' => $extraVariables,
-            ]
-        );
+        ($this->securityPostDenormalizeStage)($resourceClass,'item_query', [
+            'info' => $info,
+            'extra_variables' => $extraVariables,
+        ]);
     }
 }

tests/GraphQl/Resolver/Stage/SecurityStageTest.php

@@ -105,9 +105,15 @@ public function testNoSecurityBundleInstalled(): void
         $resourceClass = 'myResource';
         $isGranted = 'not_granted';
         $extraVariables = ['extra' => false];
-        $resourceMetadata = (new ResourceMetadata())->withGraphql([$operationName => ['security' => $isGranted]]);
+        $resourceMetadata = (new ResourceMetadata())->withGraphql([
+            $operationName => ['security' => $isGranted],
+        ]);
         $this->resourceMetadataFactoryProphecy->create($resourceClass)->willReturn($resourceMetadata);
+
         $this->expectException(\LogicException::class);
-        ($this->securityStage)($resourceClass, 'item_query', ['extra_variables' => $extraVariables]);
+
+        ($this->securityStage)($resourceClass, 'item_query', [
+            'extra_variables' => $extraVariables
+        ]);
     }
 }