diff options
Diffstat (limited to 'tests/phpunit/includes')
31 files changed, 134 insertions, 86 deletions
diff --git a/tests/phpunit/includes/Permissions/PermissionManagerTest.php b/tests/phpunit/includes/Permissions/PermissionManagerTest.php index f9710fb7e897..8eb035ab671c 100644 --- a/tests/phpunit/includes/Permissions/PermissionManagerTest.php +++ b/tests/phpunit/includes/Permissions/PermissionManagerTest.php @@ -2,7 +2,7 @@ namespace MediaWiki\Tests\Integration\Permissions; -use Action; +use MediaWiki\Actions\Action; use MediaWiki\Api\ApiMessage; use MediaWiki\Block\AnonIpBlockTarget; use MediaWiki\Block\BlockActionInfo; diff --git a/tests/phpunit/includes/Storage/DerivedPageDataUpdaterTest.php b/tests/phpunit/includes/Storage/DerivedPageDataUpdaterTest.php index 882a0a8212b9..2338bf3ed8de 100644 --- a/tests/phpunit/includes/Storage/DerivedPageDataUpdaterTest.php +++ b/tests/phpunit/includes/Storage/DerivedPageDataUpdaterTest.php @@ -3,8 +3,6 @@ namespace MediaWiki\Tests\Storage; use DummyContentHandlerForTesting; -use LogPage; -use ManualLogEntry; use MediaWiki\CommentStore\CommentStoreComment; use MediaWiki\Config\ServiceOptions; use MediaWiki\Content\Content; @@ -18,6 +16,8 @@ use MediaWiki\Deferred\DeferredUpdates; use MediaWiki\Deferred\LinksUpdate\LinksUpdate; use MediaWiki\Deferred\MWCallableUpdate; use MediaWiki\Edit\ParsoidRenderID; +use MediaWiki\Logging\LogPage; +use MediaWiki\Logging\ManualLogEntry; use MediaWiki\MainConfigNames; use MediaWiki\Message\Message; use MediaWiki\Page\Event\PageRevisionUpdatedEvent; @@ -44,6 +44,7 @@ use MediaWikiIntegrationTestCase; use MockTitleTrait; use PHPUnit\Framework\Assert; use PHPUnit\Framework\MockObject\MockObject; +use RecentChange; use Wikimedia\ObjectCache\BagOStuff; use Wikimedia\Rdbms\Platform\ISQLPlatform; use Wikimedia\TestingAccessWrapper; @@ -1423,17 +1424,24 @@ class DerivedPageDataUpdaterTest extends MediaWikiIntegrationTestCase { public static function provideEnqueueRevertedTagUpdateJob() { return [ - 'approved' => [ true, 1 ], - 'not approved' => [ false, 0 ] + 'not patrolled' => [ true, 0, 0 ], + 'patrolled' => [ true, RecentChange::PRC_AUTOPATROLLED, 1 ], + 'autopatrolled' => [ true, RecentChange::PRC_AUTOPATROLLED, 1 ], + 'patrolling disabled' => [ false, 0, 1 ] ]; } /** * @covers \MediaWiki\Storage\DerivedPageDataUpdater::doUpdates - * @covers \MediaWiki\Storage\DerivedPageDataUpdater::maybeEnqueueRevertedTagUpdateJob + * @covers \MediaWiki\RecentChanges\ChangeTrackingEventIngress::updateRevertTagAfterPageUpdated * @dataProvider provideEnqueueRevertedTagUpdateJob */ - public function testEnqueueRevertedTagUpdateJob( bool $approved, int $queueSize ) { + public function testEnqueueRevertedTagUpdateJob( + bool $useRcPatrol, + int $rcPatrolStatus, + int $expectQueueSize + ) { + $this->overrideConfigValue( MainConfigNames::UseRCPatrol, $useRcPatrol ); $page = $this->getPage( __METHOD__ ); $content = [ SlotRecord::MAIN => new WikitextContent( '1' ) ]; @@ -1453,7 +1461,7 @@ class DerivedPageDataUpdaterTest extends MediaWikiIntegrationTestCase { $updater->prepareUpdate( $rev, [ 'editResult' => $editResult, - 'approved' => $approved + 'rcPatrolStatus' => $rcPatrolStatus, ] ); $updater->doUpdates(); @@ -1467,23 +1475,16 @@ class DerivedPageDataUpdaterTest extends MediaWikiIntegrationTestCase { ) ); - if ( $approved ) { - $this->assertNull( - $editResultCache->get( $rev->getId() ), - 'EditResult should not be cached when the revert is approved' - ); - } else { - $this->assertEquals( - $editResult, - $editResultCache->get( $rev->getId() ), - 'EditResult should be cached when the revert is not approved' - ); - } + $this->assertEquals( + $editResult, + $editResultCache->get( $rev->getId() ), + 'EditResult should be cached when the revert is not approved' + ); $jobQueueGroup = $this->getServiceContainer()->getJobQueueGroup(); $jobQueue = $jobQueueGroup->get( 'revertedTagUpdate' ); $this->assertSame( - $queueSize, + $expectQueueSize, $jobQueue->getSize() ); } diff --git a/tests/phpunit/includes/Storage/PageUpdaterTest.php b/tests/phpunit/includes/Storage/PageUpdaterTest.php index 7b4892c8f757..30cd345e604e 100644 --- a/tests/phpunit/includes/Storage/PageUpdaterTest.php +++ b/tests/phpunit/includes/Storage/PageUpdaterTest.php @@ -1418,7 +1418,9 @@ class PageUpdaterTest extends MediaWikiIntegrationTestCase { $services->getWikiPageFactory(), $services->getHookContainer(), $services->getUserNameUtils(), - $services->getTalkPageNotificationManager() + $services->getTalkPageNotificationManager(), + $services->getMainConfig(), + $services->getJobQueueGroup() ); $services->getDomainEventSource() diff --git a/tests/phpunit/includes/actions/RollbackActionTest.php b/tests/phpunit/includes/actions/RollbackActionTest.php index 8cf9557775ce..589ff979a049 100644 --- a/tests/phpunit/includes/actions/RollbackActionTest.php +++ b/tests/phpunit/includes/actions/RollbackActionTest.php @@ -4,6 +4,7 @@ namespace MediaWiki\Tests\Action; use Article; use ErrorPageError; +use MediaWiki\Actions\RollbackAction; use MediaWiki\Context\DerivativeContext; use MediaWiki\Context\RequestContext; use MediaWiki\Request\FauxRequest; @@ -11,10 +12,9 @@ use MediaWiki\Request\WebRequest; use MediaWiki\Title\Title; use MediaWiki\User\User; use MediaWikiIntegrationTestCase; -use RollbackAction; /** - * @covers \RollbackAction + * @covers \MediaWiki\Actions\RollbackAction * @group Database */ class RollbackActionTest extends MediaWikiIntegrationTestCase { diff --git a/tests/phpunit/includes/api/ApiBlockTest.php b/tests/phpunit/includes/api/ApiBlockTest.php index a702f29a256a..87fe35a794fa 100644 --- a/tests/phpunit/includes/api/ApiBlockTest.php +++ b/tests/phpunit/includes/api/ApiBlockTest.php @@ -2,12 +2,12 @@ namespace MediaWiki\Tests\Api; -use LogEntryBase; use MediaWiki\Block\DatabaseBlock; use MediaWiki\Block\DatabaseBlockStore; use MediaWiki\Block\Restriction\ActionRestriction; use MediaWiki\Block\Restriction\NamespaceRestriction; use MediaWiki\Block\Restriction\PageRestriction; +use MediaWiki\Logging\LogEntryBase; use MediaWiki\MainConfigNames; use MediaWiki\Permissions\Authority; use MediaWiki\Permissions\UltimateAuthority; diff --git a/tests/phpunit/includes/api/ApiUploadTest.php b/tests/phpunit/includes/api/ApiUploadTest.php index 6456e89be356..9821428fefe4 100644 --- a/tests/phpunit/includes/api/ApiUploadTest.php +++ b/tests/phpunit/includes/api/ApiUploadTest.php @@ -2,12 +2,12 @@ namespace MediaWiki\Tests\Api; -use LocalRepo; +use MediaWiki\FileRepo\LocalRepo; +use MediaWiki\FileRepo\RepoGroup; use MediaWiki\MainConfigNames; use MediaWiki\Permissions\Authority; use MediaWiki\Title\Title; use MediaWiki\WikiMap\WikiMap; -use RepoGroup; use Wikimedia\FileBackend\FSFileBackend; use Wikimedia\Mime\MimeAnalyzer; diff --git a/tests/phpunit/includes/api/query/ApiQueryImageInfoTest.php b/tests/phpunit/includes/api/query/ApiQueryImageInfoTest.php index 56d07f711b46..8a63da254292 100644 --- a/tests/phpunit/includes/api/query/ApiQueryImageInfoTest.php +++ b/tests/phpunit/includes/api/query/ApiQueryImageInfoTest.php @@ -2,8 +2,8 @@ namespace MediaWiki\Tests\Api\Query; -use File; use MediaWiki\Api\ApiQueryImageInfo; +use MediaWiki\FileRepo\File\File; use MediaWiki\Request\FauxRequest; use MediaWiki\Tests\Api\ApiTestCase; use MediaWiki\Tests\Unit\Permissions\MockAuthorityTrait; diff --git a/tests/phpunit/includes/auth/AuthManagerTest.php b/tests/phpunit/includes/auth/AuthManagerTest.php index aa0da5ddd1e0..534d3863917e 100644 --- a/tests/phpunit/includes/auth/AuthManagerTest.php +++ b/tests/phpunit/includes/auth/AuthManagerTest.php @@ -3,7 +3,6 @@ namespace MediaWiki\Tests\Auth; use Closure; -use DatabaseLogEntry; use DomainException; use DummySessionProvider; use DynamicPropertyTestHelper; @@ -44,6 +43,7 @@ use MediaWiki\HookContainer\StaticHookRegistry; use MediaWiki\Language\Language; use MediaWiki\Languages\LanguageConverterFactory; use MediaWiki\Logger\LoggerFactory; +use MediaWiki\Logging\DatabaseLogEntry; use MediaWiki\MainConfigNames; use MediaWiki\MediaWikiServices; use MediaWiki\Message\Message; diff --git a/tests/phpunit/includes/collation/CustomUppercaseCollationTest.php b/tests/phpunit/includes/collation/CustomUppercaseCollationTest.php index 5428153294f3..603b896bdad6 100644 --- a/tests/phpunit/includes/collation/CustomUppercaseCollationTest.php +++ b/tests/phpunit/includes/collation/CustomUppercaseCollationTest.php @@ -18,7 +18,8 @@ class CustomUppercaseCollationTest extends MediaWikiIntegrationTestCase { 'D', 'C', 'Cs', - 'B' + [ 'V', 'W' ], + 'B', ], 'en' // digital transformation language ); @@ -44,7 +45,9 @@ class CustomUppercaseCollationTest extends MediaWikiIntegrationTestCase { [ 'c', 'b', 'lowercase' ], [ 'x', 'z', 'lowercase original' ], [ 'Cz', 'Cs', 'digraphs' ], - [ 'C50D', 'C100', 'Numbers' ] + [ 'C50D', 'C100', 'Numbers' ], + [ 'V', 'b', 'Equal weight groups' ], + [ 'W', 'b', 'Equal weight groups' ], ]; } @@ -64,13 +67,17 @@ class CustomUppercaseCollationTest extends MediaWikiIntegrationTestCase { [ "\u{F3000}Foo", 'D' ], [ "\u{F3001}Foo", 'C' ], [ "\u{F3002}Foo", 'Cs' ], - [ "\u{F3003}Foo", 'B' ], - [ "\u{F3004}Foo", "\u{F3004}" ], + [ "\u{F3004}Foo", 'B' ], + [ "\u{F3005}Foo", "\u{F3005}" ], [ 'C', 'C' ], [ 'Cz', 'C' ], [ 'Cs', 'Cs' ], [ 'CS', 'Cs' ], [ 'cs', 'Cs' ], + [ 'V', 'V' ], + [ 'v', 'V' ], + [ 'w', 'V' ], + [ 'W', 'V' ], ]; } } diff --git a/tests/phpunit/includes/editpage/IntroMessageBuilderTest.php b/tests/phpunit/includes/editpage/IntroMessageBuilderTest.php index 8c8fd7b0cc91..d60bc37f8a8c 100644 --- a/tests/phpunit/includes/editpage/IntroMessageBuilderTest.php +++ b/tests/phpunit/includes/editpage/IntroMessageBuilderTest.php @@ -2,10 +2,11 @@ namespace MediaWiki\Tests\EditPage; -use File; -use FileRepo; use MediaWiki\Config\HashConfig; use MediaWiki\EditPage\IntroMessageBuilder; +use MediaWiki\FileRepo\File\File; +use MediaWiki\FileRepo\FileRepo; +use MediaWiki\FileRepo\RepoGroup; use MediaWiki\MainConfigNames; use MediaWiki\Page\ProperPageIdentity; use MediaWiki\Permissions\UltimateAuthority; @@ -16,7 +17,6 @@ use MediaWiki\User\UserFactory; use MediaWiki\User\UserIdentityValue; use MediaWikiIntegrationTestCase; use MockMessageLocalizer; -use RepoGroup; /** * @covers \MediaWiki\EditPage\IntroMessageBuilder diff --git a/tests/phpunit/includes/logging/BlockLogFormatterTest.php b/tests/phpunit/includes/logging/BlockLogFormatterTest.php index 889204d98668..b3276cf7a263 100644 --- a/tests/phpunit/includes/logging/BlockLogFormatterTest.php +++ b/tests/phpunit/includes/logging/BlockLogFormatterTest.php @@ -3,7 +3,7 @@ use MediaWiki\Title\TitleValue; /** - * @covers \BlockLogFormatter + * @covers \MediaWiki\Logging\BlockLogFormatter */ class BlockLogFormatterTest extends LogFormatterTestCase { diff --git a/tests/phpunit/includes/logging/ContentModelLogFormatterTest.php b/tests/phpunit/includes/logging/ContentModelLogFormatterTest.php index bb30f9ee337e..06cd7754fc35 100644 --- a/tests/phpunit/includes/logging/ContentModelLogFormatterTest.php +++ b/tests/phpunit/includes/logging/ContentModelLogFormatterTest.php @@ -1,7 +1,7 @@ <?php /** - * @covers \ContentModelLogFormatter + * @covers \MediaWiki\Logging\ContentModelLogFormatter */ class ContentModelLogFormatterTest extends LogFormatterTestCase { public static function provideContentModelLogDatabaseRows() { diff --git a/tests/phpunit/includes/logging/DatabaseLogEntryTest.php b/tests/phpunit/includes/logging/DatabaseLogEntryTest.php index 307de195b54e..35a8fae988cc 100644 --- a/tests/phpunit/includes/logging/DatabaseLogEntryTest.php +++ b/tests/phpunit/includes/logging/DatabaseLogEntryTest.php @@ -1,5 +1,6 @@ <?php +use MediaWiki\Logging\DatabaseLogEntry; use MediaWiki\User\ActorStore; use MediaWiki\User\UserIdentity; use MediaWiki\User\UserIdentityValue; @@ -11,8 +12,8 @@ use Wikimedia\Rdbms\IReadableDatabase; class DatabaseLogEntryTest extends MediaWikiIntegrationTestCase { /** - * @covers \DatabaseLogEntry::newFromId - * @covers \DatabaseLogEntry::getSelectQueryData + * @covers \MediaWiki\Logging\DatabaseLogEntry::newFromId + * @covers \MediaWiki\Logging\DatabaseLogEntry::getSelectQueryData * * @dataProvider provideNewFromId * @@ -33,7 +34,7 @@ class DatabaseLogEntryTest extends MediaWikiIntegrationTestCase { ->with( $selectFields['tables'], $selectFields['fields'], $selectFields['conds'], - 'DatabaseLogEntry::newFromId', + DatabaseLogEntry::class . '::newFromId', $selectFields['options'], $selectFields['join_conds'] ) @@ -140,7 +141,7 @@ class DatabaseLogEntryTest extends MediaWikiIntegrationTestCase { /** * @dataProvider provideGetPerformerIdentity - * @covers \DatabaseLogEntry::getPerformerIdentity + * @covers \MediaWiki\Logging\DatabaseLogEntry::getPerformerIdentity */ public function testGetPerformer( array $actorRowFields, UserIdentity $expected ) { $logEntry = DatabaseLogEntry::newFromRow( [ diff --git a/tests/phpunit/includes/logging/DeleteLogFormatterTest.php b/tests/phpunit/includes/logging/DeleteLogFormatterTest.php index 81bdf1379db3..d54dfd927c64 100644 --- a/tests/phpunit/includes/logging/DeleteLogFormatterTest.php +++ b/tests/phpunit/includes/logging/DeleteLogFormatterTest.php @@ -1,7 +1,7 @@ <?php /** - * @covers \DeleteLogFormatter + * @covers \MediaWiki\Logging\DeleteLogFormatter */ class DeleteLogFormatterTest extends LogFormatterTestCase { diff --git a/tests/phpunit/includes/logging/ImportLogFormatterTest.php b/tests/phpunit/includes/logging/ImportLogFormatterTest.php index cf3612d7c27a..b1c3e4d9c603 100644 --- a/tests/phpunit/includes/logging/ImportLogFormatterTest.php +++ b/tests/phpunit/includes/logging/ImportLogFormatterTest.php @@ -3,7 +3,7 @@ use MediaWiki\Tests\Unit\DummyServicesTrait; /** - * @covers \ImportLogFormatter + * @covers \MediaWiki\Logging\ImportLogFormatter */ class ImportLogFormatterTest extends LogFormatterTestCase { use DummyServicesTrait; diff --git a/tests/phpunit/includes/logging/LogFormatterTest.php b/tests/phpunit/includes/logging/LogFormatterTest.php index 268e69f69ac0..1b7aa6274d80 100644 --- a/tests/phpunit/includes/logging/LogFormatterTest.php +++ b/tests/phpunit/includes/logging/LogFormatterTest.php @@ -4,6 +4,8 @@ use MediaWiki\Api\ApiResult; use MediaWiki\Context\DerivativeContext; use MediaWiki\Context\RequestContext; use MediaWiki\Linker\Linker; +use MediaWiki\Logging\LogFormatter; +use MediaWiki\Logging\LogPage; use MediaWiki\MainConfigNames; use MediaWiki\Message\Message; use MediaWiki\Permissions\SimpleAuthority; @@ -93,7 +95,7 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::setShowUserToolLinks + * @covers \MediaWiki\Logging\LogFormatter::setShowUserToolLinks */ public function testNormalLogParams() { $entry = $this->newLogEntry( 'test', [] ); @@ -136,7 +138,7 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getActionText + * @covers \MediaWiki\Logging\LogFormatter::getActionText */ public function testLogParamsTypeRaw() { $params = [ '4:raw:raw' => Linker::link( $this->title, null, [], [] ) ]; @@ -152,7 +154,7 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getActionText + * @covers \MediaWiki\Logging\LogFormatter::getActionText */ public function testLogParamsTypeMsg() { $params = [ '4:msg:msg' => 'log-description-phpunit' ]; @@ -168,7 +170,7 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getActionText + * @covers \MediaWiki\Logging\LogFormatter::getActionText */ public function testLogParamsTypeMsgContent() { $params = [ '4:msg-content:msgContent' => 'log-description-phpunit' ]; @@ -184,7 +186,7 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getActionText + * @covers \MediaWiki\Logging\LogFormatter::getActionText */ public function testLogParamsTypeNumber() { global $wgLang; @@ -202,7 +204,7 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getActionText + * @covers \MediaWiki\Logging\LogFormatter::getActionText */ public function testLogParamsTypeUserLink() { $params = [ '4:user-link:userLink' => $this->user->getName() ]; @@ -221,7 +223,7 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getActionText + * @covers \MediaWiki\Logging\LogFormatter::getActionText */ public function testLogParamsTypeUserLink_empty() { $params = [ '4:user-link:userLink' => ':' ]; @@ -237,7 +239,7 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getActionText + * @covers \MediaWiki\Logging\LogFormatter::getActionText */ public function testLogParamsTypeTitleLink() { $params = [ '4:title-link:titleLink' => $this->title->getText() ]; @@ -253,7 +255,7 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getActionText + * @covers \MediaWiki\Logging\LogFormatter::getActionText */ public function testLogParamsTypePlain() { $params = [ '4:plain:plain' => 'Some plain text' ]; @@ -269,7 +271,7 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getPerformerElement + * @covers \MediaWiki\Logging\LogFormatter::getPerformerElement * @dataProvider provideLogElement */ public function testGetPerformerElement( $deletedFlag, $allowedAction ) { @@ -318,7 +320,7 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getComment + * @covers \MediaWiki\Logging\LogFormatter::getComment * @dataProvider provideLogElement */ public function testLogComment( $deletedFlag, $allowedAction ) { @@ -380,8 +382,8 @@ class LogFormatterTest extends MediaWikiLangTestCase { /** * @dataProvider provideApiParamFormatting - * @covers \LogFormatter::formatParametersForApi - * @covers \LogFormatter::formatParameterValueForApi + * @covers \MediaWiki\Logging\LogFormatter::formatParametersForApi + * @covers \MediaWiki\Logging\LogFormatter::formatParameterValueForApi */ public function testApiParamFormatting( $key, $value, $expected ) { $entry = $this->newLogEntry( 'param', [ $key => $value ] ); @@ -474,8 +476,8 @@ class LogFormatterTest extends MediaWikiLangTestCase { */ /** - * @covers \LogFormatter::getIRCActionComment - * @covers \LogFormatter::getIRCActionText + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionComment + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionText */ public function testIrcMsgForLogTypeBlock() { $sep = $this->context->msg( 'colon-separator' )->text(); @@ -525,8 +527,8 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getIRCActionComment - * @covers \LogFormatter::getIRCActionText + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionComment + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionText */ public function testIrcMsgForLogTypeDelete() { $sep = $this->context->msg( 'colon-separator' )->text(); @@ -549,8 +551,8 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getIRCActionComment - * @covers \LogFormatter::getIRCActionText + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionComment + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionText */ public function testIrcMsgForLogTypeNewusers() { $this->assertIRCComment( @@ -576,8 +578,8 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getIRCActionComment - * @covers \LogFormatter::getIRCActionText + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionComment + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionText */ public function testIrcMsgForLogTypeMove() { $move_params = [ @@ -606,8 +608,8 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getIRCActionComment - * @covers \LogFormatter::getIRCActionText + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionComment + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionText */ public function testIrcMsgForLogTypePatrol() { # patrol/patrol @@ -623,8 +625,8 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getIRCActionComment - * @covers \LogFormatter::getIRCActionText + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionComment + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionText */ public function testIrcMsgForLogTypeProtect() { $protectParams = [ @@ -673,8 +675,8 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getIRCActionComment - * @covers \LogFormatter::getIRCActionText + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionComment + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionText */ public function testIrcMsgForLogTypeUpload() { $sep = $this->context->msg( 'colon-separator' )->text(); @@ -697,8 +699,8 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getIRCActionComment - * @covers \LogFormatter::getIRCActionText + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionComment + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionText */ public function testIrcMsgForLogTypeMerge() { $sep = $this->context->msg( 'colon-separator' )->text(); @@ -717,8 +719,8 @@ class LogFormatterTest extends MediaWikiLangTestCase { } /** - * @covers \LogFormatter::getIRCActionComment - * @covers \LogFormatter::getIRCActionText + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionComment + * @covers \MediaWiki\Logging\LogFormatter::getIRCActionText */ public function testIrcMsgForLogTypeImport() { $sep = $this->context->msg( 'colon-separator' )->text(); diff --git a/tests/phpunit/includes/logging/LogFormatterTestCase.php b/tests/phpunit/includes/logging/LogFormatterTestCase.php index 83a16a02ebbf..d09c662e314e 100644 --- a/tests/phpunit/includes/logging/LogFormatterTestCase.php +++ b/tests/phpunit/includes/logging/LogFormatterTestCase.php @@ -6,6 +6,8 @@ use MediaWiki\Config\ServiceOptions; use MediaWiki\Context\RequestContext; use MediaWiki\Linker\LinkRenderer; use MediaWiki\Linker\LinkTarget; +use MediaWiki\Logging\LogEntryBase; +use MediaWiki\Logging\LogPage; use MediaWiki\Page\ExistingPageRecord; use MediaWiki\Page\PageStore; use MediaWiki\Tests\Unit\Permissions\MockAuthorityTrait; diff --git a/tests/phpunit/includes/logging/MergeLogFormatterTest.php b/tests/phpunit/includes/logging/MergeLogFormatterTest.php index 694efb7268ef..855aa0325cad 100644 --- a/tests/phpunit/includes/logging/MergeLogFormatterTest.php +++ b/tests/phpunit/includes/logging/MergeLogFormatterTest.php @@ -1,7 +1,7 @@ <?php /** - * @covers \MergeLogFormatter + * @covers \MediaWiki\Logging\MergeLogFormatter */ class MergeLogFormatterTest extends LogFormatterTestCase { diff --git a/tests/phpunit/includes/logging/MoveLogFormatterTest.php b/tests/phpunit/includes/logging/MoveLogFormatterTest.php index 719d0aa68825..913b3f3579c1 100644 --- a/tests/phpunit/includes/logging/MoveLogFormatterTest.php +++ b/tests/phpunit/includes/logging/MoveLogFormatterTest.php @@ -1,7 +1,7 @@ <?php /** - * @covers \MoveLogFormatter + * @covers \MediaWiki\Logging\MoveLogFormatter */ class MoveLogFormatterTest extends LogFormatterTestCase { diff --git a/tests/phpunit/includes/logging/NewUsersLogFormatterTest.php b/tests/phpunit/includes/logging/NewUsersLogFormatterTest.php index 60bd6f9244c2..5245b0517b67 100644 --- a/tests/phpunit/includes/logging/NewUsersLogFormatterTest.php +++ b/tests/phpunit/includes/logging/NewUsersLogFormatterTest.php @@ -1,7 +1,9 @@ <?php +use MediaWiki\Logging\NewUsersLogFormatter; + /** - * @covers \NewUsersLogFormatter + * @covers \MediaWiki\Logging\NewUsersLogFormatter * @group Database */ class NewUsersLogFormatterTest extends LogFormatterTestCase { diff --git a/tests/phpunit/includes/logging/PageLangLogFormatterTest.php b/tests/phpunit/includes/logging/PageLangLogFormatterTest.php index 79d99457db34..100e74f8870a 100644 --- a/tests/phpunit/includes/logging/PageLangLogFormatterTest.php +++ b/tests/phpunit/includes/logging/PageLangLogFormatterTest.php @@ -1,10 +1,11 @@ <?php +use MediaWiki\Logging\PageLangLogFormatter; use MediaWiki\MainConfigNames; use MediaWiki\MainConfigSchema; /** - * @covers \PageLangLogFormatter + * @covers \MediaWiki\Logging\PageLangLogFormatter */ class PageLangLogFormatterTest extends LogFormatterTestCase { diff --git a/tests/phpunit/includes/logging/PatrolLogFormatterTest.php b/tests/phpunit/includes/logging/PatrolLogFormatterTest.php index 5c0137b2abaa..98629bee75bf 100644 --- a/tests/phpunit/includes/logging/PatrolLogFormatterTest.php +++ b/tests/phpunit/includes/logging/PatrolLogFormatterTest.php @@ -1,7 +1,7 @@ <?php /** - * @covers \PatrolLogFormatter + * @covers \MediaWiki\Logging\PatrolLogFormatter */ class PatrolLogFormatterTest extends LogFormatterTestCase { diff --git a/tests/phpunit/includes/logging/ProtectLogFormatterTest.php b/tests/phpunit/includes/logging/ProtectLogFormatterTest.php index ecdb77583f7b..64bd55461af9 100644 --- a/tests/phpunit/includes/logging/ProtectLogFormatterTest.php +++ b/tests/phpunit/includes/logging/ProtectLogFormatterTest.php @@ -11,7 +11,7 @@ use Wikimedia\Rdbms\IDatabase; use Wikimedia\Rdbms\LBFactory; /** - * @covers \ProtectLogFormatter + * @covers \MediaWiki\Logging\ProtectLogFormatter */ class ProtectLogFormatterTest extends LogFormatterTestCase { @@ -466,7 +466,7 @@ class ProtectLogFormatterTest extends LogFormatterTestCase { * @param string[] $permissions * @param bool $shouldMatch * @dataProvider provideGetActionLinks - * @covers \ProtectLogFormatter::getActionLinks + * @covers \MediaWiki\Logging\ProtectLogFormatter::getActionLinks */ public function testGetActionLinks( array $permissions, $shouldMatch ) { RequestContext::resetMain(); diff --git a/tests/phpunit/includes/logging/RightsLogFormatterTest.php b/tests/phpunit/includes/logging/RightsLogFormatterTest.php index 5fe31e6dc5eb..83a007236b6a 100644 --- a/tests/phpunit/includes/logging/RightsLogFormatterTest.php +++ b/tests/phpunit/includes/logging/RightsLogFormatterTest.php @@ -4,7 +4,7 @@ use Wikimedia\Rdbms\IDatabase; use Wikimedia\Rdbms\LBFactory; /** - * @covers \RightsLogFormatter + * @covers \MediaWiki\Logging\RightsLogFormatter */ class RightsLogFormatterTest extends LogFormatterTestCase { diff --git a/tests/phpunit/includes/logging/UploadLogFormatterTest.php b/tests/phpunit/includes/logging/UploadLogFormatterTest.php index b5199588af32..0e2722a90432 100644 --- a/tests/phpunit/includes/logging/UploadLogFormatterTest.php +++ b/tests/phpunit/includes/logging/UploadLogFormatterTest.php @@ -1,7 +1,7 @@ <?php /** - * @covers \UploadLogFormatter + * @covers \MediaWiki\Logging\UploadLogFormatter */ class UploadLogFormatterTest extends LogFormatterTestCase { diff --git a/tests/phpunit/includes/media/BmpHandlerTest.php b/tests/phpunit/includes/media/BmpHandlerTest.php index 86ba6ad3bcee..64448141550d 100644 --- a/tests/phpunit/includes/media/BmpHandlerTest.php +++ b/tests/phpunit/includes/media/BmpHandlerTest.php @@ -22,8 +22,8 @@ namespace MediaWiki\Tests\Media; use BmpHandler; -use File; use MediaHandlerState; +use MediaWiki\FileRepo\File\File; use MediaWikiMediaTestCase; /** diff --git a/tests/phpunit/includes/pager/HistoryPagerTest.php b/tests/phpunit/includes/pager/HistoryPagerTest.php index 709e1c5029db..94455968de01 100644 --- a/tests/phpunit/includes/pager/HistoryPagerTest.php +++ b/tests/phpunit/includes/pager/HistoryPagerTest.php @@ -1,5 +1,6 @@ <?php +use MediaWiki\Actions\HistoryAction; use MediaWiki\Context\RequestContext; use MediaWiki\Output\OutputPage; use MediaWiki\Pager\HistoryPager; diff --git a/tests/phpunit/includes/recentchanges/ChangesListTest.php b/tests/phpunit/includes/recentchanges/ChangesListTest.php index 1d077e7be3e0..3df3cc6af587 100644 --- a/tests/phpunit/includes/recentchanges/ChangesListTest.php +++ b/tests/phpunit/includes/recentchanges/ChangesListTest.php @@ -1,5 +1,6 @@ <?php +use MediaWiki\Logging\DatabaseLogEntry; use MediaWiki\User\UserIdentity; use Wikimedia\Parsoid\Utils\DOMCompat; use Wikimedia\Parsoid\Utils\DOMUtils; diff --git a/tests/phpunit/includes/recentchanges/rcfeed/RCFeedIntegrationTest.php b/tests/phpunit/includes/recentchanges/rcfeed/RCFeedIntegrationTest.php index b923508f72e2..c96e198ce1e2 100644 --- a/tests/phpunit/includes/recentchanges/rcfeed/RCFeedIntegrationTest.php +++ b/tests/phpunit/includes/recentchanges/rcfeed/RCFeedIntegrationTest.php @@ -1,5 +1,6 @@ <?php +use MediaWiki\Logging\LogEntryBase; use MediaWiki\MainConfigNames; use MediaWiki\RCFeed\FormattedRCFeed; use MediaWiki\RCFeed\JSONRCFeedFormatter; diff --git a/tests/phpunit/includes/specials/SpecialBlockTest.php b/tests/phpunit/includes/specials/SpecialBlockTest.php index 2ad9c89f4926..d29c855091ab 100644 --- a/tests/phpunit/includes/specials/SpecialBlockTest.php +++ b/tests/phpunit/includes/specials/SpecialBlockTest.php @@ -945,6 +945,33 @@ class SpecialBlockTest extends SpecialPageTestBase { ]; } + /** + * @covers ::validateTarget + * @covers ::getTargetInternal + */ + public function testValidateTargetFromId(): void { + $badActor = $this->getTestUser()->getUser(); + $block = $this->blockStore->insertBlockWithParams( [ + 'targetUser' => $badActor, + 'by' => $this->getTestSysop()->getUser(), + 'expiry' => 'infinity', + 'sitewide' => 1, + ] ); + + $wrappedPage = TestingAccessWrapper::newFromObject( $this->newSpecialPage() ); + $target = $wrappedPage->getTargetInternal( '', new FauxRequest( [ + 'id' => $block->getId(), + ] ) ); + $this->assertSame( $badActor->getName(), $target->toString() ); + + // Invalid ID. + $fauxRequest = new FauxRequest( [ 'id' => 999999 ] ); + $target = $wrappedPage->getTargetInternal( null, $fauxRequest ); + $this->assertNull( $target ); + $wrappedPage->validateTarget( $fauxRequest ); + $this->assertSame( 'block-invalid-id', $wrappedPage->preErrors[ 0 ]->getKey() ); + } + protected function insertBlock() { $badActor = $this->getTestUser()->getUser(); $sysop = $this->getTestSysop()->getUser(); diff --git a/tests/phpunit/includes/user/UserGroupManagerTest.php b/tests/phpunit/includes/user/UserGroupManagerTest.php index bf2e350d80b5..61ca2d7479d6 100644 --- a/tests/phpunit/includes/user/UserGroupManagerTest.php +++ b/tests/phpunit/includes/user/UserGroupManagerTest.php @@ -21,12 +21,12 @@ namespace MediaWiki\Tests\User; use InvalidArgumentException; -use LogEntryBase; use LogicException; use MediaWiki\Block\DatabaseBlock; use MediaWiki\Config\ServiceOptions; use MediaWiki\Config\SiteConfiguration; use MediaWiki\Context\RequestContext; +use MediaWiki\Logging\LogEntryBase; use MediaWiki\MainConfigNames; use MediaWiki\Permissions\SimpleAuthority; use MediaWiki\Request\WebRequest; |