Quality Assurance > Missing PHPDoc comment

app/code/Magento/AuthorizenetAcceptjs/Test/Unit/Gateway/ConfigTest.php

@@ -81,6 +81,9 @@ public function testGetSolutionIdSandbox($environment, $expectedSolution)
         $this->assertEquals($expectedSolution, $this->model->getSolutionId(123));
     }
 
+    /**
+     * @return array
+     */
     public function configMapProvider()
     {
         return [
@@ -97,6 +100,10 @@ public function configMapProvider()
             ['getTransactionInfoSyncKeys', 'transactionSyncKeys', 'a,b,c', ['a', 'b', 'c']],
         ];
     }
+
+    /**
+     * @return array
+     */
     public function environmentUrlProvider()
     {
         return [
@@ -105,6 +112,9 @@ public function environmentUrlProvider()
         ];
     }
 
+    /**
+     * @return array
+     */
     public function environmentSolutionProvider()
     {
         return [

app/code/Magento/AuthorizenetAcceptjs/Test/Unit/Gateway/Request/AddressDataBuilderTest.php

@@ -108,13 +108,22 @@ public function testBuildWithBothAddresses()
         $this->assertEquals('abc', $result['transactionRequest']['customerIP']);
     }
 
+    /**
+     * @param $responseData
+     * @param $addressPrefix
+     */
     private function validateAddressData($responseData, $addressPrefix)
     {
         foreach ($this->mockAddressData as $fieldValue => $field) {
             $this->assertEquals($addressPrefix . $field['sampleData'], $responseData[$fieldValue]);
         }
     }
 
+    /**
+     * @param $prefix
+     *
+     * @return \PHPUnit\Framework\MockObject\MockObject
+     */
     private function createAddressMock($prefix)
     {
         $addressAdapterMock = $this->createMock(AddressAdapterInterface::class);

app/code/Magento/AuthorizenetAcceptjs/Test/Unit/Gateway/Response/PaymentReviewStatusHandlerTest.php

@@ -112,6 +112,9 @@ public function testDoesNothingWhenPending(string $status)
         $this->handler->handle($subject, $response);
     }
 
+    /**
+     * @return array
+     */
     public function pendingTransactionStatusesProvider()
     {
         return [
@@ -120,6 +123,9 @@ public function pendingTransactionStatusesProvider()
         ];
     }
 
+    /**
+     * @return array
+     */
     public function declinedTransactionStatusesProvider()
     {
         return [

app/code/Magento/Catalog/Test/Unit/Model/Category/FileInfoTest.php

@@ -191,6 +191,9 @@ public function testIsExist($fileName, $fileMediaPath)
         $this->assertTrue($this->model->isExist($fileName));
     }
 
+    /**
+     * @return array
+     */
     public function isExistProvider()
     {
         return [
@@ -213,6 +216,9 @@ public function testIsBeginsWithMediaDirectoryPath($fileName, $expected)
         $this->assertEquals($expected, $this->model->isBeginsWithMediaDirectoryPath($fileName));
     }
 
+    /**
+     * @return array
+     */
     public function isBeginsWithMediaDirectoryPathProvider()
     {
         return [

app/code/Magento/Catalog/Test/Unit/Model/ProductTest.php

@@ -486,6 +486,9 @@ public function testGetStoreSingleSiteModelIds(
         $this->assertEquals($websiteIDs, $this->model->getStoreIds());
     }
 
+    /**
+     * @return array
+     */
     public function getSingleStoreIds()
     {
         return [

app/code/Magento/Checkout/Test/Unit/Model/Cart/RequestQuantityProcessorTest.php

@@ -48,6 +48,9 @@ public function testProcess($cartData, $expected)
         $this->assertEquals($this->requestProcessor->process($cartData), $expected);
     }
 
+    /**
+     * @return array
+     */
     public function cartDataProvider()
     {
         return [

app/code/Magento/ImportExport/Test/Unit/Model/ImportTest.php

@@ -640,6 +640,9 @@ public function testGetUnknownEntity($entity)
         $import->getEntity();
     }
 
+    /**
+     * @return array
+     */
     public function unknownEntitiesProvider()
     {
         return [

app/code/Magento/LayeredNavigation/Test/Unit/Block/NavigationTest.php

@@ -100,7 +100,7 @@ public function testCanShowBlock()
             ->method('isEnabled')
             ->with($this->catalogLayerMock, $filters)
             ->will($this->returnValue($enabled));
-        
+
         $category = $this->createMock(Category::class);
         $this->catalogLayerMock->expects($this->atLeastOnce())->method('getCurrentCategory')->willReturn($category);
         $category->expects($this->once())->method('getDisplayMode')->willReturn(Category::DM_PRODUCT);
@@ -119,12 +119,12 @@ public function testCanShowBlock()
     public function testCanShowBlockWithDifferentDisplayModes(string $mode, bool $result)
     {
         $filters = ['To' => 'be', 'or' => 'not', 'to' => 'be'];
-        
+
         $this->filterListMock->expects($this->atLeastOnce())->method('getFilters')
             ->with($this->catalogLayerMock)
             ->will($this->returnValue($filters));
         $this->assertEquals($filters, $this->model->getFilters());
-        
+
         $this->visibilityFlagMock
             ->expects($this->any())
             ->method('isEnabled')
@@ -137,6 +137,9 @@ public function testCanShowBlockWithDifferentDisplayModes(string $mode, bool $re
         $this->assertEquals($result, $this->model->canShowBlock());
     }
 
+    /**
+     * @return array
+     */
     public function canShowBlockDataProvider()
     {
         return [

app/code/Magento/Sales/Test/Unit/Model/OrderTest.php

@@ -1253,6 +1253,9 @@ public function testGetCreatedAtFormattedUsesCorrectLocale()
         $this->order->getCreatedAtFormatted(\IntlDateFormatter::SHORT);
     }
 
+    /**
+     * @return array
+     */
     public function notInvoicingStatesProvider()
     {
         return [
@@ -1262,6 +1265,9 @@ public function notInvoicingStatesProvider()
         ];
     }
 
+    /**
+     * @return array
+     */
     public function canNotCreditMemoStatesProvider()
     {
         return [

app/code/Magento/Sales/Test/Unit/Model/ResourceModel/Order/Handler/StateTest.php

@@ -103,6 +103,9 @@ public function testCheck(
         $this->assertEquals($expectedState, $this->orderMock->getState());
     }
 
+    /**
+     * @return array
+     */
     public function stateCheckDataProvider()
     {
         return [

app/code/Magento/Search/Test/Unit/Model/Search/PageSizeProviderTest.php

@@ -50,6 +50,9 @@ public function testGetPageSize($searchEngine, $size)
         $this->assertEquals($size, $this->model->getMaxPageSize());
     }
 
+    /**
+     * @return array
+     */
     public function getPageSizeDataProvider()
     {
         return [

app/functions.php

@@ -13,6 +13,9 @@
  * @return \Magento\Framework\Phrase
  */
 if (!function_exists('__')) {
+    /**
+     * @return \Magento\Framework\Phrase
+     */
     function __()
     {
         $argc = func_get_args();

lib/internal/Magento/Framework/Encryption/Test/Unit/Adapter/McryptTest.php

@@ -28,6 +28,11 @@ protected function setUp()
         $this->key = substr(__CLASS__, -32, 32);
     }
 
+    /**
+     * @param int $length
+     *
+     * @return string
+     */
     protected function getRandomString(int $length): string
     {
         $result = '';
@@ -48,18 +53,33 @@ private function requireCipherInfo()
         }
     }
 
+    /**
+     * @param string $cipherName
+     * @param string $modeName
+     *
+     * @return int
+     */
     private function getKeySize(string $cipherName, string $modeName): int
     {
         $this->requireCipherInfo();
         return self::$cipherInfo[$cipherName][$modeName]['key_size'];
     }
 
+    /**
+     * @param string $cipherName
+     * @param string $modeName
+     *
+     * @return int
+     */
     private function getInitVectorSize(string $cipherName, string $modeName): int
     {
         $this->requireCipherInfo();
         return self::$cipherInfo[$cipherName][$modeName]['iv_size'];
     }
 
+    /**
+     * @return array
+     */
     public function getCipherModeCombinations(): array
     {
         $result = [];
@@ -87,6 +107,9 @@ public function testConstructor(string $cipher, string $mode)
         $this->assertEquals($initVector, $crypt->getInitVector());
     }
 
+    /**
+     * @return array
+     */
     public function getConstructorExceptionData(): array
     {
         $key = substr(__CLASS__, -32, 32);
@@ -130,6 +153,9 @@ public function testConstructorDefaults()
         $this->assertEquals($cryptExpected->getInitVector(), $cryptActual->getInitVector());
     }
 
+    /**
+     * @return array
+     */
     public function getCryptData(): array
     {
         $fixturesFilename = __DIR__ . '/../Crypt/_files/_crypt_fixtures.php';

lib/internal/Magento/Framework/Encryption/Test/Unit/KeyValidatorTest.php

@@ -33,6 +33,9 @@ public function testIsValid($key, $expected = true)
         $this->assertEquals($expected, $this->keyValidator->isValid($key));
     }
 
+    /**
+     * @return array
+     */
     public function isValidDataProvider() : array
     {
         return [

lib/internal/Magento/Framework/Indexer/Test/Unit/MultiDimensionProviderTest.php

@@ -210,6 +210,11 @@ public function testMultiDimensionProviderWithMixedDataProvider()
         }
     }
 
+    /**
+     * @param $dimensions
+     *
+     * @return \PHPUnit\Framework\MockObject\MockObject
+     */
     private function getDimensionProviderMock($dimensions)
     {
         $dimensionProviderMock = $this->getMockBuilder(DimensionProviderInterface::class)
@@ -233,6 +238,12 @@ function () use ($dimensions) {
         return $dimensionProviderMock;
     }
 
+    /**
+     * @param string $name
+     * @param string $value
+     *
+     * @return \PHPUnit\Framework\MockObject\MockObject
+     */
     private function getDimensionMock(string $name, string $value)
     {
         $dimensionMock = $this->getMockBuilder(Dimension::class)

lib/internal/Magento/Framework/Serialize/Test/Unit/Serializer/JsonHexTagTest.php

@@ -17,7 +17,7 @@ class JsonHexTagTest extends \PHPUnit\Framework\TestCase
      * @var \Magento\Framework\Serialize\Serializer\Json
      */
     private $json;
-    
+
     protected function setUp()
     {
         $objectManager = new \Magento\Framework\TestFramework\Unit\Helper\ObjectManager($this);
@@ -37,6 +37,9 @@ public function testSerialize($value, $expected)
         );
     }
 
+    /**
+     * @return array
+     */
     public function serializeDataProvider()
     {
         $dataObject = new DataObject(['something']);

setup/src/Magento/Setup/Test/Unit/Console/Command/ModuleConfigStatusCommandTest.php

@@ -49,6 +49,9 @@ public function testExecute(array $currentConfig, array $correctConfig, string $
         $this->assertEquals($expectedOutput, $tester->getDisplay());
     }
 
+    /**
+     * @return array
+     */
     public function executeDataProvider()
     {
         $successMessage = 'The modules configuration is up to date.' . PHP_EOL;

setup/src/Magento/Setup/Test/Unit/Controller/EnvironmentTest.php

@@ -294,6 +294,13 @@ public function testIndexAction()
         $this->assertInstanceOf(\Zend\View\Model\JsonModel::class, $model);
     }
 
+    /**
+     * @param \PHPUnit\Framework\MockObject\MockObject $request
+     * @param \PHPUnit\Framework\MockObject\MockObject $response
+     * @param \PHPUnit\Framework\MockObject\MockObject $routeMatch
+     *
+     * @return \PHPUnit\Framework\MockObject\MockObject
+     */
     protected function getMvcEventMock(
         MockObject $request,
         MockObject $response,