Test Failed
Pull Request — master (#896)
by Alexander
09:18
created
src/Security/tests/PermissionManagerTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -90,8 +90,8 @@
 block discarded – undo
90 90
         $this->assertEquals($allowRule, $manager->getRule(static::ROLE, static::PERMISSION));
91 91
 
92 92
         // test pattern permission
93
-        $this->assertEquals($manager, $manager->associate(static::ROLE, static::PERMISSION . '.*', AllowRule::class));
94
-        $this->assertEquals($allowRule, $manager->getRule(static::ROLE, static::PERMISSION . '.' . static::PERMISSION));
93
+        $this->assertEquals($manager, $manager->associate(static::ROLE, static::PERMISSION.'.*', AllowRule::class));
94
+        $this->assertEquals($allowRule, $manager->getRule(static::ROLE, static::PERMISSION.'.'.static::PERMISSION));
95 95
 
96 96
         $this->assertEquals($manager, $manager->deassociate(static::ROLE, static::PERMISSION));
97 97
         $this->assertEquals($forbidRule, $manager->getRule(static::ROLE, static::PERMISSION));
Please login to merge, or discard this patch.
src/Security/tests/Traits/GuardedTraitTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -100,7 +100,7 @@
 block discarded – undo
100 100
         $this->assertEquals(static::OPERATION, $guarded->resolvePermission(static::OPERATION));
101 101
 
102 102
         $guarded = new GuardedWithNamespace();
103
-        $resolvedPermission = GuardedWithNamespace::GUARD_NAMESPACE . '.' . static::OPERATION;
103
+        $resolvedPermission = GuardedWithNamespace::GUARD_NAMESPACE.'.'.static::OPERATION;
104 104
         $this->assertEquals($resolvedPermission, $guarded->resolvePermission(static::OPERATION));
105 105
     }
106 106
 }
Please login to merge, or discard this patch.
src/Security/tests/RuleManagerTest.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@
 block discarded – undo
60 60
         // other rule types
61 61
         $manager->set('RuleInterface', $this->rule);
62 62
         $this->assertEquals($this->rule, $manager->get('RuleInterface'));
