http://www.mediawiki.org/wiki/Special:Code/MediaWiki/74121

Revision: 74121
Author:   platonides
Date:     2010-10-01 22:59:57 +0000 (Fri, 01 Oct 2010)

Log Message:
-----------
Remove suite included twice in suite.xml
Rename MediaWikiParserTestSuite class to MediaWikiParserTest. This way, it 
matches its file basename, which makes phpunit not to consider, when loading 
MediaWikiParserText.php, the ParserUnitTest (coming from ParserHelpers.php 
require) as a standalone test class, which is what was producing the warning: 
No tests found in class "ParserUnitTest".

All tests pass for me now:
Tests: 752, Assertions: 3706, Incomplete: 2.

Incomplete tests are testApiListPages() and testWatchRollback()

Modified Paths:
--------------
    
trunk/phase3/maintenance/tests/phpunit/includes/parser/MediaWikiParserTest.php
    trunk/phase3/maintenance/tests/phpunit/suite.xml

Modified: 
trunk/phase3/maintenance/tests/phpunit/includes/parser/MediaWikiParserTest.php
===================================================================
--- 
trunk/phase3/maintenance/tests/phpunit/includes/parser/MediaWikiParserTest.php  
    2010-10-01 22:59:04 UTC (rev 74120)
+++ 
trunk/phase3/maintenance/tests/phpunit/includes/parser/MediaWikiParserTest.php  
    2010-10-01 22:59:57 UTC (rev 74121)
@@ -2,7 +2,7 @@
 
 require_once( dirname( __FILE__ ) . '/ParserHelpers.php' );
 
-class MediaWikiParserTestSuite extends PHPUnit_Framework_TestSuite {
+class MediaWikiParserTest extends PHPUnit_Framework_TestSuite {
        private $count;
        public $backend;
 

Modified: trunk/phase3/maintenance/tests/phpunit/suite.xml
===================================================================
--- trunk/phase3/maintenance/tests/phpunit/suite.xml    2010-10-01 22:59:04 UTC 
(rev 74120)
+++ trunk/phase3/maintenance/tests/phpunit/suite.xml    2010-10-01 22:59:57 UTC 
(rev 74121)
@@ -11,9 +11,6 @@
                <testsuite name="includes">
                        <directory>./includes</directory>
                </testsuite>
-               <testsuite name="includes">
-                       <directory>./includes</directory>
-               </testsuite>
                <testsuite name="languages">
                        <directory>./languages</directory>
                </testsuite>



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

Reply via email to