From 8689fe1a2eda9bfc0a870e893a71bee634ed986b Mon Sep 17 00:00:00 2001 From: Bernhard Posselt Date: Tue, 16 Dec 2014 20:01:49 +0100 Subject: [PATCH] use two tests instead of one --- tests/lib/user/session.php | 22 ++++++++++++++++++---- 1 file changed, 18 insertions(+), 4 deletions(-) diff --git a/tests/lib/user/session.php b/tests/lib/user/session.php index d441f802087..fb263d876b8 100644 --- a/tests/lib/user/session.php +++ b/tests/lib/user/session.php @@ -39,7 +39,7 @@ class Session extends \Test\TestCase { $session->expects($this->once()) ->method('get') ->with('user_id') - ->will($this->returnValue(null)); + ->will($this->returnValue('foo')); $backend = $this->getMock('OC_User_Dummy'); $backend->expects($this->once()) @@ -52,14 +52,28 @@ class Session extends \Test\TestCase { $userSession = new \OC\User\Session($manager, $session); $isLoggedIn = $userSession->isLoggedIn(); - $this->assertFalse($isLoggedIn); + $this->assertTrue($isLoggedIn); + } + public function testNotLoggedIn() { + $session = $this->getMock('\OC\Session\Memory', array(), array('')); $session->expects($this->once()) ->method('get') ->with('user_id') - ->will($this->returnValue('foo')); + ->will($this->returnValue(null)); + + $backend = $this->getMock('OC_User_Dummy'); + $backend->expects($this->once()) + ->method('userExists') + ->with('foo') + ->will($this->returnValue(true)); + + $manager = new \OC\User\Manager(); + $manager->registerBackend($backend); + + $userSession = new \OC\User\Session($manager, $session); $isLoggedIn = $userSession->isLoggedIn(); - $this->assertTrue($isLoggedIn); + $this->assertFalse($isLoggedIn); } public function testSetUser() { -- 2.39.5