cross-solution / YAWIK

module/Core/test/CoreTest/Listener/LanguageRouteListener/OnRouteCallbackTest.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 
     private $target = [
         LanguageRouteListener::class,
-        'mock' => [ 'detectLanguage' => ['return' => 'xx'], 'setLocale' ],
+        'mock' => ['detectLanguage' => ['return' => 'xx'], 'setLocale'],
         'args' => 'getConstructorArgs'
     ];
     

module/Core/test/CoreTest/Listener/LanguageRouteListener/OnDispatchErrorCallbackTest.php

Indentation +20 added lines, -20 removed lines

@@ -46,21 +46,21 @@ 
     private $moduleOptions;
 
     public function testHandleConsoleRequests()
-   {
-       $event = $this
-           ->getMockBuilder(MvcEvent::class)
-           ->disableOriginalConstructor()
-           ->setMethods(['getRequest', 'getError'])
-           ->getMock()
-       ;
+    {
+        $event = $this
+            ->getMockBuilder(MvcEvent::class)
+            ->disableOriginalConstructor()
+            ->setMethods(['getRequest', 'getError'])
+            ->getMock()
+        ;
 
-       $request = new \Zend\Console\Request();
+        $request = new \Zend\Console\Request();
 
-       $event->expects($this->once())->method('getRequest')->willReturn($request);
-       $event->expects($this->never())->method('getError');
+        $event->expects($this->once())->method('getRequest')->willReturn($request);
+        $event->expects($this->never())->method('getError');
 
-       $this->assertNull($this->target->onDispatchError($event));
-   }
+        $this->assertNull($this->target->onDispatchError($event));
+    }
 
     public function testHandleNonRouteMatchErrors()
     {
@@ -134,8 +134,8 @@ 
         $event = $this->getEventMock('/base/uri', '/see/no/lang');
 
         $event->expects($this->once())
-              ->method('stopPropagation')
-              ->with(true)
+                ->method('stopPropagation')
+                ->with(true)
         ;
         $response = new Response();
         $event->setResponse($response);
@@ -148,18 +148,18 @@ 
                     \PHPUnit_Framework_Assert::assertEquals('/base/uri/xx/see/no/lang', (string) $v->getUri());
                     return true;
                 }
-              ))
+                ))
             ->willReturn($routeMatch)
         ;
 
         $this->target
-	        ->expects($this->once())
-	        ->method('redirect')
-	        ->with($response, '/base/uri/xx/see/no/lang')
+            ->expects($this->once())
+            ->method('redirect')
+            ->with($response, '/base/uri/xx/see/no/lang')
         ;
         $this->target
-	        ->expects($this->never())
-	        ->method('setLocale')
+            ->expects($this->never())
+            ->method('setLocale')
         ;
 
         $this->target->onDispatchError($event);

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
 
     private $target = [
         LanguageRouteListener::class,
-        'mock' => [ 'detectLanguage' => ['return' => 'xx'], 'setLocale', 'isSupportedLanguage', 'redirect'],
+        'mock' => ['detectLanguage' => ['return' => 'xx'], 'setLocale', 'isSupportedLanguage', 'redirect'],
         'args' => 'getConstructorArgs'
     ];
 
@@ -221,7 +221,7 @@ 
 
         $target->expects($this->once())
             ->method('redirect')
-            ->with($response,'/base/uri/default/see/no/lang')
+            ->with($response, '/base/uri/default/see/no/lang')
         ;
 
         $target->onDispatchError($event);
@@ -230,7 +230,7 @@ 
     public function getConstructorArgs()
     {
         $this->moduleOptions = new ModuleOptions();
-        return [new LocaleService([]),$this->moduleOptions];
+        return [new LocaleService([]), $this->moduleOptions];
     }
 
 

module/Core/test/CoreTest/Listener/LanguageRouteListener/BaseTest.php

Indentation +7 added lines, -7 removed lines

