nextcloud / server

apps/dav/tests/unit/CalDAV/CalendarImplTest.php

Spacing +1 added lines, -1 removed lines

@@ -257,7 +257,7 @@
 		$iTipMessage->sender = $attendee;
 		$iTipMessage->uid = isset($vEvent->{'UID'}) ? $vEvent->{'UID'}->getValue() : '';
 		$iTipMessage->component = 'VEVENT';
-		$iTipMessage->sequence = isset($vEvent->{'SEQUENCE'}) ? (int)$vEvent->{'SEQUENCE'}->getValue() : 0;
+		$iTipMessage->sequence = isset($vEvent->{'SEQUENCE'}) ? (int) $vEvent->{'SEQUENCE'}->getValue() : 0;
 		$iTipMessage->message = $vObject;
 		return $iTipMessage;
 	}

Indentation +253 added lines, -253 removed lines

@@ -23,108 +23,108 @@ 
 use Sabre\VObject\Reader;
 
 class CalendarImplTest extends \Test\TestCase {
-	private Calendar&MockObject $calendar;
-	private array $calendarInfo;
-	private CalDavBackend&MockObject $backend;
-	private CalendarImpl $calendarImpl;
-	private array $mockExportCollection;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->calendar = $this->createMock(Calendar::class);
-		$this->calendarInfo = [
-			'id' => 1,
-			'{DAV:}displayname' => 'user readable name 123',
-			'{http://apple.com/ns/ical/}calendar-color' => '#AABBCC',
-			'uri' => '/this/is/a/uri',
-			'principaluri' => 'principal/users/foobar'
-		];
-		$this->backend = $this->createMock(CalDavBackend::class);
-
-		$this->calendarImpl = new CalendarImpl(
-			$this->calendar,
-			$this->calendarInfo,
-			$this->backend
-		);
-	}
-
-
-	public function testGetKey(): void {
-		$this->assertEquals($this->calendarImpl->getKey(), 1);
-	}
-
-	public function testGetDisplayname(): void {
-		$this->assertEquals($this->calendarImpl->getDisplayName(), 'user readable name 123');
-	}
-
-	public function testGetDisplayColor(): void {
-		$this->assertEquals($this->calendarImpl->getDisplayColor(), '#AABBCC');
-	}
-
-	public function testSearch(): void {
-		$this->backend->expects($this->once())
-			->method('search')
-			->with($this->calendarInfo, 'abc', ['def'], ['ghi'], 42, 1337)
-			->willReturn(['SEARCHRESULTS']);
-
-		$result = $this->calendarImpl->search('abc', ['def'], ['ghi'], 42, 1337);
-		$this->assertEquals($result, ['SEARCHRESULTS']);
-	}
-
-	public function testGetPermissionRead(): void {
-		$this->calendar->expects($this->once())
-			->method('getACL')
-			->with()
-			->willReturn([
-				['privilege' => '{DAV:}read', 'principal' => 'principal/users/foobar'],
-				['privilege' => '{DAV:}read', 'principal' => 'principal/users/other'],
-				['privilege' => '{DAV:}write', 'principal' => 'principal/users/other'],
-				['privilege' => '{DAV:}all', 'principal' => 'principal/users/other'],
-			]);
-
-		$this->assertEquals(1, $this->calendarImpl->getPermissions());
-	}
-
-	public function testGetPermissionWrite(): void {
-		$this->calendar->expects($this->once())
-			->method('getACL')
-			->with()
-			->willReturn([
-				['privilege' => '{DAV:}write', 'principal' => 'principal/users/foobar'],
-				['privilege' => '{DAV:}read', 'principal' => 'principal/users/other'],
-				['privilege' => '{DAV:}all', 'principal' => 'principal/users/other'],
-			]);
-
-		$this->assertEquals(6, $this->calendarImpl->getPermissions());
-	}
-
-	public function testGetPermissionReadWrite(): void {
-		$this->calendar->expects($this->once())
-			->method('getACL')
-			->with()
-			->willReturn([
-				['privilege' => '{DAV:}write', 'principal' => 'principal/users/foobar'],
-				['privilege' => '{DAV:}read', 'principal' => 'principal/users/foobar'],
-				['privilege' => '{DAV:}all', 'principal' => 'principal/users/other'],
-			]);
-
-		$this->assertEquals(7, $this->calendarImpl->getPermissions());
-	}
-
-	public function testGetPermissionAll(): void {
-		$this->calendar->expects($this->once())
-			->method('getACL')
-			->with()
-			->willReturn([
-				['privilege' => '{DAV:}all', 'principal' => 'principal/users/foobar'],
-			]);
-
-		$this->assertEquals(31, $this->calendarImpl->getPermissions());
-	}
-
-	public function testHandleImipMessage(): void {
-		$message = <<<EOF
+    private Calendar&MockObject $calendar;
+    private array $calendarInfo;
+    private CalDavBackend&MockObject $backend;
+    private CalendarImpl $calendarImpl;
+    private array $mockExportCollection;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->calendar = $this->createMock(Calendar::class);
+        $this->calendarInfo = [
+            'id' => 1,
+            '{DAV:}displayname' => 'user readable name 123',
+            '{http://apple.com/ns/ical/}calendar-color' => '#AABBCC',
+            'uri' => '/this/is/a/uri',
+            'principaluri' => 'principal/users/foobar'
+        ];
+        $this->backend = $this->createMock(CalDavBackend::class);
+
+        $this->calendarImpl = new CalendarImpl(
+            $this->calendar,
+            $this->calendarInfo,
+            $this->backend
+        );
+    }
+
+
+    public function testGetKey(): void {
+        $this->assertEquals($this->calendarImpl->getKey(), 1);
+    }
+
+    public function testGetDisplayname(): void {
+        $this->assertEquals($this->calendarImpl->getDisplayName(), 'user readable name 123');
+    }
+
+    public function testGetDisplayColor(): void {
+        $this->assertEquals($this->calendarImpl->getDisplayColor(), '#AABBCC');
+    }
+
+    public function testSearch(): void {
+        $this->backend->expects($this->once())
+            ->method('search')
+            ->with($this->calendarInfo, 'abc', ['def'], ['ghi'], 42, 1337)
+            ->willReturn(['SEARCHRESULTS']);
+
+        $result = $this->calendarImpl->search('abc', ['def'], ['ghi'], 42, 1337);
+        $this->assertEquals($result, ['SEARCHRESULTS']);
+    }
+
+    public function testGetPermissionRead(): void {
+        $this->calendar->expects($this->once())
+            ->method('getACL')
+            ->with()
+            ->willReturn([
+                ['privilege' => '{DAV:}read', 'principal' => 'principal/users/foobar'],
+                ['privilege' => '{DAV:}read', 'principal' => 'principal/users/other'],
+                ['privilege' => '{DAV:}write', 'principal' => 'principal/users/other'],
+                ['privilege' => '{DAV:}all', 'principal' => 'principal/users/other'],
+            ]);
+
+        $this->assertEquals(1, $this->calendarImpl->getPermissions());
+    }
+
+    public function testGetPermissionWrite(): void {
+        $this->calendar->expects($this->once())
+            ->method('getACL')
+            ->with()
+            ->willReturn([
+                ['privilege' => '{DAV:}write', 'principal' => 'principal/users/foobar'],
+                ['privilege' => '{DAV:}read', 'principal' => 'principal/users/other'],
+                ['privilege' => '{DAV:}all', 'principal' => 'principal/users/other'],
+            ]);
+
+        $this->assertEquals(6, $this->calendarImpl->getPermissions());
+    }
+
+    public function testGetPermissionReadWrite(): void {
+        $this->calendar->expects($this->once())
+            ->method('getACL')
+            ->with()
+            ->willReturn([
+                ['privilege' => '{DAV:}write', 'principal' => 'principal/users/foobar'],
+                ['privilege' => '{DAV:}read', 'principal' => 'principal/users/foobar'],
+                ['privilege' => '{DAV:}all', 'principal' => 'principal/users/other'],
+            ]);
+
+        $this->assertEquals(7, $this->calendarImpl->getPermissions());
+    }
+
+    public function testGetPermissionAll(): void {
+        $this->calendar->expects($this->once())
+            ->method('getACL')
+            ->with()
+            ->willReturn([
+                ['privilege' => '{DAV:}all', 'principal' => 'principal/users/foobar'],
+            ]);
+
+        $this->assertEquals(31, $this->calendarImpl->getPermissions());
+    }
+
+    public function testHandleImipMessage(): void {
+        $message = <<<EOF
 BEGIN:VCALENDAR
 PRODID:-//Nextcloud/Nextcloud CalDAV Server//EN
 METHOD:REPLY
@@ -139,92 +139,92 @@ 
 END:VCALENDAR
 EOF;
 
-		/** @var CustomPrincipalPlugin|MockObject $authPlugin */
-		$authPlugin = $this->createMock(CustomPrincipalPlugin::class);
-		$authPlugin->expects(self::once())
-			->method('setCurrentPrincipal')
-			->with($this->calendar->getPrincipalURI());
-
-		/** @var \Sabre\DAVACL\Plugin|MockObject $aclPlugin */
-		$aclPlugin = $this->createMock(\Sabre\DAVACL\Plugin::class);
-
-		/** @var Plugin|MockObject $schedulingPlugin */
-		$schedulingPlugin = $this->createMock(Plugin::class);
-		$iTipMessage = $this->getITipMessage($message);
-		$iTipMessage->recipient = 'mailto:lewis@stardew-tent-living.com';
-
-		$server = $this->createMock(Server::class);
-		$server->expects($this->any())
-			->method('getPlugin')
-			->willReturnMap([
-				['auth', $authPlugin],
-				['acl', $aclPlugin],
-				['caldav-schedule', $schedulingPlugin]
-			]);
-		$server->expects(self::once())
-			->method('emit');
-
-		$invitationResponseServer = $this->createPartialMock(InvitationResponseServer::class, ['getServer', 'isExternalAttendee']);
-		$invitationResponseServer->server = $server;
-		$invitationResponseServer->expects($this->any())
-			->method('getServer')
-			->willReturn($server);
-		$invitationResponseServer->expects(self::once())
-			->method('isExternalAttendee')
-			->willReturn(false);
-
-		$calendarImpl = $this->getMockBuilder(CalendarImpl::class)
-			->setConstructorArgs([$this->calendar, $this->calendarInfo, $this->backend])
-			->onlyMethods(['getInvitationResponseServer'])
-			->getMock();
-		$calendarImpl->expects($this->once())
-			->method('getInvitationResponseServer')
-			->willReturn($invitationResponseServer);
-
-		$calendarImpl->handleIMipMessage('filename.ics', $message);
-	}
-
-	public function testHandleImipMessageNoCalendarUri(): void {
-		/** @var CustomPrincipalPlugin|MockObject $authPlugin */
-		$authPlugin = $this->createMock(CustomPrincipalPlugin::class);
-		$authPlugin->expects(self::once())
-			->method('setCurrentPrincipal')
-			->with($this->calendar->getPrincipalURI());
-		unset($this->calendarInfo['uri']);
-
-		/** @var Plugin|MockObject $schedulingPlugin */
-		$schedulingPlugin = $this->createMock(Plugin::class);
-
-		/** @var \Sabre\DAVACL\Plugin|MockObject $schedulingPlugin */
-		$aclPlugin = $this->createMock(\Sabre\DAVACL\Plugin::class);
-
-		$server =
-			$this->createMock(Server::class);
-		$server->expects($this->any())
-			->method('getPlugin')
-			->willReturnMap([
-				['auth', $authPlugin],
-				['acl', $aclPlugin],
-				['caldav-schedule', $schedulingPlugin]
-			]);
-		$server->expects(self::never())
-			->method('emit');
-
-		$invitationResponseServer = $this->createPartialMock(InvitationResponseServer::class, ['getServer']);
-		$invitationResponseServer->server = $server;
-		$invitationResponseServer->expects($this->any())
-			->method('getServer')
-			->willReturn($server);
-
-		$calendarImpl = $this->getMockBuilder(CalendarImpl::class)
-			->setConstructorArgs([$this->calendar, $this->calendarInfo, $this->backend])
-			->onlyMethods(['getInvitationResponseServer'])
-			->getMock();
-		$calendarImpl->expects($this->once())
-			->method('getInvitationResponseServer')
-			->willReturn($invitationResponseServer);
-
-		$message = <<<EOF
+        /** @var CustomPrincipalPlugin|MockObject $authPlugin */
+        $authPlugin = $this->createMock(CustomPrincipalPlugin::class);
+        $authPlugin->expects(self::once())
+            ->method('setCurrentPrincipal')
+            ->with($this->calendar->getPrincipalURI());
+
+        /** @var \Sabre\DAVACL\Plugin|MockObject $aclPlugin */
+        $aclPlugin = $this->createMock(\Sabre\DAVACL\Plugin::class);
+
+        /** @var Plugin|MockObject $schedulingPlugin */
+        $schedulingPlugin = $this->createMock(Plugin::class);
+        $iTipMessage = $this->getITipMessage($message);
+        $iTipMessage->recipient = 'mailto:lewis@stardew-tent-living.com';
+
+        $server = $this->createMock(Server::class);
+        $server->expects($this->any())
+            ->method('getPlugin')
+            ->willReturnMap([
+                ['auth', $authPlugin],
+                ['acl', $aclPlugin],
+                ['caldav-schedule', $schedulingPlugin]
+            ]);
+        $server->expects(self::once())
+            ->method('emit');
+
+        $invitationResponseServer = $this->createPartialMock(InvitationResponseServer::class, ['getServer', 'isExternalAttendee']);
+        $invitationResponseServer->server = $server;
+        $invitationResponseServer->expects($this->any())
+            ->method('getServer')
+            ->willReturn($server);
+        $invitationResponseServer->expects(self::once())
+            ->method('isExternalAttendee')
+            ->willReturn(false);
+
+        $calendarImpl = $this->getMockBuilder(CalendarImpl::class)
+            ->setConstructorArgs([$this->calendar, $this->calendarInfo, $this->backend])
+            ->onlyMethods(['getInvitationResponseServer'])
+            ->getMock();
+        $calendarImpl->expects($this->once())
+            ->method('getInvitationResponseServer')
+            ->willReturn($invitationResponseServer);
+
+        $calendarImpl->handleIMipMessage('filename.ics', $message);
+    }
+
+    public function testHandleImipMessageNoCalendarUri(): void {
+        /** @var CustomPrincipalPlugin|MockObject $authPlugin */
+        $authPlugin = $this->createMock(CustomPrincipalPlugin::class);
+        $authPlugin->expects(self::once())
+            ->method('setCurrentPrincipal')
+            ->with($this->calendar->getPrincipalURI());
+        unset($this->calendarInfo['uri']);
+
+        /** @var Plugin|MockObject $schedulingPlugin */
+        $schedulingPlugin = $this->createMock(Plugin::class);
+
+        /** @var \Sabre\DAVACL\Plugin|MockObject $schedulingPlugin */
+        $aclPlugin = $this->createMock(\Sabre\DAVACL\Plugin::class);
+
+        $server =
+            $this->createMock(Server::class);
+        $server->expects($this->any())
+            ->method('getPlugin')
+            ->willReturnMap([
+                ['auth', $authPlugin],
+                ['acl', $aclPlugin],
+                ['caldav-schedule', $schedulingPlugin]
+            ]);
+        $server->expects(self::never())
+            ->method('emit');
+
+        $invitationResponseServer = $this->createPartialMock(InvitationResponseServer::class, ['getServer']);
+        $invitationResponseServer->server = $server;
+        $invitationResponseServer->expects($this->any())
+            ->method('getServer')
+            ->willReturn($server);
+
+        $calendarImpl = $this->getMockBuilder(CalendarImpl::class)
+            ->setConstructorArgs([$this->calendar, $this->calendarInfo, $this->backend])
+            ->onlyMethods(['getInvitationResponseServer'])
+            ->getMock();
+        $calendarImpl->expects($this->once())
+            ->method('getInvitationResponseServer')
+            ->willReturn($invitationResponseServer);
+
+        $message = <<<EOF
 BEGIN:VCALENDAR
 PRODID:-//Nextcloud/Nextcloud CalDAV Server//EN
 METHOD:REPLY
@@ -239,70 +239,70 @@ 
 END:VCALENDAR
 EOF;
 
-		$this->expectException(CalendarException::class);
-		$calendarImpl->handleIMipMessage('filename.ics', $message);
-	}
-
-	private function getITipMessage($calendarData): Message {
-		$iTipMessage = new Message();
-		/** @var VCalendar $vObject */
-		$vObject = Reader::read($calendarData);
-		/** @var VEvent $vEvent */
-		$vEvent = $vObject->{'VEVENT'};
-		$orgaizer = $vEvent->{'ORGANIZER'}->getValue();
-		$attendee = $vEvent->{'ATTENDEE'}->getValue();
-
-		$iTipMessage->method = $vObject->{'METHOD'}->getValue();
-		$iTipMessage->recipient = $orgaizer;
-		$iTipMessage->sender = $attendee;
-		$iTipMessage->uid = isset($vEvent->{'UID'}) ? $vEvent->{'UID'}->getValue() : '';
-		$iTipMessage->component = 'VEVENT';
-		$iTipMessage->sequence = isset($vEvent->{'SEQUENCE'}) ? (int)$vEvent->{'SEQUENCE'}->getValue() : 0;
-		$iTipMessage->message = $vObject;
-		return $iTipMessage;
-	}
-
-	protected function mockExportGenerator(): Generator {
-		foreach ($this->mockExportCollection as $entry) {
-			yield $entry;
-		}
-	}
-
-	public function testExport(): void {
-		// Arrange
-		// construct calendar with a 1 hour event and same start/end time zones
-		$vCalendar = new VCalendar();
-		/** @var VEvent $vEvent */
-		$vEvent = $vCalendar->add('VEVENT', []);
-		$vEvent->UID->setValue('96a0e6b1-d886-4a55-a60d-152b31401dcc');
-		$vEvent->add('DTSTART', '20240701T080000', ['TZID' => 'America/Toronto']);
-		$vEvent->add('DTEND', '20240701T090000', ['TZID' => 'America/Toronto']);
-		$vEvent->add('SUMMARY', 'Test Recurrence Event');
-		$vEvent->add('ORGANIZER', 'mailto:organizer@testing.com', ['CN' => 'Organizer']);
-		$vEvent->add('ATTENDEE', 'mailto:attendee1@testing.com', [
-			'CN' => 'Attendee One',
-			'CUTYPE' => 'INDIVIDUAL',
-			'PARTSTAT' => 'NEEDS-ACTION',
-			'ROLE' => 'REQ-PARTICIPANT',
-			'RSVP' => 'TRUE'
-		]);
-		// construct data store return
-		$this->mockExportCollection[] = [
-			'id' => 1,
-			'calendardata' => $vCalendar->serialize()
-		];
-		$this->backend->expects($this->once())
-			->method('exportCalendar')
-			->with(1, $this->backend::CALENDAR_TYPE_CALENDAR, null)
-			->willReturn($this->mockExportGenerator());
-
-		// Act
-		foreach ($this->calendarImpl->export(null) as $entry) {
-			$exported[] = $entry;
-		}
-
-		// Assert
-		$this->assertCount(1, $exported, 'Invalid exported items count');
-	}
+        $this->expectException(CalendarException::class);
+        $calendarImpl->handleIMipMessage('filename.ics', $message);
+    }
+
+    private function getITipMessage($calendarData): Message {
+        $iTipMessage = new Message();
+        /** @var VCalendar $vObject */
+        $vObject = Reader::read($calendarData);
+        /** @var VEvent $vEvent */
+        $vEvent = $vObject->{'VEVENT'};
+        $orgaizer = $vEvent->{'ORGANIZER'}->getValue();
+        $attendee = $vEvent->{'ATTENDEE'}->getValue();
+
+        $iTipMessage->method = $vObject->{'METHOD'}->getValue();
+        $iTipMessage->recipient = $orgaizer;
+        $iTipMessage->sender = $attendee;
+        $iTipMessage->uid = isset($vEvent->{'UID'}) ? $vEvent->{'UID'}->getValue() : '';
+        $iTipMessage->component = 'VEVENT';
+        $iTipMessage->sequence = isset($vEvent->{'SEQUENCE'}) ? (int)$vEvent->{'SEQUENCE'}->getValue() : 0;
+        $iTipMessage->message = $vObject;
+        return $iTipMessage;
+    }
+
+    protected function mockExportGenerator(): Generator {
+        foreach ($this->mockExportCollection as $entry) {
+            yield $entry;
+        }
+    }
+
+    public function testExport(): void {
+        // Arrange
+        // construct calendar with a 1 hour event and same start/end time zones
+        $vCalendar = new VCalendar();
+        /** @var VEvent $vEvent */
+        $vEvent = $vCalendar->add('VEVENT', []);
+        $vEvent->UID->setValue('96a0e6b1-d886-4a55-a60d-152b31401dcc');
+        $vEvent->add('DTSTART', '20240701T080000', ['TZID' => 'America/Toronto']);
+        $vEvent->add('DTEND', '20240701T090000', ['TZID' => 'America/Toronto']);
+        $vEvent->add('SUMMARY', 'Test Recurrence Event');
+        $vEvent->add('ORGANIZER', 'mailto:organizer@testing.com', ['CN' => 'Organizer']);
+        $vEvent->add('ATTENDEE', 'mailto:attendee1@testing.com', [
+            'CN' => 'Attendee One',
+            'CUTYPE' => 'INDIVIDUAL',
+            'PARTSTAT' => 'NEEDS-ACTION',
+            'ROLE' => 'REQ-PARTICIPANT',
+            'RSVP' => 'TRUE'
+        ]);
+        // construct data store return
+        $this->mockExportCollection[] = [
+            'id' => 1,
+            'calendardata' => $vCalendar->serialize()
+        ];
+        $this->backend->expects($this->once())
+            ->method('exportCalendar')
+            ->with(1, $this->backend::CALENDAR_TYPE_CALENDAR, null)
+            ->willReturn($this->mockExportGenerator());
+
+        // Act
+        foreach ($this->calendarImpl->export(null) as $entry) {
+            $exported[] = $entry;
+        }
+
+        // Assert
+        $this->assertCount(1, $exported, 'Invalid exported items count');
+    }
 
 }

