From 2a06bc44ca1059395f86fd1f6c5abab80c9b303d Mon Sep 17 00:00:00 2001 From: Huji Lee Date: Sat, 11 Nov 2017 20:44:00 -0500 Subject: [PATCH] UserGroupsChanged hook should specify the performer of the change Otherwise, there will be a unit-testing error when Echo is enabled Bug: T180292 Change-Id: Ibc185c82ad2a03e06e5727a633e6ab6bccce3345 --- .../auth/AuthPluginPrimaryAuthenticationProviderTest.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/phpunit/includes/auth/AuthPluginPrimaryAuthenticationProviderTest.php b/tests/phpunit/includes/auth/AuthPluginPrimaryAuthenticationProviderTest.php index 6970313498..82b0f8286a 100644 --- a/tests/phpunit/includes/auth/AuthPluginPrimaryAuthenticationProviderTest.php +++ b/tests/phpunit/includes/auth/AuthPluginPrimaryAuthenticationProviderTest.php @@ -73,7 +73,7 @@ class AuthPluginPrimaryAuthenticationProviderTest extends \MediaWikiTestCase { ); $provider = new AuthPluginPrimaryAuthenticationProvider( $plugin ); - \Hooks::run( 'UserGroupsChanged', [ $user, [ 'added' ], [ 'removed' ] ] ); + \Hooks::run( 'UserGroupsChanged', [ $user, [ 'added' ], [ 'removed' ], false, false, [], [] ] ); } public function testOnUserLoggedIn() { -- 2.20.1