diff options
author | Frédéric Guillot <fred@kanboard.net> | 2014-11-09 12:47:49 -0500 |
---|---|---|
committer | Frédéric Guillot <fred@kanboard.net> | 2014-11-09 12:47:49 -0500 |
commit | 3df63e051fac84cec98c912668722f87d6e8183e (patch) | |
tree | 648a92db37f3295669ae6cc59126fc003235cfa8 /tests | |
parent | e0117cb8ed6326e8c77a02ce265c2b5f195e08d8 (diff) |
Add projects to the dashboard and rename some methods
Diffstat (limited to 'tests')
-rw-r--r-- | tests/units/NotificationTest.php | 4 | ||||
-rw-r--r-- | tests/units/ProjectPermissionTest.php | 32 | ||||
-rw-r--r-- | tests/units/ProjectTest.php | 4 |
3 files changed, 20 insertions, 20 deletions
diff --git a/tests/units/NotificationTest.php b/tests/units/NotificationTest.php index 1c8fb6e5..6c0539c2 100644 --- a/tests/units/NotificationTest.php +++ b/tests/units/NotificationTest.php @@ -31,7 +31,7 @@ class NotificationTest extends Base $this->assertTrue($u->create(array('username' => 'user4'))); // Nobody is member of any projects - $this->assertEmpty($pp->getAllowedUsers(1)); + $this->assertEmpty($pp->getMembers(1)); $this->assertEmpty($n->getUsersWithNotification(1)); // We allow all users to be member of our projects @@ -40,7 +40,7 @@ class NotificationTest extends Base $this->assertTrue($pp->allowUser(1, 3)); $this->assertTrue($pp->allowUser(1, 4)); - $this->assertNotEmpty($pp->getAllowedUsers(1)); + $this->assertNotEmpty($pp->getMembers(1)); $users = $n->getUsersWithNotification(1); $this->assertNotEmpty($users); diff --git a/tests/units/ProjectPermissionTest.php b/tests/units/ProjectPermissionTest.php index ee608d03..fd0c7331 100644 --- a/tests/units/ProjectPermissionTest.php +++ b/tests/units/ProjectPermissionTest.php @@ -22,16 +22,16 @@ class ProjectPermissionTest extends Base $this->assertTrue($pp->isUserAllowed(1, 1)); $this->assertFalse($pp->isUserAllowed(1, 2)); $this->assertFalse($pp->isUserAllowed(1, 3)); - $this->assertEquals(array(), $pp->getAllowedUsers(1)); - $this->assertEquals(array('Unassigned'), $pp->getUsersList(1)); + $this->assertEquals(array(), $pp->getMembers(1)); + $this->assertEquals(array('Unassigned'), $pp->getMemberList(1)); $this->assertTrue($p->update(array('id' => 1, 'is_everybody_allowed' => 1))); $this->assertTrue($pp->isEverybodyAllowed(1)); $this->assertTrue($pp->isUserAllowed(1, 1)); $this->assertTrue($pp->isUserAllowed(1, 2)); $this->assertTrue($pp->isUserAllowed(1, 3)); - $this->assertEquals(array('1' => 'admin', '2' => 'unittest#1', '3' => 'unittest#2'), $pp->getAllowedUsers(1)); - $this->assertEquals(array('Unassigned', '1' => 'admin', '2' => 'unittest#1', '3' => 'unittest#2'), $pp->getUsersList(1)); + $this->assertEquals(array('1' => 'admin', '2' => 'unittest#1', '3' => 'unittest#2'), $pp->getMembers(1)); + $this->assertEquals(array('Unassigned', '1' => 'admin', '2' => 'unittest#1', '3' => 'unittest#2'), $pp->getMemberList(1)); } public function testDisallowEverybody() @@ -45,7 +45,7 @@ class ProjectPermissionTest extends Base $this->assertEquals(1, $p->create(array('name' => 'UnitTest'))); - $this->assertEmpty($pp->getAllowedUsers(1)); // Nobody is specified for the given project + $this->assertEmpty($pp->getMembers(1)); // Nobody is specified for the given project $this->assertTrue($pp->isUserAllowed(1, 1)); // Admin should be allowed $this->assertFalse($pp->isUserAllowed(1, 2)); // Regular user should be denied } @@ -72,7 +72,7 @@ class ProjectPermissionTest extends Base $this->assertFalse($pp->allowUser(1, 50)); // Both users should be allowed - $this->assertEquals(array('1' => 'admin', '2' => 'unittest'), $pp->getAllowedUsers(1)); + $this->assertEquals(array('1' => 'admin', '2' => 'unittest'), $pp->getMembers(1)); $this->assertTrue($pp->isUserAllowed(1, 1)); $this->assertTrue($pp->isUserAllowed(1, 2)); } @@ -92,7 +92,7 @@ class ProjectPermissionTest extends Base $this->assertFalse($pp->revokeUser(1, 1)); // We should have nobody in the users list - $this->assertEmpty($pp->getAllowedUsers(1)); + $this->assertEmpty($pp->getMembers(1)); // Only admin is allowed $this->assertTrue($pp->isUserAllowed(1, 1)); @@ -106,11 +106,11 @@ class ProjectPermissionTest extends Base $this->assertTrue($pp->isUserAllowed(1, 2)); // However, we should have only our regular user in the list - $this->assertEquals(array('2' => 'unittest'), $pp->getAllowedUsers(1)); + $this->assertEquals(array('2' => 'unittest'), $pp->getMembers(1)); // We allow our admin, we should have both in the list $this->assertTrue($pp->allowUser(1, 1)); - $this->assertEquals(array('1' => 'admin', '2' => 'unittest'), $pp->getAllowedUsers(1)); + $this->assertEquals(array('1' => 'admin', '2' => 'unittest'), $pp->getMembers(1)); $this->assertTrue($pp->isUserAllowed(1, 1)); $this->assertTrue($pp->isUserAllowed(1, 2)); @@ -122,11 +122,11 @@ class ProjectPermissionTest extends Base $this->assertFalse($pp->isUserAllowed(1, 2)); // We should have only admin in the list - $this->assertEquals(array('1' => 'admin'), $pp->getAllowedUsers(1)); + $this->assertEquals(array('1' => 'admin'), $pp->getMembers(1)); // We revoke the admin user $this->assertTrue($pp->revokeUser(1, 1)); - $this->assertEmpty($pp->getAllowedUsers(1)); + $this->assertEmpty($pp->getMembers(1)); // Only admin should be allowed again $this->assertTrue($pp->isUserAllowed(1, 1)); @@ -147,7 +147,7 @@ class ProjectPermissionTest extends Base // No restriction, we should have no body $this->assertEquals( array('Unassigned'), - $pp->getUsersList(1) + $pp->getMemberList(1) ); // We allow only the regular user @@ -155,7 +155,7 @@ class ProjectPermissionTest extends Base $this->assertEquals( array(0 => 'Unassigned', 2 => 'unittest'), - $pp->getUsersList(1) + $pp->getMemberList(1) ); // We allow the admin user @@ -163,7 +163,7 @@ class ProjectPermissionTest extends Base $this->assertEquals( array(0 => 'Unassigned', 1 => 'admin', 2 => 'unittest'), - $pp->getUsersList(1) + $pp->getMemberList(1) ); // We revoke only the regular user @@ -171,7 +171,7 @@ class ProjectPermissionTest extends Base $this->assertEquals( array(0 => 'Unassigned', 1 => 'admin'), - $pp->getUsersList(1) + $pp->getMemberList(1) ); // We revoke only the admin user, we should have everybody @@ -179,7 +179,7 @@ class ProjectPermissionTest extends Base $this->assertEquals( array(0 => 'Unassigned'), - $pp->getUsersList(1) + $pp->getMemberList(1) ); } } diff --git a/tests/units/ProjectTest.php b/tests/units/ProjectTest.php index cec8d93d..c507739b 100644 --- a/tests/units/ProjectTest.php +++ b/tests/units/ProjectTest.php @@ -167,7 +167,7 @@ class ProjectTest extends Base $pp = new ProjectPermission($this->registry); - $this->assertEquals(array(1 => 'admin'), $pp->getAllowedUsers(3)); - $this->assertEquals(array(1 => 'admin'), $pp->getAllowedUsers(4)); + $this->assertEquals(array(1 => 'admin'), $pp->getMembers(3)); + $this->assertEquals(array(1 => 'admin'), $pp->getMembers(4)); } } |