@@ -45,17 +45,17 @@ |
||
| 45 | 45 | )); |
| 46 | 46 | |
| 47 | 47 | $services = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager') |
| 48 | - ->disableOriginalConstructor() |
|
| 49 | - ->getMock(); |
|
| 48 | + ->disableOriginalConstructor() |
|
| 49 | + ->getMock(); |
|
| 50 | 50 | |
| 51 | 51 | $services->expects($this->exactly(3)) |
| 52 | - ->method('get') |
|
| 53 | - ->withConsecutive( |
|
| 54 | - array('Core/MailService'), |
|
| 55 | - array('Jobs/Options'), |
|
| 56 | - array('Core/Options') |
|
| 57 | - ) |
|
| 58 | - ->will($this->onConsecutiveCalls($mailService, $jobsOptions, $coreOptions)); |
|
| 52 | + ->method('get') |
|
| 53 | + ->withConsecutive( |
|
| 54 | + array('Core/MailService'), |
|
| 55 | + array('Jobs/Options'), |
|
| 56 | + array('Core/Options') |
|
| 57 | + ) |
|
| 58 | + ->will($this->onConsecutiveCalls($mailService, $jobsOptions, $coreOptions)); |
|
| 59 | 59 | |
| 60 | 60 | $expectedOptions = array( |
| 61 | 61 | 'siteName' => $coreOptions->getSiteName(), |
@@ -78,21 +78,21 @@ |
||
| 78 | 78 | $this->user = $user; |
| 79 | 79 | |
| 80 | 80 | $auth = $this->getMockBuilder('Auth\AuthenticationService') |
| 81 | - ->disableOriginalConstructor() |
|
| 82 | - ->getMock(); |
|
| 81 | + ->disableOriginalConstructor() |
|
| 82 | + ->getMock(); |
|
| 83 | 83 | |
| 84 | 84 | $auth->expects($this->once()) |
| 85 | - ->method('getUser') |
|
| 86 | - ->willReturn($user); |
|
| 85 | + ->method('getUser') |
|
| 86 | + ->willReturn($user); |
|
| 87 | 87 | |
| 88 | 88 | $services = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager') |
| 89 | - ->disableOriginalConstructor() |
|
| 90 | - ->getMock(); |
|
| 89 | + ->disableOriginalConstructor() |
|
| 90 | + ->getMock(); |
|
| 91 | 91 | |
| 92 | 92 | $services->expects($this->once()) |
| 93 | - ->method('get') |
|
| 94 | - ->with('AuthenticationService') |
|
| 95 | - ->willReturn($auth); |
|
| 93 | + ->method('get') |
|
| 94 | + ->with('AuthenticationService') |
|
| 95 | + ->willReturn($auth); |
|
| 96 | 96 | |
| 97 | 97 | $this->services = $services; |
| 98 | 98 | } |
@@ -48,19 +48,19 @@ discard block |
||
| 48 | 48 | $channelOptions = new ChannelOptions(); |
| 49 | 49 | |
| 50 | 50 | $channelOptions->setCategory('testCat') |
| 51 | - ->setCurrency('€') |
|
| 52 | - ->setDescription('testDesc') |
|
| 53 | - ->setExternalkey('testExtKey') |
|
| 54 | - ->setHeadLine('testHL') |
|
| 55 | - ->setKey('testKey') |
|
| 56 | - ->setLabel('testLabel') |
|
| 57 | - ->setLinkText('testLinkTxt') |
|
| 58 | - ->setLogo('testLogo') |
|
| 59 | - ->setParams(array('test' => 'params')) |
|
| 60 | - ->setPrice('test', 1234) |
|
| 61 | - ->setPublishDuration(10) |
|
| 62 | - ->setRoute('testRoute') |
|
| 63 | - ->setTax(19); |
|
| 51 | + ->setCurrency('€') |
|
| 52 | + ->setDescription('testDesc') |
|
| 53 | + ->setExternalkey('testExtKey') |
|
| 54 | + ->setHeadLine('testHL') |
|
| 55 | + ->setKey('testKey') |
|
| 56 | + ->setLabel('testLabel') |
|
| 57 | + ->setLinkText('testLinkTxt') |
|
| 58 | + ->setLogo('testLogo') |
|
| 59 | + ->setParams(array('test' => 'params')) |
|
| 60 | + ->setPrice('test', 1234) |
|
| 61 | + ->setPublishDuration(10) |
|
| 62 | + ->setRoute('testRoute') |
|
| 63 | + ->setTax(19); |
|
| 64 | 64 | |
| 65 | 65 | $providerOptions->addChannel($channelOptions); |
| 66 | 66 | |
@@ -106,8 +106,8 @@ discard block |
||
| 106 | 106 | ); |
| 107 | 107 | |
| 108 | 108 | $services->expects($this->exactly(3)) |
| 109 | - ->method('get') |
|
| 110 | - ->will($this->returnValueMap($servicesMap)) |
|
| 109 | + ->method('get') |
|
| 110 | + ->will($this->returnValueMap($servicesMap)) |
|
| 111 | 111 | ; |
| 112 | 112 | |
| 113 | 113 | $target = new MultipostingSelectFactory(); |
@@ -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() |
@@ -43,8 +43,8 @@ |
||
| 43 | 43 | $provider=new ProviderOptions(); |
| 44 | 44 | |
| 45 | 45 | $serviceManagerMock = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager') |
| 46 | - ->disableOriginalConstructor() |
|
| 47 | - ->getMock(); |
|
| 46 | + ->disableOriginalConstructor() |
|
| 47 | + ->getMock(); |
|
| 48 | 48 | |
| 49 | 49 | |
| 50 | 50 | $serviceManagerMock |
@@ -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() |
@@ -57,12 +57,12 @@ |
||
| 57 | 57 | ->willReturn($urlHelper); |
| 58 | 58 | |
| 59 | 59 | $serviceManagerMock = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager') |
| 60 | - ->disableOriginalConstructor() |
|
| 61 | - ->getMock(); |
|
| 60 | + ->disableOriginalConstructor() |
|
| 61 | + ->getMock(); |
|
| 62 | 62 | |
| 63 | 63 | $serviceManagerMock->expects($this->once())->method('get') |
| 64 | - ->with('ViewHelperManager') |
|
| 65 | - ->willReturn($helpers); |
|
| 64 | + ->with('ViewHelperManager') |
|
| 65 | + ->willReturn($helpers); |
|
| 66 | 66 | |
| 67 | 67 | $service = $target->__invoke($serviceManagerMock, 'irrelevant'); |
| 68 | 68 | |
@@ -62,12 +62,12 @@ discard block |
||
| 62 | 62 | default: |
| 63 | 63 | $this->mailServiceMock = $this->getMockBuilder('\Core\Mail\MailService')->disableOriginalConstructor()->getMock(); |
| 64 | 64 | $this->mailServiceMock->expects($this->atLeastOnce()) |
| 65 | - ->method('send') |
|
| 66 | - ->with($this->isInstanceOf(HTMLTemplateMessage::class)) |
|
| 65 | + ->method('send') |
|
| 66 | + ->with($this->isInstanceOf(HTMLTemplateMessage::class)) |
|
| 67 | 67 | ; |
| 68 | 68 | $this->mailServiceMock->expects($this->any()) |
| 69 | - ->method('get')->with('htmltemplate') |
|
| 70 | - ->will($this->returnCallback(array($this, 'pushMailMock'))); |
|
| 69 | + ->method('get')->with('htmltemplate') |
|
| 70 | + ->will($this->returnCallback(array($this, 'pushMailMock'))); |
|
| 71 | 71 | |
| 72 | 72 | |
| 73 | 73 | $this->targetOptions = array('siteName' => 'Test', 'adminEmail' => 'test@admin'); |
@@ -106,29 +106,29 @@ discard block |
||
| 106 | 106 | public function testAttachsToAndDetachsFromJobEvents() |
| 107 | 107 | { |
| 108 | 108 | $events = $this->getMockBuilder('\Zend\EventManager\EventManager') |
| 109 | - ->disableOriginalConstructor() |
|
| 110 | - ->getMock(); |
|
| 109 | + ->disableOriginalConstructor() |
|
| 110 | + ->getMock(); |
|
| 111 | 111 | |
| 112 | 112 | $callback1 = [new MailSenderListenerMock(),'listen1']; |
| 113 | 113 | $callback2 = [new MailSenderListenerMock(),'listen2']; |
| 114 | 114 | $callback3 = [new MailSenderListenerMock(),'listen3']; |
| 115 | 115 | $events->expects($this->exactly(3)) |
| 116 | - ->method('attach') |
|
| 117 | - ->withConsecutive( |
|
| 118 | - array(JobEvent::EVENT_JOB_CREATED, array($this->target, 'onJobCreated')), |
|
| 119 | - array(JobEvent::EVENT_JOB_ACCEPTED, array($this->target, 'onJobAccepted')), |
|
| 120 | - array(JobEvent::EVENT_JOB_REJECTED, array($this->target, 'onJobRejected')) |
|
| 121 | - ) |
|
| 122 | - ->will($this->onConsecutiveCalls($callback1, $callback2, $callback3)) |
|
| 116 | + ->method('attach') |
|
| 117 | + ->withConsecutive( |
|
| 118 | + array(JobEvent::EVENT_JOB_CREATED, array($this->target, 'onJobCreated')), |
|
| 119 | + array(JobEvent::EVENT_JOB_ACCEPTED, array($this->target, 'onJobAccepted')), |
|
| 120 | + array(JobEvent::EVENT_JOB_REJECTED, array($this->target, 'onJobRejected')) |
|
| 121 | + ) |
|
| 122 | + ->will($this->onConsecutiveCalls($callback1, $callback2, $callback3)) |
|
| 123 | 123 | ; |
| 124 | 124 | |
| 125 | 125 | $events->expects($this->exactly(3)) |
| 126 | - ->method('detach') |
|
| 127 | - ->withConsecutive( |
|
| 128 | - array($callback1), |
|
| 129 | - array($callback2), |
|
| 130 | - array($callback3) |
|
| 131 | - )->willReturn(true); |
|
| 126 | + ->method('detach') |
|
| 127 | + ->withConsecutive( |
|
| 128 | + array($callback1), |
|
| 129 | + array($callback2), |
|
| 130 | + array($callback3) |
|
| 131 | + )->willReturn(true); |
|
| 132 | 132 | |
| 133 | 133 | $this->target->attach($events); |
| 134 | 134 | $this->assertAttributeEquals(array($callback1, $callback2, $callback3), 'listeners', $this->target); |
@@ -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( |
@@ -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 | ); |