apps/dav/tests/unit/Avatars/AvatarNodeTest.php

Indentation +14 added lines, -14 removed lines

@@ -14,20 +14,20 @@
 use Test\TestCase;
 
 class AvatarNodeTest extends TestCase {
-	public function testGetName(): void {
-		/** @var IAvatar&MockObject $a */
-		$a = $this->createMock(IAvatar::class);
-		$n = new AvatarNode(1024, 'png', $a);
-		$this->assertEquals('1024.png', $n->getName());
-	}
+    public function testGetName(): void {
+        /** @var IAvatar&MockObject $a */
+        $a = $this->createMock(IAvatar::class);
+        $n = new AvatarNode(1024, 'png', $a);
+        $this->assertEquals('1024.png', $n->getName());
+    }
 
-	public function testGetContentType(): void {
-		/** @var IAvatar&MockObject $a */
-		$a = $this->createMock(IAvatar::class);
-		$n = new AvatarNode(1024, 'png', $a);
-		$this->assertEquals('image/png', $n->getContentType());
+    public function testGetContentType(): void {
+        /** @var IAvatar&MockObject $a */
+        $a = $this->createMock(IAvatar::class);
+        $n = new AvatarNode(1024, 'png', $a);
+        $this->assertEquals('image/png', $n->getContentType());
 
-		$n = new AvatarNode(1024, 'jpeg', $a);
-		$this->assertEquals('image/jpeg', $n->getContentType());
-	}
+        $n = new AvatarNode(1024, 'jpeg', $a);
+        $this->assertEquals('image/jpeg', $n->getContentType());
+    }
 }

apps/dav/tests/unit/Avatars/AvatarHomeTest.php

Indentation +84 added lines, -84 removed lines

