This removes most of the pre-actor user and user_text columns, and the $wgActorTableSchemaMigrationStage setting that used to determine whether the columns were used. rev_user and rev_user_text remain in the code, as on Wikimedia wikis the revision table is too large to alter at this time. A future change will combine that with the removal of rev_comment, rev_content_model, and rev_content_format (and the addition of rev_comment_id and rev_actor). ActorMigration's constructor continues to take a $stage parameter, and continues to have the logic for handling it, for the benefit of extensions that might need their own migration process. Code using ActorMigration for accessing the core fields should be updated to use the new actor fields directly. That will be done for in a followup. Bug: T188327 Change-Id: Id35544b879af1cd708f3efd303fce8d9a1b9eb02
319 lines
7.1 KiB
PHP
319 lines
7.1 KiB
PHP
<?php
|
|
|
|
namespace MediaWiki\Tests\Maintenance;
|
|
|
|
use DeleteAutoPatrolLogs;
|
|
|
|
/**
|
|
* @group Database
|
|
* @covers DeleteAutoPatrolLogs
|
|
*/
|
|
class DeleteAutoPatrolLogsTest extends MaintenanceBaseTestCase {
|
|
|
|
public function getMaintenanceClass() {
|
|
return DeleteAutoPatrolLogs::class;
|
|
}
|
|
|
|
public function setUp() {
|
|
parent::setUp();
|
|
$this->tablesUsed = [ 'logging' ];
|
|
|
|
$this->cleanLoggingTable();
|
|
$this->insertLoggingData();
|
|
}
|
|
|
|
private function cleanLoggingTable() {
|
|
wfGetDB( DB_MASTER )->delete( 'logging', '*' );
|
|
}
|
|
|
|
private function insertLoggingData() {
|
|
$dbw = wfGetDB( DB_MASTER );
|
|
$logs = [];
|
|
|
|
$comment = \MediaWiki\MediaWikiServices::getInstance()->getCommentStore()
|
|
->createComment( $dbw, '' );
|
|
|
|
// Manual patrolling
|
|
$logs[] = [
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'patrol',
|
|
'log_actor' => 7251,
|
|
'log_params' => '',
|
|
'log_timestamp' => $dbw->timestamp( '20041223210426' ),
|
|
'log_namespace' => NS_MAIN,
|
|
'log_title' => 'DeleteAutoPatrolLogs',
|
|
'log_comment_id' => $comment->id,
|
|
];
|
|
|
|
// Autopatrol #1
|
|
$logs[] = [
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'autopatrol',
|
|
'log_actor' => 7252,
|
|
'log_params' => '',
|
|
'log_timestamp' => $dbw->timestamp( '20051223210426' ),
|
|
'log_namespace' => NS_MAIN,
|
|
'log_title' => 'DeleteAutoPatrolLogs',
|
|
'log_comment_id' => $comment->id,
|
|
];
|
|
|
|
// Block
|
|
$logs[] = [
|
|
'log_type' => 'block',
|
|
'log_action' => 'block',
|
|
'log_actor' => 7253,
|
|
'log_params' => '',
|
|
'log_timestamp' => $dbw->timestamp( '20061223210426' ),
|
|
'log_namespace' => NS_MAIN,
|
|
'log_title' => 'DeleteAutoPatrolLogs',
|
|
'log_comment_id' => $comment->id,
|
|
];
|
|
|
|
// Very old/ invalid patrol
|
|
$logs[] = [
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'patrol',
|
|
'log_actor' => 7253,
|
|
'log_params' => 'nanana',
|
|
'log_timestamp' => $dbw->timestamp( '20061223210426' ),
|
|
'log_namespace' => NS_MAIN,
|
|
'log_title' => 'DeleteAutoPatrolLogs',
|
|
'log_comment_id' => $comment->id,
|
|
];
|
|
|
|
// Autopatrol #2
|
|
$logs[] = [
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'autopatrol',
|
|
'log_actor' => 7254,
|
|
'log_params' => '',
|
|
'log_timestamp' => $dbw->timestamp( '20071223210426' ),
|
|
'log_namespace' => NS_MAIN,
|
|
'log_title' => 'DeleteAutoPatrolLogs',
|
|
'log_comment_id' => $comment->id,
|
|
];
|
|
|
|
// Autopatrol #3 old way
|
|
$logs[] = [
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'patrol',
|
|
'log_actor' => 7255,
|
|
'log_params' => serialize( [ '6::auto' => true ] ),
|
|
'log_timestamp' => $dbw->timestamp( '20081223210426' ),
|
|
'log_namespace' => NS_MAIN,
|
|
'log_title' => 'DeleteAutoPatrolLogs',
|
|
'log_comment_id' => $comment->id,
|
|
];
|
|
|
|
// Manual patrol #2 old way
|
|
$logs[] = [
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'patrol',
|
|
'log_actor' => 7256,
|
|
'log_params' => serialize( [ '6::auto' => false ] ),
|
|
'log_timestamp' => $dbw->timestamp( '20091223210426' ),
|
|
'log_namespace' => NS_MAIN,
|
|
'log_title' => 'DeleteAutoPatrolLogs',
|
|
'log_comment_id' => $comment->id,
|
|
];
|
|
|
|
// Autopatrol #4 very old way
|
|
$logs[] = [
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'patrol',
|
|
'log_actor' => 7257,
|
|
'log_params' => "9227851\n0\n1",
|
|
'log_timestamp' => $dbw->timestamp( '20081223210426' ),
|
|
'log_namespace' => NS_MAIN,
|
|
'log_title' => 'DeleteAutoPatrolLogs',
|
|
'log_comment_id' => $comment->id,
|
|
];
|
|
|
|
// Manual patrol #3 very old way
|
|
$logs[] = [
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'patrol',
|
|
'log_actor' => 7258,
|
|
'log_params' => "9227851\n0\n0",
|
|
'log_timestamp' => $dbw->timestamp( '20091223210426' ),
|
|
'log_namespace' => NS_MAIN,
|
|
'log_title' => 'DeleteAutoPatrolLogs',
|
|
'log_comment_id' => $comment->id,
|
|
];
|
|
|
|
$dbw->insert( 'logging', $logs );
|
|
}
|
|
|
|
public function runProvider() {
|
|
$allRows = [
|
|
(object)[
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'patrol',
|
|
'log_actor' => '7251',
|
|
],
|
|
(object)[
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'autopatrol',
|
|
'log_actor' => '7252',
|
|
],
|
|
(object)[
|
|
'log_type' => 'block',
|
|
'log_action' => 'block',
|
|
'log_actor' => '7253',
|
|
],
|
|
(object)[
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'patrol',
|
|
'log_actor' => '7253',
|
|
],
|
|
(object)[
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'autopatrol',
|
|
'log_actor' => '7254',
|
|
],
|
|
(object)[
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'patrol',
|
|
'log_actor' => '7255',
|
|
],
|
|
(object)[
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'patrol',
|
|
'log_actor' => '7256',
|
|
],
|
|
(object)[
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'patrol',
|
|
'log_actor' => '7257',
|
|
],
|
|
(object)[
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'patrol',
|
|
'log_actor' => '7258',
|
|
],
|
|
];
|
|
|
|
$cases = [
|
|
'dry run' => [
|
|
$allRows,
|
|
[ '--sleep', '0', '--dry-run', '-q' ]
|
|
],
|
|
'basic run' => [
|
|
[
|
|
$allRows[0],
|
|
$allRows[2],
|
|
$allRows[3],
|
|
$allRows[5],
|
|
$allRows[6],
|
|
$allRows[7],
|
|
$allRows[8],
|
|
],
|
|
[ '--sleep', '0', '-q' ]
|
|
],
|
|
'run with before' => [
|
|
[
|
|
$allRows[0],
|
|
$allRows[2],
|
|
$allRows[3],
|
|
$allRows[4],
|
|
$allRows[5],
|
|
$allRows[6],
|
|
$allRows[7],
|
|
$allRows[8],
|
|
],
|
|
[ '--sleep', '0', '--before', '20060123210426', '-q' ]
|
|
],
|
|
'run with check-old' => [
|
|
[
|
|
$allRows[0],
|
|
$allRows[1],
|
|
$allRows[2],
|
|
$allRows[3],
|
|
$allRows[4],
|
|
$allRows[6],
|
|
$allRows[8],
|
|
],
|
|
[ '--sleep', '0', '--check-old', '-q' ]
|
|
],
|
|
];
|
|
|
|
foreach ( $cases as $key => $case ) {
|
|
yield $key . '-batch-size-1' => [
|
|
$case[0],
|
|
array_merge( $case[1], [ '--batch-size', '1' ] )
|
|
];
|
|
yield $key . '-batch-size-5' => [
|
|
$case[0],
|
|
array_merge( $case[1], [ '--batch-size', '5' ] )
|
|
];
|
|
yield $key . '-batch-size-1000' => [
|
|
$case[0],
|
|
array_merge( $case[1], [ '--batch-size', '1000' ] )
|
|
];
|
|
}
|
|
}
|
|
|
|
/**
|
|
* @dataProvider runProvider
|
|
*/
|
|
public function testRun( $expected, $args ) {
|
|
// FIXME: fails under postgres
|
|
$this->markTestSkippedIfDbType( 'postgres' );
|
|
|
|
$this->maintenance->loadWithArgv( $args );
|
|
|
|
$this->maintenance->execute();
|
|
|
|
$remainingLogs = wfGetDB( DB_REPLICA )->select(
|
|
[ 'logging' ],
|
|
[ 'log_type', 'log_action', 'log_actor' ],
|
|
[],
|
|
__METHOD__,
|
|
[ 'ORDER BY' => 'log_id' ]
|
|
);
|
|
|
|
$this->assertEquals( $expected, iterator_to_array( $remainingLogs, false ) );
|
|
}
|
|
|
|
public function testFromId() {
|
|
$fromId = wfGetDB( DB_REPLICA )->selectField(
|
|
'logging',
|
|
'log_id',
|
|
[ 'log_params' => 'nanana' ]
|
|
);
|
|
|
|
$this->maintenance->loadWithArgv( [ '--sleep', '0', '--from-id', strval( $fromId ), '-q' ] );
|
|
|
|
$this->maintenance->execute();
|
|
|
|
$remainingLogs = wfGetDB( DB_REPLICA )->select(
|
|
[ 'logging' ],
|
|
[ 'log_type', 'log_action', 'log_actor' ],
|
|
[],
|
|
__METHOD__,
|
|
[ 'ORDER BY' => 'log_id' ]
|
|
);
|
|
|
|
$deleted = [
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'autopatrol',
|
|
'log_actor' => '7254',
|
|
];
|
|
$notDeleted = [
|
|
'log_type' => 'patrol',
|
|
'log_action' => 'autopatrol',
|
|
'log_actor' => '7252',
|
|
];
|
|
|
|
$remainingLogs = array_map(
|
|
function ( $val ) {
|
|
return (array)$val;
|
|
},
|
|
iterator_to_array( $remainingLogs, false )
|
|
);
|
|
|
|
$this->assertNotContains( $deleted, $remainingLogs );
|
|
$this->assertContains( $notDeleted, $remainingLogs );
|
|
}
|
|
|
|
}
|