UserGroupsChanged hook should specify the performer of the change
authorHuji Lee <huji.huji@gmail.com>
Sun, 12 Nov 2017 01:44:00 +0000 (20:44 -0500)
committerHuji <huji.huji@gmail.com>
Thu, 16 Nov 2017 02:56:32 +0000 (02:56 +0000)
Otherwise, there will be a unit-testing error when Echo is enabled

Bug: T180292
Change-Id: Ibc185c82ad2a03e06e5727a633e6ab6bccce3345

tests/phpunit/includes/auth/AuthPluginPrimaryAuthenticationProviderTest.php

index 6970313..82b0f82 100644 (file)
@@ -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() {