@@ -71,7 +71,7 @@ 
      */
     public function testDetach()
     {
-    	$callableListener = [new CallableListenerMock(),'doSomething'];
+        $callableListener = [new CallableListenerMock(),'doSomething'];
     	
         $events = $this
             ->getMockBuilder(EventManager::class)
@@ -80,11 +80,11 @@ 
             ->getMock();
 
         $events
-	        ->expects($this->any())
-	        ->method('attach')
-	        ->will(
-	        	$this->onConsecutiveCalls($callableListener, $callableListener)
-	        )
+            ->expects($this->any())
+            ->method('attach')
+            ->will(
+                $this->onConsecutiveCalls($callableListener, $callableListener)
+            )
         ;
 
         $events
@@ -103,5 +103,5 @@ 
 
 class CallableListenerMock
 {
-	public function doSomething(){}
+    public function doSomething(){}
 }

Spacing +4 added lines, -4 removed lines

@@ -35,11 +35,11 @@ 
 
     private $target;
 
-    private $inheritance = [ ListenerAggregateInterface::class ];
+    private $inheritance = [ListenerAggregateInterface::class];
 
     public function setUp()
     {
-        $this->target = new LanguageRouteListener(new LocaleService(['xx' => 'xx_XX']),new ModuleOptions());
+        $this->target = new LanguageRouteListener(new LocaleService(['xx' => 'xx_XX']), new ModuleOptions());
     }
     
     public function testAttachsToExpectedEvents()
@@ -71,7 +71,7 @@ 
      */
     public function testDetach()
     {
-    	$callableListener = [new CallableListenerMock(),'doSomething'];
+    	$callableListener = [new CallableListenerMock(), 'doSomething'];
     	
         $events = $this
             ->getMockBuilder(EventManager::class)
@@ -103,5 +103,5 @@ 
 
 class CallableListenerMock
 {
-	public function doSomething(){}
+	public function doSomething() {}
 }

module/Core/test/CoreTest/Listener/Events/AjaxEventTest.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 
     private $target = AjaxEvent::class;
 
-    private $inheritance = [ Event::class ];
+    private $inheritance = [Event::class];
 
     public function propertiesProvider()
     {

module/Core/test/CoreTest/Listener/Events/CreatePaginatorEventTest.php

Spacing +8 added lines, -8 removed lines

@@ -24,7 +24,7 @@ 
  */
 class CreatePaginatorEventTest extends \PHPUnit_Framework_TestCase
 {
-    use TestSetterGetterTrait,SetupTargetTrait;
+    use TestSetterGetterTrait, SetupTargetTrait;
 
     protected $target = [
         'class' => CreatePaginatorEvent::class
@@ -41,16 +41,16 @@ 
             ->getMock()
         ;
         return [
-            ['paginatorParams',[
+            ['paginatorParams', [
                 'value'     =>['name'=>'value'],
                 'default'   => array(),
             ]],
-            ['paginatorName',[
+            ['paginatorName', [
                 'value' => 'Some/Paginator',
                 'default'   => null,
             ]],
-            ['paginators',$paginators],
-            ['paginator',$paginator]
+            ['paginators', $paginators],
+            ['paginator', $paginator]
         ];
     }
 
@@ -72,9 +72,9 @@ 
         $target = $this->target;
         $target->setParams($params);
 
-        $this->assertEquals($params['paginatorName'],$target->getPaginatorName());
-        $this->assertEquals($params['paginatorParams'],$target->getPaginatorParams());
-        $this->assertEquals($params['paginators'],$target->getPaginators());
+        $this->assertEquals($params['paginatorName'], $target->getPaginatorName());
+        $this->assertEquals($params['paginatorParams'], $target->getPaginatorParams());
+        $this->assertEquals($params['paginators'], $target->getPaginators());
         $this->assertEquals(
             ['foo' => 'bar'],
             $target->getParams(),

module/Core/test/CoreTest/Listener/Events/FileEventTest.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
 
     private $target = FileEvent::class;
 
-    private $inheritance = [ Event::class ];
+    private $inheritance = [Event::class];
 
     public function propertiesProvider()
     {
@@ -43,7 +43,7 @@ 
             ]],
             ['file', $file],
             ['file', [
-                'pre' => function() use ($file) { $this->target->setParam('file', $file);},
+                'pre' => function() use ($file) { $this->target->setParam('file', $file); },
                 'ignore_setter' => true,
                 'value' => $file
             ]]

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

Indentation +2 added lines, -2 removed lines

@@ -110,8 +110,8 @@
         $results->push($result1);
         $this->ajaxEventManagerMock->expects($this->once())->method('getEvent')->with('test', $this->target)->willReturn(new AjaxEvent());
         $this->ajaxEventManagerMock->expects($this->once())->method('triggerEventUntil')
-                                   ->with($this->isInstanceOf('\Closure'), $this->isInstanceOf(AjaxEvent::class))
-                                   ->willReturn($results);
+                                    ->with($this->isInstanceOf('\Closure'), $this->isInstanceOf(AjaxEvent::class))
+                                    ->willReturn($results);
 
         $response = $this->target->onRoute($event);
 

Spacing +4 added lines, -4 removed lines

@@ -40,13 +40,13 @@ 
         '@testConstruction' => false,
         '@testInheritance' => '@testUsesTraits',
         '@testDefaultAttributes' => '@testUsesTraits',
-        '@testUsesTraits' => [ 'as_reflection' => true ],
+        '@testUsesTraits' => ['as_reflection' => true],
     ];
 
-    private $traits = [ ListenerAggregateTrait::class ];
+    private $traits = [ListenerAggregateTrait::class];
 
     private $attributes = [
-        'events' => [ [ MvcEvent::EVENT_ROUTE, 'onRoute', 100 ] ],
+        'events' => [[MvcEvent::EVENT_ROUTE, 'onRoute', 100]],
     ];
 
     private function getTargetArgs()
@@ -54,7 +54,7 @@ 
         $this->ajaxEventManagerMock = $this->getMockBuilder(EventManager::class)->disableOriginalConstructor()
             ->setMethods(['getEvent', 'triggerEventUntil'])->getMock();
 
-        return [ $this->ajaxEventManagerMock ];
+        return [$this->ajaxEventManagerMock];
     }
 
     public function testConstruction()

module/Core/test/CoreTest/Acl/FileAccessAssertionTest.php

Spacing +5 added lines, -5 removed lines

@@ -57,17 +57,17 @@ 
         $failMessage = 'Assert should return false when role is not a user or resource is not file';
 
         $this->assertFalse(
-            $target->assert($acl,$roleInterface,$resourceInterface),
+            $target->assert($acl, $roleInterface, $resourceInterface),
             $failMessage
         );
 
         $this->assertFalse(
-            $target->assert($acl,$userRole,$resourceInterface),
+            $target->assert($acl, $userRole, $resourceInterface),
             $failMessage
         );
 
         $this->assertFalse(
-            $this->target->assert($acl,$roleInterface,$file),
+            $this->target->assert($acl, $roleInterface, $file),
             $failMessage
         );
     }
@@ -89,11 +89,11 @@ 
 
         $permissions->expects($this->once())
             ->method('isGranted')
-            ->with($role,PermissionsInterface::PERMISSION_VIEW)
+            ->with($role, PermissionsInterface::PERMISSION_VIEW)
             ->willReturn(true)
         ;
         $this->assertTrue(
-            $target->assert($acl,$role,$file,PermissionsInterface::PERMISSION_VIEW)
+            $target->assert($acl, $role, $file, PermissionsInterface::PERMISSION_VIEW)
         );
     }
 }

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

Indentation +9 added lines, -9 removed lines

@@ -76,7 +76,7 @@ 
     {
         $entry = 'value';
         $count = count($this->wrappedCollection);
-		$this->identityWrapper->add($entry);
+        $this->identityWrapper->add($entry);
         $this->assertSame($count + 1, count($this->identityWrapper));
         $this->assertSame($count + 1, count($this->wrappedCollection));
         $this->assertTrue($this->identityWrapper->contains($entry));
@@ -181,7 +181,7 @@ 
     public function testGetValues()
     {
         $expected = array_values($this->entries);
-		$this->assertSame($expected, $this->identityWrapper->getValues());
+        $this->assertSame($expected, $this->identityWrapper->getValues());
         $this->assertSame($expected, $this->wrappedCollection->getValues());
     }
 
@@ -234,8 +234,8 @@ 
     public function testFirst()
     {
         $expected = reset($this->entries);
-		$this->assertSame($expected, $this->identityWrapper->first());
-		$this->assertSame($expected, $this->wrappedCollection->first());
+        $this->assertSame($expected, $this->identityWrapper->first());
+        $this->assertSame($expected, $this->wrappedCollection->first());
     }
 
     /**
@@ -244,8 +244,8 @@ 
     public function testLast()
     {
         $expected = end($this->entries);
-		$this->assertSame($expected, $this->identityWrapper->last());
-		$this->assertSame($expected, $this->wrappedCollection->last());
+        $this->assertSame($expected, $this->identityWrapper->last());
+        $this->assertSame($expected, $this->wrappedCollection->last());
     }
 
     /**
@@ -324,9 +324,9 @@ 
             return $value === $searchValue;
         };
         $filtered = $this->identityWrapper->filter($filter);
-		$this->assertInstanceOf(IdentityWrapper::class, $filtered);
-		$this->assertNotSame($this->identityWrapper, $filtered);
-		$this->assertSame(array_filter($this->entries, $filter), $filtered->toArray());
+        $this->assertInstanceOf(IdentityWrapper::class, $filtered);
+        $this->assertNotSame($this->identityWrapper, $filtered);
+        $this->assertSame(array_filter($this->entries, $filter), $filtered->toArray());
     }
 
     /**

Spacing +16 added lines, -16 removed lines

@@ -32,7 +32,7 @@ 
     
     public function setUp()
     {
-        $getEntryMock = function ($id)
+        $getEntryMock = function($id)
         {
             $entry = $this->getMockBuilder(\stdClass::class)
                 ->setMethods(['getId'])
@@ -290,11 +290,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;
         }));
     }
@@ -305,11 +305,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;
         }));
     }
@@ -320,7 +320,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);
@@ -335,11 +335,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);
         }));
     }
@@ -350,11 +350,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);
         }));
     }
@@ -364,7 +364,7 @@ 
      */
     public function testMapValidOperation()
     {
-        $map = function ($value) {
+        $map = function($value) {
             $value->random = rand();
             return $value;
         };
@@ -380,7 +380,7 @@ 
      */
     public function testMapInvalidOperation()
     {
-        $map = function ($value) {
+        $map = function($value) {
             return sprintf('id: %s', $value->getId());
         };
         $this->identityWrapper->map($map);
@@ -392,7 +392,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);
@@ -411,7 +411,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);
@@ -553,8 +553,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));