nextcloud / server

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());
+    }
 }

apps/dav/tests/unit/CalDAV/BirthdayCalendar/EnablePluginTest.php

Indentation +150 added lines, -150 removed lines

@@ -17,205 +17,205 @@
 use Test\TestCase;
 
 class EnablePluginTest extends TestCase {
-	protected \Sabre\DAV\Server&MockObject $server;
-	protected IConfig&MockObject $config;
-	protected BirthdayService&MockObject $birthdayService;
-	protected IUser&MockObject $user;
-	protected EnablePlugin $plugin;
+    protected \Sabre\DAV\Server&MockObject $server;
+    protected IConfig&MockObject $config;
+    protected BirthdayService&MockObject $birthdayService;
+    protected IUser&MockObject $user;
+    protected EnablePlugin $plugin;
 
-	protected $request;
+    protected $request;
 
-	protected $response;
+    protected $response;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->server = $this->createMock(\Sabre\DAV\Server::class);
-		$this->server->tree = $this->createMock(\Sabre\DAV\Tree::class);
-		$this->server->httpResponse = $this->createMock(\Sabre\HTTP\Response::class);
-		$this->server->xml = $this->createMock(\Sabre\DAV\Xml\Service::class);
+        $this->server = $this->createMock(\Sabre\DAV\Server::class);
+        $this->server->tree = $this->createMock(\Sabre\DAV\Tree::class);
+        $this->server->httpResponse = $this->createMock(\Sabre\HTTP\Response::class);
+        $this->server->xml = $this->createMock(\Sabre\DAV\Xml\Service::class);
 
-		$this->config = $this->createMock(IConfig::class);
-		$this->birthdayService = $this->createMock(BirthdayService::class);
-		$this->user = $this->createMock(IUser::class);
+        $this->config = $this->createMock(IConfig::class);
+        $this->birthdayService = $this->createMock(BirthdayService::class);
+        $this->user = $this->createMock(IUser::class);
 
-		$this->plugin = new EnablePlugin($this->config, $this->birthdayService, $this->user);
-		$this->plugin->initialize($this->server);
+        $this->plugin = new EnablePlugin($this->config, $this->birthdayService, $this->user);
+        $this->plugin->initialize($this->server);
 
-		$this->request = $this->createMock(\Sabre\HTTP\RequestInterface::class);
-		$this->response = $this->createMock(\Sabre\HTTP\ResponseInterface::class);
-	}
+        $this->request = $this->createMock(\Sabre\HTTP\RequestInterface::class);
+        $this->response = $this->createMock(\Sabre\HTTP\ResponseInterface::class);
+    }
 
-	public function testGetFeatures(): void {
-		$this->assertEquals(['nc-enable-birthday-calendar'], $this->plugin->getFeatures());
-	}
+    public function testGetFeatures(): void {
+        $this->assertEquals(['nc-enable-birthday-calendar'], $this->plugin->getFeatures());
+    }
 
-	public function testGetName(): void {
-		$this->assertEquals('nc-enable-birthday-calendar', $this->plugin->getPluginName());
-	}
+    public function testGetName(): void {
+        $this->assertEquals('nc-enable-birthday-calendar', $this->plugin->getPluginName());
+    }
 
-	public function testInitialize(): void {
-		$server = $this->createMock(\Sabre\DAV\Server::class);
+    public function testInitialize(): void {
+        $server = $this->createMock(\Sabre\DAV\Server::class);
 
-		$plugin = new EnablePlugin($this->config, $this->birthdayService, $this->user);
+        $plugin = new EnablePlugin($this->config, $this->birthdayService, $this->user);
 
-		$server->expects($this->once())
-			->method('on')
-			->with('method:POST', [$plugin, 'httpPost']);
+        $server->expects($this->once())
+            ->method('on')
+            ->with('method:POST', [$plugin, 'httpPost']);
 
-		$plugin->initialize($server);
-	}
+        $plugin->initialize($server);
+    }
 
-	public function testHttpPostNoCalendarHome(): void {
-		$calendar = $this->createMock(Calendar::class);
+    public function testHttpPostNoCalendarHome(): void {
+        $calendar = $this->createMock(Calendar::class);
 
-		$this->server->expects($this->once())
-			->method('getRequestUri')
-			->willReturn('/bar/foo');
-		$this->server->tree->expects($this->once())
-			->method('getNodeForPath')
-			->with('/bar/foo')
-			->willReturn($calendar);
+        $this->server->expects($this->once())
+            ->method('getRequestUri')
+            ->willReturn('/bar/foo');
+        $this->server->tree->expects($this->once())
+            ->method('getNodeForPath')
+            ->with('/bar/foo')
+            ->willReturn($calendar);
 
-		$this->config->expects($this->never())
-			->method('setUserValue');
+        $this->config->expects($this->never())
+            ->method('setUserValue');
 
-		$this->birthdayService->expects($this->never())
-			->method('syncUser');
+        $this->birthdayService->expects($this->never())
+            ->method('syncUser');
 
-		$this->plugin->httpPost($this->request, $this->response);
-	}
+        $this->plugin->httpPost($this->request, $this->response);
+    }
 
