http://www.mediawiki.org/wiki/Special:Code/MediaWiki/100576

Revision: 100576
Author:   bawolff
Date:     2011-10-24 02:47:22 +0000 (Mon, 24 Oct 2011)
Log Message:
-----------
(follow-up r100575 / r99477) unit-tests for jpegMetadataExtractor dealing with 
padding bytes.

Modified Paths:
--------------
    trunk/phase3/tests/phpunit/data/media/README
    trunk/phase3/tests/phpunit/includes/media/JpegMetadataExtractorTest.php

Added Paths:
-----------
    trunk/phase3/tests/phpunit/data/media/jpeg-padding-even.jpg
    trunk/phase3/tests/phpunit/data/media/jpeg-padding-odd.jpg

Modified: trunk/phase3/tests/phpunit/data/media/README
===================================================================
--- trunk/phase3/tests/phpunit/data/media/README        2011-10-24 02:41:34 UTC 
(rev 100575)
+++ trunk/phase3/tests/phpunit/data/media/README        2011-10-24 02:47:22 UTC 
(rev 100576)
@@ -25,7 +25,8 @@
 jpeg-comment-iso8859-1.jpg, jpeg-comment-binary.jpg, jpeg-xmp-psir.jpg,
 jpeg-xmp-alt.jpg, animated.gif, exif-user-comment.jpg, animated-xmp.gif,
 iptc-timetest-invalid.jpg, jpeg-iptc-bad-hash.jpg, iptc-timetest.jpg,
-xmp.png, nonanimated.gif, exif-gps.jpg, jpeg-xmp-psir.xmp, 
jpeg-iptc-good-hash.jpg
+xmp.png, nonanimated.gif, exif-gps.jpg, jpeg-xmp-psir.xmp, 
jpeg-iptc-good-hash.jpg,
+jpeg-padding-even.jpg, jpeg-padding-odd.jpg
 Are all by Bawolff. I don't think they contain enough originality to
 claim copyright, but on the off chance they do, feel free to use them
 however you feel fit, without restriction.

Added: trunk/phase3/tests/phpunit/data/media/jpeg-padding-even.jpg
===================================================================
(Binary files differ)


Property changes on: trunk/phase3/tests/phpunit/data/media/jpeg-padding-even.jpg
___________________________________________________________________
Added: svn:mime-type
   + image/jpeg

Added: trunk/phase3/tests/phpunit/data/media/jpeg-padding-odd.jpg
===================================================================
(Binary files differ)


Property changes on: trunk/phase3/tests/phpunit/data/media/jpeg-padding-odd.jpg
___________________________________________________________________
Added: svn:mime-type
   + image/jpeg

Modified: 
trunk/phase3/tests/phpunit/includes/media/JpegMetadataExtractorTest.php
===================================================================
--- trunk/phase3/tests/phpunit/includes/media/JpegMetadataExtractorTest.php     
2011-10-24 02:41:34 UTC (rev 100575)
+++ trunk/phase3/tests/phpunit/includes/media/JpegMetadataExtractorTest.php     
2011-10-24 02:47:22 UTC (rev 100576)
@@ -12,10 +12,25 @@
                $this->filePath = dirname( __FILE__ ) . '/../../data/media/';
        }
 
-       public function testUtf8Comment() {
-               $res = JpegMetadataExtractor::segmentSplitter( $this->filePath 
. 'jpeg-comment-utf.jpg' );
+       /**
+        * We also use this test to test padding bytes don't
+        * screw stuff up
+        *
+        * @param $file filename
+        *
+        * @dataProvider dataUtf8Comment
+        */
+       public function testUtf8Comment( $file ) {
+               $res = JpegMetadataExtractor::segmentSplitter( $this->filePath 
. $file );
                $this->assertEquals( array( 'UTF-8 JPEG Comment — ¼' ), 
$res['COM'] );
        }
+       public function dataUtf8Comment() {
+               return array(
+                       array( 'jpeg-comment-utf.jpg' ),
+                       array( 'jpeg-padding-even.jpg' ),
+                       array( 'jpeg-padding-odd.jpg' ),
+               );
+       }
        /** The file is iso-8859-1, but it should get auto converted */
        public function testIso88591Comment() {
                $res = JpegMetadataExtractor::segmentSplitter( $this->filePath 
. 'jpeg-comment-iso8859-1.jpg' );


_______________________________________________
MediaWiki-CVS mailing list
MediaWiki-CVS@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs

Reply via email to