(bug 23375) Added ogv, oga, spx as extensions for ogg files. Patch by Derk-Jan Hartman.
[lhc/web/wiklou.git] / maintenance / parserTests.inc
index 64b41cc..29b7fc7 100644 (file)
@@ -1,5 +1,5 @@
 <?php
-# Copyright (C) 2004 Brion Vibber <brion@pobox.com>
+# Copyright (C) 2004, 2010 Brion Vibber <brion@pobox.com>
 # http://www.mediawiki.org/
 #
 # This program is free software; you can redistribute it and/or modify
  */
 
 /** */
-$options = array( 'quick', 'color', 'quiet', 'help', 'show-output', 'record'. 'run-disabled' );
+$options = array( 'quick', 'color', 'quiet', 'help', 'show-output', 'record', 'run-disabled' );
 $optionsWithArgs = array( 'regex', 'seed', 'setversion' );
 
-require_once( dirname(__FILE__) . '/commandLine.inc' );
+if ( !defined( "NO_COMMAND_LINE" ) ) {
+       require_once( dirname(__FILE__) . '/commandLine.inc' );
+}
 require_once( "$IP/maintenance/parserTestsParserHook.php" );
 require_once( "$IP/maintenance/parserTestsStaticParserHook.php" );
 require_once( "$IP/maintenance/parserTestsParserTime.php" );
@@ -97,7 +99,7 @@ class ParserTest {
                        isset( $options['quiet'] )
                        && ( isset( $options['record'] )
                                || isset( $options['compare'] ) ) ); // redundant output
-               
+
                $this->showOutput = isset( $options['show-output'] );
 
 
@@ -118,6 +120,8 @@ class ParserTest {
                        $this->recorder = new DbTestPreviewer( $this );
                } elseif( isset( $options['upload'] ) ) {
                        $this->recorder = new RemoteTestRecorder( $this );
+               } elseif( class_exists( 'PHPUnitTestRecorder' ) ) {
+                       $this->recorder = new PHPUnitTestRecorder( $this );
                } else {
                        $this->recorder = new TestRecorder( $this );
                }
