Merge "Code style cleanups to parserTest.inc and NewParserTest.php."
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Sat, 3 Aug 2013 00:07:11 +0000 (00:07 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Sat, 3 Aug 2013 00:07:11 +0000 (00:07 +0000)
1  2 
tests/parser/parserTest.inc

@@@ -434,12 -434,7 +434,12 @@@ class ParserTest 
         */
        public function runTestsFromFiles( $filenames ) {
                $ok = false;
 +
 +              // be sure, ParserTest::addArticle has correct language set,
 +              // so that system messages gets into the right language cache
 +              $GLOBALS['wgLanguageCode'] = 'en';
                $GLOBALS['wgContLang'] = Language::factory( 'en' );
 +
                $this->recorder->start();
                try {
                        $this->setupDatabase();
                }
  
                # Update certain things in site_stats
-               $this->db->insert( 'site_stats', array( 'ss_row_id' => 1, 'ss_images' => 2, 'ss_good_articles' => 1 ) );
+               $this->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();
                $tables = $this->listTables();
  
                foreach ( $tables as $table ) {
-                       $sql = $this->db->getType() == 'oracle' ? "DROP TABLE pt_$table DROP CONSTRAINTS" : "DROP TABLE `parsertest_$table`";
-                       $this->db->query( $sql );
+                       if ( $this->db->getType() == 'oracle' ) {
+                               $this->db->query( "DROP TABLE pt_$table DROP CONSTRAINTS" );
+                       } else {
+                               $this->db->query( "DROP TABLE `parsertest_$table`" );
+                       }
                }
  
                if ( $this->db->getType() == 'oracle' ) {
         * @param $outFileTail String: tailing for the output file name
         * @return String
         */
-       protected function quickDiff( $input, $output, $inFileTail = 'expected', $outFileTail = 'actual' ) {
+       protected function quickDiff( $input, $output,
+               $inFileTail = 'expected', $outFileTail = 'actual'
+       ) {
                # Windows, or at least the fc utility, is retarded
                $slash = wfIsWindows() ? '\\' : '/';
                $prefix = wfTempDir() . "{$slash}mwParser-" . mt_rand();