@@ -126,14 +126,14 @@ |
||
126 | 126 | |
127 | 127 | $formData = ['data' => 'value']; |
128 | 128 | $hydrator = $this->getMockBuilder(HydratorInterface::class) |
129 | - ->setMethods(['hydrate', 'extract']) |
|
130 | - ->getMockForAbstractClass(); |
|
129 | + ->setMethods(['hydrate', 'extract']) |
|
130 | + ->getMockForAbstractClass(); |
|
131 | 131 | $hydrator->expects($this->once())->method('extract')->with($this->isInstanceOf(Parameters::class))->willReturn($formData); |
132 | 132 | $hydrator->expects($this->once())->method('hydrate')->with($formData, $this->isInstanceOf(Parameters::class)); |
133 | 133 | |
134 | 134 | $form = $this->getMockBuilder(Form::class) |
135 | - ->setMethods(['getHydrator', 'setData']) |
|
136 | - ->getMock(); |
|
135 | + ->setMethods(['getHydrator', 'setData']) |
|
136 | + ->getMock(); |
|
137 | 137 | $form->expects($this->once())->method('gethydrator')->willReturn($hydrator); |
138 | 138 | $form->expects($this->once())->method('setData')->with($formData); |
139 | 139 | $this->formElementManagerMock |
@@ -47,16 +47,16 @@ discard block |
||
47 | 47 | $services = new ServiceManager(); |
48 | 48 | $this->assertTrue( |
49 | 49 | $this->target->canCreate( |
50 | - $services, |
|
51 | - 'Any.string/Value/Events' |
|
52 | - ), |
|
50 | + $services, |
|
51 | + 'Any.string/Value/Events' |
|
52 | + ), |
|
53 | 53 | 'Checking correct name failed.' |
54 | 54 | ); |
55 | 55 | $this->assertFalse( |
56 | 56 | $this->target->canCreate( |
57 | - $services, |
|
58 | - 'Any.string.not.ending/in/Events.but has it in the middle!' |
|
59 | - ), |
|
57 | + $services, |
|
58 | + 'Any.string.not.ending/in/Events.but has it in the middle!' |
|
59 | + ), |
|
60 | 60 | 'Checking invalid name failed.' |
61 | 61 | ); |
62 | 62 | } |
@@ -98,8 +98,8 @@ discard block |
||
98 | 98 | |
99 | 99 | /* @var EventManagerAbstractFactory|\PHPUnit_Framework_MockObject_MockObject $target */ |
100 | 100 | $target = $this->getMockBuilder('\Core\Factory\EventManager\EventManagerAbstractFactory') |
101 | - ->setMethods([ 'createEventManager', 'attachListeners' ]) |
|
102 | - ->getMock(); |
|
101 | + ->setMethods([ 'createEventManager', 'attachListeners' ]) |
|
102 | + ->getMock(); |
|
103 | 103 | |
104 | 104 | $services = new ServiceManager(); |
105 | 105 | $services->setService('Config', $config); |
@@ -153,11 +153,11 @@ discard block |
||
153 | 153 | 'delay' => 100, |
154 | 154 | ]; |
155 | 155 | $resultMock = $this->getMockBuilder(\MongoDB\InsertOneResult::class)->disableOriginalConstructor() |
156 | - ->setMethods(['getInsertedId'])->getMock(); |
|
156 | + ->setMethods(['getInsertedId'])->getMock(); |
|
157 | 157 | $resultMock->expects($this->once())->method('getInsertedId')->willReturn('ID'); |
158 | 158 | $this->mongoCollectionMock->expects($this->once())->method('insertOne') |
159 | - ->with($this->callback(function ($value) use ($options) { |
|
160 | - return isset($value['queue']) && $value['queue'] == $this->queueName |
|
159 | + ->with($this->callback(function ($value) use ($options) { |
|
160 | + return isset($value['queue']) && $value['queue'] == $this->queueName |
|
161 | 161 | && isset($value['status']) && $value['status'] == MongoQueue::STATUS_PENDING |
162 | 162 | && !isset($value['tried']) |
163 | 163 | && !isset($value['message']) && !isset($value['trace']) |
@@ -166,9 +166,9 @@ discard block |
||
166 | 166 | && $value['created'] instanceof \MongoDB\BSON\UTCDateTime |
167 | 167 | && (int) ((string) $value['scheduled']) - (int) ((string)$value['created']) == $options['delay'] * 1000 |
168 | 168 | && isset($value['priority']) && $value['priority'] == $options['priority'] |
169 | - ; |
|
170 | - })) |
|
171 | - ->willReturn($resultMock); |
|
169 | + ; |
|
170 | + })) |
|
171 | + ->willReturn($resultMock); |
|
172 | 172 | |
173 | 173 | $this->target->push($job, $options); |
174 | 174 | |
@@ -184,21 +184,21 @@ discard block |
||
184 | 184 | |
185 | 185 | |
186 | 186 | $this->mongoCollectionMock->expects($this->once())->method('findOneAndUpdate') |
187 | - ->with( |
|
188 | - $this->equalTo(['_id' => new \MongoDB\BSON\ObjectID($job->getId())]), |
|
189 | - $this->callback( |
|
190 | - function ($value) { |
|
191 | - if (!array_key_exists('$set', $value)) { |
|
192 | - return false; |
|
193 | - } |
|
194 | - $value = $value['$set']; |
|
195 | - return isset($value['tried']) && $value['tried'] == 10 |
|
187 | + ->with( |
|
188 | + $this->equalTo(['_id' => new \MongoDB\BSON\ObjectID($job->getId())]), |
|
189 | + $this->callback( |
|
190 | + function ($value) { |
|
191 | + if (!array_key_exists('$set', $value)) { |
|
192 | + return false; |
|
193 | + } |
|
194 | + $value = $value['$set']; |
|
195 | + return isset($value['tried']) && $value['tried'] == 10 |
|
196 | 196 | && !array_key_exists('created', $value) |
197 | 197 | ; |
198 | - } |
|
199 | - ) |
|
200 | - ) |
|
201 | - ->willReturn(null); |
|
198 | + } |
|
199 | + ) |
|
200 | + ) |
|
201 | + ->willReturn(null); |
|
202 | 202 | |
203 | 203 | $this->target->retry($job); |
204 | 204 | } |
@@ -215,33 +215,33 @@ discard block |
||
215 | 215 | |
216 | 216 | $this->jobManagerMock->expects($this->once())->method('get')->with(get_class($job))->willReturn(new $this->testJobClass()); |
217 | 217 | $this->mongoCollectionMock->expects($this->once())->method('findOneAndUpdate') |
218 | - ->with( |
|
219 | - $this->callback(function ($value) { |
|
220 | - return |
|
218 | + ->with( |
|
219 | + $this->callback(function ($value) { |
|
220 | + return |
|
221 | 221 | isset($value['queue']) && $value['queue'] == $this->queueName |
222 | 222 | && isset($value['status']) && $value['status'] == MongoQueue::STATUS_PENDING |
223 | 223 | && isset($value['scheduled']['$lte']) && $value['scheduled']['$lte'] instanceof \MongoDB\BSON\UTCDateTime |
224 | - ; |
|
225 | - }), |
|
226 | - $this->callback( |
|
227 | - function ($value) { |
|
228 | - if (!array_key_exists('$set', $value)) { |
|
229 | - return false; |
|
230 | - } |
|
231 | - $value = $value['$set']; |
|
232 | - return isset($value['status']) && $value['status'] == MongoQueue::STATUS_RUNNING |
|
224 | + ; |
|
225 | + }), |
|
226 | + $this->callback( |
|
227 | + function ($value) { |
|
228 | + if (!array_key_exists('$set', $value)) { |
|
229 | + return false; |
|
230 | + } |
|
231 | + $value = $value['$set']; |
|
232 | + return isset($value['status']) && $value['status'] == MongoQueue::STATUS_RUNNING |
|
233 | 233 | && array_key_exists('executed', $value) |
234 | - ; |
|
235 | - } |
|
236 | - ), |
|
237 | - $this->callback(function ($value) { |
|
238 | - return |
|
234 | + ; |
|
235 | + } |
|
236 | + ), |
|
237 | + $this->callback(function ($value) { |
|
238 | + return |
|
239 | 239 | isset($value['sort']) && $value['sort'] == ['priority' => 1, 'scheduled' => 1] |
240 | 240 | && isset($value['returnDocument'])&&$value['returnDocument'] = \MongoDB\Operation\FindOneAndUpdate::RETURN_DOCUMENT_AFTER |
241 | - ; |
|
242 | - }) |
|
243 | - ) |
|
244 | - ->willReturn($envelope); |
|
241 | + ; |
|
242 | + }) |
|
243 | + ) |
|
244 | + ->willReturn($envelope); |
|
245 | 245 | |
246 | 246 | $actualJob = $this->target->pop(); |
247 | 247 | |
@@ -251,7 +251,7 @@ discard block |
||
251 | 251 | public function testPopEmptyQueue() |
252 | 252 | { |
253 | 253 | $this->mongoCollectionMock->expects($this->once())->method('findOneAndUpdate') |
254 | - ->willReturn(null); |
|
254 | + ->willReturn(null); |
|
255 | 255 | |
256 | 256 | $actualJob = $this->target->pop(); |
257 | 257 | |
@@ -335,8 +335,8 @@ discard block |
||
335 | 335 | }; |
336 | 336 | |
337 | 337 | $this->mongoCollectionMock->expects($this->once())->method('find') |
338 | - ->with($filter, $opt) |
|
339 | - ->willReturn($cursor); |
|
338 | + ->with($filter, $opt) |
|
339 | + ->willReturn($cursor); |
|
340 | 340 | |
341 | 341 | $this->target->listing($options); |
342 | 342 | } |
@@ -348,23 +348,23 @@ discard block |
||
348 | 348 | $job->setId(new \MongoDB\BSON\ObjectID()); |
349 | 349 | $options = ['message' => 'test failure message']; |
350 | 350 | $this->mongoCollectionMock->expects($this->once())->method('findOneAndUpdate') |
351 | - ->with( |
|
352 | - $this->equalTo(['_id' => new \MongoDB\BSON\ObjectID($job->getId())]), |
|
353 | - $this->callback( |
|
354 | - function ($value) use ($options) { |
|
355 | - if (!array_key_exists('$set', $value)) { |
|
356 | - return false; |
|
357 | - } |
|
358 | - $value = $value['$set']; |
|
359 | - return isset($value['status']) && $value['status'] == MongoQueue::STATUS_FAILED |
|
351 | + ->with( |
|
352 | + $this->equalTo(['_id' => new \MongoDB\BSON\ObjectID($job->getId())]), |
|
353 | + $this->callback( |
|
354 | + function ($value) use ($options) { |
|
355 | + if (!array_key_exists('$set', $value)) { |
|
356 | + return false; |
|
357 | + } |
|
358 | + $value = $value['$set']; |
|
359 | + return isset($value['status']) && $value['status'] == MongoQueue::STATUS_FAILED |
|
360 | 360 | && !array_key_exists('created', $value) |
361 | 361 | && !array_key_exists('scheduled', $value) |
362 | 362 | && isset($value['message']) && $value['message'] == $options['message'] |
363 | - ; |
|
364 | - } |
|
365 | - ) |
|
366 | - ) |
|
367 | - ->willReturn(null); |
|
363 | + ; |
|
364 | + } |
|
365 | + ) |
|
366 | + ) |
|
367 | + ->willReturn(null); |
|
368 | 368 | |
369 | 369 | $this->target->fail($job, $options); |
370 | 370 | } |
@@ -122,9 +122,9 @@ |
||
122 | 122 | public function testHasPropertyReturnsExpectedResults($target, $expects) |
123 | 123 | { |
124 | 124 | $modes = [EntityInterface::PROPERTY_FACILE, |
125 | - EntityInterface::PROPERTY_GETTER, |
|
126 | - EntityInterface::PROPERTY_SETTER, |
|
127 | - EntityInterface::PROPERTY_STRICT]; |
|
125 | + EntityInterface::PROPERTY_GETTER, |
|
126 | + EntityInterface::PROPERTY_SETTER, |
|
127 | + EntityInterface::PROPERTY_STRICT]; |
|
128 | 128 | |
129 | 129 | foreach ($modes as $mode) { |
130 | 130 | $actual = $target->hasProperty('testProperty', $mode); |
@@ -64,7 +64,7 @@ discard block |
||
64 | 64 | public function testCloneCreatesNewResourceCollection() |
65 | 65 | { |
66 | 66 | $resource = $this->getMockBuilder('\Core\Entity\PermissionsResourceInterface') |
67 | - ->getMockForAbstractClass(); |
|
67 | + ->getMockForAbstractClass(); |
|
68 | 68 | |
69 | 69 | $target1 = new Permissions(); |
70 | 70 | $target1->grant($resource, Permissions::PERMISSION_ALL); |
@@ -121,23 +121,23 @@ discard block |
||
121 | 121 | $allParams = array($user, PermissionsInterface::PERMISSION_ALL); |
122 | 122 | |
123 | 123 | $target = $this->getMockBuilder('\Core\Entity\Permissions')->disableOriginalConstructor() |
124 | - ->setMethods(array('isGranted', 'grant', 'revoke'))->getMock(); |
|
124 | + ->setMethods(array('isGranted', 'grant', 'revoke'))->getMock(); |
|
125 | 125 | |
126 | 126 | |
127 | 127 | $target->expects($this->exactly(4)) |
128 | - ->method('isGranted') |
|
129 | - ->withConsecutive($viewParams, $changeParams, $noneParams, $allParams) |
|
130 | - ->willReturn(null); |
|
128 | + ->method('isGranted') |
|
129 | + ->withConsecutive($viewParams, $changeParams, $noneParams, $allParams) |
|
130 | + ->willReturn(null); |
|
131 | 131 | |
132 | 132 | $target->expects($this->exactly(4)) |
133 | - ->method('grant') |
|
134 | - ->withConsecutive($viewParams, $changeParams, $noneParams, $allParams) |
|
135 | - ->willReturn(null); |
|
133 | + ->method('grant') |
|
134 | + ->withConsecutive($viewParams, $changeParams, $noneParams, $allParams) |
|
135 | + ->willReturn(null); |
|
136 | 136 | |
137 | 137 | $target->expects($this->exactly(4)) |
138 | - ->method('revoke') |
|
139 | - ->withConsecutive($viewParams, $changeParams, $noneParams, $allParams) |
|
140 | - ->willReturn(null); |
|
138 | + ->method('revoke') |
|
139 | + ->withConsecutive($viewParams, $changeParams, $noneParams, $allParams) |
|
140 | + ->willReturn(null); |
|
141 | 141 | |
142 | 142 | /* Test starts here */ |
143 | 143 | |
@@ -167,11 +167,11 @@ discard block |
||
167 | 167 | $target = new Permissions(); |
168 | 168 | |
169 | 169 | foreach (array( |
170 | - PermissionsInterface::PERMISSION_VIEW => array(true, false, false, false), |
|
171 | - PermissionsInterface::PERMISSION_CHANGE => array(true, true, true, false), |
|
172 | - PermissionsInterface::PERMISSION_ALL => array(true, true, true, false), |
|
173 | - PermissionsInterface::PERMISSION_NONE => array(false, false, false, true), |
|
174 | - ) as $perm => $expected |
|
170 | + PermissionsInterface::PERMISSION_VIEW => array(true, false, false, false), |
|
171 | + PermissionsInterface::PERMISSION_CHANGE => array(true, true, true, false), |
|
172 | + PermissionsInterface::PERMISSION_ALL => array(true, true, true, false), |
|
173 | + PermissionsInterface::PERMISSION_NONE => array(false, false, false, true), |
|
174 | + ) as $perm => $expected |
|
175 | 175 | ) { |
176 | 176 | $target->grant($resource, $perm); |
177 | 177 | $this->assertEquals($expected[0], $target->isGranted($resource, PermissionsInterface::PERMISSION_VIEW)); |
@@ -302,20 +302,20 @@ discard block |
||
302 | 302 | public function testGrantWorksWhenPassingPermissionsResourceInterfaces() |
303 | 303 | { |
304 | 304 | $resource = $this->getMockBuilder('\Core\Entity\PermissionsResourceInterface') |
305 | - ->getMockForAbstractClass(); |
|
305 | + ->getMockForAbstractClass(); |
|
306 | 306 | |
307 | 307 | $resource->expects($this->any()) |
308 | - ->method('getPermissionsResourceId')->willReturn('resource'); |
|
308 | + ->method('getPermissionsResourceId')->willReturn('resource'); |
|
309 | 309 | |
310 | 310 | $userIds1 = array('user1', 'user2'); |
311 | 311 | $userIds2 = array('view' => array('user1', 'user2'), 'all' => array('user3', 'user4')); |
312 | 312 | $resource->expects($this->exactly(4)) |
313 | - ->method('getPermissionsUserIds') |
|
314 | - ->will($this->onConsecutiveCalls( |
|
315 | - $userIds1, |
|
316 | - $userIds2, |
|
317 | - array(), |
|
318 | - null |
|
313 | + ->method('getPermissionsUserIds') |
|
314 | + ->will($this->onConsecutiveCalls( |
|
315 | + $userIds1, |
|
316 | + $userIds2, |
|
317 | + array(), |
|
318 | + null |
|
319 | 319 | )); |
320 | 320 | |
321 | 321 | |
@@ -383,9 +383,9 @@ discard block |
||
383 | 383 | |
384 | 384 | /* @var $target \Core\Entity\PermissionsInterface|\PHPUnit_Framework_MockObject_MockObject */ |
385 | 385 | $target = $this->getMockBuilder('\Core\Entity\Permissions') |
386 | - ->disableOriginalConstructor() |
|
387 | - ->setMethods(array('grant', 'isAssigned')) |
|
388 | - ->getMock(); |
|
386 | + ->disableOriginalConstructor() |
|
387 | + ->setMethods(array('grant', 'isAssigned')) |
|
388 | + ->getMock(); |
|
389 | 389 | |
390 | 390 | if (PermissionsInterface::PERMISSION_NONE == $perm) { |
391 | 391 | $target->expects($this->never())->method('isAssigned'); |
@@ -400,7 +400,7 @@ discard block |
||
400 | 400 | : PermissionsInterface::PERMISSION_NONE; |
401 | 401 | |
402 | 402 | $target->expects($this->once())->method('grant')->with($resource, $expPerm, $build) |
403 | - ->will($this->returnSelf()); |
|
403 | + ->will($this->returnSelf()); |
|
404 | 404 | } |
405 | 405 | |
406 | 406 | /* Test start here */ |
@@ -455,11 +455,11 @@ discard block |
||
455 | 455 | } |
456 | 456 | |
457 | 457 | foreach (array( |
458 | - PermissionsInterface::PERMISSION_NONE, |
|
459 | - PermissionsInterface::PERMISSION_VIEW, |
|
460 | - PermissionsInterface::PERMISSION_CHANGE, |
|
461 | - PermissionsInterface::PERMISSION_ALL |
|
462 | - ) as $i => $perm |
|
458 | + PermissionsInterface::PERMISSION_NONE, |
|
459 | + PermissionsInterface::PERMISSION_VIEW, |
|
460 | + PermissionsInterface::PERMISSION_CHANGE, |
|
461 | + PermissionsInterface::PERMISSION_ALL |
|
462 | + ) as $i => $perm |
|
463 | 463 | ) { |
464 | 464 | $this->assertEquals($expected[$i], $target->isGranted($user, $perm)); |
465 | 465 | } |
@@ -476,13 +476,13 @@ discard block |
||
476 | 476 | public function testClear() |
477 | 477 | { |
478 | 478 | $resource = $this->getMockBuilder('\Core\Entity\PermissionsResourceInterface') |
479 | - ->getMockForAbstractClass(); |
|
479 | + ->getMockForAbstractClass(); |
|
480 | 480 | |
481 | 481 | $resource->expects($this->any()) |
482 | - ->method('getPermissionsResourceId')->willReturn('resource'); |
|
482 | + ->method('getPermissionsResourceId')->willReturn('resource'); |
|
483 | 483 | |
484 | 484 | $resource->expects($this->exactly(1))->method('getPermissionsUserIds') |
485 | - ->willReturn(array('test')); |
|
485 | + ->willReturn(array('test')); |
|
486 | 486 | |
487 | 487 | $target = new Permissions(); |
488 | 488 | $target->grant('test', Permissions::PERMISSION_ALL); |
@@ -516,13 +516,13 @@ discard block |
||
516 | 516 | public function testInherit() |
517 | 517 | { |
518 | 518 | $resource = $this->getMockBuilder('\Core\Entity\PermissionsResourceInterface') |
519 | - ->getMockForAbstractClass(); |
|
519 | + ->getMockForAbstractClass(); |
|
520 | 520 | |
521 | 521 | $resource->expects($this->any()) |
522 | - ->method('getPermissionsResourceId')->willReturn('resource'); |
|
522 | + ->method('getPermissionsResourceId')->willReturn('resource'); |
|
523 | 523 | |
524 | 524 | $resource->expects($this->any())->method('getPermissionsUserIds') |
525 | - ->willReturn(array('test')); |
|
525 | + ->willReturn(array('test')); |
|
526 | 526 | |
527 | 527 | $target1 = new Permissions(); |
528 | 528 | $target2 = new Permissions(); |
@@ -618,7 +618,7 @@ discard block |
||
618 | 618 | public function testGetResourcesReturnsCollectionOfResources() |
619 | 619 | { |
620 | 620 | $resource = $this->getMockBuilder('\Core\Entity\PermissionsResourceInterface') |
621 | - ->getMockForAbstractClass(); |
|
621 | + ->getMockForAbstractClass(); |
|
622 | 622 | |
623 | 623 | $target = new Permissions(); |
624 | 624 | $target->grant($resource, PermissionsInterface::PERMISSION_ALL); |
@@ -641,13 +641,13 @@ discard block |
||
641 | 641 | $grantedResource->expects($this->atLeastOnce()) |
642 | 642 | ->method('getPermissionsUserIds') |
643 | 643 | ->will($this->onConsecutiveCalls(array( |
644 | - 'all' => array('user1', 'user2'), |
|
645 | - 'view' => array('popel', 'dopel'), |
|
646 | - ), array('all' => array('user3')))); |
|
644 | + 'all' => array('user1', 'user2'), |
|
645 | + 'view' => array('popel', 'dopel'), |
|
646 | + ), array('all' => array('user3')))); |
|
647 | 647 | $grantedResource->method('getPermissionsResourceId')->willReturn('grantResource'); |
648 | 648 | |
649 | 649 | $ungrantedResource = $this->getMockBuilder('\Core\Entity\PermissionsResourceInterface') |
650 | - ->getMockForAbstractClass(); |
|
650 | + ->getMockForAbstractClass(); |
|
651 | 651 | $ungrantedResource->method('getPermissionsResourceId')->willReturn('ungrant'); |
652 | 652 | |
653 | 653 | $target = new Permissions(); |
@@ -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 | } |
@@ -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 | } |
@@ -282,10 +282,10 @@ |
||
282 | 282 | 'return' => isset($methodSpec['@return']) |
283 | 283 | ? $call((array) $methodSpec['@return']) |
284 | 284 | : ( |
285 | - isset($methodSpec['return']) |
|
285 | + isset($methodSpec['return']) |
|
286 | 286 | ? ('__self__' === $methodSpec['return'] ? $this->returnSelf() : $this->returnValue($methodSpec['return'])) |
287 | 287 | : null |
288 | - ), |
|
288 | + ), |
|
289 | 289 | ]; |
290 | 290 | } |
291 | 291 | } |
@@ -185,7 +185,7 @@ discard block |
||
185 | 185 | $errTmpl = __METHOD__ . ': ' . get_class($this); |
186 | 186 | if (!property_exists($this, 'target') || (!is_object($this->target) && !isset($spec['target']))) { |
187 | 187 | throw new PHPUnitException($errTmpl |
188 | - . ' must define the property "target" and the value must be an object.'); |
|
188 | + . ' must define the property "target" and the value must be an object.'); |
|
189 | 189 | } |
190 | 190 | |
191 | 191 | if (!is_array($spec)) { |
@@ -399,7 +399,7 @@ discard block |
||
399 | 399 | } |
400 | 400 | |
401 | 401 | $err = __TRAIT__ . ': ' . get_class($this) . ': Setter ' . get_class($this->target) . '::' . $setter |
402 | - . ($expect === $this->target ? ' breaks fluent interface.' : ' does not return expected value.'); |
|
402 | + . ($expect === $this->target ? ' breaks fluent interface.' : ' does not return expected value.'); |
|
403 | 403 | |
404 | 404 | |
405 | 405 | if (false === $args) { |