Last modified: 2013-11-14 14:28:23 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 T51435, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 49435 - phpunit doesn't run TextContentTest
phpunit doesn't run TextContentTest
Status: NEW
Product: MediaWiki
Classification: Unclassified
ContentHandler (Other open bugs)
1.22.0
All Linux
: High major (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-06-11 14:12 UTC by Daniel Kinzler
Modified: 2013-11-14 14:28 UTC (History)
4 users (show)

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


Attachments

Description Daniel Kinzler 2013-06-11 14:12:53 UTC
when running phpunit.php without specifying a target file or class (so, running all tests or using --group and/or --filter), it does not find/run TextContentTest and JavaScriptContentTest. The related test classes CssContentTest and WikitextContentTest work fine.

There seems to be something odd going on here involving inheritance:  CssContentTest extends JavaScriptContentTest extends TextContentTest extends MediaWikiLangTestCase. Note that the top class, CssContentTest, works. If I remove it, JavaScriptContentTest suddenly works! As if phpunit was skipping any tests class that serve as a base class for another test...

But when I tried to verified this with a minimal test case, I couldn't: both trivial test classes, the parent and the child, where found and executed. Strange.

This is with PHPUnit 3.7.19 and PHP 5.4.9-4ubuntu2on Ubuntu. I access my MediaWiki installation via a symlinked path, but don't use symlinks inside the install.
Comment 1 Daniel Kinzler 2013-06-11 14:15:11 UTC
Point in case: https://gerrit.wikimedia.org/r/#/c/28199/11 should not have passed the gateway verification, since it causes a fatal error in TextContentTest.
Comment 2 Daniel Kinzler 2013-06-11 14:26:17 UTC
Setting to "major", since it can and has caused broken code to be merged into master. If we rely on phpunit for gateway testing, it should not just skip some tests.

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


Navigation
Links