summaryrefslogtreecommitdiffstats
path: root/apps/dav/tests/unit/CalDAV
diff options
context:
space:
mode:
authorThomas Citharel <tcit@tcit.fr>2020-01-21 10:32:48 +0100
committerThomas Citharel <tcit@tcit.fr>2020-04-26 11:11:50 +0200
commitdeb2ea9cead3dbf428877faed5136fe73198b6c7 (patch)
tree603142da0bedb9d90cae4a23592be498db78822c /apps/dav/tests/unit/CalDAV
parent0eeb6606796260bc3e0da3f2e6deb3a7cc522c3c (diff)
downloadnextcloud-server-deb2ea9cead3dbf428877faed5136fe73198b6c7.tar.gz
nextcloud-server-deb2ea9cead3dbf428877faed5136fe73198b6c7.zip
Provide links to calendar in event creation/update activities
Signed-off-by: Thomas Citharel <tcit@tcit.fr>
Diffstat (limited to 'apps/dav/tests/unit/CalDAV')
-rw-r--r--apps/dav/tests/unit/CalDAV/Activity/BackendTest.php22
-rw-r--r--apps/dav/tests/unit/CalDAV/Activity/Provider/BaseTest.php64
-rw-r--r--apps/dav/tests/unit/CalDAV/Activity/Provider/EventTest.php164
3 files changed, 196 insertions, 54 deletions
diff --git a/apps/dav/tests/unit/CalDAV/Activity/BackendTest.php b/apps/dav/tests/unit/CalDAV/Activity/BackendTest.php
index f4179c60665..7b977423765 100644
--- a/apps/dav/tests/unit/CalDAV/Activity/BackendTest.php
+++ b/apps/dav/tests/unit/CalDAV/Activity/BackendTest.php
@@ -29,40 +29,47 @@ use OCA\DAV\CalDAV\Activity\Backend;
use OCA\DAV\CalDAV\Activity\Provider\Calendar;
use OCP\Activity\IEvent;
use OCP\Activity\IManager;
+use OCP\App\IAppManager;
use OCP\IGroup;
use OCP\IGroupManager;
use OCP\IUser;
use OCP\IUserSession;
+use PHPUnit\Framework\MockObject\MockObject;
use Test\TestCase;
class BackendTest extends TestCase {
- /** @var IManager|\PHPUnit_Framework_MockObject_MockObject */
+ /** @var IManager|MockObject */
protected $activityManager;
- /** @var IGroupManager|\PHPUnit_Framework_MockObject_MockObject */
+ /** @var IGroupManager|MockObject */
protected $groupManager;
- /** @var IUserSession|\PHPUnit_Framework_MockObject_MockObject */
+ /** @var IUserSession|MockObject */
protected $userSession;
+ /** @var IAppManager|MockObject */
+ protected $appManager;
+
protected function setUp(): void {
parent::setUp();
$this->activityManager = $this->createMock(IManager::class);
$this->groupManager = $this->createMock(IGroupManager::class);
$this->userSession = $this->createMock(IUserSession::class);
+ $this->appManager = $this->createMock(IAppManager::class);
}
/**
* @param array $methods
- * @return Backend|\PHPUnit_Framework_MockObject_MockObject
+ * @return Backend|MockObject
*/
protected function getBackend(array $methods = []) {
if (empty($methods)) {
return new Backend(
$this->activityManager,
$this->groupManager,
- $this->userSession
+ $this->userSession,
+ $this->appManager
);
} else {
return $this->getMockBuilder(Backend::class)
@@ -70,6 +77,7 @@ class BackendTest extends TestCase {
$this->activityManager,
$this->groupManager,
$this->userSession,
+ $this->appManager,
])
->setMethods($methods)
->getMock();
@@ -339,7 +347,7 @@ class BackendTest extends TestCase {
/**
* @param string[] $users
- * @return IUser[]|\PHPUnit_Framework_MockObject_MockObject[]
+ * @return IUser[]|MockObject[]
*/
protected function getUsers(array $users) {
$list = [];
@@ -351,7 +359,7 @@ class BackendTest extends TestCase {
/**
* @param string $uid
- * @return IUser|\PHPUnit_Framework_MockObject_MockObject
+ * @return IUser|MockObject
*/
protected function getUserMock($uid) {
$user = $this->createMock(IUser::class);
diff --git a/apps/dav/tests/unit/CalDAV/Activity/Provider/BaseTest.php b/apps/dav/tests/unit/CalDAV/Activity/Provider/BaseTest.php
index eb0ac34c15a..e4fa05452cb 100644
--- a/apps/dav/tests/unit/CalDAV/Activity/Provider/BaseTest.php
+++ b/apps/dav/tests/unit/CalDAV/Activity/Provider/BaseTest.php
@@ -5,6 +5,7 @@
* @author Joas Schilling <coding@schilljs.com>
* @author John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* @author Roeland Jago Douma <roeland@famdouma.nl>
+ * @author Thomas Citharel <nextcloud@tcit.fr>
*
* @license GNU AGPL version 3 or any later version
*
@@ -30,29 +31,36 @@ use OCP\Activity\IEvent;
use OCP\Activity\IProvider;
use OCP\IGroupManager;
use OCP\IL10N;
+use OCP\IURLGenerator;
use OCP\IUser;
use OCP\IUserManager;
+use PHPUnit\Framework\MockObject\MockObject;
use Test\TestCase;
class BaseTest extends TestCase {
- /** @var IUserManager|\PHPUnit_Framework_MockObject_MockObject */
+ /** @var IUserManager|MockObject */
protected $userManager;
- /** @var IGroupManager|\PHPUnit_Framework_MockObject_MockObject */
+ /** @var IGroupManager|MockObject */
protected $groupManager;
- /** @var IProvider|Base|\PHPUnit_Framework_MockObject_MockObject */
+ /** @var IURLGenerator|MockObject */
+ protected $url;
+
+ /** @var IProvider|Base|MockObject */
protected $provider;
protected function setUp(): void {
parent::setUp();
$this->userManager = $this->createMock(IUserManager::class);
$this->groupManager = $this->createMock(IGroupManager::class);
+ $this->url = $this->createMock(IURLGenerator::class);
$this->provider = $this->getMockBuilder(Base::class)
->setConstructorArgs([
$this->userManager,
- $this->groupManager
+ $this->groupManager,
+ $this->url,
])
->setMethods(['parse'])
->getMock();
@@ -71,7 +79,7 @@ class BaseTest extends TestCase {
* @param array $parameters
* @param string $parsedSubject
*/
- public function testSetSubjects($subject, array $parameters, $parsedSubject) {
+ public function testSetSubjects(string $subject, array $parameters, string $parsedSubject) {
$event = $this->createMock(IEvent::class);
$event->expects($this->once())
->method('setRichSubject')
@@ -85,44 +93,6 @@ class BaseTest extends TestCase {
$this->invokePrivate($this->provider, 'setSubjects', [$event, $subject, $parameters]);
}
- public function dataGenerateObjectParameter() {
- return [
- [23, 'c1'],
- [42, 'c2'],
- ];
- }
-
- /**
- * @dataProvider dataGenerateObjectParameter
- * @param int $id
- * @param string $name
- */
- public function testGenerateObjectParameter($id, $name) {
- $this->assertEquals([
- 'type' => 'calendar-event',
- 'id' => $id,
- 'name' => $name,
- ], $this->invokePrivate($this->provider, 'generateObjectParameter', [['id' => $id, 'name' => $name]]));
- }
-
- public function dataGenerateObjectParameterThrows() {
- return [
- ['event'],
- [['name' => 'event']],
- [['id' => 42]],
- ];
- }
-
- /**
- * @dataProvider dataGenerateObjectParameterThrows
- * @param mixed $eventData
- */
- public function testGenerateObjectParameterThrows($eventData) {
- $this->expectException(\InvalidArgumentException::class);
-
- $this->invokePrivate($this->provider, 'generateObjectParameter', [$eventData]);
- }
-
public function dataGenerateCalendarParameter() {
return [
[['id' => 23, 'uri' => 'foo', 'name' => 'bar'], 'bar'],
@@ -137,7 +107,7 @@ class BaseTest extends TestCase {
* @param array $data
* @param string $name
*/
- public function testGenerateCalendarParameter(array $data, $name) {
+ public function testGenerateCalendarParameter(array $data, string $name) {
$l = $this->createMock(IL10N::class);
$l->expects($this->any())
->method('t')
@@ -164,7 +134,7 @@ class BaseTest extends TestCase {
* @param int $id
* @param string $name
*/
- public function testGenerateLegacyCalendarParameter($id, $name) {
+ public function testGenerateLegacyCalendarParameter(int $id, string $name) {
$this->assertEquals([
'type' => 'calendar',
'id' => $id,
@@ -183,7 +153,7 @@ class BaseTest extends TestCase {
* @dataProvider dataGenerateGroupParameter
* @param string $gid
*/
- public function testGenerateGroupParameter($gid) {
+ public function testGenerateGroupParameter(string $gid) {
$this->assertEquals([
'type' => 'user-group',
'id' => $gid,
@@ -208,7 +178,7 @@ class BaseTest extends TestCase {
* @param string $displayName
* @param IUser|null $user
*/
- public function testGenerateUserParameter($uid, $displayName, $user) {
+ public function testGenerateUserParameter(string $uid, string $displayName, ?IUser $user) {
$this->userManager->expects($this->once())
->method('get')
->with($uid)
diff --git a/apps/dav/tests/unit/CalDAV/Activity/Provider/EventTest.php b/apps/dav/tests/unit/CalDAV/Activity/Provider/EventTest.php
new file mode 100644
index 00000000000..2ae78444c11
--- /dev/null
+++ b/apps/dav/tests/unit/CalDAV/Activity/Provider/EventTest.php
@@ -0,0 +1,164 @@
+<?php
+/**
+ * @copyright Copyright (c) 2020 Thomas Citharel <nextcloud@tcit.fr>
+ *
+ * @author Thomas Citharel <nextcloud@tcit.fr>
+ *
+ * @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/>.
+ *
+ */
+
+namespace OCA\DAV\Tests\unit\CalDAV\Activity\Provider;
+
+use InvalidArgumentException;
+use OCA\DAV\CalDAV\Activity\Provider\Base;
+use OCA\DAV\CalDAV\Activity\Provider\Event;
+use OCP\Activity\IEventMerger;
+use OCP\Activity\IManager;
+use OCP\Activity\IProvider;
+use OCP\App\IAppManager;
+use OCP\IGroupManager;
+use OCP\IURLGenerator;
+use OCP\IUserManager;
+use OCP\L10N\IFactory;
+use PHPUnit\Framework\MockObject\MockObject;
+use Test\TestCase;
+use TypeError;
+
+class EventTest extends TestCase {
+
+ /** @var IUserManager|MockObject */
+ protected $userManager;
+
+ /** @var IGroupManager|MockObject */
+ protected $groupManager;
+
+ /** @var IURLGenerator|MockObject */
+ protected $url;
+
+ /** @var IProvider|Base|MockObject */
+ protected $provider;
+
+ /** @var IAppManager|MockObject */
+ protected $appManager;
+
+ /** @var IFactory|MockObject */
+ protected $i10nFactory;
+
+ /** @var IManager|MockObject */
+ protected $activityManager;
+
+ /** @var IEventMerger|MockObject */
+ protected $eventMerger;
+
+ protected function setUp(): void {
+ parent::setUp();
+ $this->i10nFactory = $this->createMock(IFactory::class);
+ $this->userManager = $this->createMock(IUserManager::class);
+ $this->groupManager = $this->createMock(IGroupManager::class);
+ $this->activityManager = $this->createMock(IManager::class);
+ $this->url = $this->createMock(IURLGenerator::class);
+ $this->appManager = $this->createMock(IAppManager::class);
+ $this->eventMerger = $this->createMock(IEventMerger::class);
+ $this->provider = $this->getMockBuilder(Event::class)
+ ->setConstructorArgs([
+ $this->i10nFactory,
+ $this->url,
+ $this->activityManager,
+ $this->userManager,
+ $this->groupManager,
+ $this->eventMerger,
+ $this->appManager
+ ])
+ ->setMethods(['parse'])
+ ->getMock();
+ }
+
+ public function dataGenerateObjectParameter() {
+ $link = [
+ 'object_uri' => 'someuuid.ics',
+ 'calendar_uri' => 'personal',
+ 'owner' => 'someuser'
+ ];
+
+ return [
+ [23, 'c1', $link, true],
+ [23, 'c1', $link, false],
+ [42, 'c2', null],
+ ];
+ }
+
+ /**
+ * @dataProvider dataGenerateObjectParameter
+ * @param int $id
+ * @param string $name
+ * @param array|null $link
+ * @param bool $calendarAppEnabled
+ */
+ public function testGenerateObjectParameter(int $id, string $name, ?array $link, bool $calendarAppEnabled = true) {
+ if ($link) {
+ $generatedLink = [
+ 'view' => 'dayGridMonth',
+ 'timeRange' => 'now',
+ 'mode' => 'sidebar',
+ 'objectId' => base64_encode('/remote.php/dav/calendars/' . $link['owner'] . '/' . $link['calendar_uri'] . '/' . $link['object_uri']),
+ 'recurrenceId' => 'next'
+ ];
+ $this->appManager->expects($this->once())
+ ->method('isEnabledForUser')
+ ->with('calendar')
+ ->willReturn($calendarAppEnabled);
+ if ($calendarAppEnabled) {
+ $this->url->expects($this->once())
+ ->method('linkToRouteAbsolute')
+ ->with('calendar.view.indexview.timerange.edit', $generatedLink)
+ ->willReturn('fullLink');
+ }
+ }
+ $objectParameter = ['id' => $id, 'name' => $name];
+ if ($link) {
+ $objectParameter['link'] = $link;
+ }
+ $result = [
+ 'type' => 'calendar-event',
+ 'id' => $id,
+ 'name' => $name,
+ ];
+ if ($link && $calendarAppEnabled) {
+ $result['link'] = 'fullLink';
+ }
+ $this->assertEquals($result, $this->invokePrivate($this->provider, 'generateObjectParameter', [$objectParameter]));
+ }
+
+ public function dataGenerateObjectParameterThrows() {
+ return [
+ ['event', TypeError::class],
+ [['name' => 'event']],
+ [['id' => 42]],
+ ];
+ }
+
+ /**
+ * @dataProvider dataGenerateObjectParameterThrows
+ * @param mixed $eventData
+ * @param string $exception
+ */
+ public function testGenerateObjectParameterThrows($eventData, string $exception = InvalidArgumentException::class) {
+ $this->expectException($exception);
+
+ $this->invokePrivate($this->provider, 'generateObjectParameter', [$eventData]);
+ }
+}