1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
|
<?php
namespace MediaWiki\Tests\Maintenance;
use MediaWiki\User\UserIdentity;
use MoveBatch;
/**
* @covers \MoveBatch
* @group Database
* @author Dreamy Jazz
*/
class MoveBatchTest extends MaintenanceBaseTestCase {
private static UserIdentity $testPerformer;
public function getMaintenanceClass() {
return MoveBatch::class;
}
public function addDBDataOnce() {
self::$testPerformer = $this->getTestUser()->getUserIdentity();
}
private function getFileWithContent( string $content ): string {
$testFilename = $this->getNewTempFile();
$testFile = fopen( $testFilename, 'w' );
fwrite( $testFile, $content );
fclose( $testFile );
return $testFilename;
}
/** @dataProvider provideInvalidPerformerUsernames */
public function testExecuteForInvalidPerformer( $performerUsername ) {
$this->expectCallToFatalError();
$this->expectOutputRegex( '/Invalid username/' );
$this->maintenance->setArg( 'listfile', $this->getNewTempFile() );
$this->maintenance->setOption( 'u', $performerUsername );
$this->maintenance->execute();
}
public static function provideInvalidPerformerUsernames() {
return [
'Username is invalid' => [ 'Template:Testing#test' ],
'Username does not exist' => [ 'Non-existing-test-user-1234' ],
];
}
/** @dataProvider provideExecuteOnFailedMove */
public function testExecuteOnFailedMove( $fileInputContent, $expectedOutputRegex ) {
$this->expectOutputRegex( $expectedOutputRegex );
$this->maintenance->setArg( 'listfile', $this->getFileWithContent( $fileInputContent ) );
$this->maintenance->execute();
}
public static function provideExecuteOnFailedMove() {
return [
'Line is missing line break' => [ 'Template:Test#testing', '/Error on line 1, no pipe character/' ],
'Invalid source title' => [ ':::|Abc', '/Invalid title on line 1/' ],
'Invalid destination title' => [ 'Abc|:::', '/Invalid title on line 1/' ],
'Non-existing source title' => [ 'Abc|Def', '/Abc --> Def FAILED[\s\S]*Abc doesn\'t exist/' ],
];
}
/** @dataProvider provideExecuteForGoodMove */
public function testExecuteForGoodMove(
$options, $shouldCreateRedirect, $expectedReason, $expectedPerformerUsernameCallback
) {
foreach ( $options as $name => $value ) {
if ( is_callable( $value ) ) {
$value = $value();
}
$this->maintenance->setOption( $name, $value );
}
// Get a source page and destination page
$sourcePage = $this->getExistingTestPage();
$sourcePageContentBeforeMove = $sourcePage->getContent()->getWikitextForTransclusion();
$destPage = $this->getNonexistingTestPage();
// Move the page using the maintenance script
$this->maintenance->setArg(
'listfile',
$this->getFileWithContent( "$sourcePage|$destPage" )
);
$this->maintenance->execute();
// Validate that the move occurred
$sourcePage->clear();
$destPage->clear();
// First check that the source page either is a redirect or is deleted, depending on the
// --noredirects option being provided.
if ( $shouldCreateRedirect ) {
$this->assertTrue( $sourcePage->getContent()->isRedirect() );
} else {
$this->assertFalse( $sourcePage->exists() );
}
// Next check that the content of the destination page is the same as the source page (as the move should not
// have modified the content).
$this->assertSame( $sourcePageContentBeforeMove, $destPage->getContent()->getWikitextForTransclusion() );
// Check the reason for the move is as expected
$this->newSelectQueryBuilder()
->select( 'comment_text' )
->from( 'logging' )
->join( 'comment', null, 'log_comment_id=comment_id' )
->where( [ 'log_type' => 'move' ] )
->assertFieldValue( $expectedReason );
// Check the performer of the move is as expected
$this->newSelectQueryBuilder()
->select( 'actor_name' )
->from( 'logging' )
->join( 'actor', null, 'log_actor=actor_id' )
->where( [ 'log_type' => 'move' ] )
->assertFieldValue( $expectedPerformerUsernameCallback() );
$this->expectOutputString( "$sourcePage --> $destPage\n" );
}
public static function provideExecuteForGoodMove() {
return [
'No options provided' => [ [], true, '', fn () => 'Move page script' ],
'--noredirects set, custom reason, and custom performer' => [
[ 'noredirects' => 1, 'r' => 'Test', 'u' => fn () => static::$testPerformer->getName() ],
false, 'Test', fn () => static::$testPerformer->getName()
],
];
}
}
|
