cross-solution / YAWIK

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/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')
         );
     }

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

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
             ->method('trigger')
             ->with($event, 'some_target')
             ->willReturn([
-                'test_template',$viewModel
+                'test_template', $viewModel
             ])
         ;
 

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

Spacing +13 added lines, -13 removed lines

@@ -91,20 +91,20 @@ 
         $this->viewResolver->expects($this->any())
             ->method('resolve')
             ->willReturnMap([
-                ['mail/null-template',null,null],
-                ['coretest/mail/null-template',null,__DIR__.'/fixtures/mail-template.phtml'],
-                ['mail/template',null,__DIR__.'/fixtures/mail-template.phtml'],
-                ['mail/exception',null,__DIR__.'/fixtures/error.phtml']
+                ['mail/null-template', null, null],
+                ['coretest/mail/null-template', null, __DIR__.'/fixtures/mail-template.phtml'],
+                ['mail/template', null, __DIR__.'/fixtures/mail-template.phtml'],
+                ['mail/exception', null, __DIR__.'/fixtures/error.phtml']
             ])
         ;
 
         $container->expects($this->any())
             ->method('get')
             ->willReturnMap([
-                ['Log/Core/Mail',$this->mailLog],
-                ['ViewResolver',$this->viewResolver],
-                ['EventManager',$this->events],
-                ['ModuleManager',$this->moduleManager]
+                ['Log/Core/Mail', $this->mailLog],
+                ['ViewResolver', $this->viewResolver],
+                ['EventManager', $this->events],
+                ['ModuleManager', $this->moduleManager]
             ])
         ;
 
@@ -132,7 +132,7 @@ 
     public function testTemplate()
     {
         $target = $this->target;
-        $params = ['Hello' => 'World','this'=>'this should not rendered'];
+        $params = ['Hello' => 'World', 'this'=>'this should not rendered'];
 
         $target($params);
         $target->template('null-template');
@@ -193,7 +193,7 @@ 
         $resolver->expects($this->any())
             ->method('resolve')
             ->willReturnMap([
-                ['mail/exception',null,$tplFile]
+                ['mail/exception', null, $tplFile]
             ])
         ;
 
@@ -221,9 +221,9 @@ 
     public function getTestInformationCompleteException()
     {
         return [
-            ['$from','A from email address'],
-            ['$fromName','A from name'],
-            ['$subject','A subject must be'],
+            ['$from', 'A from email address'],
+            ['$fromName', 'A from name'],
+            ['$subject', 'A subject must be'],
         ];
     }
 

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

Indentation +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
             ->getMock()
         ;
         $container = $this->getMockBuilder(ContainerInterface::class)
-                          ->getMock();
+                            ->getMock();
         $container->expects($this->once())
             ->method('get')
             ->with('ServiceManager')

Spacing +5 added lines, -5 removed lines

@@ -51,10 +51,10 @@ 
     public function providePaginatorCreationData()
     {
         return [
-            [ 'Test/Paginator', ['test' => 'value'], ['merged' => 'yes'], false, ['page' => 1, 'count' => 10, 'range' => 5] ],
-            [ 'Test2/YetAnotherPager', ['page' => 2], [], true, ['page' => 2, 'count' => 10, 'range' => 5] ],
-            [ 'Yet/Another', ['page' => 3, 'count' => 90, 'range' => 2], false, false, ['page' => 3, 'count' => 90, 'range' => 2] ],
-            [ 'Even/Another', ['test' => 'value'], [], new \ArrayObject(['test' => 'value']), ['page' => 1, 'count' => 10, 'range' => 5]],
+            ['Test/Paginator', ['test' => 'value'], ['merged' => 'yes'], false, ['page' => 1, 'count' => 10, 'range' => 5]],
+            ['Test2/YetAnotherPager', ['page' => 2], [], true, ['page' => 2, 'count' => 10, 'range' => 5]],
+            ['Yet/Another', ['page' => 3, 'count' => 90, 'range' => 2], false, false, ['page' => 3, 'count' => 90, 'range' => 2]],
+            ['Even/Another', ['test' => 'value'], [], new \ArrayObject(['test' => 'value']), ['page' => 1, 'count' => 10, 'range' => 5]],
         ];
     }
 
@@ -107,7 +107,7 @@ 
             
         $em = $this->getMockBuilder(EventManager::class)
             ->disableOriginalConstructor()
-            ->setMethods(['getEvent','triggerEvent'])
+            ->setMethods(['getEvent', 'triggerEvent'])
             ->getMock()
         ;