ezsystems / ezpublish-kernel

eZ/Publish/Core/Persistence/Legacy/Tests/Content/Type/MapperTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -383,7 +383,7 @@ 
     /**
      * Returns a Mapper with conversion methods mocked.
      *
-     * @return Mapper
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getNonConvertingMapper()
     {
@@ -413,7 +413,7 @@ 
     /**
      * Returns a converter registry mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\FieldValue\ConverterRegistry
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getConverterRegistryMock()
     {

eZ/Publish/SPI/Tests/FieldType/ImageIntegrationTest.php

Doc Comments +3 added lines

@@ -100,6 +100,9 @@
         );
     }
 
+    /**
+     * @return DeprecationWarnerInterface
+     */
     public function getDeprecationWarnerMock()
     {
         if (!isset($this->deprecationWarnerMock)) {

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

@@ -1038,7 +1038,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
      */
@@ -1112,10 +1112,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,
@@ -2000,7 +1999,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,
@@ -4809,6 +4808,10 @@ 
         );
     }
 
+    /**
+     * @param Field[] $existingFields
+     * @param FieldDefinition[] $fieldDefinitions
+     */
     public function assertForTestUpdateContentRequiredField(
         $initialLanguageCode,
         $structFields,
@@ -5010,6 +5013,10 @@ 
         }
     }
 
+    /**
+     * @param Field[] $existingFields
+     * @param FieldDefinition[] $fieldDefinitions
+     */
     public function assertForTestUpdateContentThrowsContentFieldValidationException(
         $initialLanguageCode,
         $structFields,
@@ -5740,7 +5747,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/Bundle/EzPublishCoreBundle/EventListener/ConfigScopeListener.php

Doc Comments +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
     /**
      * Sets the complete list of view providers.
      *
-     * @param array $viewProviders
+     * @param \PHPUnit\Framework\MockObject\MockObject $viewProviders
      */
     public function setViewProviders(array $viewProviders)
     {

eZ/Bundle/EzPublishCoreBundle/Tests/EventListener/BackgroundIndexingTerminateListenerTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
     /**
      * @dataProvider indexingProvider
      *
-     * @param array|null $value
+     * @param array|null $values
      * @param \Psr\Log\LoggerInterface|\PHPUnit\Framework\MockObject\MockObject|null $logger
      */
     public function testIndexing(array $values = null, $logger = null)

eZ/Publish/Core/Helper/Tests/ContentInfoLocationLoader/SudoMainLocationLoaderTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
     }
 
     /**
-     * @return \eZ\Publish\Core\Repository\Repository|\PHPUnit\Framework\MockObject\MockObject
+     * @return \eZ\Publish\API\Repository\Repository
      */
     private function getRepositoryMock()
     {

eZ/Publish/Core/MVC/Symfony/Matcher/Tests/AbstractMatcherFactoryTest.php

Doc Comments -1 removed lines

@@ -169,7 +169,6 @@
     }
 
     /**
-     * @param array $properties
      *
      * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\MVC\Symfony\View\ContentView
      */

eZ/Publish/Core/MVC/Symfony/Templating/Tests/Twig/Extension/FileSizeExtensionTest.php

Doc Comments +5 added lines, -5 removed lines

@@ -55,7 +55,7 @@ 
     }
 
     /**
-     * @return string $locale
+     * @return string[] $locale
      */
     public function getLocale()
     {
@@ -63,7 +63,7 @@ 
     }
 
     /**
-     * @return array
+     * @return FileSizeExtension[]
      */
     protected function getExtensions()
     {
@@ -81,7 +81,7 @@ 
     }
 
     /**
-     * @return ConfigResolverInterface|\PHPUnit\Framework\MockObject\MockObject
+     * @return ConfigResolverInterface
      */
     protected function getConfigResolverInterfaceMock()
     {
@@ -95,7 +95,7 @@ 
     }
 
     /**
-     * @return LocaleConverterInterface|\PHPUnit\Framework\MockObject\MockObject
+     * @return LocaleConverterInterface
      */
     protected function getLocaleConverterInterfaceMock()
     {
@@ -115,7 +115,7 @@ 
     }
 
     /**
-     * @return TranslatorInterface|\PHPUnit\Framework\MockObject\MockObject
+     * @return TranslatorInterface
      */
     protected function getTranslatorInterfaceMock()
     {

eZ/Publish/Core/Persistence/Legacy/Tests/Content/ContentHandlerTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -1438,7 +1438,7 @@ 
      *
      * @param string[] $methods
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Handler
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getPartlyMockedHandler(array $methods)
     {
@@ -1462,7 +1462,7 @@ 
     /**
      * Returns a TreeHandler mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\TreeHandler
+     * @return TreeHandler
      */
     protected function getTreeHandlerMock()
     {