@@ -222,7 +222,7 @@ discard block |
||
222 | 222 | } |
223 | 223 | |
224 | 224 | /** |
225 | - * @param $itemId |
|
225 | + * @param ItemId $itemId |
|
226 | 226 | * @return \PHPUnit\Framework\MockObject\MockObject |
227 | 227 | */ |
228 | 228 | private function getOutputPageEntityIdReaderReturningEntity( $itemId ) { |
@@ -235,8 +235,8 @@ discard block |
||
235 | 235 | } |
236 | 236 | |
237 | 237 | /** |
238 | - * @param $itemId |
|
239 | - * @return MockObject |
|
238 | + * @param ItemId $itemId |
|
239 | + * @return EntityRevisionLookup |
|
240 | 240 | */ |
241 | 241 | private function getEntityRevisionLookupReturningEntity( $itemId ): EntityRevisionLookup { |
242 | 242 | $entityRevisionLookup = $this->createMock( EntityRevisionLookup::class ); |
@@ -224,6 +224,7 @@ |
||
224 | 224 | } |
225 | 225 | |
226 | 226 | /** |
227 | + * @param ItemId $id |
|
227 | 228 | * @return MockObject|OutputPageEntityIdReader |
228 | 229 | */ |
229 | 230 | protected function newEntityIdReaderReturningEntityId( $id ) { |
@@ -189,7 +189,7 @@ discard block |
||
189 | 189 | } |
190 | 190 | |
191 | 191 | /** |
192 | - * @return LocalizedTextProvider|MockObject |
|
192 | + * @return LocalizedTextProvider |
|
193 | 193 | */ |
194 | 194 | private function newLocalizedTextProvider(): LocalizedTextProvider { |
195 | 195 | return $this->createMock( LocalizedTextProvider::class ); |
@@ -211,6 +211,10 @@ discard block |
||
211 | 211 | ); |
212 | 212 | } |
213 | 213 | |
214 | + /** |
|
215 | + * @param ItemId|null $itemId |
|
216 | + * @param string $editLinkUrl |
|
217 | + */ |
|
214 | 218 | private function newLinkingSpecialPageLinker( $itemId, $editLinkUrl ) { |
215 | 219 | $specialPageLinker = $this->newSpecialPageLinker(); |
216 | 220 | $specialPageLinker->expects( $this->once() ) |
@@ -221,7 +225,7 @@ discard block |
||
221 | 225 | } |
222 | 226 | |
223 | 227 | /** |
224 | - * @return MockObject|SpecialPageLinker |
|
228 | + * @return SpecialPageLinker |
|
225 | 229 | */ |
226 | 230 | private function newSpecialPageLinker() { |
227 | 231 | return $this->createMock( SpecialPageLinker::class ); |
@@ -173,6 +173,9 @@ |
||
173 | 173 | $this->assertLastMessageContains( 'Q4' ); |
174 | 174 | } |
175 | 175 | |
176 | + /** |
|
177 | + * @param string $expectedString |
|
178 | + */ |
|
176 | 179 | private function assertLastMessageContains( $expectedString ) { |
177 | 180 | $messages = $this->progressReporter->getMessages(); |
178 | 181 |
@@ -433,6 +433,9 @@ |
||
433 | 433 | )->map(); |
434 | 434 | } |
435 | 435 | |
436 | + /** |
|
437 | + * @param string $idString |
|
438 | + */ |
|
436 | 439 | private function getMockEntityId( $idString ) { |
437 | 440 | $entityId = $this->createMock( EntityId::class ); |
438 | 441 | $entityId->method( '__toString' )->willReturn( $idString ); |
@@ -86,7 +86,7 @@ |
||
86 | 86 | } |
87 | 87 | |
88 | 88 | /** |
89 | - * @return AliasesProvider|EntityDocument |
|
89 | + * @return EntityDocument |
|
90 | 90 | */ |
91 | 91 | abstract protected function getEntity(); |
92 | 92 |
@@ -67,7 +67,7 @@ |
||
67 | 67 | } |
68 | 68 | |
69 | 69 | /** |
70 | - * @return DescriptionsProvider|EntityDocument |
|
70 | + * @return EntityDocument |
|
71 | 71 | */ |
72 | 72 | abstract protected function getEntity(); |
73 | 73 |
@@ -65,7 +65,7 @@ |
||
65 | 65 | } |
66 | 66 | |
67 | 67 | /** |
68 | - * @return LabelsProvider|EntityDocument |
|
68 | + * @return EntityDocument |
|
69 | 69 | */ |
70 | 70 | abstract protected function getEntity(); |
71 | 71 |
@@ -190,6 +190,10 @@ |
||
190 | 190 | $this->fail( 'Should throw specific exception' ); |
191 | 191 | } |
192 | 192 | |
193 | + /** |
|
194 | + * @param integer $revisionId |
|
195 | + * @param string $slotRole |
|
196 | + */ |
|
193 | 197 | private function newRevisionRecord( $revisionId, $slotRole ) { |
194 | 198 | $revision = $this->createMock( RevisionRecord::class ); |
195 | 199 | $revision |