cross-solution / YAWIK

module/Organizations/test/OrganizationsTest/Paginator/ListJobPaginatorFactoryTest.php

Spacing +2 added lines, -2 removed lines

@@ -41,8 +41,8 @@
         $container->expects($this->any())
             ->method('get')
             ->willReturnMap([
-                ['FilterManager',$filterManager],
-                ['repositories',$repositories]
+                ['FilterManager', $filterManager],
+                ['repositories', $repositories]
             ])
         ;
 

module/Organizations/test/Bootstrap.php

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@
 <?php
 
-$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/AclTest/Assertion/AssertionManagerFactoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
 
         $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()

Indentation +6 added lines, -6 removed lines

@@ -32,11 +32,11 @@ 
     public function testCreateServiceReturnsAssertionManager()
     {
         $services = $this->getMockBuilder('\Laminas\ServiceManager\ServiceManager')
-                         ->disableOriginalConstructor()
-                         ->getMock();
+                            ->disableOriginalConstructor()
+                            ->getMock();
 
         $services->expects($this->once())->method('get')->with('Config')
-                 ->willReturn(array());
+                    ->willReturn(array());
 
         $target = new AssertionManagerFactory();
         $manager = $target->__invoke($services, AssertionManager::class);
@@ -54,11 +54,11 @@ 
     public function testCorrectConfigIsUsedToConfigureTheManager($config, $testName, $testResult)
     {
         $services = $this->getMockBuilder('\Laminas\ServiceManager\ServiceManager')
-                         ->disableOriginalConstructor()
-                         ->getMock();
+                            ->disableOriginalConstructor()
+                            ->getMock();
 
         $services->expects($this->once())->method('get')->with('Config')
-                 ->willReturn($config);
+                    ->willReturn($config);
 
         $target = new AssertionManagerFactory();
 

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

Spacing +1 added lines, -1 removed lines

@@ -156,7 +156,7 @@
         $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->assertSame($acl, $event->getAcl());

Indentation +22 added lines, -22 removed lines

@@ -86,12 +86,12 @@ 
         $acl    = new Acl();
 
         $events = $this->getMockBuilder('\Laminas\EventManager\EventManager')
-                       ->disableOriginalConstructor()
-                       ->getMock();
+                        ->disableOriginalConstructor()
+                        ->getMock();
 
         $events->expects($this->once())
-               ->method('triggerUntil')
-               ->willReturn(new ResponseCollection());
+                ->method('triggerUntil')
+                ->willReturn(new ResponseCollection());
 
         $target->setEventManager($events);
 
@@ -113,12 +113,12 @@ 
         $responseFalse = $this->createResponseMock(false);
 
         $events = $this->getMockBuilder('\Laminas\EventManager\EventManager')
-                       ->disableOriginalConstructor()
-                       ->getMock();
+                        ->disableOriginalConstructor()
+                        ->getMock();
 
         $events->expects($this->exactly(5))
-               ->method('triggerUntil')
-               ->will($this->onConsecutiveCalls($responseNull, $responseEmpty, $responseZero, $responseTrue, $responseFalse));
+                ->method('triggerUntil')
+                ->will($this->onConsecutiveCalls($responseNull, $responseEmpty, $responseZero, $responseTrue, $responseFalse));
 
         $target->setEventManager($events);
 
@@ -134,8 +134,8 @@ 
     protected function createResponseMock($returnValue)
     {
         $response = $this->getMockBuilder('\Laminas\EventManager\ResponseCollection')
-                             ->disableOriginalConstructor()
-                             ->getMock();
+                                ->disableOriginalConstructor()
+                                ->getMock();
         $response->method('last')->willReturn($returnValue);
 
         return $response;
@@ -146,8 +146,8 @@ 
         $target = new TargetMock();
 
         $events = $this->getMockBuilder('\Laminas\EventManager\EventManager')
-                       ->disableOriginalConstructor()
-                       ->getMock();
+                        ->disableOriginalConstructor()
+                        ->getMock();
 
         $acl = new Acl();
         $role = new GenericRole('testRole');
@@ -156,16 +156,16 @@ 
         $self = $this;
 
         $events->expects($this->once())->method('triggerUntil')
-               ->will($this->returnCallback(function ($callback, $eventName, $event) use ($acl, $role, $resource, $privilege, $self) {
-                   $self->assertTrue(is_callable($callback));
-                   $self->assertEquals('assert', $eventName);
-                   $self->assertSame($acl, $event->getAcl());
-                   $self->assertSame($role, $event->getRole());
-                   $self->assertSame($resource, $event->getResource());
-                   $self->assertSame($privilege, $event->getPrivilege());
-
-                   return new ResponseCollection();
-               }));
+                ->will($this->returnCallback(function ($callback, $eventName, $event) use ($acl, $role, $resource, $privilege, $self) {
+                    $self->assertTrue(is_callable($callback));
+                    $self->assertEquals('assert', $eventName);
+                    $self->assertSame($acl, $event->getAcl());
+                    $self->assertSame($role, $event->getRole());
+                    $self->assertSame($resource, $event->getResource());
+                    $self->assertSame($privilege, $event->getPrivilege());
+
+                    return new ResponseCollection();
+                }));
 
         $target->setEventManager($events);
         $target->assert($acl, $role, $resource, $privilege);

module/Auth/test/AuthTest/Controller/Plugin/UserSwitcherTest.php

Indentation +6 added lines, -6 removed lines

@@ -211,11 +211,11 @@ 
         $this->assertTrue($this->target->switchUser('switchedUser'));
         $this->assertEquals(
             [
-                 'isSwitchedUser' => true,
-                 'originalUser' => 'originalUser',
-                 'params' => [], 'ref' => '/some/ref',
-                 'session' => serialize($oldSession)
-             ],
+                    'isSwitchedUser' => true,
+                    'originalUser' => 'originalUser',
+                    'params' => [], 'ref' => '/some/ref',
+                    'session' => serialize($oldSession)
+                ],
             $_SESSION[UserSwitcher::SESSION_NAMESPACE]->getArrayCopy()
         );
         $_SESSION = [];
@@ -240,7 +240,7 @@ 
         $acl->expects($this->once())->method('setUser')->with($this->isInstanceOf(User::class));
 
         $user = $this->getMockBuilder(User::class)->disableOriginalConstructor()
-                     ->setMethods(['getId'])->getMock();
+                        ->setMethods(['getId'])->getMock();
         $user->expects($this->any())->method('getId')->willReturn('switchedUser');
 
         $this->target->switchUser($user, ['ref' => 'ref']);

Spacing +9 added lines, -9 removed lines

@@ -45,10 +45,10 @@ 
     private $target = [
         UserSwitcher::class,
         'getSimpleAuthMock',
-        '@testInheritance' => [ 'as_reflection' => true ],
+        '@testInheritance' => ['as_reflection' => true],
         '@testInvokationProxiesToCorrectMethods' => [
             'args' => false,
-            'mock' => [ 'clear' => 1, 'switchUser' => ['with' => 'testUserId', 'count' => 1]],
+            'mock' => ['clear' => 1, 'switchUser' => ['with' => 'testUserId', 'count' => 1]],
         ],
         '@testClearRestoresOriginalUserAndClearsSession' => [
             'args' => 'getComplexAuthMock',
@@ -64,14 +64,14 @@ 
         ],
     ];
 
-    private $inheritance = [ AbstractPlugin::class ];
+    private $inheritance = [AbstractPlugin::class];
 
     public function propertiesProvider()
     {
-        $createSession = function () {
+        $createSession = function() {
             $_SESSION[UserSwitcher::SESSION_NAMESPACE]['params'] = ['param' => 'value'];
         };
-        $clearSession = function () {
+        $clearSession = function() {
             $_SESSION = [];
         };
         return [
@@ -113,7 +113,7 @@ 
             ->disableOriginalConstructor()
             ->getMock();
 
-        return [ $auth ];
+        return [$auth];
     }
 
     private function getComplexAuthMock()
@@ -138,7 +138,7 @@ 
 
         $auth->expects($this->any())->method('getUser')->willReturn(new User());
 
-        return [ $auth ];
+        return [$auth];
     }
 
     public function testInvokationProxiesToCorrectMethods()
@@ -155,7 +155,7 @@ 
     public function returnReferenceProvider()
     {
         return [
-            [ null ], [ 'some/ref/uri' ],
+            [null], ['some/ref/uri'],
         ];
     }
 
@@ -169,7 +169,7 @@ 
         $session = new Container(UserSwitcher::SESSION_NAMESPACE);
         $session->isSwitchedUser = true;
         $session->originalUser = 'switchedUser';
-        $oldSession      = [
+        $oldSession = [
             'oldSession' => true,
             'must'       => 'be same'
         ];

module/Auth/test/AuthTest/Controller/ManageControllerTest.php

Indentation -1 removed lines

@@ -19,7 +19,6 @@
 
 /**
  * Class ManageControllerTest
-
  * @package AuthTest\Controller
  */
 class ManageControllerTest extends AbstractFunctionalControllerTestCase

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

Spacing +2 added lines, -2 removed lines

@@ -41,11 +41,11 @@
     private $target = [
         UserSwitcherFactory::class,
         '@testCreateServiceInvokesItself' => [
-            'mock' => [ '__invoke' ],
+            'mock' => ['__invoke'],
         ],
     ];
 
-    private $inheritance = [ FactoryInterface::class ];
+    private $inheritance = [FactoryInterface::class];
 
     public function testCreateServiceInvokesItself()
     {

module/Auth/test/AuthTest/Listener/DeactivatedUserListenerTest.php

Indentation +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
                 [$this->equalTo(MvcEvent::EVENT_DISPATCH_ERROR), $this->identicalTo([$this->listener, 'prepareExceptionViewModel'], $this->identicalTo(null))],
                 [$this->equalTo(MvcEvent::EVENT_RENDER_ERROR), $this->identicalTo([$this->listener, 'prepareExceptionViewModel'], $this->identicalTo(null))],
                 [$this->equalTo(MvcEvent::EVENT_ROUTE), $this->identicalTo([$this->listener, 'checkDeactivatedUser'], $this->identicalTo(null))]
-             );
+                );
         
         $this->listener->attach($eventManager);
     }

Spacing +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
             ->getMock();
         $this->serviceManager->expects($this->any())
             ->method('get')
-            ->will($this->returnCallback(function ($name) {
+            ->will($this->returnCallback(function($name) {
                 switch ($name) {
                     case 'AuthenticationService':
                         return $this->auth;

module/Auth/test/TestConfig.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
     // This should be an array of module namespaces used in the application.
 
 use PHPUnit\Framework\TestCase;
-    'modules' => array_merge($commonModules,array(
+    'modules' => array_merge($commonModules, array(
         'Core',
 	    'Auth',
 	    'Jobs',

Indentation +6 added lines, -6 removed lines

@@ -8,9 +8,9 @@ 
 use PHPUnit\Framework\TestCase;
     'modules' => array_merge($commonModules,array(
         'Core',
-	    'Auth',
-	    'Jobs',
-	    'Organizations',
+        'Auth',
+        'Jobs',
+        'Organizations',
     )),
 
     // These are various options for the listeners attached to the ModuleManager
@@ -68,7 +68,7 @@ 
     //     ),
     // )
 
-   // Initial configuration with which to seed the ServiceManager.
-   // Should be compatible with Laminas\ServiceManager\Config.
-   // 'service_manager' => array(),
+    // Initial configuration with which to seed the ServiceManager.
+    // Should be compatible with Laminas\ServiceManager\Config.
+    // 'service_manager' => array(),
 );