63
-        $manager->set('Closure', function () {
63
+        $manager->set('Closure', function (){
64 64
             return true;
65 65
         });
66 66
         $this->assertTrue($manager->get('Closure') instanceof CallableRule);
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -60,7 +60,8 @@
 block discarded – undo
60 60
         // other rule types
61 61
         $manager->set('RuleInterface', $this->rule);
62 62
         $this->assertEquals($this->rule, $manager->get('RuleInterface'));
63
-        $manager->set('Closure', function () {
63
+        $manager->set('Closure', function ()
64
+        {
64 65
             return true;
65 66
         });
66 67
         $this->assertTrue($manager->get('Closure') instanceof CallableRule);
Please login to merge, or discard this patch.
src/Security/src/Guard.php 2 patches
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -48,8 +48,10 @@  discard block
 block discarded – undo
48 48
     public function allows(string $permission, array $context = []): bool
49 49
     {
50 50
         $allows = false;
51
-        foreach ($this->getRoles() as $role) {
52
-            if (!$this->permissions->hasRole($role)) {
51
+        foreach ($this->getRoles() as $role)
52
+        {
53
+            if (!$this->permissions->hasRole($role))
54
+            {
53 55
                 continue;
54 56
             }
55 57
 
@@ -95,7 +97,8 @@  discard block
 block discarded – undo
95 97
      */
96 98
     public function getActor(): ActorInterface
97 99
     {
98
-        if (empty($this->actor)) {
100
+        if (empty($this->actor))
101
+        {
99 102
             throw new GuardException('Unable to get Guard Actor, no value set');
100 103
         }
101 104
 
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -18,14 +18,14 @@  discard block
 block discarded – undo
18 18
         private readonly PermissionsInterface $permissions,
19 19
         private ?ActorInterface $actor = null,
20 20
         private array $roles = []
21
-    ) {
21
+    ){
22 22
     }
23 23
 
24 24
     public function allows(string $permission, array $context = []): bool
25 25
     {
26 26
         $allows = false;
27
-        foreach ($this->getRoles() as $role) {
28
-            if (!$this->permissions->hasRole($role)) {
27
+        foreach ($this->getRoles() as $role){
28
+            if (!$this->permissions->hasRole($role)){
29 29
                 continue;
30 30
             }
31 31
 
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
      */
65 65
     public function getActor(): ActorInterface
66 66
     {
67
-        if (empty($this->actor)) {
67
+        if (empty($this->actor)){
68 68
             throw new GuardException('Unable to get Guard Actor, no value set');
69 69
         }
70 70
 
Please login to merge, or discard this patch.
src/Scaffolder/tests/Command/ConfigTest.php 2 patches
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -115,17 +115,17 @@  discard block
 block discarded – undo
115 115
         ];
116 116
 
117 117
         $reflectionMethods = [];
118
-        foreach ($reflection->getMethods() as $method) {
119
-            if ($method->getDeclaringClass()->name !== $reflection->name) {
118
+        foreach ($reflection->getMethods() as $method){
119
+            if ($method->getDeclaringClass()->name !== $reflection->name){
120 120
                 continue;
121 121
             }
122 122
 
123 123
             $reflectionMethods[$method->name] = $method;
124 124
             $this->assertArrayHasKey($method->name, $methods);
125 125
 
126
-            if (!$method->hasReturnType()) {
126
+            if (!$method->hasReturnType()){
127 127
                 $this->assertNull($methods[$method->name]['hint']);
128
-            } else {
128
+            }else{
129 129
                 $this->assertEquals($methods[$method->name]['hint'], $method->getReturnType()->getName());
130 130
             }
131 131
 
@@ -169,8 +169,8 @@  discard block
 block discarded – undo
169 169
         ];
170 170
 
171 171
         $reflectionMethods = [];
172
-        foreach ($reflection->getMethods() as $method) {
173
-            if ($method->getDeclaringClass()->name !== $reflection->name) {
172
+        foreach ($reflection->getMethods() as $method){
173
+            if ($method->getDeclaringClass()->name !== $reflection->name){
174 174
                 continue;
175 175
             }
176 176
             $reflectionMethods[$method->name] = $method;
@@ -239,7 +239,7 @@  discard block
 block discarded – undo
239 239
 
240 240
         clearstatcache();
241 241
 
242
-        $filename = $this->app->directory('config') . "$name.php";
242
+        $filename = $this->app->directory('config')."$name.php";
243 243
         $this->assertFileExists($filename);
244 244
 
245 245
         return $filename;
Please login to merge, or discard this patch.
Braces   +13 added lines, -6 removed lines patch added patch discarded remove patch
@@ -115,17 +115,22 @@  discard block
 block discarded – undo
115 115
         ];
116 116
 
117 117
         $reflectionMethods = [];
118
-        foreach ($reflection->getMethods() as $method) {
119
-            if ($method->getDeclaringClass()->name !== $reflection->name) {
118
+        foreach ($reflection->getMethods() as $method)
119
+        {
120
+            if ($method->getDeclaringClass()->name !== $reflection->name)
121
+            {
120 122
                 continue;
121 123
             }
122 124
 
123 125
             $reflectionMethods[$method->name] = $method;
124 126
             $this->assertArrayHasKey($method->name, $methods);
125 127
 
126
-            if (!$method->hasReturnType()) {
128
+            if (!$method->hasReturnType())
129
+            {
127 130
                 $this->assertNull($methods[$method->name]['hint']);
128
-            } else {
131
+            }
132
+            else
133
+            {
129 134
                 $this->assertEquals($methods[$method->name]['hint'], $method->getReturnType()->getName());
130 135
             }
131 136
 
@@ -169,8 +174,10 @@  discard block
 block discarded – undo
169 174
         ];
170 175
 
171 176
         $reflectionMethods = [];
172
-        foreach ($reflection->getMethods() as $method) {
173
-            if ($method->getDeclaringClass()->name !== $reflection->name) {
177
+        foreach ($reflection->getMethods() as $method)
178
+        {
179
+            if ($method->getDeclaringClass()->name !== $reflection->name)
180
+            {
174 181
                 continue;
175 182
             }
176 183
             $reflectionMethods[$method->name] = $method;
Please login to merge, or discard this patch.
src/Scaffolder/tests/Command/CommandTest.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@
 block discarded – undo
33 33
             'alias'         => $alias,
34 34
             '--description' => 'My sample command description',
35 35
         ];
36
-        if ($alias === null) {
36
+        if ($alias === null){
37 37
             unset($input['alias']);
38 38
         }
39 39
 
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -33,7 +33,8 @@
 block discarded – undo
33 33
             'alias'         => $alias,
34 34
             '--description' => 'My sample command description',
35 35
         ];
36
-        if ($alias === null) {
36
+        if ($alias === null)
37
+        {
37 38
             unset($input['alias']);
38 39
         }
39 40
 
Please login to merge, or discard this patch.
src/Scaffolder/tests/Command/AbstractCommandTest.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -24,11 +24,11 @@
 block discarded – undo
24 24
      */
25 25
     protected function deleteDeclaration(string $class): void
26 26
     {
27
-        if (class_exists($class)) {
28
-            try {
27
+        if (class_exists($class)){
28
+            try{
29 29
                 $reflection = new ReflectionClass($class);
30 30
                 $this->files()->delete($reflection->getFileName());
31
-            } catch (Throwable $exception) {
31
+            }catch (Throwable $exception){
32 32
             }
33 33
         }
34 34
     }
Please login to merge, or discard this patch.
Braces   +7 added lines, -3 removed lines patch added patch discarded remove patch
@@ -24,11 +24,15 @@
 block discarded – undo
24 24
      */
25 25
     protected function deleteDeclaration(string $class): void
26 26
     {
27
-        if (class_exists($class)) {
28
-            try {
27
+        if (class_exists($class))
28
+        {
29
+            try
30
+            {
29 31
                 $reflection = new ReflectionClass($class);
30 32
                 $this->files()->delete($reflection->getFileName());
31
-            } catch (Throwable $exception) {
33
+            }
34
+            catch (Throwable $exception)
35
+            {
32 36
             }
33 37
         }
34 38
     }
Please login to merge, or discard this patch.
src/Scaffolder/tests/App/TestApp.php 2 patches
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -61,11 +61,13 @@
 block discarded – undo
61 61
      */
62 62
     protected function mapDirectories(array $directories): array
63 63
     {
64
-        if (!isset($directories['root'])) {
64
+        if (!isset($directories['root']))
65
+        {
65 66
             throw new Boot\Exception\BootException('Missing required directory `root`.');
66 67
         }
67 68
 
68
-        if (!isset($directories['app'])) {
69
+        if (!isset($directories['app']))
70
+        {
69 71
             $directories['app'] = $directories['root'] . '/';
70 72
         }
71 73
 
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -52,19 +52,19 @@
 block discarded – undo
52 52
      */
53 53
     protected function mapDirectories(array $directories): array
54 54
     {
55
-        if (!isset($directories['root'])) {
55
+        if (!isset($directories['root'])){
56 56
             throw new Boot\Exception\BootException('Missing required directory `root`.');
57 57
         }
58 58
 
59
-        if (!isset($directories['app'])) {
60
-            $directories['app'] = $directories['root'] . '/';
59
+        if (!isset($directories['app'])){
60
+            $directories['app'] = $directories['root'].'/';
61 61
         }
62 62
 
63 63
         return array_merge([
64
-            'vendor'  => $directories['root'] . '/vendor/',
65
-            'runtime' => $directories['root'] . '/runtime/',
66
-            'config'  => $directories['app'] . '/config/',
67
-            'resources' => $directories['root'] . '/resources/',
64
+            'vendor'  => $directories['root'].'/vendor/',
65
+            'runtime' => $directories['root'].'/runtime/',
66
+            'config'  => $directories['app'].'/config/',
67
+            'resources' => $directories['root'].'/resources/',
68 68
         ], $directories);
69 69
     }
70 70
 }
Please login to merge, or discard this patch.
src/Prototype/src/ClassNode/ConflictResolver/AbstractEntity.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@
 block discarded – undo
32 32
     public function fullName(): string
33 33
     {
34 34
         $name = $this->name;
35
-        if ($this->sequence > 0) {
35
+        if ($this->sequence > 0){
36 36
             $name .= $this->sequence;
37 37
         }
38 38
 
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,8 @@
 block discarded – undo
32 32
     public function fullName(): string
33 33
     {
34 34
         $name = $this->name;
35
-        if ($this->sequence > 0) {
35
+        if ($this->sequence > 0)
36
+        {
36 37
             $name .= $this->sequence;
37 38
         }
38 39
 
Please login to merge, or discard this patch.