Fixing tests ...

owncloud · Sep 29, 2016 · 43d9b75 · 43d9b75
1 parent 202e102
commit 43d9b75
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/apps/dav/tests/unit/DAV/HookManagerTest.php b/apps/dav/tests/unit/DAV/HookManagerTest.php
@@ -60,7 +60,6 @@ public function test() {
 		$userManager = $this->getMockBuilder(IUserManager::class)
 			->disableOriginalConstructor()
 			->getMock();
-		$userManager->expects($this->once())->method('get')->willReturn($user);
 
 		/** @var SyncService | \PHPUnit_Framework_MockObject_MockObject $syncService */
 		$syncService = $this->getMockBuilder(SyncService::class)
@@ -90,7 +89,7 @@ public function test() {
 			'contacts', ['{DAV:}displayname' => $this->l10n->t('Contacts')]);
 
 		$hm = new HookManager($userManager, $syncService, $cal, $card, $this->l10n);
-		$hm->postLogin(['uid' => 'newUser']);
+		$hm->firstLogin($user);
 	}
 
 	public function testWithExisting() {
@@ -103,7 +102,6 @@ public function testWithExisting() {
 		$userManager = $this->getMockBuilder(IUserManager::class)
 			->disableOriginalConstructor()
 			->getMock();
-		$userManager->expects($this->once())->method('get')->willReturn($user);
 
 		/** @var SyncService | \PHPUnit_Framework_MockObject_MockObject $syncService */
 		$syncService = $this->getMockBuilder(SyncService::class)
@@ -129,7 +127,7 @@ public function testWithExisting() {
 		$card->expects($this->never())->method('createAddressBook');
 
 		$hm = new HookManager($userManager, $syncService, $cal, $card, $this->l10n);
-		$hm->postLogin(['uid' => 'newUser']);
+		$hm->firstLogin($user);
 	}
 
 	public function testWithBirthdayCalendar() {
@@ -142,7 +140,6 @@ public function testWithBirthdayCalendar() {
 		$userManager = $this->getMockBuilder(IUserManager::class)
 			->disableOriginalConstructor()
 			->getMock();
-		$userManager->expects($this->once())->method('get')->willReturn($user);
 
 		/** @var SyncService | \PHPUnit_Framework_MockObject_MockObject $syncService */
 		$syncService = $this->getMockBuilder(SyncService::class)
@@ -174,7 +171,7 @@ public function testWithBirthdayCalendar() {
 			'contacts', ['{DAV:}displayname' => $this->l10n->t('Contacts')]);
 
 		$hm = new HookManager($userManager, $syncService, $cal, $card, $this->l10n);
-		$hm->postLogin(['uid' => 'newUser']);
+		$hm->firstLogin($user);
 	}
 
 	public function testDeleteCalendar() {

diff --git a/tests/lib/TestCase.php b/tests/lib/TestCase.php
@@ -332,6 +332,7 @@ static protected function tearDownAfterClassCleanStrayLocks() {
 	static protected function loginAsUser($user = '') {
 		self::logout();
 		\OC\Files\Filesystem::tearDown();
+		\OC::$server->getConfig()->setUserValue($user, 'login', 'lastLogin', time());
 		\OC_User::setUserId($user);
 		\OC_Util::setupFS($user);
 		if (\OC_User::userExists($user)) {