Last modified: 2014-07-01 19:18:44 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T69343, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 67343 - Scribunto fetchLanguageName tests fail if CLDR is not installed
Scribunto fetchLanguageName tests fail if CLDR is not installed
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Scribunto (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-07-01 00:57 UTC by Tim Starling
Modified: 2014-07-01 19:18 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Tim Starling 2014-07-01 00:57:35 UTC
6 Scribunto PHPUnit tests fail if CLDR is not installed. These tests should be skipped instead. 

On the Lua side, testframework.getTestProvider() gives name, func, args and expect parameters. A "skip" parameter could be added which skips tests in Scribunto_LuaEngineTestBase::testLua() if a given "skip tag" string is registered by the PHP layer.

builder@hhvm-syd:/mnt/build/src/mediawiki/core$ hhvm -c ~/etc/server.hdf tests/phpunit/phpunit.php ../extensions/Scribunto/tests/engines/LuaCommon/LanguageLibraryTest.php
PHPUnit 4.1.3 by Sebastian Bergmann.

Configuration read from /mnt/build/src/mediawiki/core/tests/phpunit/suite.xml

S..FF...F........................................S..FF...F....... 65 / 98 ( 66%)
.................................

Time: 15.13 seconds, Memory: 52.67Mb

There were 6 failures:

1) LuaSandbox: LanguageLibraryTests[3]: fetchLanguageName (en,ru)
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
 {
-  "английский",
+  "English",
 }

/mnt/build/src/mediawiki/extensions/Scribunto/tests/engines/LuaCommon/LuaEngineTestBase.php:235
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiTestCase.php:133
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiPHPUnitCommand.php:42

2) LuaSandbox: LanguageLibraryTests[4]: fetchLanguageName (ru,en)
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
 {
-  "Russian",
+  "русский",
 }

/mnt/build/src/mediawiki/extensions/Scribunto/tests/engines/LuaCommon/LuaEngineTestBase.php:235
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiTestCase.php:133
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiPHPUnitCommand.php:42

3) LuaSandbox: LanguageLibraryTests[8]: fetchLanguageNames (de)
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
 {
   {
-    ["en"] = "Englisch",
-    ["ru"] = "Russisch",
+    ["en"] = "English",
+    ["ru"] = "русский",
   },
 }

/mnt/build/src/mediawiki/extensions/Scribunto/tests/engines/LuaCommon/LuaEngineTestBase.php:235
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiTestCase.php:133
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiPHPUnitCommand.php:42

4) LuaStandalone: LanguageLibraryTests[3]: fetchLanguageName (en,ru)
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
 {
-  "английский",
+  "English",
 }

/mnt/build/src/mediawiki/extensions/Scribunto/tests/engines/LuaCommon/LuaEngineTestBase.php:235
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiTestCase.php:133
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiPHPUnitCommand.php:42

5) LuaStandalone: LanguageLibraryTests[4]: fetchLanguageName (ru,en)
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
 {
-  "Russian",
+  "русский",
 }

/mnt/build/src/mediawiki/extensions/Scribunto/tests/engines/LuaCommon/LuaEngineTestBase.php:235
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiTestCase.php:133
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiPHPUnitCommand.php:42

6) LuaStandalone: LanguageLibraryTests[8]: fetchLanguageNames (de)
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
 {
   {
-    ["en"] = "Englisch",
-    ["ru"] = "Russisch",
+    ["en"] = "English",
+    ["ru"] = "русский",
   },
 }

/mnt/build/src/mediawiki/extensions/Scribunto/tests/engines/LuaCommon/LuaEngineTestBase.php:235
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiTestCase.php:133
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiPHPUnitCommand.php:42

--

There were 2 skipped tests:

1) LuaSandbox: Scribunto_LuaLanguageLibraryTests::testFormatDateTTLs
Language::sprintfDate does not set a TTL

/mnt/build/src/mediawiki/extensions/Scribunto/tests/engines/LuaCommon/LanguageLibraryTest.php:25
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiTestCase.php:133
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiPHPUnitCommand.php:42

2) LuaStandalone: Scribunto_LuaLanguageLibraryTests::testFormatDateTTLs
Language::sprintfDate does not set a TTL

/mnt/build/src/mediawiki/extensions/Scribunto/tests/engines/LuaCommon/LanguageLibraryTest.php:25
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiTestCase.php:133
/mnt/build/src/mediawiki/core/tests/phpunit/MediaWikiPHPUnitCommand.php:42
                                                   
FAILURES!                                          
Tests: 98, Assertions: 96, Failures: 6, Skipped: 2.
Comment 1 Gerrit Notification Bot 2014-07-01 14:49:14 UTC
Change 143325 had a related patch set uploaded by Anomie:
Allow for skipping Lua tests

https://gerrit.wikimedia.org/r/143325
Comment 2 Gerrit Notification Bot 2014-07-01 18:46:01 UTC
Change 143325 merged by jenkins-bot:
Allow for skipping Lua tests

https://gerrit.wikimedia.org/r/143325

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links