@@ -106,7 +106,7 @@ |
||
106 | 106 | [ |
107 | 107 | [ |
108 | 108 | 0 => ['name' => 'test', 'options' => ['opt' => 'value', 'label' => 'Overridden']], |
109 | - 1 => [ 't' => 'y', 'test' => 'AlsoOverridden'] |
|
109 | + 1 => ['t' => 'y', 'test' => 'AlsoOverridden'] |
|
110 | 110 | ] |
111 | 111 | ], |
112 | 112 | $this->target->addCalledWithArgs |
@@ -146,7 +146,7 @@ |
||
146 | 146 | |
147 | 147 | $this->assertEquals( |
148 | 148 | ['test' => ['required' => true], |
149 | - 'test2' => ['filters' => []]], |
|
149 | + 'test2' => ['filters' => []]], |
|
150 | 150 | $actual |
151 | 151 | ); |
152 | 152 | } |
@@ -32,14 +32,14 @@ |
||
32 | 32 | |
33 | 33 | private $target = ManagementForm::class; |
34 | 34 | |
35 | - private $inheritance = [ SummaryForm::class, HeadscriptProviderInterface::class ]; |
|
35 | + private $inheritance = [SummaryForm::class, HeadscriptProviderInterface::class]; |
|
36 | 36 | |
37 | 37 | private $attributes = [ |
38 | 38 | 'baseFieldset' => 'Core/Tree/ManagementFieldset', |
39 | - 'attributes' => [ 'method' => 'POST', 'class' => 'yk-tree-management-form' ], |
|
39 | + 'attributes' => ['method' => 'POST', 'class' => 'yk-tree-management-form'], |
|
40 | 40 | ]; |
41 | 41 | |
42 | 42 | private $properties = [ |
43 | - [ 'headscripts', ['value' => ['testHeadScript'], 'default' => [ 'modules/Core/js/html.sortable.min.js', 'modules/Core/js/forms.tree-management.js' ]] ] |
|
43 | + ['headscripts', ['value' => ['testHeadScript'], 'default' => ['modules/Core/js/html.sortable.min.js', 'modules/Core/js/forms.tree-management.js']]] |
|
44 | 44 | ]; |
45 | 45 | } |
@@ -40,12 +40,12 @@ |
||
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', |
@@ -32,7 +32,7 @@ |
||
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 | } |
@@ -37,7 +37,7 @@ discard block |
||
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 |
||
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 | ]; |
@@ -51,19 +51,19 @@ |
||
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 | } |
@@ -38,7 +38,7 @@ |
||
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 | { |
@@ -92,7 +92,7 @@ discard block |
||
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 |
||
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 |
@@ -36,7 +36,7 @@ |
||
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 |
@@ -110,8 +110,8 @@ |
||
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 |
@@ -40,13 +40,13 @@ discard block |
||
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 |
||
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() |