@@ -18,88 +18,88 @@
 use Test\TestCase;
 
 class AvatarHomeTest extends TestCase {
-	private AvatarHome $home;
-	private IAvatarManager&MockObject $avatarManager;
-
-	protected function setUp(): void {
-		parent::setUp();
-		$this->avatarManager = $this->createMock(IAvatarManager::class);
-		$this->home = new AvatarHome(['uri' => 'principals/users/admin'], $this->avatarManager);
-	}
-
-	/**
-	 * @dataProvider providesForbiddenMethods
-	 */
-	public function testForbiddenMethods($method): void {
-		$this->expectException(\Sabre\DAV\Exception\Forbidden::class);
-
-		$this->home->$method('');
-	}
-
-	public static function providesForbiddenMethods(): array {
-		return [
-			['createFile'],
-			['createDirectory'],
-			['delete'],
-			['setName']
-		];
-	}
-
-	public function testGetName(): void {
-		$n = $this->home->getName();
-		self::assertEquals('admin', $n);
-	}
-
-	public static function providesTestGetChild(): array {
-		return [
-			[MethodNotAllowed::class, false, ''],
-			[MethodNotAllowed::class, false, 'bla.foo'],
-			[MethodNotAllowed::class, false, 'bla.png'],
-			[NotFound::class, false, '512.png'],
-			[null, true, '512.png'],
-		];
-	}
-
-	/**
-	 * @dataProvider providesTestGetChild
-	 */
-	public function testGetChild(?string $expectedException, bool $hasAvatar, string $path): void {
-		if ($expectedException !== null) {
-			$this->expectException($expectedException);
-		}
-
-		$avatar = $this->createMock(IAvatar::class);
-		$avatar->method('exists')->willReturn($hasAvatar);
-
-		$this->avatarManager->expects($this->any())->method('getAvatar')->with('admin')->willReturn($avatar);
-		$avatarNode = $this->home->getChild($path);
-		$this->assertInstanceOf(AvatarNode::class, $avatarNode);
-	}
-
-	public function testGetChildren(): void {
-		$avatarNodes = $this->home->getChildren();
-		self::assertEquals(0, count($avatarNodes));
-
-		$avatar = $this->createMock(IAvatar::class);
-		$avatar->expects($this->once())->method('exists')->willReturn(true);
-		$this->avatarManager->expects($this->any())->method('getAvatar')->with('admin')->willReturn($avatar);
-		$avatarNodes = $this->home->getChildren();
-		self::assertEquals(1, count($avatarNodes));
-	}
-
-	/**
-	 * @dataProvider providesTestGetChild
-	 */
-	public function testChildExists(?string $expectedException, bool $hasAvatar, string $path): void {
-		$avatar = $this->createMock(IAvatar::class);
-		$avatar->method('exists')->willReturn($hasAvatar);
-
-		$this->avatarManager->expects($this->any())->method('getAvatar')->with('admin')->willReturn($avatar);
-		$childExists = $this->home->childExists($path);
-		$this->assertEquals($hasAvatar, $childExists);
-	}
-
-	public function testGetLastModified(): void {
-		self::assertNull($this->home->getLastModified());
-	}
+    private AvatarHome $home;
+    private IAvatarManager&MockObject $avatarManager;
+
+    protected function setUp(): void {
+        parent::setUp();
+        $this->avatarManager = $this->createMock(IAvatarManager::class);
+        $this->home = new AvatarHome(['uri' => 'principals/users/admin'], $this->avatarManager);
+    }
+
+    /**
+     * @dataProvider providesForbiddenMethods
+     */
+    public function testForbiddenMethods($method): void {
+        $this->expectException(\Sabre\DAV\Exception\Forbidden::class);
+
+        $this->home->$method('');
+    }
+
+    public static function providesForbiddenMethods(): array {
+        return [
+            ['createFile'],
+            ['createDirectory'],
+            ['delete'],
+            ['setName']
+        ];
+    }
+
+    public function testGetName(): void {
+        $n = $this->home->getName();
+        self::assertEquals('admin', $n);
+    }
+
+    public static function providesTestGetChild(): array {
+        return [
+            [MethodNotAllowed::class, false, ''],
+            [MethodNotAllowed::class, false, 'bla.foo'],
+            [MethodNotAllowed::class, false, 'bla.png'],
+            [NotFound::class, false, '512.png'],
+            [null, true, '512.png'],
+        ];
+    }
+
+    /**
+     * @dataProvider providesTestGetChild
+     */
+    public function testGetChild(?string $expectedException, bool $hasAvatar, string $path): void {
+        if ($expectedException !== null) {
+            $this->expectException($expectedException);
+        }
+
+        $avatar = $this->createMock(IAvatar::class);
+        $avatar->method('exists')->willReturn($hasAvatar);
+
+        $this->avatarManager->expects($this->any())->method('getAvatar')->with('admin')->willReturn($avatar);
+        $avatarNode = $this->home->getChild($path);
+        $this->assertInstanceOf(AvatarNode::class, $avatarNode);
+    }
+
+    public function testGetChildren(): void {
+        $avatarNodes = $this->home->getChildren();
+        self::assertEquals(0, count($avatarNodes));
+
+        $avatar = $this->createMock(IAvatar::class);
+        $avatar->expects($this->once())->method('exists')->willReturn(true);
+        $this->avatarManager->expects($this->any())->method('getAvatar')->with('admin')->willReturn($avatar);
+        $avatarNodes = $this->home->getChildren();
+        self::assertEquals(1, count($avatarNodes));
+    }
+
+    /**
+     * @dataProvider providesTestGetChild
+     */
+    public function testChildExists(?string $expectedException, bool $hasAvatar, string $path): void {
+        $avatar = $this->createMock(IAvatar::class);
+        $avatar->method('exists')->willReturn($hasAvatar);
+
+        $this->avatarManager->expects($this->any())->method('getAvatar')->with('admin')->willReturn($avatar);
+        $childExists = $this->home->childExists($path);
+        $this->assertEquals($hasAvatar, $childExists);
+    }
+
+    public function testGetLastModified(): void {
+        self::assertNull($this->home->getLastModified());
+    }
 }

apps/dav/tests/unit/CalDAV/PublicCalendarTest.php

Indentation +95 added lines, -95 removed lines

@@ -16,55 +16,55 @@ 
 
 class PublicCalendarTest extends CalendarTest {
 
-	/**
-	 * @dataProvider providesConfidentialClassificationData
-	 */
-	public function testPrivateClassification(int $expectedChildren, bool $isShared): void {
-		$calObject0 = ['uri' => 'event-0', 'classification' => CalDavBackend::CLASSIFICATION_PUBLIC];
-		$calObject1 = ['uri' => 'event-1', 'classification' => CalDavBackend::CLASSIFICATION_CONFIDENTIAL];
-		$calObject2 = ['uri' => 'event-2', 'classification' => CalDavBackend::CLASSIFICATION_PRIVATE];
-
-		/** @var CalDavBackend&MockObject $backend */
-		$backend = $this->getMockBuilder(CalDavBackend::class)->disableOriginalConstructor()->getMock();
-		$backend->expects($this->any())->method('getCalendarObjects')->willReturn([
-			$calObject0, $calObject1, $calObject2
-		]);
-		$backend->expects($this->any())->method('getMultipleCalendarObjects')
-			->with(666, ['event-0', 'event-1', 'event-2'])
-			->willReturn([
-				$calObject0, $calObject1, $calObject2
-			]);
-		$backend->expects($this->any())->method('getCalendarObject')
-			->willReturn($calObject2)->with(666, 'event-2');
-		$backend->expects($this->any())->method('applyShareAcl')->willReturnArgument(1);
-
-		$calendarInfo = [
-			'{http://owncloud.org/ns}owner-principal' => 'user2',
-			'principaluri' => 'user2',
-			'id' => 666,
-			'uri' => 'cal',
-		];
-		/** @var IConfig&MockObject $config */
-		$config = $this->createMock(IConfig::class);
-		/** @var LoggerInterface&MockObject $logger */
-		$logger = $this->createMock(LoggerInterface::class);
-		$c = new PublicCalendar($backend, $calendarInfo, $this->l10n, $config, $logger);
-		$children = $c->getChildren();
-		$this->assertEquals(2, count($children));
-		$children = $c->getMultipleChildren(['event-0', 'event-1', 'event-2']);
-		$this->assertEquals(2, count($children));
-
-		$this->assertFalse($c->childExists('event-2'));
-	}
-
-	/**
-	 * @dataProvider providesConfidentialClassificationData
-	 */
-	public function testConfidentialClassification(int $expectedChildren, bool $isShared): void {
-		$start = '20160609';
-		$end = '20160610';
-
-		$calData = <<<EOD
+    /**
+     * @dataProvider providesConfidentialClassificationData
+     */
+    public function testPrivateClassification(int $expectedChildren, bool $isShared): void {
+        $calObject0 = ['uri' => 'event-0', 'classification' => CalDavBackend::CLASSIFICATION_PUBLIC];
+        $calObject1 = ['uri' => 'event-1', 'classification' => CalDavBackend::CLASSIFICATION_CONFIDENTIAL];
+        $calObject2 = ['uri' => 'event-2', 'classification' => CalDavBackend::CLASSIFICATION_PRIVATE];
+
+        /** @var CalDavBackend&MockObject $backend */
+        $backend = $this->getMockBuilder(CalDavBackend::class)->disableOriginalConstructor()->getMock();
+        $backend->expects($this->any())->method('getCalendarObjects')->willReturn([
+            $calObject0, $calObject1, $calObject2
+        ]);
+        $backend->expects($this->any())->method('getMultipleCalendarObjects')
+            ->with(666, ['event-0', 'event-1', 'event-2'])
+            ->willReturn([
+                $calObject0, $calObject1, $calObject2
+            ]);
+        $backend->expects($this->any())->method('getCalendarObject')
+            ->willReturn($calObject2)->with(666, 'event-2');
+        $backend->expects($this->any())->method('applyShareAcl')->willReturnArgument(1);
+
+        $calendarInfo = [
+            '{http://owncloud.org/ns}owner-principal' => 'user2',
+            'principaluri' => 'user2',
+            'id' => 666,
+            'uri' => 'cal',
+        ];
+        /** @var IConfig&MockObject $config */
+        $config = $this->createMock(IConfig::class);
+        /** @var LoggerInterface&MockObject $logger */
+        $logger = $this->createMock(LoggerInterface::class);
+        $c = new PublicCalendar($backend, $calendarInfo, $this->l10n, $config, $logger);
+        $children = $c->getChildren();
+        $this->assertEquals(2, count($children));
+        $children = $c->getMultipleChildren(['event-0', 'event-1', 'event-2']);
+        $this->assertEquals(2, count($children));
+
+        $this->assertFalse($c->childExists('event-2'));
+    }
+
+    /**
+     * @dataProvider providesConfidentialClassificationData
+     */
+    public function testConfidentialClassification(int $expectedChildren, bool $isShared): void {
+        $start = '20160609';
+        $end = '20160610';
+
+        $calData = <<<EOD
 BEGIN:VCALENDAR
 PRODID:-//ownCloud calendar v1.2.2
 BEGIN:VEVENT
@@ -106,50 +106,50 @@ 
 END:VCALENDAR
 EOD;
 
-		$calObject0 = ['uri' => 'event-0', 'classification' => CalDavBackend::CLASSIFICATION_PUBLIC];
-		$calObject1 = ['uri' => 'event-1', 'classification' => CalDavBackend::CLASSIFICATION_CONFIDENTIAL, 'calendardata' => $calData];
-		$calObject2 = ['uri' => 'event-2', 'classification' => CalDavBackend::CLASSIFICATION_PRIVATE];
-
-		/** @var CalDavBackend&MockObject $backend */
-		$backend = $this->getMockBuilder(CalDavBackend::class)->disableOriginalConstructor()->getMock();
-		$backend->expects($this->any())->method('getCalendarObjects')->willReturn([
-			$calObject0, $calObject1, $calObject2
-		]);
-		$backend->expects($this->any())->method('getMultipleCalendarObjects')
-			->with(666, ['event-0', 'event-1', 'event-2'])
-			->willReturn([
-				$calObject0, $calObject1, $calObject2
-			]);
-		$backend->expects($this->any())->method('getCalendarObject')
-			->willReturn($calObject1)->with(666, 'event-1');
-		$backend->expects($this->any())->method('applyShareAcl')->willReturnArgument(1);
-
-		$calendarInfo = [
-			'{http://owncloud.org/ns}owner-principal' => 'user1',
-			'principaluri' => 'user2',
-			'id' => 666,
-			'uri' => 'cal',
-		];
-		/** @var IConfig&MockObject $config */
-		$config = $this->createMock(IConfig::class);
-		/** @var LoggerInterface&MockObject $logger */
-		$logger = $this->createMock(LoggerInterface::class);
-		$c = new PublicCalendar($backend, $calendarInfo, $this->l10n, $config, $logger);
-
-		$this->assertEquals(count($c->getChildren()), 2);
-
-		// test private event
-		$privateEvent = $c->getChild('event-1');
-		$calData = $privateEvent->get();
-		$event = Reader::read($calData);
-
-		$this->assertEquals($start, $event->VEVENT->DTSTART->getValue());
-		$this->assertEquals($end, $event->VEVENT->DTEND->getValue());
-
-		$this->assertEquals('Busy', $event->VEVENT->SUMMARY->getValue());
-		$this->assertArrayNotHasKey('ATTENDEE', $event->VEVENT);
-		$this->assertArrayNotHasKey('LOCATION', $event->VEVENT);
-		$this->assertArrayNotHasKey('DESCRIPTION', $event->VEVENT);
-		$this->assertArrayNotHasKey('ORGANIZER', $event->VEVENT);
-	}
+        $calObject0 = ['uri' => 'event-0', 'classification' => CalDavBackend::CLASSIFICATION_PUBLIC];
+        $calObject1 = ['uri' => 'event-1', 'classification' => CalDavBackend::CLASSIFICATION_CONFIDENTIAL, 'calendardata' => $calData];
+        $calObject2 = ['uri' => 'event-2', 'classification' => CalDavBackend::CLASSIFICATION_PRIVATE];
+
+        /** @var CalDavBackend&MockObject $backend */
+        $backend = $this->getMockBuilder(CalDavBackend::class)->disableOriginalConstructor()->getMock();
+        $backend->expects($this->any())->method('getCalendarObjects')->willReturn([
+            $calObject0, $calObject1, $calObject2
+        ]);
+        $backend->expects($this->any())->method('getMultipleCalendarObjects')
+            ->with(666, ['event-0', 'event-1', 'event-2'])
+            ->willReturn([
+                $calObject0, $calObject1, $calObject2
+            ]);
+        $backend->expects($this->any())->method('getCalendarObject')
+            ->willReturn($calObject1)->with(666, 'event-1');
+        $backend->expects($this->any())->method('applyShareAcl')->willReturnArgument(1);
+
+        $calendarInfo = [
+            '{http://owncloud.org/ns}owner-principal' => 'user1',
+            'principaluri' => 'user2',
+            'id' => 666,
+            'uri' => 'cal',
+        ];
+        /** @var IConfig&MockObject $config */
+        $config = $this->createMock(IConfig::class);
+        /** @var LoggerInterface&MockObject $logger */
+        $logger = $this->createMock(LoggerInterface::class);
+        $c = new PublicCalendar($backend, $calendarInfo, $this->l10n, $config, $logger);
+
+        $this->assertEquals(count($c->getChildren()), 2);
+
+        // test private event
+        $privateEvent = $c->getChild('event-1');
+        $calData = $privateEvent->get();
+        $event = Reader::read($calData);
+
+        $this->assertEquals($start, $event->VEVENT->DTSTART->getValue());
+        $this->assertEquals($end, $event->VEVENT->DTEND->getValue());
+
+        $this->assertEquals('Busy', $event->VEVENT->SUMMARY->getValue());
+        $this->assertArrayNotHasKey('ATTENDEE', $event->VEVENT);
+        $this->assertArrayNotHasKey('LOCATION', $event->VEVENT);
+        $this->assertArrayNotHasKey('DESCRIPTION', $event->VEVENT);
+        $this->assertArrayNotHasKey('ORGANIZER', $event->VEVENT);
+    }
 }

