aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files/tests/Activity/ProviderTest.php
diff options
context:
space:
mode:
Diffstat (limited to 'apps/files/tests/Activity/ProviderTest.php')
-rw-r--r--apps/files/tests/Activity/ProviderTest.php197
1 files changed, 88 insertions, 109 deletions
diff --git a/apps/files/tests/Activity/ProviderTest.php b/apps/files/tests/Activity/ProviderTest.php
index 4a835f42d75..b6ba095ecfe 100644
--- a/apps/files/tests/Activity/ProviderTest.php
+++ b/apps/files/tests/Activity/ProviderTest.php
@@ -1,37 +1,25 @@
<?php
+
+declare(strict_types=1);
/**
- * @copyright Copyright (c) 2017 Joas Schilling <coding@schilljs.com>
- *
- * @author Joas Schilling <coding@schilljs.com>
- *
- * @license GNU AGPL version 3 or any later version
- *
- * This program is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Affero General Public License as
- * published by the Free Software Foundation, either version 3 of the
- * License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU Affero General Public License for more details.
- *
- * You should have received a copy of the GNU Affero General Public License
- * along with this program. If not, see <http://www.gnu.org/licenses/>.
- *
+ * SPDX-FileCopyrightText: 2017 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
*/
-
namespace OCA\Files\Tests\Activity;
-
use OCA\Files\Activity\Provider;
+use OCP\Activity\Exceptions\UnknownActivityException;
use OCP\Activity\IEvent;
use OCP\Activity\IEventMerger;
use OCP\Activity\IManager;
+use OCP\Contacts\IManager as IContactsManager;
+use OCP\Federation\ICloudId;
+use OCP\Federation\ICloudIdManager;
+use OCP\Files\IRootFolder;
use OCP\IURLGenerator;
-use OCP\IUser;
use OCP\IUserManager;
use OCP\L10N\IFactory;
+use PHPUnit\Framework\MockObject\MockObject;
use Test\TestCase;
/**
@@ -40,31 +28,31 @@ use Test\TestCase;
* @package OCA\Files\Tests\Activity
*/
class ProviderTest extends TestCase {
-
- /** @var IFactory|\PHPUnit_Framework_MockObject_MockObject */
- protected $l10nFactory;
- /** @var IURLGenerator|\PHPUnit_Framework_MockObject_MockObject */
- protected $url;
- /** @var IManager|\PHPUnit_Framework_MockObject_MockObject */
- protected $activityManager;
- /** @var IUserManager|\PHPUnit_Framework_MockObject_MockObject */
- protected $userManager;
- /** @var IEventMerger|\PHPUnit_Framework_MockObject_MockObject */
- protected $eventMerger;
-
- public function setUp() {
+ protected IFactory&MockObject $l10nFactory;
+ protected IURLGenerator&MockObject $url;
+ protected IManager&MockObject $activityManager;
+ protected IUserManager&MockObject $userManager;
+ protected IRootFolder&MockObject $rootFolder;
+ protected ICloudIdManager&MockObject $cloudIdManager;
+ protected IContactsManager&MockObject $contactsManager;
+ protected IEventMerger&MockObject $eventMerger;
+
+ protected function setUp(): void {
parent::setUp();
$this->l10nFactory = $this->createMock(IFactory::class);
$this->url = $this->createMock(IURLGenerator::class);
$this->activityManager = $this->createMock(IManager::class);
$this->userManager = $this->createMock(IUserManager::class);
+ $this->rootFolder = $this->createMock(IRootFolder::class);
+ $this->cloudIdManager = $this->createMock(ICloudIdManager::class);
+ $this->contactsManager = $this->createMock(IContactsManager::class);
$this->eventMerger = $this->createMock(IEventMerger::class);
}
/**
* @param string[] $methods
- * @return Provider|\PHPUnit_Framework_MockObject_MockObject
+ * @return Provider|MockObject
*/
protected function getProvider(array $methods = []) {
if (!empty($methods)) {
@@ -74,9 +62,12 @@ class ProviderTest extends TestCase {
$this->url,
$this->activityManager,
$this->userManager,
+ $this->rootFolder,
+ $this->cloudIdManager,
+ $this->contactsManager,
$this->eventMerger,
])
- ->setMethods($methods)
+ ->onlyMethods($methods)
->getMock();
}
return new Provider(
@@ -84,27 +75,23 @@ class ProviderTest extends TestCase {
$this->url,
$this->activityManager,
$this->userManager,
+ $this->rootFolder,
+ $this->cloudIdManager,
+ $this->contactsManager,
$this->eventMerger
);
}
- public function dataGetFile() {
+ public static function dataGetFile(): array {
return [
[[42 => '/FortyTwo.txt'], null, '42', 'FortyTwo.txt', 'FortyTwo.txt'],
[['23' => '/Twenty/Three.txt'], null, '23', 'Three.txt', 'Twenty/Three.txt'],
- ['/Foo/Bar.txt', '128', '128', 'Bar.txt', 'Foo/Bar.txt'], // Legacy from ownCloud 8.2 and before
+ ['/Foo/Bar.txt', 128, '128', 'Bar.txt', 'Foo/Bar.txt'], // Legacy from ownCloud 8.2 and before
];
}
- /**
- * @dataProvider dataGetFile
- * @param mixed $parameter
- * @param mixed $eventId
- * @param int $id
- * @param string $name
- * @param string $path
- */
- public function testGetFile($parameter, $eventId, $id, $name, $path) {
+ #[\PHPUnit\Framework\Attributes\DataProvider('dataGetFile')]
+ public function testGetFile(array|string $parameter, ?int $eventId, string $id, string $name, string $path): void {
$provider = $this->getProvider();
if ($eventId !== null) {
@@ -130,81 +117,73 @@ class ProviderTest extends TestCase {
$this->assertSame('link-' . $id, $result['link']);
}
- /**
- * @expectedException \InvalidArgumentException
- */
- public function testGetFileThrows() {
- $provider = $this->getProvider();
- self::invokePrivate($provider, 'getFile', ['/Foo/Bar.txt', null]);
- }
- public function dataGetUser() {
- return [
- ['test', [], false, 'Test'],
- ['foo', ['admin' => 'Admin'], false, 'Bar'],
- ['admin', ['admin' => 'Administrator'], true, 'Administrator'],
- ];
- }
+ public function testGetFileThrows(): void {
+ $this->expectException(UnknownActivityException::class);
- /**
- * @dataProvider dataGetUser
- * @param string $uid
- * @param array $cache
- * @param bool $cacheHit
- * @param string $name
- */
- public function testGetUser($uid, $cache, $cacheHit, $name) {
- $provider = $this->getProvider(['getDisplayName']);
-
- self::invokePrivate($provider, 'displayNames', [$cache]);
-
- if (!$cacheHit) {
- $provider->expects($this->once())
- ->method('getDisplayName')
- ->with($uid)
- ->willReturn($name);
- } else {
- $provider->expects($this->never())
- ->method('getDisplayName');
- }
-
- $result = self::invokePrivate($provider, 'getUser', [$uid]);
- $this->assertSame('user', $result['type']);
- $this->assertSame($uid, $result['id']);
- $this->assertSame($name, $result['name']);
+ $provider = $this->getProvider();
+ self::invokePrivate($provider, 'getFile', ['/Foo/Bar.txt', null]);
}
- public function dataGetDisplayName() {
+ public static function dataGetUser(): array {
return [
- ['test', true, 'Test'],
- ['foo', false, 'foo'],
+ ['test', 'Test user', null, ['type' => 'user', 'id' => 'test', 'name' => 'Test user']],
+ ['test@http://localhost', null, ['user' => 'test', 'displayId' => 'test@localhost', 'remote' => 'localhost', 'name' => null], ['type' => 'user', 'id' => 'test', 'name' => 'test@localhost', 'server' => 'localhost']],
+ ['test@http://localhost', null, ['user' => 'test', 'displayId' => 'test@localhost', 'remote' => 'localhost', 'name' => 'Remote user'], ['type' => 'user', 'id' => 'test', 'name' => 'Remote user (test@localhost)', 'server' => 'localhost']],
+ ['test', null, null, ['type' => 'user', 'id' => 'test', 'name' => 'test']],
];
}
- /**
- * @dataProvider dataGetDisplayName
- * @param string $uid
- * @param string $name
- */
- public function testGetDisplayNamer($uid, $validUser, $name) {
+ #[\PHPUnit\Framework\Attributes\DataProvider('dataGetUser')]
+ public function testGetUser(string $uid, ?string $userDisplayName, ?array $cloudIdData, array $expected): void {
$provider = $this->getProvider();
- if ($validUser) {
- $user = $this->createMock(IUser::class);
- $user->expects($this->once())
- ->method('getDisplayName')
- ->willReturn($name);
+ if ($userDisplayName !== null) {
$this->userManager->expects($this->once())
- ->method('get')
+ ->method('getDisplayName')
->with($uid)
- ->willReturn($user);
- } else {
- $this->userManager->expects($this->once())
- ->method('get')
+ ->willReturn($userDisplayName);
+ }
+ if ($cloudIdData !== null) {
+ $this->cloudIdManager->expects($this->once())
+ ->method('isValidCloudId')
+ ->willReturn(true);
+
+ $cloudId = $this->createMock(ICloudId::class);
+ $cloudId->expects($this->once())
+ ->method('getUser')
+ ->willReturn($cloudIdData['user']);
+ $cloudId->expects($this->once())
+ ->method('getDisplayId')
+ ->willReturn($cloudIdData['displayId']);
+ $cloudId->expects($this->once())
+ ->method('getRemote')
+ ->willReturn($cloudIdData['remote']);
+
+ $this->cloudIdManager->expects($this->once())
+ ->method('resolveCloudId')
->with($uid)
- ->willReturn(null);
+ ->willReturn($cloudId);
+
+ if ($cloudIdData['name'] !== null) {
+ $this->contactsManager->expects($this->once())
+ ->method('search')
+ ->with($cloudIdData['displayId'], ['CLOUD'])
+ ->willReturn([
+ [
+ 'CLOUD' => $cloudIdData['displayId'],
+ 'FN' => $cloudIdData['name'],
+ ]
+ ]);
+ } else {
+ $this->contactsManager->expects($this->once())
+ ->method('search')
+ ->with($cloudIdData['displayId'], ['CLOUD'])
+ ->willReturn([]);
+ }
}
- $this->assertSame($name, self::invokePrivate($provider, 'getDisplayName', [$uid]));
+ $result = self::invokePrivate($provider, 'getUser', [$uid]);
+ $this->assertEquals($expected, $result);
}
}
generated by cgit v1.2.3 (git 2.39.1) at 2025-08-16 01:36:40 +0000