cross-solution / YAWIK

module/Core/test/CoreTest/Form/Tree/AddItemFieldsetTest.php

Spacing +3 added lines, -3 removed lines

@@ -37,7 +37,7 @@ 
         '@testInitializesItself' => 'setupMock'
     ];
 
-    private $inheritance = [ Fieldset::class, ViewPartialProviderInterface::class, InputFilterProviderInterface::class];
+    private $inheritance = [Fieldset::class, ViewPartialProviderInterface::class, InputFilterProviderInterface::class];
 
     private $traits = [ViewPartialProviderTrait::class];
 
@@ -77,13 +77,13 @@ 
             'name' => [
                 'required' => true,
                 'filters' => [
-                    [ 'name' => 'StringTrim' ],
+                    ['name' => 'StringTrim'],
                 ],
             ],
             'value' => [
                 'required' => false,
                 'filters' => [
-                    [ 'name' => 'StringTrim' ],
+                    ['name' => 'StringTrim'],
                 ],
             ],
         ];

Indentation +9 added lines, -9 removed lines

@@ -51,19 +51,19 @@
     {
         $mock = $this->getMockBuilder(AddItemFieldset::class)
             ->setMethods(['setObject', 'add'])
-             ->disableOriginalConstructor()->getMock();
+                ->disableOriginalConstructor()->getMock();
 
         $mock->expects($this->once())->method('setObject')->with($this->isInstanceOf(\ArrayObject::class));
 
         $mock->expects($this->exactly(6))->method('add')
-             ->withConsecutive(
-                 [['name' => 'id', 'type' => 'Hidden']],
-                 [['name' => 'current', 'type' => 'Hidden']],
-                 [['name' => 'do', 'type' => 'Hidden']],
-                 [['name' => 'name', 'type' => 'Text', 'options' => ['label' => 'Name'], 'attributes' => ['required' => 'required']]],
-                 [['name' => 'value', 'type' => 'Text', 'options' => ['label' => 'Value']]],
-                 [['name' => 'priority', 'type' => 'Text']]
-             );
+                ->withConsecutive(
+                    [['name' => 'id', 'type' => 'Hidden']],
+                    [['name' => 'current', 'type' => 'Hidden']],
+                    [['name' => 'do', 'type' => 'Hidden']],
+                    [['name' => 'name', 'type' => 'Text', 'options' => ['label' => 'Name'], 'attributes' => ['required' => 'required']]],
+                    [['name' => 'value', 'type' => 'Text', 'options' => ['label' => 'Value']]],
+                    [['name' => 'priority', 'type' => 'Text']]
+                );
 
         return $mock;
     }

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

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/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/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/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/EventManager/EventManager/BaseTest.php

Spacing +16 added lines, -16 removed lines

@@ -31,7 +31,7 @@ 
      */
     protected $target = '\Core\EventManager\EventManager';
 
