Passed
Push — master ( c58a40...6d37ad )
by Mathias
10:02 queued 04:27
created
module/Core/test/CoreTest/Form/Tree/ManagementFieldsetTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -40,12 +40,12 @@
 block discarded – undo
40 40
      */
41 41
     private $target = [
42 42
         ManagementFieldset::class,
43
-        '@testInitialize' => [ 'mock' => ['add'] ],
43
+        '@testInitialize' => ['mock' => ['add']],
44 44
     ];
45 45
 
46
-    private $inheritance = [ Fieldset::class, ViewPartialProviderInterface::class ];
46
+    private $inheritance = [Fieldset::class, ViewPartialProviderInterface::class];
47 47
 
48
-    private $traits = [ ViewPartialProviderTrait::class ];
48
+    private $traits = [ViewPartialProviderTrait::class];
49 49
 
50 50
     private $attributes = [
51 51
         'defaultPartial' => 'core/form/tree-manage',
Please login to merge, or discard this patch.
module/Core/test/CoreTest/Form/Tree/SelectTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@
 block discarded – undo
32 32
 
33 33
     private $target = Select::class;
34 34
 
35
-    private $inheritance = [ ZfSelect::class, HydratorStrategyProviderInterface::class ];
35
+    private $inheritance = [ZfSelect::class, HydratorStrategyProviderInterface::class];
36 36
 
37
-    private $traits = [ HydratorStrategyProviderTrait::class ];
37
+    private $traits = [HydratorStrategyProviderTrait::class];
38 38
 }
Please login to merge, or discard this patch.
module/Core/test/CoreTest/Form/Tree/AddItemFieldsetTest.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
         '@testInitializesItself' => 'setupMock'
38 38
     ];
39 39
 
40
-    private $inheritance = [ Fieldset::class, ViewPartialProviderInterface::class, InputFilterProviderInterface::class];
40
+    private $inheritance = [Fieldset::class, ViewPartialProviderInterface::class, InputFilterProviderInterface::class];
41 41
 
42 42
     private $traits = [ViewPartialProviderTrait::class];
43 43
 
@@ -77,13 +77,13 @@  discard block
 block discarded – undo
77 77
             'name' => [
78 78
                 'required' => true,
79 79
                 'filters' => [
80
-                    [ 'name' => 'StringTrim' ],
80
+                    ['name' => 'StringTrim'],
81 81
                 ],
82 82
             ],
83 83
             'value' => [
84 84
                 'required' => false,
85 85
                 'filters' => [
86
-                    [ 'name' => 'StringTrim' ],
86
+                    ['name' => 'StringTrim'],
87 87
                 ],
88 88
             ],
89 89
         ];
Please login to merge, or discard this patch.
Indentation   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -51,19 +51,19 @@
 block discarded – undo
51 51
     {
52 52
         $mock = $this->getMockBuilder(AddItemFieldset::class)
53 53
             ->setMethods(['setObject', 'add'])
54
-             ->disableOriginalConstructor()->getMock();
54
+                ->disableOriginalConstructor()->getMock();
55 55
 
56 56
         $mock->expects($this->once())->method('setObject')->with($this->isInstanceOf(\ArrayObject::class));
57 57
 
58 58
         $mock->expects($this->exactly(6))->method('add')
59
-             ->withConsecutive(
60
-                 [['name' => 'id', 'type' => 'Hidden']],
61
-                 [['name' => 'current', 'type' => 'Hidden']],
62
-                 [['name' => 'do', 'type' => 'Hidden']],
63
-                 [['name' => 'name', 'type' => 'Text', 'options' => ['label' => 'Name'], 'attributes' => ['required' => 'required']]],
64
-                 [['name' => 'value', 'type' => 'Text', 'options' => ['label' => 'Value']]],
65
-                 [['name' => 'priority', 'type' => 'Text']]
66
-             );
59
+                ->withConsecutive(
60
+                    [['name' => 'id', 'type' => 'Hidden']],
61
+                    [['name' => 'current', 'type' => 'Hidden']],
62
+                    [['name' => 'do', 'type' => 'Hidden']],
63
+                    [['name' => 'name', 'type' => 'Text', 'options' => ['label' => 'Name'], 'attributes' => ['required' => 'required']]],
64
+                    [['name' => 'value', 'type' => 'Text', 'options' => ['label' => 'Value']]],
65
+                    [['name' => 'priority', 'type' => 'Text']]
66
+                );
67 67
 
68 68
         return $mock;
69 69
     }