apps/dav/tests/unit/CalDAV/TimezoneServiceTest.php

Indentation +112 added lines, -112 removed lines

@@ -21,122 +21,122 @@
 use Test\TestCase;
 
 class TimezoneServiceTest extends TestCase {
-	private IConfig&MockObject $config;
-	private PropertyMapper&MockObject $propertyMapper;
-	private IManager&MockObject $calendarManager;
-	private TimezoneService $service;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->config = $this->createMock(IConfig::class);
-		$this->propertyMapper = $this->createMock(PropertyMapper::class);
-		$this->calendarManager = $this->createMock(IManager::class);
-
-		$this->service = new TimezoneService(
-			$this->config,
-			$this->propertyMapper,
-			$this->calendarManager,
-		);
-	}
-
-	public function testGetUserTimezoneFromSettings(): void {
-		$this->config->expects(self::once())
-			->method('getUserValue')
-			->with('test123', 'core', 'timezone', '')
-			->willReturn('Europe/Warsaw');
-
-		$timezone = $this->service->getUserTimezone('test123');
-
-		self::assertSame('Europe/Warsaw', $timezone);
-	}
-
-	public function testGetUserTimezoneFromAvailability(): void {
-		$this->config->expects(self::once())
-			->method('getUserValue')
-			->with('test123', 'core', 'timezone', '')
-			->willReturn('');
-		$property = new Property();
-		$property->setPropertyvalue('BEGIN:VCALENDAR
+    private IConfig&MockObject $config;
+    private PropertyMapper&MockObject $propertyMapper;
+    private IManager&MockObject $calendarManager;
+    private TimezoneService $service;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->config = $this->createMock(IConfig::class);
+        $this->propertyMapper = $this->createMock(PropertyMapper::class);
+        $this->calendarManager = $this->createMock(IManager::class);
+
+        $this->service = new TimezoneService(
+            $this->config,
+            $this->propertyMapper,
+            $this->calendarManager,
+        );
+    }
+
+    public function testGetUserTimezoneFromSettings(): void {
+        $this->config->expects(self::once())
+            ->method('getUserValue')
+            ->with('test123', 'core', 'timezone', '')
+            ->willReturn('Europe/Warsaw');
+
+        $timezone = $this->service->getUserTimezone('test123');
+
+        self::assertSame('Europe/Warsaw', $timezone);
+    }
+
+    public function testGetUserTimezoneFromAvailability(): void {
+        $this->config->expects(self::once())
+            ->method('getUserValue')
+            ->with('test123', 'core', 'timezone', '')
+            ->willReturn('');
+        $property = new Property();
+        $property->setPropertyvalue('BEGIN:VCALENDAR
 PRODID:Nextcloud DAV app
 BEGIN:VTIMEZONE
 TZID:Europe/Vienna
 END:VTIMEZONE
 END:VCALENDAR');
-		$this->propertyMapper->expects(self::once())
-			->method('findPropertyByPathAndName')
-			->willReturn([
-				$property,
-			]);
-
-		$timezone = $this->service->getUserTimezone('test123');
-
-		self::assertNotNull($timezone);
-		self::assertEquals('Europe/Vienna', $timezone);
-	}
-
-	public function testGetUserTimezoneFromPersonalCalendar(): void {
-		$this->config->expects(self::exactly(2))
-			->method('getUserValue')
-			->willReturnMap([
-				['test123', 'core', 'timezone', '', ''],
-				['test123', 'dav', 'defaultCalendar', '', 'personal-1'],
-			]);
-		$other = $this->createMock(ICalendar::class);
-		$other->method('getUri')->willReturn('other');
-		$personal = $this->createMock(CalendarImpl::class);
-		$personal->method('getUri')->willReturn('personal-1');
-		$tz = new DateTimeZone('Europe/Berlin');
-		$vtz = $this->createMock(VTimeZone::class);
-		$vtz->method('getTimeZone')->willReturn($tz);
-		$personal->method('getSchedulingTimezone')->willReturn($vtz);
-		$this->calendarManager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->with('principals/users/test123')
-			->willReturn([
-				$other,
-				$personal,
-			]);
-
-		$timezone = $this->service->getUserTimezone('test123');
-
-		self::assertNotNull($timezone);
-		self::assertEquals('Europe/Berlin', $timezone);
-	}
-
-	public function testGetUserTimezoneFromAny(): void {
-		$this->config->expects(self::exactly(2))
-			->method('getUserValue')
-			->willReturnMap([
-				['test123', 'core', 'timezone', '', ''],
-				['test123', 'dav', 'defaultCalendar', '', 'personal-1'],
-			]);
-		$other = $this->createMock(ICalendar::class);
-		$other->method('getUri')->willReturn('other');
-		$personal = $this->createMock(CalendarImpl::class);
-		$personal->method('getUri')->willReturn('personal-2');
-		$tz = new DateTimeZone('Europe/Prague');
-		$vtz = $this->createMock(VTimeZone::class);
-		$vtz->method('getTimeZone')->willReturn($tz);
-		$personal->method('getSchedulingTimezone')->willReturn($vtz);
-		$this->calendarManager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->with('principals/users/test123')
-			->willReturn([
-				$other,
-				$personal,
-			]);
-
-		$timezone = $this->service->getUserTimezone('test123');
-
-		self::assertNotNull($timezone);
-		self::assertEquals('Europe/Prague', $timezone);
-	}
-
-	public function testGetUserTimezoneNoneFound(): void {
-		$timezone = $this->service->getUserTimezone('test123');
-
-		self::assertNull($timezone);
-	}
+        $this->propertyMapper->expects(self::once())
+            ->method('findPropertyByPathAndName')
+            ->willReturn([
+                $property,
+            ]);
+
+        $timezone = $this->service->getUserTimezone('test123');
+
+        self::assertNotNull($timezone);
+        self::assertEquals('Europe/Vienna', $timezone);
+    }
+
+    public function testGetUserTimezoneFromPersonalCalendar(): void {
+        $this->config->expects(self::exactly(2))
+            ->method('getUserValue')
+            ->willReturnMap([
+                ['test123', 'core', 'timezone', '', ''],
+                ['test123', 'dav', 'defaultCalendar', '', 'personal-1'],
+            ]);
+        $other = $this->createMock(ICalendar::class);
+        $other->method('getUri')->willReturn('other');
+        $personal = $this->createMock(CalendarImpl::class);
+        $personal->method('getUri')->willReturn('personal-1');
+        $tz = new DateTimeZone('Europe/Berlin');
+        $vtz = $this->createMock(VTimeZone::class);
+        $vtz->method('getTimeZone')->willReturn($tz);
+        $personal->method('getSchedulingTimezone')->willReturn($vtz);
+        $this->calendarManager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->with('principals/users/test123')
+            ->willReturn([
+                $other,
+                $personal,
+            ]);
+
+        $timezone = $this->service->getUserTimezone('test123');
+
+        self::assertNotNull($timezone);
+        self::assertEquals('Europe/Berlin', $timezone);
+    }
+
+    public function testGetUserTimezoneFromAny(): void {
+        $this->config->expects(self::exactly(2))
+            ->method('getUserValue')
+            ->willReturnMap([
+                ['test123', 'core', 'timezone', '', ''],
+                ['test123', 'dav', 'defaultCalendar', '', 'personal-1'],
+            ]);
+        $other = $this->createMock(ICalendar::class);
+        $other->method('getUri')->willReturn('other');
+        $personal = $this->createMock(CalendarImpl::class);
+        $personal->method('getUri')->willReturn('personal-2');
+        $tz = new DateTimeZone('Europe/Prague');
+        $vtz = $this->createMock(VTimeZone::class);
+        $vtz->method('getTimeZone')->willReturn($tz);
+        $personal->method('getSchedulingTimezone')->willReturn($vtz);
+        $this->calendarManager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->with('principals/users/test123')
+            ->willReturn([
+                $other,
+                $personal,
+            ]);
+
+        $timezone = $this->service->getUserTimezone('test123');
+
+        self::assertNotNull($timezone);
+        self::assertEquals('Europe/Prague', $timezone);
+    }
+
+    public function testGetUserTimezoneNoneFound(): void {
+        $timezone = $this->service->getUserTimezone('test123');
+
+        self::assertNull($timezone);
+    }
 
 }

apps/dav/tests/unit/CalDAV/Integration/ExternalCalendarTest.php

Indentation +88 added lines, -88 removed lines

