Passed
Push — master ( c58a40...6d37ad )
by Mathias
10:02 queued 04:27
created
module/Applications/test/ApplicationsTest/Form/FactsFieldsetTest.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 
49 49
     public function testSetGetEmptySummaryText()
50 50
     {
51
-        $input="Klick here";
51
+        $input = "Klick here";
52 52
         $this->target->setEmptySummaryNotice($input);
53 53
         $this->assertSame($this->target->getEmptySummaryNotice(), $input);
54 54
     }
@@ -65,8 +65,8 @@  discard block
 block discarded – undo
65 65
     public function providerDisableCapable()
66 66
     {
67 67
         return [
68
-            [true,true],
69
-            [false,false],
68
+            [true, true],
69
+            [false, false],
70 70
             [null, true]
71 71
         ];
72 72
     }
@@ -83,8 +83,8 @@  discard block
 block discarded – undo
83 83
     public function providerDisableElementCapable()
84 84
     {
85 85
         return [
86
-            [true,true],
87
-            [false,false],
86
+            [true, true],
87
+            [false, false],
88 88
             [null, true]
89 89
         ];
90 90
     }
Please login to merge, or discard this patch.
module/Applications/test/ApplicationsTest/Form/Element/JobSelectTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
         ]]
47 47
     ];
48 48
 
49
-    private $inheritance = [ Select::class, HeadscriptProviderInterface::class ];
49
+    private $inheritance = [Select::class, HeadscriptProviderInterface::class];
50 50
 
51 51
     public function propertiesProvider()
52 52
     {
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
                 'getter_method' => 'getValue',
64 64
                 'expect' => $job->getId(),
65 65
                 'value' => $job,
66
-                'post' => function () use ($job) {
66
+                'post' => function() use ($job) {
67 67
                     $this->assertEquals([0 => '', $job->getId() => $job->getTitle()], $this->target->getValueOptions());
68 68
                 }
69 69
             ]]
Please login to merge, or discard this patch.
module/Applications/test/ApplicationsTest/Form/ApplicationsFilterTest.php 2 patches
Indentation   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -84,15 +84,15 @@
 block discarded – undo
84 84
         ];
85 85
 
86 86
         $add3 = ['type' => 'ToggleButton',
87
-                 'name' => 'unread',
88
-                 'options' => [
89
-                     'checked_value' => '1',
90
-                     'unchecked_value' => '0',
91
-                     'label' => 'unread',
92
-                 ],
93
-                 'attributes' => [
94
-                     'data-submit-on-change' => 'true',
95
-                 ]
87
+                    'name' => 'unread',
88
+                    'options' => [
89
+                        'checked_value' => '1',
90
+                        'unchecked_value' => '0',
91
+                        'label' => 'unread',
92
+                    ],
93
+                    'attributes' => [
94
+                        'data-submit-on-change' => 'true',
95
+                    ]
96 96
         ];
97 97
 
98 98
         /** @noinspection PhpUndefinedMethodInspection */
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
         '@testAddElements' => ['mock' => ['add', 'setButtonElement' => ['with' => 'unread', 'count' => 1]]],
39 39
     ];
40 40
 
41
-    private $inheritance = [ SearchForm::class ];
41
+    private $inheritance = [SearchForm::class];
42 42
 
