Completed
Pull Request — master (#361)
by Ramūnas
37:16
created
SecurityBundle/Tests/Unit/Acl/Domain/PermissionGrantingStrategyTest.php 2 patches
Doc Comments   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -346,6 +346,9 @@  discard block
 block discarded – undo
346 346
         );
347 347
     }
348 348
 
349
+    /**
350
+     * @param ObjectIdentity $oid
351
+     */
349 352
     protected function getAcl($oid = null, $entriesInheriting = true)
350 353
     {
351 354
         static $id = 1;
@@ -385,7 +388,7 @@  discard block
 block discarded – undo
385 388
 
386 389
     /**
387 390
      * @param  string     $permission
388
-     * @param  object     $object
391
+     * @param  TestEntity     $object
389 392
      * @return array|null may return null if permission/object combination is not supported
390 393
      */
391 394
     private function getMasks($permission, $object)
@@ -395,7 +398,7 @@  discard block
 block discarded – undo
395 398
 
396 399
     /**
397 400
      * @param  string      $permission
398
-     * @param  mixed       $object
401
+     * @param  TestEntity       $object
399 402
      * @return MaskBuilder
400 403
      */
401 404
     private function getMaskBuilder($permission, $object)
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 namespace Oro\Component\PropertyAccess\Tests\Unit;
4 4
 
5 5
 use Symfony\Component\PropertyAccess\Exception\NoSuchPropertyException;
6
-
7 6
 use Oro\Component\PropertyAccess\PropertyAccessor;
8 7
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClass;
9 8
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClassMagicCall;
Please login to merge, or discard this patch.
Bundle/SecurityBundle/Tests/Unit/Acl/Group/ChainAclGroupProviderTest.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@
 block discarded – undo
65 65
     /**
66 66
      * @param bool $isSupports
67 67
      * @param string $group
68
-     * @return \PHPUnit_Framework_MockObject_MockObject|AclGroupProviderInterface
68
+     * @return AclGroupProviderInterface
69 69
      */
70 70
     protected function getAclGroupProviderMock($isSupports = true, $group = '')
71 71
     {
Please login to merge, or discard this patch.
SecurityBundle/Tests/Unit/Acl/Persistence/AceManipulationHelperTest.php 1 patch
Doc Comments   +5 added lines patch added patch discarded remove patch
@@ -436,6 +436,11 @@
 block discarded – undo
436 436
         );
437 437
     }
438 438
 
439
+    /**
440
+     * @param boolean $granting
441
+     * @param integer $mask
442
+     * @param string $strategy
443
+     */
439 444
     private function getAce(
440 445
         $sid,
441 446
         $granting = null,
Please login to merge, or discard this patch.
SecurityBundle/Tests/Unit/Acl/Persistence/AclPrivilegeRepositoryTest.php 1 patch
Doc Comments   +9 added lines patch added patch discarded remove patch
@@ -352,6 +352,10 @@  discard block
 block discarded – undo
352 352
         $this->assertFalse($p->containsKey('EDIT'));
353 353
     }
354 354
 
355
+    /**
356
+     * @param string $extensionKey
357
+     * @param ObjectIdentity $rootOid
358
+     */
355 359
     private function initSavePrivileges($extensionKey, $rootOid)
356 360
     {
357 361
         $this->extension->expects($this->any())
@@ -364,6 +368,11 @@  discard block
 block discarded – undo
364 368
             ->method('adaptRootMask')
365 369
             ->will(
366 370
                 $this->returnCallback(
371
+
372
+                    /**
373
+                     * @param string $mask
374
+                     * @param ObjectIdentity $object
375
+                     */
367 376
                     function ($mask, $object) {
368 377
                         return $mask;
369 378
                     }
Please login to merge, or discard this patch.
Tests/Unit/DependencyInjection/Extension/SecurityExtensionHelperTest.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@
 block discarded – undo
49 49
 
50 50
     /**
51 51
      * @param array $securityConfig
52
-     * @return \PHPUnit_Framework_MockObject_MockObject|ExtendedContainerBuilder
52
+     * @return ExtendedContainerBuilder
53 53
      */
54 54
     protected function getContainerBuilder(array $securityConfig)
55 55
     {
Please login to merge, or discard this patch.
SecurityBundle/Tests/Unit/Owner/ChainEntityOwnershipDecisionMakerTest.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -173,7 +173,7 @@
 block discarded – undo
173 173
 
174 174
     /**
175 175
      * @param bool $supports
176
-     * @return AccessLevelOwnershipDecisionMakerInterface|\PHPUnit_Framework_MockObject_MockObject
176
+     * @return AccessLevelOwnershipDecisionMakerInterface
177 177
      */
178 178
     protected function getOwnershipDecisionMakerMock($supports = true)
179 179
     {
Please login to merge, or discard this patch.
Oro/Bundle/SecurityBundle/Tests/Unit/Owner/ChainOwnerTreeProviderTest.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -123,7 +123,7 @@
 block discarded – undo
123 123
     }
124 124
 
125 125
     /**
126
-     * @return \PHPUnit_Framework_MockObject_MockObject|OwnerTreeProviderInterface
126
+     * @return OwnerTreeProviderInterface
127 127
      */
128 128
     protected function getProviderMock()
129 129
     {
Please login to merge, or discard this patch.
SecurityBundle/Tests/Unit/Owner/Metadata/ChainMetadataProviderTest.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -175,7 +175,7 @@
 block discarded – undo
175 175
     /**
176 176
      * @param bool $isSupports
177 177
      * @param array $metadata
178
-     * @return MetadataProviderInterface|\PHPUnit_Framework_MockObject_MockObject
178
+     * @return MetadataProviderInterface
179 179
      */
180 180
     protected function getMetadataProviderMock($isSupports = true, array $metadata = [])
181 181
     {
Please login to merge, or discard this patch.
src/Oro/Bundle/SecurityBundle/Tests/Unit/Owner/OwnerTreeProviderTest.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -184,7 +184,7 @@
 block discarded – undo
184 184
 
185 185
     /**
186 186
      * @param object $object
187
-     * @param mixed $value
187
+     * @param integer $value
188 188
      */
189 189
     protected function setId($object, $value)
190 190
     {
Please login to merge, or discard this patch.
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -4,9 +4,7 @@
 block discarded – undo
4 4
 
5 5
 use Doctrine\Common\EventManager;
6 6
 use Doctrine\DBAL\Driver\Connection;
7
-
8 7
 use Symfony\Component\Filesystem\Filesystem;
9
-
10 8
 use Oro\Component\TestUtils\ORM\Mocks\DriverMock;
11 9
 use Oro\Component\TestUtils\ORM\Mocks\EntityManagerMock;
12 10
 
Please login to merge, or discard this patch.