cross-solution / YAWIK

module/Core/test/CoreTest/Controller/AdminControllerEventTest.php

Spacing +11 added lines, -11 removed lines

@@ -30,7 +30,7 @@ 
 
     protected $target = '\Core\Controller\AdminControllerEvent';
 
-    protected $inheritance = [ '\Laminas\EventManager\Event' ];
+    protected $inheritance = ['\Laminas\EventManager\Event'];
 
     public function testCreatesModelPriorityListUponCreation()
     {
@@ -64,10 +64,10 @@ 
     public function provideAddViewTemplateTestData()
     {
         return [
-            [ [ 'test', 'template' ], ['test', 'template', [], 0] ],
-            [ [ 'test', 'template', [ 'var' => 'iable']], [ 'test', 'template', [ 'var' => 'iable'], 0] ],
-            [ [ 'test', 'template', 10] , ['test', 'template', [], 10] ],
-            [ [ 'test', 'template', [ 'var' => 'iable'], 10 ], [ 'test', 'template', [ 'var' => 'iable'], 10 ]],
+            [['test', 'template'], ['test', 'template', [], 0]],
+            [['test', 'template', ['var' => 'iable']], ['test', 'template', ['var' => 'iable'], 0]],
+            [['test', 'template', 10], ['test', 'template', [], 10]],
+            [['test', 'template', ['var' => 'iable'], 10], ['test', 'template', ['var' => 'iable'], 10]],
         ];
     }
 
@@ -94,12 +94,12 @@ 
     public function provideAddViewVariablesTestData()
     {
         return [
-            [ [ 'test' ] , ['test', ['name' => 'test'], 0] ],
-            [ [ 'test', [ 'var' => 'iable'] ], [ 'test', [ 'name' => 'test', 'var' => 'iable'], 0 ] ],
-            [ [ 'test', [ 'var' => 'iable'], 10], [ 'test', [ 'name' => 'test', 'var' => 'iable'], 10 ]],
-            [ [ [ 'name' => 'other', 'var' => 'test' ], 10], ['other', ['name' => 'other', 'var' => 'test' ], 10] ],
-            [ [ 'test', ['name' => 'other'] ], [ 'test', [ 'name' => 'other' ], 0] ],
-            [ [ 'test', 10 ], ['test', ['name' => 'test'], 10]],
+            [['test'], ['test', ['name' => 'test'], 0]],
+            [['test', ['var' => 'iable']], ['test', ['name' => 'test', 'var' => 'iable'], 0]],
+            [['test', ['var' => 'iable'], 10], ['test', ['name' => 'test', 'var' => 'iable'], 10]],
+            [[['name' => 'other', 'var' => 'test'], 10], ['other', ['name' => 'other', 'var' => 'test'], 10]],
+            [['test', ['name' => 'other']], ['test', ['name' => 'other'], 0]],
+            [['test', 10], ['test', ['name' => 'test'], 10]],
         ];
     }
 

module/Core/test/CoreTest/Controller/AdminControllerTest.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
      */
     protected $target = '\Core\Controller\AdminController';
 
-    protected $inheritance = [ 'Laminas\Mvc\Controller\AbstractActionController' ];
+    protected $inheritance = ['Laminas\Mvc\Controller\AbstractActionController'];
 
     protected function setUp(): void
     {

module/Core/test/CoreTest/Mail/MailService/SendMailTest.php

Spacing +6 added lines, -6 removed lines

@@ -40,7 +40,7 @@ 
     protected function setUp(): void
     {
         $test = $this;
-        $sendCallback = function ($value) use ($test) {
+        $sendCallback = function($value) use ($test) {
             return $value === $test->expectedMail;
         };
         $transport = $this->getMockForAbstractClass('\Laminas\Mail\Transport\TransportInterface');
@@ -109,9 +109,9 @@ 
     public function testOverrideRecipient()
     {
         $overrideEmail = 'overidden@email';
-        $ccEmail="cc@email";
-        $bccEmail="bcc@email";
-        $toEmail="to@email";
+        $ccEmail = "cc@email";
+        $bccEmail = "bcc@email";
+        $toEmail = "to@email";
         
         $recipients = new AddressList();
         $recipients->add($overrideEmail);
@@ -128,13 +128,13 @@ 
         $this->target->send($mail);
 
         $headers = $mail->getHeaders();
-        $expectedTo = 'To: ' . $overrideEmail;
+        $expectedTo = 'To: '.$overrideEmail;
         $this->assertFalse($headers->has('cc'));
         $this->assertFalse($headers->has('bcc'));
         $this->assertTrue($headers->has('X-Original-Recipients'));
 
         $this->assertEquals($expectedTo, $headers->get('to')->toString());
-        $this->assertEquals('X-Original-Recipients: To: ' . $toEmail . '; Cc: ' . $ccEmail . '; Bcc: ' . $bccEmail, $headers->get('X-Original-Recipients')->toString());
+        $this->assertEquals('X-Original-Recipients: To: '.$toEmail.'; Cc: '.$ccEmail.'; Bcc: '.$bccEmail, $headers->get('X-Original-Recipients')->toString());
     }
 
     public function testSetsXMailerHeader()

module/Core/test/CoreTest/Factory/OptionsAbstractFactoryTest.php

Spacing +20 added lines, -20 removed lines

@@ -56,10 +56,10 @@ 
         ];
 
         return [
-            [ [], 'testnameone', 'Test/Name.One', false ],
-            [ $cfg1, 'testnametwo', 'Test.Name/Two', true ],
-            [ $cfg1, 'othername', '', false ],
-            [ $cfg1, 'nonexistant', 'Non.Existant', false ],
+            [[], 'testnameone', 'Test/Name.One', false],
+            [$cfg1, 'testnametwo', 'Test.Name/Two', true],
+            [$cfg1, 'othername', '', false],
+            [$cfg1, 'nonexistant', 'Non.Existant', false],
         ];
     }
 
@@ -79,7 +79,7 @@ 
 
         $services = $this->getServiceLocatorMock($optionsConfig);
 
-        $method = "assert" . ($expected ? 'True' : 'False');
+        $method = "assert".($expected ? 'True' : 'False');
         $this->$method($target->canCreateServiceWithName($services, $name, $requestedName));
     }
 
@@ -91,10 +91,10 @@ 
         ];
 
         return [
-            [ [], 'Test/Name.One', false ],
-            [ $cfg1, 'Test.Name/Two', true ],
-            [ $cfg1, 'othername', true ],
-            [ $cfg1, 'Non.Existant', false ],
+            [[], 'Test/Name.One', false],
+            [$cfg1, 'Test.Name/Two', true],
+            [$cfg1, 'othername', true],
+            [$cfg1, 'Non.Existant', false],
         ];
     }
 
