From: rxy Date: Fri, 31 May 2019 20:04:09 +0000 (+0000) Subject: Fix testsuites for LogFormatter X-Git-Tag: 1.34.0-rc.0~1310 X-Git-Url: https://git.cyclocoop.org/%7B%24www_url%7Dadmin/compta/banques/%22http:/wikimediafoundation.org/fundraising/%7B%7B%20url_for%28%27user_settings%27%2C%20userid=session.userid%29%20%7D%7D?a=commitdiff_plain;h=d0c2da82616a280f51feafa015dd67608c13e29b;p=lhc%2Fweb%2Fwiklou.git Fix testsuites for LogFormatter Change-Id: I064f563cb54a50c3292b1f9e24b0bb467f69af9b --- diff --git a/tests/phpunit/includes/logging/BlockLogFormatterTest.php b/tests/phpunit/includes/logging/BlockLogFormatterTest.php index bc0ca2ad85..1d653281e6 100644 --- a/tests/phpunit/includes/logging/BlockLogFormatterTest.php +++ b/tests/phpunit/includes/logging/BlockLogFormatterTest.php @@ -307,6 +307,81 @@ class BlockLogFormatterTest extends LogFormatterTestCase { * @dataProvider provideSuppressBlockLogDatabaseRows */ public function testSuppressBlockLogDatabaseRows( $row, $extra ) { + $this->setMwGlobals( + 'wgGroupPermissions', + [ + 'oversight' => [ + 'viewsuppressed' => true, + 'suppressionlog' => true, + ], + ] + ); + $this->doTestLogFormatter( $row, $extra, [ 'oversight' ] ); + } + + /** + * Provide different rows from the logging table to test + * for backward compatibility. + * Do not change the existing data, just add a new database row + */ + public static function provideSuppressBlockLogDatabaseRowsNonPrivileged() { + return [ + // Current log format + [ + [ + 'type' => 'suppress', + 'action' => 'block', + 'comment' => 'Block comment', + 'user' => 0, + 'user_text' => 'Sysop', + 'namespace' => NS_USER, + 'title' => 'Logtestuser', + 'params' => [ + '5::duration' => 'infinite', + '6::flags' => 'anononly', + ], + ], + [ + 'text' => '(username removed) (log details removed)', + 'api' => [ + 'duration' => 'infinite', + 'flags' => [ 'anononly' ], + ], + ], + ], + + // legacy log + [ + [ + 'type' => 'suppress', + 'action' => 'block', + 'comment' => 'Block comment', + 'user' => 0, + 'user_text' => 'Sysop', + 'namespace' => NS_USER, + 'title' => 'Logtestuser', + 'params' => [ + 'infinite', + 'anononly', + ], + ], + [ + 'legacy' => true, + 'text' => '(username removed) (log details removed)', + 'api' => [ + 'duration' => 'infinite', + 'flags' => [ 'anononly' ], + ], + ], + ], + ]; + } + + /** + * @dataProvider provideSuppressBlockLogDatabaseRowsNonPrivileged + */ + public function testSuppressBlockLogDatabaseRowsNonPrivileged( $row, $extra ) { + $this->user = $this->getTestUser()->getUser(); $this->doTestLogFormatter( $row, $extra ); } @@ -374,6 +449,81 @@ class BlockLogFormatterTest extends LogFormatterTestCase { * @dataProvider provideSuppressReblockLogDatabaseRows */ public function testSuppressReblockLogDatabaseRows( $row, $extra ) { + $this->setMwGlobals( + 'wgGroupPermissions', + [ + 'oversight' => [ + 'viewsuppressed' => true, + 'suppressionlog' => true, + ], + ] + ); + $this->doTestLogFormatter( $row, $extra, [ 'oversight' ] ); + } + + /** + * Provide different rows from the logging table to test + * for backward compatibility. + * Do not change the existing data, just add a new database row + */ + public static function provideSuppressReblockLogDatabaseRowsNonPrivileged() { + return [ + // Current log format + [ + [ + 'type' => 'suppress', + 'action' => 'reblock', + 'comment' => 'Block comment', + 'user' => 0, + 'user_text' => 'Sysop', + 'namespace' => NS_USER, + 'title' => 'Logtestuser', + 'params' => [ + '5::duration' => 'infinite', + '6::flags' => 'anononly', + ], + ], + [ + 'text' => '(username removed) (log details removed)', + 'api' => [ + 'duration' => 'infinite', + 'flags' => [ 'anononly' ], + ], + ], + ], + + // Legacy format + [ + [ + 'type' => 'suppress', + 'action' => 'reblock', + 'comment' => 'Block comment', + 'user' => 0, + 'user_text' => 'Sysop', + 'namespace' => NS_USER, + 'title' => 'Logtestuser', + 'params' => [ + 'infinite', + 'anononly', + ], + ], + [ + 'legacy' => true, + 'text' => '(username removed) (log details removed)', + 'api' => [ + 'duration' => 'infinite', + 'flags' => [ 'anononly' ], + ], + ], + ], + ]; + } + + /** + * @dataProvider provideSuppressReblockLogDatabaseRowsNonPrivileged + */ + public function testSuppressReblockLogDatabaseRowsNonPrivileged( $row, $extra ) { + $this->user = $this->getTestUser()->getUser(); $this->doTestLogFormatter( $row, $extra ); } diff --git a/tests/phpunit/includes/logging/DeleteLogFormatterTest.php b/tests/phpunit/includes/logging/DeleteLogFormatterTest.php index 6648c31c25..f1d58fdf09 100644 --- a/tests/phpunit/includes/logging/DeleteLogFormatterTest.php +++ b/tests/phpunit/includes/logging/DeleteLogFormatterTest.php @@ -409,6 +409,109 @@ class DeleteLogFormatterTest extends LogFormatterTestCase { * @dataProvider provideSuppressRevisionLogDatabaseRows */ public function testSuppressRevisionLogDatabaseRows( $row, $extra ) { + $this->setMwGlobals( + 'wgGroupPermissions', + [ + 'oversight' => [ + 'viewsuppressed' => true, + 'suppressionlog' => true, + ], + ] + ); + $this->doTestLogFormatter( $row, $extra, [ 'oversight' ] ); + } + + /** + * Provide different rows from the logging table to test + * for backward compatibility. + * Do not change the existing data, just add a new database row + */ + public static function provideSuppressRevisionLogDatabaseRowsNonPrivileged() { + return [ + // Current format + [ + [ + 'type' => 'suppress', + 'action' => 'revision', + 'comment' => 'Suppress comment', + 'namespace' => NS_MAIN, + 'title' => 'Page', + 'params' => [ + '4::type' => 'archive', + '5::ids' => [ '1', '3', '4' ], + '6::ofield' => '1', + '7::nfield' => '10', + ], + ], + [ + 'text' => '(username removed) (log details removed)', + 'api' => [ + 'type' => 'archive', + 'ids' => [ '1', '3', '4' ], + 'old' => [ + 'bitmask' => 1, + 'content' => true, + 'comment' => false, + 'user' => false, + 'restricted' => false, + ], + 'new' => [ + 'bitmask' => 10, + 'content' => false, + 'comment' => true, + 'user' => false, + 'restricted' => true, + ], + ], + ], + ], + + // Legacy format + [ + [ + 'type' => 'suppress', + 'action' => 'revision', + 'comment' => 'Suppress comment', + 'namespace' => NS_MAIN, + 'title' => 'Page', + 'params' => [ + 'archive', + '1,3,4', + 'ofield=1', + 'nfield=10', + ], + ], + [ + 'legacy' => true, + 'text' => '(username removed) (log details removed)', + 'api' => [ + 'type' => 'archive', + 'ids' => [ '1', '3', '4' ], + 'old' => [ + 'bitmask' => 1, + 'content' => true, + 'comment' => false, + 'user' => false, + 'restricted' => false, + ], + 'new' => [ + 'bitmask' => 10, + 'content' => false, + 'comment' => true, + 'user' => false, + 'restricted' => true, + ], + ], + ], + ], + ]; + } + + /** + * @dataProvider provideSuppressRevisionLogDatabaseRowsNonPrivileged + */ + public function testSuppressRevisionLogDatabaseRowsNonPrivileged( $row, $extra ) { + $this->user = $this->getTestUser()->getUser(); $this->doTestLogFormatter( $row, $extra ); } @@ -523,6 +626,107 @@ class DeleteLogFormatterTest extends LogFormatterTestCase { * @dataProvider provideSuppressEventLogDatabaseRows */ public function testSuppressEventLogDatabaseRows( $row, $extra ) { + $this->setMwGlobals( + 'wgGroupPermissions', + [ + 'oversight' => [ + 'viewsuppressed' => true, + 'suppressionlog' => true, + ], + ] + ); + $this->doTestLogFormatter( $row, $extra, [ 'oversight' ] ); + } + + /** + * Provide different rows from the logging table to test + * for backward compatibility. + * Do not change the existing data, just add a new database row + */ + public static function provideSuppressEventLogDatabaseRowsNonPrivileged() { + return [ + // Current format + [ + [ + 'type' => 'suppress', + 'action' => 'event', + 'comment' => 'Suppress comment', + 'namespace' => NS_MAIN, + 'title' => 'Page', + 'params' => [ + '4::ids' => [ '1', '3', '4' ], + '5::ofield' => '1', + '6::nfield' => '10', + ], + ], + [ + 'text' => '(username removed) (log details removed)', + 'api' => [ + 'type' => 'logging', + 'ids' => [ '1', '3', '4' ], + 'old' => [ + 'bitmask' => 1, + 'content' => true, + 'comment' => false, + 'user' => false, + 'restricted' => false, + ], + 'new' => [ + 'bitmask' => 10, + 'content' => false, + 'comment' => true, + 'user' => false, + 'restricted' => true, + ], + ], + ], + ], + + // Legacy format + [ + [ + 'type' => 'suppress', + 'action' => 'event', + 'comment' => 'Suppress comment', + 'namespace' => NS_MAIN, + 'title' => 'Page', + 'params' => [ + '1,3,4', + 'ofield=1', + 'nfield=10', + ], + ], + [ + 'legacy' => true, + 'text' => '(username removed) (log details removed)', + 'api' => [ + 'type' => 'logging', + 'ids' => [ '1', '3', '4' ], + 'old' => [ + 'bitmask' => 1, + 'content' => true, + 'comment' => false, + 'user' => false, + 'restricted' => false, + ], + 'new' => [ + 'bitmask' => 10, + 'content' => false, + 'comment' => true, + 'user' => false, + 'restricted' => true, + ], + ], + ], + ], + ]; + } + + /** + * @dataProvider provideSuppressEventLogDatabaseRowsNonPrivileged + */ + public function testSuppressEventLogDatabaseRowsNonPrivileged( $row, $extra ) { + $this->user = $this->getTestUser()->getUser(); $this->doTestLogFormatter( $row, $extra ); } @@ -572,6 +776,65 @@ class DeleteLogFormatterTest extends LogFormatterTestCase { * @dataProvider provideSuppressDeleteLogDatabaseRows */ public function testSuppressDeleteLogDatabaseRows( $row, $extra ) { + $this->setMwGlobals( + 'wgGroupPermissions', + [ + 'oversight' => [ + 'viewsuppressed' => true, + 'suppressionlog' => true, + ], + ] + ); + $this->doTestLogFormatter( $row, $extra, [ 'oversight' ] ); + } + + /** + * Provide different rows from the logging table to test + * for backward compatibility. + * Do not change the existing data, just add a new database row + */ + public static function provideSuppressDeleteLogDatabaseRowsNonPrivileged() { + return [ + // Current format + [ + [ + 'type' => 'suppress', + 'action' => 'delete', + 'comment' => 'delete comment', + 'namespace' => NS_MAIN, + 'title' => 'Page', + 'params' => [], + ], + [ + 'text' => '(username removed) (log details removed)', + 'api' => [], + ], + ], + + // Legacy format + [ + [ + 'type' => 'suppress', + 'action' => 'delete', + 'comment' => 'delete comment', + 'namespace' => NS_MAIN, + 'title' => 'Page', + 'params' => [], + ], + [ + 'legacy' => true, + 'text' => '(username removed) (log details removed)', + 'api' => [], + ], + ], + ]; + } + + /** + * @dataProvider provideSuppressDeleteLogDatabaseRowsNonPrivileged + */ + public function testSuppressDeleteLogDatabaseRowsNonPrivileged( $row, $extra ) { + $this->user = $this->getTestUser()->getUser(); $this->doTestLogFormatter( $row, $extra ); } } diff --git a/tests/phpunit/includes/logging/LogFormatterTestCase.php b/tests/phpunit/includes/logging/LogFormatterTestCase.php index 883af71240..71ab14783a 100644 --- a/tests/phpunit/includes/logging/LogFormatterTestCase.php +++ b/tests/phpunit/includes/logging/LogFormatterTestCase.php @@ -5,11 +5,15 @@ */ abstract class LogFormatterTestCase extends MediaWikiLangTestCase { - public function doTestLogFormatter( $row, $extra ) { + public function doTestLogFormatter( $row, $extra, $userGroups = [] ) { RequestContext::resetMain(); $row = $this->expandDatabaseRow( $row, $this->isLegacy( $extra ) ); + $context = new RequestContext(); + $context->setUser( $this->getTestUser( $userGroups )->getUser() ); + $formatter = LogFormatter::newFromRow( $row ); + $formatter->setContext( $context ); $this->assertEquals( $extra['text'],