cross-solution / YAWIK

module/Jobs/test/JobsTest/Acl/WriteAssertionTest.php

Spacing +21 added lines, -21 removed lines

@@ -65,19 +65,19 @@ 
 

         $privileges = array('new', 'test', 'edit');

 

-        foreach(array($role, $user) as $testObj)

+        foreach (array($role, $user) as $testObj)

         {

             // non user is always false?

-            foreach($privileges as $p)

+            foreach ($privileges as $p)

             {

-                $this->assertFalse((bool)$target->assert($acl, $testObj, null, $p));

+                $this->assertFalse((bool) $target->assert($acl, $testObj, null, $p));

             }

 

             // user and wrong privilege is false?

-            foreach($privileges as $p)

+            foreach ($privileges as $p)

             {

                 // Casting null to false is safe here, 'edit' for pair user-privilege tested after

-                $this->assertFalse((bool)$target->assert($acl, $testObj, $job, $p));

+                $this->assertFalse((bool) $target->assert($acl, $testObj, $job, $p));

             }

         }

 

@@ -117,15 +117,15 @@ 
         //$user->setOrganization($organization);

 

         /** Organization without user **/

-        $this->assertFalse( $assertion->checkOrganizationPermissions( $user, $job ) );

+        $this->assertFalse($assertion->checkOrganizationPermissions($user, $job));

     }

 

     /**

      * @dataProvider assertParametersWithoutOrganization

      */

