Extract project permissions to a separate class

1 files changed, 160 insertions, 0 deletions

diff --git a/tests/units/ProjectPermissionTest.php b/tests/units/ProjectPermissionTest.php
new file mode 100644
index 00000000..309fa63b
--- /dev/null
+++ b/tests/units/ProjectPermissionTest.php
@@ -0,0 +1,160 @@
+<?php
+
+require_once __DIR__.'/Base.php';
+
+use Model\Project;
+use Model\ProjectPermission;
+use Model\User;
+
+class ProjectPermissionTest extends Base
+{
+    public function testAllowEverybody()
+    {
+        // We create a regular user
+        $user = new User($this->registry);
+        $user->create(array('username' => 'unittest', 'password' => 'unittest'));
+
+        $p = new Project($this->registry);
+        $pp = new ProjectPermission($this->registry);
+
+        $this->assertEquals(1, $p->create(array('name' => 'UnitTest')));
+
+        $this->assertEmpty($pp->getAllowedUsers(1)); // Nobody is specified for the given project
+        $this->assertTrue($pp->isUserAllowed(1, 1)); // Everybody should be allowed
+        $this->assertTrue($pp->isUserAllowed(1, 2)); // Everybody should be allowed
+    }
+
+    public function testAllowUser()
+    {
+        $p = new Project($this->registry);
+        $pp = new ProjectPermission($this->registry);
+        $user = new User($this->registry);
+
+        $user->create(array('username' => 'unittest', 'password' => 'unittest'));
+
+        // We create a project
+        $this->assertEquals(1, $p->create(array('name' => 'UnitTest')));
+
+        // We allow the admin user
+        $this->assertTrue($pp->allowUser(1, 1));
+
+        // Non-existant project
+        $this->assertFalse($pp->allowUser(50, 1));
+
+        // Non-existant user
+        $this->assertFalse($pp->allowUser(1, 50));
+
+        // Our admin user should be allowed
+        $this->assertEquals(array('1' => 'admin'), $pp->getAllowedUsers(1));
+        $this->assertTrue($pp->isUserAllowed(1, 1));
+
+        // Our regular user should be forbidden
+        $this->assertFalse($pp->isUserAllowed(1, 2));
+    }
+
+    public function testRevokeUser()
+    {
+        $p = new Project($this->registry);
+        $pp = new ProjectPermission($this->registry);
+        $user = new User($this->registry);
+
+        $user->create(array('username' => 'unittest', 'password' => 'unittest'));
+
+        // We create a project
+        $this->assertEquals(1, $p->create(array('name' => 'UnitTest')));
+
+        // We revoke our admin user (not existing row)
+        $this->assertFalse($pp->revokeUser(1, 1));
+
+        // We should have nobody in the users list
+        $this->assertEmpty($pp->getAllowedUsers(1));
+
+        // Our admin user and our regular user should be allowed
+        $this->assertTrue($pp->isUserAllowed(1, 1));
+        $this->assertTrue($pp->isUserAllowed(1, 2));
+
+        // We allow only the regular user
+        $this->assertTrue($pp->allowUser(1, 2));
+
+        // All users should be allowed (admin and regular)
+        $this->assertTrue($pp->isUserAllowed(1, 1));
+        $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));
+
+        // 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->assertTrue($pp->isUserAllowed(1, 1));
+        $this->assertTrue($pp->isUserAllowed(1, 2));
+
+        // We revoke the regular user
+        $this->assertTrue($pp->revokeUser(1, 2));
+
+        // Only admin should be allowed
+        $this->assertTrue($pp->isUserAllowed(1, 1));
+        $this->assertFalse($pp->isUserAllowed(1, 2));
+
+        // We should have only admin in the list
+        $this->assertEquals(array('1' => 'admin'), $pp->getAllowedUsers(1));
+
+        // We revoke the admin user
+        $this->assertTrue($pp->revokeUser(1, 1));
+        $this->assertEmpty($pp->getAllowedUsers(1));
+
+        // Everybody should be allowed again
+        $this->assertTrue($pp->isUserAllowed(1, 1));
+        $this->assertTrue($pp->isUserAllowed(1, 2));
+    }
+
+    public function testUsersList()
+    {
+        $p = new Project($this->registry);
+        $pp = new ProjectPermission($this->registry);
+
+        $user = new User($this->registry);
+        $user->create(array('username' => 'unittest', 'password' => 'unittest'));
+
+        // We create project
+        $this->assertEquals(1, $p->create(array('name' => 'UnitTest')));
+
+        // No restriction, we should have everybody
+        $this->assertEquals(
+            array('Unassigned', 'admin', 'unittest'),
+            $pp->getUsersList(1)
+        );
+
+        // We allow only the regular user
+        $this->assertTrue($pp->allowUser(1, 2));
+
+        $this->assertEquals(
+            array(0 => 'Unassigned', 2 => 'unittest'),
+            $pp->getUsersList(1)
+        );
+
+        // We allow the admin user
+        $this->assertTrue($pp->allowUser(1, 1));
+
+        $this->assertEquals(
+            array(0 => 'Unassigned', 1 => 'admin', 2 => 'unittest'),
+            $pp->getUsersList(1)
+        );
+
+        // We revoke only the regular user
+        $this->assertTrue($pp->revokeUser(1, 2));
+
+        $this->assertEquals(
+            array(0 => 'Unassigned', 1 => 'admin'),
+            $pp->getUsersList(1)
+        );
+
+        // We revoke only the admin user, we should have everybody
+        $this->assertTrue($pp->revokeUser(1, 1));
+
+        $this->assertEquals(
+            array(0 => 'Unassigned', 1 => 'admin', 2 => 'unittest'),
+            $pp->getUsersList(1)
+        );
+    }
+}