cross-solution / YAWIK

module/Core/test/CoreTest/Form/SummaryFormButtonsFieldsetTest.php

Spacing +5 added lines, -5 removed lines

@@ -21,7 +21,7 @@ 
      */
     protected $target;
 
-    public function setUp(){
+    public function setUp() {
         $this->target = new SummaryFormButtonsFieldset();
     }
 
@@ -39,20 +39,20 @@ 
 
     public function testSetGetFormId()
     {
-        $id=123;
+        $id = 123;
         $this->target->setFormId($id);
-        $this->assertSame($this->target->getFormId(), (string)$id);
+        $this->assertSame($this->target->getFormId(), (string) $id);
     }
 
     public function testSetAttribute()
     {
-        $this->target->setAttribute('id',123);
+        $this->target->setAttribute('id', 123);
         $this->assertSame($this->target->getFormId(), "123");
     }
 
     public function testSetAttributeSomeAttribute()
     {
-        $this->target->setAttribute('foo','bar');
+        $this->target->setAttribute('foo', 'bar');
         $this->assertSame($this->target->getFormId(), false);
     }
 }
\ No newline at end of file

module/Core/test/CoreTest/Form/MetaDataFieldsetTest.php

Spacing +2 added lines, -2 removed lines

@@ -31,9 +31,9 @@
 
     private $target = MetaDataFieldset::class;
 
-    private $inheritance = [ Fieldset::class ];
+    private $inheritance = [Fieldset::class];
 
     private $properties = [
-        [ 'hydrator', ['default@' => MetaDataHydrator::class, '@value' => EntityHydrator::class]]
+        ['hydrator', ['default@' => MetaDataHydrator::class, '@value' => EntityHydrator::class]]
     ];
 }
\ No newline at end of file

module/Core/test/CoreTest/Form/DefaultButtonsFieldsetTest.php

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
      */
     protected $target;
 
-    public function setUp(){
+    public function setUp() {
         $this->target = new DefaultButtonsFieldset();
     }
 
@@ -32,7 +32,7 @@ 
     }
 
 
-    public function testInit(){
+    public function testInit() {
         /*@todo*/
     }
 }
\ No newline at end of file

module/Core/test/CoreTest/I18n/LocaleTest.php

Indentation +6 added lines, -6 removed lines

