wikimedia / mediawiki-extensions-Wikibase

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

lib/tests/phpunit/EntityRevisionLookupTestCase.php

Doc Comments +3 added lines

@@ -56,6 +56,9 @@
 		];
 	}
 
+	/**
+	 * @param integer $revision
+	 */
 	protected function resolveLogicalRevision( $revision ) {
 		return $revision;
 	}

client/includes/Changes/AffectedPagesFinder.php

Doc Comments +1 added lines, -1 removed lines

@@ -355,7 +355,7 @@
 	}
 
 	/**
-	 * @param iterable<PageEntityUsages> $usages
+	 * @param Traversable $usages
 	 * @param EntityId $entityId
 	 * @param string[] $changedAspects
 	 *

client/includes/DataAccess/Scribunto/CachingFallbackBasedTermLookup.php

Doc Comments +1 added lines, -1 removed lines

@@ -139,7 +139,7 @@
 	/**
 	 * @param EntityId $entityId
 	 * @param Language $language
-	 * @param $termType
+	 * @param string $termType
 	 * @return TermFallback|null
 	 */
 	private function lookupWithoutCache( EntityId $entityId, Language $language, $termType ): ?TermFallback {

client/tests/phpunit/unit/includes/DataAccess/Scribunto/CachingFallbackBasedTermLookupTest.php

Doc Comments +21 added lines

@@ -97,6 +97,11 @@ 
 		);
 	}
 
+	/**
+	 * @param string $term
+	 * @param string $requestLanguageCode
+	 * @param string $actualLanguageCode
+	 */
 	private function getTermFallback( $term, $requestLanguageCode, $actualLanguageCode = null ): ?TermFallback {
 		if ( $term === null ) {
 			return null;
@@ -280,6 +285,9 @@ 
 		yield [ TermTypes::TYPE_DESCRIPTION ];
 	}
 
+	/**
+	 * @param ItemId $itemId
+	 */
 	private function mockCacheWithContent( string $term, $itemId ): void {
 		$termFallback = new TermFallback( 'en', $term, 'en', 'en' );
 		$this->revisionLookup->method( 'lookupLatestRevisionResolvingRedirect' )->willReturn( [ 1, $itemId ] );
@@ -300,6 +308,9 @@ 
 			->willReturn( [ self::ITEM_Q1_REVISION, $itemId ] );
 	}
 
+	/**
+	 * @param ItemId $itemId
+	 */
 	private function mockCacheEmpty( $itemId ): void {
 		$this->mockRevisionLookup( $itemId );
 
@@ -309,6 +320,12 @@ 
 			->willReturn( TermFallbackCacheFacade::NO_VALUE );
 	}
 
+	/**
+	 * @param TermFallback|null $termFallback
+	 * @param ItemId $targetEntityId
+	 * @param integer $revisionId
+	 * @param string $termType
+	 */
 	private function mockCacheSetExpectation(
 		$termFallback,
 		$targetEntityId,
@@ -323,6 +340,10 @@ 
 			->with( $termFallback, $targetEntityId, $revisionId, $languageCode, $termType );
 	}
 
+	/**
+	 * @param ItemId $itemId
+	 * @param TermFallback|null $termFallback
+	 */
 	private function mockInternalLookupWithContent(
 		$itemId,
 		?TermFallback $termFallback,