-    public function testAssertWithoutOrganisation($input, $expected){

+    public function testAssertWithoutOrganisation($input, $expected) {

 

-            $method="assert".($expected?"True":"False");

+            $method = "assert".($expected ? "True" : "False");

 

             $this->$method(

                 $this->target->assert(

@@ -138,7 +138,7 @@ 
 

     }

 

-    public function assertParametersWithoutOrganization(){

+    public function assertParametersWithoutOrganization() {

 

         $userId = 1234;

         $user = new User();

@@ -163,13 +163,13 @@ 
 

 

         return [

-            [[new Acl(), null, null, null] , false ],

-            [[new Acl(), null, null, Permissions::PERMISSION_CHANGE] , false ],

-            [[new Acl(), $user, $jobMock, 'edit'] , true ],

+            [[new Acl(), null, null, null], false],

+            [[new Acl(), null, null, Permissions::PERMISSION_CHANGE], false],

+            [[new Acl(), $user, $jobMock, 'edit'], true],

         ];

     }

 

-    public function testAssertUserIsOrganizationAdmin(){

+    public function testAssertUserIsOrganizationAdmin() {

 

         $userId = 1234;

         $user = new User();

@@ -192,13 +192,13 @@ 
         $this->assertTrue(

             $this->target->assert(

                 new Acl(), // acl

-                $user,     // role

-                $jobMock,  // resource

+                $user, // role

+                $jobMock, // resource

                 'edit'     // privilege

             ));

     }

 

-    public function testAssertUserIsOwnerOfTheParentOrganization(){

+    public function testAssertUserIsOwnerOfTheParentOrganization() {

 

         $userId = 1234;

         $user = new User();

@@ -225,13 +225,13 @@ 
         $this->assertTrue(

             $this->target->assert(

                 new Acl(), // acl

-                $user,     // role

-                $jobMock,  // resource

+                $user, // role

+                $jobMock, // resource

                 'edit'     // privilege

             ));

     }

 

-    public function testUserIsEmployeeWithJobsChangePermissions(){

+    public function testUserIsEmployeeWithJobsChangePermissions() {

 

         $userId = 1234;

         $user = new User();

@@ -265,8 +265,8 @@ 
         $this->assertTrue(

             $this->target->assert(

                 new Acl(), // acl

-                $user,     // role

-                $jobMock,  // resource

+                $user, // role

+                $jobMock, // resource

                 'edit'     // privilege

             ));

     }

module/Jobs/test/TestConfig.php

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 $commonModules = include_once __DIR__.'/../../../config/common.modules.php';
 
 return array(
-    'modules' => array_merge($commonModules,array(
+    'modules' => array_merge($commonModules, array(
         'Core',
         'Auth',
         'Jobs',

module/Auth/test/Bootstrap.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@
 <?php
 
 /** @var array $testConfig */
-$testConfig = include __DIR__ . '/TestConfig.php';
+$testConfig = include __DIR__.'/TestConfig.php';
 
-require_once __DIR__ . '/../../../test/Bootstrap.php';
+require_once __DIR__.'/../../../test/Bootstrap.php';

module/Auth/test/TestConfig.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 
 return array(
     // This should be an array of module namespaces used in the application.
-    'modules' => array_merge($commonModules,array(
+    'modules' => array_merge($commonModules, array(
         'Core',
 	    'Auth',
 	    'Jobs',

module/Auth/test/AclTest/Assertion/AbstractEventManagerAwareAssertionTest.php

Spacing +2 added lines, -2 removed lines

@@ -155,9 +155,9 @@
         $self = $this;
 
         $events->expects($this->once())->method('triggerUntil')
-               ->will($this->returnCallback(function ($callback,$eventName,$event) use ($acl, $role, $resource, $privilege, $self) {
+               ->will($this->returnCallback(function($callback, $eventName, $event) use ($acl, $role, $resource, $privilege, $self) {
                	    $self->assertTrue(is_callable($callback));
-	                $self->assertEquals('assert',$eventName);
+	                $self->assertEquals('assert', $eventName);
                     $self->assertSame($acl, $event->getAcl());
                     $self->assertSame($role, $event->getRole());
                     $self->assertSame($resource, $event->getResource());

module/Auth/test/AclTest/Assertion/AssertionManagerFactoryTest.php

Spacing +3 added lines, -3 removed lines

@@ -38,7 +38,7 @@ 
                  ->willReturn(array());
 
         $target = new AssertionManagerFactory();
-        $manager = $target->__invoke($services,AssertionManager::class);
+        $manager = $target->__invoke($services, AssertionManager::class);
 
         $this->assertInstanceOf('\Acl\Assertion\AssertionManager', $manager);
         //$this->assertFalse($manager->shareByDefault(), 'The managers\' shareByDefault value must be set to FALSE by the factory.');
@@ -61,9 +61,9 @@ 
 
         $target = new AssertionManagerFactory();
 
-        $manager = $target->__invoke($services,AssertionManager::class);
+        $manager = $target->__invoke($services, AssertionManager::class);
 
-        $this->assertTrue($testResult === $manager->has($testName), 'Expected managers\' has method to return ' . ($testResult ? 'TRUE' : 'FALSE') . ' on ' .$testName);
+        $this->assertTrue($testResult === $manager->has($testName), 'Expected managers\' has method to return '.($testResult ? 'TRUE' : 'FALSE').' on '.$testName);
     }
 
     public function provideConfigArrays()

module/Auth/test/AuthTest/Factory/Controller/RemoveControllerFactoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -49,6 +49,6 @@
             ]));
         
         $controllerFactory = new RemoveControllerFactory();
-        $this->assertInstanceOf(RemoveController::class, $controllerFactory($serviceLocator,'irrelevant'));
+        $this->assertInstanceOf(RemoveController::class, $controllerFactory($serviceLocator, 'irrelevant'));
     }
 }

module/Auth/test/AuthTest/Factory/Controller/Plugin/UserSwitcherFactoryTest.php

Spacing +3 added lines, -3 removed lines

@@ -39,11 +39,11 @@ 
     private $target = [
         UserSwitcherFactory::class,
         '@testCreateServiceInvokesItself' => [
-            'mock' => [ '__invoke' ],
+            'mock' => ['__invoke'],
         ],
     ];
 
-    private $inheritance = [ FactoryInterface::class ];
+    private $inheritance = [FactoryInterface::class];
 
     public function testCreateServiceInvokesItself()
     {
@@ -63,7 +63,7 @@ 
         $auth = $this->getMockBuilder(AuthenticationService::class)->disableOriginalConstructor()->getMock();
         $acl = $this->getMockBuilder(Acl::class)->disableOriginalConstructor()->getMock();
         $sm = $this->createServiceManagerMock();
-        $controllerPlugins = $this->getPluginManagerMock(['Acl' => ['service' => $acl, 'count_get' => 1]],$sm);
+        $controllerPlugins = $this->getPluginManagerMock(['Acl' => ['service' => $acl, 'count_get' => 1]], $sm);
         $container = $this->getServiceManagerMock([
                 'AuthenticationService' => ['service' => $auth, 'count_get' => 1],
                 'ControllerPluginManager' => ['service' => $controllerPlugins, 'count_get' => 1]

module/Auth/test/AuthTest/Factory/View/Helper/AuthFactoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
         
         $hm = new HelperPluginManager($sm);
         
-        $helper = $f->__invoke($sm,Auth::class);
+        $helper = $f->__invoke($sm, Auth::class);
         
         $this->assertInstanceOf('\Auth\View\Helper\Auth', $helper);
         $this->assertSame($auth, $helper->getService());