Brion VIBBER has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/332640 )

Change subject: Add test case for Opus file check
......................................................................

Add test case for Opus file check

Test case for Opus file check so we don't regress!
Confirms the fix for T151352 that Opus files return
type 'audio' and not 'multimedia'.

Bug: T151352
Change-Id: Ib868793d90b60609ccec25726e6fbaa28bffebaf
---
A tests/phpunit/data/media/say-test.opus
M tests/phpunit/includes/libs/mime/MimeAnalyzerTest.php
2 files changed, 10 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/40/332640/1

diff --git a/tests/phpunit/data/media/say-test.opus 
b/tests/phpunit/data/media/say-test.opus
new file mode 100644
index 0000000..168d218
--- /dev/null
+++ b/tests/phpunit/data/media/say-test.opus
Binary files differ
diff --git a/tests/phpunit/includes/libs/mime/MimeAnalyzerTest.php 
b/tests/phpunit/includes/libs/mime/MimeAnalyzerTest.php
index 85927a3..53a36c5 100644
--- a/tests/phpunit/includes/libs/mime/MimeAnalyzerTest.php
+++ b/tests/phpunit/includes/libs/mime/MimeAnalyzerTest.php
@@ -59,4 +59,14 @@
                $actualType = $this->mimeAnalyzer->getMediaType( $oggFile, 
'application/ogg' );
                $this->assertEquals( $actualType, MEDIATYPE_AUDIO );
        }
+
+       /**
+        * Test to make sure that Opus audio files don't trigger
+        * MEDIATYPE_MULTIMEDIA (bug T151352)
+        */
+       function testOpusRecognize() {
+               $oggFile = __DIR__ . '/../../../data/media/say-test.opus';
+               $actualType = $this->mimeAnalyzer->getMediaType( $oggFile, 
'application/ogg' );
+               $this->assertEquals( $actualType, MEDIATYPE_AUDIO );
+       }
 }

-- 
To view, visit https://gerrit.wikimedia.org/r/332640
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib868793d90b60609ccec25726e6fbaa28bffebaf
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Brion VIBBER <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to