ezsystems / ezpublish-kernel

eZ/Publish/Core/Repository/Tests/Service/Mock/ContentTest.php

Unused Use Statements +1 added lines, -2 removed lines

@@ -14,7 +14,7 @@ 
 use eZ\Publish\API\Repository\ContentTypeService as APIContentTypeService;
 use eZ\Publish\API\Repository\LocationService as APILocationService;
 use eZ\Publish\API\Repository\Exceptions\NotFoundException as APINotFoundException;
-use eZ\Publish\API\Repository\Values\Content\ContentInfo as APIContentInfo;
+use eZ\Publish\API\Repository\Values\Content\ContentInfo;
 use eZ\Publish\API\Repository\Values\ContentType\ContentType as APIContentType;
 use eZ\Publish\API\Repository\Values\Content\Location as APILocation;
 use eZ\Publish\API\Repository\Values\ContentType\FieldDefinition as APIFieldDefinition;
@@ -33,7 +33,6 @@ 
 use eZ\Publish\Core\Repository\Helper\NameSchemaService;
 use eZ\Publish\API\Repository\Values\Content\Field;
 use eZ\Publish\Core\FieldType\Value;
-use eZ\Publish\API\Repository\Values\Content\ContentInfo;
 use eZ\Publish\API\Repository\Values\Content\VersionInfo as APIVersionInfo;
 use eZ\Publish\Core\Repository\Values\ContentType\ContentType;
 use eZ\Publish\Core\Repository\Values\ContentType\FieldDefinition;

Doc Comments +12 added lines, -5 removed lines

@@ -1082,7 +1082,7 @@ 
      * @param string $mainLanguageCode
      * @param \eZ\Publish\API\Repository\Values\Content\Field[] $structFields
      * @param \eZ\Publish\API\Repository\Values\ContentType\FieldDefinition[] $fieldDefinitions
-     * @param array $languageCodes
+     * @param string[] $languageCodes
      *
      * @return array
      *
@@ -1158,10 +1158,9 @@ 
      * @param \eZ\Publish\SPI\Persistence\Content\Field[] $spiFields
      * @param \eZ\Publish\API\Repository\Values\ContentType\FieldDefinition[] $fieldDefinitions
      * @param \eZ\Publish\API\Repository\Values\Content\LocationCreateStruct[] $locationCreateStructs
-     * @param \eZ\Publish\SPI\Persistence\Content\ObjectState\Group[] $objectStateGroups
      * @param bool $execute
      *
-     * @return mixed
+     * @return ContentCreateStruct
      */
     protected function assertForTestCreateContentNonRedundantFieldSet(
         $mainLanguageCode,
@@ -2045,7 +2044,7 @@ 
      * @param \eZ\Publish\API\Repository\Values\Content\Field[] $structFields
      * @param \eZ\Publish\API\Repository\Values\ContentType\FieldDefinition[] $fieldDefinitions
      *
-     * @return mixed
+     * @return ContentCreateStruct
      */
     protected function assertForTestCreateContentRequiredField(
         $mainLanguageCode,
@@ -4873,6 +4872,10 @@ 
         );
     }
 
+    /**
+     * @param Field[] $existingFields
+     * @param FieldDefinition[] $fieldDefinitions
+     */
     public function assertForTestUpdateContentRequiredField(
         $initialLanguageCode,
         $structFields,
@@ -5081,6 +5084,10 @@ 
         }
     }
 
