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

Revision: 91211
Author:   demon
Date:     2011-06-30 19:47:26 +0000 (Thu, 30 Jun 2011)
Log Message:
-----------
Remove --upload option to parserTests.php and backend RemoteTestRecorder -- was 
only used for since-removed CR integration

Modified Paths:
--------------
    trunk/phase3/includes/AutoLoader.php
    trunk/phase3/tests/parser/parserTest.inc
    trunk/phase3/tests/parserTests.php
    trunk/phase3/tests/testHelpers.inc

Modified: trunk/phase3/includes/AutoLoader.php
===================================================================
--- trunk/phase3/includes/AutoLoader.php        2011-06-30 19:45:08 UTC (rev 
91210)
+++ trunk/phase3/includes/AutoLoader.php        2011-06-30 19:47:26 UTC (rev 
91211)
@@ -846,7 +846,6 @@
        'DbTestPreviewer' => 'tests/testHelpers.inc',
        'DbTestRecorder' => 'tests/testHelpers.inc',
        'DummyTermColorer' => 'tests/testHelpers.inc',
-       'RemoteTestRecorder' => 'tests/testHelpers.inc',
        'TestFileIterator' => 'tests/testHelpers.inc',
        'TestRecorder' => 'tests/testHelpers.inc',
 

Modified: trunk/phase3/tests/parser/parserTest.inc
===================================================================
--- trunk/phase3/tests/parser/parserTest.inc    2011-06-30 19:45:08 UTC (rev 
91210)
+++ trunk/phase3/tests/parser/parserTest.inc    2011-06-30 19:47:26 UTC (rev 
91211)
@@ -185,8 +185,6 @@
                                        $options['setversion'] : 
SpecialVersion::getVersion();
                } elseif ( isset( $options['compare'] ) ) {
                        $this->recorder = new DbTestPreviewer( $this );
-               } elseif ( isset( $options['upload'] ) ) {
-                       $this->recorder = new RemoteTestRecorder( $this );
                } else {
                        $this->recorder = new TestRecorder( $this );
                }

Modified: trunk/phase3/tests/parserTests.php
===================================================================
--- trunk/phase3/tests/parserTests.php  2011-06-30 19:45:08 UTC (rev 91210)
+++ trunk/phase3/tests/parserTests.php  2011-06-30 19:47:26 UTC (rev 91211)
@@ -51,7 +51,6 @@
   --seed <n>       Start the fuzz test from the specified seed
   --help           Show this help message
   --run-disabled   run disabled tests
-  --upload         Upload test results to remote wiki (per 
\$wgParserTestRemote)
 
 ENDS;
        exit( 0 );

Modified: trunk/phase3/tests/testHelpers.inc
===================================================================
--- trunk/phase3/tests/testHelpers.inc  2011-06-30 19:45:08 UTC (rev 91210)
+++ trunk/phase3/tests/testHelpers.inc  2011-06-30 19:47:26 UTC (rev 91211)
@@ -349,99 +349,6 @@
        }
 }
 
-class RemoteTestRecorder extends TestRecorder {
-       function start() {
-               parent::start();
-
-               $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 $status string: "running" - tell it we've started
-        *                        "complete" - provide test results array
-        *                        "abort" - something went horribly awry
-        * @param $results array of test name => true/false
-        */
-       function ping( $status, $results = false ) {
-               global $wgParserTestRemote, $IP;
-
-               $remote = $wgParserTestRemote;
-               $revId = SpecialVersion::getSvnRevision( $IP );
-               $jsonResults = FormatJson::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'],
-                       $remote['suite'],
-                       $revId,
-                       $status,
-               );
-
-               if ( $status == "complete" ) {
-                       $message[] = $jsonResults;
-               }
-               $hmac = hash_hmac( "sha1", implode( "|", $message ), 
$remote['secret'] );
-
-               $postData = array(
-                       'action' => 'codetestupload',
-                       'format' => 'json',
-                       'repo'   => $remote['repo'],
-                       'suite'  => $remote['suite'],
-                       'rev'    => $revId,
-                       'status' => $status,
-                       'hmac'   => $hmac,
-               );
-
-               if ( $status == "complete" ) {
-                       $postData['results'] = $jsonResults;
-               }
-
-               $response = $this->post( $remote['api-url'], $postData );
-
-               if ( $response === false ) {
-                       print "CodeReview info upload failed to reach 
server.\n";
-                       exit( 1 );
-               }
-
-               $responseData = FormatJson::decode( $response, true );
-
-               if ( !is_array( $responseData ) ) {
-                       print "CodeReview API response not recognized...\n";
-                       wfDebug( "Unrecognized CodeReview API response: 
$response\n" );
-                       exit( 1 );
-               }
-
-               if ( isset( $responseData['error'] ) ) {
-                       $code = $responseData['error']['code'];
-                       $info = $responseData['error']['info'];
-                       print "CodeReview info upload failed: $code $info\n";
-                       exit( 1 );
-               }
-       }
-
-       function post( $url, $data ) {
-               return Http::post( $url, array( 'postData' => $data ) );
-       }
-}
-
 class TestFileIterator implements Iterator {
        private $file;
        private $fh;


_______________________________________________
MediaWiki-CVS mailing list
MediaWiki-CVS@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs

Reply via email to