larriereguichet / AdminBundle

Admin/Admin.php

Doc Comments +3 added lines, -3 removed lines

@@ -156,7 +156,7 @@ 
     /**
      * Check if user is allowed to be here
      *
-     * @param UserInterface|string $user
+     * @param null|\Symfony\Component\Security\Core\User\User $user
      * @throws Exception
      */
     public function checkPermissions($user)
@@ -325,7 +325,7 @@ 
     /**
      * Return loaded entities
      *
-     * @return mixed
+     * @return ArrayCollection
      */
     public function getEntities()
     {
@@ -401,7 +401,7 @@ 
     }
 
     /**
-     * @return array
+     * @return integer[]
      */
     public function getActionNames()
     {

Tests/Base.php

Doc Comments +7 added lines, -7 removed lines

@@ -124,8 +124,8 @@ 
     }
 
     /**
-     * @param $name
-     * @param $configuration
+     * @param string $name
+     * @param \LAG\AdminBundle\Admin\Configuration\AdminConfiguration $configuration
      * @return Admin
      */
     protected function mockAdmin($name, $configuration)
@@ -140,7 +140,7 @@ 
 
     /**
      * @param $name
-     * @return ActionInterface|PHPUnit_Framework_MockObject_MockObject
+     * @return ActionInterface
      */
     protected function mockAction($name)
     {
@@ -242,7 +242,7 @@ 
     /**
      * Return a mock of an entity repository
      *
-     * @return RepositoryInterface|PHPUnit_Framework_MockObject_MockObject
+     * @return RepositoryInterface
      */
     protected function mockEntityRepository()
     {
@@ -280,7 +280,7 @@ 
     }
 
     /**
-     * @return ActionFactory|PHPUnit_Framework_MockObject_MockObject
+     * @return ActionFactory
      */
     protected function mockActionFactory()
     {
@@ -322,7 +322,7 @@ 
     }
 
     /**
-     * @return MessageHandlerInterface|PHPUnit_Framework_MockObject_MockObject
+     * @return MessageHandlerInterface
      */
     protected function mockMessageHandler()
     {
@@ -345,7 +345,7 @@ 
     }
 
     /**
-     * @return DataProviderInterface|PHPUnit_Framework_MockObject_MockObject
+     * @return DataProviderInterface
      */
     protected function mockDataProvider()
     {

Tests/AdminBundle/Admin/AdminTest.php

Spacing +5 added lines, -5 removed lines

@@ -40,13 +40,13 @@ 
 
 
             // with no action, handleRequest method SHOULD throw an exception
-            $this->assertExceptionRaised('Exception', function () use ($admin) {
+            $this->assertExceptionRaised('Exception', function() use ($admin) {
                 $request = new Request();
                 $admin->handleRequest($request);
             });
 
             // with a wrong action, handleRequest method SHOULD throw an exception
-            $this->assertExceptionRaised('Exception', function () use ($admin) {
+            $this->assertExceptionRaised('Exception', function() use ($admin) {
                 $request = new Request([], [], [
                     '_route_params' => [
                         '_action' => 'bad_action'
@@ -75,13 +75,13 @@ 
             $this->doTestAdmin($admin, $configuration, $adminName);
 
             // with a current action unset, checkPermissions method SHOULD throw an exception
-            $this->assertExceptionRaised('Exception', function () use ($admin) {
+            $this->assertExceptionRaised('Exception', function() use ($admin) {
                 $user = new User('JohnKrovitch', 'john1234');
                 $admin->checkPermissions($user);
             });
 
             // with the wrong roles, checkPermissions method SHOULD throw an exception
-            $this->assertExceptionRaised(NotFoundHttpException::class, function () use ($admin) {
+            $this->assertExceptionRaised(NotFoundHttpException::class, function() use ($admin) {
                 $request = new Request([], [], [
                     '_route_params' => [
                         '_action' => 'custom_list'
@@ -93,7 +93,7 @@ 
             });
 
             // with the wrong roles, checkPermissions method SHOULD throw an exception
-            $this->assertExceptionRaised(NotFoundHttpException::class, function () use ($admin) {
+            $this->assertExceptionRaised(NotFoundHttpException::class, function() use ($admin) {
                 $request = new Request([], [], [
                     '_route_params' => [
                         '_action' => 'custom_list'