Please login to merge, or discard this patch.
module/Core/test/CoreTest/View/Helper/SnippetTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
         '@testConstruction' => false,
39 39
     ];
40 40
 
41
-    private $inheritance = [ AbstractHelper::class ];
41
+    private $inheritance = [AbstractHelper::class];
42 42
 
43 43
     private function getTargetArgs()
44 44
     {
Please login to merge, or discard this patch.
module/Core/test/CoreTest/Service/ClearCacheServiceTest.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
      * @throws \Exception
93 93
      * @dataProvider    getTestClearCache
94 94
      */
95
-    public function testClearCache($cacheDir, $expectThrow=false, $message='Test Cache Directory')
95
+    public function testClearCache($cacheDir, $expectThrow = false, $message = 'Test Cache Directory')
96 96
     {
97 97
         $options = $this->getMockBuilder(ListenerOptions::class)
98 98
             ->disableOriginalConstructor()
@@ -115,9 +115,9 @@  discard block
 block discarded – undo
115 115
     {
116 116
         $cacheDir = sys_get_temp_dir().'/yawik/test-cache';
117 117
         return [
118
-            [null,true,'Test with null cache directory'],
119
-            [sys_get_temp_dir().'/foo',true,'Test with non existent directory'],
120
-            [$cacheDir,false,'Test with valid cache directory']
118
+            [null, true, 'Test with null cache directory'],
119
+            [sys_get_temp_dir().'/foo', true, 'Test with non existent directory'],
120
+            [$cacheDir, false, 'Test with valid cache directory']
121 121
         ];
122 122
     }
123 123
 
Please login to merge, or discard this patch.
Core/test/CoreTest/Listener/LanguageRouteListener/OnRouteCallbackTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@
 block discarded – undo
36 36
 
37 37
     private $target = [
38 38
         LanguageRouteListener::class,
39
-        'mock' => [ 'detectLanguage' => ['return' => 'xx'], 'setLocale' ],
39
+        'mock' => ['detectLanguage' => ['return' => 'xx'], 'setLocale'],
40 40
         'args' => 'getConstructorArgs'
41 41
     ];
42 42
     
Please login to merge, or discard this patch.
module/Core/test/CoreTest/Listener/AjaxRouteListenerTest.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -110,8 +110,8 @@
 block discarded – undo
110 110
         $results->push($result1);
111 111
         $this->ajaxEventManagerMock->expects($this->once())->method('getEvent')->with('test', $this->target)->willReturn(new AjaxEvent());
112 112
         $this->ajaxEventManagerMock->expects($this->once())->method('triggerEventUntil')
113
-                                   ->with($this->isInstanceOf('\Closure'), $this->isInstanceOf(AjaxEvent::class))
114
-                                   ->willReturn($results);
113
+                                    ->with($this->isInstanceOf('\Closure'), $this->isInstanceOf(AjaxEvent::class))
114
+                                    ->willReturn($results);
115 115
 
116 116
         $response = $this->target->onRoute($event);
117 117
 
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -40,13 +40,13 @@  discard block
 block discarded – undo
40 40
         '@testConstruction' => false,
41 41
         '@testInheritance' => '@testUsesTraits',
42 42
         '@testDefaultAttributes' => '@testUsesTraits',
43
-        '@testUsesTraits' => [ 'as_reflection' => true ],
43
+        '@testUsesTraits' => ['as_reflection' => true],
44 44
     ];
45 45
 
46
-    private $traits = [ ListenerAggregateTrait::class ];
46
+    private $traits = [ListenerAggregateTrait::class];
47 47
 
48 48
     private $attributes = [
49
-        'events' => [ [ MvcEvent::EVENT_ROUTE, 'onRoute', 100 ] ],
49
+        'events' => [[MvcEvent::EVENT_ROUTE, 'onRoute', 100]],
50 50
     ];
51 51
 
52 52
     private function getTargetArgs()
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
         $this->ajaxEventManagerMock = $this->getMockBuilder(EventManager::class)->disableOriginalConstructor()
55 55
             ->setMethods(['getEvent', 'triggerEventUntil'])->getMock();
56 56
 
57
-        return [ $this->ajaxEventManagerMock ];
57
+        return [$this->ajaxEventManagerMock];
58 58
     }
