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

Indentation +18 added lines, -18 removed lines

@@ -58,11 +58,11 @@ 
             default:
                 $this->mailServiceMock = $this->getMockBuilder('\Core\Mail\MailService')->disableOriginalConstructor()->getMock();
                 $this->mailServiceMock->expects($this->atLeastOnce())
-                                      ->method('send')
-                                      ->with($this->callback(array($this, 'popMailMock')));
+                                        ->method('send')
+                                        ->with($this->callback(array($this, 'popMailMock')));
                 $this->mailServiceMock->expects($this->any())
-                                      ->method('get')->with('htmltemplate')
-                                      ->will($this->returnCallback(array($this, 'pushMailMock')));
+                                        ->method('get')->with('htmltemplate')
+                                        ->will($this->returnCallback(array($this, 'pushMailMock')));
 
 
                 $this->targetOptions = array('siteName' => 'Test', 'adminEmail' => 'test@admin');
@@ -101,27 +101,27 @@ 
     public function testAttachsToAndDetachsFromJobEvents()
     {
         $events = $this->getMockBuilder('\Zend\EventManager\EventManager')
-                       ->disableOriginalConstructor()
-                       ->getMock();
+                        ->disableOriginalConstructor()
+                        ->getMock();
 
         $callback1 = [new MailSenderListenerMock(),'listen1'];
-	    $callback2 = [new MailSenderListenerMock(),'listen2'];
-	    $callback3 = [new MailSenderListenerMock(),'listen3'];
+        $callback2 = [new MailSenderListenerMock(),'listen2'];
+        $callback3 = [new MailSenderListenerMock(),'listen3'];
         $events->expects($this->exactly(3))
-               ->method('attach')
-               ->withConsecutive(
+                ->method('attach')
+                ->withConsecutive(
                     array(JobEvent::EVENT_JOB_CREATED, array($this->target, 'onJobCreated')),
                     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))
-               ->method('detach')
-               ->withConsecutive(
+                ->method('detach')
+                ->withConsecutive(
                     array($callback1), array($callback2), array($callback3)
-               )->willReturn(true);
+                )->willReturn(true);
 
         $this->target->attach($events);
         $this->assertAttributeEquals(array($callback1, $callback2, $callback3), 'listeners', $this->target);
@@ -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

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

Indentation +43 added lines, -43 removed lines

@@ -129,62 +129,62 @@ 
                 ->will($this->returnValue('<html />'));
 
         $this->viewPhpRendererStrategy = $this->getMockBuilder('\Zend\View\Strategy\PhpRendererStrategy')
-                          ->disableOriginalConstructor()
-                          ->getMock();
+                            ->disableOriginalConstructor()
+                            ->getMock();
 
         $this->viewPhpRendererStrategy->expects($this->once())
-                          ->method('getRenderer')
-                          ->will($this->returnValue($this->renderer));
+                            ->method('getRenderer')
+                            ->will($this->returnValue($this->renderer));
 
         $this->response = $this->getMockBuilder('\Zend\Http\Response')
-                                              ->disableOriginalConstructor()
-                                              ->getMock();
+                                                ->disableOriginalConstructor()
+                                                ->getMock();
 
         $this->jobEvent = $this->getMockBuilder('\Jobs\Listener\Events\JobEvent')
-                               ->disableOriginalConstructor()
-                               ->getMock();
+                                ->disableOriginalConstructor()
+                                ->getMock();
 
         $this->job = $this->getMockBuilder('\Jobs\Entity\Job')
-                          ->disableOriginalConstructor()
-                          ->getMock();
+                            ->disableOriginalConstructor()
+                            ->getMock();
 
         $this->publisher = $this->getMockBuilder('\Jobs\Entity\Publisher')
-                          ->disableOriginalConstructor()
-                          ->getMock();
+                            ->disableOriginalConstructor()
+                            ->getMock();
 
         $this->publisher->expects($this->at(0))
-                           ->method('__get')
-                           ->with('externalId')
-                           ->will($this->returnValue('externalId32'));
+                            ->method('__get')
+                            ->with('externalId')
+                            ->will($this->returnValue('externalId32'));
 
         $this->publisher->expects($this->at(1))
-                           ->method('__get')
-                           ->with('reference')
-                           ->will($this->returnValue('reference32'));
+                            ->method('__get')
+                            ->with('reference')
+                            ->will($this->returnValue('reference32'));
 
         $this->publisher->expects($this->any())
-                              ->method('__set')
-                              ->will($this->returnCallback($staticClassPrefix . 'publisherSetter'));
+                                ->method('__set')
+                                ->will($this->returnCallback($staticClassPrefix . 'publisherSetter'));
 
         $this->htmlAbsPathFilter = $this->getMockBuilder('\Core\Filter\HtmlAbsPathFilter')
                                     ->disableOriginalConstructor()
                                     ->getMock();
 
         $this->htmlAbsPathFilter->expects($this->any())
-                              ->method('filter')
-                              ->will($this->returnCallback($staticClassPrefix . 'absPathFilter'));
+                                ->method('filter')
+                                ->will($this->returnCallback($staticClassPrefix . 'absPathFilter'));
 
         $this->filterManager = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager')
-                                     ->disableOriginalConstructor()
-                                     ->getMock();
+                                        ->disableOriginalConstructor()
+                                        ->getMock();
 
         $this->filterManager->expects($this->at(0))
                         ->method('get')
                         ->willReturn($this->htmlAbsPathFilter);
 
         $this->restClient = $this->getMockBuilder('\Core\Service\RestClient')
-                                 ->disableOriginalConstructor()
-                                 ->getMock();
+                                    ->disableOriginalConstructor()
+                                    ->getMock();
 
         //$this->restClient->expects($this->at(0))
         //                   ->method('getHost')
@@ -200,8 +200,8 @@ 
             ->will($this->returnValue($this->response));
 
         $this->organization = $this->getMockBuilder('\Organizations\Entity\Organization')
-                                 ->disableOriginalConstructor()
-                                 ->getMock();
+                                    ->disableOriginalConstructor()
+                                    ->getMock();
 
         $this->organization->expects($this->at(0))
                         ->method('__get')
@@ -209,21 +209,21 @@ 
                         ->will($this->returnValue('name32'));
 
         $this->provider = $this->getMockBuilder('\Jobs\Options\ProviderOptions')
-                                   ->disableOriginalConstructor()
-                                   ->getMock();
+                                    ->disableOriginalConstructor()
+                                    ->getMock();
 
         $this->providerChannel = $this->getMockBuilder('\Jobs\Options\ChannelOptions')
-                                      ->disableOriginalConstructor()
-                                      ->getMock();
+                                        ->disableOriginalConstructor()
+                                        ->getMock();
 
         $this->providerChannel->expects($this->any())
-                              ->method('__get')
-                              ->will($this->returnCallback($staticClassPrefix . 'providerChannelGetter'));
+                                ->method('__get')
+                                ->will($this->returnCallback($staticClassPrefix . 'providerChannelGetter'));
 
         $this->provider->expects($this->any(0))
-                       ->method('__get')
-                       ->with('channels')
-                       ->willReturn(array('bbb value' => $this->providerChannel));
+                        ->method('__get')
+                        ->with('channels')
+                        ->willReturn(array('bbb value' => $this->providerChannel));
 
         $this->viewModel = $this->getMockBuilder('\Zend\View\Model\ViewModel')
                                 ->disableOriginalConstructor()
@@ -304,12 +304,12 @@ 
     protected function setRestReturn($referenceUpdate, $applyIdUpdate)
     {
         $this->response->expects($this->any())
-                       ->method('getBody')
-                       ->will($this->returnValue(json_encode(
-                           array(
-                               'referenceUpdate' => $referenceUpdate,
-                               'applyIdUpdate' => $applyIdUpdate
-                           ))));
+                        ->method('getBody')
+                        ->will($this->returnValue(json_encode(
+                            array(
+                                'referenceUpdate' => $referenceUpdate,
+                                'applyIdUpdate' => $applyIdUpdate
+                            ))));
     }
 
 

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

Indentation +1 added lines, -1 removed lines

@@ -107,7 +107,7 @@
 
         $wf = new WorkflowSettings();
         $wf->setAcceptApplicationByDepartmentManager($flag('acceptApplication'))
-           ->setAssignDepartmentManagersToJobs($flag('assignManagers'));
+            ->setAssignDepartmentManagersToJobs($flag('assignManagers'));
 
         $org->expects($this->once())->method('getWorkflowSettings')->willReturn($wf);
 

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

Indentation +6 added lines, -6 removed lines

@@ -6,9 +6,9 @@ 
     // This should be an array of module namespaces used in the application.
     'modules' => array_merge($commonModules,array(
         'Core',
-	    'Auth',
-	    'Jobs',
-	    'Organizations',
+        'Auth',
+        'Jobs',
+        'Organizations',
     )),
 
     // These are various options for the listeners attached to the ModuleManager
@@ -64,7 +64,7 @@ 
     //     ),
     // )
 
-   // Initial configuration with which to seed the ServiceManager.
-   // Should be compatible with Zend\ServiceManager\Config.
-   // 'service_manager' => array(),
+    // Initial configuration with which to seed the ServiceManager.
+    // Should be compatible with Zend\ServiceManager\Config.
+    // 'service_manager' => array(),
 );

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