@@ -139,7 +139,7 @@  | 
                                                    ||
| 139 | 139 | /**  | 
                                                        
| 140 | 140 | * @param EntityId $entityId  | 
                                                        
| 141 | 141 | * @param Language $language  | 
                                                        
| 142 | - * @param $termType  | 
                                                        |
| 142 | + * @param string $termType  | 
                                                        |
| 143 | 143 | * @return TermFallback|null  | 
                                                        
| 144 | 144 | */  | 
                                                        
| 145 | 145 |  	private function lookupWithoutCache( EntityId $entityId, Language $language, $termType ): ?TermFallback { | 
                                                        
@@ -110,6 +110,11 @@ discard block  | 
                                                    ||
| 110 | 110 | );  | 
                                                        
| 111 | 111 | }  | 
                                                        
| 112 | 112 | |
| 113 | + /**  | 
                                                        |
| 114 | + * @param string $term  | 
                                                        |
| 115 | + * @param string $requestLanguageCode  | 
                                                        |
| 116 | + * @param string $actualLanguageCode  | 
                                                        |
| 117 | + */  | 
                                                        |
| 113 | 118 |  	private function getTermFallback( $term, $requestLanguageCode, $actualLanguageCode = null ): ?TermFallback { | 
                                                        
| 114 | 119 |  		if ( $term === null ) { | 
                                                        
| 115 | 120 | return null;  | 
                                                        
@@ -264,6 +269,9 @@ discard block  | 
                                                    ||
| 264 | 269 | );  | 
                                                        
| 265 | 270 | }  | 
                                                        
| 266 | 271 | |
| 272 | + /**  | 
                                                        |
| 273 | + * @param ItemId $itemId  | 
                                                        |
| 274 | + */  | 
                                                        |
| 267 | 275 |  	private function mockCacheWithContent( string $term, $itemId ): void { | 
                                                        
| 268 | 276 | $termFallback = new TermFallback( 'en', $term, 'en', 'en' );  | 
                                                        
| 269 | 277 | $this->revisionLookup->method( 'lookupLatestRevisionResolvingRedirect' )->willReturn( [ 1, $itemId ] );  | 
                                                        
@@ -284,6 +292,9 @@ discard block  | 
                                                    ||
| 284 | 292 | ->willReturn( [ self::ITEM_Q1_REVISION, $itemId ] );  | 
                                                        
| 285 | 293 | }  | 
                                                        
| 286 | 294 | |
| 295 | + /**  | 
                                                        |
| 296 | + * @param ItemId $itemId  | 
                                                        |
| 297 | + */  | 
                                                        |
| 287 | 298 |  	private function mockCacheEmpty( $itemId ): void { | 
                                                        
| 288 | 299 | $this->mockRevisionLookup( $itemId );  | 
                                                        
| 289 | 300 | |
@@ -293,6 +304,12 @@ discard block  | 
                                                    ||
| 293 | 304 | ->willReturn( TermFallbackCacheFacade::NO_VALUE );  | 
                                                        
| 294 | 305 | }  | 
                                                        
| 295 | 306 | |
| 307 | + /**  | 
                                                        |
| 308 | + * @param TermFallback|null $termFallback  | 
                                                        |
| 309 | + * @param ItemId $targetEntityId  | 
                                                        |
| 310 | + * @param integer $revisionId  | 
                                                        |
| 311 | + * @param string $termType  | 
                                                        |
| 312 | + */  | 
                                                        |
| 296 | 313 | private function mockCacheSetExpectation(  | 
                                                        
| 297 | 314 | $termFallback,  | 
                                                        
| 298 | 315 | $targetEntityId,  | 
                                                        
@@ -307,6 +324,10 @@ discard block  | 
                                                    ||
| 307 | 324 | ->with( $termFallback, $targetEntityId, $revisionId, $languageCode, $termType );  | 
                                                        
| 308 | 325 | }  | 
                                                        
| 309 | 326 | |
| 327 | + /**  | 
                                                        |
| 328 | + * @param ItemId $itemId  | 
                                                        |
| 329 | + * @param TermFallback|null $termFallback  | 
                                                        |
| 330 | + */  | 
                                                        |
| 310 | 331 | private function mockInternalLookupWithContent(  | 
                                                        
| 311 | 332 | $itemId,  | 
                                                        
| 312 | 333 | ?TermFallback $termFallback,  |