Greetings All, I've been editing http://www.mediawiki.org/wiki/Unit_Testing (and am happy for feedback and suggestions.)
While editing, I took at look at the support files in tests/phpunit and have some questions (along with a patch to fix a few problems I noted.) In tests/phpunit/Makefile: * Do we need the install target? Given that the installation script seems lightly broken, I'd guess that it is not used often. I've removed the target. We should also remove the supporting file (install-phpunit.sh). * The path for the coverage target was broken by r78383. It's fixed in the patch. * In targets noparser, safe and databaseless, do we need to exclude group Broken? It is already excluded in suite.xml. I can see an argument for always exclude Broken, so that people don't accidentally run broken tests when they supply their own XML configuration file with the CONFIG_FILE option. However, if you're supplying your own config file, then you should know what you're doing. * Also, I've done some light copy editing. I've done a light edit on tests/phpunit/README. In particular, I've removed the recommendation to use the system packaging tools to install PHPUnit. PEAR works quite well in my experience – following the installation instructions in the PHPUnit manual will help ensure that people are running current versions of PHPUnit. I also fixed the path in docs/code-coverage (which was broken by r78383). Cheers! -- Zak Greant (Wikimedia Foundation Contractor) MediaWiki Activity Log at http://twitter.com/#!/zwmf Plans, reports + more at http://mediawiki.org/wiki/User:Zakgreant Want to talk about the Mediawiki developer docs? Catch me on irc://irc.freenode.net#mediawiki
_______________________________________________ Wikitech-l mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/wikitech-l
