Thijs/wiki.techinc.nl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Remove --upload option to parserTests.php and backend RemoteTestRecorder -- was only used for since-removed CR integration

Browse source
This commit is contained in:
Chad Horohoe 2011-06-30 19:47:26 +00:00
parent c82602778b
commit 3ddafb719a
4 changed files with 0 additions and 97 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
includes/AutoLoader.php
View file

@ -846,7 +846,6 @@ $wgAutoloadLocalClasses = array(
'DbTestPreviewer' => 'tests/testHelpers.inc',
'DbTestRecorder' => 'tests/testHelpers.inc',
'DummyTermColorer' => 'tests/testHelpers.inc',
'RemoteTestRecorder' => 'tests/testHelpers.inc',
'TestFileIterator' => 'tests/testHelpers.inc',
'TestRecorder' => 'tests/testHelpers.inc',

2
tests/parser/parserTest.inc
View file

@ -185,8 +185,6 @@ class ParserTest {
$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 );
}

1
tests/parserTests.php
View file

@ -51,7 +51,6 @@ Options:
--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 );

93
tests/testHelpers.inc
View file

@ -349,99 +349,6 @@ class DbTestRecorder extends DbTestPreviewer {
}
}
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;