mbstring.func_overload test should be skipped properly.

This avoids PHPUnit notices related to "risky" tests.

See #36016.

git-svn-id: https://develop.svn.wordpress.org/trunk@36788 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Boone Gorges 2016-02-29 22:37:42 +00:00
parent 7e55710025
commit 2b19ef7b76

View File

@ -127,13 +127,13 @@ class Tests_POMO_MO extends WP_UnitTestCase {
}
function test_overloaded_mb_functions() {
// skipIf() method skips the whole test case, not only this method
// that's why we are skipping it the stupid way
if ((ini_get("mbstring.func_overload") & 2) != 0) {
$mo = new MO();
$mo->import_from_file(DIR_TESTDATA . '/pomo/overload.mo');
$this->assertEquals(array('Табло'), $mo->entries['Dashboard']->translations);
if ((ini_get("mbstring.func_overload") & 2) == 0) {
$this->markTestSkipped( __METHOD__ . ' only runs when mbstring.func_overload is enabled.' );
}
$mo = new MO();
$mo->import_from_file(DIR_TESTDATA . '/pomo/overload.mo');
$this->assertEquals(array('Табло'), $mo->entries['Dashboard']->translations);
}
function test_load_pot_file() {