Passed
Push — master ( 45ac95...d04c5a )
by Carsten
12:05
created
module/Organizations/test/OrganizationsTest/Entity/OrganizationTest.php 1 patch
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -61,74 +61,74 @@
 block discarded – undo
61 61
         $parent = new Organization();
62 62
         $permissions = $this->createMock(Permissions::class);
63 63
         return [
64
-            [ 'externalId', '1234' ],
65
-            [ 'name', [ 'value'=> '', 'ignore_setter' => true] ],
66
-            [ 'name', [ 'pre' => 'setupGetName', 'value' => 'Test', 'ignore_setter' => true] ],
67
-            [ 'organizationName' , [
68
-                '@value' => [ '\Organizations\Entity\OrganizationName', [ 'Test' ] ],
64
+            ['externalId', '1234'],
65
+            ['name', ['value'=> '', 'ignore_setter' => true]],
66
+            ['name', ['pre' => 'setupGetName', 'value' => 'Test', 'ignore_setter' => true]],
67
+            ['organizationName', [
68
+                '@value' => ['\Organizations\Entity\OrganizationName', ['Test']],
69 69
                 'post' => 'organizationNamePostAsserts'
70 70
             ]],
71
-            [ 'organizationName', [
71
+            ['organizationName', [
72 72
                 'pre' => 'setupOrganizationName',
73
-                '@value' => [ '\Organizations\Entity\OrganizationName', [ 'TestNew' ] ],
73
+                '@value' => ['\Organizations\Entity\OrganizationName', ['TestNew']],
74 74
                 'post' => 'assertOrganizationNameCounter'
75 75
             ]],
76
-            [ 'permissions',[
76
+            ['permissions', [
77 77
                 'value' => $permissions
78 78
             ]],
79
-            [ 'image',[
79
+            ['image', [
80 80
                 'value' => new OrganizationImage()
81 81
             ]],
82
-            [ 'images',[
82
+            ['images', [
83 83
                 'value' => new ImageSet(),
84 84
             ]],
85
-            [ 'isDraft',[
85
+            ['isDraft', [
86 86
                 'default' => false,
87 87
                 'value' => true,
88 88
                 'getter_method' => 'isDraft'
89 89
             ]],
90
-            [ 'contact',[
90
+            ['contact', [
91 91
                 'value' => new OrganizationContact()
92 92
             ]],
93
-            [ 'description',[
93
+            ['description', [
94 94
                 'default' => null,
95 95
                 'value' => 'Some Description'
96 96
             ]],
97
-            [ 'parent',[
97
+            ['parent', [
98 98
                 'default' => null,
99 99
                 'value' => $parent,
100 100
             ]],
101
-            [ 'hiringOrganizations',[
101
+            ['hiringOrganizations', [
102 102
                 'default' => null,
103 103
             ]],
104 104
 
105
-            [ 'employees',[
105
+            ['employees', [
106 106
                 '@default' => ArrayCollection::class,
107 107
                 'value' => new ArrayCollection()
108 108
             ]],
109 109
 
110
-            [ 'jobs',[
110
+            ['jobs', [
111 111
                 'default' => null,
112 112
             ]],
113 113
 
114
-            [ 'user',[
114
+            ['user', [
115 115
                 'value' => new User()
116 116
             ]],
117 117
 
118
-            [ 'template',[
118
+            ['template', [
119 119
                 'value' => new Template(),
120 120
             ]],
121 121
 
122
-            [ 'workflowSettings',[
122
+            ['workflowSettings', [
123 123
                 'value' => new WorkflowSettings(),
124 124
             ]],
125 125
 
126
-            [ 'profileSetting',[
126
+            ['profileSetting', [
127 127
                 'value' => Organization::PROFILE_ALWAYS_ENABLE,
128 128
                 'default' => null,
129 129
             ]],
130 130
 
131
-            [ 'hydrator', [
131
+            ['hydrator', [
132 132
                 '@default' => EntityHydrator::class
133 133
             ]]
134 134
 
Please login to merge, or discard this patch.
test/OrganizationsTest/Paginator/ListJobPaginatorFactoryTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -41,8 +41,8 @@
 block discarded – undo
41 41
         $container->expects($this->any())
42 42
             ->method('get')
43 43
             ->willReturnMap([
44
-                ['FilterManager',$filterManager],
45
-                ['repositories',$repositories]
44
+                ['FilterManager', $filterManager],
45
+                ['repositories', $repositories]
46 46
             ])
47 47
         ;
48 48
 
Please login to merge, or discard this patch.
module/Organizations/test/Bootstrap.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@
 block discarded – undo
1 1
 <?php
2 2
 
3
-$testConfig = include __DIR__ . '/TestConfig.php';
3
+$testConfig = include __DIR__.'/TestConfig.php';
4 4
 
5
-require_once __DIR__ . '/../../../test/Bootstrap.php';
5
+require_once __DIR__.'/../../../test/Bootstrap.php';
Please login to merge, or discard this patch.
module/Auth/test/AclTest/Assertion/AssertionManagerFactoryTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@
 block discarded – undo
64 64
 
65 65
         $manager = $target->__invoke($services, AssertionManager::class);
66 66
 
67
-        $this->assertTrue($testResult === $manager->has($testName), 'Expected managers\' has method to return ' . ($testResult ? 'TRUE' : 'FALSE') . ' on ' .$testName);
67
+        $this->assertTrue($testResult === $manager->has($testName), 'Expected managers\' has method to return '.($testResult ? 'TRUE' : 'FALSE').' on '.$testName);
68 68
     }
69 69
 
70 70
     public function provideConfigArrays()
Please login to merge, or discard this patch.
Auth/test/AclTest/Assertion/AbstractEventManagerAwareAssertionTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -156,7 +156,7 @@
 block discarded – undo
156 156
         $self = $this;
157 157
 
158 158
         $events->expects($this->once())->method('triggerUntil')
159
-               ->will($this->returnCallback(function ($callback, $eventName, $event) use ($acl, $role, $resource, $privilege, $self) {
159
+               ->will($this->returnCallback(function($callback, $eventName, $event) use ($acl, $role, $resource, $privilege, $self) {
160 160
                    $self->assertTrue(is_callable($callback));
161 161
                    $self->assertEquals('assert', $eventName);
162 162
                    $self->assertSame($acl, $event->getAcl());
Please login to merge, or discard this patch.
module/Auth/test/AuthTest/Controller/Plugin/UserSwitcherTest.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -45,10 +45,10 @@  discard block
 block discarded – undo
45 45
     private $target = [
46 46
         UserSwitcher::class,
47 47
         'getSimpleAuthMock',
48
-        '@testInheritance' => [ 'as_reflection' => true ],
48
+        '@testInheritance' => ['as_reflection' => true],
49 49
         '@testInvokationProxiesToCorrectMethods' => [
50 50
             'args' => false,
51
-            'mock' => [ 'clear' => 1, 'switchUser' => ['with' => 'testUserId', 'count' => 1]],
51
+            'mock' => ['clear' => 1, 'switchUser' => ['with' => 'testUserId', 'count' => 1]],
52 52
         ],
53 53
         '@testClearRestoresOriginalUserAndClearsSession' => [
54 54
             'args' => 'getComplexAuthMock',
@@ -64,14 +64,14 @@  discard block
 block discarded – undo
64 64
         ],
65 65
     ];
66 66
 
67
-    private $inheritance = [ AbstractPlugin::class ];
67
+    private $inheritance = [AbstractPlugin::class];
68 68
 
69 69
     public function propertiesProvider()
70 70
     {
71
-        $createSession = function () {
71
+        $createSession = function() {
72 72
             $_SESSION[UserSwitcher::SESSION_NAMESPACE]['params'] = ['param' => 'value'];
73 73
         };
74
-        $clearSession = function () {
74
+        $clearSession = function() {
75 75
             $_SESSION = [];
76 76
         };
77 77
         return [
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
             ->disableOriginalConstructor()
114 114
             ->getMock();
115 115
 
116
-        return [ $auth ];
116
+        return [$auth];
117 117
     }
118 118
 
119 119
     private function getComplexAuthMock()
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
 
139 139
         $auth->expects($this->any())->method('getUser')->willReturn(new User());
140 140
 
141
-        return [ $auth ];
141
+        return [$auth];
142 142
     }
143 143
 
144 144
     public function testInvokationProxiesToCorrectMethods()
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
     public function returnReferenceProvider()
156 156
     {
157 157
         return [
158
-            [ null ], [ 'some/ref/uri' ],
158
+            [null], ['some/ref/uri'],
159 159
         ];
160 160
     }
161 161
 
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
         $session = new Container(UserSwitcher::SESSION_NAMESPACE);
170 170
         $session->isSwitchedUser = true;
171 171
         $session->originalUser = 'switchedUser';
172
-        $oldSession      = [
172
+        $oldSession = [
173 173
             'oldSession' => true,
174 174
             'must'       => 'be same'
175 175
         ];
Please login to merge, or discard this patch.
Auth/test/AuthTest/Factory/Controller/Plugin/UserSwitcherFactoryTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -41,11 +41,11 @@
 block discarded – undo
41 41
     private $target = [
42 42
         UserSwitcherFactory::class,
43 43
         '@testCreateServiceInvokesItself' => [
44
-            'mock' => [ '__invoke' ],
44
+            'mock' => ['__invoke'],
45 45
         ],
46 46
     ];
47 47
 
48
-    private $inheritance = [ FactoryInterface::class ];
48
+    private $inheritance = [FactoryInterface::class];
49 49
 
50 50
     public function testCreateServiceInvokesItself()
51 51
     {
Please login to merge, or discard this patch.
module/Auth/test/AuthTest/Listener/DeactivatedUserListenerTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -83,7 +83,7 @@
 block discarded – undo
83 83
             ->getMock();
84 84
         $this->serviceManager->expects($this->any())
85 85
             ->method('get')
86
-            ->will($this->returnCallback(function ($name) {
86
+            ->will($this->returnCallback(function($name) {
87 87
                 switch ($name) {
88 88
                     case 'AuthenticationService':
89 89
                         return $this->auth;
Please login to merge, or discard this patch.
module/Auth/test/AuthTest/Service/RegisterTest.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
     public function testProceed_WhenUserAlreadyExists()
116 116
     {
117 117
         $name = uniqid('name');
118
-        $email = uniqid('email') . '@' . uniqid('host') . '.com.pl';
118
+        $email = uniqid('email').'@'.uniqid('host').'.com.pl';
119 119
         $user = UserEntityProvider::createEntityWithRandomData();
120 120
         $role = 'user';
121 121
 
@@ -160,8 +160,8 @@  discard block
 block discarded – undo
160 160
      */
161 161
     public function testProceed($role)
162 162
     {
163
-        $name = uniqid('name') . ' ' . uniqid('surname');
164
-        $email = uniqid('email') . '@' . uniqid('host') . '.com.pl';
163
+        $name = uniqid('name').' '.uniqid('surname');
164
+        $email = uniqid('email').'@'.uniqid('host').'.com.pl';
165 165
         $user = UserEntityProvider::createEntityWithRandomData();
166 166
         $confirmationLink = uniqid('confirmationLink');
167 167
         $self = $this;
@@ -194,7 +194,7 @@  discard block
 block discarded – undo
194 194
 
195 195
         $this->userRepositoryMock->expects($this->once())
196 196
             ->method('store')
197
-            ->with($this->callback(function ($user) {
197
+            ->with($this->callback(function($user) {
198 198
                 return $user instanceof User ? true:false;
199 199
             }));
200 200
 
Please login to merge, or discard this patch.