cross-solution / YAWIK

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(),
 );

module/Core/test/CoreTest/Repository/DoctrineMongoODM/Event/AbstractUpdateFilesPermissionsSubscriberTest.php

Spacing +5 added lines, -5 removed lines

@@ -54,7 +54,7 @@ 
     ];
 
     /** @noinspection PhpUnusedPrivateFieldInspection */
-    private $inheritance = [ EventSubscriber::class ];
+    private $inheritance = [EventSubscriber::class];
 
 
     private function getTarget()
@@ -66,7 +66,7 @@ 
 
     public function testSubscribesToOnFlushEvent()
     {
-        $this->assertEquals([ Events::onFlush ], $this->target->getSubscribedEvents());
+        $this->assertEquals([Events::onFlush], $this->target->getSubscribedEvents());
     }
 
     public function testUpdatesFilesPermissionsOnFlush()
@@ -99,8 +99,8 @@ 
         $document->singleFile = $file;
         $document->fileCollection = $collection;
 
-        $inserts = [ $document ];
-        $updates = [ $document ];
+        $inserts = [$document];
+        $updates = [$document];
 
         $filePermissions->expects($this->exactly(4))->method('clear')->will($this->returnSelf());
         $filePermissions->expects($this->exactly(4))->method('inherit')->with($permissions)->will($this->returnSelf());
@@ -157,7 +157,7 @@ 
 
 class ConcreteUpdateFilesPermissionsSubScriber extends AbstractUpdateFilesPermissionsSubscriber
 {
-    protected $filesProperties = [ 'singleFile', 'fileCollection' ];
+    protected $filesProperties = ['singleFile', 'fileCollection'];
     protected $targetDocument  = Ufps_TargetDocument::class;
 }
 

module/Core/test/CoreTest/Controller/Plugin/NotificationTest.php

Spacing +11 added lines, -11 removed lines

@@ -101,7 +101,7 @@ 
         $translator->expects($this->once())
             ->method('translate')
             ->willReturnMap([
-                ['some message','default','translated some message'],
+                ['some message', 'default', 'translated some message'],
             ])
         ;
         $listener->expects($this->exactly(2))
@@ -137,7 +137,7 @@ 
             ->willReturn($mock)
         ;
 
-        $callback = array($mock,$method);
+        $callback = array($mock, $method);
         $this->assertSame(
             $mock,
             call_user_func($callback, $message)
@@ -147,11 +147,11 @@ 
     public function getTestCreateNamespacedMessages()
     {
         return [
-            ['info','info message'],
-            ['warning','warning message'],
-            ['success','success message'],
-            ['danger','danger message'],
-            ['error','error message']
+            ['info', 'info message'],
+            ['warning', 'warning message'],
+            ['success', 'success message'],
+            ['danger', 'danger message'],
+            ['error', 'error message']
         ];
     }
 
@@ -174,8 +174,8 @@ 
         $events->expects($this->exactly(2))
             ->method('getNotifications')
             ->willReturnOnConsecutiveCalls(
-                [],// setup for empty array test
-                [$entity1,$entity2] // setup for non empty array tests
+                [], // setup for empty array test
+                [$entity1, $entity2] // setup for non empty array tests
             )
         ;
 
@@ -192,8 +192,8 @@ 
         $mock->expects($this->exactly(2))
             ->method('renderMessage')
             ->withConsecutive(
-                ['some notification',Notification::NAMESPACE_INFO],
-                ['some notification',Notification::NAMESPACE_DANGER]
+                ['some notification', Notification::NAMESPACE_INFO],
+                ['some notification', Notification::NAMESPACE_DANGER]
             )
         ;
         $mock->createOutput($events);

module/Core/test/CoreTest/Controller/Plugin/PaginationParamsTest.php

Spacing +6 added lines, -6 removed lines

@@ -62,7 +62,7 @@ 
     {
         $mock = $this->getMockBuilder(PaginationParams::class)
             ->disableOriginalConstructor()
-            ->setMethods(['getParams','getList'])
+            ->setMethods(['getParams', 'getList'])
             ->getMock()
         ;
         $repository = $this->createMock(RepositoryInterface::class);
@@ -78,8 +78,8 @@ 
         $mock->expects($this->exactly(2))
             ->method('getList')
             ->withConsecutive(
-                ['some/namespace',[$this,'methodToTest']],
-                ['some/namespace',$repository]
+                ['some/namespace', [$this, 'methodToTest']],
+                ['some/namespace', $repository]
             )
             ->willReturn('getList called')
         ;
@@ -97,7 +97,7 @@ 
         );
         $this->assertEquals(
             'getList called',
-            $mock('some/namespace', [$this,'methodToTest']),
+            $mock('some/namespace', [$this, 'methodToTest']),
             '__invoke() should call ::getList when defaults is callable'
         );
         $this->assertEquals(
@@ -179,7 +179,7 @@ 
         $session->list = null;
         $mock = $this->target;
 
-        $callback = [$this,'methodToTest'];
+        $callback = [$this, 'methodToTest'];
         $output = $mock->getList('namespace', $callback);
         $this->assertEquals(
             'methodToTest called',
@@ -222,7 +222,7 @@ 
             ->willReturn('next')
         ;
         $this->assertEquals(
-            ['previous','next'],
+            ['previous', 'next'],
             $mock->getNeighbours('namespace', 'callback', 'id')
         );
     }