-    protected $inheritance = [ '\Zend\EventManager\EventManager', '\Core\EventManager\EventProviderInterface' ];
+    protected $inheritance = ['\Zend\EventManager\EventManager', '\Core\EventManager\EventProviderInterface'];
 
     public function testSetEventPrototype()
     {
@@ -44,23 +44,23 @@ 
     {
         $testTarget = new \stdClass();
         $testTarget2 = new \stdClass();
-        $testParams1 = [ 'params1' => 'value1' ];
-        $testParams2 = [ 'name' => 'test6', 'param2' => 'value2' ];
-        $testParams3 = [ 'name' => 'test8', 'target' => $testTarget, 'param3' => 'value3' ];
-        $expect2     = [ 'param2' => 'value2' ];
-        $expect3     = [ 'param3' => 'value3' ];
+        $testParams1 = ['params1' => 'value1'];
+        $testParams2 = ['name' => 'test6', 'param2' => 'value2'];
+        $testParams3 = ['name' => 'test8', 'target' => $testTarget, 'param3' => 'value3'];
+        $expect2     = ['param2' => 'value2'];
+        $expect3     = ['param3' => 'value3'];
 
         return [
-            [ 'test1', null, [], [ 'name' => 'test1', 'target' => null, 'params' => []]],
-            [ 'test2', $testTarget, [], [ 'name' => 'test2', 'target' => $testTarget, 'params' => []]],
-            [ 'test3', null, $testParams1, [ 'name' => 'test3', 'target' => null, 'params' => $testParams1]],
-            'setParamsAsFirstArg' => [ $testParams1, null, ['ignoreme' => 'yes' ], [ 'name' => null, 'target' => null, 'params' => $testParams1 ]],
-            'setParamsAsSecondArg' => [ 'test5', $testParams1, ['ignoreme' => 'yes' ], ['name' => 'test5', 'target' => null, 'params' => $testParams1 ]],
-            'setNameInParamsAsFirstArg' => [ $testParams2, null, [], [ 'name' => 'test6', 'target' => null, 'params' => $expect2]],
-            'setNameInParamsAsFirstArgAndTarget' => [ $testParams2, $testTarget, ['ignoreme' => 'yes'], [ 'name' => 'test6', 'target' => $testTarget, 'params' => $expect2]],
-            'setNameAndTargetInParams' => [ $testParams3, null, [], [ 'name' => 'test8', 'target' => $testTarget, 'params' => $expect3]],
-            [ 'test9', $testTarget2, $testParams3, ['name' => 'test9', 'target' => $testTarget2, 'params' => $testParams3 ]],
-            [ null, $testParams2, [], ['name' => 'test6', 'target' => null, 'params' => $expect2 ]],
+            ['test1', null, [], ['name' => 'test1', 'target' => null, 'params' => []]],
+            ['test2', $testTarget, [], ['name' => 'test2', 'target' => $testTarget, 'params' => []]],
+            ['test3', null, $testParams1, ['name' => 'test3', 'target' => null, 'params' => $testParams1]],
+            'setParamsAsFirstArg' => [$testParams1, null, ['ignoreme' => 'yes'], ['name' => null, 'target' => null, 'params' => $testParams1]],
+            'setParamsAsSecondArg' => ['test5', $testParams1, ['ignoreme' => 'yes'], ['name' => 'test5', 'target' => null, 'params' => $testParams1]],
+            'setNameInParamsAsFirstArg' => [$testParams2, null, [], ['name' => 'test6', 'target' => null, 'params' => $expect2]],
+            'setNameInParamsAsFirstArgAndTarget' => [$testParams2, $testTarget, ['ignoreme' => 'yes'], ['name' => 'test6', 'target' => $testTarget, 'params' => $expect2]],
+            'setNameAndTargetInParams' => [$testParams3, null, [], ['name' => 'test8', 'target' => $testTarget, 'params' => $expect3]],
+            ['test9', $testTarget2, $testParams3, ['name' => 'test9', 'target' => $testTarget2, 'params' => $testParams3]],
+            [null, $testParams2, [], ['name' => 'test6', 'target' => null, 'params' => $expect2]],
 
         ];
     }

module/Core/test/CoreTestUtils/TestCase/SimpleSetterAndGetterTrait.php

Spacing +3 added lines, -3 removed lines

@@ -21,14 +21,14 @@
         if (!is_object($object)) {
             throw new \InvalidArgumentException('Data Provider first argument should be an object');
         }
-        $setter = 'set' . $propertyName;
-        $getter = 'get' . $propertyName;
+        $setter = 'set'.$propertyName;
+        $getter = 'get'.$propertyName;
 
         call_user_func(array($object, $setter), $propertyValue);
         $this->assertSame(
             $propertyValue,
             call_user_func(array($object, $getter)),
-            '::' . $setter . '() and ::' . $getter . '() should executed properly'
+            '::'.$setter.'() and ::'.$getter.'() should executed properly'
         );
     }
 

module/Core/test/CoreTestUtils/TestCase/TestUsesTraitsTrait.php

Indentation +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     {
         if (!property_exists($this, 'traits') || !property_exists($this, 'target')) {
             throw new \PHPUnit_Framework_Exception(self::class . ': ' . static::class
-                                                   . ' must define the properties $target and $traits.');
+                                                    . ' must define the properties $target and $traits.');
         }
 
         $this->assertUsesTraits($this->traits, $this->target);

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
     public function testUsesTraits()
     {
         if (!property_exists($this, 'traits') || !property_exists($this, 'target')) {
-            throw new \PHPUnit_Framework_Exception(self::class . ': ' . static::class
+            throw new \PHPUnit_Framework_Exception(self::class.': '.static::class
                                                    . ' must define the properties $target and $traits.');
         }
 

module/Core/test/CoreTestUtils/TestCase/TestInheritanceTrait.php

Spacing +4 added lines, -4 removed lines

@@ -40,18 +40,18 @@
      */
     public function testInheritance()
     {
-        $errTmpl = __TRAIT__ . ': ' . get_class($this);
+        $errTmpl = __TRAIT__.': '.get_class($this);
 
         if (!property_exists($this, 'inheritance') || !property_exists($this, 'target')) {
-            throw new \PHPUnit_Framework_Exception($errTmpl . ' must define the properties "$inheritance" and "$target"');
+            throw new \PHPUnit_Framework_Exception($errTmpl.' must define the properties "$inheritance" and "$target"');
         }
 
         if (!is_array($this->inheritance)) {
-            throw new \PHPUnit_Framework_Exception($errTmpl . ': Property $inheritance must be an array');
+            throw new \PHPUnit_Framework_Exception($errTmpl.': Property $inheritance must be an array');
         }
 
         if (!is_object($this->target)) {
-            throw new \PHPUnit_Framework_Exception($errTmpl . ': Property $target must be an object');
+            throw new \PHPUnit_Framework_Exception($errTmpl.': Property $target must be an object');
         }
 
         $this->assertInheritance($this->inheritance, $this->target);