jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/379296 )

Change subject: Set wgUseRCPatrol in ChangesListSpecialPageTest
......................................................................


Set wgUseRCPatrol in ChangesListSpecialPageTest

wgUseRCPatrol = false makes the tests fail

Change-Id: I18965d33171137ff182800512c8c0e8e63de124e
---
M tests/phpunit/includes/specialpage/AbstractChangesListSpecialPageTestCase.php
M tests/phpunit/includes/specialpage/ChangesListSpecialPageTest.php
2 files changed, 6 insertions(+), 1 deletion(-)

Approvals:
  Krinkle: Looks good to me, approved
  jenkins-bot: Verified



diff --git 
a/tests/phpunit/includes/specialpage/AbstractChangesListSpecialPageTestCase.php 
b/tests/phpunit/includes/specialpage/AbstractChangesListSpecialPageTestCase.php
index b101857..04b12e6 100644
--- 
a/tests/phpunit/includes/specialpage/AbstractChangesListSpecialPageTestCase.php
+++ 
b/tests/phpunit/includes/specialpage/AbstractChangesListSpecialPageTestCase.php
@@ -19,7 +19,10 @@
                global $wgGroupPermissions;
 
                parent::setUp();
-               $this->setMwGlobals( 'wgRCWatchCategoryMembership', true );
+               $this->setMwGlobals( [
+                       'wgRCWatchCategoryMembership' => true,
+                       'wgUseRCPatrol' => true,
+               ] );
 
                if ( isset( $wgGroupPermissions['patrollers'] ) ) {
                        $this->oldPatrollersGroup = 
$wgGroupPermissions['patrollers'];
diff --git a/tests/phpunit/includes/specialpage/ChangesListSpecialPageTest.php 
b/tests/phpunit/includes/specialpage/ChangesListSpecialPageTest.php
index f494785..fd7e0f1 100644
--- a/tests/phpunit/includes/specialpage/ChangesListSpecialPageTest.php
+++ b/tests/phpunit/includes/specialpage/ChangesListSpecialPageTest.php
@@ -293,6 +293,7 @@
        }
 
        public function testRcHidepatrolledDisabledFilter() {
+               $this->setMwGlobals( 'wgUseRCPatrol', false );
                $user = $this->getTestUser()->getUser();
                $this->assertConditions(
                        [ # expected
@@ -306,6 +307,7 @@
        }
 
        public function testRcHideunpatrolledDisabledFilter() {
+               $this->setMwGlobals( 'wgUseRCPatrol', false );
                $user = $this->getTestUser()->getUser();
                $this->assertConditions(
                        [ # expected

-- 
To view, visit https://gerrit.wikimedia.org/r/379296
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I18965d33171137ff182800512c8c0e8e63de124e
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Umherirrender <[email protected]>
Gerrit-Reviewer: Krinkle <[email protected]>
Gerrit-Reviewer: Mattflaschen <[email protected]>
Gerrit-Reviewer: Sbisson <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to