@@ -12,92 +12,92 @@
 use Test\TestCase;
 
 class ExternalCalendarTest extends TestCase {
-	private ExternalCalendar&MockObject $abstractExternalCalendar;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->abstractExternalCalendar =
-			$this->getMockForAbstractClass(ExternalCalendar::class, ['example-app-id', 'calendar-uri-in-backend']);
-	}
-
-	public function testGetName():void {
-		// Check that the correct name is returned
-		$this->assertEquals('app-generated--example-app-id--calendar-uri-in-backend',
-			$this->abstractExternalCalendar->getName());
-
-		// Check that the method is final and can't be overridden by other classes
-		$reflectionMethod = new \ReflectionMethod(ExternalCalendar::class, 'getName');
-		$this->assertTrue($reflectionMethod->isFinal());
-	}
-
-	public function testSetName():void {
-		// Check that the method is final and can't be overridden by other classes
-		$reflectionMethod = new \ReflectionMethod(ExternalCalendar::class, 'setName');
-		$this->assertTrue($reflectionMethod->isFinal());
-
-		$this->expectException(\Sabre\DAV\Exception\MethodNotAllowed::class);
-		$this->expectExceptionMessage('Renaming calendars is not yet supported');
-
-		$this->abstractExternalCalendar->setName('other-name');
-	}
-
-	public function createDirectory(): void {
-		// Check that the method is final and can't be overridden by other classes
-		$reflectionMethod = new \ReflectionMethod(ExternalCalendar::class, 'createDirectory');
-		$this->assertTrue($reflectionMethod->isFinal());
-
-		$this->expectException(\Sabre\DAV\Exception\MethodNotAllowed::class);
-		$this->expectExceptionMessage('Creating collections in calendar objects is not allowed');
-
-		$this->abstractExternalCalendar->createDirectory('other-name');
-	}
-
-	public function testIsAppGeneratedCalendar():void {
-		$this->assertFalse(ExternalCalendar::isAppGeneratedCalendar('personal'));
-		$this->assertFalse(ExternalCalendar::isAppGeneratedCalendar('work'));
-		$this->assertFalse(ExternalCalendar::isAppGeneratedCalendar('contact_birthdays'));
-		$this->assertFalse(ExternalCalendar::isAppGeneratedCalendar('company'));
-		$this->assertFalse(ExternalCalendar::isAppGeneratedCalendar('app-generated'));
-		$this->assertFalse(ExternalCalendar::isAppGeneratedCalendar('app-generated--example'));
-
-		$this->assertTrue(ExternalCalendar::isAppGeneratedCalendar('app-generated--deck--board-1'));
-		$this->assertTrue(ExternalCalendar::isAppGeneratedCalendar('app-generated--example--foo-2'));
-		$this->assertTrue(ExternalCalendar::isAppGeneratedCalendar('app-generated--example--foo--2'));
-	}
-
-	/**
-	 * @dataProvider splitAppGeneratedCalendarUriDataProvider
-	 */
-	public function testSplitAppGeneratedCalendarUriInvalid(string $name):void {
-		$this->expectException(\InvalidArgumentException::class);
-		$this->expectExceptionMessage('Provided calendar uri was not app-generated');
-
-		ExternalCalendar::splitAppGeneratedCalendarUri($name);
-	}
-
-	public static function splitAppGeneratedCalendarUriDataProvider():array {
-		return [
-			['personal'],
-			['foo_shared_by_admin'],
-			['contact_birthdays'],
-		];
-	}
-
-	public function testSplitAppGeneratedCalendarUri():void {
-		$this->assertEquals(['deck', 'board-1'], ExternalCalendar::splitAppGeneratedCalendarUri('app-generated--deck--board-1'));
-		$this->assertEquals(['example', 'foo-2'], ExternalCalendar::splitAppGeneratedCalendarUri('app-generated--example--foo-2'));
-		$this->assertEquals(['example', 'foo--2'], ExternalCalendar::splitAppGeneratedCalendarUri('app-generated--example--foo--2'));
-	}
-
-	public function testDoesViolateReservedName():void {
-		$this->assertFalse(ExternalCalendar::doesViolateReservedName('personal'));
-		$this->assertFalse(ExternalCalendar::doesViolateReservedName('work'));
-		$this->assertFalse(ExternalCalendar::doesViolateReservedName('contact_birthdays'));
-		$this->assertFalse(ExternalCalendar::doesViolateReservedName('company'));
-
-		$this->assertTrue(ExternalCalendar::doesViolateReservedName('app-generated'));
-		$this->assertTrue(ExternalCalendar::doesViolateReservedName('app-generated-calendar'));
-		$this->assertTrue(ExternalCalendar::doesViolateReservedName('app-generated--deck-123'));
-	}
+    private ExternalCalendar&MockObject $abstractExternalCalendar;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->abstractExternalCalendar =
+            $this->getMockForAbstractClass(ExternalCalendar::class, ['example-app-id', 'calendar-uri-in-backend']);
+    }
+
+    public function testGetName():void {
+        // Check that the correct name is returned
+        $this->assertEquals('app-generated--example-app-id--calendar-uri-in-backend',
+            $this->abstractExternalCalendar->getName());
+
+        // Check that the method is final and can't be overridden by other classes
+        $reflectionMethod = new \ReflectionMethod(ExternalCalendar::class, 'getName');
+        $this->assertTrue($reflectionMethod->isFinal());
+    }
+
+    public function testSetName():void {
+        // Check that the method is final and can't be overridden by other classes
+        $reflectionMethod = new \ReflectionMethod(ExternalCalendar::class, 'setName');
+        $this->assertTrue($reflectionMethod->isFinal());
+
+        $this->expectException(\Sabre\DAV\Exception\MethodNotAllowed::class);
+        $this->expectExceptionMessage('Renaming calendars is not yet supported');
+
+        $this->abstractExternalCalendar->setName('other-name');
+    }
+
+    public function createDirectory(): void {
+        // Check that the method is final and can't be overridden by other classes
+        $reflectionMethod = new \ReflectionMethod(ExternalCalendar::class, 'createDirectory');
+        $this->assertTrue($reflectionMethod->isFinal());
+
+        $this->expectException(\Sabre\DAV\Exception\MethodNotAllowed::class);
+        $this->expectExceptionMessage('Creating collections in calendar objects is not allowed');
+
+        $this->abstractExternalCalendar->createDirectory('other-name');
+    }
+
+    public function testIsAppGeneratedCalendar():void {
+        $this->assertFalse(ExternalCalendar::isAppGeneratedCalendar('personal'));
+        $this->assertFalse(ExternalCalendar::isAppGeneratedCalendar('work'));
+        $this->assertFalse(ExternalCalendar::isAppGeneratedCalendar('contact_birthdays'));
+        $this->assertFalse(ExternalCalendar::isAppGeneratedCalendar('company'));
+        $this->assertFalse(ExternalCalendar::isAppGeneratedCalendar('app-generated'));
+        $this->assertFalse(ExternalCalendar::isAppGeneratedCalendar('app-generated--example'));
+
+        $this->assertTrue(ExternalCalendar::isAppGeneratedCalendar('app-generated--deck--board-1'));
+        $this->assertTrue(ExternalCalendar::isAppGeneratedCalendar('app-generated--example--foo-2'));
+        $this->assertTrue(ExternalCalendar::isAppGeneratedCalendar('app-generated--example--foo--2'));
+    }
+
+    /**
+     * @dataProvider splitAppGeneratedCalendarUriDataProvider
+     */
+    public function testSplitAppGeneratedCalendarUriInvalid(string $name):void {
+        $this->expectException(\InvalidArgumentException::class);
+        $this->expectExceptionMessage('Provided calendar uri was not app-generated');
+
+        ExternalCalendar::splitAppGeneratedCalendarUri($name);
+    }
+
+    public static function splitAppGeneratedCalendarUriDataProvider():array {
+        return [
+            ['personal'],
+            ['foo_shared_by_admin'],
+            ['contact_birthdays'],
+        ];
+    }
+
+    public function testSplitAppGeneratedCalendarUri():void {
+        $this->assertEquals(['deck', 'board-1'], ExternalCalendar::splitAppGeneratedCalendarUri('app-generated--deck--board-1'));
+        $this->assertEquals(['example', 'foo-2'], ExternalCalendar::splitAppGeneratedCalendarUri('app-generated--example--foo-2'));
+        $this->assertEquals(['example', 'foo--2'], ExternalCalendar::splitAppGeneratedCalendarUri('app-generated--example--foo--2'));
+    }
+
+    public function testDoesViolateReservedName():void {
+        $this->assertFalse(ExternalCalendar::doesViolateReservedName('personal'));
+        $this->assertFalse(ExternalCalendar::doesViolateReservedName('work'));
+        $this->assertFalse(ExternalCalendar::doesViolateReservedName('contact_birthdays'));
+        $this->assertFalse(ExternalCalendar::doesViolateReservedName('company'));
+
+        $this->assertTrue(ExternalCalendar::doesViolateReservedName('app-generated'));
+        $this->assertTrue(ExternalCalendar::doesViolateReservedName('app-generated-calendar'));
+        $this->assertTrue(ExternalCalendar::doesViolateReservedName('app-generated--deck-123'));
+    }
 }

apps/dav/tests/unit/CalDAV/Status/StatusServiceTest.php

Indentation +415 added lines, -415 removed lines

