SimLibaud / SilRouteSecurityBundle

Tests/Twig/RouteSecurityExtensionTest.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
         $twig_functions = $routeSecurityExtension->getFunctions();
         $this->assertEquals(3, count($twig_functions));
 
-        $functions =  ['hasUserAccessToRoute', 'hasUserAccessToRoutes', 'hasUserAccessAtLeastOneRoute'];
+        $functions = ['hasUserAccessToRoute', 'hasUserAccessToRoutes', 'hasUserAccessAtLeastOneRoute'];
         foreach ($twig_functions as $index => $twig_function) {
             $this->assertTrue(in_array($twig_function->getName(), $functions));
         }

Tests/Security/AccessControlTest.php

Spacing +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
         $routeToRoleConverter = $this->createMock(NamingStrategyInterface::class);
         $routeToRoleConverter
             ->method('generateRoleForRoute')
-            ->will($this->returnCallback(function ($route) {
+            ->will($this->returnCallback(function($route) {
                 return 'ROLE_'.strtoupper($route);
             }));
         $configuration = [

Security/AccessControl.php

Spacing +1 added lines, -1 removed lines

@@ -135,7 +135,7 @@
         $all_secured_routes = [];
         $configured_routes = array_keys($this->router->getRouteCollection()->all());
 
-        foreach ($configured_routes as $configured_route) if (true === $this->isRouteSecure($configured_route)){
+        foreach ($configured_routes as $configured_route) if (true === $this->isRouteSecure($configured_route)) {
             $all_secured_routes[] = $configured_route;
         }
 

Braces +4 added lines, -2 removed lines

@@ -135,8 +135,10 @@
         $all_secured_routes = [];
         $configured_routes = array_keys($this->router->getRouteCollection()->all());
 
-        foreach ($configured_routes as $configured_route) if (true === $this->isRouteSecure($configured_route)){
-            $all_secured_routes[] = $configured_route;
+        foreach ($configured_routes as $configured_route) {

+            if (true === $this->isRouteSecure($configured_route)){
+            $all_secured_routes[] = $configured_route;

+        }

         }
 
         return $all_secured_routes;

Tests/Role/RolesProviderTest.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
         $routeToRoleConverter
             ->expects($this->exactly(count($secured_routes)))
             ->method('generateRoleForRoute')
-            ->will($this->returnCallback(function ($route) {
+            ->will($this->returnCallback(function($route) {
                 return 'ROLE_'.strtoupper($route);
             }));
         $rolesProvider = new RolesProvider($routeToRoleConverter, $secured_routes);

Tests/Listener/AccessControlListenerTest.php

Spacing +1 added lines, -1 removed lines

@@ -167,7 +167,7 @@
             ->expects($this->once())
             ->method('dispatch')
             ->with($this->isInstanceOf(AccessDeniedToRouteEvent::class), AccessDeniedToRouteEvent::ON_ACCESS_DENIED_TO_ROUTE)
-            ->will($this->returnCallback(function ($access_denied_event, $name) use ($response) {
+            ->will($this->returnCallback(function($access_denied_event, $name) use ($response) {
                 $access_denied_event->setResponse($response);
                 return $access_denied_event;
             }));

Controller/ExportJsSecuredRoutesController.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
 
         $cache = new FilesystemAdapter('sil_route_security_bundle', 0, $this->cacheDir);
 
-        $securedRoutesWithUserPermission = $cache->get($cacheKey, function (ItemInterface $item) use ($user){
+        $securedRoutesWithUserPermission = $cache->get($cacheKey, function(ItemInterface $item) use ($user){
             $item->expiresAfter(3600);
 
             $securedRoutesWithUserPermission = [];

Tests/Controller/ExportJsSecuredRoutesControllerTest.php

Indentation +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 SilRouteSecurity.addSecuredRoutes('app_secured_route_test', true);
 
 JAVASCRIPT
-       , $response->getContent());
+        , $response->getContent());
     }
 
     public function testWithUserHasNotAccessToRoute()

Spacing +3 added lines, -3 removed lines

@@ -11,7 +11,7 @@ 
 {
     public function testWithUnauthenticatedUser()
     {
-        $client  = static::createClient();
+        $client = static::createClient();
 
         $crawler  = $client->request('GET', '/sil-route-security/export-js-secured-routes.js');
         $response = $client->getResponse();
@@ -21,7 +21,7 @@ 
 
     public function testWithUserHasAccessToRoute()
     {
-        $client  = static::createClient();
+        $client = static::createClient();
 
         $user = $this->createMock(UserInterface::class);
         $user->method('getRoles')->willReturn(['ROLE_APP_SECURED_ROUTE_TEST']);
@@ -41,7 +41,7 @@ 
 
     public function testWithUserHasNotAccessToRoute()
     {
-        $client  = static::createClient();
+        $client = static::createClient();
 
         $user = $this->createMock(UserInterface::class);
         $user->method('getRoles')->willReturn([]);