@@ -143,7 +143,7 @@ 
 
         $services = $this->getServiceLocatorMock([
             'TestOption' => [
-                'class' => __NAMESPACE__ . '\SimpleOptionsMock',
+                'class' => __NAMESPACE__.'\SimpleOptionsMock',
                 'mode' => 'invalid',
             ]
         ]);
@@ -158,11 +158,11 @@ 
      */
     public function testCreatesSimpleOptionsInstance()
     {
-        $optionsMockClass = __NAMESPACE__ . '\SimpleOptionsMock';
+        $optionsMockClass = __NAMESPACE__.'\SimpleOptionsMock';
         $cfg = [
             'SimpleOptions' => [
                 'class' => $optionsMockClass,
-                'options' => [ 'one' => 'three' ]
+                'options' => ['one' => 'three']
             ]
         ];
 
@@ -203,8 +203,8 @@ 
      */
     public function testCreateNestedOptionsInstance()
     {
-        $nestedOptionsMockClass = __NAMESPACE__ . '\NestedOptionsMock';
-        $simpleOptionsMockClass = __NAMESPACE__ . '\SimpleOptionsMock';
+        $nestedOptionsMockClass = __NAMESPACE__.'\NestedOptionsMock';
+        $simpleOptionsMockClass = __NAMESPACE__.'\SimpleOptionsMock';
 
         $cfg = [
             'NestedOptions' => [
@@ -212,9 +212,9 @@ 
                 'mode' => OptionsAbstractFactory::MODE_NESTED,
                 'options' => [
                     'skalar' => 'itsworking',
-                    'array' => [ 'its' => 'working' ],
-                    'opt1' => [ '__class__' => $simpleOptionsMockClass ],
-                    'opt2' => [ '__class__' => $simpleOptionsMockClass, 'two' => 'four' ],
+                    'array' => ['its' => 'working'],
+                    'opt1' => ['__class__' => $simpleOptionsMockClass],
+                    'opt2' => ['__class__' => $simpleOptionsMockClass, 'two' => 'four'],
                 ],
             ],
         ];
@@ -228,14 +228,14 @@ 
 
         $this->assertInstanceOf($nestedOptionsMockClass, $options);
         $this->assertEquals('itsworking', $options->getSkalar());
-        $this->assertEquals([ 'its' => 'working' ], $options->getArray());
+        $this->assertEquals(['its' => 'working'], $options->getArray());
         $this->assertEquals('One', $options->getOpt1()->getOne());
         $this->assertEquals('four', $options->getOpt2()->getTwo());
     }
 
     protected function getServiceLocatorMock($optionsConfig = [])
     {
-        $optionsConfig = [ 'options' => $optionsConfig ];
+        $optionsConfig = ['options' => $optionsConfig];
         $services = $this->getMockBuilder('Laminas\ServiceManager\ServiceManager')->disableOriginalConstructor()->getMock();
 
         $services->expects($this->once())->method('get')->with('config')->willReturn($optionsConfig);
@@ -247,7 +247,7 @@ 
 class SimpleOptionsMock extends AbstractOptions
 {
     protected $one = "One";
-    protected $two= "Two";
+    protected $two = "Two";
 
     /**
      * @param string $one

module/Core/test/CoreTest/Factory/Navigation/DefaultNavigationFactoryTest.php

Spacing +3 added lines, -3 removed lines

@@ -35,7 +35,7 @@ 
      */
     private $target = DefaultNavigationFactory::class;
 
-    private $inheritance = [ '\Laminas\Navigation\Service\DefaultNavigationFactory' ];
+    private $inheritance = ['\Laminas\Navigation\Service\DefaultNavigationFactory'];
 
     public function testSetsActiveFlagOnPagesProvidingTheActiveOnOption()
     {
@@ -47,7 +47,7 @@ 
                 'active_on' => 'notMatchedRoute',
             ],
             'page3' => [
-                'active_on' => [ 'matchedRouteName', 'anotherRoute' ],
+                'active_on' => ['matchedRouteName', 'anotherRoute'],
             ],
             'page4' => []
         ];
@@ -77,7 +77,7 @@ 
                 'active_on' => 'notMatchedRoute',
             ],
             'page3' => [
-                'active_on' => [ 'matchedRouteName', 'anotherRoute' ],
+                'active_on' => ['matchedRouteName', 'anotherRoute'],
             ],
             'page4' => []
         ];

