cross-solution / YAWIK

module/Core/test/CoreTest/Entity/EntityTraitTest.php

Indentation +3 added lines, -3 removed lines

@@ -120,9 +120,9 @@
     public function testHasPropertyReturnsExpectedResults($target, $expects)
     {
         $modes = [EntityInterface::PROPERTY_FACILE,
-                  EntityInterface::PROPERTY_GETTER,
-                  EntityInterface::PROPERTY_SETTER,
-                  EntityInterface::PROPERTY_STRICT];
+                    EntityInterface::PROPERTY_GETTER,
+                    EntityInterface::PROPERTY_SETTER,
+                    EntityInterface::PROPERTY_STRICT];
 
         foreach ($modes  as $mode) {
 

Spacing +19 added lines, -19 removed lines

@@ -34,7 +34,7 @@ 
     }
 
 
-    public function setUp(){
+    public function setUp() {
         $this->target = new TraitEntity();
     }
 
@@ -45,19 +45,19 @@ 
         $countable = new \ArrayObject(['not', 'empty']);
 
         return [
-            [ null, false ],
-            [ '', false ],
-            [ 'something', true],
-            [ 0, false ],
-            [ 1, true ],
-            [ false, false ],
-            [ true, true ],
-            [ [], false ],
-            [ ['not', 'empty'], true ],
-            [ new \stdClass(), true ],
-            [ $emptyCountable, false ],
-            [ $countable, true ],
-            [ self::$fileResource, true ],
+            [null, false],
+            ['', false],
+            ['something', true],
+            [0, false],
+            [1, true],
+            [false, false],
+            [true, true],
+            [[], false],
+            [['not', 'empty'], true],
+            [new \stdClass(), true],
+            [$emptyCountable, false],
+            [$countable, true],
+            [self::$fileResource, true],
 
         ];
     }
@@ -104,10 +104,10 @@ 
         $setterTarget = new HasPropertyAndSetterTestTraitEntity();
         $strictTarget = new HasPropertyAndGetterAndSetterTestTraitEntity();
         return [
-            [ $facileTarget, [true, false ,false, false]],
-            [ $getterTarget, [true, true, false, false]],
-            [ $setterTarget, [true, false, true, false]],
-            [ $strictTarget, [true, true, true, true]],
+            [$facileTarget, [true, false, false, false]],
+            [$getterTarget, [true, true, false, false]],
+            [$setterTarget, [true, false, true, false]],
+            [$strictTarget, [true, true, true, true]],
         ];
     }
 
@@ -128,7 +128,7 @@ 
 
             $actual = $target->hasProperty('testProperty', $mode);
             $expect = array_shift($expects);
-            $assert = 'assert' . ($expect ? 'true' : 'false');
+            $assert = 'assert'.($expect ? 'true' : 'false');
 
             $this->$assert($actual);
         }

module/Core/test/CoreTest/Entity/StatusAwareEntityTraitTest.php

Spacing +7 added lines, -7 removed lines