-	public function testHttpPostWrongRequest(): void {
-		$calendarHome = $this->createMock(CalendarHome::class);
+    public function testHttpPostWrongRequest(): void {
+        $calendarHome = $this->createMock(CalendarHome::class);
 
-		$this->server->expects($this->once())
-			->method('getRequestUri')
-			->willReturn('/bar/foo');
-		$this->server->tree->expects($this->once())
-			->method('getNodeForPath')
-			->with('/bar/foo')
-			->willReturn($calendarHome);
+        $this->server->expects($this->once())
+            ->method('getRequestUri')
+            ->willReturn('/bar/foo');
+        $this->server->tree->expects($this->once())
+            ->method('getNodeForPath')
+            ->with('/bar/foo')
+            ->willReturn($calendarHome);
 
-		$this->request->expects($this->once())
-			->method('getBodyAsString')
-			->willReturn('<nc:disable-birthday-calendar xmlns:nc="http://nextcloud.com/ns"/>');
+        $this->request->expects($this->once())
+            ->method('getBodyAsString')
+            ->willReturn('<nc:disable-birthday-calendar xmlns:nc="http://nextcloud.com/ns"/>');
 
-		$this->request->expects($this->once())
-			->method('getUrl')
-			->willReturn('url_abc');
+        $this->request->expects($this->once())
+            ->method('getUrl')
+            ->willReturn('url_abc');
 
-		$this->server->xml->expects($this->once())
-			->method('parse')
-			->willReturnCallback(function ($requestBody, $url, &$documentType): void {
-				$documentType = '{http://nextcloud.com/ns}disable-birthday-calendar';
-			});
+        $this->server->xml->expects($this->once())
+            ->method('parse')
+            ->willReturnCallback(function ($requestBody, $url, &$documentType): void {
+                $documentType = '{http://nextcloud.com/ns}disable-birthday-calendar';
+            });
 
-		$this->config->expects($this->never())
-			->method('setUserValue');
+        $this->config->expects($this->never())
+            ->method('setUserValue');
 
-		$this->birthdayService->expects($this->never())
-			->method('syncUser');
+        $this->birthdayService->expects($this->never())
+            ->method('syncUser');
 
-		$this->plugin->httpPost($this->request, $this->response);
-	}
+        $this->plugin->httpPost($this->request, $this->response);
+    }
 
-	public function testHttpPostNotAuthorized(): void {
-		$calendarHome = $this->createMock(CalendarHome::class);
+    public function testHttpPostNotAuthorized(): void {
+        $calendarHome = $this->createMock(CalendarHome::class);
 
-		$this->server->expects($this->once())
-			->method('getRequestUri')
-			->willReturn('/bar/foo');
-		$this->server->tree->expects($this->once())
-			->method('getNodeForPath')
-			->with('/bar/foo')
-			->willReturn($calendarHome);
+        $this->server->expects($this->once())
+            ->method('getRequestUri')
+            ->willReturn('/bar/foo');
+        $this->server->tree->expects($this->once())
+            ->method('getNodeForPath')
+            ->with('/bar/foo')
+            ->willReturn($calendarHome);
 
-		$calendarHome->expects($this->once())
-			->method('getOwner')
-			->willReturn('principals/users/BlaBlub');
+        $calendarHome->expects($this->once())
+            ->method('getOwner')
+            ->willReturn('principals/users/BlaBlub');
 
-		$this->request->expects($this->once())
-			->method('getBodyAsString')
-			->willReturn('<nc:enable-birthday-calendar xmlns:nc="http://nextcloud.com/ns"/>');
+        $this->request->expects($this->once())
+            ->method('getBodyAsString')
+            ->willReturn('<nc:enable-birthday-calendar xmlns:nc="http://nextcloud.com/ns"/>');
 
-		$this->request->expects($this->once())
-			->method('getUrl')
-			->willReturn('url_abc');
+        $this->request->expects($this->once())
+            ->method('getUrl')
+            ->willReturn('url_abc');
 
-		$this->server->xml->expects($this->once())
-			->method('parse')
-			->willReturnCallback(function ($requestBody, $url, &$documentType): void {
-				$documentType = '{http://nextcloud.com/ns}enable-birthday-calendar';
-			});
+        $this->server->xml->expects($this->once())
+            ->method('parse')
+            ->willReturnCallback(function ($requestBody, $url, &$documentType): void {
+                $documentType = '{http://nextcloud.com/ns}enable-birthday-calendar';
+            });
 
-		$this->user->expects(self::once())
-			->method('getUID')
-			->willReturn('admin');
+        $this->user->expects(self::once())
+            ->method('getUID')
+            ->willReturn('admin');
 
-		$this->server->httpResponse->expects($this->once())
-			->method('setStatus')
-			->with(403);
+        $this->server->httpResponse->expects($this->once())
+            ->method('setStatus')
+            ->with(403);
 
-		$this->config->expects($this->never())
-			->method('setUserValue');
+        $this->config->expects($this->never())
+            ->method('setUserValue');
 
-		$this->birthdayService->expects($this->never())
-			->method('syncUser');
+        $this->birthdayService->expects($this->never())
+            ->method('syncUser');
 
 
-		$result = $this->plugin->httpPost($this->request, $this->response);
+        $result = $this->plugin->httpPost($this->request, $this->response);
 
-		$this->assertEquals(false, $result);
-	}
+        $this->assertEquals(false, $result);
+    }
 
