cross-solution / YAWIK

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

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

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

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/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

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/AuthTest/Service/RegisterTest.php

Spacing +4 added lines, -4 removed lines

@@ -115,7 +115,7 @@ 
     public function testProceed_WhenUserAlreadyExists()
     {
         $name = uniqid('name');
-        $email = uniqid('email') . '@' . uniqid('host') . '.com.pl';
+        $email = uniqid('email').'@'.uniqid('host').'.com.pl';
         $user = UserEntityProvider::createEntityWithRandomData();
         $role = 'user';
 
@@ -160,8 +160,8 @@ 
      */
     public function testProceed($role)
     {
-        $name = uniqid('name') . ' ' . uniqid('surname');
-        $email = uniqid('email') . '@' . uniqid('host') . '.com.pl';
+        $name = uniqid('name').' '.uniqid('surname');
+        $email = uniqid('email').'@'.uniqid('host').'.com.pl';
         $user = UserEntityProvider::createEntityWithRandomData();
         $confirmationLink = uniqid('confirmationLink');
         $self = $this;
@@ -194,7 +194,7 @@ 
 
         $this->userRepositoryMock->expects($this->once())
             ->method('store')
-            ->with($this->callback(function ($user) {
+            ->with($this->callback(function($user) {
                 return $user instanceof User ? true:false;
             }));
 

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

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