NewParserTest.php can now work independently
authorAntoine Musso <hashar@users.mediawiki.org>
Tue, 31 May 2011 19:51:48 +0000 (19:51 +0000)
committerAntoine Musso <hashar@users.mediawiki.org>
Tue, 31 May 2011 19:51:48 +0000 (19:51 +0000)
PHPUnit load class according to their order on the filesystem I guess.
On the cruisecontrol host, it first load NewParserTest.php and run
the test then load MediaWikiParserTest.php which run the tests too!

On my host the order is reversed, and NewParserTest.php is never run
for a reason I have not investigated.

Anyway, deleting the MediaWikiParserTest.php is harmless since it is
redundant with NewParserTest.php

(should fix cruise control)

tests/phpunit/includes/parser/MediaWikiParserTest.php [deleted file]

diff --git a/tests/phpunit/includes/parser/MediaWikiParserTest.php b/tests/phpunit/includes/parser/MediaWikiParserTest.php
deleted file mode 100644 (file)
index 8d6230d..0000000
+++ /dev/null
@@ -1,35 +0,0 @@
-<?php
-
-require_once( dirname( __FILE__ ) . '/ParserHelpers.php' );
-require_once( dirname( __FILE__ ) . '/NewParserTest.php' );
-require_once( dirname(dirname(dirname( __FILE__ ))) . '/bootstrap.php' );
-
-/**
- * The UnitTest must be either a class that inherits from PHPUnit_Framework_TestCase 
- * or a class that provides a public static suite() method which returns 
- * an PHPUnit_Framework_Test object
- * 
- * @group Parser
- * @group Database
- */
-class MediaWikiParserTest {
-
-       public static function suite() {
-               global $wgParserTestFiles;
-
-               $suite = new PHPUnit_Framework_TestSuite;
-
-               foreach ( $wgParserTestFiles as $filename ) {
-                       $testsName = basename( $filename, '.txt' );
-                       $className = /*ucfirst( basename( dirname( $filename ) ) ) .*/ ucfirst( basename( $filename, '.txt' ) );
-                       
-                       eval( "/** @group Database\n@group Parser\n*/ class $className extends NewParserTest { protected \$file = \"" . addslashes( $filename ) . "\"; } " );
-
-                       $parserTester = new $className( $testsName );
-                       $suite->addTestSuite( new ReflectionClass ( $parserTester ) );
-               }
-               
-
-               return $suite;
-       }
-}