43 43
     private $attributes = [
44 44
         'options' => [
Please login to merge, or discard this patch.
Applications/test/ApplicationsTest/Acl/ApplicationAccessAssertionTest.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -70,13 +70,13 @@
 block discarded – undo
70 70
         $app  = new Application();
71 71
         $app2 = new Application();
72 72
         $app2->getPermissions()->grant($user, PermissionsInterface::PERMISSION_VIEW)
73
-                               ->grant($user2, PermissionsInterface::PERMISSION_CHANGE);
73
+                                ->grant($user2, PermissionsInterface::PERMISSION_CHANGE);
74 74
 
75 75
         $app3 = new Application();
76 76
         $app3->setIsDraft(true);
77 77
         //$app3->setUser($user);
78 78
         $app3->getPermissions()->grant($user, PermissionsInterface::PERMISSION_VIEW)
79
-                               ->grant($user2, PermissionsInterface::PERMISSION_CHANGE);
79
+                                ->grant($user2, PermissionsInterface::PERMISSION_CHANGE);
80 80
 
81 81
         return array(
82 82
             'nouser-noapp'     => array($role, $resource, null, false),
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -89,8 +89,8 @@
 block discarded – undo
89 89
             'change-not-granted2' => array($user, $app2, 'change', false),
90 90
             'subsequentAttachmentUpload-not-granted' => array($user, $app, Application::PERMISSION_SUBSEQUENT_ATTACHMENT_UPLOAD, false),
91 91
             'subsequentAttachmentUpload-granted' => array($user, $app2, Application::PERMISSION_SUBSEQUENT_ATTACHMENT_UPLOAD, true),
92
-            'allow-draft-view' => [ $user2, $app3, 'change', true ],
93
-            'disallow-draft-view' => [ $user, $app3, 'change', false ],
92
+            'allow-draft-view' => [$user2, $app3, 'change', true],
93
+            'disallow-draft-view' => [$user, $app3, 'change', false],
94 94
 
95 95
         );
96 96
     }
Please login to merge, or discard this patch.
module/Applications/test/TestConfig.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -3,8 +3,8 @@
 block discarded – undo
3 3
 
4 4
 return array(
5 5
     // This should be an array of module namespaces used in the application.
6
-    'modules' => array_merge($commonModules,array(
7
-        'Core', 'Auth', 'Jobs', 'Organizations',  'Applications', 'Cv', 'Settings'
6
+    'modules' => array_merge($commonModules, array(
7
+        'Core', 'Auth', 'Jobs', 'Organizations', 'Applications', 'Cv', 'Settings'
8 8
     )),
9 9
 
10 10
     // These are various options for the listeners attached to the ModuleManager
Please login to merge, or discard this patch.
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@
 block discarded – undo
64 64
     //     ),
65 65
     // )
66 66
 
67
-   // Initial configuration with which to seed the ServiceManager.
68
-   // Should be compatible with Laminas\ServiceManager\Config.
69
-   // 'service_manager' => array(),
67
+    // Initial configuration with which to seed the ServiceManager.
68
+    // Should be compatible with Laminas\ServiceManager\Config.
69
+    // 'service_manager' => array(),
70 70
 );
Please login to merge, or discard this patch.
Organizations/test/OrganizationsTest/Controller/ProfileControllerTest.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 
Please login to merge, or discard this patch.
test/OrganizationsTest/Controller/Plugin/InvitationHandlerTest.php 2 patches
Indentation   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -61,11 +61,11 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -65,14 +65,14 @@  discard block
 block discarded – undo
65 65
                                              ->disableOriginalConstructor()->getMock();
66 66
         }
67 67
         if (in_array('mailerPlugin', $mocks)) {
68
-            $this->mailerPluginMock   = $this->getMockBuilder('\Core\Controller\Plugin\Mailer')
68
+            $this->mailerPluginMock = $this->getMockBuilder('\Core\Controller\Plugin\Mailer')
69 69
                                          ->disableOriginalConstructor()->getMock();
70 70
         }
71 71
 
72 72
         if ($inject) {
73 73
             foreach ($mocks as $key) {
74
-                $setter = 'set' . $key;
75
-                $this->target->$setter($this->{$key . 'Mock'});
74
+                $setter = 'set'.$key;
75
+                $this->target->$setter($this->{$key.'Mock'});
76 76
             }
77 77
         }
78 78
     }
@@ -107,9 +107,9 @@  discard block
 block discarded – undo
107 107
     {
108 108
         $this->setupMocks($mockKey);
109 109
 
110
-        $setter = 'set' . ucfirst($mockKey);
111
-        $getter = 'get' . ucfirst($mockKey);
112
-        $value  = $this->{$mockKey . 'Mock'};
110
+        $setter = 'set'.ucfirst($mockKey);
111
+        $getter = 'get'.ucfirst($mockKey);
112
+        $value  = $this->{$mockKey.'Mock'};
113 113
 
114 114
         $this->assertSame($this->target, $this->target->$setter($value), 'Fluent interface broken!');
115 115
         $this->assertSame($value, $this->target->$getter());
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
      */
124 124
     public function testGetterThrowExceptionIfDependencyMissing($getterName)
125 125
     {
126
-        $getter = "get" . $getterName;
126
+        $getter = "get".$getterName;
127 127
 
128 128
         $this->target->$getter();
129 129
     }
Please login to merge, or discard this patch.
test/OrganizationsTest/Controller/Plugin/AcceptInvitationHandlerTest.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -134,7 +134,7 @@
 block discarded – undo
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
 
Please login to merge, or discard this patch.
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 
49 49
         $this->authMock = $this->getMockBuilder('\Auth\AuthenticationService')->disableOriginalConstructor()->getMock();
50 50
         $this->organizationRepositoryMock = $this->getMockBuilder('Organizations\Repository\Organization')
51
-                                                 ->disableOriginalConstructor()->getMock();
51
+                                                    ->disableOriginalConstructor()->getMock();
52 52
         $this->userRepositoryMock = $this->getMockBuilder('\Auth\Repository\User')->disableOriginalConstructor()->getMock();
53 53
 
54 54
         if (false !== strpos($this->getName(false), 'Setter')) {
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
         $employee->expects($this->once())->method('setStatus')->with(Employee::STATUS_ASSIGNED);
78 78
 
79 79
         $this->organizationMock->expects($this->once())->method('getEmployee')->with($this->userMock->getId())
80
-                               ->willReturn($employee);
80
+                                ->willReturn($employee);
81 81
 
82 82
         $this->organizationMock->expects($this->any())->method('getId')->willReturn('testOrgId');
83 83
         $this->userRepositoryMock->expects($this->once())->method('findByToken')->with('testToken')->willReturn($this->userMock);
@@ -104,9 +104,9 @@  discard block
 block discarded – undo
104 104
                                 ->willReturn($unassignedEmp);
105 105
 
106 106
         $this->organizationRepositoryMock->expects($this->once())
107
-                                         ->method('findPendingOrganizationsByEmployee')
108
-                                         ->with($this->userMock->getId())
109
-                                         ->willReturn(array($sameOrganization, $assignedEmpOrganization, $unassignedEmpOrganization));
107
+                                            ->method('findPendingOrganizationsByEmployee')
108
+                                            ->with($this->userMock->getId())
109
+                                            ->willReturn(array($sameOrganization, $assignedEmpOrganization, $unassignedEmpOrganization));
110 110
 
111 111
         $storageMock = $this->getMockForAbstractClass('\Laminas\Authentication\Storage\StorageInterface');
112 112
         $storageMock->expects($this->once())->method('write')->with($this->userMock->getId());
@@ -177,7 +177,7 @@  discard block
 block discarded – undo
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
 
Please login to merge, or discard this patch.
test/OrganizationsTest/Controller/InviteEmployeeControllerTest.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
         return isset($this->pluginsMockMap[$name]) ? $this->pluginsMockMap[$name] : null;
95 95
     }
96 96
 
97
-    public function retrieveQueryParam($name, $default=null)
97
+    public function retrieveQueryParam($name, $default = null)
98 98
     {
99 99
         return isset($this->queryParams[$name]) ? $this->queryParams[$name] : $default;
100 100
     }
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
 
128 128
     private function setupOrganizationEntityRetrievalMocks($name)
129 129
     {
130
-        $id = $name. "Id";
130
+        $id = $name."Id";
131 131
         $organization = new Organization();
132 132
         $organization->setOrganizationName(new OrganizationName($name));
133 133
 
Please login to merge, or discard this patch.
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -74,8 +74,8 @@
 block discarded – undo
74 74
         $this->target->setPluginManager($plugins);
75 75
 
76 76
         $params = $this->getMockBuilder('\Laminas\Mvc\Controller\Plugin\Params')
77
-                       ->disableOriginalConstructor()
78
-                       ->getMock()
77
+                        ->disableOriginalConstructor()
78
+                        ->getMock()
79 79
         ;
80 80
         $params
81 81
             ->expects($this->any())
Please login to merge, or discard this patch.