@@ -44,15 +44,15 @@ discard block |
||
| 44 | 44 | |
| 45 | 45 | $headscript = $this->getMockBuilder('\Zend\View\Helper\HeadScript')->disableOriginalConstructor()->getMock(); |
| 46 | 46 | $headscript->expects($this->once()) |
| 47 | - ->method('__call') |
|
| 48 | - ->with('appendFile', array('modules/Organizations/js/organizations.employees.js')) |
|
| 49 | - ->willReturn(null); |
|
| 47 | + ->method('__call') |
|
| 48 | + ->with('appendFile', array('modules/Organizations/js/organizations.employees.js')) |
|
| 49 | + ->willReturn(null); |
|
| 50 | 50 | |
| 51 | 51 | $basepath = $this->getMockBuilder('\Zend\View\Helper\BasePath')->disableOriginalConstructor()->getMock(); |
| 52 | 52 | $basepath->expects($this->once()) |
| 53 | - ->method('__invoke') |
|
| 54 | - ->with('modules/Organizations/js/organizations.employees.js') |
|
| 55 | - ->will($this->returnArgument(0)); |
|
| 53 | + ->method('__invoke') |
|
| 54 | + ->with('modules/Organizations/js/organizations.employees.js') |
|
| 55 | + ->will($this->returnArgument(0)); |
|
| 56 | 56 | |
| 57 | 57 | $helpers = $this->getMockBuilder('\Zend\View\HelperPluginManager')->disableOriginalConstructor()->getMock(); |
| 58 | 58 | $helpers->expects($this->exactly(2)) |
@@ -64,12 +64,12 @@ discard block |
||
| 64 | 64 | ->will($this->onConsecutiveCalls($headscript, $basepath)); |
| 65 | 65 | |
| 66 | 66 | $services = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager')->disableOriginalConstructor() |
| 67 | - ->getMock(); |
|
| 67 | + ->getMock(); |
|
| 68 | 68 | |
| 69 | 69 | $services->expects($this->once()) |
| 70 | - ->method('get') |
|
| 71 | - ->with('ViewHelperManager') |
|
| 72 | - ->willReturn($helpers); |
|
| 70 | + ->method('get') |
|
| 71 | + ->with('ViewHelperManager') |
|
| 72 | + ->willReturn($helpers); |
|
| 73 | 73 | |
| 74 | 74 | $fieldset = $target->__invoke($services, 'irrelevant'); |
| 75 | 75 | |
@@ -48,7 +48,7 @@ |
||
| 48 | 48 | ->with('appendFile', array('modules/Organizations/js/organizations.employees.js')) |
| 49 | 49 | ->willReturn(null); |
| 50 | 50 | |
| 51 | - $basepath = $this->getMockBuilder('\Zend\View\Helper\BasePath')->disableOriginalConstructor()->getMock(); |
|
| 51 | + $basepath = $this->getMockBuilder('\Zend\View\Helper\BasePath')->disableOriginalConstructor()->getMock(); |
|
| 52 | 52 | $basepath->expects($this->once()) |
| 53 | 53 | ->method('__invoke') |
| 54 | 54 | ->with('modules/Organizations/js/organizations.employees.js') |
@@ -47,11 +47,11 @@ |
||
| 47 | 47 | |
| 48 | 48 | $repos = $this->getMockBuilder('\Core\Repository\RepositoryService')->disableOriginalConstructor()->getMock(); |
| 49 | 49 | $repos->expects($this->once()) |
| 50 | - ->method('get')->with('Auth/User')->willReturn($users); |
|
| 50 | + ->method('get')->with('Auth/User')->willReturn($users); |
|
| 51 | 51 | |
| 52 | 52 | $services = $this->getMockBuilder('\Zend\ServiceManager\ServiceManager')->disableOriginalConstructor()->getMock(); |
| 53 | 53 | $services->expects($this->once()) |
| 54 | - ->method('get')->with('repositories')->willReturn($repos); |
|
| 54 | + ->method('get')->with('repositories')->willReturn($repos); |
|
| 55 | 55 | |
| 56 | 56 | $fieldset = $target->__invoke($services,'irrelevant'); |
| 57 | 57 | |
@@ -53,7 +53,7 @@ |
||
| 53 | 53 | $services->expects($this->once()) |
| 54 | 54 | ->method('get')->with('repositories')->willReturn($repos); |
| 55 | 55 | |
| 56 | - $fieldset = $target->__invoke($services,'irrelevant'); |
|
| 56 | + $fieldset = $target->__invoke($services, 'irrelevant'); |
|
| 57 | 57 | |
| 58 | 58 | $this->assertInstanceOf('\Organizations\Form\EmployeeFieldset', $fieldset); |
| 59 | 59 | |
@@ -38,7 +38,7 @@ |
||
| 38 | 38 | ])); |
| 39 | 39 | |
| 40 | 40 | $factory = new ODMListenerFactory(); |
| 41 | - $listener = $factory->__invoke($serviceLocator,'irrelevant'); |
|
| 41 | + $listener = $factory->__invoke($serviceLocator, 'irrelevant'); |
|
| 42 | 42 | $this->assertInstanceOf(ODMListener::class, $listener); |
| 43 | 43 | } |
| 44 | 44 | } |
@@ -34,7 +34,7 @@ |
||
| 34 | 34 | ])); |
| 35 | 35 | |
| 36 | 36 | $factory = new ManagerFactory(); |
| 37 | - $listener = $factory->__invoke($serviceLocator,'irrelevant'); |
|
| 37 | + $listener = $factory->__invoke($serviceLocator, 'irrelevant'); |
|
| 38 | 38 | $this->assertInstanceOf(Manager::class, $listener); |
| 39 | 39 | } |
| 40 | 40 | } |
@@ -51,7 +51,7 @@ |
||
| 51 | 51 | ])); |
| 52 | 52 | |
| 53 | 53 | $factory = new ApplicationListenerFactory(); |
| 54 | - $listener = $factory->__invoke($serviceLocator,'irrelevant'); |
|
| 54 | + $listener = $factory->__invoke($serviceLocator, 'irrelevant'); |
|
| 55 | 55 | $this->assertInstanceOf(ApplicationListener::class, $listener); |
| 56 | 56 | } |
| 57 | 57 | } |
@@ -90,7 +90,7 @@ |
||
| 90 | 90 | $view = $this->getMockBuilder(View::class) |
| 91 | 91 | ->getMock(); |
| 92 | 92 | |
| 93 | - $actual = $this->listListener->getItems($user, $view , 10); |
|
| 93 | + $actual = $this->listListener->getItems($user, $view, 10); |
|
| 94 | 94 | |
| 95 | 95 | $this->assertInternalType('array', $actual); |
| 96 | 96 | $this->assertCount($expected, $actual); |
@@ -103,7 +103,7 @@ discard block |
||
| 103 | 103 | $pluginManager->expects($this->any()) |
| 104 | 104 | ->method('get') |
| 105 | 105 | ->willReturnMap([ |
| 106 | - ['params',null,$params] |
|
| 106 | + ['params', null, $params] |
|
| 107 | 107 | ]) |
| 108 | 108 | ; |
| 109 | 109 | |
@@ -126,7 +126,7 @@ discard block |
||
| 126 | 126 | $plugins->expects($this->any()) |
| 127 | 127 | ->method('get') |
| 128 | 128 | ->willReturnMap([ |
| 129 | - ['params',null,$params] |
|
| 129 | + ['params', null, $params] |
|
| 130 | 130 | ]) |
| 131 | 131 | ; |
| 132 | 132 | |
@@ -150,7 +150,7 @@ discard block |
||
| 150 | 150 | $plugins->expects($this->any()) |
| 151 | 151 | ->method('get') |
| 152 | 152 | ->willReturnMap([ |
| 153 | - ['params',null,$params] |
|
| 153 | + ['params', null, $params] |
|
| 154 | 154 | ]) |
| 155 | 155 | ; |
| 156 | 156 | |
@@ -182,7 +182,7 @@ discard block |
||
| 182 | 182 | $pluginManager->expects($this->any()) |
| 183 | 183 | ->method('get') |
| 184 | 184 | ->willReturnMap([ |
| 185 | - ['params',null,$params] |
|
| 185 | + ['params', null, $params] |
|
| 186 | 186 | ]) |
| 187 | 187 | ; |
| 188 | 188 | |
@@ -217,7 +217,7 @@ discard block |
||
| 217 | 217 | $pluginManager->expects($this->any()) |
| 218 | 218 | ->method('get') |
| 219 | 219 | ->willReturnMap([ |
| 220 | - ['params',null,$params] |
|
| 220 | + ['params', null, $params] |
|
| 221 | 221 | ]) |
| 222 | 222 | ; |
| 223 | 223 | |
@@ -61,11 +61,11 @@ discard block |
||
| 61 | 61 | } |
| 62 | 62 | if (in_array('userTokenGenerator', $mocks)) { |
| 63 | 63 | $this->userTokenGeneratorMock = $this->getMockBuilder('\Auth\Service\UserUniqueTokenGenerator') |
| 64 | - ->disableOriginalConstructor()->getMock(); |
|
| 64 | + ->disableOriginalConstructor()->getMock(); |
|
| 65 | 65 | } |
| 66 | 66 | if (in_array('mailerPlugin', $mocks)) { |
| 67 | 67 | $this->mailerPluginMock = $this->getMockBuilder('\Core\Controller\Plugin\Mailer') |
| 68 | - ->disableOriginalConstructor()->getMock(); |
|
| 68 | + ->disableOriginalConstructor()->getMock(); |
|
| 69 | 69 | } |
| 70 | 70 | |
| 71 | 71 | if ($inject) { |
@@ -133,9 +133,9 @@ discard block |
||
| 133 | 133 | $email = 'invalidEmailAddress'; |
| 134 | 134 | $message = 'Email address is invalid.'; |
| 135 | 135 | $this->translatorMock->expects($this->exactly(5)) |
| 136 | - ->method('translate') |
|
| 137 | - ->with($message) |
|
| 138 | - ->will($this->returnArgument(0)); |
|
| 136 | + ->method('translate') |
|
| 137 | + ->with($message) |
|
| 138 | + ->will($this->returnArgument(0)); |
|
| 139 | 139 | |
| 140 | 140 | $expected = array( |
| 141 | 141 | 'ok' => false, |
@@ -160,7 +160,7 @@ discard block |
||
| 160 | 160 | $user->getInfo()->setEmail($email); |
| 161 | 161 | |
| 162 | 162 | $this->userRepositoryMock->expects($this->once())->method('findByEmail')->with($email, null) |
| 163 | - ->willReturn($user); |
|
| 163 | + ->willReturn($user); |
|
| 164 | 164 | $this->userRepositoryMock->expects($this->never())->method('create'); |
| 165 | 165 | $this->userTokenGeneratorMock->expects($this->once())->method('generate')->with($user, 7)->willReturn('testToken'); |
| 166 | 166 | |
@@ -192,7 +192,7 @@ discard block |
||
| 192 | 192 | $user->getInfo()->setEmail($email); |
| 193 | 193 | |
| 194 | 194 | $this->userRepositoryMock->expects($this->once())->method('findByEmail')->with($email, null) |
| 195 | - ->willReturn(null); |
|
| 195 | + ->willReturn(null); |
|
| 196 | 196 | $this->userRepositoryMock->expects($this->once())->method('create')->willReturn($user); |
| 197 | 197 | $this->userTokenGeneratorMock->expects($this->once())->method('generate')->with($user, 7)->willReturn('testToken'); |
| 198 | 198 | |
@@ -227,12 +227,12 @@ discard block |
||
| 227 | 227 | |
| 228 | 228 | $message = 'Sending invitation mail failed.'; |
| 229 | 229 | $this->translatorMock->expects($this->once()) |
| 230 | - ->method('translate') |
|
| 231 | - ->with($message) |
|
| 232 | - ->will($this->returnArgument(0)); |
|
| 230 | + ->method('translate') |
|
| 231 | + ->with($message) |
|
| 232 | + ->will($this->returnArgument(0)); |
|
| 233 | 233 | |
| 234 | 234 | $this->userRepositoryMock->expects($this->once())->method('findByEmail')->with($email, null) |
| 235 | - ->willReturn($user); |
|
| 235 | + ->willReturn($user); |
|
| 236 | 236 | $this->userRepositoryMock->expects($this->never())->method('create'); |
| 237 | 237 | $this->userTokenGeneratorMock->expects($this->once())->method('generate')->with($user, 7)->willReturn('testToken'); |
| 238 | 238 | |
@@ -64,14 +64,14 @@ discard block |
||
| 64 | 64 | ->disableOriginalConstructor()->getMock(); |
| 65 | 65 | } |
| 66 | 66 | if (in_array('mailerPlugin', $mocks)) { |
| 67 | - $this->mailerPluginMock = $this->getMockBuilder('\Core\Controller\Plugin\Mailer') |
|
| 67 | + $this->mailerPluginMock = $this->getMockBuilder('\Core\Controller\Plugin\Mailer') |
|
| 68 | 68 | ->disableOriginalConstructor()->getMock(); |
| 69 | 69 | } |
| 70 | 70 | |
| 71 | 71 | if ($inject) { |
| 72 | 72 | foreach ($mocks as $key) { |
| 73 | - $setter = 'set' . $key; |
|
| 74 | - $this->target->$setter($this->{$key . 'Mock'}); |
|
| 73 | + $setter = 'set'.$key; |
|
| 74 | + $this->target->$setter($this->{$key.'Mock'}); |
|
| 75 | 75 | } |
| 76 | 76 | } |
| 77 | 77 | } |
@@ -106,9 +106,9 @@ discard block |
||
| 106 | 106 | { |
| 107 | 107 | $this->setupMocks($mockKey); |
| 108 | 108 | |
| 109 | - $setter = 'set' . ucfirst($mockKey); |
|
| 110 | - $getter = 'get' . ucfirst($mockKey); |
|
| 111 | - $value = $this->{$mockKey . 'Mock'}; |
|
| 109 | + $setter = 'set'.ucfirst($mockKey); |
|
| 110 | + $getter = 'get'.ucfirst($mockKey); |
|
| 111 | + $value = $this->{$mockKey.'Mock'}; |
|
| 112 | 112 | |
| 113 | 113 | $this->assertSame($this->target, $this->target->$setter($value), 'Fluent interface broken!'); |
| 114 | 114 | $this->assertSame($value, $this->target->$getter()); |
@@ -122,7 +122,7 @@ discard block |
||
| 122 | 122 | */ |
| 123 | 123 | public function testGetterThrowExceptionIfDependencyMissing($getterName) |
| 124 | 124 | { |
| 125 | - $getter = "get" . $getterName; |
|
| 125 | + $getter = "get".$getterName; |
|
| 126 | 126 | |
| 127 | 127 | $this->target->$getter(); |
| 128 | 128 | } |
@@ -142,7 +142,7 @@ discard block |
||
| 142 | 142 | 'message' => $message, |
| 143 | 143 | ); |
| 144 | 144 | |
| 145 | - foreach (array($email, null, '', 0, array()) as $testEmail) { |
|
| 145 | + foreach (array($email, null, '', 0, array()) as $testEmail) { |
|
| 146 | 146 | $result = $this->target->process($testEmail); |
| 147 | 147 | |
| 148 | 148 | $this->assertEquals($expected, $result); |
@@ -47,7 +47,7 @@ discard block |
||
| 47 | 47 | |
| 48 | 48 | $this->authMock = $this->getMockBuilder('\Auth\AuthenticationService')->disableOriginalConstructor()->getMock(); |
| 49 | 49 | $this->organizationRepositoryMock = $this->getMockBuilder('Organizations\Repository\Organization') |
| 50 | - ->disableOriginalConstructor()->getMock(); |
|
| 50 | + ->disableOriginalConstructor()->getMock(); |
|
| 51 | 51 | $this->userRepositoryMock = $this->getMockBuilder('\Auth\Repository\User')->disableOriginalConstructor()->getMock(); |
| 52 | 52 | |
| 53 | 53 | if (false !== strpos($this->getName(false), 'Setter')) { |
@@ -76,7 +76,7 @@ discard block |
||
| 76 | 76 | $employee->expects($this->once())->method('setStatus')->with(Employee::STATUS_ASSIGNED); |
| 77 | 77 | |
| 78 | 78 | $this->organizationMock->expects($this->once())->method('getEmployee')->with($this->userMock->getId()) |
| 79 | - ->willReturn($employee); |
|
| 79 | + ->willReturn($employee); |
|
| 80 | 80 | |
| 81 | 81 | $this->organizationMock->expects($this->any())->method('getId')->willReturn('testOrgId'); |
| 82 | 82 | $this->userRepositoryMock->expects($this->once())->method('findByToken')->with('testToken')->willReturn($this->userMock); |
@@ -103,9 +103,9 @@ discard block |
||
| 103 | 103 | ->willReturn($unassignedEmp); |
| 104 | 104 | |
| 105 | 105 | $this->organizationRepositoryMock->expects($this->once()) |
| 106 | - ->method('findPendingOrganizationsByEmployee') |
|
| 107 | - ->with($this->userMock->getId()) |
|
| 108 | - ->willReturn(array($sameOrganization, $assignedEmpOrganization, $unassignedEmpOrganization)); |
|
| 106 | + ->method('findPendingOrganizationsByEmployee') |
|
| 107 | + ->with($this->userMock->getId()) |
|
| 108 | + ->willReturn(array($sameOrganization, $assignedEmpOrganization, $unassignedEmpOrganization)); |
|
| 109 | 109 | |
| 110 | 110 | $storageMock = $this->getMockForAbstractClass('\Zend\Authentication\Storage\StorageInterface'); |
| 111 | 111 | $storageMock->expects($this->once())->method('write')->with($this->userMock->getId()); |
@@ -177,7 +177,7 @@ discard block |
||
| 177 | 177 | $orgRef = $this->getMockBuilder('\Organizations\Entity\OrganizationReference')->disableOriginalConstructor()->getMock(); |
| 178 | 178 | $orgRef->expects($this->once())->method('hasAssociation')->willReturn(true); |
| 179 | 179 | $orgRef->expects($this->once())->method('getEmployee')->with($this->userMock->getId()) |
| 180 | - ->willReturn($empMock); |
|
| 180 | + ->willReturn($empMock); |
|
| 181 | 181 | |
| 182 | 182 | $this->userMock->setOrganization($orgRef); |
| 183 | 183 | |
@@ -134,7 +134,7 @@ |
||
| 134 | 134 | continue; |
| 135 | 135 | } |
| 136 | 136 | |
| 137 | - $this->fail('Expected exception was not thrown for "' . $method . '"'); |
|
| 137 | + $this->fail('Expected exception was not thrown for "'.$method.'"'); |
|
| 138 | 138 | } |
| 139 | 139 | } |
| 140 | 140 | |