diff options
author | Frédéric Guillot <fred@kanboard.net> | 2019-01-30 22:25:57 -0800 |
---|---|---|
committer | Frédéric Guillot <fred@kanboard.net> | 2019-01-30 22:25:57 -0800 |
commit | 928f80d569141601c4d8f6652b2ac0a2497c7be4 (patch) | |
tree | e87dcd2712b3b433332f14bb5a2fa84f930dd224 /tests | |
parent | 83deec2e3621c40d15a06e2491f27571d32fe10f (diff) |
Update unit tests
Diffstat (limited to 'tests')
-rw-r--r-- | tests/units/Auth/DatabaseAuthTest.php | 15 | ||||
-rw-r--r-- | tests/units/Core/Security/AuthenticationManagerTest.php | 4 | ||||
-rw-r--r-- | tests/units/Middleware/AuthenticationMiddlewareTest.php | 1 |
3 files changed, 13 insertions, 7 deletions
diff --git a/tests/units/Auth/DatabaseAuthTest.php b/tests/units/Auth/DatabaseAuthTest.php index 50ae3c7e..e2236837 100644 --- a/tests/units/Auth/DatabaseAuthTest.php +++ b/tests/units/Auth/DatabaseAuthTest.php @@ -46,17 +46,24 @@ class DatabaseAuthTest extends Base $this->assertFalse($provider->isValidSession()); - $this->assertEquals(2, $userModel->create(array('username' => 'foobar'))); + $this->assertEquals(2, $userModel->create(array('username' => 'user1'))); + $this->assertEquals(3, $userModel->create(array('username' => 'user2'))); - $_SESSION['user'] = array('id' => 2); + $_SESSION['user'] = array('id' => 2, 'role' => 'app-user'); $this->assertTrue($provider->isValidSession()); - $_SESSION['user'] = array('id' => 3); + $_SESSION['user'] = array('id' => 4, 'role' => 'app-user'); $this->assertFalse($provider->isValidSession()); $this->assertTrue($userModel->disable(2)); - $_SESSION['user'] = array('id' => 2); + $_SESSION['user'] = array('id' => 2, 'role' => 'app-user'); + $this->assertFalse($provider->isValidSession()); + + $_SESSION['user'] = array('id' => 3, 'role' => 'app-user'); + $this->assertTrue($provider->isValidSession()); + + $_SESSION['user'] = array('id' => 3, 'role' => 'app-admin'); $this->assertFalse($provider->isValidSession()); } } diff --git a/tests/units/Core/Security/AuthenticationManagerTest.php b/tests/units/Core/Security/AuthenticationManagerTest.php index a5a23c7e..616b7fa8 100644 --- a/tests/units/Core/Security/AuthenticationManagerTest.php +++ b/tests/units/Core/Security/AuthenticationManagerTest.php @@ -56,7 +56,7 @@ class AuthenticationManagerTest extends Base $authManager = new AuthenticationManager($this->container); $authManager->register(new DatabaseAuth($this->container)); - $_SESSION['user'] = array('id' => 1, 'username' => 'test'); + $_SESSION['user'] = array('id' => 1, 'username' => 'test', 'role' => 'app-admin'); $this->assertTrue($this->container['userSession']->isLogged()); $this->assertTrue($authManager->checkCurrentSession()); @@ -67,7 +67,7 @@ class AuthenticationManagerTest extends Base $authManager = new AuthenticationManager($this->container); $authManager->register(new DatabaseAuth($this->container)); - $_SESSION['user'] = array('id' => 42, 'username' => 'test'); + $_SESSION['user'] = array('id' => 42, 'username' => 'test', 'role' => 'app-admin'); $this->assertTrue($this->container['userSession']->isLogged()); $this->assertFalse($authManager->checkCurrentSession()); diff --git a/tests/units/Middleware/AuthenticationMiddlewareTest.php b/tests/units/Middleware/AuthenticationMiddlewareTest.php index b6819adc..2cbcc123 100644 --- a/tests/units/Middleware/AuthenticationMiddlewareTest.php +++ b/tests/units/Middleware/AuthenticationMiddlewareTest.php @@ -60,7 +60,6 @@ class AuthenticationMiddlewareTest extends Base ->expects($this->never()) ->method('execute'); - $this->setExpectedException('Kanboard\Core\Controller\AccessForbiddenException'); $this->middleware->execute(); } |