59 59
 
60 60
     public function testConstruction()
Please login to merge, or discard this patch.
module/Core/test/CoreTestUtils/TestCase/SimpleSetterAndGetterTrait.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -21,14 +21,14 @@
 block discarded – undo
21 21
         if (!is_object($object)) {
22 22
             throw new \InvalidArgumentException('Data Provider first argument should be an object');
23 23
         }
24
-        $setter = 'set' . $propertyName;
25
-        $getter = 'get' . $propertyName;
24
+        $setter = 'set'.$propertyName;
25
+        $getter = 'get'.$propertyName;
26 26
 
27 27
         call_user_func(array($object, $setter), $propertyValue);
28 28
         $this->assertSame(
29 29
             $propertyValue,
30 30
             call_user_func(array($object, $getter)),
31
-            '::' . $setter . '() and ::' . $getter . '() should executed properly'
31
+            '::'.$setter.'() and ::'.$getter.'() should executed properly'
32 32
         );
33 33
     }
34 34
 
Please login to merge, or discard this patch.
module/Core/src/Module.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -97,8 +97,8 @@  discard block
 block discarded – undo
97 97
             // assets-install info
98 98
             'assets-install [--symlink] [--relative] <target>' => 'Install assets in the given target',
99 99
             'The assets-install command will install assets in the given <target> directory. If no option given this command will copy assets into the target.',
100
-            ['--symlink','This option will install assets using absolute symlink directory'],
101
-            ['--relative','This option will install assets using relative symlink'],
100
+            ['--symlink', 'This option will install assets using absolute symlink directory'],
101
+            ['--relative', 'This option will install assets using relative symlink'],
102 102
             ""
103 103
         ];
104 104
     }
@@ -164,7 +164,7 @@  discard block
 block discarded – undo
164 164
 
165 165
         $eventManager->attach(
166 166
             MvcEvent::EVENT_DISPATCH_ERROR,
167
-            function ($event) {
167
+            function($event) {
168 168
                 if ($event instanceof MvcEvent) {
169 169
                     $application = $event->getApplication();
170 170
                     
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
         );
181 181
         $eventManager->attach(
182 182
             MvcEvent::EVENT_DISPATCH,
183
-            function ($event) use ($eventManager) {
183
+            function($event) use ($eventManager) {
184 184
                 $eventManager->trigger('postDispatch', $event);
185 185
             },
186 186
             -150
@@ -196,7 +196,7 @@  discard block
 block discarded – undo
196 196
      */
197 197
     public function getConfig()
198 198
     {
199
-        $config = include __DIR__ . '/../config/module.config.php';
199
+        $config = include __DIR__.'/../config/module.config.php';
200 200
         return $config;
201 201
     }
202 202
 
@@ -206,7 +206,7 @@  discard block
 block discarded – undo
206 206
     public function init(ModuleManager $manager)
207 207
     {
208 208
         $events = $manager->getEventManager();
209
-        $events->attach(ModuleEvent::EVENT_MERGE_CONFIG, [$this,'onMergeConfig']);
209
+        $events->attach(ModuleEvent::EVENT_MERGE_CONFIG, [$this, 'onMergeConfig']);
210 210
     }
211 211
 
212 212
     /**
Please login to merge, or discard this patch.