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

Reply via email to