wiki.techinc.nl/tests/phpunit/includes/user/UserGroupMembershipTest.php

152 lines
4.9 KiB
PHP
Raw Normal View History

User group memberships that expire This patch adds an ug_expiry column to the user_groups table, a timestamp giving a date when the user group expires. A new UserGroupMembership class, based on the Block class, manages entries in this table. When the expiry date passes, the row in user_groups is ignored, and will eventually be purged from the DB when UserGroupMembership::insert is next called. Old, expired user group memberships are not kept; instead, the log entries are available to find the history of these memberships, similar to the way it has always worked for blocks and protections. Anyone getting user group info through the User object will get correct information. However, code that reads the user_groups table directly will now need to skip over rows with ug_expiry < wfTimestampNow(). See UsersPager for an example of how to do this. NULL is used to represent infinite (no) expiry, rather than a string 'infinity' or similar (except in the API). This allows existing user group assignments and log entries, which are all infinite in duration, to be treated the same as new, infinite-length memberships, without special casing everything. The whole thing is behind the temporary feature flag $wgDisableUserGroupExpiry, in accordance with the WMF schema change policy. The opportunity has been taken to refactor some static user-group-related functions out of User into UserGroupMembership, and also to add a primary key (ug_user, ug_group) to the user_groups table. There are a few breaking changes: - UserRightsProxy-like objects are now required to have a getGroupMemberships() function. - $user->mGroups (on a User object) is no longer present. - Some protected functions in UsersPager are altered or removed. - The UsersPagerDoBatchLookups hook (unused in any Wikimedia Git-hosted extension) has a change of parameter. Bug: T12493 Depends-On: Ia9616e1e35184fed9058d2d39afbe1038f56d7fa Depends-On: I86eb1d5619347ce54a5f33a591417742ebe5d6f8 Change-Id: I93c955dc7a970f78e32aa503c01c67da30971d1a
2017-01-12 06:07:56 +00:00
<?php
/**
* @group Database
*/
class UserGroupMembershipTest extends MediaWikiTestCase {
/**
* @var User Belongs to no groups
*/
protected $userNoGroups;
/**
* @var User Belongs to the 'unittesters' group indefinitely, and the
* 'testwriters' group with expiry
*/
protected $userTester;
/**
* @var string The timestamp, in TS_MW format, of the expiry of $userTester's
* membership in the 'testwriters' group
*/
protected $expiryTime;
protected function setUp() {
parent::setUp();
$this->setMwGlobals( [
'wgGroupPermissions' => [
'unittesters' => [
'runtest' => true,
],
'testwriters' => [
'writetest' => true,
]
]
] );
$this->userNoGroups = new User;
$this->userNoGroups->setName( 'NoGroups' );
$this->userNoGroups->addToDatabase();
$this->userTester = new User;
$this->userTester->setName( 'Tester' );
$this->userTester->addToDatabase();
$this->userTester->addGroup( 'unittesters' );
$this->expiryTime = wfTimestamp( TS_MW, time() + 100500 );
$this->userTester->addGroup( 'testwriters', $this->expiryTime );
}
/**
* @covers UserGroupMembership::insert
* @covers UserGroupMembership::delete
*/
public function testAddAndRemoveGroups() {
$user = new User;
$user->addToDatabase();
// basic tests
$ugm = new UserGroupMembership( $user->getId(), 'unittesters' );
$this->assertTrue( $ugm->insert() );
$user->clearInstanceCache();
$this->assertContains( 'unittesters', $user->getGroups() );
$this->assertArrayHasKey( 'unittesters', $user->getGroupMemberships() );
$this->assertTrue( $user->isAllowed( 'runtest' ) );
// try updating without allowUpdate. Should fail
$ugm = new UserGroupMembership( $user->getId(), 'unittesters', $this->expiryTime );
$this->assertFalse( $ugm->insert() );
// now try updating with allowUpdate
$this->assertTrue( $ugm->insert( 2 ) );
$user->clearInstanceCache();
$this->assertContains( 'unittesters', $user->getGroups() );
$this->assertArrayHasKey( 'unittesters', $user->getGroupMemberships() );
$this->assertTrue( $user->isAllowed( 'runtest' ) );
// try removing the group
$ugm->delete();
$user->clearInstanceCache();
$this->assertThat( $user->getGroups(),
$this->logicalNot( $this->contains( 'unittesters' ) ) );
$this->assertThat( $user->getGroupMemberships(),
$this->logicalNot( $this->arrayHasKey( 'unittesters' ) ) );
$this->assertFalse( $user->isAllowed( 'runtest' ) );
// check that the user group is now in user_former_groups
$this->assertContains( 'unittesters', $user->getFormerGroups() );
}
private function addUserTesterToExpiredGroup() {
// put $userTester in a group with expiry in the past
$ugm = new UserGroupMembership( $this->userTester->getId(), 'sysop', '20010102030405' );
$ugm->insert();
}
/**
* @covers UserGroupMembership::getMembershipsForUser
*/
public function testGetMembershipsForUser() {
$this->addUserTesterToExpiredGroup();
// check that the user in no groups has no group memberships
$ugms = UserGroupMembership::getMembershipsForUser( $this->userNoGroups->getId() );
$this->assertEmpty( $ugms );
// check that the user in 2 groups has 2 group memberships
$testerUserId = $this->userTester->getId();
$ugms = UserGroupMembership::getMembershipsForUser( $testerUserId );
$this->assertCount( 2, $ugms );
// check that the required group memberships are present on $userTester,
// with the correct user IDs and expiries
$expectedGroups = [ 'unittesters', 'testwriters' ];
foreach ( $expectedGroups as $group ) {
$this->assertArrayHasKey( $group, $ugms );
$this->assertEquals( $ugms[$group]->getUserId(), $testerUserId );
$this->assertEquals( $ugms[$group]->getGroup(), $group );
if ( $group === 'unittesters' ) {
$this->assertNull( $ugms[$group]->getExpiry() );
} elseif ( $group === 'testwriters' ) {
$this->assertEquals( $ugms[$group]->getExpiry(), $this->expiryTime );
}
}
}
/**
* @covers UserGroupMembership::getMembership
*/
public function testGetMembership() {
$this->addUserTesterToExpiredGroup();
// groups that the user doesn't belong to shouldn't be returned
$ugm = UserGroupMembership::getMembership( $this->userNoGroups->getId(), 'sysop' );
$this->assertFalse( $ugm );
// implicit groups shouldn't be returned
$ugm = UserGroupMembership::getMembership( $this->userNoGroups->getId(), 'user' );
$this->assertFalse( $ugm );
// expired groups shouldn't be returned
$ugm = UserGroupMembership::getMembership( $this->userTester->getId(), 'sysop' );
$this->assertFalse( $ugm );
// groups that the user does belong to should be returned with correct properties
$ugm = UserGroupMembership::getMembership( $this->userTester->getId(), 'unittesters' );
$this->assertInstanceOf( UserGroupMembership::class, $ugm );
$this->assertEquals( $ugm->getUserId(), $this->userTester->getId() );
$this->assertEquals( $ugm->getGroup(), 'unittesters' );
$this->assertNull( $ugm->getExpiry() );
}
}