@@ -27,419 +27,419 @@
 use Test\TestCase;
 
 class StatusServiceTest extends TestCase {
-	private ITimeFactory&MockObject $timeFactory;
-	private IManager&MockObject $calendarManager;
-	private IUserManager&MockObject $userManager;
-	private UserStatusService&MockObject $userStatusService;
-	private IAvailabilityCoordinator&MockObject $availabilityCoordinator;
-	private ICacheFactory&MockObject $cacheFactory;
-	private LoggerInterface&MockObject $logger;
-	private ICache&MockObject $cache;
-	private StatusService $service;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->timeFactory = $this->createMock(ITimeFactory::class);
-		$this->calendarManager = $this->createMock(IManager::class);
-		$this->userManager = $this->createMock(IUserManager::class);
-		$this->userStatusService = $this->createMock(UserStatusService::class);
-		$this->availabilityCoordinator = $this->createMock(IAvailabilityCoordinator::class);
-		$this->cacheFactory = $this->createMock(ICacheFactory::class);
-		$this->logger = $this->createMock(LoggerInterface::class);
-		$this->cache = $this->createMock(ICache::class);
-		$this->cacheFactory->expects(self::once())
-			->method('createLocal')
-			->with('CalendarStatusService')
-			->willReturn($this->cache);
-
-		$this->service = new StatusService($this->timeFactory,
-			$this->calendarManager,
-			$this->userManager,
-			$this->userStatusService,
-			$this->availabilityCoordinator,
-			$this->cacheFactory,
-			$this->logger,
-		);
-	}
-
-	public function testNoUser(): void {
-		$this->userManager->expects(self::once())
-			->method('get')
-			->willReturn(null);
-		$this->availabilityCoordinator->expects(self::never())
-			->method('getCurrentOutOfOfficeData');
-		$this->availabilityCoordinator->expects(self::never())
-			->method('isInEffect');
-		$this->logger->expects(self::never())
-			->method('debug');
-		$this->cache->expects(self::never())
-			->method('get');
-		$this->cache->expects(self::never())
-			->method('set');
-		$this->calendarManager->expects(self::never())
-			->method('getCalendarsForPrincipal');
-		$this->calendarManager->expects(self::never())
-			->method('newQuery');
-		$this->timeFactory->expects(self::never())
-			->method('getDateTime');
-		$this->calendarManager->expects(self::never())
-			->method('searchForPrincipal');
-		$this->userStatusService->expects(self::never())
-			->method('revertUserStatus');
-		$this->userStatusService->expects(self::never())
-			->method('setUserStatus');
-		$this->userStatusService->expects(self::never())
-			->method('findByUserId');
-
-		$this->service->processCalendarStatus('admin');
-	}
-
-	public function testOOOInEffect(): void {
-		$user = $this->createConfiguredMock(IUser::class, [
-			'getUID' => 'admin',
-		]);
-
-		$this->userManager->expects(self::once())
-			->method('get')
-			->willReturn($user);
-		$this->availabilityCoordinator->expects(self::once())
-			->method('getCurrentOutOfOfficeData')
-			->willReturn($this->createMock(IOutOfOfficeData::class));
-		$this->availabilityCoordinator->expects(self::once())
-			->method('isInEffect')
-			->willReturn(true);
-		$this->logger->expects(self::once())
-			->method('debug');
-		$this->cache->expects(self::never())
-			->method('get');
-		$this->cache->expects(self::never())
-			->method('set');
-		$this->calendarManager->expects(self::never())
-			->method('getCalendarsForPrincipal');
-		$this->calendarManager->expects(self::never())
-			->method('newQuery');
-		$this->timeFactory->expects(self::never())
-			->method('getDateTime');
-		$this->calendarManager->expects(self::never())
-			->method('searchForPrincipal');
-		$this->userStatusService->expects(self::never())
-			->method('revertUserStatus');
-		$this->userStatusService->expects(self::never())
-			->method('setUserStatus');
-		$this->userStatusService->expects(self::never())
-			->method('findByUserId');
-
-		$this->service->processCalendarStatus('admin');
-	}
-
-	public function testNoCalendars(): void {
-		$user = $this->createConfiguredMock(IUser::class, [
-			'getUID' => 'admin',
-		]);
-
-		$this->userManager->expects(self::once())
-			->method('get')
-			->willReturn($user);
-		$this->availabilityCoordinator->expects(self::once())
-			->method('getCurrentOutOfOfficeData')
-			->willReturn(null);
-		$this->availabilityCoordinator->expects(self::never())
-			->method('isInEffect');
-		$this->cache->expects(self::once())
-			->method('get')
-			->willReturn(null);
-		$this->cache->expects(self::once())
-			->method('set');
-		$this->calendarManager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([]);
-		$this->calendarManager->expects(self::never())
-			->method('newQuery');
-		$this->timeFactory->expects(self::never())
-			->method('getDateTime');
-		$this->calendarManager->expects(self::never())
-			->method('searchForPrincipal');
-		$this->userStatusService->expects(self::once())
-			->method('revertUserStatus');
-		$this->logger->expects(self::once())
-			->method('debug');
-		$this->userStatusService->expects(self::never())
-			->method('setUserStatus');
-		$this->userStatusService->expects(self::never())
-			->method('findByUserId');
-
-		$this->service->processCalendarStatus('admin');
-	}
-
-	public function testNoCalendarEvents(): void {
-		$user = $this->createConfiguredMock(IUser::class, [
-			'getUID' => 'admin',
-		]);
-
-		$this->userManager->expects(self::once())
-			->method('get')
-			->willReturn($user);
-		$this->availabilityCoordinator->expects(self::once())
-			->method('getCurrentOutOfOfficeData')
-			->willReturn(null);
-		$this->availabilityCoordinator->expects(self::never())
-			->method('isInEffect');
-		$this->cache->expects(self::once())
-			->method('get')
-			->willReturn(null);
-		$this->cache->expects(self::once())
-			->method('set');
-		$this->calendarManager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$this->createMock(CalendarImpl::class)]);
-		$this->calendarManager->expects(self::once())
-			->method('newQuery')
-			->willReturn(new CalendarQuery('admin'));
-		$this->timeFactory->expects(self::exactly(2))
-			->method('getDateTime')
-			->willReturn(new \DateTime());
-		$this->calendarManager->expects(self::once())
-			->method('searchForPrincipal')
-			->willReturn([]);
-		$this->userStatusService->expects(self::once())
-			->method('revertUserStatus');
-		$this->logger->expects(self::once())
-			->method('debug');
-		$this->userStatusService->expects(self::never())
-			->method('setUserStatus');
-		$this->userStatusService->expects(self::never())
-			->method('findByUserId');
-
-		$this->service->processCalendarStatus('admin');
-	}
-
-	public function testCalendarNoEventObjects(): void {
-		$user = $this->createConfiguredMock(IUser::class, [
-			'getUID' => 'admin',
-		]);
-
-		$this->userManager->expects(self::once())
-			->method('get')
-			->willReturn($user);
-		$this->availabilityCoordinator->expects(self::once())
-			->method('getCurrentOutOfOfficeData')
-			->willReturn(null);
-		$this->availabilityCoordinator->expects(self::never())
-			->method('isInEffect');
-		$this->cache->expects(self::once())
-			->method('get')
-			->willReturn(null);
-		$this->cache->expects(self::once())
-			->method('set');
-		$this->calendarManager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$this->createMock(CalendarImpl::class)]);
-		$this->calendarManager->expects(self::once())
-			->method('newQuery')
-			->willReturn(new CalendarQuery('admin'));
-		$this->timeFactory->expects(self::exactly(2))
-			->method('getDateTime')
-			->willReturn(new \DateTime());
-		$this->userStatusService->expects(self::once())
-			->method('findByUserId')
-			->willThrowException(new DoesNotExistException(''));
-		$this->calendarManager->expects(self::once())
-			->method('searchForPrincipal')
-			->willReturn([['objects' => []]]);
-		$this->userStatusService->expects(self::once())
-			->method('revertUserStatus');
-		$this->logger->expects(self::once())
-			->method('debug');
-		$this->userStatusService->expects(self::never())
-			->method('setUserStatus');
-
-
-		$this->service->processCalendarStatus('admin');
-	}
-
-	public function testCalendarEvent(): void {
-		$user = $this->createConfiguredMock(IUser::class, [
-			'getUID' => 'admin',
-		]);
-
-		$this->userManager->expects(self::once())
-			->method('get')
-			->willReturn($user);
-		$this->availabilityCoordinator->expects(self::once())
-			->method('getCurrentOutOfOfficeData')
-			->willReturn(null);
-		$this->availabilityCoordinator->expects(self::never())
-			->method('isInEffect');
-		$this->cache->expects(self::once())
-			->method('get')
-			->willReturn(null);
-		$this->cache->expects(self::once())
-			->method('set');
-		$this->calendarManager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$this->createMock(CalendarImpl::class)]);
-		$this->calendarManager->expects(self::once())
-			->method('newQuery')
-			->willReturn(new CalendarQuery('admin'));
-		$this->timeFactory->expects(self::exactly(2))
-			->method('getDateTime')
-			->willReturn(new \DateTime());
-		$this->userStatusService->expects(self::once())
-			->method('findByUserId')
-			->willThrowException(new DoesNotExistException(''));
-		$this->calendarManager->expects(self::once())
-			->method('searchForPrincipal')
-			->willReturn([['objects' => [[]]]]);
-		$this->userStatusService->expects(self::never())
-			->method('revertUserStatus');
-		$this->logger->expects(self::once())
-			->method('debug');
-		$this->userStatusService->expects(self::once())
-			->method('setUserStatus');
-
-
-		$this->service->processCalendarStatus('admin');
-	}
-
-	public function testCallStatus(): void {
-		$user = $this->createConfiguredMock(IUser::class, [
-			'getUID' => 'admin',
-		]);
-
-		$this->userManager->expects(self::once())
-			->method('get')
-			->willReturn($user);
-		$this->availabilityCoordinator->expects(self::once())
-			->method('getCurrentOutOfOfficeData')
-			->willReturn(null);
-		$this->availabilityCoordinator->expects(self::never())
-			->method('isInEffect');
-		$this->cache->expects(self::once())
-			->method('get')
-			->willReturn(null);
-		$this->cache->expects(self::once())
-			->method('set');
-		$this->calendarManager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$this->createMock(CalendarImpl::class)]);
-		$this->calendarManager->expects(self::once())
-			->method('newQuery')
-			->willReturn(new CalendarQuery('admin'));
-		$this->timeFactory->expects(self::exactly(2))
-			->method('getDateTime')
-			->willReturn(new \DateTime());
-		$this->calendarManager->expects(self::once())
-			->method('searchForPrincipal')
-			->willReturn([['objects' => [[]]]]);
-		$userStatus = new UserStatus();
-		$userStatus->setMessageId(IUserStatus::MESSAGE_CALL);
-		$userStatus->setStatusTimestamp(123456);
-		$this->userStatusService->expects(self::once())
-			->method('findByUserId')
-			->willReturn($userStatus);
-		$this->logger->expects(self::once())
-			->method('debug');
-		$this->userStatusService->expects(self::never())
-			->method('revertUserStatus');
-		$this->userStatusService->expects(self::never())
-			->method('setUserStatus');
-
-
-		$this->service->processCalendarStatus('admin');
-	}
-
-	public function testInvisibleStatus(): void {
-		$user = $this->createConfiguredMock(IUser::class, [
-			'getUID' => 'admin',
-		]);
-
-		$this->userManager->expects(self::once())
-			->method('get')
-			->willReturn($user);
-		$this->availabilityCoordinator->expects(self::once())
-			->method('getCurrentOutOfOfficeData')
-			->willReturn(null);
-		$this->availabilityCoordinator->expects(self::never())
-			->method('isInEffect');
-		$this->cache->expects(self::once())
-			->method('get')
-			->willReturn(null);
-		$this->cache->expects(self::once())
-			->method('set');
-		$this->calendarManager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$this->createMock(CalendarImpl::class)]);
-		$this->calendarManager->expects(self::once())
-			->method('newQuery')
-			->willReturn(new CalendarQuery('admin'));
-		$this->timeFactory->expects(self::exactly(2))
-			->method('getDateTime')
-			->willReturn(new \DateTime());
-		$this->calendarManager->expects(self::once())
-			->method('searchForPrincipal')
-			->willReturn([['objects' => [[]]]]);
-		$userStatus = new UserStatus();
-		$userStatus->setStatus(IUserStatus::INVISIBLE);
-		$userStatus->setStatusTimestamp(123456);
-		$this->userStatusService->expects(self::once())
-			->method('findByUserId')
-			->willReturn($userStatus);
-		$this->logger->expects(self::once())
-			->method('debug');
-		$this->userStatusService->expects(self::never())
-			->method('revertUserStatus');
-		$this->userStatusService->expects(self::never())
-			->method('setUserStatus');
-
-
-		$this->service->processCalendarStatus('admin');
-	}
-
-	public function testDNDStatus(): void {
-		$user = $this->createConfiguredMock(IUser::class, [
-			'getUID' => 'admin',
-		]);
-
-		$this->userManager->expects(self::once())
-			->method('get')
-			->willReturn($user);
-		$this->availabilityCoordinator->expects(self::once())
-			->method('getCurrentOutOfOfficeData')
-			->willReturn(null);
-		$this->availabilityCoordinator->expects(self::never())
-			->method('isInEffect');
-		$this->cache->expects(self::once())
-			->method('get')
-			->willReturn(null);
-		$this->cache->expects(self::once())
-			->method('set');
-		$this->calendarManager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$this->createMock(CalendarImpl::class)]);
-		$this->calendarManager->expects(self::once())
-			->method('newQuery')
-			->willReturn(new CalendarQuery('admin'));
-		$this->timeFactory->expects(self::exactly(2))
-			->method('getDateTime')
-			->willReturn(new \DateTime());
-		$this->calendarManager->expects(self::once())
-			->method('searchForPrincipal')
-			->willReturn([['objects' => [[]]]]);
-		$userStatus = new UserStatus();
-		$userStatus->setStatus(IUserStatus::DND);
-		$userStatus->setStatusTimestamp(123456);
-		$this->userStatusService->expects(self::once())
-			->method('findByUserId')
-			->willReturn($userStatus);
-		$this->logger->expects(self::once())
-			->method('debug');
-		$this->userStatusService->expects(self::never())
-			->method('revertUserStatus');
-		$this->userStatusService->expects(self::never())
-			->method('setUserStatus');
-
-
-		$this->service->processCalendarStatus('admin');
-	}
+    private ITimeFactory&MockObject $timeFactory;
+    private IManager&MockObject $calendarManager;
+    private IUserManager&MockObject $userManager;
+    private UserStatusService&MockObject $userStatusService;
+    private IAvailabilityCoordinator&MockObject $availabilityCoordinator;
+    private ICacheFactory&MockObject $cacheFactory;
+    private LoggerInterface&MockObject $logger;
+    private ICache&MockObject $cache;
+    private StatusService $service;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->timeFactory = $this->createMock(ITimeFactory::class);
+        $this->calendarManager = $this->createMock(IManager::class);
+        $this->userManager = $this->createMock(IUserManager::class);
+        $this->userStatusService = $this->createMock(UserStatusService::class);
+        $this->availabilityCoordinator = $this->createMock(IAvailabilityCoordinator::class);
+        $this->cacheFactory = $this->createMock(ICacheFactory::class);
+        $this->logger = $this->createMock(LoggerInterface::class);
+        $this->cache = $this->createMock(ICache::class);
+        $this->cacheFactory->expects(self::once())
+            ->method('createLocal')
+            ->with('CalendarStatusService')
+            ->willReturn($this->cache);
+
+        $this->service = new StatusService($this->timeFactory,
+            $this->calendarManager,
+            $this->userManager,
+            $this->userStatusService,
+            $this->availabilityCoordinator,
+            $this->cacheFactory,
+            $this->logger,
+        );
+    }
+
+    public function testNoUser(): void {
+        $this->userManager->expects(self::once())
+            ->method('get')
+            ->willReturn(null);
+        $this->availabilityCoordinator->expects(self::never())
+            ->method('getCurrentOutOfOfficeData');
+        $this->availabilityCoordinator->expects(self::never())
+            ->method('isInEffect');
+        $this->logger->expects(self::never())
+            ->method('debug');
+        $this->cache->expects(self::never())
+            ->method('get');
+        $this->cache->expects(self::never())
+            ->method('set');
+        $this->calendarManager->expects(self::never())
+            ->method('getCalendarsForPrincipal');
+        $this->calendarManager->expects(self::never())
+            ->method('newQuery');
+        $this->timeFactory->expects(self::never())
+            ->method('getDateTime');
+        $this->calendarManager->expects(self::never())
+            ->method('searchForPrincipal');
+        $this->userStatusService->expects(self::never())
+            ->method('revertUserStatus');
+        $this->userStatusService->expects(self::never())
+            ->method('setUserStatus');
+        $this->userStatusService->expects(self::never())
+            ->method('findByUserId');
+
+        $this->service->processCalendarStatus('admin');
+    }
+
+    public function testOOOInEffect(): void {
+        $user = $this->createConfiguredMock(IUser::class, [
+            'getUID' => 'admin',
+        ]);
+
+        $this->userManager->expects(self::once())
+            ->method('get')
+            ->willReturn($user);
+        $this->availabilityCoordinator->expects(self::once())
+            ->method('getCurrentOutOfOfficeData')
+            ->willReturn($this->createMock(IOutOfOfficeData::class));
+        $this->availabilityCoordinator->expects(self::once())
+            ->method('isInEffect')
+            ->willReturn(true);
+        $this->logger->expects(self::once())
+            ->method('debug');
+        $this->cache->expects(self::never())
+            ->method('get');
+        $this->cache->expects(self::never())
+            ->method('set');
+        $this->calendarManager->expects(self::never())
+            ->method('getCalendarsForPrincipal');
+        $this->calendarManager->expects(self::never())
+            ->method('newQuery');
+        $this->timeFactory->expects(self::never())
+            ->method('getDateTime');
+        $this->calendarManager->expects(self::never())
+            ->method('searchForPrincipal');
+        $this->userStatusService->expects(self::never())
+            ->method('revertUserStatus');
+        $this->userStatusService->expects(self::never())
+            ->method('setUserStatus');
+        $this->userStatusService->expects(self::never())
+            ->method('findByUserId');
+
+        $this->service->processCalendarStatus('admin');
+    }
+
+    public function testNoCalendars(): void {
+        $user = $this->createConfiguredMock(IUser::class, [
+            'getUID' => 'admin',
+        ]);
+
+        $this->userManager->expects(self::once())
+            ->method('get')
+            ->willReturn($user);
+        $this->availabilityCoordinator->expects(self::once())
+            ->method('getCurrentOutOfOfficeData')
+            ->willReturn(null);
+        $this->availabilityCoordinator->expects(self::never())
+            ->method('isInEffect');
+        $this->cache->expects(self::once())
+            ->method('get')
+            ->willReturn(null);
+        $this->cache->expects(self::once())
+            ->method('set');
+        $this->calendarManager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([]);
+        $this->calendarManager->expects(self::never())
+            ->method('newQuery');
+        $this->timeFactory->expects(self::never())
+            ->method('getDateTime');
+        $this->calendarManager->expects(self::never())
+            ->method('searchForPrincipal');
+        $this->userStatusService->expects(self::once())
+            ->method('revertUserStatus');
+        $this->logger->expects(self::once())
+            ->method('debug');
+        $this->userStatusService->expects(self::never())
+            ->method('setUserStatus');
+        $this->userStatusService->expects(self::never())
+            ->method('findByUserId');
+
+        $this->service->processCalendarStatus('admin');
+    }
+
+    public function testNoCalendarEvents(): void {
+        $user = $this->createConfiguredMock(IUser::class, [
+            'getUID' => 'admin',
+        ]);
+
+        $this->userManager->expects(self::once())
+            ->method('get')
+            ->willReturn($user);
+        $this->availabilityCoordinator->expects(self::once())
+            ->method('getCurrentOutOfOfficeData')
+            ->willReturn(null);
+        $this->availabilityCoordinator->expects(self::never())
+            ->method('isInEffect');
+        $this->cache->expects(self::once())
+            ->method('get')
+            ->willReturn(null);
+        $this->cache->expects(self::once())
+            ->method('set');
+        $this->calendarManager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$this->createMock(CalendarImpl::class)]);
+        $this->calendarManager->expects(self::once())
+            ->method('newQuery')
+            ->willReturn(new CalendarQuery('admin'));
+        $this->timeFactory->expects(self::exactly(2))
+            ->method('getDateTime')
+            ->willReturn(new \DateTime());
+        $this->calendarManager->expects(self::once())
+            ->method('searchForPrincipal')
+            ->willReturn([]);
+        $this->userStatusService->expects(self::once())
+            ->method('revertUserStatus');
+        $this->logger->expects(self::once())
+            ->method('debug');
+        $this->userStatusService->expects(self::never())
+            ->method('setUserStatus');
+        $this->userStatusService->expects(self::never())
+            ->method('findByUserId');
+
+        $this->service->processCalendarStatus('admin');
+    }
+
+    public function testCalendarNoEventObjects(): void {
+        $user = $this->createConfiguredMock(IUser::class, [
+            'getUID' => 'admin',
+        ]);
+
+        $this->userManager->expects(self::once())
+            ->method('get')
+            ->willReturn($user);
+        $this->availabilityCoordinator->expects(self::once())
+            ->method('getCurrentOutOfOfficeData')
+            ->willReturn(null);
+        $this->availabilityCoordinator->expects(self::never())
+            ->method('isInEffect');
+        $this->cache->expects(self::once())
+            ->method('get')
+            ->willReturn(null);
+        $this->cache->expects(self::once())
+            ->method('set');
+        $this->calendarManager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$this->createMock(CalendarImpl::class)]);
+        $this->calendarManager->expects(self::once())
+            ->method('newQuery')
+            ->willReturn(new CalendarQuery('admin'));
+        $this->timeFactory->expects(self::exactly(2))
+            ->method('getDateTime')
+            ->willReturn(new \DateTime());
+        $this->userStatusService->expects(self::once())
+            ->method('findByUserId')
+            ->willThrowException(new DoesNotExistException(''));
+        $this->calendarManager->expects(self::once())
+            ->method('searchForPrincipal')
+            ->willReturn([['objects' => []]]);
+        $this->userStatusService->expects(self::once())
+            ->method('revertUserStatus');
+        $this->logger->expects(self::once())
+            ->method('debug');
+        $this->userStatusService->expects(self::never())
+            ->method('setUserStatus');
+
+
+        $this->service->processCalendarStatus('admin');
+    }
+
+    public function testCalendarEvent(): void {
+        $user = $this->createConfiguredMock(IUser::class, [
+            'getUID' => 'admin',
+        ]);
+
+        $this->userManager->expects(self::once())
+            ->method('get')
+            ->willReturn($user);
+        $this->availabilityCoordinator->expects(self::once())
+            ->method('getCurrentOutOfOfficeData')
+            ->willReturn(null);
+        $this->availabilityCoordinator->expects(self::never())
+            ->method('isInEffect');
+        $this->cache->expects(self::once())
+            ->method('get')
+            ->willReturn(null);
+        $this->cache->expects(self::once())
+            ->method('set');
+        $this->calendarManager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$this->createMock(CalendarImpl::class)]);
+        $this->calendarManager->expects(self::once())
+            ->method('newQuery')
+            ->willReturn(new CalendarQuery('admin'));
+        $this->timeFactory->expects(self::exactly(2))
+            ->method('getDateTime')
+            ->willReturn(new \DateTime());
+        $this->userStatusService->expects(self::once())
+            ->method('findByUserId')
+            ->willThrowException(new DoesNotExistException(''));
+        $this->calendarManager->expects(self::once())
+            ->method('searchForPrincipal')
+            ->willReturn([['objects' => [[]]]]);
+        $this->userStatusService->expects(self::never())
+            ->method('revertUserStatus');
+        $this->logger->expects(self::once())
+            ->method('debug');
+        $this->userStatusService->expects(self::once())
+            ->method('setUserStatus');
+
+
+        $this->service->processCalendarStatus('admin');
+    }
+
+    public function testCallStatus(): void {
+        $user = $this->createConfiguredMock(IUser::class, [
+            'getUID' => 'admin',
+        ]);
+
+        $this->userManager->expects(self::once())
+            ->method('get')
+            ->willReturn($user);
+        $this->availabilityCoordinator->expects(self::once())
+            ->method('getCurrentOutOfOfficeData')
+            ->willReturn(null);
+        $this->availabilityCoordinator->expects(self::never())
+            ->method('isInEffect');
+        $this->cache->expects(self::once())
+            ->method('get')
+            ->willReturn(null);
+        $this->cache->expects(self::once())
+            ->method('set');
+        $this->calendarManager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$this->createMock(CalendarImpl::class)]);
+        $this->calendarManager->expects(self::once())
+            ->method('newQuery')
+            ->willReturn(new CalendarQuery('admin'));
+        $this->timeFactory->expects(self::exactly(2))
+            ->method('getDateTime')
+            ->willReturn(new \DateTime());
+        $this->calendarManager->expects(self::once())
+            ->method('searchForPrincipal')
+            ->willReturn([['objects' => [[]]]]);
+        $userStatus = new UserStatus();
+        $userStatus->setMessageId(IUserStatus::MESSAGE_CALL);
+        $userStatus->setStatusTimestamp(123456);
+        $this->userStatusService->expects(self::once())
+            ->method('findByUserId')
+            ->willReturn($userStatus);
+        $this->logger->expects(self::once())
+            ->method('debug');
+        $this->userStatusService->expects(self::never())
+            ->method('revertUserStatus');
+        $this->userStatusService->expects(self::never())
+            ->method('setUserStatus');
+
+
+        $this->service->processCalendarStatus('admin');
+    }
+
+    public function testInvisibleStatus(): void {
+        $user = $this->createConfiguredMock(IUser::class, [
+            'getUID' => 'admin',
+        ]);
+
+        $this->userManager->expects(self::once())
+            ->method('get')
+            ->willReturn($user);
+        $this->availabilityCoordinator->expects(self::once())
+            ->method('getCurrentOutOfOfficeData')
+            ->willReturn(null);
+        $this->availabilityCoordinator->expects(self::never())
+            ->method('isInEffect');
+        $this->cache->expects(self::once())
+            ->method('get')
+            ->willReturn(null);
+        $this->cache->expects(self::once())
+            ->method('set');
+        $this->calendarManager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$this->createMock(CalendarImpl::class)]);
+        $this->calendarManager->expects(self::once())
+            ->method('newQuery')
+            ->willReturn(new CalendarQuery('admin'));
+        $this->timeFactory->expects(self::exactly(2))
+            ->method('getDateTime')
+            ->willReturn(new \DateTime());
+        $this->calendarManager->expects(self::once())
+            ->method('searchForPrincipal')
+            ->willReturn([['objects' => [[]]]]);
+        $userStatus = new UserStatus();
+        $userStatus->setStatus(IUserStatus::INVISIBLE);
+        $userStatus->setStatusTimestamp(123456);
+        $this->userStatusService->expects(self::once())
+            ->method('findByUserId')
+            ->willReturn($userStatus);
+        $this->logger->expects(self::once())
+            ->method('debug');
+        $this->userStatusService->expects(self::never())
+            ->method('revertUserStatus');
+        $this->userStatusService->expects(self::never())
+            ->method('setUserStatus');
+
+
+        $this->service->processCalendarStatus('admin');
+    }
+
+    public function testDNDStatus(): void {
+        $user = $this->createConfiguredMock(IUser::class, [
+            'getUID' => 'admin',
+        ]);
+
+        $this->userManager->expects(self::once())
+            ->method('get')
+            ->willReturn($user);
+        $this->availabilityCoordinator->expects(self::once())
+            ->method('getCurrentOutOfOfficeData')
+            ->willReturn(null);
+        $this->availabilityCoordinator->expects(self::never())
+            ->method('isInEffect');
+        $this->cache->expects(self::once())
+            ->method('get')
+            ->willReturn(null);
+        $this->cache->expects(self::once())
+            ->method('set');
+        $this->calendarManager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$this->createMock(CalendarImpl::class)]);
+        $this->calendarManager->expects(self::once())
+            ->method('newQuery')
+            ->willReturn(new CalendarQuery('admin'));
+        $this->timeFactory->expects(self::exactly(2))
+            ->method('getDateTime')
+            ->willReturn(new \DateTime());
+        $this->calendarManager->expects(self::once())
+            ->method('searchForPrincipal')
+            ->willReturn([['objects' => [[]]]]);
+        $userStatus = new UserStatus();
+        $userStatus->setStatus(IUserStatus::DND);
+        $userStatus->setStatusTimestamp(123456);
+        $this->userStatusService->expects(self::once())
+            ->method('findByUserId')
+            ->willReturn($userStatus);
+        $this->logger->expects(self::once())
+            ->method('debug');
+        $this->userStatusService->expects(self::never())
+            ->method('revertUserStatus');
+        $this->userStatusService->expects(self::never())
+            ->method('setUserStatus');
+
+
+        $this->service->processCalendarStatus('admin');
+    }
 }