-	public function testHttpPost(): void {
-		$calendarHome = $this->createMock(CalendarHome::class);
+    public function testHttpPost(): void {
+        $calendarHome = $this->createMock(CalendarHome::class);
 
-		$this->server->expects($this->once())
-			->method('getRequestUri')
-			->willReturn('/bar/foo');
-		$this->server->tree->expects($this->once())
-			->method('getNodeForPath')
-			->with('/bar/foo')
-			->willReturn($calendarHome);
+        $this->server->expects($this->once())
+            ->method('getRequestUri')
+            ->willReturn('/bar/foo');
+        $this->server->tree->expects($this->once())
+            ->method('getNodeForPath')
+            ->with('/bar/foo')
+            ->willReturn($calendarHome);
 
-		$calendarHome->expects($this->once())
-			->method('getOwner')
-			->willReturn('principals/users/BlaBlub');
+        $calendarHome->expects($this->once())
+            ->method('getOwner')
+            ->willReturn('principals/users/BlaBlub');
 
-		$this->request->expects($this->once())
-			->method('getBodyAsString')
-			->willReturn('<nc:enable-birthday-calendar xmlns:nc="http://nextcloud.com/ns"/>');
+        $this->request->expects($this->once())
+            ->method('getBodyAsString')
+            ->willReturn('<nc:enable-birthday-calendar xmlns:nc="http://nextcloud.com/ns"/>');
 
-		$this->request->expects($this->once())
-			->method('getUrl')
-			->willReturn('url_abc');
+        $this->request->expects($this->once())
+            ->method('getUrl')
+            ->willReturn('url_abc');
 
-		$this->server->xml->expects($this->once())
-			->method('parse')
-			->willReturnCallback(function ($requestBody, $url, &$documentType): void {
-				$documentType = '{http://nextcloud.com/ns}enable-birthday-calendar';
-			});
+        $this->server->xml->expects($this->once())
+            ->method('parse')
+            ->willReturnCallback(function ($requestBody, $url, &$documentType): void {
+                $documentType = '{http://nextcloud.com/ns}enable-birthday-calendar';
+            });
 
-		$this->user->expects(self::exactly(3))
-			->method('getUID')
-			->willReturn('BlaBlub');
+        $this->user->expects(self::exactly(3))
+            ->method('getUID')
+            ->willReturn('BlaBlub');
 
-		$this->config->expects($this->once())
-			->method('setUserValue')
-			->with('BlaBlub', 'dav', 'generateBirthdayCalendar', 'yes');
+        $this->config->expects($this->once())
+            ->method('setUserValue')
+            ->with('BlaBlub', 'dav', 'generateBirthdayCalendar', 'yes');
 
-		$this->birthdayService->expects($this->once())
-			->method('syncUser')
-			->with('BlaBlub');
+        $this->birthdayService->expects($this->once())
+            ->method('syncUser')
+            ->with('BlaBlub');
 
-		$this->server->httpResponse->expects($this->once())
-			->method('setStatus')
-			->with(204);
+        $this->server->httpResponse->expects($this->once())
+            ->method('setStatus')
+            ->with(204);
 
-		$result = $this->plugin->httpPost($this->request, $this->response);
+        $result = $this->plugin->httpPost($this->request, $this->response);
 
-		$this->assertEquals(false, $result);
-	}
+        $this->assertEquals(false, $result);
+    }
 }