From 0939c3147e24b91b0ac6087c4a9371372aa88d45 Mon Sep 17 00:00:00 2001
From: Bernhard Posselt <dev@bernhard-posselt.com>
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 d441f80208..fb263d876b 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() {
--
GitLab