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()

diff --git a/maintenance/tests/phpunit/includes/parser/MediaWikiParserTest.php b/maintenance/tests/phpunit/includes/parser/MediaWikiParserTest.php
index 49d1336..4b0c259 100644 (file)
--- a/maintenance/tests/phpunit/includes/parser/MediaWikiParserTest.php
+++ b/maintenance/tests/phpunit/includes/parser/MediaWikiParserTest.php
@@ -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;
 

diff --git a/maintenance/tests/phpunit/suite.xml b/maintenance/tests/phpunit/suite.xml
index 7a3b90e..fdda2b8 100644 (file)
--- a/maintenance/tests/phpunit/suite.xml
+++ b/maintenance/tests/phpunit/suite.xml
@@ -8,9 +8,6 @@
          convertWarningsToExceptions="true"
                 stopOnFailure="false">
        <testsuites>
-               <testsuite name="includes">
-                       <directory>./includes</directory>
-               </testsuite>
                <testsuite name="includes">
                        <directory>./includes</directory>
                </testsuite>