@@ -39,9 +39,9 @@ |
||
39 | 39 | |
40 | 40 | private $target = AdminCategoriesController::class; |
41 | 41 | |
42 | - private $inheritance = [ AbstractActionController::class ]; |
|
42 | + private $inheritance = [AbstractActionController::class]; |
|
43 | 43 | |
44 | - private function getConfiguredServiceManager($rootExists=true, $postRequest = false) |
|
44 | + private function getConfiguredServiceManager($rootExists = true, $postRequest = false) |
|
45 | 45 | { |
46 | 46 | $form = $this |
47 | 47 | ->getMockBuilder(CategoriesContainer::class) |
@@ -76,7 +76,7 @@ |
||
76 | 76 | vfsStream::setup('/'); |
77 | 77 | vfsStream::create($isJobs ? ['module' => ['Jobs' => []]] : ['module' => []]); |
78 | 78 | $expectModulePath = $isJobs ? vfsStream::url('./module/Jobs/config/') : '.'; |
79 | - $config['module_paths'] = array_map(function ($i) { |
|
79 | + $config['module_paths'] = array_map(function($i) { |
|
80 | 80 | return vfsStream::url($i); |
81 | 81 | }, $config['module_paths']); |
82 | 82 |
@@ -54,7 +54,7 @@ |
||
54 | 54 | |
55 | 55 | public function testTrimAndStripTagsFromJobTitle() |
56 | 56 | { |
57 | - $expected="title (m/w)"; |
|
57 | + $expected = "title (m/w)"; |
|
58 | 58 | $output = $this->inputFilter->getInputs(); |
59 | 59 | $filter = $output['title']->getFilterChain(); |
60 | 60 | $this->assertEquals($expected, $filter->filter(' <b>title</b> (m/w)')); |
@@ -45,7 +45,7 @@ |
||
45 | 45 | '@testCreateService' => ['mock' => ['__invoke' => ['count' => 1]]], |
46 | 46 | ]; |
47 | 47 | |
48 | - private $inheritance = [ FactoryInterface::class ]; |
|
48 | + private $inheritance = [FactoryInterface::class]; |
|
49 | 49 | |
50 | 50 | |
51 | 51 | public function testCreateService() |
@@ -40,7 +40,7 @@ |
||
40 | 40 | */ |
41 | 41 | public function testInvokation() |
42 | 42 | { |
43 | - $provider=new ProviderOptions(); |
|
43 | + $provider = new ProviderOptions(); |
|
44 | 44 | |
45 | 45 | $serviceManagerMock = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager') |
46 | 46 | ->disableOriginalConstructor() |
@@ -109,9 +109,9 @@ |
||
109 | 109 | ->disableOriginalConstructor() |
110 | 110 | ->getMock(); |
111 | 111 | |
112 | - $callback1 = [new MailSenderListenerMock(),'listen1']; |
|
113 | - $callback2 = [new MailSenderListenerMock(),'listen2']; |
|
114 | - $callback3 = [new MailSenderListenerMock(),'listen3']; |
|
112 | + $callback1 = [new MailSenderListenerMock(), 'listen1']; |
|
113 | + $callback2 = [new MailSenderListenerMock(), 'listen2']; |
|
114 | + $callback3 = [new MailSenderListenerMock(), 'listen3']; |
|
115 | 115 | $events->expects($this->exactly(3)) |
116 | 116 | ->method('attach') |
117 | 117 | ->withConsecutive( |
@@ -94,7 +94,7 @@ |
||
94 | 94 | |
95 | 95 | private function createOrganization(array $spec) |
96 | 96 | { |
97 | - $flag = function ($key) use ($spec) { |
|
97 | + $flag = function($key) use ($spec) { |
|
98 | 98 | return isset($spec[$key]) && $spec[$key]; |
99 | 99 | }; |
100 | 100 | $org = $this |
@@ -70,13 +70,13 @@ discard block |
||
70 | 70 | foreach (array($role, $user) as $testObj) { |
71 | 71 | // non user is always false? |
72 | 72 | foreach ($privileges as $p) { |
73 | - $this->assertFalse((bool)$target->assert($acl, $testObj, null, $p)); |
|
73 | + $this->assertFalse((bool) $target->assert($acl, $testObj, null, $p)); |
|
74 | 74 | } |
75 | 75 | |
76 | 76 | // user and wrong privilege is false? |
77 | 77 | foreach ($privileges as $p) { |
78 | 78 | // Casting null to false is safe here, 'edit' for pair user-privilege tested after |
79 | - $this->assertFalse((bool)$target->assert($acl, $testObj, $job, $p)); |
|
79 | + $this->assertFalse((bool) $target->assert($acl, $testObj, $job, $p)); |
|
80 | 80 | } |
81 | 81 | } |
82 | 82 | } |
@@ -123,7 +123,7 @@ discard block |
||
123 | 123 | */ |
124 | 124 | public function testAssertWithoutOrganisation($input, $expected) |
125 | 125 | { |
126 | - $method="assert".($expected?"True":"False"); |
|
126 | + $method = "assert".($expected ? "True" : "False"); |
|
127 | 127 | |
128 | 128 | $this->$method( |
129 | 129 | $this->target->assert( |
@@ -160,9 +160,9 @@ discard block |
||
160 | 160 | |
161 | 161 | |
162 | 162 | return [ |
163 | - [[new Acl(), null, null, null] , false ], |
|
164 | - [[new Acl(), null, null, Permissions::PERMISSION_CHANGE] , false ], |
|
165 | - [[new Acl(), $user, $jobMock, 'edit'] , true ], |
|
163 | + [[new Acl(), null, null, null], false], |
|
164 | + [[new Acl(), null, null, Permissions::PERMISSION_CHANGE], false], |
|
165 | + [[new Acl(), $user, $jobMock, 'edit'], true], |
|
166 | 166 | ]; |
167 | 167 | } |
168 | 168 | |
@@ -189,8 +189,8 @@ discard block |
||
189 | 189 | $this->assertTrue( |
190 | 190 | $this->target->assert( |
191 | 191 | new Acl(), // acl |
192 | - $user, // role |
|
193 | - $jobMock, // resource |
|
192 | + $user, // role |
|
193 | + $jobMock, // resource |
|
194 | 194 | 'edit' // privilege |
195 | 195 | ) |
196 | 196 | ); |
@@ -223,8 +223,8 @@ discard block |
||
223 | 223 | $this->assertTrue( |
224 | 224 | $this->target->assert( |
225 | 225 | new Acl(), // acl |
226 | - $user, // role |
|
227 | - $jobMock, // resource |
|
226 | + $user, // role |
|
227 | + $jobMock, // resource |
|
228 | 228 | 'edit' // privilege |
229 | 229 | ) |
230 | 230 | ); |
@@ -264,8 +264,8 @@ discard block |
||
264 | 264 | $this->assertTrue( |
265 | 265 | $this->target->assert( |
266 | 266 | new Acl(), // acl |
267 | - $user, // role |
|
268 | - $jobMock, // resource |
|
267 | + $user, // role |
|
268 | + $jobMock, // resource |
|
269 | 269 | 'edit' // privilege |
270 | 270 | ) |
271 | 271 | ); |
@@ -72,7 +72,7 @@ |
||
72 | 72 | ->getMock(); |
73 | 73 | $translator->expects($this->once()) |
74 | 74 | ->method('translate') |
75 | - ->with($this->callback(function ($string) { |
|
75 | + ->with($this->callback(function($string) { |
|
76 | 76 | return is_string($string); |
77 | 77 | })) |
78 | 78 | ->willReturn($expected); |