wikimedia / mediawiki-extensions-Wikibase

repo/tests/phpunit/includes/FederatedProperties/FederatedPropertiesTestTrait.php

Doc Comments +3 added lines

@@ -23,6 +23,9 @@
 		$this->setWbSetting( 'federatedPropertiesEnabled', true );
 	}
 
+	/**
+	 * @param string|boolean $value
+	 */
 	private function setWbSetting( string $name, $value ) {
 		$this->setWbSettingInGlobalIfMwIntegrationTest( $name, $value );
 		$this->setWbSettingInSettings( $name, $value );

repo/includes/FederatedProperties/FederatedPropertiesError.php

Doc Comments +1 added lines

@@ -21,6 +21,7 @@
 	 * @param LabelsProvider $entity
 	 * @param string $msg Message key (string) for error text
 	 * @param array $params Array with parameters to wfMessage()
+	 * @param string|null $languageCode
 	 *
 	 * @suppress SecurityCheck-DoubleEscaped
 	 */

repo/includes/FederatedProperties/SummaryParsingPrefetchHelper.php

Doc Comments +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
 
 	/**
 	 * @param IResultWrapper|array $rows
-	 * @param array $languageCodes
+	 * @param string[] $languageCodes
 	 * @param array $termTypes
 	 */
 	public function prefetchFederatedProperties( $rows, array $languageCodes, array $termTypes ) : void {

repo/tests/phpunit/includes/FederatedProperties/Hooks/DifferenceEngineViewHeaderHookHandlerTest.php

Doc Comments +3 added lines

@@ -182,6 +182,9 @@
 		return $rows;
 	}
 
+	/**
+	 * @param string $titleText
+	 */
 	private function getMockedDiffEngine( $getOldRevision, $getNewRevision, $titleText ) {
 		$diffEngine = $this->createMock( DifferenceEngine::class );
 		$diffEngine->expects( $this->once() )

repo/tests/phpunit/includes/Hooks/LabelPrefetchHookHandlerTestBase.php

Doc Comments +4 added lines, -1 removed lines

@@ -55,7 +55,6 @@ 
 	 * @param string[] $termTypes
 	 * @param string[] $languageCodes
 	 * @param PrefetchingTermLookup|null $prefetchingTermLookup
-	 * @param EntityLookup|null $entityLookup
 	 * @param bool $federatedPropertiesEnabled
 	 * @return LabelPrefetchHookHandler
 	 */
@@ -103,6 +102,10 @@ 
 		);
 	}
 
+	/**
+	 * @param string[] $expectedTermTypes
+	 * @param string[] $expectedLanguageCodes
+	 */
 	protected function getPrefetchTermsCallback( $expectedIds, $expectedTermTypes, $expectedLanguageCodes ) {
 		$prefetchTerms = function (
 			array $entityIds,

repo/tests/phpunit/includes/FederatedProperties/ParserOutput/FederatedPropertiesEntityParserOutputGeneratorTest.php

Doc Comments +6 added lines

@@ -155,6 +155,9 @@ 
 		];
 	}
 
+	/**
+	 * @param PropertyId[] $expectedIds
+	 */
 	protected function getPrefetchTermsCallback( $expectedIds ) {
 		$prefetchTerms = function (
 			array $entityIds,
@@ -178,6 +181,9 @@ 
 		return $prefetchTerms;
 	}
 
+	/**
+	 * @param FullEntityParserOutputGenerator $fullGenerator
+	 */
 	private function newEntityParserOutputGenerator( $prefetchingTermLookup, $fullGenerator, $languageCode = 'en' ) {
 		return new FederatedPropertiesEntityParserOutputGenerator(
 			$fullGenerator,

repo/tests/phpunit/includes/WikibaseRepoTest.php

Doc Comments +8 added lines

@@ -1071,6 +1071,10 @@ 
 		$this->invokeMethodIfNoRequiredParameters( $wbRepo, $reflectionClass->getMethod( $methodName ) );
 	}
 
+	/**
+	 * @param WikibaseRepo $wbRepo
+	 * @param ReflectionMethod $method
+	 */
 	private function invokeMethodIfNoRequiredParameters( $wbRepo, $method ) {
 		if ( $method->getNumberOfRequiredParameters() === 0 ) {
 			$method->invoke( $wbRepo );
@@ -1086,6 +1090,10 @@ 
 		);
 	}
 
+	/**
+	 * @param string $settingName
+	 * @param boolean $settingValue
+	 */
 	private function getSettingsCopyWithSettingSet( $settingName, $settingValue ) {
 		$settings = new SettingsArray( WikibaseRepo::getDefaultInstance()->getSettings()->getArrayCopy() );
 		$settings->setSetting( $settingName, $settingValue );

data-access/src/SingleEntitySourceServices.php

Doc Comments +3 added lines

@@ -342,6 +342,9 @@
 		);
 	}
 
+	/**
+	 * @return PrefetchingTermLookup
+	 */
 	public function getPrefetchingTermLookup() {
 		if ( $this->prefetchingTermLookup === null ) {
 			$this->prefetchingTermLookup = new ByTypeDispatchingPrefetchingTermLookup(

repo/tests/phpunit/includes/FederatedProperties/ApiEntitySearchHelperTest.php

Doc Comments +8 added lines, -4 removed lines

@@ -56,6 +56,9 @@ 
 		return new ApiEntitySearchHelper( $api, $dataTypes );
 	}
 
+	/**
+	 * @param string $responseDataFile
+	 */
 	private function setupTestApi( &$params, $langCode, $responseDataFile, $statusCode = 200 ) {
 		$params = array_merge( $params, [ 'language' => $langCode, 'uselang' => $langCode, 'format' => 'json' ] );
 		$api = $this->createMock( GenericActionApiClient::class );
@@ -152,8 +155,8 @@ 
 	/**
 	 * @dataProvider paramsAndExpectedResponseProvider
 	 * @param string $responseDataFile
-	 * @param int $expectedResultCount
-	 * @param array $expectedResultsEntityId
+	 * @param int $expectedResultsEntityIds
+	 * @param array $expectedResultsEntityIds
 	 * @throws ApiRequestException
 	 */
 	public function testGetRankedSearchResults( $langCode, $params, $responseDataFile, $expectedResultsEntityIds ) {
@@ -220,6 +223,9 @@ 
 		}
 	}
 
+	/**
+	 * @return string
+	 */
 	private function getResponseDataForId( array $searchResponses, $resultId ) {
 		$searchResponses = array_filter( $searchResponses, function ( $response ) use ( $resultId ) {
 			return $response->id === $resultId;
@@ -233,8 +239,6 @@ 
 	/**
 	 * @dataProvider invalidParamsAndUnexpectedResponseProvider
 	 * @param string $responseDataFile
-	 * @param int $expectedResultCount
-	 * @param array $expectedResultsEntityId
 	 */
 	public function testApiResponseStructureIsValid( $langCode, $params, $responseDataFile, $statusCode ) {
 		$api = $this->setupTestApi( $params, $langCode, $responseDataFile, $statusCode );