module/Core/test/CoreTest/Factory/View/Helper/SocialButtonsFactoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
         $options = new ModuleOptions();
         $config = ['testwert'];
 
-        $HauptServiceLocator =  $this->getMockBuilder('Laminas\ServiceManager\ServiceManager')->disableOriginalConstructor()->getMock();
+        $HauptServiceLocator = $this->getMockBuilder('Laminas\ServiceManager\ServiceManager')->disableOriginalConstructor()->getMock();
         $HauptServiceLocator->expects($this->exactly(2))->method('get')->withConsecutive(['Auth/Options'], ['Config'])->will($this->onConsecutiveCalls($options, $config));
 
         $target = new SocialButtonsFactory();

module/Core/test/CoreTest/Factory/EventManager/EventManagerAbstractFactory/AttachListenersTest.php

Spacing +101 added lines, -101 removed lines

@@ -46,7 +46,7 @@ 
     protected function setUp(): void
     {
         $this->target = $this->getMockBuilder('\Core\Factory\EventManager\EventManagerAbstractFactory')
-                             ->setMethods([ 'createEventManager', 'getConfig' ])
+                             ->setMethods(['createEventManager', 'getConfig'])
                              ->getMock();
 
         $events = $this->getMockBuilder('\Laminas\EventManager\EventManager')
@@ -60,7 +60,7 @@ 
 
     protected function setTargetListenerConfig($config)
     {
-        $cfg = [ 'listeners' => $config ];
+        $cfg = ['listeners' => $config];
         $this->target->expects($this->once())->method('getConfig')->willReturn($cfg);
     }
 
@@ -71,14 +71,14 @@ 
                          ->getMock();
 
 
-        $hasMap = [ ];
-        $getMap = [ ];
+        $hasMap = [];
+        $getMap = [];
         $defaultListenerMock = new AttachListenerTestListenerMock();
 
         $lazyAggregateMock = $this->getLazyAggregateMock($expectLazyListeners);
         if ($expectLazyListeners) {
             $lazyAggregateMock->expects($this->once())->method('attach');
-            $getMap[] = [ 'Core/Listener/DeferredListenerAggregate', $lazyAggregateMock ];
+            $getMap[] = ['Core/Listener/DeferredListenerAggregate', $lazyAggregateMock];
         }
 
         foreach ($listeners as $serviceName => $listenerMock) {
@@ -88,10 +88,10 @@ 
             }
 
             if (false === $listenerMock) {
-                $hasMap[] = [ $serviceName, false];
+                $hasMap[] = [$serviceName, false];
             } else {
-                $hasMap[] = [ $serviceName, true ];
-                $getMap[] = [ $serviceName, $listenerMock ];
+                $hasMap[] = [$serviceName, true];
+                $getMap[] = [$serviceName, $listenerMock];
             }
         }
 
