Completed
Pull Request — develop (#542)
by Mathias
09:01
created
module/Jobs/test/JobsTest/Controller/AdminCategoriesControllerTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -39,9 +39,9 @@
 block discarded – undo
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)
Please login to merge, or discard this patch.
test/JobsTest/Factory/Repository/DefaultCategoriesBuilderFactoryTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -76,7 +76,7 @@
 block discarded – undo
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
 
Please login to merge, or discard this patch.
module/Jobs/test/JobsTest/Factory/Form/InputFilter/TestJobLocationEdit.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@
 block discarded – undo
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)'));
Please login to merge, or discard this patch.
Jobs/test/JobsTest/Factory/Form/ActiveOrganizationSelectFactoryTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@
 block discarded – undo
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()
Please login to merge, or discard this patch.
module/Jobs/test/JobsTest/Factory/Filter/ChannelPricesFactoryTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@
 block discarded – undo
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()
Please login to merge, or discard this patch.
module/Jobs/test/JobsTest/Listener/MailSenderTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -109,9 +109,9 @@
 block discarded – undo
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(
Please login to merge, or discard this patch.
module/Jobs/test/JobsTest/Listener/GetOrganizationManagersTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -94,7 +94,7 @@
 block discarded – undo
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
Please login to merge, or discard this patch.
module/Jobs/test/JobsTest/Acl/WriteAssertionTest.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -70,13 +70,13 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
         );
Please login to merge, or discard this patch.
module/Jobs/test/JobsTest/Auth/Dependency/ListListenerTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -72,7 +72,7 @@
 block discarded – undo
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);
Please login to merge, or discard this patch.