apps/dav/tests/unit/CalDAV/PublicCalendarRootTest.php

Indentation +106 added lines, -106 removed lines

@@ -32,110 +32,110 @@
  * @package OCA\DAV\Tests\unit\CalDAV
  */
 class PublicCalendarRootTest extends TestCase {
-	public const UNIT_TEST_USER = '';
-	private CalDavBackend $backend;
-	private PublicCalendarRoot $publicCalendarRoot;
-	private IL10N&MockObject $l10n;
-	private Principal&MockObject $principal;
-	protected IUserManager&MockObject $userManager;
-	protected IGroupManager&MockObject $groupManager;
-	protected IConfig&MockObject $config;
-	private ISecureRandom $random;
-	private LoggerInterface&MockObject $logger;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$db = Server::get(IDBConnection::class);
-		$this->principal = $this->createMock('OCA\DAV\Connector\Sabre\Principal');
-		$this->userManager = $this->createMock(IUserManager::class);
-		$this->groupManager = $this->createMock(IGroupManager::class);
-		$this->random = Server::get(ISecureRandom::class);
-		$this->logger = $this->createMock(LoggerInterface::class);
-		$dispatcher = $this->createMock(IEventDispatcher::class);
-		$config = $this->createMock(IConfig::class);
-		$sharingBackend = $this->createMock(\OCA\DAV\CalDAV\Sharing\Backend::class);
-
-		$this->principal->expects($this->any())->method('getGroupMembership')
-			->withAnyParameters()
-			->willReturn([]);
-
-		$this->principal->expects($this->any())->method('getCircleMembership')
-			->withAnyParameters()
-			->willReturn([]);
-
-		$this->backend = new CalDavBackend(
-			$db,
-			$this->principal,
-			$this->userManager,
-			$this->random,
-			$this->logger,
-			$dispatcher,
-			$config,
-			$sharingBackend,
-			false,
-		);
-		$this->l10n = $this->createMock(IL10N::class);
-		$this->config = $this->createMock(IConfig::class);
-
-		$this->publicCalendarRoot = new PublicCalendarRoot($this->backend,
-			$this->l10n, $this->config, $this->logger);
-	}
-
-	protected function tearDown(): void {
-		parent::tearDown();
-
-		if (is_null($this->backend)) {
-			return;
-		}
-		$this->principal->expects($this->any())->method('getGroupMembership')
-			->withAnyParameters()
-			->willReturn([]);
-
-		$this->principal->expects($this->any())->method('getCircleMembership')
-			->withAnyParameters()
-			->willReturn([]);
-
-		$books = $this->backend->getCalendarsForUser(self::UNIT_TEST_USER);
-		foreach ($books as $book) {
-			$this->backend->deleteCalendar($book['id'], true);
-		}
-	}
-
-	public function testGetName(): void {
-		$name = $this->publicCalendarRoot->getName();
-		$this->assertEquals('public-calendars', $name);
-	}
-
-	public function testGetChild(): void {
-		$calendar = $this->createPublicCalendar();
-
-		$publicCalendars = $this->backend->getPublicCalendars();
-		$this->assertEquals(1, count($publicCalendars));
-		$this->assertEquals(true, $publicCalendars[0]['{http://owncloud.org/ns}public']);
-
-		$publicCalendarURI = $publicCalendars[0]['uri'];
-
-		$calendarResult = $this->publicCalendarRoot->getChild($publicCalendarURI);
-		$this->assertEquals($calendar, $calendarResult);
-	}
-
-	public function testGetChildren(): void {
-		$this->createPublicCalendar();
-		$calendarResults = $this->publicCalendarRoot->getChildren();
-		$this->assertSame([], $calendarResults);
-	}
-
-	protected function createPublicCalendar(): Calendar {
-		$this->backend->createCalendar(self::UNIT_TEST_USER, 'Example', []);
-
-		$calendarInfo = $this->backend->getCalendarsForUser(self::UNIT_TEST_USER)[0];
-		$calendar = new PublicCalendar($this->backend, $calendarInfo, $this->l10n, $this->config, $this->logger);
-		$publicUri = $calendar->setPublishStatus(true);
-
-		$calendarInfo = $this->backend->getPublicCalendar($publicUri);
-		$calendar = new PublicCalendar($this->backend, $calendarInfo, $this->l10n, $this->config, $this->logger);
-
-		return $calendar;
-	}
+    public const UNIT_TEST_USER = '';
+    private CalDavBackend $backend;
+    private PublicCalendarRoot $publicCalendarRoot;
+    private IL10N&MockObject $l10n;
+    private Principal&MockObject $principal;
+    protected IUserManager&MockObject $userManager;
+    protected IGroupManager&MockObject $groupManager;
+    protected IConfig&MockObject $config;
+    private ISecureRandom $random;
+    private LoggerInterface&MockObject $logger;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $db = Server::get(IDBConnection::class);
+        $this->principal = $this->createMock('OCA\DAV\Connector\Sabre\Principal');
+        $this->userManager = $this->createMock(IUserManager::class);
+        $this->groupManager = $this->createMock(IGroupManager::class);
+        $this->random = Server::get(ISecureRandom::class);
+        $this->logger = $this->createMock(LoggerInterface::class);
+        $dispatcher = $this->createMock(IEventDispatcher::class);
+        $config = $this->createMock(IConfig::class);
+        $sharingBackend = $this->createMock(\OCA\DAV\CalDAV\Sharing\Backend::class);
+
+        $this->principal->expects($this->any())->method('getGroupMembership')
+            ->withAnyParameters()
+            ->willReturn([]);
+
+        $this->principal->expects($this->any())->method('getCircleMembership')
+            ->withAnyParameters()
+            ->willReturn([]);
+
+        $this->backend = new CalDavBackend(
+            $db,
+            $this->principal,
+            $this->userManager,
+            $this->random,
+            $this->logger,
+            $dispatcher,
+            $config,
+            $sharingBackend,
+            false,
+        );
+        $this->l10n = $this->createMock(IL10N::class);
+        $this->config = $this->createMock(IConfig::class);
+
+        $this->publicCalendarRoot = new PublicCalendarRoot($this->backend,
+            $this->l10n, $this->config, $this->logger);
+    }
+
+    protected function tearDown(): void {
+        parent::tearDown();
+
+        if (is_null($this->backend)) {
+            return;
+        }
+        $this->principal->expects($this->any())->method('getGroupMembership')
+            ->withAnyParameters()
+            ->willReturn([]);
+
+        $this->principal->expects($this->any())->method('getCircleMembership')
+            ->withAnyParameters()
+            ->willReturn([]);
+
+        $books = $this->backend->getCalendarsForUser(self::UNIT_TEST_USER);
+        foreach ($books as $book) {
+            $this->backend->deleteCalendar($book['id'], true);
+        }
+    }
+
+    public function testGetName(): void {
+        $name = $this->publicCalendarRoot->getName();
+        $this->assertEquals('public-calendars', $name);
+    }
+
+    public function testGetChild(): void {
+        $calendar = $this->createPublicCalendar();
+
+        $publicCalendars = $this->backend->getPublicCalendars();
+        $this->assertEquals(1, count($publicCalendars));
+        $this->assertEquals(true, $publicCalendars[0]['{http://owncloud.org/ns}public']);
+
+        $publicCalendarURI = $publicCalendars[0]['uri'];
+
+        $calendarResult = $this->publicCalendarRoot->getChild($publicCalendarURI);
+        $this->assertEquals($calendar, $calendarResult);
+    }
+
+    public function testGetChildren(): void {
+        $this->createPublicCalendar();
+        $calendarResults = $this->publicCalendarRoot->getChildren();
+        $this->assertSame([], $calendarResults);
+    }
+
+    protected function createPublicCalendar(): Calendar {
+        $this->backend->createCalendar(self::UNIT_TEST_USER, 'Example', []);
+
+        $calendarInfo = $this->backend->getCalendarsForUser(self::UNIT_TEST_USER)[0];
+        $calendar = new PublicCalendar($this->backend, $calendarInfo, $this->l10n, $this->config, $this->logger);
+        $publicUri = $calendar->setPublishStatus(true);
+
+        $calendarInfo = $this->backend->getPublicCalendar($publicUri);
+        $calendar = new PublicCalendar($this->backend, $calendarInfo, $this->l10n, $this->config, $this->logger);
+
+        return $calendar;
+    }
 }

