cross-solution / YAWIK

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

Indentation +8 added lines, -8 removed lines

@@ -41,8 +41,8 @@ 
     protected function setUp()
     {
         $this->target = $this->getMockBuilder('\Core\Factory\EventManager\EventManagerAbstractFactory')
-                             ->setMethods([ 'attachListeners', 'getConfig' ])
-                             ->getMock();
+                                ->setMethods([ 'attachListeners', 'getConfig' ])
+                                ->getMock();
     }
 
     protected function setTargetConfig($config)
@@ -69,8 +69,8 @@ 
     protected function getServiceManagerMock($events, $args = [])
     {
         $services = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager')
-                         ->disableOriginalConstructor()
-                         ->getMock();
+                            ->disableOriginalConstructor()
+                            ->getMock();
 
 
         if (is_Array($events)) {
@@ -101,7 +101,7 @@ 
             ->will($this->returnValueMap($hasMap))
         ;
         $services->expects($this->exactly(count($getMap)))
-                 ->method('get')->will($this->returnValueMap($getMap));
+                    ->method('get')->will($this->returnValueMap($getMap));
 
         return $services;
     }
@@ -179,9 +179,9 @@ 
     {
         $event = new \Zend\EventManager\Event();
         $events = $this->getMockBuilder('\Zend\EventManager\EventManager')
-                       ->disableOriginalConstructor()
-                       ->setMethods(['setIdentifiers', 'setEventPrototype'])
-                       ->getMock()
+                        ->disableOriginalConstructor()
+                        ->setMethods(['setIdentifiers', 'setEventPrototype'])
+                        ->getMock()
         ;
         $events
             ->expects($this->once())

Spacing +12 added lines, -12 removed lines

@@ -41,7 +41,7 @@ 
     protected function setUp()
     {
         $this->target = $this->getMockBuilder('\Core\Factory\EventManager\EventManagerAbstractFactory')
-                             ->setMethods([ 'attachListeners', 'getConfig' ])
+                             ->setMethods(['attachListeners', 'getConfig'])
                              ->getMock();
     }
 
@@ -54,7 +54,7 @@ 
             $config['listeners'] = [];
         }
         if (!isset($config['identifiers'])) {
-            $config['identifiers'] = [ 'Test/Events/Manager' ];
+            $config['identifiers'] = ['Test/Events/Manager'];
         }
         if (!isset($config['event'])) {
             $config['event'] = '\Zend\EventManager\Event';
@@ -81,15 +81,15 @@ 
             $eventsService = 'Test/Events/Manager';
         }
 
-        $hasMap = [ [ $eventsService, true ] ];
-        $getMap = [ [ $eventsService, $events ] ];
+        $hasMap = [[$eventsService, true]];
+        $getMap = [[$eventsService, $events]];
 
         foreach ($args as $serviceName => $serviceValue) {
             if (false === $serviceValue) {
-                $hasMap[] = [ $serviceName, true ];
+                $hasMap[] = [$serviceName, true];
             } else {
-                $hasMap[] = [ $serviceName, true];
-                $getMap[] = [ $serviceName, $serviceValue ];
+                $hasMap[] = [$serviceName, true];
+                $getMap[] = [$serviceName, $serviceValue];
             }
         }
 
@@ -113,7 +113,7 @@ 
 
         $services->setService('Test/Events/Manager', $events);
 
-        $this->setTargetConfig([ 'service' => 'Test/Events/Manager', 'configure' => false, 'listeners' => []]);
+        $this->setTargetConfig(['service' => 'Test/Events/Manager', 'configure' => false, 'listeners' => []]);
 
         $this->target->expects($this->once())->method('attachListeners')->with($services, $events, []);
         $this->target->createServiceWithName($services, 'irrelevant', 'Test/Events');
@@ -121,7 +121,7 @@ 
 
     public function testInstatiatesEventManagerFromClassName()
     {
-        $this->setTargetConfig([ 'service' => '\Zend\EventManager\EventManager', 'configure' => false, 'listeners' => []]);
+        $this->setTargetConfig(['service' => '\Zend\EventManager\EventManager', 'configure' => false, 'listeners' => []]);
 
         $services = new ServiceManager();
         $events = $this->target->createServiceWithName($services, 'irrelevant', 'irrelevant');
@@ -134,7 +134,7 @@ 
         $this->expectException(\UnexpectedValueException::class);
         $this->expectExceptionMessage('Cannot create');
 
-        $this->setTargetConfig(['service' => 'NonExistantClass' ]);
+        $this->setTargetConfig(['service' => 'NonExistantClass']);
 
         $this->target->createServiceWithName(new ServiceManager(), 'irrelevant', 'irrelevant');
     }
@@ -149,7 +149,7 @@ 
 
     public function testSetsIdentifiersOnEventManagerInstance()
     {
-        $ids = [ 'testID', 'TestId2' ];
+        $ids = ['testID', 'TestId2'];
         $events = new EventManager();
         $services = $this->getServiceManagerMock(['EventManager' => $events]);
         $this->setTargetConfig(['configure' => true, 'identifiers' => $ids, 'service' => 'EventManager']);
@@ -170,7 +170,7 @@ 
             ->with($event)
         ;
 
-        $services = $this->getServiceManagerMock(['EventManager' => $events], [ 'Event' => $event ]);
+        $services = $this->getServiceManagerMock(['EventManager' => $events], ['Event' => $event]);
         $this->setTargetConfig(['configure' => true, 'service' => 'EventManager', 'event' => 'Event']);
         $this->target->createServiceWithName($services, 'irrelevant', 'Test/Events');
     }

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

Indentation +10 added lines, -10 removed lines

@@ -46,13 +46,13 @@ 
     protected function setUp()
     {
         $this->target = $this->getMockBuilder('\Core\Factory\EventManager\EventManagerAbstractFactory')
-                             ->setMethods([ 'createEventManager', 'getConfig' ])
-                             ->getMock();
+                                ->setMethods([ 'createEventManager', 'getConfig' ])
+                                ->getMock();
 
         $events = $this->getMockBuilder('\Zend\EventManager\EventManager')
-                       ->disableOriginalConstructor()
-                       ->setMethods(['attach'])
-                       ->getMock();
+                        ->disableOriginalConstructor()
+                        ->setMethods(['attach'])
+                        ->getMock();
 
         $this->target->expects($this->once())->method('createEventManager')->willReturn($events);
         $this->events = $events;
@@ -67,8 +67,8 @@ 
     protected function getServiceManagerMock($listeners, $expectLazyListeners = false)
     {
         $services = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager')
-                         ->disableOriginalConstructor()
-                         ->getMock();
+                            ->disableOriginalConstructor()
+                            ->getMock();
 
 
         $hasMap = [ ];
@@ -101,8 +101,8 @@ 
         ;
 
         $services->expects($this->exactly(count($getMap)))
-                 ->method('get')
-                 ->will($this->returnValueMap($getMap))
+                    ->method('get')
+                    ->will($this->returnValueMap($getMap))
         ;
 
         return $services;
@@ -166,7 +166,7 @@ 
     public function testCallsAttachOnListenerAggregates()
     {
         $aggregateMock = $this->getMockBuilder('\Zend\EventManager\ListenerAggregateInterface')
-                              ->getMockForAbstractClass();
+                                ->getMockForAbstractClass();
         $aggregateMock->expects($this->exactly(2))->method('attach')->withConsecutive([ $this->events, 1], [$this->events, -100]);
         
         $this->setTargetListenerConfig(['TestAggregate', 'TestAggregate2' => -100]);

Spacing +101 added lines, -101 removed lines

@@ -46,7 +46,7 @@ 
     protected function setUp()
     {
         $this->target = $this->getMockBuilder('\Core\Factory\EventManager\EventManagerAbstractFactory')
-                             ->setMethods([ 'createEventManager', 'getConfig' ])
+                             ->setMethods(['createEventManager', 'getConfig'])
                              ->getMock();
 
         $events = $this->getMockBuilder('\Zend\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('\Zend\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/View/Helper/SnippetFactoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
         '@testCreateService' => ['mock' => ['__invoke']],
     ];
 
-    private $inheritance = [ FactoryInterface::class ];
+    private $inheritance = [FactoryInterface::class];
 
 
     public function testCreateService()

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('Zend\ServiceManager\ServiceManager')->disableOriginalConstructor()->getMock();
+        $HauptServiceLocator = $this->getMockBuilder('Zend\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/Paginator/RepositoryAbstractFactoryTest.php

Indentation +3 added lines, -3 removed lines

@@ -100,7 +100,7 @@ 
         $qb->expects($this->once())->method('getQuery')->willReturn($q);
 
         $repositories = $this->getMockBuilder('\Core\Repository\RepositoryService')
-                             ->disableOriginalConstructor()->getMock();
+                                ->disableOriginalConstructor()->getMock();
 
         $repositories->expects($this->once())->method('createQueryBuilder')->willReturn($qb);
 
@@ -120,8 +120,8 @@ 
         $sm = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager')->disableOriginalConstructor()->getMock();
 
         $sm->expects($this->exactly(2))->method('get')
-                                       ->withConsecutive([ 'repositories' ], [ 'FilterManager'])
-                                       ->will($this->onConsecutiveCalls($repositories, $filters));
+                                        ->withConsecutive([ 'repositories' ], [ 'FilterManager'])
+                                        ->will($this->onConsecutiveCalls($repositories, $filters));
 
         $target = $this->target;
         $paginator = $target($sm, $serviceName, $options);

Spacing +10 added lines, -10 removed lines

@@ -44,10 +44,10 @@ 
     public function serviceNamesProvider()
     {
         return [
-            [ 'Repository/Core/FileEntity', true ],
-            [ 'Core/FileEntity', true ],
-            [ 'Repository/NoModuleWillBeEverCalledThat/SomeEntity', false ],
-            [ 'ThisIsAHighlyUnCommonModuleName/ObscureEntity', false ],
+            ['Repository/Core/FileEntity', true],
+            ['Core/FileEntity', true],
+            ['Repository/NoModuleWillBeEverCalledThat/SomeEntity', false],
+            ['ThisIsAHighlyUnCommonModuleName/ObscureEntity', false],
         ];
     }
 
@@ -65,7 +65,7 @@ 
             ->disableOriginalConstructor()
             ->getMock()
         ;
-        $method = "assert" . ($expected ? 'True' : 'False');
+        $method = "assert".($expected ? 'True' : 'False');
 
         $this->{$method}($this->target->canCreate($sm, $serviceName));
     }
@@ -73,8 +73,8 @@ 
     public function servicesProvider()
     {
         return [
-            [ 'Repository/Core/TestEntity', '\Core\Entity\TestEntity', [], false ],
-            [ 'Core/TestEntity', '\Core\Entity\TestEntity', [ 'testOption' => 'testValue' ], true ]
+            ['Repository/Core/TestEntity', '\Core\Entity\TestEntity', [], false],
+            ['Core/TestEntity', '\Core\Entity\TestEntity', ['testOption' => 'testValue'], true]
         ];
     }
 
@@ -108,19 +108,19 @@ 
                         ->disableOriginalConstructor()
                         ->setMethods(['has', 'get'])->getMockForAbstractClass();
 
-        $filters->expects($this->once())->method('has')->with('PaginationQuery/' . $serviceName)->willReturn($hasFilter);
+        $filters->expects($this->once())->method('has')->with('PaginationQuery/'.$serviceName)->willReturn($hasFilter);
 
         if ($hasFilter) {
             $filter = $this->getMockBuilder('\Zend\Filter\FilterInterface')->getMockForAbstractClass();
             $filter->expects($this->once())->method('filter')->with($options, $qb)->willReturn($qb);
 
-            $filters->expects($this->once())->method('get')->with('PaginationQuery/' . $serviceName)->willReturn($filter);
+            $filters->expects($this->once())->method('get')->with('PaginationQuery/'.$serviceName)->willReturn($filter);
         }
 
         $sm = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager')->disableOriginalConstructor()->getMock();
 
         $sm->expects($this->exactly(2))->method('get')
-                                       ->withConsecutive([ 'repositories' ], [ 'FilterManager'])
+                                       ->withConsecutive(['repositories'], ['FilterManager'])
                                        ->will($this->onConsecutiveCalls($repositories, $filters));
 
         $target = $this->target;

module/Core/test/CoreTest/Collection/IdentityWrapperTest.php

Spacing +16 added lines, -16 removed lines

@@ -34,7 +34,7 @@ 
     
     protected function setUp()
     {
-        $getEntryMock = function ($id) {
+        $getEntryMock = function($id) {
             $entry = $this->getMockBuilder(\stdClass::class)
                 ->setMethods(['getId'])
                 ->getMock();
@@ -291,11 +291,11 @@ 
     public function testExistsWithKey()
     {
         $searchKey = key($this->entries);
-        $this->assertTrue($this->identityWrapper->exists(function ($key) use ($searchKey) {
+        $this->assertTrue($this->identityWrapper->exists(function($key) use ($searchKey) {
             return $key === $searchKey;
         }));
         $searchKey = 'non-existent';
-        $this->assertFalse($this->identityWrapper->exists(function ($key) use ($searchKey) {
+        $this->assertFalse($this->identityWrapper->exists(function($key) use ($searchKey) {
             return $key === $searchKey;
         }));
     }
@@ -306,11 +306,11 @@ 
     public function testExistsWithValue()
     {
         $searchValue = current($this->entries);
-        $this->assertTrue($this->identityWrapper->exists(function ($key, $value) use ($searchValue) {
+        $this->assertTrue($this->identityWrapper->exists(function($key, $value) use ($searchValue) {
             return $value === $searchValue;
         }));
         $searchValue = 'non-existent';
-        $this->assertFalse($this->identityWrapper->exists(function ($key, $value) use ($searchValue) {
+        $this->assertFalse($this->identityWrapper->exists(function($key, $value) use ($searchValue) {
             return $value === $searchValue;
         }));
     }
@@ -321,7 +321,7 @@ 
     public function testFilter()
     {
         $searchValue = current($this->entries);
-        $filter = function ($value) use ($searchValue) {
+        $filter = function($value) use ($searchValue) {
             return $value === $searchValue;
         };
         $filtered = $this->identityWrapper->filter($filter);
@@ -336,11 +336,11 @@ 
     public function testForAllWithKey()
     {
         $keys = array_keys($this->entries);
-        $this->assertTrue($this->identityWrapper->forAll(function ($key) use ($keys) {
+        $this->assertTrue($this->identityWrapper->forAll(function($key) use ($keys) {
             return in_array($key, $keys);
         }));
         $keys = ['non-existent'];
-        $this->assertFalse($this->identityWrapper->forAll(function ($key) use ($keys) {
+        $this->assertFalse($this->identityWrapper->forAll(function($key) use ($keys) {
             return in_array($key, $keys);
         }));
     }
@@ -351,11 +351,11 @@ 
     public function testForAllWithValue()
     {
         $values = array_values($this->entries);
-        $this->assertTrue($this->identityWrapper->forAll(function ($key, $value) use ($values) {
+        $this->assertTrue($this->identityWrapper->forAll(function($key, $value) use ($values) {
             return in_array($value, $values, true);
         }));
         $values = ['non-existent'];
-        $this->assertFalse($this->identityWrapper->forAll(function ($key, $value) use ($values) {
+        $this->assertFalse($this->identityWrapper->forAll(function($key, $value) use ($values) {
             return in_array($value, $values, true);
         }));
     }
@@ -365,7 +365,7 @@ 
      */
     public function testMapValidOperation()
     {
-        $map = function ($value) {
+        $map = function($value) {
             $value->random = rand();
             return $value;
         };
@@ -381,7 +381,7 @@ 
      */
     public function testMapInvalidOperation()
     {
-        $map = function ($value) {
+        $map = function($value) {
             return sprintf('id: %s', $value->getId());
         };
         $this->identityWrapper->map($map);
@@ -393,7 +393,7 @@ 
     public function testPartitionWithKey()
     {
         $searchKey = key($this->entries);
-        $partitions = $this->identityWrapper->partition(function ($key) use ($searchKey) {
+        $partitions = $this->identityWrapper->partition(function($key) use ($searchKey) {
             return $key === $searchKey;
         });
         $expectedFirstPartiton = array_slice($this->entries, 0, 1, true);
@@ -412,7 +412,7 @@ 
     public function testPartitionWithValue()
     {
         $searchValue = reset($this->entries);
-        $partitions = $this->identityWrapper->partition(function ($key, $value) use ($searchValue) {
+        $partitions = $this->identityWrapper->partition(function($key, $value) use ($searchValue) {
             return $value === $searchValue;
         });
         $expectedFirstPartiton = array_slice($this->entries, 0, 1, true);
@@ -554,8 +554,8 @@ 
     public function testSetIdentityExtractor()
     {
         $suffix = '.suffix';
-        $identityExtractor = function ($entry) use ($suffix) {
-            return $entry->getId() . $suffix;
+        $identityExtractor = function($entry) use ($suffix) {
+            return $entry->getId().$suffix;
         };
         
         $this->assertSame($this->identityWrapper, $this->identityWrapper->setIdentityExtractor($identityExtractor));

module/Core/test/CoreTest/Options/ImageSetOptionsTest.php

Spacing +2 added lines, -2 removed lines

@@ -32,12 +32,12 @@
 
     private $target = ImageSetOptions::class;
 
-    private $inheritance = [ AbstractOptions::class ];
+    private $inheritance = [AbstractOptions::class];
 
     private $properties = [
         ['entityClass', ['default' => Image::class, '@value' => Image::class]],
         ['formElementName', ['default' => ImageSetInterface::ORIGINAL, 'value' => 'formElementNameValue']],
-        ['images', ['default'=>[ImageSetInterface::THUMBNAIL => [100,100]], 'value' => ['a' => [1,1]]]],
+        ['images', ['default'=>[ImageSetInterface::THUMBNAIL => [100, 100]], 'value' => ['a' => [1, 1]]]],
         ['entity', ['ignore_setter' => true, 'value@' => Image::class]],
     ];
 }

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

Indentation +4 added lines, -4 removed lines

@@ -84,12 +84,12 @@
         ));
 
         $translator = $this->getMockBuilder('\Zend\I18n\Translator\Translator')
-                           ->disableOriginalConstructor()
-                           ->getMock();
+                            ->disableOriginalConstructor()
+                            ->getMock();
 
         $services = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager')
-                         ->disableOriginalConstructor()
-                         ->getMock();
+                            ->disableOriginalConstructor()
+                            ->getMock();
         
 
         $services

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
             'stringtemplate' => '\Core\Mail\StringTemplateMessage',
         );
         $factories = array(
-            'htmltemplate'   => [HTMLTemplateMessage::class,'factory']
+            'htmltemplate'   => [HTMLTemplateMessage::class, 'factory']
         );
 
         $this->assertAttributeEquals(false, 'shareByDefault', $target, 'shareByDefault is not set to FALSE');

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

Spacing +6 added lines, -6 removed lines

@@ -40,7 +40,7 @@ 
     protected function setUp()
     {
         $test = $this;
-        $sendCallback = function ($value) use ($test) {
+        $sendCallback = function($value) use ($test) {
             return $value === $test->expectedMail;
         };
         $transport = $this->getMockForAbstractClass('\Zend\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()