@@ -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 |
@@ -714,6 +714,9 @@ |
||
714 | 714 | ); |
715 | 715 | } |
716 | 716 | |
717 | + /** |
|
718 | + * @param integer $langCount |
|
719 | + */ |
|
717 | 720 | private function generateLanguageValuePairs( $langCount ) { |
718 | 721 | $result = []; |
719 | 722 | $langCodes = WikibaseRepo::getDefaultInstance()->getTermsLanguages()->getLanguages(); |
@@ -25,6 +25,9 @@ |
||
25 | 25 | $this->termValidatorFactory = $termValidatorFactory; |
26 | 26 | } |
27 | 27 | |
28 | + /** |
|
29 | + * @param NullChangeOp $changeOps |
|
30 | + */ |
|
28 | 31 | public function add( $changeOps ) { |
29 | 32 | $this->innerChangeOps->add( $changeOps ); |
30 | 33 | } |
@@ -213,6 +213,9 @@ discard block |
||
213 | 213 | } |
214 | 214 | } |
215 | 215 | |
216 | + /** |
|
217 | + * @param string|false $termType |
|
218 | + */ |
|
216 | 219 | private function getEntityTerm( EntityId $entityId, $lang, $termType ): string { |
217 | 220 | if ( $termType === 'label' ) { |
218 | 221 | return $this->termLookup->getLabel( $entityId, $lang ) ?? ''; |
@@ -223,6 +226,10 @@ discard block |
||
223 | 226 | throw new InvalidArgumentException( "\$termType can only be 'label' or 'property'. '{$termType}' was given" ); |
224 | 227 | } |
225 | 228 | |
229 | + /** |
|
230 | + * @param string $code |
|
231 | + * @param EntityId $collidingEntityId |
|
232 | + */ |
|
226 | 233 | private function collisionToError( $code, $collidingEntityId, $lang, $label ) { |
227 | 234 | return new UniquenessViolation( |
228 | 235 | $collidingEntityId, |