@@ -34,30 +34,30 @@
 
         return [
 
-            [ 'status', [
+            ['status', [
                 'value' => new Status(Status::STATE_ONE),
                 'target' => new StatusAwareEntity(),
             ]],
 
-            [ 'status', [
+            ['status', [
                 'target' => new ProvideEntityStatusAwareEntity(),
                 'default' => true,
-                'default_assert' => function ($g, $return) { $this->assertEquals(new Status(Status::STATE_ONE), $return); },
+                'default_assert' => function($g, $return) { $this->assertEquals(new Status(Status::STATE_ONE), $return); },
                 'value' => Status::STATE_TWO,
                 'getter_assert' => function($g, $return, $v) { $this->assertEquals(new Status(Status::STATE_TWO), $return); },
             ]],
 
-            [ 'status', [
+            ['status', [
                 'target' => new StatusAwareEntity(),
                 'value' => 'state',
-                'setter_exception' => [ 'RuntimeException', 'No status entity' ]
+                'setter_exception' => ['RuntimeException', 'No status entity']
             ]],
 
-            [ 'status', [
+            ['status', [
                 'target' => new StatusAwareEntity(),
                 'value' => new Status(Status::STATE_TWO),
                 'getter_method' => 'has*',
-                'getter_args' => [ Status::STATE_TWO ],
+                'getter_args' => [Status::STATE_TWO],
                 'expect' => true,
             ]],
         ];

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

Indentation +12 added lines, -12 removed lines

@@ -124,18 +124,18 @@
     private function getPreparedTarget()
     {
         return new FieldsetCustomizationOptions([
-           'fields' => [
-               'field' => [
-                   'attributes' => ['attr' => 'attrVal'],
-                   'options' => ['opt1' => 'optVal' ],
-                   'label' => 'label',
-                   'required' => true,
-                   'type' => 'ElementType',
-                   'priority' => 12,
-                   'flags' => ['priority' => 10],
-                   'input_filter' => ['test' => 'works'],
-               ],
-           ],
+            'fields' => [
+                'field' => [
+                    'attributes' => ['attr' => 'attrVal'],
+                    'options' => ['opt1' => 'optVal' ],
+                    'label' => 'label',
+                    'required' => true,
+                    'type' => 'ElementType',
+                    'priority' => 12,
+                    'flags' => ['priority' => 10],
+                    'input_filter' => ['test' => 'works'],
+                ],
+            ],
         ]);
     }
 

Spacing +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
         '@testCopyArrayValues' => Fcot_Mock::class,
     ];
 
-    private $inheritance = [ AbstractOptions::class ];
+    private $inheritance = [AbstractOptions::class];
 
     public function propertiesProvider()
     {
@@ -127,7 +127,7 @@ 
            'fields' => [
                'field' => [
                    'attributes' => ['attr' => 'attrVal'],
-                   'options' => ['opt1' => 'optVal' ],
+                   'options' => ['opt1' => 'optVal'],
                    'label' => 'label',
                    'required' => true,
                    'type' => 'ElementType',

module/Core/test/CoreTest/Options/Exception/MissingOptionExceptionTest.php

Spacing +7 added lines, -7 removed lines

@@ -29,7 +29,7 @@ 
 
     protected $target = [
         'class' => '\Core\Options\Exception\MissingOptionException',
-        'args' => [ 'testOptionKey', 'TestTarget' ]
+        'args' => ['testOptionKey', 'TestTarget']
     ];
 
     protected $inheritance = [
@@ -44,29 +44,29 @@ 
         $target2 = new MissingOptionException('-', new \stdClass());
 
         return [
-            [ 'OptionKey', [
+            ['OptionKey', [
                 'value' => 'testOptionKey',
                 'ignore_setter' => true,
             ]],
-            [ 'target', [
+            ['target', [
                 'value' => 'TestTarget',
                 'ignore_setter' => true,
             ]],
-            [ 'targetFQCN', [
+            ['targetFQCN', [
                 'value' => 'stdClass',
                 'target' => $target2,
                 'ignore_setter' => true,
             ]],
-            [ 'targetFQCN', [
+            ['targetFQCN', [
                 'value' => 'TestTarget',
                 'ignore_setter' => true,
             ]],
-            [ 'previous', [
+            ['previous', [
                 'value' => $ex,
                 'target' => $target,
                 'ignore_setter' => true,
             ]],
-            [ 'message', [
+            ['message', [
                 'value' => 'Missing value for option "testOptionKey" in "TestTarget"',
                 'ignore_setter' => true,
             ]]

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

Indentation +2 added lines, -2 removed lines

@@ -104,8 +104,8 @@
     public function testGetSiteNameThrowsExceptionIfNotSet()
     {
         $this->setExpectedException(
-             '\Core\Options\Exception\MissingOptionException',
-             'Missing value for option "siteName"'
+                '\Core\Options\Exception\MissingOptionException',
+                'Missing value for option "siteName"'
         );
 
         $this->target->setSiteName('');

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

Spacing +2 added lines, -2 removed lines

@@ -30,12 +30,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/Form/FormTest.php

Indentation +3 added lines, -3 removed lines

@@ -84,7 +84,7 @@ 
     }
 
     public function testDisableElement() {
-       /* @todo */
+        /* @todo */
     }
 
     /**
@@ -118,9 +118,9 @@ 
             foreach ($classes as $class) {
                 $this->target->addClass($class);
             }
-         }elseif (is_string($classes)){
+            }elseif (is_string($classes)){
             $this->target->addClass($classes);
-         }
+            }
 
         $x=$this->target->getAttribute('class');
         $this->assertEquals($x, $expected);

Spacing +15 added lines, -15 removed lines

@@ -22,13 +22,13 @@ 
      */
     protected $target;
 
-    public function setUp(){
-        $fs = new Fieldset('test',['type'=>"text"]);
-        $fs->setAttribute('class','myclass1 myclass2');
+    public function setUp() {
+        $fs = new Fieldset('test', ['type'=>"text"]);
+        $fs->setAttribute('class', 'myclass1 myclass2');
         $fs->setName('myField');
         $target = new Form();
         $target->add($fs);
-        $this->target=$target;
+        $this->target = $target;
     }
 
     public function testConstructor()
@@ -44,10 +44,10 @@ 
     public function testSetGetIsDescriptionEnabled() {
         $input = true;
         $this->target->setIsDescriptionsEnabled($input);
-        $this->assertEquals($this->target->isDescriptionsEnabled(),$input);
+        $this->assertEquals($this->target->isDescriptionsEnabled(), $input);
         $input = false;
         $this->target->setIsDescriptionsEnabled($input);
-        $this->assertEquals($this->target->isDescriptionsEnabled(),$input);
+        $this->assertEquals($this->target->isDescriptionsEnabled(), $input);
     }
 
     public function testSetDescription() {
@@ -59,27 +59,27 @@ 
 
     public function testSetDescriptionWithAdditionalParams() {
         $input = "this is my description";
-        $this->target->setDescription($input,['p1','p2']);
+        $this->target->setDescription($input, ['p1', 'p2']);
         $this->assertAttributeSame(['description' => $input,
-                                    'description_params' => ['p1','p2']], 'options', $this->target);
+                                    'description_params' => ['p1', 'p2']], 'options', $this->target);
     }
 
-    public function testSetGetIsDisableCapable(){
+    public function testSetGetIsDisableCapable() {
         $input = true;
         $this->target->setIsDisableCapable($input);
         $this->assertAttributeSame(['is_disable_capable' => $input], 'options', $this->target);
-        $this->assertEquals($this->target->isDisableCapable(),$input);
+        $this->assertEquals($this->target->isDisableCapable(), $input);
     }
 
     /**
      * @covers Core\Form\Form::setIsDisableElementsCapable
      * @covers Core\Form\Form::isDisableElementsCapable
      */
-    public function testSetGetIsDisableElementsCapable(){
+    public function testSetGetIsDisableElementsCapable() {
         $input = true;
         $this->target->setIsDisableElementsCapable($input);
         $this->assertAttributeSame(['is_disable_elements_capable' => $input], 'options', $this->target);
-        $this->assertEquals($this->target->isDisableElementsCapable(),$input);
+        $this->assertEquals($this->target->isDisableElementsCapable(), $input);
 
     }
 
@@ -113,16 +113,16 @@ 
      * @param string $class     the expected name for the status
      * @param string $expected  the expected order for the status
      */
-    public function testAddClass($classes,$expected) {
+    public function testAddClass($classes, $expected) {
         if (is_array($classes)) {
             foreach ($classes as $class) {
                 $this->target->addClass($class);
             }
-         }elseif (is_string($classes)){
+         }elseif (is_string($classes)) {
             $this->target->addClass($classes);
          }
 
-        $x=$this->target->getAttribute('class');
+        $x = $this->target->getAttribute('class');
         $this->assertEquals($x, $expected);
     }
 

Braces +1 added lines, -1 removed lines

@@ -118,7 +118,7 @@
             foreach ($classes as $class) {
                 $this->target->addClass($class);
             }
-         }elseif (is_string($classes)){
+         } elseif (is_string($classes)){
             $this->target->addClass($classes);
          }
 

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

Spacing +28 added lines, -28 removed lines

@@ -27,7 +27,7 @@ 
      */
     protected $target;
 
-    public function setUp(){
+    public function setUp() {
         $this->target = new Container();
     }
 
@@ -40,19 +40,19 @@ 
         $this->assertInstanceOf(\Core\Form\FormParentInterface::class, $this->target);
     }
 
-    public function testSetFormElementManager(){
+    public function testSetFormElementManager() {
         $input = new ServiceManager();
-        $this->assertSame($this->target,$this->target->setFormElementManager($input));
+        $this->assertSame($this->target, $this->target->setFormElementManager($input));
     }
 
     /**
      * @dataProvider provideBool
      * @param $input
      */
-    public function testSetGetIsDisableCapable($input){
+    public function testSetGetIsDisableCapable($input) {
         $this->target->setIsDisableCapable($input);
-        $this->assertSame($this->target->isDisableCapable(),$input);
-        $this->assertSame($this->target->getOption('is_disable_capable'),$input);
+        $this->assertSame($this->target->isDisableCapable(), $input);
+        $this->assertSame($this->target->getOption('is_disable_capable'), $input);
 
     }
 
@@ -60,13 +60,13 @@ 
      * @dataProvider provideBool
      * @param $input
      */
-    public function testSetGetIsDisableElementCapable($input){
+    public function testSetGetIsDisableElementCapable($input) {
         $this->target->setIsDisableElementsCapable($input);
-        $this->assertSame($this->target->isDisableElementsCapable(),$input);
-        $this->assertSame($this->target->getOption('is_disable_elements_capable'),$input);
+        $this->assertSame($this->target->isDisableElementsCapable(), $input);
+        $this->assertSame($this->target->getOption('is_disable_elements_capable'), $input);
     }
 
-    public function provideBool (){
+    public function provideBool() {
         return [
             [true, true],
             [false, false],
@@ -77,11 +77,11 @@ 
      * @covers Core\Form\Container::setParent()
      * @covers Core\Form\Container::getParent()
      */
-    public function testSetGetParent(){
+    public function testSetGetParent() {
         $input = "improve this";
         $this->target->setParent($input);
-        $this->assertSame($this->target->getParent(),$input);
-        $this->assertSame($this->target->hasParent(),true);
+        $this->assertSame($this->target->getParent(), $input);
+        $this->assertSame($this->target->hasParent(), true);
     }
 
 
@@ -98,17 +98,17 @@ 
      * @param $enabled
      * @dataProvider provideSetFormData
      */
-    public function testSetForm($key,$spec,$enabled, $expected){
+    public function testSetForm($key, $spec, $enabled, $expected) {
         /* @var $target Container */
         $target = $this->getMockBuilder('\Core\Form\Container')
             ->disableOriginalConstructor()
             ->setMethods(array('setParent'))
             ->getMock();
-        $target->setForm($key,$spec,$enabled);
+        $target->setForm($key, $spec, $enabled);
         if ($expected) {
-            $this->assertAttributeContains($key,'activeForms',$target);
-        }else{
-            $this->assertAttributeNotContains($key,'activeForms',$target);
+            $this->assertAttributeContains($key, 'activeForms', $target);
+        } else {
+            $this->assertAttributeNotContains($key, 'activeForms', $target);
         }
         $this->assertAttributeSame(
             [$key => [
@@ -124,12 +124,12 @@ 
 
     public function provideSetFormData() {
         return [
-            ['fieldname','input',true,true ],
-            ['fieldname', 'foobar', false, false ],
+            ['fieldname', 'input', true, true],
+            ['fieldname', 'foobar', false, false],
         ];
     }
 
-    public function testSetFormWithArray(){
+    public function testSetFormWithArray() {
         $key1 = "key1";
         $key2 = "key2";
         $spec = [
@@ -144,7 +144,7 @@ 
             ->disableOriginalConstructor()
             ->setMethods(array('setParent'))
             ->getMock();
-        $target->setForm($key1,$spec,true);
+        $target->setForm($key1, $spec, true);
 
         $this->assertAttributeSame(
 
@@ -164,7 +164,7 @@ 
     /**
      * @dataProvider provideEntities
      */
-    public function testSetGetEntity($input, $key, $expected )
+    public function testSetGetEntity($input, $key, $expected)
     {
         $this->target->setEntity($input, $key, $expected);
         $this->assertEquals($expected, $this->target->getEntity($key));
@@ -180,10 +180,10 @@ 
 
 
         return [
-            [$e['application'], null , $e['application'] ],
-            [$e['job'], 'job' , $e['job'] ],
-            [$e['user'], null , $e['user'] ],
-            [$e['organization'], null , $e['organization'] ],
+            [$e['application'], null, $e['application']],
+            [$e['job'], 'job', $e['job']],
+            [$e['user'], null, $e['user']],
+            [$e['organization'], null, $e['organization']],
         ];
     }
     
@@ -201,7 +201,7 @@ 
         $childName = 'child';
         $child = new Container($childName);
         $this->target->setForm($childName, $child);
-        $this->assertSame($childName .'.' . $name, $child->formatAction($name));
+        $this->assertSame($childName.'.'.$name, $child->formatAction($name));
     }
     
     public function testGetActionFor()

Braces +1 added lines, -1 removed lines

@@ -107,7 +107,7 @@
         $target->setForm($key,$spec,$enabled);
         if ($expected) {
             $this->assertAttributeContains($key,'activeForms',$target);
-        }else{
+        } else{
             $this->assertAttributeNotContains($key,'activeForms',$target);
         }
         $this->assertAttributeSame(

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

Indentation +3 added lines, -3 removed lines

@@ -40,8 +40,8 @@ 
      * @param string $expected  render mode
      */
     public function testSetGetRenderModer($input,$expected) {
-       $this->target->setRenderMode($input);
-       $this->assertSame($this->target->getRenderMode(),$expected);
+        $this->target->setRenderMode($input);
+        $this->assertSame($this->target->getRenderMode(),$expected);
     }
 
     public function providerRenderMode()
@@ -75,7 +75,7 @@ 
     }
 
     public function testIsValid(){
-       /* @todo */
+        /* @todo */
     }
 
     /**

Spacing +12 added lines, -12 removed lines

@@ -21,7 +21,7 @@ 
      */
     protected $target;
 
-    public function setUp(){
+    public function setUp() {
         $this->target = new SummaryForm();
     }
 
@@ -39,16 +39,16 @@ 
      * @param string $input     render mode
      * @param string $expected  render mode
      */
-    public function testSetGetRenderModer($input,$expected) {
+    public function testSetGetRenderModer($input, $expected) {
        $this->target->setRenderMode($input);
-       $this->assertSame($this->target->getRenderMode(),$expected);
+       $this->assertSame($this->target->getRenderMode(), $expected);
     }
 
     public function providerRenderMode()
     {
         return [
-            [SummaryFormInterface::RENDER_FORM,SummaryFormInterface::RENDER_FORM],
-            [SummaryFormInterface::RENDER_SUMMARY,SummaryFormInterface::RENDER_SUMMARY],
+            [SummaryFormInterface::RENDER_FORM, SummaryFormInterface::RENDER_FORM],
+            [SummaryFormInterface::RENDER_SUMMARY, SummaryFormInterface::RENDER_SUMMARY],
             [SummaryFormInterface::RENDER_ALL, SummaryFormInterface::RENDER_ALL],
         ];
     }
@@ -61,20 +61,20 @@ 
      * @param string $input     display mode
      * @param string $expected  display mode
      */
-    public function testSetGetDisplayModer($input,$expected) {
+    public function testSetGetDisplayModer($input, $expected) {
         $this->target->setDisplayMode($input);
-        $this->assertSame($this->target->getDisplayMode(),$expected);
+        $this->assertSame($this->target->getDisplayMode(), $expected);
     }
 
     public function providerDisplayMode()
     {
         return [
-            [SummaryFormInterface::DISPLAY_FORM,SummaryFormInterface::DISPLAY_FORM],
-            [SummaryFormInterface::DISPLAY_SUMMARY,SummaryFormInterface::DISPLAY_SUMMARY],
+            [SummaryFormInterface::DISPLAY_FORM, SummaryFormInterface::DISPLAY_FORM],
+            [SummaryFormInterface::DISPLAY_SUMMARY, SummaryFormInterface::DISPLAY_SUMMARY],
         ];
     }
 
-    public function testIsValid(){
+    public function testIsValid() {
        /* @todo */
     }
 
@@ -86,7 +86,7 @@ 
      * @param string $expected  display mode
      */
     public function testSetOptions($input, $expected) {
-        $this->target->setOptions(['display_mode' => $input ]);
-        $this->assertSame($this->target->getDisplayMode(),$expected);
+        $this->target->setOptions(['display_mode' => $input]);
+        $this->assertSame($this->target->getDisplayMode(), $expected);
     }
 }
\ No newline at end of file