apps/dav/tests/unit/CalDAV/CachedSubscriptionProviderTest.php

Indentation +45 added lines, -45 removed lines

@@ -17,56 +17,56 @@
 
 class CachedSubscriptionProviderTest extends TestCase {
 
-	private CalDavBackend&MockObject $backend;
-	private CachedSubscriptionProvider $provider;
+    private CalDavBackend&MockObject $backend;
+    private CachedSubscriptionProvider $provider;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->backend = $this->createMock(CalDavBackend::class);
-		$this->backend
-			->expects(self::once())
-			->method('getSubscriptionsForUser')
-			->with('user-principal-123')
-			->willReturn([
-				[
-					'id' => 'subscription-1',
-					'uri' => 'subscription-1',
-					'principaluris' => 'user-principal-123',
-					'source' => 'https://localhost/subscription-1',
-					// A subscription array has actually more properties.
-				],
-				[
-					'id' => 'subscription-2',
-					'uri' => 'subscription-2',
-					'principaluri' => 'user-principal-123',
-					'source' => 'https://localhost/subscription-2',
-					// A subscription array has actually more properties.
-				]
-			]);
+        $this->backend = $this->createMock(CalDavBackend::class);
+        $this->backend
+            ->expects(self::once())
+            ->method('getSubscriptionsForUser')
+            ->with('user-principal-123')
+            ->willReturn([
+                [
+                    'id' => 'subscription-1',
+                    'uri' => 'subscription-1',
+                    'principaluris' => 'user-principal-123',
+                    'source' => 'https://localhost/subscription-1',
+                    // A subscription array has actually more properties.
+                ],
+                [
+                    'id' => 'subscription-2',
+                    'uri' => 'subscription-2',
+                    'principaluri' => 'user-principal-123',
+                    'source' => 'https://localhost/subscription-2',
+                    // A subscription array has actually more properties.
+                ]
+            ]);
 
-		$this->provider = new CachedSubscriptionProvider($this->backend);
-	}
+        $this->provider = new CachedSubscriptionProvider($this->backend);
+    }
 
-	public function testGetCalendars(): void {
-		$calendars = $this->provider->getCalendars(
-			'user-principal-123',
-			[]
-		);
+    public function testGetCalendars(): void {
+        $calendars = $this->provider->getCalendars(
+            'user-principal-123',
+            []
+        );
 
-		$this->assertCount(2, $calendars);
-		$this->assertInstanceOf(CachedSubscriptionImpl::class, $calendars[0]);
-		$this->assertInstanceOf(CachedSubscriptionImpl::class, $calendars[1]);
-	}
+        $this->assertCount(2, $calendars);
+        $this->assertInstanceOf(CachedSubscriptionImpl::class, $calendars[0]);
+        $this->assertInstanceOf(CachedSubscriptionImpl::class, $calendars[1]);
+    }
 
-	public function testGetCalendarsFilterByUri(): void {
-		$calendars = $this->provider->getCalendars(
-			'user-principal-123',
-			['subscription-1']
-		);
+    public function testGetCalendarsFilterByUri(): void {
+        $calendars = $this->provider->getCalendars(
+            'user-principal-123',
+            ['subscription-1']
+        );
 
-		$this->assertCount(1, $calendars);
-		$this->assertInstanceOf(CachedSubscriptionImpl::class, $calendars[0]);
-		$this->assertEquals('subscription-1', $calendars[0]->getUri());
-	}
+        $this->assertCount(1, $calendars);
+        $this->assertInstanceOf(CachedSubscriptionImpl::class, $calendars[0]);
+        $this->assertEquals('subscription-1', $calendars[0]->getUri());
+    }
 }