Thiemo Mättig (WMDE) has uploaded a new change for review.
https://gerrit.wikimedia.org/r/265228
Change subject: Add missing test case for PageImagesDataUpdater
......................................................................
Add missing test case for PageImagesDataUpdater
When there is a preferred statement somewhere, this should be used,
even if other properties do have a higher priority.
Change-Id: Id3e5b84fa507bb06e0f6752666fbb284fad03d74
---
M repo/tests/phpunit/includes/ParserOutput/PageImagesDataUpdaterTest.php
1 file changed, 3 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase
refs/changes/28/265228/1
diff --git
a/repo/tests/phpunit/includes/ParserOutput/PageImagesDataUpdaterTest.php
b/repo/tests/phpunit/includes/ParserOutput/PageImagesDataUpdaterTest.php
index 3849361..ffe9548 100644
--- a/repo/tests/phpunit/includes/ParserOutput/PageImagesDataUpdaterTest.php
+++ b/repo/tests/phpunit/includes/ParserOutput/PageImagesDataUpdaterTest.php
@@ -131,6 +131,8 @@
'Property not found' => array( $statements, array(
'P9999' ), null ),
'Not a property id' => array( $statements, array( 'Q1'
), null ),
'Invalid id' => array( $statements, array( 'invalid' ),
null ),
+
+ // Configuration.
'Ignore misconfiguration' => array( $statements, array(
'P1', 'P2', 'P1' ), '1.jpg' ),
'Ignore keys' => array( $statements, array( 2 => 'P1',
1 => 'P2' ), '1.jpg' ),
@@ -145,7 +147,7 @@
'Decreasing order' => array( $statements, array( 'P3',
'P2', 'P1' ), '3a.jpg' ),
'Skip deprecated' => array( $statements, array( 'P4' ),
'Four_2.jpg' ),
- 'Prefer preferred' => array( $statements, array( 'P5'
), '5c.jpg' ),
+ 'Prefer preferred' => array( $statements, array( 'P3',
'P4', 'P5' ), '5c.jpg' ),
);
}
--
To view, visit https://gerrit.wikimedia.org/r/265228
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id3e5b84fa507bb06e0f6752666fbb284fad03d74
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Thiemo Mättig (WMDE) <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits