wikimedia / mediawiki-extensions-Wikibase

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 );

lib/includes/Store/Sql/Terms/PrefetchingEntityTermLookupBase.php

Doc Comments +3 added lines

@@ -133,6 +133,9 @@
 		return $this->getFromBuffer( $entityId, TermTypes::TYPE_ALIAS, $languageCode ) ?? false;
 	}
 
+	/**
+	 * @param string $languageCode
+	 */
 	private function getFromBuffer( EntityId $entityId, $termType, $languageCode ) {
 		if ( !( $entityId instanceof Int32EntityId ) ) {
 			throw new InvalidArgumentException( "Not an Int32EntityId: " . $entityId->getSerialization() );

repo/tests/phpunit/includes/HttpResponseMockerTrait.php

Doc Comments +1 added lines

@@ -14,6 +14,7 @@
 
 	/**
 	 * Some test appear to want to simulate null status codes, hence the type hint
+	 * @param integer $statusCode
 	 */
 	private function newMockResponse( $response, ?int $statusCode ): ResponseInterface {
 		$mockStream = $this->createMock( StreamInterface::class );