cross-solution / YAWIK

module/Jobs/test/JobsTest/Listener/PublisherTest.php

Spacing +4 added lines, -4 removed lines

@@ -110,7 +110,7 @@ 
         
         $this->target = new Publisher($this->serviceManager);
 
-        $staticClassPrefix = '\\' . __CLASS__ . '::';
+        $staticClassPrefix = '\\'.__CLASS__.'::';
 
         $this->log = $this->getMockBuilder('\Zend\Log\Logger')
             ->disableOriginalConstructor()
@@ -164,7 +164,7 @@ 
 
         $this->publisher->expects($this->any())
                               ->method('__set')
-                              ->will($this->returnCallback($staticClassPrefix . 'publisherSetter'));
+                              ->will($this->returnCallback($staticClassPrefix.'publisherSetter'));
 
         $this->htmlAbsPathFilter = $this->getMockBuilder('\Core\Filter\HtmlAbsPathFilter')
                                     ->disableOriginalConstructor()
@@ -172,7 +172,7 @@ 
 
         $this->htmlAbsPathFilter->expects($this->any())
                               ->method('filter')
-                              ->will($this->returnCallback($staticClassPrefix . 'absPathFilter'));
+                              ->will($this->returnCallback($staticClassPrefix.'absPathFilter'));
 
         $this->filterManager = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager')
                                      ->disableOriginalConstructor()
@@ -218,7 +218,7 @@ 
 
         $this->providerChannel->expects($this->any())
                               ->method('__get')
-                              ->will($this->returnCallback($staticClassPrefix . 'providerChannelGetter'));
+                              ->will($this->returnCallback($staticClassPrefix.'providerChannelGetter'));
 
         $this->provider->expects($this->any(0))
                        ->method('__get')

module/Jobs/test/JobsTest/Listener/GetOrganizationManagersTest.php

Spacing +1 added lines, -1 removed lines

@@ -199,7 +199,7 @@
 
         $actual = $this->target->__invoke($event);
 
-        $this->assertTrue(is_array($actual),'Listener did not return an array!');
+        $this->assertTrue(is_array($actual), 'Listener did not return an array!');
         $this->assertArrayHasKey('status', $actual);
         $this->assertEquals('disabled', $actual['status']);
     }

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'));
     }
 }