Fix UserRightsTest unit tests

This commit is contained in:
Brion Vibber 2009-12-15 14:49:53 -08:00
parent 0158f4f73d
commit a998bda4a5
2 changed files with 19 additions and 6 deletions

View File

@ -329,7 +329,7 @@ class User extends Memcached_DataObject
$profile->query('COMMIT');
if ($email && !$user->email) {
if (!empty($email) && !$user->email) {
mail_confirm_address($user, $confirm->code, $profile->nickname, $email);
}

View File

@ -16,15 +16,27 @@ class UserRightsTest extends PHPUnit_Framework_TestCase
function setUp()
{
$user = User::staticGet('nickname', 'userrightstestuser');
if ($user) {
// Leftover from a broken test run?
$profile = $user->getProfile();
$user->delete();
$profile->delete();
}
$this->user = User::register(array('nickname' => 'userrightstestuser'));
if (!$this->user) {
throw new Exception("Couldn't register userrightstestuser");
}
}
function tearDown()
{
if ($this->user) {
$profile = $this->user->getProfile();
$this->user->delete();
$profile->delete();
}
}
function testInvalidRole()
{
@ -33,7 +45,8 @@ class UserRightsTest extends PHPUnit_Framework_TestCase
function standardRoles()
{
return array('admin', 'moderator');
return array(array('admin'),
array('moderator'));
}
/**
@ -54,6 +67,6 @@ class UserRightsTest extends PHPUnit_Framework_TestCase
function testGrantedRole($role)
{
$this->user->grantRole($role);
$this->assertFalse($this->user->hasRole($role));
$this->assertTrue($this->user->hasRole($role));
}
}