@@ -135,7 +135,7 @@ 
 
     public function testPullsListenersFromServiceManager()
     {
-        $this->setTargetListenerConfig(['TestListener' => ['test-event','doSomething',1]]);
+        $this->setTargetListenerConfig(['TestListener' => ['test-event', 'doSomething', 1]]);
         $services = $this->getServiceManagerMock(['TestListener']);
         $this->target->createServiceWithName($services, 'irrelevant', 'irrelevant');
     }
@@ -143,7 +143,7 @@ 
     public function testCreatesListenersInstances()
     {
         $this->setTargetListenerConfig(
-            ['\CoreTest\Factory\EventManager\EventManagerAbstractFactory\AttachListenerTestListenerMock' => ['test-event','doSomething',1]]
+            ['\CoreTest\Factory\EventManager\EventManagerAbstractFactory\AttachListenerTestListenerMock' => ['test-event', 'doSomething', 1]]
         );
         $services = $this->getServiceManagerMock([
             '\CoreTest\Factory\EventManager\EventManagerAbstractFactory\AttachListenerTestListenerMock' => false
@@ -167,7 +167,7 @@ 
     {
         $aggregateMock = $this->getMockBuilder('\Laminas\EventManager\ListenerAggregateInterface')
                               ->getMockForAbstractClass();
-        $aggregateMock->expects($this->exactly(2))->method('attach')->withConsecutive([ $this->events, 1], [$this->events, -100]);
+        $aggregateMock->expects($this->exactly(2))->method('attach')->withConsecutive([$this->events, 1], [$this->events, -100]);
         
         $this->setTargetListenerConfig(['TestAggregate', 'TestAggregate2' => -100]);
         $services = $this->getServiceManagerMock(['TestAggregate' => $aggregateMock, 'TestAggregate2' => $aggregateMock]);
@@ -179,8 +179,8 @@ 
     {
         $listenerMock = new AttachListenerTestListenerMock();
         $this->setTargetListenerConfig([
-            'TestListener1' => ['test-event1','doSomething',1],
-            'TestListener2' => ['test-event2','doSomething',2]
+            'TestListener1' => ['test-event1', 'doSomething', 1],
+            'TestListener2' => ['test-event2', 'doSomething', 2]
         ]);
         $services = $this->getServiceManagerMock([
             'TestListener1' => $listenerMock,
@@ -190,8 +190,8 @@ 
             ->expects($this->exactly(2))
             ->method('attach')
             ->withConsecutive(
-                ['test-event1', [$listenerMock,'doSomething'], 1 ],
-                ['test-event2', [$listenerMock, 'doSomething'], 2 ]
+                ['test-event1', [$listenerMock, 'doSomething'], 1],
+                ['test-event2', [$listenerMock, 'doSomething'], 2]
             )
         ;
 
@@ -203,7 +203,7 @@ 
         $listenerMock = new AttachListenerTestListenerMock();
         $singleEvent = 'event';
         $expectSingleEvent = $singleEvent;
-        $multiEvent  = [ 'event1', 'event2' ];
+        $multiEvent  = ['event1', 'event2'];
         $method = 'doSomething';
         $additionalMethod = 'doSomethingElse';
         $priority = 1;
@@ -212,30 +212,30 @@ 
         $notLazy = false;
 
         $listeners = [
-            'Test01' => [ $singleEvent, $method],
-            'Test02' => [ $singleEvent, $method ],
-            'Test03' => [ $multiEvent, $method ],
-            'Test04' => [ $singleEvent, $method ],
-            'Test05' => [ $multiEvent, $method ],
-            'Test06' => [ $singleEvent, $method, $priority ],
-            'Test07' => [ $multiEvent, $method, $priority ],
-            'Test08' => [ $singleEvent, $method, $priority, $lazy ],
-            'Test09' => [ $multiEvent, $method, $priority, $lazy ],
-            'Test10' => [ $singleEvent,$method, $priority ],
-            'Test11' => [ $multiEvent, $method, $priority ],
-            'Test12' => [ $singleEvent,$method, $lazy ],
-            'Test13' => [ $multiEvent,$method, $lazy ],
-            'Test14' => [ $singleEvent, $method, $additionalMethod ],
-            'Test15' => [ $multiEvent, $method, $lazy, $priority, $notLazy, $additionalPriority ],
+            'Test01' => [$singleEvent, $method],
+            'Test02' => [$singleEvent, $method],
+            'Test03' => [$multiEvent, $method],
+            'Test04' => [$singleEvent, $method],
+            'Test05' => [$multiEvent, $method],
+            'Test06' => [$singleEvent, $method, $priority],
+            'Test07' => [$multiEvent, $method, $priority],
+            'Test08' => [$singleEvent, $method, $priority, $lazy],
+            'Test09' => [$multiEvent, $method, $priority, $lazy],
+            'Test10' => [$singleEvent, $method, $priority],
+            'Test11' => [$multiEvent, $method, $priority],
+            'Test12' => [$singleEvent, $method, $lazy],
+            'Test13' => [$multiEvent, $method, $lazy],
+            'Test14' => [$singleEvent, $method, $additionalMethod],
+            'Test15' => [$multiEvent, $method, $lazy, $priority, $notLazy, $additionalPriority],
         ];
 
         $expectedLazyListeners = [
-            [ 'service' => 'Test08', 'event' => $expectSingleEvent, 'method' => $method, 'priority' => $priority ],
-            [ 'service' => 'Test09', 'event' => 'event1', 'method' => $method, 'priority' => $priority ],
-            [ 'service' => 'Test09', 'event' => 'event2', 'method' => $method, 'priority' => $priority ],
-            [ 'service' => 'Test12', 'event' => $expectSingleEvent, 'method' => $method, 'priority' => 1 ],
-            [ 'service' => 'Test13', 'event' => 'event1', 'method' => $method, 'priority' => 1],
-            [ 'service' => 'Test13', 'event' => 'event2', 'method' => $method, 'priority' => 1],
+            ['service' => 'Test08', 'event' => $expectSingleEvent, 'method' => $method, 'priority' => $priority],
+            ['service' => 'Test09', 'event' => 'event1', 'method' => $method, 'priority' => $priority],
+            ['service' => 'Test09', 'event' => 'event2', 'method' => $method, 'priority' => $priority],
+            ['service' => 'Test12', 'event' => $expectSingleEvent, 'method' => $method, 'priority' => 1],
+            ['service' => 'Test13', 'event' => 'event1', 'method' => $method, 'priority' => 1],
+            ['service' => 'Test13', 'event' => 'event2', 'method' => $method, 'priority' => 1],
         ];
 
         $servicesCfg = array_fill_keys([
@@ -252,26 +252,26 @@ 
 
         $this->setTargetListenerConfig($listeners);
         $services = $this->getServiceManagerMock($servicesCfg, true);
-        $expectedCallback = [$listenerMock,'doSomething'];
+        $expectedCallback = [$listenerMock, 'doSomething'];
         $this->events->expects($this->exactly(16))
             ->method('attach')
             ->withConsecutive(
-                [ $expectSingleEvent, $expectedCallback, 1 ],
-                [ $expectSingleEvent, $expectedCallback, 1 ],
-                [ 'event1', $expectedCallback, 1],
-                [ 'event2', $expectedCallback, 1],
-                [ $expectSingleEvent, $expectedCallback, 1],
-                [ 'event1', $expectedCallback, 1],
-                [ 'event2', $expectedCallback, 1],
-                [ $expectSingleEvent, $expectedCallback, $priority],
-                [ 'event1', $expectedCallback, $priority],
-                [ 'event2', $expectedCallback, $priority],
-                [ $expectSingleEvent, $expectedCallback, $priority ],
-                [ 'event1', $expectedCallback, $priority ],
-                [ 'event2', $expectedCallback, $priority ],
-                [ $expectSingleEvent, [$listenerMock,$additionalMethod], 1],
-                [ 'event1', $expectedCallback, $additionalPriority ],
-                [ 'event2', $expectedCallback, $additionalPriority ]
+                [$expectSingleEvent, $expectedCallback, 1],
+                [$expectSingleEvent, $expectedCallback, 1],
+                ['event1', $expectedCallback, 1],
+                ['event2', $expectedCallback, 1],
+                [$expectSingleEvent, $expectedCallback, 1],
+                ['event1', $expectedCallback, 1],
+                ['event2', $expectedCallback, 1],
+                [$expectSingleEvent, $expectedCallback, $priority],
+                ['event1', $expectedCallback, $priority],
+                ['event2', $expectedCallback, $priority],
+                [$expectSingleEvent, $expectedCallback, $priority],
+                ['event1', $expectedCallback, $priority],
+                ['event2', $expectedCallback, $priority],
+                [$expectSingleEvent, [$listenerMock, $additionalMethod], 1],
+                ['event1', $expectedCallback, $additionalPriority],
+                ['event2', $expectedCallback, $additionalPriority]
         );
 
         $this->target->createServiceWithName($services, '', '');
@@ -280,32 +280,32 @@ 
     public function testNormalizeListenerOptionsWithArrayHashSpecification()
     {
         $listenerMock = new AttachListenerTestListenerMock();
-        $singleEvent = [ 'singleEvent' ];
-        $priorityEvent = [ 'prioEvent' => 10 ];
-        $methodEvent = [ 'methodEvent' => 'doSomething' ];
-        $singleAndPriorityAndMethodEvent = [ 'single', 'priority' => 10, 'method' => 'doSomething'];
-        $verboseEvent = [ 'verbose' => [ 'method' => 'doSomething', 'priority' => 20]];
-        $verboseFullEvent = [ 'verbose' => [ 'method' => ['doSomething', 'doSomethingElse' => 25], 'priority' => 30]];
-        $multiEvent = [ 'multi1', 'multi2' ];
+        $singleEvent = ['singleEvent'];
+        $priorityEvent = ['prioEvent' => 10];
+        $methodEvent = ['methodEvent' => 'doSomething'];
+        $singleAndPriorityAndMethodEvent = ['single', 'priority' => 10, 'method' => 'doSomething'];
+        $verboseEvent = ['verbose' => ['method' => 'doSomething', 'priority' => 20]];
+        $verboseFullEvent = ['verbose' => ['method' => ['doSomething', 'doSomethingElse' => 25], 'priority' => 30]];
+        $multiEvent = ['multi1', 'multi2'];
 
         $listeners = [
-            'Test01' => [ 'events' => $singleEvent, 'method' => 'doSomething'],
-            'Test02' => [ 'events' => $priorityEvent, 'method' => 'doSomething'],
-            'Test03' => [ 'events' => $methodEvent, 'method' => 'doSomething'],
-            'Test04' => [ 'events' => $singleEvent, 'method'=>'doSomething', 'priority' => 12],
-            'Test05' => [ 'events' => $priorityEvent, 'method'=>'doSomething', 'priority' => 12],
-            'Test06' => [ 'events' => $methodEvent,'method'=>'doSomething', 'priority' => 12],
-            'Test07' => [ 'events' => $singleEvent, 'method' => 'doSomething'],
-            'Test08' => [ 'events' => $priorityEvent, 'method' => 'doSomething'],
-            'Test09' => [ 'events' => $methodEvent, 'method' => 'doSomethingElse'],
-            'Test10' => [ 'events' => $singleEvent, 'method' => 'doSomething', 'priority' => 12],
-            'Test11' => [ 'events' => $priorityEvent, 'method' => 'doSomethingElse', 'priority' => 12],
-            'Test12' => [ 'events' => $methodEvent, 'method' => 'doSomethingElse', 'priority' => 12],
-            'Test13' => [ 'events' => $singleAndPriorityAndMethodEvent, 'method'=> 'doSomething'],
-            'Test14' => [ 'events' => $verboseEvent],
-            'Test15' => [ 'events' => $verboseFullEvent],
-            'Test16' => [ 'events' => $singleEvent, 'lazy' => true],
-            'Test17' => [ 'events' => $multiEvent, 'method' => 'multiMethod' ],
+            'Test01' => ['events' => $singleEvent, 'method' => 'doSomething'],
+            'Test02' => ['events' => $priorityEvent, 'method' => 'doSomething'],
+            'Test03' => ['events' => $methodEvent, 'method' => 'doSomething'],
+            'Test04' => ['events' => $singleEvent, 'method'=>'doSomething', 'priority' => 12],
+            'Test05' => ['events' => $priorityEvent, 'method'=>'doSomething', 'priority' => 12],
+            'Test06' => ['events' => $methodEvent, 'method'=>'doSomething', 'priority' => 12],
+            'Test07' => ['events' => $singleEvent, 'method' => 'doSomething'],
+            'Test08' => ['events' => $priorityEvent, 'method' => 'doSomething'],
+            'Test09' => ['events' => $methodEvent, 'method' => 'doSomethingElse'],
+            'Test10' => ['events' => $singleEvent, 'method' => 'doSomething', 'priority' => 12],
+            'Test11' => ['events' => $priorityEvent, 'method' => 'doSomethingElse', 'priority' => 12],
+            'Test12' => ['events' => $methodEvent, 'method' => 'doSomethingElse', 'priority' => 12],
+            'Test13' => ['events' => $singleAndPriorityAndMethodEvent, 'method'=> 'doSomething'],
+            'Test14' => ['events' => $verboseEvent],
+            'Test15' => ['events' => $verboseFullEvent],
+            'Test16' => ['events' => $singleEvent, 'lazy' => true],
+            'Test17' => ['events' => $multiEvent, 'method' => 'multiMethod'],
         ];
     
         $servicesCfg = array_fill_keys([
@@ -314,7 +314,7 @@ 
             'Test15', 'Test17',
         ], $listenerMock);
         $expectedLazyListeners = [
-            [ 'service' => 'Test16', 'event' => $singleEvent[0], 'method' => null, 'priority' => 1 ],
+            ['service' => 'Test16', 'event' => $singleEvent[0], 'method' => null, 'priority' => 1],
         ];
 
         $lazyAggregateMock = $this->getLazyAggregateMock();
@@ -328,33 +328,33 @@ 
         $this->setTargetListenerConfig($listeners);
         $services = $this->getServiceManagerMock($servicesCfg, true);
         
-        $callback1 = [$listenerMock,'doSomething'];
-        $callback2 = [$listenerMock,'doSomethingElse'];
+        $callback1 = [$listenerMock, 'doSomething'];
+        $callback2 = [$listenerMock, 'doSomethingElse'];
         $this->events
             ->expects($this->exactly(20))
             ->method('attach')
             ->withConsecutive(
-                [ $singleEvent[0], $callback1, 1 ],
-                [ 'prioEvent', $callback1, 10 ],
-                [ 'methodEvent', $callback1, 1],
-                [ $singleEvent[0], $callback1, 12],
-                [ 'prioEvent', $callback1, 10],
-                [ 'methodEvent', $callback1, 12],
-                [ $singleEvent[0], $callback1, 1],
-                [ 'prioEvent' , $callback1, 10],
-                [ 'methodEvent', $callback1, 1],
-                [ $singleEvent[0], $callback1, 12],
-                [ 'prioEvent', $callback2, 10],
-                [ 'methodEvent', $callback1, 12],
-                [ 'single', $callback1, 1 ],
-                [ 'method' , $callback1, 1],
-                [ 'priority', $callback1, 10],
+                [$singleEvent[0], $callback1, 1],
+                ['prioEvent', $callback1, 10],
+                ['methodEvent', $callback1, 1],
+                [$singleEvent[0], $callback1, 12],
+                ['prioEvent', $callback1, 10],
+                ['methodEvent', $callback1, 12],
+                [$singleEvent[0], $callback1, 1],
+                ['prioEvent', $callback1, 10],
+                ['methodEvent', $callback1, 1],
+                [$singleEvent[0], $callback1, 12],
+                ['prioEvent', $callback2, 10],
+                ['methodEvent', $callback1, 12],
+                ['single', $callback1, 1],
+                ['method', $callback1, 1],
+                ['priority', $callback1, 10],
                 //[ [ 'method' ], $callback1, 1],
-                [ 'verbose', $callback1, 20],
-                [ 'verbose', $callback1, 30],
-                [ 'verbose', $callback2, 25],
-                [ 'multi1', [$listenerMock,'multiMethod'], 1],
-                [ 'multi2', [$listenerMock,'multiMethod'], 1]
+                ['verbose', $callback1, 20],
+                ['verbose', $callback1, 30],
+                ['verbose', $callback2, 25],
+                ['multi1', [$listenerMock, 'multiMethod'], 1],
+                ['multi2', [$listenerMock, 'multiMethod'], 1]
             )
         ;
 

module/Core/test/CoreTest/Factory/EventManager/EventManagerAbstractFactory/InheritanceAndConfigMergingTest.php

Spacing +8 added lines, -8 removed lines

@@ -40,7 +40,7 @@ 
         '@testCanCreateServiceWithName' => ['mock' => ['canCreate' => 1]],
     ];
 
-    protected $inheritance = [ '\Laminas\ServiceManager\Factory\AbstractFactoryInterface' ];
+    protected $inheritance = ['\Laminas\ServiceManager\Factory\AbstractFactoryInterface'];
 
     public function testDeterminesIfItCanCreateAnEventManagerByName()
     {
@@ -66,17 +66,17 @@ 
         $config1 = [];
 
         $config2 = [
-            'event_manager' => [ 'Test2/Events' => [
+            'event_manager' => ['Test2/Events' => [
                 'service' => 'TestService',
                 'configure' => false,
-                'identifiers' => [ 'TestEvents', 'AnotherId' ],
+                'identifiers' => ['TestEvents', 'AnotherId'],
                 'event' => 'SomeEventClass',
-                'listeners' => [ 'listener' => 'event' ]
+                'listeners' => ['listener' => 'event']
             ]]
         ];
         return [
-            [ $config1, 'Test1/Events' ],
-            [ $config2, 'Test2/Events' ],
+            [$config1, 'Test1/Events'],
+            [$config2, 'Test2/Events'],
         ];
     }
 
@@ -91,14 +91,14 @@ 
         $defaults = [
             'service' => 'EventManager',
             'configure' => true,
-            'identifiers' => [ $reqName ],
+            'identifiers' => [$reqName],
             'event' => '\Laminas\EventManager\Event',
             'listeners' => [],
         ];
 
         /* @var EventManagerAbstractFactory|\PHPUnit_Framework_MockObject_MockObject $target */
         $target = $this->getMockBuilder('\Core\Factory\EventManager\EventManagerAbstractFactory')
-                       ->setMethods([ 'createEventManager', 'attachListeners' ])
+                       ->setMethods(['createEventManager', 'attachListeners'])
                        ->getMock();
 
         $services = new ServiceManager();

module/Core/test/CoreTest/Form/Event/FormEventTest.php

Spacing +21 added lines, -21 removed lines

@@ -36,41 +36,41 @@
      */
     protected $target = '\Core\Form\Event\FormEvent';
 
-    protected $inheritance = [ '\Laminas\EventManager\Event' ];
+    protected $inheritance = ['\Laminas\EventManager\Event'];
 
     public function propertiesProvider()
     {
         $form = new Form();
         $container = new Container();
         return [
-            [ 'Form', '@\Laminas\Form\Form' ],
-            [ 'Form', '@\Core\Form\Container' ],
-            [ 'Form', [ 'value' => 'invalid', 'setter_exception' => ['\InvalidArgumentException', 'Form must either' ] ] ],
-            [ 'Target', 'TestTarget' ],
-            [ 'Target', [
+            ['Form', '@\Laminas\Form\Form'],
+            ['Form', '@\Core\Form\Container'],
+            ['Form', ['value' => 'invalid', 'setter_exception' => ['\InvalidArgumentException', 'Form must either']]],
+            ['Target', 'TestTarget'],
+            ['Target', [
                 'value' => $form,
-                'expect_property' => [ 'form', $form ]
+                'expect_property' => ['form', $form]
             ]],
-            [ 'Target', [
+            ['Target', [
                 'value' => $container,
-                'expect_property' => [ 'form', $container ],
+                'expect_property' => ['form', $container],
             ]],
-            [ 'Target', [
+            ['Target', [
                 'value' => 'NonFormOrContainer',
-                'expect_property' => [ 'form', null ],
+                'expect_property' => ['form', null],
             ]],
-            [ 'Params', [ 'value' => [ 'param1' => 'value1' ]]],
-            [ 'Params', [
-                'value' => [ 'form' => $form ],
-                'expect_property' => [ 'form', $form ]
+            ['Params', ['value' => ['param1' => 'value1']]],
+            ['Params', [
+                'value' => ['form' => $form],
+                'expect_property' => ['form', $form]
             ]],
-            [ 'Params', [
-                'value' => [ 'form' => $container ],
-                'expect_property' => [ 'form', $container ]
+            ['Params', [
+                'value' => ['form' => $container],
+                'expect_property' => ['form', $container]
             ]],
-            [ 'Params', [
-                'value' => [ 'form' => 'notAnFormInstance' ],
-                'expect_property' => [ 'form', null ]
+            ['Params', [
+                'value' => ['form' => 'notAnFormInstance'],
+                'expect_property' => ['form', null]
             ]]
         ];
     }