@@ -32,13 +32,13 @@
     protected $localeService;
     
     /**
-	 * @see PHPUnit_Framework_TestCase::setUp()
-	 */
-	protected function setUp()
-	{
-		$this->localeService = new LocaleService($this->languages);
+     * @see PHPUnit_Framework_TestCase::setUp()
+     */
+    protected function setUp()
+    {
+        $this->localeService = new LocaleService($this->languages);
 		
-	}
+    }
 
     public function testGetDefaultLanguage()
     {

module/Core/test/CoreTest/View/Helper/AjaxUrlTest.php

Spacing +4 added lines, -4 removed lines

@@ -37,18 +37,18 @@ 
     private $target = [
         AjaxUrl::class,
         [self::BASEPATH],
-        '@testInheritance' => [ 'as_reflection' => true ],
+        '@testInheritance' => ['as_reflection' => true],
         '@testConstruction' => false,
     ];
 
-    private $inheritance = [ AbstractHelper::class ];
+    private $inheritance = [AbstractHelper::class];
 
     public function testConstruction()
     {
         $basepath = '/test/path/no/slash';
         $target   = new AjaxUrl($basepath);
 
-        $this->assertAttributeSame($basepath . '/' , 'basePath', $target);
+        $this->assertAttributeSame($basepath.'/', 'basePath', $target);
 
         $basepath = '/test/path/slash/';
         $target   = new AjaxUrl($basepath);
@@ -75,7 +75,7 @@ 
     {
         $url = $this->target->__invoke(...$args);
 
-        $expect = self::BASEPATH . $expect;
+        $expect = self::BASEPATH.$expect;
         $this->assertEquals($expect, $url);
     }
 

module/Core/test/CoreTest/View/Helper/SnippetTest.php

Indentation +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
 
     public function testReturnsEmptyString()
     {
-    	$event = $this->args['event'];
+        $event = $this->args['event'];
         $event->setName('nothingToDo');
         $event->setTarget($this->target);
 

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
         '@testConstruction' => false,
     ];
 
-    private $inheritance = [ AbstractHelper::class ];
+    private $inheritance = [AbstractHelper::class];
 
     private function getTargetArgs()
     {

module/Core/test/CoreTest/View/Helper/ProxyTest.php

Indentation +6 added lines, -6 removed lines

@@ -60,19 +60,19 @@ 
 
     private function injectHelperManager($services = [])
     {
-    	$sm = $this->createServiceManagerMock();
+        $sm = $this->createServiceManagerMock();
         $manager = $this->createPluginManagerMock($services,$sm);
 
         $renderer = $this->getMockBuilder(PhpRenderer::class)
-			->disableOriginalConstructor()
+            ->disableOriginalConstructor()
             ->setMethods(['getHelperPluginManager'])
             ->getMock()
         ;
 
         $renderer
-	        ->expects($this->any())
-	        ->method('getHelperPluginManager')
-	        ->will($this->returnValue($manager))
+            ->expects($this->any())
+            ->method('getHelperPluginManager')
+            ->will($this->returnValue($manager))
         ;
 	    
         $this->target->setView($renderer);
@@ -153,7 +153,7 @@ 
         $options = ['option' => 'value'];
         $this->injectHelperManager(['helper' => PtHelperDummy::class]);
         $this->helperManager
-	        ->setExpectedCallCount('get', 'helper', $options, 1)
+            ->setExpectedCallCount('get', 'helper', $options, 1)
         ;
 
         $this->target->plugin('helper', $options);

Spacing +2 added lines, -2 removed lines

@@ -49,7 +49,7 @@ 
         ],
     ];
 
-    private $inheritance = [ AbstractHelper::class ];
+    private $inheritance = [AbstractHelper::class];
 
     /**
      *
@@ -61,7 +61,7 @@ 
     private function injectHelperManager($services = [])
     {
     	$sm = $this->createServiceManagerMock();
-        $manager = $this->createPluginManagerMock($services,$sm);
+        $manager = $this->createPluginManagerMock($services, $sm);
 
         $renderer = $this->getMockBuilder(PhpRenderer::class)
 			->disableOriginalConstructor()

module/Core/test/CoreTest/Service/ClearCacheServiceTest.php

Spacing +4 added lines, -4 removed lines

@@ -92,7 +92,7 @@ 
      * @throws \Exception
      * @dataProvider    getTestClearCache
      */
-    public function testClearCache($cacheDir, $expectThrow=false, $message='Test Cache Directory')
+    public function testClearCache($cacheDir, $expectThrow = false, $message = 'Test Cache Directory')
     {
         $options = $this->getMockBuilder(ListenerOptions::class)
             ->disableOriginalConstructor()
@@ -115,9 +115,9 @@ 
     {
         $cacheDir = sys_get_temp_dir().'/yawik/test-cache';
         return [
-            [null,true,'Test with null cache directory'],
-            [sys_get_temp_dir().'/foo',true,'Test with non existent directory'],
-            [$cacheDir,false,'Test with valid cache directory']
+            [null, true, 'Test with null cache directory'],
+            [sys_get_temp_dir().'/foo', true, 'Test with non existent directory'],
+            [$cacheDir, false, 'Test with valid cache directory']
         ];
     }
 

module/Core/test/CoreTest/Listener/DeferredListenerAggregateTest.php

Indentation +8 added lines, -8 removed lines

@@ -126,7 +126,7 @@ 
      */
     public function testAttachesAndDetachesToAndFromAnEventManager()
     {
-    	$testListener = new DLATListenerMock();
+        $testListener = new DLATListenerMock();
         $this->target->setListener('test01', 'service01');
         $this->target->setListener('test02', 'service02', 10);
 		
@@ -136,20 +136,20 @@ 
             ->setMethods(['attach', 'detach'])
             ->getMock();
         $events->expects($this->exactly(2))
-			->method('attach')
+            ->method('attach')
             ->withConsecutive(
-            	[ $this->equalTo('test01'), $this->anything(), $this->equalTo(0) ],
-				[ $this->equalTo('test02'), $this->anything(), $this->equalTo(10) ]
+                [ $this->equalTo('test01'), $this->anything(), $this->equalTo(0) ],
+                [ $this->equalTo('test02'), $this->anything(), $this->equalTo(10) ]
             )
             ->will($this->onConsecutiveCalls(
-            	[$testListener,'callback'], [$testListener,'callback'])
+                [$testListener,'callback'], [$testListener,'callback'])
             );
 
         $events
-	        ->expects($this->exactly(2))
-	        ->method('detach')
+            ->expects($this->exactly(2))
+            ->method('detach')
             ->withConsecutive(
-            	[$callback], [$callback] )
+                [$callback], [$callback] )
             ->will($this->onConsecutiveCalls([true, false]))
         ;
 

Spacing +34 added lines, -34 removed lines

@@ -40,7 +40,7 @@ 
         '@testFactoryMethodReturnsInstance' => false
     ];
 
-    protected $inheritance = [ '\Zend\EventManager\ListenerAggregateInterface' ];
+    protected $inheritance = ['\Zend\EventManager\ListenerAggregateInterface'];
     
     protected $services;
 
@@ -55,39 +55,39 @@ 
     public function propertiesProvider()
     {
         return [
-            [ 'listeners', [
-                'value' => [ [] ],
-                'setter_exception' => [ '\DomainException', 'Listener specification must be an array' ]
+            ['listeners', [
+                'value' => [[]],
+                'setter_exception' => ['\DomainException', 'Listener specification must be an array']
             ]],
-            [ 'listeners', [
-                'value' => [ [ 'event' => 'test' ] ],
-                'setter_exception' => [ '\DomainException', 'Listener specification must be an array' ]
+            ['listeners', [
+                'value' => [['event' => 'test']],
+                'setter_exception' => ['\DomainException', 'Listener specification must be an array']
             ]],
-            [ 'listeners', [
-                'value' => [ [ 'service' => 'test' ] ],
-                'setter_exception' => [ '\DomainException', 'Listener specification must be an array' ]
+            ['listeners', [
+                'value' => [['service' => 'test']],
+                'setter_exception' => ['\DomainException', 'Listener specification must be an array']
             ]],
-            [ 'listeners', [
-                'value' => [ [ 'event' => 'test', 'service' => 'service' ] ],
+            ['listeners', [
+                'value' => [['event' => 'test', 'service' => 'service']],
                 'ignore_getter' => true,
-                'post' => ['assertListenerSpecsProperty', [ '', '@target', '###' ] ],
+                'post' => ['assertListenerSpecsProperty', ['', '@target', '###']],
             ]],
-            [ 'listeners', [
-                'value' => [ [ 'event' => 'test2', 'service' => 'someClass', 'method' => 'method', 'priority' => 12 ] ],
+            ['listeners', [
+                'value' => [['event' => 'test2', 'service' => 'someClass', 'method' => 'method', 'priority' => 12]],
                 'ignore_getter' => true,
-                'post' => ['assertListenerSpecsProperty', [ '', '@target', '###' ] ],
+                'post' => ['assertListenerSpecsProperty', ['', '@target', '###']],
             ]],
-            [ 'listener', [
+            ['listener', [
                 'value' => 'test',
-                'setter_args' => [ 'service', 'method', 10 ],
-                'setter_value' => [ 'event' => 'test', 'service' => 'service', 'method' => 'method', 'priority' => 10, 'instance' => null ],
+                'setter_args' => ['service', 'method', 10],
+                'setter_value' => ['event' => 'test', 'service' => 'service', 'method' => 'method', 'priority' => 10, 'instance' => null],
                 'setter_assert' => 'assertListenerSpecsProperty',
                 'ignore_getter' => true,
             ]],
-            [ 'listener', [
+            ['listener', [
                 'value' => 'test',
-                'setter_args' => [ 'service', 10 ],
-                'setter_value' => [ 'event' => 'test', 'service' => 'service', 'method' => null, 'priority' => 10, 'instance' => null ],
+                'setter_args' => ['service', 10],
+                'setter_value' => ['event' => 'test', 'service' => 'service', 'method' => null, 'priority' => 10, 'instance' => null],
                 'setter_assert' => 'assertListenerSpecsProperty',
                 'ignore_getter' => true,
             ]]
@@ -131,18 +131,18 @@ 
         $this->target->setListener('test02', 'service02', 10);
 		
         //In ZF3 EventManager detach should be a callable or it will throws an error
-        $callback = [$testListener,'callback'];
+        $callback = [$testListener, 'callback'];
         $events = $this->getMockBuilder(EventManager::class)
             ->setMethods(['attach', 'detach'])
             ->getMock();
         $events->expects($this->exactly(2))
 			->method('attach')
             ->withConsecutive(
-            	[ $this->equalTo('test01'), $this->anything(), $this->equalTo(0) ],
-				[ $this->equalTo('test02'), $this->anything(), $this->equalTo(10) ]
+            	[$this->equalTo('test01'), $this->anything(), $this->equalTo(0)],
+				[$this->equalTo('test02'), $this->anything(), $this->equalTo(10)]
             )
             ->will($this->onConsecutiveCalls(
-            	[$testListener,'callback'], [$testListener,'callback'])
+            	[$testListener, 'callback'], [$testListener, 'callback'])
             );
 
         $events
@@ -178,13 +178,13 @@ 
     public function listenerProvider()
     {
         return [
-            [ 'nonExistant', null ],
-            [ '\CoreTest\Listener\DLATListenerMock', null ],
-            [ 'listener', new DLATListenerMock() ],
-            [ 'listener', new DLATListenerMock(), 'invoke' ],
-            [ 'listener', new DLATListenerMock(), 'method', 'callback' ],
-            [ 'listener', new DLATListenerMock(), 'methodButNone', 'noMethod' ],
-            [ 'listener', new DLATNonInvokableListenerMock() ],
+            ['nonExistant', null],
+            ['\CoreTest\Listener\DLATListenerMock', null],
+            ['listener', new DLATListenerMock()],
+            ['listener', new DLATListenerMock(), 'invoke'],
+            ['listener', new DLATListenerMock(), 'method', 'callback'],
+            ['listener', new DLATListenerMock(), 'methodButNone', 'noMethod'],
+            ['listener', new DLATNonInvokableListenerMock()],
         ];
     }
 
@@ -202,7 +202,7 @@ 
     {
         if (null === $listener) {
             if (!class_exists($service, true)) {
-                $this->setExpectedException('\UnexpectedValueException', 'Cannot create deferred listener "' . $service );
+                $this->setExpectedException('\UnexpectedValueException', 'Cannot create deferred listener "'.$service);
             }
             $this->services->expects($this->once())->method('has')->with($service)->willReturn(false);
             $this->services->expects($this->never())->method('get');