@@ -136,7 +140,7 @@ class ParserTest {
        /**
         * Remove last character if it is a newline
         */
-       private function chomp($s) {
+       public function chomp($s) {
                if (substr($s, -1) === "\n") {
                        return substr($s, 0, -1);
                }
@@ -266,15 +270,16 @@ class ParserTest {
         * Prints status updates on stdout and counts up the total
         * number and percentage of passed tests.
         *
-        * @param array of strings $filenames
-        * @return bool True if passed all tests, false if any tests failed.
+        * @param $filenames Array of strings
+        * @return Boolean: true if passed all tests, false if any tests failed.
         */
        public function runTestsFromFiles( $filenames ) {
                $this->recorder->start();
                $this->setupDatabase();
                $ok = true;
                foreach( $filenames as $filename ) {
-                       $ok = $this->runFile( $filename ) && $ok;
+                       $tests = new TestFileIterator( $filename, $this );
+                       $ok = $this->runTests( $tests ) && $ok;
                }
                $this->teardownDatabase();
                $this->recorder->report();
@@ -282,124 +287,18 @@ class ParserTest {
                return $ok;
        }
 
-       private function runFile( $filename ) {
-               $infile = fopen( $filename, 'rt' );
-               if( !$infile ) {
-                       wfDie( "Couldn't open file '$filename'\n" );
-               } else {
-                       global $IP;
-                       $relative = wfRelativePath( $filename, $IP );
-                       $this->showRunFile( $relative );
-               }
-
-               $data = array();
-               $section = null;
-               $n = 0;
+       function runTests($tests) {
                $ok = true;
-               while( false !== ($line = fgets( $infile ) ) ) {
-                       $n++;
-                       $matches = array();
-                       if( preg_match( '/^!!\s*(\w+)/', $line, $matches ) ) {
-                               $section = strtolower( $matches[1] );
-                               if( $section == 'endarticle') {
-                                       if( !isset( $data['text'] ) ) {
-                                               wfDie( "'endarticle' without 'text' at line $n of $filename\n" );
-                                       }
-                                       if( !isset( $data['article'] ) ) {
-                                               wfDie( "'endarticle' without 'article' at line $n of $filename\n" );
-                                       }
-                                       $this->addArticle($this->chomp($data['article']), $this->chomp($data['text']), $n);
-                                       $data = array();
-                                       $section = null;
-                                       continue;
-                               }
-                               if( $section == 'endhooks' ) {
-                                       if( !isset( $data['hooks'] ) ) {
-                                               wfDie( "'endhooks' without 'hooks' at line $n of $filename\n" );
-                                       }
-                                       foreach( explode( "\n", $data['hooks'] ) as $line ) {
-                                               $line = trim( $line );
-                                               if( $line ) {
-                                                       $this->requireHook( $line );
-                                               }
-                                       }
-                                       $data = array();
-                                       $section = null;
-                                       continue;
-                               }
-                               if( $section == 'endfunctionhooks' ) {
-                                       if( !isset( $data['functionhooks'] ) ) {
-                                               wfDie( "'endfunctionhooks' without 'functionhooks' at line $n of $filename\n" );
-                                       }
-                                       foreach( explode( "\n", $data['functionhooks'] ) as $line ) {
-                                               $line = trim( $line );
-                                               if( $line ) {
-                                                       $this->requireFunctionHook( $line );
-                                               }
-                                       }
-                                       $data = array();
-                                       $section = null;
-                                       continue;
-                               }
-                               if( $section == 'end' ) {
-                                       if( !isset( $data['test'] ) ) {
-                                               wfDie( "'end' without 'test' at line $n of $filename\n" );
-                                       }
-                                       if( !isset( $data['input'] ) ) {
-                                               wfDie( "'end' without 'input' at line $n of $filename\n" );
-                                       }
-                                       if( !isset( $data['result'] ) ) {
-                                               wfDie( "'end' without 'result' at line $n of $filename\n" );
-                                       }
-                                       if( !isset( $data['options'] ) ) {
-                                               $data['options'] = '';
-                                       }
-                                       else {
-                                               $data['options'] = $this->chomp( $data['options'] );
-                                       }
-                                       if (!isset( $data['config'] ) )
-                                               $data['config'] = '';
-                                       
-                                       if ( (preg_match('/\\bdisabled\\b/i', $data['options'])
-                                               || !preg_match("/{$this->regex}/i", $data['test'])) && !$this->runDisabled ) {
-                                               # disabled test
-                                               $data = array();
-                                               $section = null;
-                                               continue;
-                                       }
-                                       if ( preg_match('/\\bmath\\b/i', $data['options']) && !$this->savedGlobals['wgUseTeX'] ) {
-                                               # don't run math tests if $wgUseTeX is set to false in LocalSettings
-                                               $data = array();
-                                               $section = null;
-                                               continue;
-                                       }
-                                       $result = $this->runTest(
-                                               $this->chomp( $data['test'] ),
-                                               $this->chomp( $data['input'] ),
-                                               $this->chomp( $data['result'] ),
-                                               $this->chomp( $data['options'] ),
-                                               $this->chomp( $data['config']   )
-                                               );
-                                       $ok = $ok && $result;
-                                       $this->recorder->record( $this->chomp( $data['test'] ), $result );
-                                       $data = array();
-                                       $section = null;
-                                       continue;
-                               }
-                               if ( isset ($data[$section] ) ) {
-                                       wfDie( "duplicate section '$section' at line $n of $filename\n" );
-                               }
-                               $data[$section] = '';
-                               continue;
-                       }
-                       if( $section ) {
-                               $data[$section] .= $line;
-                       }
+  foreach($tests as $i => $t) {
+                       $result =
+                               $this->runTest($t['test'], $t['input'], $t['result'], $t['options'], $t['config']);
+                       $ok = $ok && $result;
+                       $this->recorder->record( $t['test'], $result );
                }
                if ( $this->showProgress ) {
                        print "\n";
                }
-               return $ok;
+  return $ok;
        }
 
        /**
@@ -425,11 +324,14 @@ class ParserTest {
         * and compare the output against the expected results.
         * Prints status and explanatory messages to stdout.
         *
-        * @param string $input Wikitext to try rendering
-        * @param string $result Result to output
-        * @return bool
+        * @param $desc String: test's description
+        * @param $input String: wikitext to try rendering
+        * @param $result String: result to output
+        * @param $opts Array: test's options
+        * @param $config String: overrides for global variables, one per line
+        * @return Boolean
         */
-       private function runTest( $desc, $input, $result, $opts, $config ) {
+       public function runTest( $desc, $input, $result, $opts, $config ) {
                if( $this->showProgress ) {
                        $this->showTesting( $desc );
                }
@@ -440,11 +342,6 @@ class ParserTest {
                $user = new User();
                $options = ParserOptions::newFromUser( $user );
 
-               if ( isset( $opts['math'] ) ) {
-                       # XXX this should probably be done by the ParserOptions
-                       $options->setUseTex(true);
-               }
-
                $m = array();
                if (isset( $opts['title'] ) ) {
                        $titleText = $opts['title'];
@@ -473,10 +370,16 @@ class ParserTest {
                } elseif( isset( $opts['comment'] ) ) {
                        $linker = $user->getSkin();
                        $out = $linker->formatComment( $input, $title, $local );
+               } elseif( isset( $opts['preload'] ) ) {
+                       $out = $parser->getpreloadText( $input, $title, $options );
                } else {
                        $output = $parser->parse( $input, $title, $options, true, true, 1337 );
                        $out = $output->getText();
 
+                       if ( isset( $opts['showtitle'] ) ) {
+                               if($output->getTitleText()) $title = $output->getTitleText();
+                               $out = "$title\n$out";
+                       }
                        if (isset( $opts['ill'] ) ) {
                                $out = $this->tidy( implode( ' ', $output->getLanguageLinks() ) );
                        } elseif( isset( $opts['cat'] ) ) {
@@ -493,6 +396,7 @@ class ParserTest {
                        $result = $this->tidy($result);
                }
 
+
                $this->teardownGlobals();
 
                if( $result === $out && ( $noxml === true || $this->wellFormed( $out ) ) ) {
@@ -505,9 +409,9 @@ class ParserTest {
 
        /**
         * Use a regex to find out the value of an option
-        * @param $key name of option val to retrieve
+        * @param $key String: name of option val to retrieve
         * @param $opts Options array to look in
-        * @param $defaults Default value returned if not found
+        * @param $default Mixed: default value returned if not found
         */
        private static function getOptionValue( $key, $opts, $default ) {
                $key = strtolower( $key );
@@ -517,7 +421,7 @@ class ParserTest {
                        return $default;
                }
        }
-       
+
        private function parseOptions( $instring ) {
                $opts = array();
                $lines = explode( "\n", $instring );
@@ -573,7 +477,7 @@ class ParserTest {
                }
                return $opts;
        }
-       
+
        private function cleanupOption( $opt ) {
                if( substr( $opt, 0, 1 ) == '"' ) {
                        return substr( $opt, 1, -1 );
@@ -589,6 +493,7 @@ class ParserTest {
         * Ideally this should replace the global configuration entirely.
         */
        private function setupGlobals($opts = '', $config = '') {
+               global $wgDBtype;
                if( !isset( $this->uploadDir ) ) {
                        $this->uploadDir = $this->setupUploadDir();
                }
@@ -600,7 +505,7 @@ class ParserTest {
                        self::getOptionValue( 'variant', $opts, false );
                $maxtoclevel =
                        self::getOptionValue( 'wgMaxTocLevel', $opts, 999 );
-               $linkHolderBatchSize = 
+               $linkHolderBatchSize =
                        self::getOptionValue( 'wgLinkHolderBatchSize', $opts, 1000 );
 
                $settings = array(
@@ -623,7 +528,7 @@ class ParserTest {
                        'wgServerName' => 'Britney-Spears',
                        'wgLanguageCode' => $lang,
                        'wgContLanguageCode' => $lang,
-                       'wgDBprefix' => 'parsertest_',
+                       'wgDBprefix' => $wgDBtype != 'oracle' ? 'parsertest_' : 'pt_',
                        'wgRawHtml' => isset( $opts['rawhtml'] ),
                        'wgLang' => null,
                        'wgContLang' => null,
@@ -633,7 +538,9 @@ class ParserTest {
                        'wgNoFollowLinks' => true,
                        'wgNoFollowDomainExceptions' => array(),
                        'wgThumbnailScriptPath' => false,
-                       'wgUseTeX' => false,
+                       'wgUseImageResize' => false,
+                       'wgUseTeX' => isset( $opts['math'] ),
+                       'wgMathDirectory' => $this->uploadDir . '/math',
                        'wgLocaltimezone' => 'UTC',
                        'wgAllowExternalImages' => true,
                        'wgUseTidy' => false,
@@ -650,26 +557,29 @@ class ParserTest {
                        'wgDefaultExternalStore' => array(),
                        'wgForeignFileRepos' => array(),
                        'wgLinkHolderBatchSize' => $linkHolderBatchSize,
-                       'wgEnforceHtmlIds' => true,
+                       'wgExperimentalHtmlIds' => false,
                        'wgExternalLinkTarget' => false,
                        'wgAlwaysUseTidy' => false,
                        'wgHtml5' => true,
                        'wgWellFormedXml' => true,
-                       );
+                       'wgAllowMicrodataAttributes' => true,
+               );
 
                if ($config) {
                        $configLines = explode( "\n", $config );
-                       
+
                        foreach( $configLines as $line ) {
                                list( $var, $value ) = explode( '=', $line, 2 );
-                               
+
                                $settings[$var] = eval("return $value;" );
                        }
                }
-               
+
                $this->savedGlobals = array();
                foreach( $settings as $var => $val ) {
-                       $this->savedGlobals[$var] = $GLOBALS[$var];
+                       if( array_key_exists( $var, $GLOBALS ) ) {
+                               $this->savedGlobals[$var] = $GLOBALS[$var];
+                       }
                        $GLOBALS[$var] = $val;
                }
                $langObj = Language::factory( $lang );
@@ -678,8 +588,6 @@ class ParserTest {
                $GLOBALS['wgMemc'] = new FakeMemCachedClient;
                $GLOBALS['wgOut'] = new OutputPage;
 
-               //$GLOBALS['wgMessageCache'] = new MessageCache( new BagOStuff(), false, 0, $GLOBALS['wgDBname'] );
-
                MagicWord::clearCache();
 
                global $wgUser;
@@ -694,16 +602,16 @@ class ParserTest {
                global $wgDBtype;
                $tables = array('user', 'page', 'page_restrictions',
                        'protected_titles', 'revision', 'text', 'pagelinks', 'imagelinks',
-                       'categorylinks', 'templatelinks', 'externallinks', 'langlinks',
+                       'categorylinks', 'templatelinks', 'externallinks', 'langlinks', 'iwlinks',
                        'site_stats', 'hitcounter',     'ipblocks', 'image', 'oldimage',
                        'recentchanges', 'watchlist', 'math', 'interwiki',
                        'querycache', 'objectcache', 'job', 'l10n_cache', 'redirect', 'querycachetwo',
                        'archive', 'user_groups', 'page_props', 'category'
                );
 
-               if ($wgDBtype === 'mysql') 
+               if ($wgDBtype === 'mysql')
                        array_push( $tables, 'searchindex' );
-               
+
                // Allow extensions to add to the list of tables to duplicate;
                // may be necessary if they hook into page save or other code
                // which will require them while running tests.
@@ -717,12 +625,12 @@ class ParserTest {
         * Currently this will only be done once per run, and any changes to
         * the db will be visible to later tests in the run.
         */
-       private function setupDatabase() {
+       function setupDatabase() {
                global $wgDBprefix, $wgDBtype;
                if ( $this->databaseSetupDone ) {
                        return;
                }
-               if ( $wgDBprefix === 'parsertest_' ) {
+               if ( $wgDBprefix === 'parsertest_' || ($wgDBtype == 'oracle' && $wgDBprefix === 'pt_')) {
                        throw new MWException( 'setupDatabase should be called before setupGlobals' );
                }
                $this->databaseSetupDone = true;
@@ -746,17 +654,19 @@ class ParserTest {
                        # fix back and forth so tableName() works right.
                        $this->changePrefix( $this->oldTablePrefix );
                        $oldTableName = $db->tableName( $tbl );
-                       $this->changePrefix( 'parsertest_' );
+                       $this->changePrefix( $wgDBtype != 'oracle' ? 'parsertest_' : 'pt_' );
                        $newTableName = $db->tableName( $tbl );
 
-                       if ( $db->tableExists( $tbl ) && $wgDBtype != 'postgres' ) {
+                       if ( $db->tableExists( $tbl ) && $wgDBtype != 'postgres' && $wgDBtype != 'oracle' ) {
                                $db->query( "DROP TABLE $newTableName" );
                        }
                        # Create new table
                        $db->duplicateTableStructure( $oldTableName, $newTableName, $temporary );
                }
+               if ($wgDBtype == 'oracle')
+                       $db->query('BEGIN FILL_WIKI_INFO; END;');
 
-               $this->changePrefix( 'parsertest_' );
+               $this->changePrefix( $wgDBtype != 'oracle' ? 'parsertest_' : 'pt_' );
 
                # Hack: insert a few Wikipedia in-project interwiki prefixes,
                # for testing inter-language links
@@ -781,6 +691,21 @@ class ParserTest {
                                   'iw_local'  => 1 ),
                        ) );
 
+
+               if ($wgDBtype == 'oracle') {
+                       # Insert 0 and 1 user_ids to prevent FK violations
+
+                       #Anonymous user
+                       $db->insert( 'user', array(
+                               'user_id'         => 0,
+                               'user_name'       => 'Anonymous') );
+
+                       # Hack-on-Hack: Insert a test user to be able to insert an image
+                       $db->insert( 'user', array(
+                               'user_id'         => 1,
+                               'user_name'       => 'Tester') );
+               }
+
                # Hack: Insert an image to work with
                $db->insert( 'image', array(
                        'img_name'        => 'Foobar.jpg',
@@ -798,11 +723,32 @@ class ParserTest {
                        'img_metadata'    => serialize( array() ),
                        ) );
 
+               # This image will be blacklisted in [[MediaWiki:Bad image list]]
+               $db->insert( 'image', array(
+                       'img_name'        => 'Bad.jpg',
+                       'img_size'        => 12345,
+                       'img_description' => 'zomgnotcensored',
+                       'img_user'        => 1,
+                       'img_user_text'   => 'WikiSysop',
+                       'img_timestamp'   => $db->timestamp( '20010115123500' ),
+                       'img_width'       => 320,
+                       'img_height'      => 240,
+                       'img_bits'        => 24,
+                       'img_media_type'  => MEDIATYPE_BITMAP,
+                       'img_major_mime'  => "image",
+                       'img_minor_mime'  => "jpeg",
+                       'img_metadata'    => serialize( array() ),
+                       ) );
+
                # Update certain things in site_stats
-               $db->insert( 'site_stats', array( 'ss_row_id' => 1, 'ss_images' => 1, 'ss_good_articles' => 1 ) );
+               $db->insert( 'site_stats', array( 'ss_row_id' => 1, 'ss_images' => 2, 'ss_good_articles' => 1 ) );
 
                # Reinitialise the LocalisationCache to match the database state
                Language::getLocalisationCache()->unloadAll();
+
+               # Make a new message cache
+               global $wgMessageCache, $wgMemc;
+               $wgMessageCache = new MessageCache( $wgMemc, true, 3600, '' );
        }
 
        /**
@@ -823,7 +769,7 @@ class ParserTest {
        }
 
        private function teardownDatabase() {
-               global $wgDBprefix;
+               global $wgDBtype;
                if ( !$this->databaseSetupDone ) {
                        return;
                }
@@ -838,14 +784,19 @@ class ParserTest {
                $tables = $this->listTables();
                $db = wfGetDB( DB_MASTER );
                foreach ( $tables as $table ) {
-                       $db->query( "DROP TABLE `parsertest_$table`" );
-               }*/
+                       $sql = $wgDBtype == 'oracle' ? "DROP TABLE pt_$table DROP CONSTRAINTS" : "DROP TABLE `parsertest_$table`";
+                       $db->query( $sql );
+               }
+               if ($wgDBtype == 'oracle')
+                       $db->query('BEGIN FILL_WIKI_INFO; END;');
+               */
        }
-       
+
        /**
         * Create a dummy uploads directory which will contain a couple
         * of files in order to pass existence tests.
-        * @return string The directory
+        *
+        * @return String: the directory
         */
        private function setupUploadDir() {
                global $IP;
@@ -865,6 +816,9 @@ class ParserTest {
                }
                wfMkdirParents( $dir . '/3/3a' );
                copy( "$IP/skins/monobook/headbg.jpg", "$dir/3/3a/Foobar.jpg" );
+
+               wfMkdirParents( $dir . '/0/09' );
+               copy( "$IP/skins/monobook/headbg.jpg", "$dir/0/09/Bad.jpg" );
                return $dir;
        }
 
@@ -900,6 +854,10 @@ class ParserTest {
                                "$dir/thumb/3/3a/Foobar.jpg/200px-Foobar.jpg",
                                "$dir/thumb/3/3a/Foobar.jpg/640px-Foobar.jpg",
                                "$dir/thumb/3/3a/Foobar.jpg/120px-Foobar.jpg",
+
+                               "$dir/0/09/Bad.jpg",
+
+                               "$dir/math/f/a/5/fa50b8b616463173474302ca3e63586b.png",
                        )
                );
 
@@ -912,7 +870,14 @@ class ParserTest {
                                "$dir/thumb/3/3a/Foobar.jpg",
                                "$dir/thumb/3/3a",
                                "$dir/thumb/3",
+
+                               "$dir/0/09/",
+                               "$dir/0/",
                                "$dir/thumb",
+                               "$dir/math/f/a/5",
+                               "$dir/math/f/a",
+                               "$dir/math/f",
+                               "$dir/math",
                                "$dir",
                        )
                );
@@ -920,7 +885,7 @@ class ParserTest {
 
        /**
         * Delete the specified files, if they exist.
-        * @param array $files full paths to files to delete.
+        * @param $files Array: full paths to files to delete.
         */
        private static function deleteFiles( $files ) {
                foreach( $files as $file ) {
@@ -932,7 +897,7 @@ class ParserTest {
 
        /**
         * Delete the specified directories, if they exist. Must be empty.
-        * @param array $dirs full paths to directories to delete.
+        * @param $dirs Array: full paths to directories to delete.
         */
        private static function deleteDirs( $dirs ) {
                foreach( $dirs as $dir ) {
@@ -952,8 +917,8 @@ class ParserTest {
        /**
         * Print a happy success message.
         *
-        * @param string $desc The test name
-        * @return bool
+        * @param $desc String: the test name
+        * @return Boolean
         */
        protected function showSuccess( $desc ) {
                if( $this->showProgress ) {
@@ -966,10 +931,10 @@ class ParserTest {
         * Print a failure message and provide some explanatory output
         * about what went wrong if so configured.
         *
-        * @param string $desc The test name
-        * @param string $result Expected HTML output
-        * @param string $html Actual HTML output
-        * @return bool
+        * @param $desc String: the test name
+        * @param $result String: expected HTML output
+        * @param $html String: actual HTML output
+        * @return Boolean
         */
        protected function showFailure( $desc, $result, $html ) {
                if( $this->showFailure ) {
@@ -996,11 +961,11 @@ class ParserTest {
         * Run given strings through a diff and return the (colorized) output.
         * Requires writable /tmp directory and a 'diff' command in the PATH.
         *
-        * @param string $input
-        * @param string $output
-        * @param string $inFileTail Tailing for the input file name
-        * @param string $outFileTail Tailing for the output file name
-        * @return string
+        * @param $input String
+        * @param $output String
+        * @param $inFileTail String: tailing for the input file name
+        * @param $outFileTail String: tailing for the output file name
+        * @return String
         */
        protected function quickDiff( $input, $output, $inFileTail='expected', $outFileTail='actual' ) {
                $prefix = wfTempDir() . "/mwParser-" . mt_rand();
@@ -1021,8 +986,8 @@ class ParserTest {
        /**
         * Write the given string to a file, adding a final newline.
         *
-        * @param string $data
-        * @param string $filename
+        * @param $data String
+        * @param $filename String
         */
        private function dumpToFile( $data, $filename ) {
                $file = fopen( $filename, "wt" );
@@ -1034,8 +999,8 @@ class ParserTest {
         * Colorize unified diff output if set for ANSI color output.
         * Subtractions are colored blue, additions red.
         *
-        * @param string $text
-        * @return string
+        * @param $text String
+        * @return String
         */
        protected function colorDiff( $text ) {
                return preg_replace(
@@ -1048,9 +1013,9 @@ class ParserTest {
        /**
         * Show "Reading tests from ..."
         *
-        * @param String $path
+        * @param $path String
         */
-       protected function showRunFile( $path ){
+       public function showRunFile( $path ){
                print $this->term->color( 1 ) .
                        "Reading tests from \"$path\"..." .
                        $this->term->reset() .
@@ -1059,11 +1024,11 @@ class ParserTest {
 
        /**
         * Insert a temporary test article
-        * @param string $name the title, including any prefix
-        * @param string $text the article text
-        * @param int $line the input line number, for reporting errors
+        * @param $name String: the title, including any prefix
+        * @param $text String: the article text
+        * @param $line Integer: the input line number, for reporting errors
         */
-       private function addArticle($name, $text, $line) {
+       public function addArticle($name, $text, $line) {
                $this->setupGlobals();
                $title = Title::newFromText( $name );
                if ( is_null($title) ) {
@@ -1072,11 +1037,12 @@ class ParserTest {
 
                $aid = $title->getArticleID( GAID_FOR_UPDATE );
                if ($aid != 0) {
-                       wfDie( "duplicate article at line $line\n" );
+                       wfDie( "duplicate article '$name' at line $line\n" );
                }
 
                $art = new Article($title);
                $art->insertNewArticle($text, '', false, false );
+
                $this->teardownGlobals();
        }
 
@@ -1084,9 +1050,10 @@ class ParserTest {
         * Steal a callback function from the primary parser, save it for
         * application to our scary parser. If the hook is not installed,
         * die a painful dead to warn the others.
-        * @param string $name
+        *
+        * @param $name String
         */
-       private function requireHook( $name ) {
+       public function requireHook( $name ) {
                global $wgParser;
                $wgParser->firstCallInit( ); //make sure hooks are loaded.
                if( isset( $wgParser->mTagHooks[$name] ) ) {
@@ -1100,9 +1067,10 @@ class ParserTest {
         * Steal a callback function from the primary parser, save it for
         * application to our scary parser. If the hook is not installed,
         * die a painful dead to warn the others.
-        * @param string $name
+        *
+        * @param $name String
         */
-       private function requireFunctionHook( $name ) {
+       public function requireFunctionHook( $name ) {
                global $wgParser;
                $wgParser->firstCallInit( ); //make sure hooks are loaded.
                if( isset( $wgParser->mFunctionHooks[$name] ) ) {
@@ -1116,8 +1084,8 @@ class ParserTest {
         * Run the "tidy" command on text if the $wgUseTidy
         * global is true
         *
-        * @param string $text the text to tidy
-        * @return string
+        * @param $text String: the text to tidy
+        * @return String
         * @static
         */
        private function tidy( $text ) {
@@ -1184,8 +1152,8 @@ class AnsiTermColorer {
        /**
         * Return ANSI terminal escape code for changing text attribs/color
         *
-        * @param string $color Semicolon-separated list of attribute/color codes
-        * @return string
+        * @param $color String: semicolon-separated list of attribute/color codes
+        * @return String
         */
        public function color( $color ) {
                global $wgCommandLineDarkBg;
@@ -1196,7 +1164,7 @@ class AnsiTermColorer {
        /**
         * Return ANSI terminal escape code for restoring default text attributes
         *
-        * @return string
+        * @return String
         */
        public function reset() {
                return $this->color( 0 );
@@ -1281,11 +1249,11 @@ class DbTestPreviewer extends TestRecorder  {
         * and all that fun stuff
         */
        function start() {
-               global $wgDBtype, $wgDBprefix;
+               global $wgDBtype;
                parent::start();
 
-               if( ! $this->db->tableExists( 'testrun' ) 
-                       or ! $this->db->tableExists( 'testitem' ) ) 
+               if( ! $this->db->tableExists( 'testrun' )
+                       or ! $this->db->tableExists( 'testitem' ) )
                {
                        print "WARNING> `testrun` table not found in database.\n";
                        $this->prevRun = false;
@@ -1321,7 +1289,7 @@ class DbTestPreviewer extends TestRecorder  {
                        $res = $this->db->select( 'testitem', array( 'ti_name', 'ti_success' ),
                                array( 'ti_run' => $this->prevRun ), __METHOD__ );
                        foreach ( $res as $row ) {
-                               if ( !$this->parent->regex 
+                               if ( !$this->parent->regex
                                        || preg_match( "/{$this->parent->regex}/i", $row->ti_name ) )
                                {
                                        $prevResults[$row->ti_name] = $row->ti_success;
@@ -1394,7 +1362,7 @@ class DbTestPreviewer extends TestRecorder  {
 
                // Otherwise, this test has previous recorded results.
                // See when this test last had a different result to what we're seeing now.
-               $conds = array( 
+               $conds = array(
                        'ti_name'    => $testname,
                        'ti_success' => ($after == 'f' ? "1" : "0") );
                if ( $this->curRun ) {
@@ -1454,20 +1422,22 @@ class DbTestRecorder extends DbTestPreviewer  {
         * and all that fun stuff
         */
        function start() {
-               global $wgDBtype, $wgDBprefix, $options;
+               global $wgDBtype, $options;
                $this->db->begin();
 
-               if( ! $this->db->tableExists( 'testrun' ) 
-                       or ! $this->db->tableExists( 'testitem' ) ) 
+               if( ! $this->db->tableExists( 'testrun' )
+                       or ! $this->db->tableExists( 'testitem' ) )
                {
                        print "WARNING> `testrun` table not found in database. Trying to create table.\n";
                        if ($wgDBtype === 'postgres')
                                $this->db->sourceFile( dirname(__FILE__) . '/testRunner.postgres.sql' );
+                       elseif ($wgDBtype === 'oracle')
+                               $this->db->sourceFile( dirname(__FILE__) . '/testRunner.ora.sql' );
                        else
                                $this->db->sourceFile( dirname(__FILE__) . '/testRunner.sql' );
                        echo "OK, resuming.\n";
                }
-               
+
                parent::start();
 
                $this->db->insert( 'testrun',
@@ -1488,8 +1458,9 @@ class DbTestRecorder extends DbTestPreviewer  {
 
        /**
         * Record an individual test item's success or failure to the db
-        * @param string $test
-        * @param bool $result
+        *
+        * @param $test String
+        * @param $result Boolean
         */
        function record( $test, $result ) {
                parent::record( $test, $result );
@@ -1509,37 +1480,37 @@ class RemoteTestRecorder extends TestRecorder {
                $this->results = array();
                $this->ping( 'running' );
        }
-       
+
        function record( $test, $result ) {
                parent::record( $test, $result );
                $this->results[$test] = (bool)$result;
        }
-       
+
        function end() {
                $this->ping( 'complete', $this->results );
                parent::end();
        }
-       
+
        /**
         * Inform a CodeReview instance that we've started or completed a test run...
-        * @param $remote array: info on remote target
+        *
         * @param $status string: "running" - tell it we've started
         *                        "complete" - provide test results array
         *                        "abort" - something went horribly awry
-        * @param $data array of test name => true/false
+        * @param $results array of test name => true/false
         */
        function ping( $status, $results=false ) {
                global $wgParserTestRemote, $IP;
-               
+
                $remote = $wgParserTestRemote;
                $revId = SpecialVersion::getSvnRevision( $IP );
                $jsonResults = json_encode( $results );
-               
+
                if( !$remote ) {
                        print "Can't do remote upload without configuring \$wgParserTestRemote!\n";
                        exit( 1 );
                }
-               
+
                // Generate a hash MAC to validate our credentials
                $message = array(
                        $remote['repo'],
@@ -1551,7 +1522,7 @@ class RemoteTestRecorder extends TestRecorder {
                        $message[] = $jsonResults;
                }
                $hmac = hash_hmac( "sha1", implode( "|", $message ), $remote['secret'] );
-               
+
                $postData = array(
                        'action' => 'codetestupload',
                        'format' => 'json',
@@ -1565,7 +1536,7 @@ class RemoteTestRecorder extends TestRecorder {
                        $postData['results'] = $jsonResults;
                }
                $response = $this->post( $remote['api-url'], $postData );
-               
+
                if( $response === false ) {
                        print "CodeReview info upload failed to reach server.\n";
                        exit( 1 );
@@ -1583,10 +1554,176 @@ class RemoteTestRecorder extends TestRecorder {
                        exit( 1 );
                }
        }
-       
+
        function post( $url, $data ) {
-               // @fixme: for whatever reason, I get a 417 fail when using CURL's multipart form submit.
-               // If we do form URL encoding ourselves, though, it should work.
-               return Http::post( $url, array( 'postdata' => wfArrayToCGI( $data ) ) );
+               return Http::post( $url, array( 'postData' => $data) );
+       }
+}
+
+class TestFileIterator implements Iterator {
+    private $file;
+    private $fh;
+    private $parser;
+    private $index = 0;
+    private $test;
+       private $lineNum;
+       private $eof;
+
+       function __construct( $file, $parser = null ) {
+               global $IP;
+
+               $this->file = $file;
+       $this->fh = fopen($this->file, "rt");
+        if( !$this->fh ) {
+                       wfDie( "Couldn't open file '$file'\n" );
+               }
+
+               $this->parser = $parser;
+
+               if( $this->parser ) $this->parser->showRunFile( wfRelativePath( $this->file, $IP ) );
+               $this->lineNum = $this->index = 0;
+       }
+
+       function setParser( ParserTest $parser ) {
+               $this->parser = $parser;
+       }
+
+       function rewind() {
+               if(fseek($this->fh, 0)) {
+                       wfDie( "Couldn't fseek to the start of '$filename'\n" );
+               }
+               $this->index = 0;
+               $this->lineNum = 0;
+               $this->eof = false;
+               $this->readNextTest();
+
+               return true;
+    }
+
+    function current() {
+               return $this->test;
+    }
+
+    function key() {
+               return $this->index;
+    }
+
+    function next() {
+        if($this->readNextTest()) {
+                       $this->index++;
+                       return true;
+               } else {
+                       $this->eof = true;
+               }
+    }
+
+    function valid() {
+               return $this->eof != true;
+    }
+
+       function readNextTest() {
+               $data = array();
+               $section = null;
+
+               while( false !== ($line = fgets( $this->fh ) ) ) {
+                       $this->lineNum++;
+                       $matches = array();
+                       if( preg_match( '/^!!\s*(\w+)/', $line, $matches ) ) {
+                               $section = strtolower( $matches[1] );
+                               if( $section == 'endarticle') {
+                                       if( !isset( $data['text'] ) ) {
+                                               wfDie( "'endarticle' without 'text' at line {$this->lineNum} of $filename\n" );
+                                       }
+                                       if( !isset( $data['article'] ) ) {
+                                               wfDie( "'endarticle' without 'article' at line {$this->lineNum} of $filename\n" );
+                                       }
+                                       if( $this->parser ) $this->parser->addArticle($this->parser->chomp($data['article']), $this->parser->chomp($data['text']),
+                                                                                         $this->lineNum);
+                                       $data = array();
+                                       $section = null;
+                                       continue;
+                               }
+                               if( $section == 'endhooks' ) {
+                                       if( !isset( $data['hooks'] ) ) {
+                                               wfDie( "'endhooks' without 'hooks' at line {$this->lineNum} of $filename\n" );
+                                       }
+                                       foreach( explode( "\n", $data['hooks'] ) as $line ) {
+                                               $line = trim( $line );
+                                               if( $line ) {
+                                                       if( $this->parser ) $this->parser->requireHook( $line );
+                                               }
+                                       }
+                                       $data = array();
+                                       $section = null;
+                                       continue;
+                               }
+                               if( $section == 'endfunctionhooks' ) {
+                                       if( !isset( $data['functionhooks'] ) ) {
+                                               wfDie( "'endfunctionhooks' without 'functionhooks' at line {$this->lineNum} of $filename\n" );
+                                       }
+                                       foreach( explode( "\n", $data['functionhooks'] ) as $line ) {
+                                               $line = trim( $line );
+                                               if( $line ) {
+                                                       if( $this->parser ) $this->parser->requireFunctionHook( $line );
+                                               }
+                                       }
+                                       $data = array();
+                                       $section = null;
+                                       continue;
+                               }
+                               if( $section == 'end' ) {
+                                       if( !isset( $data['test'] ) ) {
+                                               wfDie( "'end' without 'test' at line {$this->lineNum} of $filename\n" );
+                                       }
+                                       if( !isset( $data['input'] ) ) {
+                                               wfDie( "'end' without 'input' at line {$this->lineNum} of $filename\n" );
+                                       }
+                                       if( !isset( $data['result'] ) ) {
+                                               wfDie( "'end' without 'result' at line {$this->lineNum} of $filename\n" );
+                                       }
+                                       if( !isset( $data['options'] ) ) {
+                                               $data['options'] = '';
+                                       }
+                                       if (!isset( $data['config'] ) )
+                                               $data['config'] = '';
+
+                                       if ( $this->parser && (preg_match('/\\bdisabled\\b/i', $data['options'])
+                                               || !preg_match("/{$this->parser->regex}/i", $data['test'])) && !$this->parser->runDisabled ) {
+                                               # disabled test
+                                               $data = array();
+                                               $section = null;
+                                               continue;
+                                       }
+                                       if ( $this->parser &&
+                                                preg_match('/\\bmath\\b/i', $data['options']) && !$this->parser->savedGlobals['wgUseTeX'] ) {
+                                               # don't run math tests if $wgUseTeX is set to false in LocalSettings
+                                               $data = array();
+                                               $section = null;
+                                               continue;
+                                       }
+
+                                       if( $this->parser ) {
+                                               $this->test = array(
+                                                       'test' => $this->parser->chomp( $data['test'] ),
+                                                       'input' => $this->parser->chomp( $data['input'] ),
+                                                       'result' => $this->parser->chomp( $data['result'] ),
+                                                       'options' => $this->parser->chomp( $data['options'] ),
+                                                       'config' => $this->parser->chomp( $data['config'] ) );
+                                       } else {
+                                               $this->test['test'] = $data['test'];
+                                       }
+                                       return true;
+                               }
+                               if ( isset ($data[$section] ) ) {
+                                       wfDie( "duplicate section '$section' at line {$this->lineNum} of $filename\n" );
+                               }
+                               $data[$section] = '';
+                               continue;
+                       }
+                       if( $section ) {
+                               $data[$section] .= $line;
+                       }
+               }
+               return false;
        }
 }