+    /**
+     * @param Field[] $existingFields
+     * @param FieldDefinition[] $fieldDefinitions
+     */
     public function assertForTestUpdateContentThrowsContentFieldValidationException(
         $initialLanguageCode,
         $structFields,
@@ -5820,7 +5827,7 @@ 
      * @param array $translations
      * @param bool $useAlwaysAvailable
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\API\Repository\Values\Content\Content
+     * @return APIContent
      */
     private function mockBuildContentDomainObject(SPIContent $spiContent, array $translations = null, bool $useAlwaysAvailable = null)
     {

eZ/Publish/Core/Repository/Tests/Service/Mock/DomainMapperTest.php

Doc Comments +3 added lines, -3 removed lines

@@ -177,7 +177,7 @@ 
     }
 
     /**
-     * @return \eZ\Publish\SPI\Persistence\Content\Handler|\PHPUnit\Framework\MockObject\MockObject
+     * @return \eZ\Publish\SPI\Persistence\Content\Handler
      */
     protected function getContentHandlerMock()
     {
@@ -185,7 +185,7 @@ 
     }
 
     /**
-     * @return \eZ\Publish\SPI\Persistence\Content\Language\Handler|\PHPUnit\Framework\MockObject\MockObject
+     * @return \eZ\Publish\SPI\Persistence\Content\Language\Handler
      */
     protected function getLanguageHandlerMock()
     {
@@ -193,7 +193,7 @@ 
     }
 
     /**
-     * @return \eZ\Publish\SPI\Persistence\Content\Type\Handler|\PHPUnit\Framework\MockObject\MockObject
+     * @return \eZ\Publish\SPI\Persistence\Content\Type\Handler
      */
     protected function getTypeHandlerMock()
     {

Unused Use Statements +1 added lines, -2 removed lines

@@ -17,11 +17,10 @@
 use eZ\Publish\Core\Repository\Helper\DomainMapper;
 use eZ\Publish\Core\Repository\Values\Content\Content;
 use eZ\Publish\Core\Repository\Values\Content\VersionInfo;
-use eZ\Publish\SPI\Persistence\Content\ContentInfo;
+use eZ\Publish\SPI\Persistence\Content\ContentInfo as SPIContentInfo;
 use eZ\Publish\SPI\Persistence\Content\Location;
 use eZ\Publish\API\Repository\Values\Content\Location as APILocation;
 use eZ\Publish\SPI\Persistence\Content\VersionInfo as SPIVersionInfo;
-use eZ\Publish\SPI\Persistence\Content\ContentInfo as SPIContentInfo;
 
 /**
  * Mock test case for internal DomainMapper.

eZ/Publish/Core/Search/Tests/FieldNameResolverTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -769,7 +769,7 @@
     }
 
     /**
-     * @param array $methods
+     * @param string[] $methods
      *
      * @return \eZ\Publish\Core\Search\Common\FieldNameResolver|\PHPUnit\Framework\MockObject\MockObject
      */

eZ/Publish/Core/Persistence/Legacy/Tests/User/UserHandlerTest.php

Doc Comments +3 added lines

@@ -46,6 +46,9 @@
         return $user;
     }
 
+    /**
+     * @param integer $time
+     */
     protected function getValidUserToken($time = null)
     {
         $userToken = new Persistence\User\UserTokenUpdateStruct();

eZ/Bundle/EzPublishCoreBundle/Tests/Imagine/PlaceholderAliasGeneratorTest.php

Unused Use Statements -1 removed lines

@@ -15,7 +15,6 @@
 use eZ\Publish\API\Repository\Values\Content\VersionInfo as APIVersionInfo;
 use eZ\Publish\Core\FieldType\Image\Value as ImageValue;
 use eZ\Publish\Core\FieldType\Null\Value as NullValue;
-use eZ\Publish\Core\FieldType\Value as FieldTypeValue;
 use eZ\Publish\Core\FieldType\Value;
 use eZ\Publish\Core\IO\IOServiceInterface;
 use eZ\Publish\Core\IO\Values\BinaryFileCreateStruct;

eZ/Publish/Core/Repository/Tests/Service/Mock/UrlAliasTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -3803,7 +3803,7 @@
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|\eZ\Publish\Core\Repository\Helper\NameSchemaService
+     * @return NameSchemaService
      */
     protected function getNameSchemaServiceMock()
     {

eZ/Publish/Core/Search/Legacy/Tests/Content/AbstractTestCase.php

Doc Comments +1 added lines

@@ -56,6 +56,7 @@
 
     /**
      * Assert that the elements are.
+     * @param \eZ\Publish\API\Repository\Values\Content\Search\SearchResult $searchResult
      */
     protected function assertSearchResults($expectedIds, $searchResult)
     {

eZ/Publish/Core/Search/Legacy/Tests/Content/HandlerContentSortTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -93,7 +93,7 @@ 
     /**
      * Returns a content mapper mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Mapper
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getContentMapperMock()
     {
@@ -149,7 +149,7 @@ 
     /**
      * Returns a content field handler mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\FieldHandler
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getContentFieldHandlerMock()
     {

eZ/Publish/Core/Search/Legacy/Tests/Content/HandlerLocationSortTest.php

Doc Comments +8 added lines, -1 removed lines

@@ -28,9 +28,16 @@ 
  */
 class HandlerLocationSortTest extends AbstractTestCase
 {
+    /**
+     * @param \eZ\Publish\API\Repository\Values\Content\Search\SearchResult $searchResult
+     */
     protected function getIds($searchResult)
     {
         $ids = array_map(
+
+            /**
+             * @param \eZ\Publish\API\Repository\Values\Content\Search\SearchResult $hit
+             */
             function ($hit) {
                 return $hit->valueObject->id;
             },
@@ -106,7 +113,7 @@ 
     /**
      * Returns a location mapper mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Location\Mapper
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getLocationMapperMock()
     {