cross-solution / YAWIK

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

Spacing +2 added lines, -2 removed lines

@@ -206,8 +206,8 @@
     public function provideTestCreateAttachedEntityData()
     {
         return [
-            ['EntityClass' ],
-            ['EntityClass', 'testkey' ],
+            ['EntityClass'],
+            ['EntityClass', 'testkey'],
             ['EntityClass', ['param' => 'value']],
             ['EntityClass', ['param' => 'value'], 'testkey'],
         ];

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

Spacing +8 added lines, -8 removed lines

@@ -183,14 +183,14 @@
     public function provideRolePermissionsData()
     {
         return [
-            [ 'user', 'user', false, true ],
-            [ 'recruiter', 'user', false, false ],
-            [ 'all', 'irrelevant', false, true ],
-            [ 'user', 'user', true, true ],
-            [ 'recruiter', 'user', true, false],
-            [ 'all', 'irrelevant', true, true ],
-            [ 'testUser', null, true, true ],
-            [ 'anonymous', null, true, false ],
+            ['user', 'user', false, true],
+            ['recruiter', 'user', false, false],
+            ['all', 'irrelevant', false, true],
+            ['user', 'user', true, true],
+            ['recruiter', 'user', true, false],
+            ['all', 'irrelevant', true, true],
+            ['testUser', null, true, true],
+            ['anonymous', null, true, false],
         ];
     }
 

Indentation +44 added lines, -44 removed lines

@@ -64,7 +64,7 @@ 
     public function testCloneCreatesNewResourceCollection()
     {
         $resource = $this->getMockBuilder('\Core\Entity\PermissionsResourceInterface')
-                         ->getMockForAbstractClass();
+                            ->getMockForAbstractClass();
 
         $target1 = new Permissions();
         $target1->grant($resource, Permissions::PERMISSION_ALL);
@@ -121,23 +121,23 @@ 
         $allParams    = array($user, PermissionsInterface::PERMISSION_ALL);
 
         $target = $this->getMockBuilder('\Core\Entity\Permissions')->disableOriginalConstructor()
-                       ->setMethods(array('isGranted', 'grant', 'revoke'))->getMock();
+                        ->setMethods(array('isGranted', 'grant', 'revoke'))->getMock();
 
 
         $target->expects($this->exactly(4))
-               ->method('isGranted')
-               ->withConsecutive($viewParams, $changeParams, $noneParams, $allParams)
-               ->willReturn(null);
+                ->method('isGranted')
+                ->withConsecutive($viewParams, $changeParams, $noneParams, $allParams)
+                ->willReturn(null);
 
         $target->expects($this->exactly(4))
-               ->method('grant')
-               ->withConsecutive($viewParams, $changeParams, $noneParams, $allParams)
-               ->willReturn(null);
+                ->method('grant')
+                ->withConsecutive($viewParams, $changeParams, $noneParams, $allParams)
+                ->willReturn(null);
 
         $target->expects($this->exactly(4))
-               ->method('revoke')
-               ->withConsecutive($viewParams, $changeParams, $noneParams, $allParams)
-               ->willReturn(null);
+                ->method('revoke')
+                ->withConsecutive($viewParams, $changeParams, $noneParams, $allParams)
+                ->willReturn(null);
 
         /* Test starts here */
 
@@ -167,11 +167,11 @@ 
         $target   = new Permissions();
 
         foreach (array(
-                     PermissionsInterface::PERMISSION_VIEW   => array(true, false, false, false),
-                     PermissionsInterface::PERMISSION_CHANGE => array(true, true, true, false),
-                     PermissionsInterface::PERMISSION_ALL    => array(true, true, true, false),
-                     PermissionsInterface::PERMISSION_NONE   => array(false, false, false, true),
-                 ) as $perm => $expected
+                        PermissionsInterface::PERMISSION_VIEW   => array(true, false, false, false),
+                        PermissionsInterface::PERMISSION_CHANGE => array(true, true, true, false),
+                        PermissionsInterface::PERMISSION_ALL    => array(true, true, true, false),
+                        PermissionsInterface::PERMISSION_NONE   => array(false, false, false, true),
+                    ) as $perm => $expected
         ) {
             $target->grant($resource, $perm);
             $this->assertEquals($expected[0], $target->isGranted($resource, PermissionsInterface::PERMISSION_VIEW));
@@ -302,20 +302,20 @@ 
     public function testGrantWorksWhenPassingPermissionsResourceInterfaces()
     {
         $resource = $this->getMockBuilder('\Core\Entity\PermissionsResourceInterface')
-                         ->getMockForAbstractClass();
+                            ->getMockForAbstractClass();
 
         $resource->expects($this->any())
-                 ->method('getPermissionsResourceId')->willReturn('resource');
+                    ->method('getPermissionsResourceId')->willReturn('resource');
 
         $userIds1 = array('user1', 'user2');
         $userIds2 = array('view' => array('user1', 'user2'), 'all' => array('user3', 'user4'));
         $resource->expects($this->exactly(4))
-                 ->method('getPermissionsUserIds')
-                 ->will($this->onConsecutiveCalls(
-                     $userIds1,
-                     $userIds2,
-                     array(),
-                     null
+                    ->method('getPermissionsUserIds')
+                    ->will($this->onConsecutiveCalls(
+                        $userIds1,
+                        $userIds2,
+                        array(),
+                        null
                         ));
 
 
@@ -383,9 +383,9 @@ 
 
         /* @var $target \Core\Entity\PermissionsInterface|\PHPUnit_Framework_MockObject_MockObject */
         $target = $this->getMockBuilder('\Core\Entity\Permissions')
-                       ->disableOriginalConstructor()
-                       ->setMethods(array('grant', 'isAssigned'))
-                       ->getMock();
+                        ->disableOriginalConstructor()
+                        ->setMethods(array('grant', 'isAssigned'))
+                        ->getMock();
 
         if (PermissionsInterface::PERMISSION_NONE == $perm) {
             $target->expects($this->never())->method('isAssigned');
@@ -400,7 +400,7 @@ 
                 : PermissionsInterface::PERMISSION_NONE;
 
             $target->expects($this->once())->method('grant')->with($resource, $expPerm, $build)
-                   ->will($this->returnSelf());
+                    ->will($this->returnSelf());
         }
 
         /* Test start here */
@@ -455,11 +455,11 @@ 
         }
 
         foreach (array(
-                     PermissionsInterface::PERMISSION_NONE,
-                     PermissionsInterface::PERMISSION_VIEW,
-                     PermissionsInterface::PERMISSION_CHANGE,
-                     PermissionsInterface::PERMISSION_ALL
-                 ) as $i => $perm
+                        PermissionsInterface::PERMISSION_NONE,
+                        PermissionsInterface::PERMISSION_VIEW,
+                        PermissionsInterface::PERMISSION_CHANGE,
+                        PermissionsInterface::PERMISSION_ALL
+                    ) as $i => $perm
         ) {
             $this->assertEquals($expected[$i], $target->isGranted($user, $perm));
         }
@@ -476,13 +476,13 @@ 
     public function testClear()
     {
         $resource = $this->getMockBuilder('\Core\Entity\PermissionsResourceInterface')
-                         ->getMockForAbstractClass();
+                            ->getMockForAbstractClass();
 
         $resource->expects($this->any())
-                 ->method('getPermissionsResourceId')->willReturn('resource');
+                    ->method('getPermissionsResourceId')->willReturn('resource');
 
         $resource->expects($this->exactly(1))->method('getPermissionsUserIds')
-                 ->willReturn(array('test'));
+                    ->willReturn(array('test'));
 
         $target = new Permissions();
         $target->grant('test', Permissions::PERMISSION_ALL);
@@ -516,13 +516,13 @@ 
     public function testInherit()
     {
         $resource = $this->getMockBuilder('\Core\Entity\PermissionsResourceInterface')
-                         ->getMockForAbstractClass();
+                            ->getMockForAbstractClass();
 
         $resource->expects($this->any())
-                 ->method('getPermissionsResourceId')->willReturn('resource');
+                    ->method('getPermissionsResourceId')->willReturn('resource');
 
         $resource->expects($this->any())->method('getPermissionsUserIds')
-                 ->willReturn(array('test'));
+                    ->willReturn(array('test'));
 
         $target1 = new Permissions();
         $target2 = new Permissions();
@@ -618,7 +618,7 @@ 
     public function testGetResourcesReturnsCollectionOfResources()
     {
         $resource = $this->getMockBuilder('\Core\Entity\PermissionsResourceInterface')
-                         ->getMockForAbstractClass();
+                            ->getMockForAbstractClass();
 
         $target = new Permissions();
         $target->grant($resource, PermissionsInterface::PERMISSION_ALL);
@@ -641,13 +641,13 @@ 
         $grantedResource->expects($this->atLeastOnce())
                         ->method('getPermissionsUserIds')
                         ->will($this->onConsecutiveCalls(array(
-                                                             'all'  => array('user1', 'user2'),
-                                                             'view' => array('popel', 'dopel'),
-                                                         ), array('all' => array('user3'))));
+                                                                'all'  => array('user1', 'user2'),
+                                                                'view' => array('popel', 'dopel'),
+                                                            ), array('all' => array('user3'))));
         $grantedResource->method('getPermissionsResourceId')->willReturn('grantResource');
 
         $ungrantedResource = $this->getMockBuilder('\Core\Entity\PermissionsResourceInterface')
-                                  ->getMockForAbstractClass();
+                                    ->getMockForAbstractClass();
         $ungrantedResource->method('getPermissionsResourceId')->willReturn('ungrant');
 
         $target = new Permissions();

module/Core/test/CoreTest/Entity/Tree/AbstractLeafsTest.php

Spacing +3 added lines, -3 removed lines

@@ -47,15 +47,15 @@
         ]
     ];
 
-    private $inheritance = [ LeafsInterface::class ];
+    private $inheritance = [LeafsInterface::class];
 
-    private $traits = [ EntityTrait::class ];
+    private $traits = [EntityTrait::class];
 
     public function propertiesProvider()
     {
 
         return [
-            [ 'items', ['default' => new ArrayCollection(), 'value' => new \Doctrine\Common\Collections\ArrayCollection()]],
+            ['items', ['default' => new ArrayCollection(), 'value' => new \Doctrine\Common\Collections\ArrayCollection()]],
         ];
     }
 

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/Form/Hydrator/TreeHydratorTest.php

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
      */
     private $target = TreeHydrator::class;
 
-    private $inheritance = [ HydratorInterface::class ];
+    private $inheritance = [HydratorInterface::class];
 
     private function getHydratedTree($mode = 'all')
     {
@@ -64,7 +64,7 @@ 
 
     private function getExtractedTree()
     {
-        return [ 'items' => [
+        return ['items' => [
             new \ArrayObject([
                 'id' => 'root-id',
                 'current' => '1',

module/Core/test/CoreTest/Form/Hydrator/MetaDataHydratorTest.php

Indentation +4 added lines, -4 removed lines

@@ -49,8 +49,8 @@ 
     {
         $expect = ['extract' => 'works'];
         $object = $this->getMockBuilder(MetaDataProviderInterface::class)
-                       ->setMethods(['getMetaData'])
-                       ->getMockForAbstractClass();
+                        ->setMethods(['getMetaData'])
+                        ->getMockForAbstractClass();
 
         $object->expects($this->once())->method('getMetaData')->willReturn($expect);
 
@@ -69,8 +69,8 @@ 
     public function testHydrationOfMetaDataProvider()
     {
         $object = $this->getMockBuilder(MetaDataProviderInterface::class)
-                       ->setMethods(['setMetaData'])
-                       ->getMockForAbstractClass();
+                        ->setMethods(['setMetaData'])
+                        ->getMockForAbstractClass();
 
         $object->expects($this->once())->method('setMetaData')->with('key', 'value');
 

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
      */
     private $target = MetaDataHydrator::class;
 
-    private $inheritance = [ HydratorInterface::class ];
+    private $inheritance = [HydratorInterface::class];
 
     public function testExtractionOfNonMetaDataProvider()
     {

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

Indentation +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
         $labeldContainer->setLabel('testLabel');
         $topContainer = new Container();
         $topContainer->setForm('child', [ '__instance__' => $object ])
-                     ->setLabel('top');
+                        ->setLabel('top');
 
         return [
             /*[ 'Headscripts', [

Spacing +35 added lines, -35 removed lines

@@ -41,7 +41,7 @@ 
         $labeldContainer = new Container();
         $labeldContainer->setLabel('testLabel');
         $topContainer = new Container();
-        $topContainer->setForm('child', [ '__instance__' => $object ])
+        $topContainer->setForm('child', ['__instance__' => $object])
                      ->setLabel('top');
 
         return [
@@ -49,85 +49,85 @@ 
                 'default' => [ '/assets/twitter-bootstrap-wizard/jquery.bootstrap.wizard.js' ],
                 'value'   => [ 'test/scripts' ],
             ]],*/
-            [ 'Form', [
+            ['Form', [
                 'value' => 'test',
-                'setter_args' => [ $object ],
-                'setter_exception' => [ '\InvalidArgumentException', 'Tab container must be of the type' ],
+                'setter_args' => [$object],
+                'setter_exception' => ['\InvalidArgumentException', 'Tab container must be of the type'],
             ]],
-            [ 'Form', [
+            ['Form', [
                 'value' => 'test',
-                'setter_args' => [ $container ],
-                'setter_exception' => [ '\InvalidArgumentException', 'Container instances must have a label' ],
+                'setter_args' => [$container],
+                'setter_exception' => ['\InvalidArgumentException', 'Container instances must have a label'],
             ]],
-            [ 'Form', [
+            ['Form', [
                 'value' => 'test',
-                'setter_args' => [ $labeldContainer ],
-                'getter_args' => [ 'test', true ],
+                'setter_args' => [$labeldContainer],
+                'getter_args' => ['test', true],
                 'expect' => $labeldContainer,
             ]],
-            [ 'Form', [
+            ['Form', [
                 'value' => 'test',
-                'setter_args' => [ 'Test/Container' ],
-                'getter_args' => [ 'test', /*asInstance*/ false ],
+                'setter_args' => ['Test/Container'],
+                'getter_args' => ['test', /*asInstance*/ false],
                 'expect' => [
                     'type' => 'Test/Container',
                     'name' => 'test',
                     'entity' => '*',
                 ],
             ]],
-            [ 'Form', [
+            ['Form', [
                 'value' => 'test',
-                'setter_args' => [ [ 'type' => 'Test/Container' ] ],
-                'getter_args' => [ 'test', false ],
+                'setter_args' => [['type' => 'Test/Container']],
+                'getter_args' => ['test', false],
                 'expect' => [
                     'type' => 'Test/Container',
                     'name' => 'test',
                     'entity' => '*',
                 ],
             ]],
-            [ 'Form', [
+            ['Form', [
                 'value' => 'test',
-                'setter_args' => [ [ ] ],
-                'getter_args' => [ 'test', false ],
+                'setter_args' => [[]],
+                'getter_args' => ['test', false],
                 'expect' => [
                     'type' => 'Core/Container',
                     'name' => 'test',
                     'entity' => '*',
                 ],
             ]],
-            [ 'Form', [
+            ['Form', [
                 'value' => 'test',
-                'setter_args' => [ [ 'forms' => [] ] ],
-                'getter_args' => [ 'test', false ],
+                'setter_args' => [['forms' => []]],
+                'getter_args' => ['test', false],
                 'expect' => [
                     'type' => 'Core/Container',
                     'name' => 'test',
-                    'options' => [ 'forms' => [] ],
+                    'options' => ['forms' => []],
                     'entity' => '*',
                 ]
             ]],
-            [ 'Form', [
+            ['Form', [
                 'value' => 'test',
-                'setter_args' => [ [ '__instance__' => $object ] ],
-                'getter_args' => [ 'test' ],
+                'setter_args' => [['__instance__' => $object]],
+                'getter_args' => ['test'],
                 'getter_exception' => '\UnexpectedValueException'
             ]],
-            [ 'Form', [
+            ['Form', [
                 'value' => 'test',
-                'setter_args' => [ [ '__instance__' => $container ] ],
-                'getter_args' => [ 'test' ],
+                'setter_args' => [['__instance__' => $container]],
+                'getter_args' => ['test'],
                 'getter_exception' => '\UnexpectedValueException'
             ]],
-            [ 'Form', [
+            ['Form', [
                 'value' => 'test',
-                'setter_args' => [ [ '__instance__' => $labeldContainer ] ],
-                'getter_args' => [ 'test' ],
+                'setter_args' => [['__instance__' => $labeldContainer]],
+                'getter_args' => ['test'],
                 'expect' => $labeldContainer
             ]],
-            [ 'Form', [
+            ['Form', [
                 'value' => 'test',
-                'setter_args' => [ [ '__instance__' => $topContainer ] ],
-                'getter_args' => [ 'test.child' ],
+                'setter_args' => [['__instance__' => $topContainer]],
+                'getter_args' => ['test.child'],
                 'expect' => $object
             ]],
 

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

Spacing +1 added lines, -1 removed lines

@@ -106,7 +106,7 @@
             [
                 [
                     0 => ['name' => 'test', 'options' => ['opt' => 'value', 'label' => 'Overridden']],
-                    1 => [ 't' => 'y', 'test' => 'AlsoOverridden']
+                    1 => ['t' => 'y', 'test' => 'AlsoOverridden']
                 ]
             ],
             $this->target->addCalledWithArgs

Indentation +1 added lines, -1 removed lines

@@ -146,7 +146,7 @@
 
         $this->assertEquals(
             ['test' => ['required' => true],
-              'test2' => ['filters' => []]],
+                'test2' => ['filters' => []]],
             $actual
         );
     }