cross-solution / YAWIK

module/Jobs/test/JobsTest/View/Helper/AdminEditLinkTest.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         'args' => 'getConstructorArgs',
     ];
 
-    private $inheritance = [ AbstractHelper::class ];
+    private $inheritance = [AbstractHelper::class];
 
     private function getConstructorArgs()
     {

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

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
 
         $this->paginatorMock = $paginator;
 
-        return [ $paginator ];
+        return [$paginator];
     }
 
     public function testConstruction()

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

Spacing +7 added lines, -7 removed lines

@@ -104,9 +104,9 @@ 
                        ->disableOriginalConstructor()
                        ->getMock();
 
-        $callback1 = [new MailSenderListenerMock(),'listen1'];
-	    $callback2 = [new MailSenderListenerMock(),'listen2'];
-	    $callback3 = [new MailSenderListenerMock(),'listen3'];
+        $callback1 = [new MailSenderListenerMock(), 'listen1'];
+	    $callback2 = [new MailSenderListenerMock(), 'listen2'];
+	    $callback3 = [new MailSenderListenerMock(), 'listen3'];
         $events->expects($this->exactly(3))
                ->method('attach')
                ->withConsecutive(
@@ -114,7 +114,7 @@ 
                     array(JobEvent::EVENT_JOB_ACCEPTED, array($this->target, 'onJobAccepted')),
                     array(JobEvent::EVENT_JOB_REJECTED, array($this->target, 'onJobRejected'))
                )
-               ->will($this->onConsecutiveCalls($callback1,$callback2,$callback3))
+               ->will($this->onConsecutiveCalls($callback1, $callback2, $callback3))
         ;
 
         $events->expects($this->exactly(3))
@@ -196,7 +196,7 @@ 
 
 class MailSenderListenerMock
 {
-	public function listen1(){}
-	public function listen2(){}
-	public function listen3(){}
+	public function listen1() {}
+	public function listen2() {}
+	public function listen3() {}
 }
\ No newline at end of file

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',