nextcloud / server

tests/lib/EmojiHelperTest.php

Indentation +3121 added lines, -3121 removed lines

@@ -12,3134 +12,3134 @@
 use OCP\IEmojiHelper;
 
 class EmojiHelperTest extends TestCase {
-	/** @var IDBConnection|\PHPUnit\Framework\MockObject\MockObject */
-	private $db;
+    /** @var IDBConnection|\PHPUnit\Framework\MockObject\MockObject */
+    private $db;
 
-	private IEmojiHelper $helper;
+    private IEmojiHelper $helper;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->db = $this->createMock(IDBConnection::class);
-		$this->helper = new EmojiHelper($this->db);
-	}
+        $this->db = $this->createMock(IDBConnection::class);
+        $this->helper = new EmojiHelper($this->db);
+    }
 
-	/**
-	 * @param bool $supports4ByteText
-	 * @param bool $expected
-	 *
-	 * @dataProvider doesPlatformSupportEmojiDataProvider
-	 */
-	public function testDoesPlatformSupportEmoji(bool $supports4ByteText, bool $expected): void {
-		$this->db->expects($this->once())
-			->method('supports4ByteText')
-			->willReturn($supports4ByteText);
+    /**
+     * @param bool $supports4ByteText
+     * @param bool $expected
+     *
+     * @dataProvider doesPlatformSupportEmojiDataProvider
+     */
+    public function testDoesPlatformSupportEmoji(bool $supports4ByteText, bool $expected): void {
+        $this->db->expects($this->once())
+            ->method('supports4ByteText')
+            ->willReturn($supports4ByteText);
 
-		$this->assertEquals($expected, $this->helper->doesPlatformSupportEmoji());
-	}
+        $this->assertEquals($expected, $this->helper->doesPlatformSupportEmoji());
+    }
 
-	public static function doesPlatformSupportEmojiDataProvider(): array {
-		return [
-			[true, true],
-			[false, false],
-		];
-	}
+    public static function doesPlatformSupportEmojiDataProvider(): array {
+        return [
+            [true, true],
+            [false, false],
+        ];
+    }
 
-	/**
-	 * @param string $emoji
-	 * @param bool $expected
-	 *
-	 * @dataProvider isValidSingleEmojiDataProvider
-	 */
-	public function testIsValidSingleEmoji(string $emoji, bool $expected): void {
-		$actual = $this->helper->isValidSingleEmoji($emoji);
+    /**
+     * @param string $emoji
+     * @param bool $expected
+     *
+     * @dataProvider isValidSingleEmojiDataProvider
+     */
+    public function testIsValidSingleEmoji(string $emoji, bool $expected): void {
+        $actual = $this->helper->isValidSingleEmoji($emoji);
 
-		$this->assertEquals($expected, $actual);
-	}
+        $this->assertEquals($expected, $actual);
+    }
 
-	public static function isValidSingleEmojiDataProvider(): array {
-		return [
-			['

tests/lib/Calendar/ManagerTest.php

Indentation +1720 added lines, -1720 removed lines

@@ -36,1591 +36,1591 @@ 
 }
 
 class ManagerTest extends TestCase {
-	/** @var Coordinator&MockObject */
-	private $coordinator;
-
-	/** @var ContainerInterface&MockObject */
-	private $container;
-
-	/** @var LoggerInterface&MockObject */
-	private $logger;
-
-	/** @var Manager */
-	private $manager;
-
-	/** @var ITimeFactory&MockObject */
-	private $time;
-
-	/** @var ISecureRandom&MockObject */
-	private ISecureRandom $secureRandom;
-
-	private IUserManager&MockObject $userManager;
-	private ServerFactory&MockObject $serverFactory;
-
-	private VCalendar $vCalendar1a;
-	private VCalendar $vCalendar2a;
-	private VCalendar $vCalendar3a;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->coordinator = $this->createMock(Coordinator::class);
-		$this->container = $this->createMock(ContainerInterface::class);
-		$this->logger = $this->createMock(LoggerInterface::class);
-		$this->time = $this->createMock(ITimeFactory::class);
-		$this->secureRandom = $this->createMock(ISecureRandom::class);
-		$this->userManager = $this->createMock(IUserManager::class);
-		$this->serverFactory = $this->createMock(ServerFactory::class);
-
-		$this->manager = new Manager(
-			$this->coordinator,
-			$this->container,
-			$this->logger,
-			$this->time,
-			$this->secureRandom,
-			$this->userManager,
-			$this->serverFactory,
-		);
-
-		// construct calendar with a 1 hour event and same start/end time zones
-		$this->vCalendar1a = new VCalendar();
-		/** @var VEvent $vEvent */
-		$vEvent = $this->vCalendar1a->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 Event');
-		$vEvent->add('SEQUENCE', 3);
-		$vEvent->add('STATUS', 'CONFIRMED');
-		$vEvent->add('TRANSP', 'OPAQUE');
-		$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 calendar with a event for reply
-		$this->vCalendar2a = new VCalendar();
-		/** @var VEvent $vEvent */
-		$vEvent = $this->vCalendar2a->add('VEVENT', []);
-		$vEvent->UID->setValue('dcc733bf-b2b2-41f2-a8cf-550ae4b67aff');
-		$vEvent->add('DTSTART', '20210820');
-		$vEvent->add('DTEND', '20220821');
-		$vEvent->add('SUMMARY', 'berry basket');
-		$vEvent->add('SEQUENCE', 3);
-		$vEvent->add('STATUS', 'CONFIRMED');
-		$vEvent->add('TRANSP', 'OPAQUE');
-		$vEvent->add('ORGANIZER', 'mailto:linus@stardew-tent-living.com', ['CN' => 'admin']);
-		$vEvent->add('ATTENDEE', 'mailto:pierre@general-store.com', [
-			'CN' => 'pierre@general-store.com',
-			'CUTYPE' => 'INDIVIDUAL',
-			'ROLE' => 'REQ-PARTICIPANT',
-			'PARTSTAT' => 'ACCEPTED',
-		]);
-
-		// construct calendar with a event for reply
-		$this->vCalendar3a = new VCalendar();
-		/** @var VEvent $vEvent */
-		$vEvent = $this->vCalendar3a->add('VEVENT', []);
-		$vEvent->UID->setValue('dcc733bf-b2b2-41f2-a8cf-550ae4b67aff');
-		$vEvent->add('DTSTART', '20210820');
-		$vEvent->add('DTEND', '20220821');
-		$vEvent->add('SUMMARY', 'berry basket');
-		$vEvent->add('SEQUENCE', 3);
-		$vEvent->add('STATUS', 'CANCELLED');
-		$vEvent->add('TRANSP', 'OPAQUE');
-		$vEvent->add('ORGANIZER', 'mailto:linus@stardew-tent-living.com', ['CN' => 'admin']);
-		$vEvent->add('ATTENDEE', 'mailto:pierre@general-store.com', [
-			'CN' => 'pierre@general-store.com',
-			'CUTYPE' => 'INDIVIDUAL',
-			'ROLE' => 'REQ-PARTICIPANT',
-			'PARTSTAT' => 'ACCEPTED',
-		]);
-
-	}
-
-	/**
-	 * @dataProvider searchProvider
-	 */
-	public function testSearch($search1, $search2, $expected): void {
-		/** @var ICalendar | MockObject $calendar1 */
-		$calendar1 = $this->createMock(ICalendar::class);
-		$calendar1->method('getKey')->willReturn('simple:1');
-		$calendar1->expects($this->once())
-			->method('search')
-			->with('', [], [], null, null)
-			->willReturn($search1);
-
-		/** @var ICalendar | MockObject $calendar2 */
-		$calendar2 = $this->createMock(ICalendar::class);
-		$calendar2->method('getKey')->willReturn('simple:2');
-		$calendar2->expects($this->once())
-			->method('search')
-			->with('', [], [], null, null)
-			->willReturn($search2);
-
-		$this->manager->registerCalendar($calendar1);
-		$this->manager->registerCalendar($calendar2);
-
-		$result = $this->manager->search('');
-		$this->assertEquals($expected, $result);
-	}
-
-	/**
-	 * @dataProvider searchProvider
-	 */
-	public function testSearchOptions($search1, $search2, $expected): void {
-		/** @var ICalendar | MockObject $calendar1 */
-		$calendar1 = $this->createMock(ICalendar::class);
-		$calendar1->method('getKey')->willReturn('simple:1');
-		$calendar1->expects($this->once())
-			->method('search')
-			->with('searchTerm', ['SUMMARY', 'DESCRIPTION'],
-				['timerange' => ['start' => null, 'end' => null]], 5, 20)
-			->willReturn($search1);
-
-		/** @var ICalendar | MockObject $calendar2 */
-		$calendar2 = $this->createMock(ICalendar::class);
-		$calendar2->method('getKey')->willReturn('simple:2');
-		$calendar2->expects($this->once())
-			->method('search')
-			->with('searchTerm', ['SUMMARY', 'DESCRIPTION'],
-				['timerange' => ['start' => null, 'end' => null]], 5, 20)
-			->willReturn($search2);
-
-		$this->manager->registerCalendar($calendar1);
-		$this->manager->registerCalendar($calendar2);
-
-		$result = $this->manager->search('searchTerm', ['SUMMARY', 'DESCRIPTION'],
-			['timerange' => ['start' => null, 'end' => null]], 5, 20);
-		$this->assertEquals($expected, $result);
-	}
-
-	public static function searchProvider(): array {
-		$search1 = [
-			[
-				'id' => 1,
-				'data' => 'foobar',
-			],
-			[
-				'id' => 2,
-				'data' => 'barfoo',
-			]
-		];
-		$search2 = [
-			[
-				'id' => 3,
-				'data' => 'blablub',
-			],
-			[
-				'id' => 4,
-				'data' => 'blubbla',
-			]
-		];
-
-		$expected = [
-			[
-				'id' => 1,
-				'data' => 'foobar',
-				'calendar-key' => 'simple:1',
-			],
-			[
-				'id' => 2,
-				'data' => 'barfoo',
-				'calendar-key' => 'simple:1',
-			],
-			[
-				'id' => 3,
-				'data' => 'blablub',
-				'calendar-key' => 'simple:2',
-			],
-			[
-				'id' => 4,
-				'data' => 'blubbla',
-				'calendar-key' => 'simple:2',
-			]
-		];
-
-		return [
-			[
-				$search1,
-				$search2,
-				$expected
-			]
-		];
-	}
-
-	public function testRegisterUnregister(): void {
-		/** @var ICalendar | MockObject $calendar1 */
-		$calendar1 = $this->createMock(ICalendar::class);
-		$calendar1->method('getKey')->willReturn('key1');
-
-		/** @var ICalendar | MockObject $calendar2 */
-		$calendar2 = $this->createMock(ICalendar::class);
-		$calendar2->method('getKey')->willReturn('key2');
-
-		$this->manager->registerCalendar($calendar1);
-		$this->manager->registerCalendar($calendar2);
-
-		$result = $this->manager->getCalendars();
-		$this->assertCount(2, $result);
-		$this->assertContains($calendar1, $result);
-		$this->assertContains($calendar2, $result);
-
-		$this->manager->unregisterCalendar($calendar1);
-
-		$result = $this->manager->getCalendars();
-		$this->assertCount(1, $result);
-		$this->assertContains($calendar2, $result);
-	}
-
-	public function testGetCalendars(): void {
-		/** @var ICalendar | MockObject $calendar1 */
-		$calendar1 = $this->createMock(ICalendar::class);
-		$calendar1->method('getKey')->willReturn('key1');
-
-		/** @var ICalendar | MockObject $calendar2 */
-		$calendar2 = $this->createMock(ICalendar::class);
-		$calendar2->method('getKey')->willReturn('key2');
-
-		$this->manager->registerCalendar($calendar1);
-		$this->manager->registerCalendar($calendar2);
-
-		$result = $this->manager->getCalendars();
-		$this->assertCount(2, $result);
-		$this->assertContains($calendar1, $result);
-		$this->assertContains($calendar2, $result);
-
-		$this->manager->clear();
-
-		$result = $this->manager->getCalendars();
-
-		$this->assertCount(0, $result);
-	}
-
-	public function testEnabledIfNot(): void {
-		$isEnabled = $this->manager->isEnabled();
-		$this->assertFalse($isEnabled);
-	}
-
-	public function testIfEnabledIfSo(): void {
-		/** @var ICalendar | MockObject $calendar */
-		$calendar = $this->createMock(ICalendar::class);
-		$this->manager->registerCalendar($calendar);
-
-		$isEnabled = $this->manager->isEnabled();
-		$this->assertTrue($isEnabled);
-	}
-
-	public function testHandleImipRequestWithNoCalendars(): void {
-		// construct calendar manager returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message could not be processed because user has no calendars');
-		// construct parameters
-		$principalUri = 'principals/user/attendee1';
-		$sender = 'organizer@testing.com';
-		$recipient = 'attendee1@testing.com';
-		$calendar = $this->vCalendar1a;
-		$calendar->add('METHOD', 'REQUEST');
-		// Act
-		$result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipRequestWithInvalidData(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('error')
-			->with('iMip message could not be processed because an error occurred while parsing the iMip message');
-		// construct parameters
-		$principalUri = 'principals/user/attendee1';
-		$sender = 'organizer@testing.com';
-		$recipient = 'attendee1@testing.com';
-		// Act
-		$result = $manager->handleIMipRequest($principalUri, $sender, $recipient, 'Invalid data');
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipRequestWithNoMethod(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message contains an incorrect or invalid method');
-		// construct parameters
-		$principalUri = 'principals/user/attendee1';
-		$sender = 'organizer@testing.com';
-		$recipient = 'attendee1@testing.com';
-		$calendar = $this->vCalendar1a;
-		// Act
-		$result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipRequestWithInvalidMethod(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message contains an incorrect or invalid method');
-		// construct parameters
-		$principalUri = 'principals/user/attendee1';
-		$sender = 'organizer@testing.com';
-		$recipient = 'attendee1@testing.com';
-		$calendar = $this->vCalendar1a;
-		$calendar->add('METHOD', 'CANCEL');
-		// Act
-		$result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipRequestWithNoEvent(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message contains no event');
-		// construct parameters
-		$principalUri = 'principals/user/attendee1';
-		$sender = 'organizer@testing.com';
-		$recipient = 'attendee1@testing.com';
-		$calendar = $this->vCalendar1a;
-		$calendar->add('METHOD', 'REQUEST');
-		$calendar->remove('VEVENT');
-		// Act
-		$result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipRequestWithNoUid(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event dose not contains a UID');
-		// construct parameters
-		$principalUri = 'principals/user/attendee1';
-		$sender = 'organizer@testing.com';
-		$recipient = 'attendee1@testing.com';
-		$calendar = $this->vCalendar1a;
-		$calendar->add('METHOD', 'REQUEST');
-		$calendar->VEVENT->remove('UID');
-		// Act
-		$result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipRequestWithNoOrganizer(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event dose not contains an organizer');
-		// construct parameters
-		$principalUri = 'principals/user/attendee1';
-		$sender = 'organizer@testing.com';
-		$recipient = 'attendee1@testing.com';
-		$calendar = $this->vCalendar1a;
-		$calendar->add('METHOD', 'REQUEST');
-		$calendar->VEVENT->remove('ORGANIZER');
-		// Act
-		$result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipRequestWithNoAttendee(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event dose not contains any attendees');
-		// construct parameters
-		$principalUri = 'principals/user/attendee1';
-		$sender = 'organizer@testing.com';
-		$recipient = 'attendee1@testing.com';
-		$calendar = $this->vCalendar1a;
-		$calendar->add('METHOD', 'REQUEST');
-		$calendar->VEVENT->remove('ATTENDEE');
-		// Act
-		$result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipRequestWithInvalidAttendee(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event does not contain a attendee that matches the recipient');
-		// construct parameters
-		$principalUri = 'principals/user/attendee1';
-		$sender = 'organizer@testing.com';
-		$recipient = 'attendee2@testing.com';
-		$calendar = $this->vCalendar1a;
-		$calendar->add('METHOD', 'REQUEST');
-		// Act
-		$result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipRequestWithNoMatch(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		$userCalendar->expects(self::once())
-			->method('isDeleted')
-			->willReturn(false);
-		$userCalendar->expects(self::once())
-			->method('isWritable')
-			->willReturn(true);
-		$userCalendar->expects(self::once())
-			->method('isShared')
-			->willReturn(false);
-		$userCalendar->expects(self::once())
-			->method('search')
-			->willReturn([]);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event could not be processed because no corresponding event was found in any calendar');
-		// construct parameters
-		$principalUri = 'principals/user/attendee1';
-		$sender = 'organizer@testing.com';
-		$recipient = 'attendee1@testing.com';
-		$calendar = $this->vCalendar1a;
-		$calendar->add('METHOD', 'REQUEST');
-		// Act
-		$result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipRequest(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		$userCalendar->expects(self::once())
-			->method('isDeleted')
-			->willReturn(false);
-		$userCalendar->expects(self::once())
-			->method('isWritable')
-			->willReturn(true);
-		$userCalendar->expects(self::once())
-			->method('isShared')
-			->willReturn(false);
-		$userCalendar->expects(self::once())
-			->method('search')
-			->willReturn([['uri' => 'principals/user/attendee1/personal']]);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct parameters
-		$principalUri = 'principals/user/attendee1';
-		$sender = 'organizer@testing.com';
-		$recipient = 'attendee1@testing.com';
-		$calendar = $this->vCalendar1a;
-		$calendar->add('METHOD', 'REQUEST');
-		// construct user calendar returns
-		$userCalendar->expects(self::once())
-			->method('handleIMipMessage')
-			->with('', $calendar->serialize());
-		// Act
-		$result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertTrue($result);
-	}
-
-	public function testHandleImipReplyWithNoCalendars(): void {
-		// construct calendar manager returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message could not be processed because user has no calendars');
-		// construct parameters
-		$principalUri = 'principals/user/linus';
-		$sender = 'pierre@general-store.com';
-		$recipient = 'linus@stardew-tent-living.com';
-		$calendar = $this->vCalendar2a;
-		$calendar->add('METHOD', 'REPLY');
-		// Act
-		$result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipReplyWithInvalidData(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('error')
-			->with('iMip message could not be processed because an error occurred while parsing the iMip message');
-		// construct parameters
-		$principalUri = 'principals/user/attendee1';
-		$sender = 'organizer@testing.com';
-		$recipient = 'attendee1@testing.com';
-		// Act
-		$result = $manager->handleIMipReply($principalUri, $sender, $recipient, 'Invalid data');
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipReplyWithNoMethod(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message contains an incorrect or invalid method');
-		// construct parameters
-		$principalUri = 'principals/user/linus';
-		$sender = 'pierre@general-store.com';
-		$recipient = 'linus@stardew-tent-living.com';
-		$calendar = $this->vCalendar2a;
-		// Act
-		$result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipReplyWithInvalidMethod(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message contains an incorrect or invalid method');
-		// construct parameters
-		$principalUri = 'principals/user/linus';
-		$sender = 'pierre@general-store.com';
-		$recipient = 'linus@stardew-tent-living.com';
-		$calendar = $this->vCalendar2a;
-		$calendar->add('METHOD', 'UNKNOWN');
-		// Act
-		$result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipReplyWithNoEvent(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message contains no event');
-		// construct parameters
-		$principalUri = 'principals/user/linus';
-		$sender = 'pierre@general-store.com';
-		$recipient = 'linus@stardew-tent-living.com';
-		$calendar = $this->vCalendar2a;
-		$calendar->add('METHOD', 'REPLY');
-		$calendar->remove('VEVENT');
-		// Act
-		$result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipReplyWithNoUid(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event dose not contains a UID');
-		// construct parameters
-		$principalUri = 'principals/user/linus';
-		$sender = 'pierre@general-store.com';
-		$recipient = 'linus@stardew-tent-living.com';
-		$calendar = $this->vCalendar2a;
-		$calendar->add('METHOD', 'REPLY');
-		$calendar->VEVENT->remove('UID');
-		// Act
-		$result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipReplyWithNoOrganizer(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event dose not contains an organizer');
-		// construct parameters
-		$principalUri = 'principals/user/linus';
-		$sender = 'pierre@general-store.com';
-		$recipient = 'linus@stardew-tent-living.com';
-		$calendar = $this->vCalendar2a;
-		$calendar->add('METHOD', 'REPLY');
-		$calendar->VEVENT->remove('ORGANIZER');
-		// Act
-		$result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipReplyWithNoAttendee(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event dose not contains any attendees');
-		// construct parameters
-		$principalUri = 'principals/user/linus';
-		$sender = 'pierre@general-store.com';
-		$recipient = 'linus@stardew-tent-living.com';
-		$calendar = $this->vCalendar2a;
-		$calendar->add('METHOD', 'REPLY');
-		$calendar->VEVENT->remove('ATTENDEE');
-		// Act
-		$result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipReplyDateInThePast(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger and time returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event could not be processed because the event is in the past');
-		$this->time->expects(self::once())
-			->method('getTime')
-			->willReturn(time());
-		// construct parameters
-		$principalUri = 'principals/user/linus';
-		$sender = 'pierre@general-store.com';
-		$recipient = 'linus@stardew-tent-living.com';
-		$calendarData = clone $this->vCalendar2a;
-		$calendarData->add('METHOD', 'REPLY');
-		$calendarData->VEVENT->DTSTART = new \DateTime('2013-04-07'); // set to in the past
-		// Act
-		$result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendarData->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipReplyEventNotFound(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		$userCalendar->expects(self::once())
-			->method('search')
-			->willReturn([]);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct time returns
-		$this->time->expects(self::once())
-			->method('getTime')
-			->willReturn(1628374233);
-		// construct parameters
-		$principalUri = 'principals/user/linus';
-		$sender = 'pierre@general-store.com';
-		$recipient = 'linus@stardew-tent-living.com';
-		$calendarData = clone $this->vCalendar2a;
-		$calendarData->add('METHOD', 'REPLY');
-		// construct logger return
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event could not be processed because no corresponding event was found in any calendar ' . $principalUri . 'and UID' . $calendarData->VEVENT->UID->getValue());
-		// Act
-		$result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendarData->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipReply(): void {
-		/** @var Manager | \PHPUnit\Framework\MockObject\MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods([
-				'getCalendarsForPrincipal'
-			])
-			->getMock();
-		$calendar = $this->createMock(ITestCalendar::class);
-		$principalUri = 'principals/user/linus';
-		$sender = 'pierre@general-store.com';
-		$recipient = 'linus@stardew-tent-living.com';
-		$calendarData = clone $this->vCalendar2a;
-		$calendarData->add('METHOD', 'REPLY');
-
-		$this->time->expects(self::once())
-			->method('getTime')
-			->willReturn(1628374233);
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$calendar]);
-		$calendar->expects(self::once())
-			->method('search')
-			->willReturn([['uri' => 'testname.ics']]);
-		$calendar->expects(self::once())
-			->method('handleIMipMessage')
-			->with('testname.ics', $calendarData->serialize());
-		// Act
-		$result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendarData->serialize());
-		// Assert
-		$this->assertTrue($result);
-	}
-
-	public function testHandleImipCancelWithNoCalendars(): void {
-		// construct calendar manager returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message could not be processed because user has no calendars');
-		// construct parameters
-		$principalUri = 'principals/user/pierre';
-		$sender = 'linus@stardew-tent-living.com';
-		$recipient = 'pierre@general-store.com';
-		$replyTo = null;
-		$calendar = $this->vCalendar3a;
-		$calendar->add('METHOD', 'CANCEL');
-		// Act
-		$result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipCancelWithInvalidData(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('error')
-			->with('iMip message could not be processed because an error occurred while parsing the iMip message');
-		// construct parameters
-		$principalUri = 'principals/user/attendee1';
-		$sender = 'organizer@testing.com';
-		$recipient = 'attendee1@testing.com';
-		$replyTo = null;
-		// Act
-		$result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, 'Invalid data');
-		// Assert
-		$this->assertFalse($result);
-	}
-
-
-	public function testHandleImipCancelWithNoMethod(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message contains an incorrect or invalid method');
-		// construct parameters
-		$principalUri = 'principals/user/pierre';
-		$sender = 'linus@stardew-tent-living.com';
-		$recipient = 'pierre@general-store.com';
-		$replyTo = null;
-		$calendar = $this->vCalendar3a;
-		// Act
-		$result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipCancelWithInvalidMethod(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message contains an incorrect or invalid method');
-		// construct parameters
-		$principalUri = 'principals/user/pierre';
-		$sender = 'linus@stardew-tent-living.com';
-		$recipient = 'pierre@general-store.com';
-		$replyTo = null;
-		$calendar = $this->vCalendar3a;
-		$calendar->add('METHOD', 'UNKNOWN');
-		// Act
-		$result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipCancelWithNoEvent(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message contains no event');
-		// construct parameters
-		$principalUri = 'principals/user/pierre';
-		$sender = 'linus@stardew-tent-living.com';
-		$recipient = 'pierre@general-store.com';
-		$replyTo = null;
-		$calendar = $this->vCalendar3a;
-		$calendar->add('METHOD', 'CANCEL');
-		$calendar->remove('VEVENT');
-		// Act
-		$result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipCancelWithNoUid(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event dose not contains a UID');
-		// construct parameters
-		$principalUri = 'principals/user/pierre';
-		$sender = 'linus@stardew-tent-living.com';
-		$recipient = 'pierre@general-store.com';
-		$replyTo = null;
-		$calendar = $this->vCalendar3a;
-		$calendar->add('METHOD', 'CANCEL');
-		$calendar->VEVENT->remove('UID');
-		// Act
-		$result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipCancelWithNoOrganizer(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event dose not contains an organizer');
-		// construct parameters
-		$principalUri = 'principals/user/pierre';
-		$sender = 'linus@stardew-tent-living.com';
-		$recipient = 'pierre@general-store.com';
-		$replyTo = null;
-		$calendar = $this->vCalendar3a;
-		$calendar->add('METHOD', 'CANCEL');
-		$calendar->VEVENT->remove('ORGANIZER');
-		// Act
-		$result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipCancelWithNoAttendee(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event dose not contains any attendees');
-		// construct parameters
-		$principalUri = 'principals/user/pierre';
-		$sender = 'pierre@general-store.com';
-		$recipient = 'linus@stardew-tent-living.com';
-		$replyTo = null;
-		$calendar = $this->vCalendar3a;
-		$calendar->add('METHOD', 'CANCEL');
-		$calendar->VEVENT->remove('ATTENDEE');
-		// Act
-		$result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendar->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipCancelAttendeeNotRecipient(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event could not be processed because recipient must be an ATTENDEE of this event');
-		// construct parameters
-		$principalUri = 'principals/user/pierre';
-		$sender = 'linus@stardew-tent-living.com';
-		$recipient = 'leah@general-store.com';
-		$replyTo = null;
-		$calendarData = clone $this->vCalendar3a;
-		$calendarData->add('METHOD', 'CANCEL');
-		// Act
-		$result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendarData->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipCancelDateInThePast(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct logger and time returns
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event could not be processed because the event is in the past');
-		$this->time->expects(self::once())
-			->method('getTime')
-			->willReturn(time());
-		// construct parameters
-		$principalUri = 'principals/user/pierre';
-		$sender = 'linus@stardew-tent-living.com';
-		$recipient = 'pierre@general-store.com';
-		$replyTo = null;
-		$calendarData = clone $this->vCalendar3a;
-		$calendarData->add('METHOD', 'CANCEL');
-		$calendarData->VEVENT->DTSTART = new \DateTime('2013-04-07'); // set to in the past
-		// Act
-		$result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendarData->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipCancelEventNotFound(): void {
-		// construct mock user calendar
-		$userCalendar = $this->createMock(ITestCalendar::class);
-		$userCalendar->expects(self::once())
-			->method('search')
-			->willReturn([]);
-		// construct mock calendar manager and returns
-		/** @var Manager&MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods(['getCalendarsForPrincipal'])
-			->getMock();
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->willReturn([$userCalendar]);
-		// construct time returns
-		$this->time->expects(self::once())
-			->method('getTime')
-			->willReturn(1628374233);
-		// construct parameters
-		$principalUri = 'principals/user/pierre';
-		$sender = 'linus@stardew-tent-living.com';
-		$recipient = 'pierre@general-store.com';
-		$replyTo = null;
-		$calendarData = clone $this->vCalendar3a;
-		$calendarData->add('METHOD', 'CANCEL');
-		// construct logger return
-		$this->logger->expects(self::once())->method('warning')
-			->with('iMip message event could not be processed because no corresponding event was found in any calendar ' . $principalUri . 'and UID' . $calendarData->VEVENT->UID->getValue());
-		// Act
-		$result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendarData->serialize());
-		// Assert
-		$this->assertFalse($result);
-	}
-
-	public function testHandleImipCancelOrganiserInReplyTo(): void {
-		/** @var Manager | \PHPUnit\Framework\MockObject\MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods([
-				'getCalendarsForPrincipal'
-			])
-			->getMock();
-
-		$principalUri = 'principals/user/pierre';
-		$sender = 'clint@stardew-tent-living.com';
-		$recipient = 'pierre@general-store.com';
-		$replyTo = 'linus@stardew-tent-living.com';
-		$calendar = $this->createMock(ITestCalendar::class);
-		$calendarData = clone $this->vCalendar3a;
-		$calendarData->add('METHOD', 'CANCEL');
-
-		$this->time->expects(self::once())
-			->method('getTime')
-			->willReturn(1628374233);
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->with($principalUri)
-			->willReturn([$calendar]);
-		$calendar->expects(self::once())
-			->method('search')
-			->willReturn([['uri' => 'testname.ics']]);
-		$calendar->expects(self::once())
-			->method('handleIMipMessage')
-			->with('testname.ics', $calendarData->serialize());
-		// Act
-		$result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendarData->serialize());
-		// Assert
-		$this->assertTrue($result);
-	}
-
-	public function testHandleImipCancel(): void {
-		/** @var Manager | \PHPUnit\Framework\MockObject\MockObject $manager */
-		$manager = $this->getMockBuilder(Manager::class)
-			->setConstructorArgs([
-				$this->coordinator,
-				$this->container,
-				$this->logger,
-				$this->time,
-				$this->secureRandom,
-				$this->userManager,
-				$this->serverFactory,
-			])
-			->onlyMethods([
-				'getCalendarsForPrincipal'
-			])
-			->getMock();
-		$principalUri = 'principals/user/pierre';
-		$sender = 'linus@stardew-tent-living.com';
-		$recipient = 'pierre@general-store.com';
-		$replyTo = null;
-		$calendar = $this->createMock(ITestCalendar::class);
-		$calendarData = clone $this->vCalendar3a;
-		$calendarData->add('METHOD', 'CANCEL');
-
-		$this->time->expects(self::once())
-			->method('getTime')
-			->willReturn(1628374233);
-		$manager->expects(self::once())
-			->method('getCalendarsForPrincipal')
-			->with($principalUri)
-			->willReturn([$calendar]);
-		$calendar->expects(self::once())
-			->method('search')
-			->willReturn([['uri' => 'testname.ics']]);
-		$calendar->expects(self::once())
-			->method('handleIMipMessage')
-			->with('testname.ics', $calendarData->serialize());
-		// Act
-		$result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendarData->serialize());
-		// Assert
-		$this->assertTrue($result);
-	}
-
-	private function getFreeBusyResponse(): string {
-		return <<<EOF
+    /** @var Coordinator&MockObject */
+    private $coordinator;
+
+    /** @var ContainerInterface&MockObject */
+    private $container;
+
+    /** @var LoggerInterface&MockObject */
+    private $logger;
+
+    /** @var Manager */
+    private $manager;
+
+    /** @var ITimeFactory&MockObject */
+    private $time;
+
+    /** @var ISecureRandom&MockObject */
+    private ISecureRandom $secureRandom;
+
+    private IUserManager&MockObject $userManager;
+    private ServerFactory&MockObject $serverFactory;
+
+    private VCalendar $vCalendar1a;
+    private VCalendar $vCalendar2a;
+    private VCalendar $vCalendar3a;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->coordinator = $this->createMock(Coordinator::class);
+        $this->container = $this->createMock(ContainerInterface::class);
+        $this->logger = $this->createMock(LoggerInterface::class);
+        $this->time = $this->createMock(ITimeFactory::class);
+        $this->secureRandom = $this->createMock(ISecureRandom::class);
+        $this->userManager = $this->createMock(IUserManager::class);
+        $this->serverFactory = $this->createMock(ServerFactory::class);
+
+        $this->manager = new Manager(
+            $this->coordinator,
+            $this->container,
+            $this->logger,
+            $this->time,
+            $this->secureRandom,
+            $this->userManager,
+            $this->serverFactory,
+        );
+
+        // construct calendar with a 1 hour event and same start/end time zones
+        $this->vCalendar1a = new VCalendar();
+        /** @var VEvent $vEvent */
+        $vEvent = $this->vCalendar1a->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 Event');
+        $vEvent->add('SEQUENCE', 3);
+        $vEvent->add('STATUS', 'CONFIRMED');
+        $vEvent->add('TRANSP', 'OPAQUE');
+        $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 calendar with a event for reply
+        $this->vCalendar2a = new VCalendar();
+        /** @var VEvent $vEvent */
+        $vEvent = $this->vCalendar2a->add('VEVENT', []);
+        $vEvent->UID->setValue('dcc733bf-b2b2-41f2-a8cf-550ae4b67aff');
+        $vEvent->add('DTSTART', '20210820');
+        $vEvent->add('DTEND', '20220821');
+        $vEvent->add('SUMMARY', 'berry basket');
+        $vEvent->add('SEQUENCE', 3);
+        $vEvent->add('STATUS', 'CONFIRMED');
+        $vEvent->add('TRANSP', 'OPAQUE');
+        $vEvent->add('ORGANIZER', 'mailto:linus@stardew-tent-living.com', ['CN' => 'admin']);
+        $vEvent->add('ATTENDEE', 'mailto:pierre@general-store.com', [
+            'CN' => 'pierre@general-store.com',
+            'CUTYPE' => 'INDIVIDUAL',
+            'ROLE' => 'REQ-PARTICIPANT',
+            'PARTSTAT' => 'ACCEPTED',
+        ]);
+
+        // construct calendar with a event for reply
+        $this->vCalendar3a = new VCalendar();
+        /** @var VEvent $vEvent */
+        $vEvent = $this->vCalendar3a->add('VEVENT', []);
+        $vEvent->UID->setValue('dcc733bf-b2b2-41f2-a8cf-550ae4b67aff');
+        $vEvent->add('DTSTART', '20210820');
+        $vEvent->add('DTEND', '20220821');
+        $vEvent->add('SUMMARY', 'berry basket');
+        $vEvent->add('SEQUENCE', 3);
+        $vEvent->add('STATUS', 'CANCELLED');
+        $vEvent->add('TRANSP', 'OPAQUE');
+        $vEvent->add('ORGANIZER', 'mailto:linus@stardew-tent-living.com', ['CN' => 'admin']);
+        $vEvent->add('ATTENDEE', 'mailto:pierre@general-store.com', [
+            'CN' => 'pierre@general-store.com',
+            'CUTYPE' => 'INDIVIDUAL',
+            'ROLE' => 'REQ-PARTICIPANT',
+            'PARTSTAT' => 'ACCEPTED',
+        ]);
+
+    }
+
+    /**
+     * @dataProvider searchProvider
+     */
+    public function testSearch($search1, $search2, $expected): void {
+        /** @var ICalendar | MockObject $calendar1 */
+        $calendar1 = $this->createMock(ICalendar::class);
+        $calendar1->method('getKey')->willReturn('simple:1');
+        $calendar1->expects($this->once())
+            ->method('search')
+            ->with('', [], [], null, null)
+            ->willReturn($search1);
+
+        /** @var ICalendar | MockObject $calendar2 */
+        $calendar2 = $this->createMock(ICalendar::class);
+        $calendar2->method('getKey')->willReturn('simple:2');
+        $calendar2->expects($this->once())
+            ->method('search')
+            ->with('', [], [], null, null)
+            ->willReturn($search2);
+
+        $this->manager->registerCalendar($calendar1);
+        $this->manager->registerCalendar($calendar2);
+
+        $result = $this->manager->search('');
+        $this->assertEquals($expected, $result);
+    }
+
+    /**
+     * @dataProvider searchProvider
+     */
+    public function testSearchOptions($search1, $search2, $expected): void {
+        /** @var ICalendar | MockObject $calendar1 */
+        $calendar1 = $this->createMock(ICalendar::class);
+        $calendar1->method('getKey')->willReturn('simple:1');
+        $calendar1->expects($this->once())
+            ->method('search')
+            ->with('searchTerm', ['SUMMARY', 'DESCRIPTION'],
+                ['timerange' => ['start' => null, 'end' => null]], 5, 20)
+            ->willReturn($search1);
+
+        /** @var ICalendar | MockObject $calendar2 */
+        $calendar2 = $this->createMock(ICalendar::class);
+        $calendar2->method('getKey')->willReturn('simple:2');
+        $calendar2->expects($this->once())
+            ->method('search')
+            ->with('searchTerm', ['SUMMARY', 'DESCRIPTION'],
+                ['timerange' => ['start' => null, 'end' => null]], 5, 20)
+            ->willReturn($search2);
+
+        $this->manager->registerCalendar($calendar1);
+        $this->manager->registerCalendar($calendar2);
+
+        $result = $this->manager->search('searchTerm', ['SUMMARY', 'DESCRIPTION'],
+            ['timerange' => ['start' => null, 'end' => null]], 5, 20);
+        $this->assertEquals($expected, $result);
+    }
+
+    public static function searchProvider(): array {
+        $search1 = [
+            [
+                'id' => 1,
+                'data' => 'foobar',
+            ],
+            [
+                'id' => 2,
+                'data' => 'barfoo',
+            ]
+        ];
+        $search2 = [
+            [
+                'id' => 3,
+                'data' => 'blablub',
+            ],
+            [
+                'id' => 4,
+                'data' => 'blubbla',
+            ]
+        ];
+
+        $expected = [
+            [
+                'id' => 1,
+                'data' => 'foobar',
+                'calendar-key' => 'simple:1',
+            ],
+            [
+                'id' => 2,
+                'data' => 'barfoo',
+                'calendar-key' => 'simple:1',
+            ],
+            [
+                'id' => 3,
+                'data' => 'blablub',
+                'calendar-key' => 'simple:2',
+            ],
+            [
+                'id' => 4,
+                'data' => 'blubbla',
+                'calendar-key' => 'simple:2',
+            ]
+        ];
+
+        return [
+            [
+                $search1,
+                $search2,
+                $expected
+            ]
+        ];
+    }
+
+    public function testRegisterUnregister(): void {
+        /** @var ICalendar | MockObject $calendar1 */
+        $calendar1 = $this->createMock(ICalendar::class);
+        $calendar1->method('getKey')->willReturn('key1');
+
+        /** @var ICalendar | MockObject $calendar2 */
+        $calendar2 = $this->createMock(ICalendar::class);
+        $calendar2->method('getKey')->willReturn('key2');
+
+        $this->manager->registerCalendar($calendar1);
+        $this->manager->registerCalendar($calendar2);
+
+        $result = $this->manager->getCalendars();
+        $this->assertCount(2, $result);
+        $this->assertContains($calendar1, $result);
+        $this->assertContains($calendar2, $result);
+
+        $this->manager->unregisterCalendar($calendar1);
+
+        $result = $this->manager->getCalendars();
+        $this->assertCount(1, $result);
+        $this->assertContains($calendar2, $result);
+    }
+
+    public function testGetCalendars(): void {
+        /** @var ICalendar | MockObject $calendar1 */
+        $calendar1 = $this->createMock(ICalendar::class);
+        $calendar1->method('getKey')->willReturn('key1');
+
+        /** @var ICalendar | MockObject $calendar2 */
+        $calendar2 = $this->createMock(ICalendar::class);
+        $calendar2->method('getKey')->willReturn('key2');
+
+        $this->manager->registerCalendar($calendar1);
+        $this->manager->registerCalendar($calendar2);
+
+        $result = $this->manager->getCalendars();
+        $this->assertCount(2, $result);
+        $this->assertContains($calendar1, $result);
+        $this->assertContains($calendar2, $result);
+
+        $this->manager->clear();
+
+        $result = $this->manager->getCalendars();
+
+        $this->assertCount(0, $result);
+    }
+
+    public function testEnabledIfNot(): void {
+        $isEnabled = $this->manager->isEnabled();
+        $this->assertFalse($isEnabled);
+    }
+
+    public function testIfEnabledIfSo(): void {
+        /** @var ICalendar | MockObject $calendar */
+        $calendar = $this->createMock(ICalendar::class);
+        $this->manager->registerCalendar($calendar);
+
+        $isEnabled = $this->manager->isEnabled();
+        $this->assertTrue($isEnabled);
+    }
+
+    public function testHandleImipRequestWithNoCalendars(): void {
+        // construct calendar manager returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message could not be processed because user has no calendars');
+        // construct parameters
+        $principalUri = 'principals/user/attendee1';
+        $sender = 'organizer@testing.com';
+        $recipient = 'attendee1@testing.com';
+        $calendar = $this->vCalendar1a;
+        $calendar->add('METHOD', 'REQUEST');
+        // Act
+        $result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipRequestWithInvalidData(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('error')
+            ->with('iMip message could not be processed because an error occurred while parsing the iMip message');
+        // construct parameters
+        $principalUri = 'principals/user/attendee1';
+        $sender = 'organizer@testing.com';
+        $recipient = 'attendee1@testing.com';
+        // Act
+        $result = $manager->handleIMipRequest($principalUri, $sender, $recipient, 'Invalid data');
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipRequestWithNoMethod(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message contains an incorrect or invalid method');
+        // construct parameters
+        $principalUri = 'principals/user/attendee1';
+        $sender = 'organizer@testing.com';
+        $recipient = 'attendee1@testing.com';
+        $calendar = $this->vCalendar1a;
+        // Act
+        $result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipRequestWithInvalidMethod(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message contains an incorrect or invalid method');
+        // construct parameters
+        $principalUri = 'principals/user/attendee1';
+        $sender = 'organizer@testing.com';
+        $recipient = 'attendee1@testing.com';
+        $calendar = $this->vCalendar1a;
+        $calendar->add('METHOD', 'CANCEL');
+        // Act
+        $result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipRequestWithNoEvent(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message contains no event');
+        // construct parameters
+        $principalUri = 'principals/user/attendee1';
+        $sender = 'organizer@testing.com';
+        $recipient = 'attendee1@testing.com';
+        $calendar = $this->vCalendar1a;
+        $calendar->add('METHOD', 'REQUEST');
+        $calendar->remove('VEVENT');
+        // Act
+        $result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipRequestWithNoUid(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event dose not contains a UID');
+        // construct parameters
+        $principalUri = 'principals/user/attendee1';
+        $sender = 'organizer@testing.com';
+        $recipient = 'attendee1@testing.com';
+        $calendar = $this->vCalendar1a;
+        $calendar->add('METHOD', 'REQUEST');
+        $calendar->VEVENT->remove('UID');
+        // Act
+        $result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipRequestWithNoOrganizer(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event dose not contains an organizer');
+        // construct parameters
+        $principalUri = 'principals/user/attendee1';
+        $sender = 'organizer@testing.com';
+        $recipient = 'attendee1@testing.com';
+        $calendar = $this->vCalendar1a;
+        $calendar->add('METHOD', 'REQUEST');
+        $calendar->VEVENT->remove('ORGANIZER');
+        // Act
+        $result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipRequestWithNoAttendee(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event dose not contains any attendees');
+        // construct parameters
+        $principalUri = 'principals/user/attendee1';
+        $sender = 'organizer@testing.com';
+        $recipient = 'attendee1@testing.com';
+        $calendar = $this->vCalendar1a;
+        $calendar->add('METHOD', 'REQUEST');
+        $calendar->VEVENT->remove('ATTENDEE');
+        // Act
+        $result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipRequestWithInvalidAttendee(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event does not contain a attendee that matches the recipient');
+        // construct parameters
+        $principalUri = 'principals/user/attendee1';
+        $sender = 'organizer@testing.com';
+        $recipient = 'attendee2@testing.com';
+        $calendar = $this->vCalendar1a;
+        $calendar->add('METHOD', 'REQUEST');
+        // Act
+        $result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipRequestWithNoMatch(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        $userCalendar->expects(self::once())
+            ->method('isDeleted')
+            ->willReturn(false);
+        $userCalendar->expects(self::once())
+            ->method('isWritable')
+            ->willReturn(true);
+        $userCalendar->expects(self::once())
+            ->method('isShared')
+            ->willReturn(false);
+        $userCalendar->expects(self::once())
+            ->method('search')
+            ->willReturn([]);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event could not be processed because no corresponding event was found in any calendar');
+        // construct parameters
+        $principalUri = 'principals/user/attendee1';
+        $sender = 'organizer@testing.com';
+        $recipient = 'attendee1@testing.com';
+        $calendar = $this->vCalendar1a;
+        $calendar->add('METHOD', 'REQUEST');
+        // Act
+        $result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipRequest(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        $userCalendar->expects(self::once())
+            ->method('isDeleted')
+            ->willReturn(false);
+        $userCalendar->expects(self::once())
+            ->method('isWritable')
+            ->willReturn(true);
+        $userCalendar->expects(self::once())
+            ->method('isShared')
+            ->willReturn(false);
+        $userCalendar->expects(self::once())
+            ->method('search')
+            ->willReturn([['uri' => 'principals/user/attendee1/personal']]);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct parameters
+        $principalUri = 'principals/user/attendee1';
+        $sender = 'organizer@testing.com';
+        $recipient = 'attendee1@testing.com';
+        $calendar = $this->vCalendar1a;
+        $calendar->add('METHOD', 'REQUEST');
+        // construct user calendar returns
+        $userCalendar->expects(self::once())
+            ->method('handleIMipMessage')
+            ->with('', $calendar->serialize());
+        // Act
+        $result = $manager->handleIMipRequest($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertTrue($result);
+    }
+
+    public function testHandleImipReplyWithNoCalendars(): void {
+        // construct calendar manager returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message could not be processed because user has no calendars');
+        // construct parameters
+        $principalUri = 'principals/user/linus';
+        $sender = 'pierre@general-store.com';
+        $recipient = 'linus@stardew-tent-living.com';
+        $calendar = $this->vCalendar2a;
+        $calendar->add('METHOD', 'REPLY');
+        // Act
+        $result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipReplyWithInvalidData(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('error')
+            ->with('iMip message could not be processed because an error occurred while parsing the iMip message');
+        // construct parameters
+        $principalUri = 'principals/user/attendee1';
+        $sender = 'organizer@testing.com';
+        $recipient = 'attendee1@testing.com';
+        // Act
+        $result = $manager->handleIMipReply($principalUri, $sender, $recipient, 'Invalid data');
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipReplyWithNoMethod(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message contains an incorrect or invalid method');
+        // construct parameters
+        $principalUri = 'principals/user/linus';
+        $sender = 'pierre@general-store.com';
+        $recipient = 'linus@stardew-tent-living.com';
+        $calendar = $this->vCalendar2a;
+        // Act
+        $result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipReplyWithInvalidMethod(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message contains an incorrect or invalid method');
+        // construct parameters
+        $principalUri = 'principals/user/linus';
+        $sender = 'pierre@general-store.com';
+        $recipient = 'linus@stardew-tent-living.com';
+        $calendar = $this->vCalendar2a;
+        $calendar->add('METHOD', 'UNKNOWN');
+        // Act
+        $result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipReplyWithNoEvent(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message contains no event');
+        // construct parameters
+        $principalUri = 'principals/user/linus';
+        $sender = 'pierre@general-store.com';
+        $recipient = 'linus@stardew-tent-living.com';
+        $calendar = $this->vCalendar2a;
+        $calendar->add('METHOD', 'REPLY');
+        $calendar->remove('VEVENT');
+        // Act
+        $result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipReplyWithNoUid(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event dose not contains a UID');
+        // construct parameters
+        $principalUri = 'principals/user/linus';
+        $sender = 'pierre@general-store.com';
+        $recipient = 'linus@stardew-tent-living.com';
+        $calendar = $this->vCalendar2a;
+        $calendar->add('METHOD', 'REPLY');
+        $calendar->VEVENT->remove('UID');
+        // Act
+        $result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipReplyWithNoOrganizer(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event dose not contains an organizer');
+        // construct parameters
+        $principalUri = 'principals/user/linus';
+        $sender = 'pierre@general-store.com';
+        $recipient = 'linus@stardew-tent-living.com';
+        $calendar = $this->vCalendar2a;
+        $calendar->add('METHOD', 'REPLY');
+        $calendar->VEVENT->remove('ORGANIZER');
+        // Act
+        $result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipReplyWithNoAttendee(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event dose not contains any attendees');
+        // construct parameters
+        $principalUri = 'principals/user/linus';
+        $sender = 'pierre@general-store.com';
+        $recipient = 'linus@stardew-tent-living.com';
+        $calendar = $this->vCalendar2a;
+        $calendar->add('METHOD', 'REPLY');
+        $calendar->VEVENT->remove('ATTENDEE');
+        // Act
+        $result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipReplyDateInThePast(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger and time returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event could not be processed because the event is in the past');
+        $this->time->expects(self::once())
+            ->method('getTime')
+            ->willReturn(time());
+        // construct parameters
+        $principalUri = 'principals/user/linus';
+        $sender = 'pierre@general-store.com';
+        $recipient = 'linus@stardew-tent-living.com';
+        $calendarData = clone $this->vCalendar2a;
+        $calendarData->add('METHOD', 'REPLY');
+        $calendarData->VEVENT->DTSTART = new \DateTime('2013-04-07'); // set to in the past
+        // Act
+        $result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendarData->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipReplyEventNotFound(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        $userCalendar->expects(self::once())
+            ->method('search')
+            ->willReturn([]);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct time returns
+        $this->time->expects(self::once())
+            ->method('getTime')
+            ->willReturn(1628374233);
+        // construct parameters
+        $principalUri = 'principals/user/linus';
+        $sender = 'pierre@general-store.com';
+        $recipient = 'linus@stardew-tent-living.com';
+        $calendarData = clone $this->vCalendar2a;
+        $calendarData->add('METHOD', 'REPLY');
+        // construct logger return
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event could not be processed because no corresponding event was found in any calendar ' . $principalUri . 'and UID' . $calendarData->VEVENT->UID->getValue());
+        // Act
+        $result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendarData->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipReply(): void {
+        /** @var Manager | \PHPUnit\Framework\MockObject\MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods([
+                'getCalendarsForPrincipal'
+            ])
+            ->getMock();
+        $calendar = $this->createMock(ITestCalendar::class);
+        $principalUri = 'principals/user/linus';
+        $sender = 'pierre@general-store.com';
+        $recipient = 'linus@stardew-tent-living.com';
+        $calendarData = clone $this->vCalendar2a;
+        $calendarData->add('METHOD', 'REPLY');
+
+        $this->time->expects(self::once())
+            ->method('getTime')
+            ->willReturn(1628374233);
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$calendar]);
+        $calendar->expects(self::once())
+            ->method('search')
+            ->willReturn([['uri' => 'testname.ics']]);
+        $calendar->expects(self::once())
+            ->method('handleIMipMessage')
+            ->with('testname.ics', $calendarData->serialize());
+        // Act
+        $result = $manager->handleIMipReply($principalUri, $sender, $recipient, $calendarData->serialize());
+        // Assert
+        $this->assertTrue($result);
+    }
+
+    public function testHandleImipCancelWithNoCalendars(): void {
+        // construct calendar manager returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message could not be processed because user has no calendars');
+        // construct parameters
+        $principalUri = 'principals/user/pierre';
+        $sender = 'linus@stardew-tent-living.com';
+        $recipient = 'pierre@general-store.com';
+        $replyTo = null;
+        $calendar = $this->vCalendar3a;
+        $calendar->add('METHOD', 'CANCEL');
+        // Act
+        $result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipCancelWithInvalidData(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('error')
+            ->with('iMip message could not be processed because an error occurred while parsing the iMip message');
+        // construct parameters
+        $principalUri = 'principals/user/attendee1';
+        $sender = 'organizer@testing.com';
+        $recipient = 'attendee1@testing.com';
+        $replyTo = null;
+        // Act
+        $result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, 'Invalid data');
+        // Assert
+        $this->assertFalse($result);
+    }
+
+
+    public function testHandleImipCancelWithNoMethod(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message contains an incorrect or invalid method');
+        // construct parameters
+        $principalUri = 'principals/user/pierre';
+        $sender = 'linus@stardew-tent-living.com';
+        $recipient = 'pierre@general-store.com';
+        $replyTo = null;
+        $calendar = $this->vCalendar3a;
+        // Act
+        $result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipCancelWithInvalidMethod(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message contains an incorrect or invalid method');
+        // construct parameters
+        $principalUri = 'principals/user/pierre';
+        $sender = 'linus@stardew-tent-living.com';
+        $recipient = 'pierre@general-store.com';
+        $replyTo = null;
+        $calendar = $this->vCalendar3a;
+        $calendar->add('METHOD', 'UNKNOWN');
+        // Act
+        $result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipCancelWithNoEvent(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message contains no event');
+        // construct parameters
+        $principalUri = 'principals/user/pierre';
+        $sender = 'linus@stardew-tent-living.com';
+        $recipient = 'pierre@general-store.com';
+        $replyTo = null;
+        $calendar = $this->vCalendar3a;
+        $calendar->add('METHOD', 'CANCEL');
+        $calendar->remove('VEVENT');
+        // Act
+        $result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipCancelWithNoUid(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event dose not contains a UID');
+        // construct parameters
+        $principalUri = 'principals/user/pierre';
+        $sender = 'linus@stardew-tent-living.com';
+        $recipient = 'pierre@general-store.com';
+        $replyTo = null;
+        $calendar = $this->vCalendar3a;
+        $calendar->add('METHOD', 'CANCEL');
+        $calendar->VEVENT->remove('UID');
+        // Act
+        $result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipCancelWithNoOrganizer(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event dose not contains an organizer');
+        // construct parameters
+        $principalUri = 'principals/user/pierre';
+        $sender = 'linus@stardew-tent-living.com';
+        $recipient = 'pierre@general-store.com';
+        $replyTo = null;
+        $calendar = $this->vCalendar3a;
+        $calendar->add('METHOD', 'CANCEL');
+        $calendar->VEVENT->remove('ORGANIZER');
+        // Act
+        $result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipCancelWithNoAttendee(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event dose not contains any attendees');
+        // construct parameters
+        $principalUri = 'principals/user/pierre';
+        $sender = 'pierre@general-store.com';
+        $recipient = 'linus@stardew-tent-living.com';
+        $replyTo = null;
+        $calendar = $this->vCalendar3a;
+        $calendar->add('METHOD', 'CANCEL');
+        $calendar->VEVENT->remove('ATTENDEE');
+        // Act
+        $result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendar->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipCancelAttendeeNotRecipient(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event could not be processed because recipient must be an ATTENDEE of this event');
+        // construct parameters
+        $principalUri = 'principals/user/pierre';
+        $sender = 'linus@stardew-tent-living.com';
+        $recipient = 'leah@general-store.com';
+        $replyTo = null;
+        $calendarData = clone $this->vCalendar3a;
+        $calendarData->add('METHOD', 'CANCEL');
+        // Act
+        $result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendarData->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipCancelDateInThePast(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct logger and time returns
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event could not be processed because the event is in the past');
+        $this->time->expects(self::once())
+            ->method('getTime')
+            ->willReturn(time());
+        // construct parameters
+        $principalUri = 'principals/user/pierre';
+        $sender = 'linus@stardew-tent-living.com';
+        $recipient = 'pierre@general-store.com';
+        $replyTo = null;
+        $calendarData = clone $this->vCalendar3a;
+        $calendarData->add('METHOD', 'CANCEL');
+        $calendarData->VEVENT->DTSTART = new \DateTime('2013-04-07'); // set to in the past
+        // Act
+        $result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendarData->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipCancelEventNotFound(): void {
+        // construct mock user calendar
+        $userCalendar = $this->createMock(ITestCalendar::class);
+        $userCalendar->expects(self::once())
+            ->method('search')
+            ->willReturn([]);
+        // construct mock calendar manager and returns
+        /** @var Manager&MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods(['getCalendarsForPrincipal'])
+            ->getMock();
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->willReturn([$userCalendar]);
+        // construct time returns
+        $this->time->expects(self::once())
+            ->method('getTime')
+            ->willReturn(1628374233);
+        // construct parameters
+        $principalUri = 'principals/user/pierre';
+        $sender = 'linus@stardew-tent-living.com';
+        $recipient = 'pierre@general-store.com';
+        $replyTo = null;
+        $calendarData = clone $this->vCalendar3a;
+        $calendarData->add('METHOD', 'CANCEL');
+        // construct logger return
+        $this->logger->expects(self::once())->method('warning')
+            ->with('iMip message event could not be processed because no corresponding event was found in any calendar ' . $principalUri . 'and UID' . $calendarData->VEVENT->UID->getValue());
+        // Act
+        $result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendarData->serialize());
+        // Assert
+        $this->assertFalse($result);
+    }
+
+    public function testHandleImipCancelOrganiserInReplyTo(): void {
+        /** @var Manager | \PHPUnit\Framework\MockObject\MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods([
+                'getCalendarsForPrincipal'
+            ])
+            ->getMock();
+
+        $principalUri = 'principals/user/pierre';
+        $sender = 'clint@stardew-tent-living.com';
+        $recipient = 'pierre@general-store.com';
+        $replyTo = 'linus@stardew-tent-living.com';
+        $calendar = $this->createMock(ITestCalendar::class);
+        $calendarData = clone $this->vCalendar3a;
+        $calendarData->add('METHOD', 'CANCEL');
+
+        $this->time->expects(self::once())
+            ->method('getTime')
+            ->willReturn(1628374233);
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->with($principalUri)
+            ->willReturn([$calendar]);
+        $calendar->expects(self::once())
+            ->method('search')
+            ->willReturn([['uri' => 'testname.ics']]);
+        $calendar->expects(self::once())
+            ->method('handleIMipMessage')
+            ->with('testname.ics', $calendarData->serialize());
+        // Act
+        $result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendarData->serialize());
+        // Assert
+        $this->assertTrue($result);
+    }
+
+    public function testHandleImipCancel(): void {
+        /** @var Manager | \PHPUnit\Framework\MockObject\MockObject $manager */
+        $manager = $this->getMockBuilder(Manager::class)
+            ->setConstructorArgs([
+                $this->coordinator,
+                $this->container,
+                $this->logger,
+                $this->time,
+                $this->secureRandom,
+                $this->userManager,
+                $this->serverFactory,
+            ])
+            ->onlyMethods([
+                'getCalendarsForPrincipal'
+            ])
+            ->getMock();
+        $principalUri = 'principals/user/pierre';
+        $sender = 'linus@stardew-tent-living.com';
+        $recipient = 'pierre@general-store.com';
+        $replyTo = null;
+        $calendar = $this->createMock(ITestCalendar::class);
+        $calendarData = clone $this->vCalendar3a;
+        $calendarData->add('METHOD', 'CANCEL');
+
+        $this->time->expects(self::once())
+            ->method('getTime')
+            ->willReturn(1628374233);
+        $manager->expects(self::once())
+            ->method('getCalendarsForPrincipal')
+            ->with($principalUri)
+            ->willReturn([$calendar]);
+        $calendar->expects(self::once())
+            ->method('search')
+            ->willReturn([['uri' => 'testname.ics']]);
+        $calendar->expects(self::once())
+            ->method('handleIMipMessage')
+            ->with('testname.ics', $calendarData->serialize());
+        // Act
+        $result = $manager->handleIMipCancel($principalUri, $sender, $replyTo, $recipient, $calendarData->serialize());
+        // Assert
+        $this->assertTrue($result);
+    }
+
+    private function getFreeBusyResponse(): string {
+        return <<<EOF
 <?xml version="1.0" encoding="utf-8"?>
 <cal:schedule-response xmlns:d="DAV:" xmlns:s="http://sabredav.org/ns" xmlns:cal="urn:ietf:params:xml:ns:caldav" xmlns:cs="http://calendarserver.org/ns/" xmlns:oc="http://owncloud.org/ns" xmlns:nc="http://nextcloud.org/ns">
   <cal:response>
@@ -1698,139 +1698,139 @@ 
   </cal:response>
 </cal:schedule-response>
 EOF;
-	}
-
-	public function testCheckAvailability(): void {
-		$organizer = $this->createMock(IUser::class);
-		$organizer->expects(self::once())
-			->method('getUID')
-			->willReturn('admin');
-		$organizer->expects(self::once())
-			->method('getEMailAddress')
-			->willReturn('admin@imap.localhost');
-
-		$user1 = $this->createMock(IUser::class);
-		$user2 = $this->createMock(IUser::class);
-
-		$this->userManager->expects(self::exactly(3))
-			->method('getByEmail')
-			->willReturnMap([
-				['user@imap.localhost', [$user1]],
-				['empty@imap.localhost', [$user2]],
-				['nouser@domain.tld', []],
-			]);
-
-		$authPlugin = $this->createMock(CustomPrincipalPlugin::class);
-		$authPlugin->expects(self::once())
-			->method('setCurrentPrincipal')
-			->with('principals/users/admin');
-
-		$server = $this->createMock(\OCA\DAV\Connector\Sabre\Server::class);
-		$server->expects(self::once())
-			->method('getPlugin')
-			->with('auth')
-			->willReturn($authPlugin);
-		$server->expects(self::once())
-			->method('invokeMethod')
-			->willReturnCallback(function (
-				RequestInterface $request,
-				ResponseInterface $response,
-				bool $sendResponse,
-			) {
-				$requestBody = file_get_contents(__DIR__ . '/../../data/ics/free-busy-request.ics');
-				$this->assertEquals('POST', $request->getMethod());
-				$this->assertEquals('calendars/admin/outbox', $request->getPath());
-				$this->assertEquals('text/calendar', $request->getHeader('Content-Type'));
-				$this->assertEquals('0', $request->getHeader('Depth'));
-				$this->assertEquals($requestBody, $request->getBodyAsString());
-				$this->assertFalse($sendResponse);
-				$response->setStatus(200);
-				$response->setBody($this->getFreeBusyResponse());
-			});
-
-		$this->serverFactory->expects(self::once())
-			->method('createAttendeeAvailabilityServer')
-			->willReturn($server);
-
-		$start = new DateTimeImmutable('2025-01-16T06:00:00Z');
-		$end = new DateTimeImmutable('2025-01-17T06:00:00Z');
-		$actual = $this->manager->checkAvailability($start, $end, $organizer, [
-			'user@imap.localhost',
-			'empty@imap.localhost',
-			'nouser@domain.tld',
-		]);
-		$expected = [
-			new AvailabilityResult('admin@imap.localhost', false),
-			new AvailabilityResult('empty@imap.localhost', true),
-			new AvailabilityResult('user@imap.localhost', false),
-		];
-		$this->assertEquals($expected, $actual);
-	}
-
-	public function testCheckAvailabilityWithMailtoPrefix(): void {
-		$organizer = $this->createMock(IUser::class);
-		$organizer->expects(self::once())
-			->method('getUID')
-			->willReturn('admin');
-		$organizer->expects(self::once())
-			->method('getEMailAddress')
-			->willReturn('admin@imap.localhost');
-
-		$user1 = $this->createMock(IUser::class);
-		$user2 = $this->createMock(IUser::class);
-
-		$this->userManager->expects(self::exactly(3))
-			->method('getByEmail')
-			->willReturnMap([
-				['user@imap.localhost', [$user1]],
-				['empty@imap.localhost', [$user2]],
-				['nouser@domain.tld', []],
-			]);
-
-		$authPlugin = $this->createMock(CustomPrincipalPlugin::class);
-		$authPlugin->expects(self::once())
-			->method('setCurrentPrincipal')
-			->with('principals/users/admin');
-
-		$server = $this->createMock(\OCA\DAV\Connector\Sabre\Server::class);
-		$server->expects(self::once())
-			->method('getPlugin')
-			->with('auth')
-			->willReturn($authPlugin);
-		$server->expects(self::once())
-			->method('invokeMethod')
-			->willReturnCallback(function (
-				RequestInterface $request,
-				ResponseInterface $response,
-				bool $sendResponse,
-			) {
-				$requestBody = file_get_contents(__DIR__ . '/../../data/ics/free-busy-request.ics');
-				$this->assertEquals('POST', $request->getMethod());
-				$this->assertEquals('calendars/admin/outbox', $request->getPath());
-				$this->assertEquals('text/calendar', $request->getHeader('Content-Type'));
-				$this->assertEquals('0', $request->getHeader('Depth'));
-				$this->assertEquals($requestBody, $request->getBodyAsString());
-				$this->assertFalse($sendResponse);
-				$response->setStatus(200);
-				$response->setBody($this->getFreeBusyResponse());
-			});
-
-		$this->serverFactory->expects(self::once())
-			->method('createAttendeeAvailabilityServer')
-			->willReturn($server);
-
-		$start = new DateTimeImmutable('2025-01-16T06:00:00Z');
-		$end = new DateTimeImmutable('2025-01-17T06:00:00Z');
-		$actual = $this->manager->checkAvailability($start, $end, $organizer, [
-			'mailto:user@imap.localhost',
-			'mailto:empty@imap.localhost',
-			'mailto:nouser@domain.tld',
-		]);
-		$expected = [
-			new AvailabilityResult('admin@imap.localhost', false),
-			new AvailabilityResult('empty@imap.localhost', true),
-			new AvailabilityResult('user@imap.localhost', false),
-		];
-		$this->assertEquals($expected, $actual);
-	}
+    }
+
+    public function testCheckAvailability(): void {
+        $organizer = $this->createMock(IUser::class);
+        $organizer->expects(self::once())
+            ->method('getUID')
+            ->willReturn('admin');
+        $organizer->expects(self::once())
+            ->method('getEMailAddress')
+            ->willReturn('admin@imap.localhost');
+
+        $user1 = $this->createMock(IUser::class);
+        $user2 = $this->createMock(IUser::class);
+
+        $this->userManager->expects(self::exactly(3))
+            ->method('getByEmail')
+            ->willReturnMap([
+                ['user@imap.localhost', [$user1]],
+                ['empty@imap.localhost', [$user2]],
+                ['nouser@domain.tld', []],
+            ]);
+
+        $authPlugin = $this->createMock(CustomPrincipalPlugin::class);
+        $authPlugin->expects(self::once())
+            ->method('setCurrentPrincipal')
+            ->with('principals/users/admin');
+
+        $server = $this->createMock(\OCA\DAV\Connector\Sabre\Server::class);
+        $server->expects(self::once())
+            ->method('getPlugin')
+            ->with('auth')
+            ->willReturn($authPlugin);
+        $server->expects(self::once())
+            ->method('invokeMethod')
+            ->willReturnCallback(function (
+                RequestInterface $request,
+                ResponseInterface $response,
+                bool $sendResponse,
+            ) {
+                $requestBody = file_get_contents(__DIR__ . '/../../data/ics/free-busy-request.ics');
+                $this->assertEquals('POST', $request->getMethod());
+                $this->assertEquals('calendars/admin/outbox', $request->getPath());
+                $this->assertEquals('text/calendar', $request->getHeader('Content-Type'));
+                $this->assertEquals('0', $request->getHeader('Depth'));
+                $this->assertEquals($requestBody, $request->getBodyAsString());
+                $this->assertFalse($sendResponse);
+                $response->setStatus(200);
+                $response->setBody($this->getFreeBusyResponse());
+            });
+
+        $this->serverFactory->expects(self::once())
+            ->method('createAttendeeAvailabilityServer')
+            ->willReturn($server);
+
+        $start = new DateTimeImmutable('2025-01-16T06:00:00Z');
+        $end = new DateTimeImmutable('2025-01-17T06:00:00Z');
+        $actual = $this->manager->checkAvailability($start, $end, $organizer, [
+            'user@imap.localhost',
+            'empty@imap.localhost',
+            'nouser@domain.tld',
+        ]);
+        $expected = [
+            new AvailabilityResult('admin@imap.localhost', false),
+            new AvailabilityResult('empty@imap.localhost', true),
+            new AvailabilityResult('user@imap.localhost', false),
+        ];
+        $this->assertEquals($expected, $actual);
+    }
+
+    public function testCheckAvailabilityWithMailtoPrefix(): void {
+        $organizer = $this->createMock(IUser::class);
+        $organizer->expects(self::once())
+            ->method('getUID')
+            ->willReturn('admin');
+        $organizer->expects(self::once())
+            ->method('getEMailAddress')
+            ->willReturn('admin@imap.localhost');
+
+        $user1 = $this->createMock(IUser::class);
+        $user2 = $this->createMock(IUser::class);
+
+        $this->userManager->expects(self::exactly(3))
+            ->method('getByEmail')
+            ->willReturnMap([
+                ['user@imap.localhost', [$user1]],
+                ['empty@imap.localhost', [$user2]],
+                ['nouser@domain.tld', []],
+            ]);
+
+        $authPlugin = $this->createMock(CustomPrincipalPlugin::class);
+        $authPlugin->expects(self::once())
+            ->method('setCurrentPrincipal')
+            ->with('principals/users/admin');
+
+        $server = $this->createMock(\OCA\DAV\Connector\Sabre\Server::class);
+        $server->expects(self::once())
+            ->method('getPlugin')
+            ->with('auth')
+            ->willReturn($authPlugin);
+        $server->expects(self::once())
+            ->method('invokeMethod')
+            ->willReturnCallback(function (
+                RequestInterface $request,
+                ResponseInterface $response,
+                bool $sendResponse,
+            ) {
+                $requestBody = file_get_contents(__DIR__ . '/../../data/ics/free-busy-request.ics');
+                $this->assertEquals('POST', $request->getMethod());
+                $this->assertEquals('calendars/admin/outbox', $request->getPath());
+                $this->assertEquals('text/calendar', $request->getHeader('Content-Type'));
+                $this->assertEquals('0', $request->getHeader('Depth'));
+                $this->assertEquals($requestBody, $request->getBodyAsString());
+                $this->assertFalse($sendResponse);
+                $response->setStatus(200);
+                $response->setBody($this->getFreeBusyResponse());
+            });
+
+        $this->serverFactory->expects(self::once())
+            ->method('createAttendeeAvailabilityServer')
+            ->willReturn($server);
+
+        $start = new DateTimeImmutable('2025-01-16T06:00:00Z');
+        $end = new DateTimeImmutable('2025-01-17T06:00:00Z');
+        $actual = $this->manager->checkAvailability($start, $end, $organizer, [
+            'mailto:user@imap.localhost',
+            'mailto:empty@imap.localhost',
+            'mailto:nouser@domain.tld',
+        ]);
+        $expected = [
+            new AvailabilityResult('admin@imap.localhost', false),
+            new AvailabilityResult('empty@imap.localhost', true),
+            new AvailabilityResult('user@imap.localhost', false),
+        ];
+        $this->assertEquals($expected, $actual);
+    }
 }

tests/lib/MemoryInfoTest.php

Indentation +65 added lines, -65 removed lines

@@ -16,77 +16,77 @@
  * This class provides tests for the MemoryInfo class.
  */
 class MemoryInfoTest extends TestCase {
-	/**
-	 * The "memory_limit" value before tests.
-	 *
-	 * @var string
-	 */
-	private $iniSettingBeforeTest;
+    /**
+     * The "memory_limit" value before tests.
+     *
+     * @var string
+     */
+    private $iniSettingBeforeTest;
 
-	/**
-	 * @beforeClass
-	 */
-	public function backupMemoryInfoIniSetting() {
-		$this->iniSettingBeforeTest = ini_get('memory_limit');
-	}
+    /**
+     * @beforeClass
+     */
+    public function backupMemoryInfoIniSetting() {
+        $this->iniSettingBeforeTest = ini_get('memory_limit');
+    }
 
-	/**
-	 * @afterClass
-	 */
-	public function restoreMemoryInfoIniSetting() {
-		ini_set('memory_limit', $this->iniSettingBeforeTest);
-	}
+    /**
+     * @afterClass
+     */
+    public function restoreMemoryInfoIniSetting() {
+        ini_set('memory_limit', $this->iniSettingBeforeTest);
+    }
 
-	public static function getMemoryLimitTestData(): array {
-		return [
-			'unlimited' => ['-1', -1,],
-			'524288000 bytes' => ['524288000', 524288000,],
-			'500M' => ['500M', 524288000,],
-			'512000K' => ['512000K', 524288000,],
-			'2G' => ['2G', 2147483648,],
-		];
-	}
+    public static function getMemoryLimitTestData(): array {
+        return [
+            'unlimited' => ['-1', -1,],
+            '524288000 bytes' => ['524288000', 524288000,],
+            '500M' => ['500M', 524288000,],
+            '512000K' => ['512000K', 524288000,],
+            '2G' => ['2G', 2147483648,],
+        ];
+    }
 
-	/**
-	 * Tests that getMemoryLimit works as expected.
-	 *
-	 * @param string $iniValue The "memory_limit" ini data.
-	 * @param int|float $expected The expected detected memory limit.
-	 * @dataProvider getMemoryLimitTestData
-	 */
-	public function testMemoryLimit(string $iniValue, int|float $expected): void {
-		ini_set('memory_limit', $iniValue);
-		$memoryInfo = new MemoryInfo();
-		self::assertEquals($expected, $memoryInfo->getMemoryLimit());
-	}
+    /**
+     * Tests that getMemoryLimit works as expected.
+     *
+     * @param string $iniValue The "memory_limit" ini data.
+     * @param int|float $expected The expected detected memory limit.
+     * @dataProvider getMemoryLimitTestData
+     */
+    public function testMemoryLimit(string $iniValue, int|float $expected): void {
+        ini_set('memory_limit', $iniValue);
+        $memoryInfo = new MemoryInfo();
+        self::assertEquals($expected, $memoryInfo->getMemoryLimit());
+    }
 
-	public static function getSufficientMemoryTestData(): array {
-		return [
-			'unlimited' => [-1, true,],
-			'512M' => [512 * 1024 * 1024, true,],
-			'1G' => [1024 * 1024 * 1024, true,],
-			'256M' => [256 * 1024 * 1024, false,],
-		];
-	}
+    public static function getSufficientMemoryTestData(): array {
+        return [
+            'unlimited' => [-1, true,],
+            '512M' => [512 * 1024 * 1024, true,],
+            '1G' => [1024 * 1024 * 1024, true,],
+            '256M' => [256 * 1024 * 1024, false,],
+        ];
+    }
 
-	/**
-	 * Tests that isMemoryLimitSufficient returns the correct values.
-	 *
-	 * @param int $memoryLimit The memory limit
-	 * @param bool $expected If the memory limit is sufficient.
-	 * @dataProvider getSufficientMemoryTestData
-	 */
-	public function testIsMemoryLimitSufficient(int $memoryLimit, bool $expected): void {
-		/* @var MemoryInfo|MockObject $memoryInfo */
-		$memoryInfo = $this->getMockBuilder(MemoryInfo::class)
-			->onlyMethods(['getMemoryLimit',])
-			->getMock();
+    /**
+     * Tests that isMemoryLimitSufficient returns the correct values.
+     *
+     * @param int $memoryLimit The memory limit
+     * @param bool $expected If the memory limit is sufficient.
+     * @dataProvider getSufficientMemoryTestData
+     */
+    public function testIsMemoryLimitSufficient(int $memoryLimit, bool $expected): void {
+        /* @var MemoryInfo|MockObject $memoryInfo */
+        $memoryInfo = $this->getMockBuilder(MemoryInfo::class)
+            ->onlyMethods(['getMemoryLimit',])
+            ->getMock();
 
-		$memoryInfo
-			->method('getMemoryLimit')
-			->willReturn($memoryLimit);
+        $memoryInfo
+            ->method('getMemoryLimit')
+            ->willReturn($memoryLimit);
 
-		$isMemoryLimitSufficient = $memoryInfo->isMemoryLimitSufficient();
-		self::assertEquals($expected, $isMemoryLimitSufficient);
-	}
+        $isMemoryLimitSufficient = $memoryInfo->isMemoryLimitSufficient();
+        self::assertEquals($expected, $isMemoryLimitSufficient);
+    }
 }

Spacing +11 added lines, -11 removed lines

@@ -39,11 +39,11 @@ 
 
 	public static function getMemoryLimitTestData(): array {
 		return [
-			'unlimited' => ['-1', -1,],
-			'524288000 bytes' => ['524288000', 524288000,],
-			'500M' => ['500M', 524288000,],
-			'512000K' => ['512000K', 524288000,],
-			'2G' => ['2G', 2147483648,],
+			'unlimited' => ['-1', -1, ],
+			'524288000 bytes' => ['524288000', 524288000, ],
+			'500M' => ['500M', 524288000, ],
+			'512000K' => ['512000K', 524288000, ],
+			'2G' => ['2G', 2147483648, ],
 		];
 	}
 
@@ -54,7 +54,7 @@ 
 	 * @param int|float $expected The expected detected memory limit.
 	 * @dataProvider getMemoryLimitTestData
 	 */
-	public function testMemoryLimit(string $iniValue, int|float $expected): void {
+	public function testMemoryLimit(string $iniValue, int | float $expected): void {
 		ini_set('memory_limit', $iniValue);
 		$memoryInfo = new MemoryInfo();
 		self::assertEquals($expected, $memoryInfo->getMemoryLimit());
@@ -62,10 +62,10 @@ 
 
 	public static function getSufficientMemoryTestData(): array {
 		return [
-			'unlimited' => [-1, true,],
-			'512M' => [512 * 1024 * 1024, true,],
-			'1G' => [1024 * 1024 * 1024, true,],
-			'256M' => [256 * 1024 * 1024, false,],
+			'unlimited' => [-1, true, ],
+			'512M' => [512 * 1024 * 1024, true, ],
+			'1G' => [1024 * 1024 * 1024, true, ],
+			'256M' => [256 * 1024 * 1024, false, ],
 		];
 	}
 
@@ -79,7 +79,7 @@ 
 	public function testIsMemoryLimitSufficient(int $memoryLimit, bool $expected): void {
 		/* @var MemoryInfo|MockObject $memoryInfo */
 		$memoryInfo = $this->getMockBuilder(MemoryInfo::class)
-			->onlyMethods(['getMemoryLimit',])
+			->onlyMethods(['getMemoryLimit', ])
 			->getMock();
 
 		$memoryInfo

tests/lib/Group/ManagerTest.php

Indentation +952 added lines, -952 removed lines

@@ -31,956 +31,956 @@
 }
 
 class ManagerTest extends TestCase {
-	/** @var Manager|MockObject */
-	protected $userManager;
-	/** @var IEventDispatcher|MockObject */
-	protected $dispatcher;
-	/** @var LoggerInterface|MockObject */
-	protected $logger;
-	/** @var ICacheFactory|MockObject */
-	private $cache;
-	/** @var IRemoteAddress|MockObject */
-	private $remoteIpAddress;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->userManager = $this->createMock(Manager::class);
-		$this->dispatcher = $this->createMock(IEventDispatcher::class);
-		$this->logger = $this->createMock(LoggerInterface::class);
-		$this->cache = $this->createMock(ICacheFactory::class);
-
-		$this->remoteIpAddress = $this->createMock(IRemoteAddress::class);
-		$this->remoteIpAddress->method('allowsAdminActions')->willReturn(true);
-	}
-
-	private function getTestUser($userId) {
-		$mockUser = $this->createMock(IUser::class);
-		$mockUser->expects($this->any())
-			->method('getUID')
-			->willReturn($userId);
-		$mockUser->expects($this->any())
-			->method('getDisplayName')
-			->willReturn($userId);
-		return $mockUser;
-	}
-
-	/**
-	 * @param null|int $implementedActions
-	 * @return \PHPUnit\Framework\MockObject\MockObject
-	 */
-	private function getTestBackend($implementedActions = null) {
-		if ($implementedActions === null) {
-			$implementedActions =
-				GroupInterface::ADD_TO_GROUP |
-				GroupInterface::REMOVE_FROM_GOUP |
-				GroupInterface::COUNT_USERS |
-				GroupInterface::CREATE_GROUP |
-				GroupInterface::DELETE_GROUP;
-		}
-		// need to declare it this way due to optional methods
-		// thanks to the implementsActions logic
-		$backend = $this->getMockBuilder(TestBackend::class)
-			->disableOriginalConstructor()
-			->onlyMethods([
-				'getGroupDetails',
-				'implementsActions',
-				'getUserGroups',
-				'inGroup',
-				'getGroups',
-				'groupExists',
-				'groupsExists',
-				'usersInGroup',
-				'createGroup',
-				'addToGroup',
-				'removeFromGroup',
-				'searchInGroup',
-			])
-			->getMock();
-		$backend->expects($this->any())
-			->method('implementsActions')
-			->willReturnCallback(function ($actions) use ($implementedActions) {
-				return (bool)($actions & $implementedActions);
-			});
-		return $backend;
-	}
-
-	public function testGet(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$backend->expects($this->any())
-			->method('groupExists')
-			->with('group1')
-			->willReturn(true);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$group = $manager->get('group1');
-		$this->assertNotNull($group);
-		$this->assertEquals('group1', $group->getGID());
-	}
-
-	public function testGetNoBackend(): void {
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-
-		$this->assertNull($manager->get('group1'));
-	}
-
-	public function testGetNotExists(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$backend->expects($this->once())
-			->method('groupExists')
-			->with('group1')
-			->willReturn(false);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$this->assertNull($manager->get('group1'));
-	}
-
-	public function testGetDeleted(): void {
-		$backend = new \Test\Util\Group\Dummy();
-		$backend->createGroup('group1');
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$group = $manager->get('group1');
-		$group->delete();
-		$this->assertNull($manager->get('group1'));
-	}
-
-	public function testGetMultipleBackends(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend1
-		 */
-		$backend1 = $this->getTestBackend();
-		$backend1->expects($this->any())
-			->method('groupExists')
-			->with('group1')
-			->willReturn(false);
-
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend2
-		 */
-		$backend2 = $this->getTestBackend();
-		$backend2->expects($this->any())
-			->method('groupExists')
-			->with('group1')
-			->willReturn(true);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend1);
-		$manager->addBackend($backend2);
-
-		$group = $manager->get('group1');
-		$this->assertNotNull($group);
-		$this->assertEquals('group1', $group->getGID());
-	}
-
-	public function testCreate(): void {
-		/** @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend */
-		$backendGroupCreated = false;
-		$backend = $this->getTestBackend();
-		$backend->expects($this->any())
-			->method('groupExists')
-			->with('group1')
-			->willReturnCallback(function () use (&$backendGroupCreated) {
-				return $backendGroupCreated;
-			});
-		$backend->expects($this->once())
-			->method('createGroup')
-			->willReturnCallback(function () use (&$backendGroupCreated) {
-				$backendGroupCreated = true;
-				return true;
-			});
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$group = $manager->createGroup('group1');
-		$this->assertEquals('group1', $group->getGID());
-	}
-
-	public function testCreateFailure(): void {
-		/** @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend */
-		$backendGroupCreated = false;
-		$backend = $this->getTestBackend(
-			GroupInterface::ADD_TO_GROUP |
-			GroupInterface::REMOVE_FROM_GOUP |
-			GroupInterface::COUNT_USERS |
-			GroupInterface::CREATE_GROUP |
-			GroupInterface::DELETE_GROUP |
-			GroupInterface::GROUP_DETAILS
-		);
-		$backend->expects($this->any())
-			->method('groupExists')
-			->with('group1')
-			->willReturn(false);
-		$backend->expects($this->once())
-			->method('createGroup')
-			->willReturn(false);
-		$backend->expects($this->once())
-			->method('getGroupDetails')
-			->willReturn([]);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$group = $manager->createGroup('group1');
-		$this->assertEquals(null, $group);
-	}
-
-	public function testCreateTooLong(): void {
-		/** @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend */
-		$backendGroupCreated = false;
-		$backend = $this->getTestBackend(
-			GroupInterface::ADD_TO_GROUP |
-			GroupInterface::REMOVE_FROM_GOUP |
-			GroupInterface::COUNT_USERS |
-			GroupInterface::CREATE_GROUP |
-			GroupInterface::DELETE_GROUP |
-			GroupInterface::GROUP_DETAILS
-		);
-		$groupName = str_repeat('x', 256);
-		$backend->expects($this->any())
-			->method('groupExists')
-			->with($groupName)
-			->willReturn(false);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$this->expectException(\Exception::class);
-		$group = $manager->createGroup($groupName);
-	}
-
-	public function testCreateExists(): void {
-		/** @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend */
-		$backend = $this->getTestBackend();
-		$backend->expects($this->any())
-			->method('groupExists')
-			->with('group1')
-			->willReturn(true);
-		$backend->expects($this->never())
-			->method('createGroup');
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$group = $manager->createGroup('group1');
-		$this->assertEquals('group1', $group->getGID());
-	}
-
-	public function testSearch(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$backend->expects($this->once())
-			->method('getGroups')
-			->with('1')
-			->willReturn(['group1']);
-		$backend->expects($this->once())
-			->method('getGroupDetails')
-			->willReturnMap([
-				['group1', ['displayName' => 'group1']],
-			]);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$groups = $manager->search('1');
-		$this->assertCount(1, $groups);
-		$group1 = reset($groups);
-		$this->assertEquals('group1', $group1->getGID());
-	}
-
-	public function testSearchMultipleBackends(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend1
-		 */
-		$backend1 = $this->getTestBackend();
-		$backend1->expects($this->once())
-			->method('getGroups')
-			->with('1')
-			->willReturn(['group1']);
-		$backend1->expects($this->any())
-			->method('getGroupDetails')
-			->willReturnMap([
-				['group1', ['displayName' => 'group1']],
-				['group12', []],
-			]);
-
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend2
-		 */
-		$backend2 = $this->getTestBackend();
-		$backend2->expects($this->once())
-			->method('getGroups')
-			->with('1')
-			->willReturn(['group12', 'group1']);
-		$backend2->expects($this->any())
-			->method('getGroupDetails')
-			->willReturnMap([
-				['group12', ['displayName' => 'group12']],
-				['group1', ['displayName' => 'group1']],
-			]);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend1);
-		$manager->addBackend($backend2);
-
-		$groups = $manager->search('1');
-		$this->assertCount(2, $groups);
-		$group1 = reset($groups);
-		$group12 = next($groups);
-		$this->assertEquals('group1', $group1->getGID());
-		$this->assertEquals('group12', $group12->getGID());
-	}
-
-	public function testSearchMultipleBackendsLimitAndOffset(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend1
-		 */
-		$backend1 = $this->getTestBackend();
-		$backend1->expects($this->once())
-			->method('getGroups')
-			->with('1', 2, 1)
-			->willReturn(['group1']);
-		$backend1->expects($this->any())
-			->method('getGroupDetails')
-			->willReturnMap([
-				[1, []],
-				[2, []],
-				['group1', ['displayName' => 'group1']],
-				['group12', []],
-			]);
-
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend2
-		 */
-		$backend2 = $this->getTestBackend();
-		$backend2->expects($this->once())
-			->method('getGroups')
-			->with('1', 2, 1)
-			->willReturn(['group12']);
-		$backend2->expects($this->any())
-			->method('getGroupDetails')
-			->willReturnMap([
-				[1, []],
-				[2, []],
-				['group1', []],
-				['group12', ['displayName' => 'group12']],
-			]);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend1);
-		$manager->addBackend($backend2);
-
-		$groups = $manager->search('1', 2, 1);
-		$this->assertCount(2, $groups);
-		$group1 = reset($groups);
-		$group12 = next($groups);
-		$this->assertEquals('group1', $group1->getGID());
-		$this->assertEquals('group12', $group12->getGID());
-	}
-
-	public function testSearchResultExistsButGroupDoesNot(): void {
-		/** @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend */
-		$backend = $this->createMock(Database::class);
-		$backend->expects($this->once())
-			->method('getGroups')
-			->with('1')
-			->willReturn(['group1']);
-		$backend->expects($this->never())
-			->method('groupExists');
-		$backend->expects($this->once())
-			->method('getGroupsDetails')
-			->with(['group1'])
-			->willReturn([]);
-
-		/** @var \OC\User\Manager $userManager */
-		$userManager = $this->createMock(Manager::class);
-
-		$manager = new \OC\Group\Manager($userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$groups = $manager->search('1');
-		$this->assertEmpty($groups);
-	}
-
-	public function testGetUserGroups(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$backend->expects($this->once())
-			->method('getUserGroups')
-			->with('user1')
-			->willReturn(['group1']);
-		$backend->expects($this->any())
-			->method('groupExists')
-			->with('group1')
-			->willReturn(true);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$groups = $manager->getUserGroups($this->getTestUser('user1'));
-		$this->assertCount(1, $groups);
-		$group1 = reset($groups);
-		$this->assertEquals('group1', $group1->getGID());
-	}
-
-	public function testGetUserGroupIds(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$backend->method('getUserGroups')
-			->with('myUID')
-			->willReturn(['123', 'abc']);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		/** @var \OC\User\User|\PHPUnit\Framework\MockObject\MockObject $user */
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')
-			->willReturn('myUID');
-
-		$groups = $manager->getUserGroupIds($user);
-		$this->assertCount(2, $groups);
-
-		foreach ($groups as $group) {
-			$this->assertIsString($group);
-		}
-	}
-
-	public function testGetUserGroupsWithDeletedGroup(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->createMock(Database::class);
-		$backend->expects($this->once())
-			->method('getUserGroups')
-			->with('user1')
-			->willReturn(['group1']);
-		$backend->expects($this->any())
-			->method('groupExists')
-			->with('group1')
-			->willReturn(false);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		/** @var \OC\User\User|\PHPUnit\Framework\MockObject\MockObject $user */
-		$user = $this->createMock(IUser::class);
-		$user->expects($this->atLeastOnce())
-			->method('getUID')
-			->willReturn('user1');
-
-		$groups = $manager->getUserGroups($user);
-		$this->assertEmpty($groups);
-	}
-
-	public function testInGroup(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$backend->expects($this->once())
-			->method('getUserGroups')
-			->with('user1')
-			->willReturn(['group1', 'admin', 'group2']);
-		$backend->expects($this->any())
-			->method('groupExists')
-			->willReturn(true);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$this->assertTrue($manager->isInGroup('user1', 'group1'));
-	}
-
-	public function testIsAdmin(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$backend->expects($this->once())
-			->method('getUserGroups')
-			->with('user1')
-			->willReturn(['group1', 'admin', 'group2']);
-		$backend->expects($this->any())
-			->method('groupExists')
-			->willReturn(true);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$this->assertTrue($manager->isAdmin('user1'));
-	}
-
-	public function testNotAdmin(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$backend->expects($this->once())
-			->method('getUserGroups')
-			->with('user1')
-			->willReturn(['group1', 'group2']);
-		$backend->expects($this->any())
-			->method('groupExists')
-			->willReturn(true);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$this->assertFalse($manager->isAdmin('user1'));
-	}
-
-	public function testGetUserGroupsMultipleBackends(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend1
-		 */
-		$backend1 = $this->getTestBackend();
-		$backend1->expects($this->once())
-			->method('getUserGroups')
-			->with('user1')
-			->willReturn(['group1']);
-		$backend1->expects($this->any())
-			->method('groupExists')
-			->willReturn(true);
-
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend2
-		 */
-		$backend2 = $this->getTestBackend();
-		$backend2->expects($this->once())
-			->method('getUserGroups')
-			->with('user1')
-			->willReturn(['group1', 'group2']);
-		$backend1->expects($this->any())
-			->method('groupExists')
-			->willReturn(true);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend1);
-		$manager->addBackend($backend2);
-
-		$groups = $manager->getUserGroups($this->getTestUser('user1'));
-		$this->assertCount(2, $groups);
-		$group1 = reset($groups);
-		$group2 = next($groups);
-		$this->assertEquals('group1', $group1->getGID());
-		$this->assertEquals('group2', $group2->getGID());
-	}
-
-	public function testDisplayNamesInGroupWithOneUserBackend(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$backend->expects($this->exactly(1))
-			->method('groupExists')
-			->with('testgroup')
-			->willReturn(true);
-
-		$backend->expects($this->any())
-			->method('inGroup')
-			->willReturnCallback(function ($uid, $gid) {
-				switch ($uid) {
-					case 'user1': return false;
-					case 'user2': return true;
-					case 'user3': return false;
-					case 'user33': return true;
-					default:
-						return null;
-				}
-			});
-
-		$this->userManager->expects($this->any())
-			->method('searchDisplayName')
-			->with('user3')
-			->willReturnCallback(function ($search, $limit, $offset) {
-				switch ($offset) {
-					case 0: return ['user3' => $this->getTestUser('user3'),
-						'user33' => $this->getTestUser('user33')];
-					case 2: return [];
-				}
-				return null;
-			});
-		$this->userManager->expects($this->any())
-			->method('get')
-			->willReturnCallback(function ($uid) {
-				switch ($uid) {
-					case 'user1': return $this->getTestUser('user1');
-					case 'user2': return $this->getTestUser('user2');
-					case 'user3': return $this->getTestUser('user3');
-					case 'user33': return $this->getTestUser('user33');
-					default:
-						return null;
-				}
-			});
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$users = $manager->displayNamesInGroup('testgroup', 'user3');
-		$this->assertCount(1, $users);
-		$this->assertFalse(isset($users['user1']));
-		$this->assertFalse(isset($users['user2']));
-		$this->assertFalse(isset($users['user3']));
-		$this->assertTrue(isset($users['user33']));
-	}
-
-	public function testDisplayNamesInGroupWithOneUserBackendWithLimitSpecified(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$backend->expects($this->exactly(1))
-			->method('groupExists')
-			->with('testgroup')
-			->willReturn(true);
-
-		$backend->expects($this->any())
-			->method('inGroup')
-			->willReturnCallback(function ($uid, $gid) {
-				switch ($uid) {
-					case 'user1': return false;
-					case 'user2': return true;
-					case 'user3': return false;
-					case 'user33': return true;
-					case 'user333': return true;
-					default:
-						return null;
-				}
-			});
-
-		$this->userManager->expects($this->any())
-			->method('searchDisplayName')
-			->with('user3')
-			->willReturnCallback(function ($search, $limit, $offset) {
-				switch ($offset) {
-					case 0: return ['user3' => $this->getTestUser('user3'),
-						'user33' => $this->getTestUser('user33')];
-					case 2: return ['user333' => $this->getTestUser('user333')];
-				}
-				return null;
-			});
-		$this->userManager->expects($this->any())
-			->method('get')
-			->willReturnCallback(function ($uid) {
-				switch ($uid) {
-					case 'user1': return $this->getTestUser('user1');
-					case 'user2': return $this->getTestUser('user2');
-					case 'user3': return $this->getTestUser('user3');
-					case 'user33': return $this->getTestUser('user33');
-					case 'user333': return $this->getTestUser('user333');
-					default:
-						return null;
-				}
-			});
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$users = $manager->displayNamesInGroup('testgroup', 'user3', 1);
-		$this->assertCount(1, $users);
-		$this->assertFalse(isset($users['user1']));
-		$this->assertFalse(isset($users['user2']));
-		$this->assertFalse(isset($users['user3']));
-		$this->assertTrue(isset($users['user33']));
-		$this->assertFalse(isset($users['user333']));
-	}
-
-	public function testDisplayNamesInGroupWithOneUserBackendWithLimitAndOffsetSpecified(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$backend->expects($this->exactly(1))
-			->method('groupExists')
-			->with('testgroup')
-			->willReturn(true);
-
-		$backend->expects($this->any())
-			->method('inGroup')
-			->willReturnCallback(function ($uid) {
-				switch ($uid) {
-					case 'user1': return false;
-					case 'user2': return true;
-					case 'user3': return false;
-					case 'user33': return true;
-					case 'user333': return true;
-					default:
-						return null;
-				}
-			});
-
-		$this->userManager->expects($this->any())
-			->method('searchDisplayName')
-			->with('user3')
-			->willReturnCallback(function ($search, $limit, $offset) {
-				switch ($offset) {
-					case 0:
-						return [
-							'user3' => $this->getTestUser('user3'),
-							'user33' => $this->getTestUser('user33'),
-							'user333' => $this->getTestUser('user333')
-						];
-				}
-				return null;
-			});
-		$this->userManager->expects($this->any())
-			->method('get')
-			->willReturnCallback(function ($uid) {
-				switch ($uid) {
-					case 'user1': return $this->getTestUser('user1');
-					case 'user2': return $this->getTestUser('user2');
-					case 'user3': return $this->getTestUser('user3');
-					case 'user33': return $this->getTestUser('user33');
-					case 'user333': return $this->getTestUser('user333');
-					default:
-						return null;
-				}
-			});
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$users = $manager->displayNamesInGroup('testgroup', 'user3', 1, 1);
-		$this->assertCount(1, $users);
-		$this->assertFalse(isset($users['user1']));
-		$this->assertFalse(isset($users['user2']));
-		$this->assertFalse(isset($users['user3']));
-		$this->assertFalse(isset($users['user33']));
-		$this->assertTrue(isset($users['user333']));
-	}
-
-	public function testDisplayNamesInGroupWithOneUserBackendAndSearchEmpty(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$backend->expects($this->exactly(1))
-			->method('groupExists')
-			->with('testgroup')
-			->willReturn(true);
-
-		$backend->expects($this->once())
-			->method('searchInGroup')
-			->with('testgroup', '', -1, 0)
-			->willReturn(['user2' => $this->getTestUser('user2'), 'user33' => $this->getTestUser('user33')]);
-
-		$this->userManager->expects($this->never())->method('get');
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$users = $manager->displayNamesInGroup('testgroup', '');
-		$this->assertCount(2, $users);
-		$this->assertFalse(isset($users['user1']));
-		$this->assertTrue(isset($users['user2']));
-		$this->assertFalse(isset($users['user3']));
-		$this->assertTrue(isset($users['user33']));
-	}
-
-	public function testDisplayNamesInGroupWithOneUserBackendAndSearchEmptyAndLimitSpecified(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$backend->expects($this->exactly(1))
-			->method('groupExists')
-			->with('testgroup')
-			->willReturn(true);
-
-		$backend->expects($this->once())
-			->method('searchInGroup')
-			->with('testgroup', '', 1, 0)
-			->willReturn([new User('user2', null, $this->dispatcher)]);
-
-		$this->userManager->expects($this->never())->method('get');
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$users = $manager->displayNamesInGroup('testgroup', '', 1);
-		$this->assertCount(1, $users);
-		$this->assertFalse(isset($users['user1']));
-		$this->assertTrue(isset($users['user2']));
-		$this->assertFalse(isset($users['user3']));
-		$this->assertFalse(isset($users['user33']));
-	}
-
-	public function testDisplayNamesInGroupWithOneUserBackendAndSearchEmptyAndLimitAndOffsetSpecified(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$backend->expects($this->exactly(1))
-			->method('groupExists')
-			->with('testgroup')
-			->willReturn(true);
-
-		$backend->expects($this->once())
-			->method('searchInGroup')
-			->with('testgroup', '', 1, 1)
-			->willReturn(['user33' => $this->getTestUser('user33')]);
-
-		$this->userManager->expects($this->never())->method('get');
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$users = $manager->displayNamesInGroup('testgroup', '', 1, 1);
-		$this->assertCount(1, $users);
-		$this->assertFalse(isset($users['user1']));
-		$this->assertFalse(isset($users['user2']));
-		$this->assertFalse(isset($users['user3']));
-		$this->assertTrue(isset($users['user33']));
-	}
-
-	public function testGetUserGroupsWithAddUser(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$expectedGroups = [];
-		$backend->expects($this->any())
-			->method('getUserGroups')
-			->with('user1')
-			->willReturnCallback(function () use (&$expectedGroups) {
-				return $expectedGroups;
-			});
-		$backend->expects($this->any())
-			->method('groupExists')
-			->with('group1')
-			->willReturn(true);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		// prime cache
-		$user1 = $this->getTestUser('user1');
-		$groups = $manager->getUserGroups($user1);
-		$this->assertEquals([], $groups);
-
-		// add user
-		$group = $manager->get('group1');
-		$group->addUser($user1);
-		$expectedGroups[] = 'group1';
-
-		// check result
-		$groups = $manager->getUserGroups($user1);
-		$this->assertCount(1, $groups);
-		$group1 = reset($groups);
-		$this->assertEquals('group1', $group1->getGID());
-	}
-
-	public function testGetUserGroupsWithRemoveUser(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$expectedGroups = ['group1'];
-		$backend->expects($this->any())
-			->method('getUserGroups')
-			->with('user1')
-			->willReturnCallback(function () use (&$expectedGroups) {
-				return $expectedGroups;
-			});
-		$backend->expects($this->any())
-			->method('groupExists')
-			->with('group1')
-			->willReturn(true);
-		$backend->expects($this->once())
-			->method('inGroup')
-			->willReturn(true);
-		$backend->expects($this->once())
-			->method('removeFromGroup')
-			->willReturn(true);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		// prime cache
-		$user1 = $this->getTestUser('user1');
-		$groups = $manager->getUserGroups($user1);
-		$this->assertCount(1, $groups);
-		$group1 = reset($groups);
-		$this->assertEquals('group1', $group1->getGID());
-
-		// remove user
-		$group = $manager->get('group1');
-		$group->removeUser($user1);
-		$expectedGroups = [];
-
-		// check result
-		$groups = $manager->getUserGroups($user1);
-		$this->assertEquals($expectedGroups, $groups);
-	}
-
-	public function testGetUserIdGroups(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend();
-		$backend->expects($this->any())
-			->method('getUserGroups')
-			->with('user1')
-			->willReturn(null);
-
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		$groups = $manager->getUserIdGroups('user1');
-		$this->assertEquals([], $groups);
-	}
-
-	public function testGroupDisplayName(): void {
-		/**
-		 * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
-		 */
-		$backend = $this->getTestBackend(
-			GroupInterface::ADD_TO_GROUP |
-			GroupInterface::REMOVE_FROM_GOUP |
-			GroupInterface::COUNT_USERS |
-			GroupInterface::CREATE_GROUP |
-			GroupInterface::DELETE_GROUP |
-			GroupInterface::GROUP_DETAILS
-		);
-		$backend->expects($this->any())
-			->method('getGroupDetails')
-			->willReturnMap([
-				['group1', ['gid' => 'group1', 'displayName' => 'Group One']],
-				['group2', ['gid' => 'group2']],
-			]);
-		$manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
-		$manager->addBackend($backend);
-
-		// group with display name
-		$group = $manager->get('group1');
-		$this->assertNotNull($group);
-		$this->assertEquals('group1', $group->getGID());
-		$this->assertEquals('Group One', $group->getDisplayName());
-
-		// group without display name
-		$group = $manager->get('group2');
-		$this->assertNotNull($group);
-		$this->assertEquals('group2', $group->getGID());
-		$this->assertEquals('group2', $group->getDisplayName());
-	}
+    /** @var Manager|MockObject */
+    protected $userManager;
+    /** @var IEventDispatcher|MockObject */
+    protected $dispatcher;
+    /** @var LoggerInterface|MockObject */
+    protected $logger;
+    /** @var ICacheFactory|MockObject */
+    private $cache;
+    /** @var IRemoteAddress|MockObject */
+    private $remoteIpAddress;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->userManager = $this->createMock(Manager::class);
+        $this->dispatcher = $this->createMock(IEventDispatcher::class);
+        $this->logger = $this->createMock(LoggerInterface::class);
+        $this->cache = $this->createMock(ICacheFactory::class);
+
+        $this->remoteIpAddress = $this->createMock(IRemoteAddress::class);
+        $this->remoteIpAddress->method('allowsAdminActions')->willReturn(true);
+    }
+
+    private function getTestUser($userId) {
+        $mockUser = $this->createMock(IUser::class);
+        $mockUser->expects($this->any())
+            ->method('getUID')
+            ->willReturn($userId);
+        $mockUser->expects($this->any())
+            ->method('getDisplayName')
+            ->willReturn($userId);
+        return $mockUser;
+    }
+
+    /**
+     * @param null|int $implementedActions
+     * @return \PHPUnit\Framework\MockObject\MockObject
+     */
+    private function getTestBackend($implementedActions = null) {
+        if ($implementedActions === null) {
+            $implementedActions =
+                GroupInterface::ADD_TO_GROUP |
+                GroupInterface::REMOVE_FROM_GOUP |
+                GroupInterface::COUNT_USERS |
+                GroupInterface::CREATE_GROUP |
+                GroupInterface::DELETE_GROUP;
+        }
+        // need to declare it this way due to optional methods
+        // thanks to the implementsActions logic
+        $backend = $this->getMockBuilder(TestBackend::class)
+            ->disableOriginalConstructor()
+            ->onlyMethods([
+                'getGroupDetails',
+                'implementsActions',
+                'getUserGroups',
+                'inGroup',
+                'getGroups',
+                'groupExists',
+                'groupsExists',
+                'usersInGroup',
+                'createGroup',
+                'addToGroup',
+                'removeFromGroup',
+                'searchInGroup',
+            ])
+            ->getMock();
+        $backend->expects($this->any())
+            ->method('implementsActions')
+            ->willReturnCallback(function ($actions) use ($implementedActions) {
+                return (bool)($actions & $implementedActions);
+            });
+        return $backend;
+    }
+
+    public function testGet(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $backend->expects($this->any())
+            ->method('groupExists')
+            ->with('group1')
+            ->willReturn(true);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $group = $manager->get('group1');
+        $this->assertNotNull($group);
+        $this->assertEquals('group1', $group->getGID());
+    }
+
+    public function testGetNoBackend(): void {
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+
+        $this->assertNull($manager->get('group1'));
+    }
+
+    public function testGetNotExists(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $backend->expects($this->once())
+            ->method('groupExists')
+            ->with('group1')
+            ->willReturn(false);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $this->assertNull($manager->get('group1'));
+    }
+
+    public function testGetDeleted(): void {
+        $backend = new \Test\Util\Group\Dummy();
+        $backend->createGroup('group1');
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $group = $manager->get('group1');
+        $group->delete();
+        $this->assertNull($manager->get('group1'));
+    }
+
+    public function testGetMultipleBackends(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend1
+         */
+        $backend1 = $this->getTestBackend();
+        $backend1->expects($this->any())
+            ->method('groupExists')
+            ->with('group1')
+            ->willReturn(false);
+
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend2
+         */
+        $backend2 = $this->getTestBackend();
+        $backend2->expects($this->any())
+            ->method('groupExists')
+            ->with('group1')
+            ->willReturn(true);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend1);
+        $manager->addBackend($backend2);
+
+        $group = $manager->get('group1');
+        $this->assertNotNull($group);
+        $this->assertEquals('group1', $group->getGID());
+    }
+
+    public function testCreate(): void {
+        /** @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend */
+        $backendGroupCreated = false;
+        $backend = $this->getTestBackend();
+        $backend->expects($this->any())
+            ->method('groupExists')
+            ->with('group1')
+            ->willReturnCallback(function () use (&$backendGroupCreated) {
+                return $backendGroupCreated;
+            });
+        $backend->expects($this->once())
+            ->method('createGroup')
+            ->willReturnCallback(function () use (&$backendGroupCreated) {
+                $backendGroupCreated = true;
+                return true;
+            });
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $group = $manager->createGroup('group1');
+        $this->assertEquals('group1', $group->getGID());
+    }
+
+    public function testCreateFailure(): void {
+        /** @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend */
+        $backendGroupCreated = false;
+        $backend = $this->getTestBackend(
+            GroupInterface::ADD_TO_GROUP |
+            GroupInterface::REMOVE_FROM_GOUP |
+            GroupInterface::COUNT_USERS |
+            GroupInterface::CREATE_GROUP |
+            GroupInterface::DELETE_GROUP |
+            GroupInterface::GROUP_DETAILS
+        );
+        $backend->expects($this->any())
+            ->method('groupExists')
+            ->with('group1')
+            ->willReturn(false);
+        $backend->expects($this->once())
+            ->method('createGroup')
+            ->willReturn(false);
+        $backend->expects($this->once())
+            ->method('getGroupDetails')
+            ->willReturn([]);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $group = $manager->createGroup('group1');
+        $this->assertEquals(null, $group);
+    }
+
+    public function testCreateTooLong(): void {
+        /** @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend */
+        $backendGroupCreated = false;
+        $backend = $this->getTestBackend(
+            GroupInterface::ADD_TO_GROUP |
+            GroupInterface::REMOVE_FROM_GOUP |
+            GroupInterface::COUNT_USERS |
+            GroupInterface::CREATE_GROUP |
+            GroupInterface::DELETE_GROUP |
+            GroupInterface::GROUP_DETAILS
+        );
+        $groupName = str_repeat('x', 256);
+        $backend->expects($this->any())
+            ->method('groupExists')
+            ->with($groupName)
+            ->willReturn(false);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $this->expectException(\Exception::class);
+        $group = $manager->createGroup($groupName);
+    }
+
+    public function testCreateExists(): void {
+        /** @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend */
+        $backend = $this->getTestBackend();
+        $backend->expects($this->any())
+            ->method('groupExists')
+            ->with('group1')
+            ->willReturn(true);
+        $backend->expects($this->never())
+            ->method('createGroup');
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $group = $manager->createGroup('group1');
+        $this->assertEquals('group1', $group->getGID());
+    }
+
+    public function testSearch(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $backend->expects($this->once())
+            ->method('getGroups')
+            ->with('1')
+            ->willReturn(['group1']);
+        $backend->expects($this->once())
+            ->method('getGroupDetails')
+            ->willReturnMap([
+                ['group1', ['displayName' => 'group1']],
+            ]);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $groups = $manager->search('1');
+        $this->assertCount(1, $groups);
+        $group1 = reset($groups);
+        $this->assertEquals('group1', $group1->getGID());
+    }
+
+    public function testSearchMultipleBackends(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend1
+         */
+        $backend1 = $this->getTestBackend();
+        $backend1->expects($this->once())
+            ->method('getGroups')
+            ->with('1')
+            ->willReturn(['group1']);
+        $backend1->expects($this->any())
+            ->method('getGroupDetails')
+            ->willReturnMap([
+                ['group1', ['displayName' => 'group1']],
+                ['group12', []],
+            ]);
+
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend2
+         */
+        $backend2 = $this->getTestBackend();
+        $backend2->expects($this->once())
+            ->method('getGroups')
+            ->with('1')
+            ->willReturn(['group12', 'group1']);
+        $backend2->expects($this->any())
+            ->method('getGroupDetails')
+            ->willReturnMap([
+                ['group12', ['displayName' => 'group12']],
+                ['group1', ['displayName' => 'group1']],
+            ]);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend1);
+        $manager->addBackend($backend2);
+
+        $groups = $manager->search('1');
+        $this->assertCount(2, $groups);
+        $group1 = reset($groups);
+        $group12 = next($groups);
+        $this->assertEquals('group1', $group1->getGID());
+        $this->assertEquals('group12', $group12->getGID());
+    }
+
+    public function testSearchMultipleBackendsLimitAndOffset(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend1
+         */
+        $backend1 = $this->getTestBackend();
+        $backend1->expects($this->once())
+            ->method('getGroups')
+            ->with('1', 2, 1)
+            ->willReturn(['group1']);
+        $backend1->expects($this->any())
+            ->method('getGroupDetails')
+            ->willReturnMap([
+                [1, []],
+                [2, []],
+                ['group1', ['displayName' => 'group1']],
+                ['group12', []],
+            ]);
+
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend2
+         */
+        $backend2 = $this->getTestBackend();
+        $backend2->expects($this->once())
+            ->method('getGroups')
+            ->with('1', 2, 1)
+            ->willReturn(['group12']);
+        $backend2->expects($this->any())
+            ->method('getGroupDetails')
+            ->willReturnMap([
+                [1, []],
+                [2, []],
+                ['group1', []],
+                ['group12', ['displayName' => 'group12']],
+            ]);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend1);
+        $manager->addBackend($backend2);
+
+        $groups = $manager->search('1', 2, 1);
+        $this->assertCount(2, $groups);
+        $group1 = reset($groups);
+        $group12 = next($groups);
+        $this->assertEquals('group1', $group1->getGID());
+        $this->assertEquals('group12', $group12->getGID());
+    }
+
+    public function testSearchResultExistsButGroupDoesNot(): void {
+        /** @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend */
+        $backend = $this->createMock(Database::class);
+        $backend->expects($this->once())
+            ->method('getGroups')
+            ->with('1')
+            ->willReturn(['group1']);
+        $backend->expects($this->never())
+            ->method('groupExists');
+        $backend->expects($this->once())
+            ->method('getGroupsDetails')
+            ->with(['group1'])
+            ->willReturn([]);
+
+        /** @var \OC\User\Manager $userManager */
+        $userManager = $this->createMock(Manager::class);
+
+        $manager = new \OC\Group\Manager($userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $groups = $manager->search('1');
+        $this->assertEmpty($groups);
+    }
+
+    public function testGetUserGroups(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $backend->expects($this->once())
+            ->method('getUserGroups')
+            ->with('user1')
+            ->willReturn(['group1']);
+        $backend->expects($this->any())
+            ->method('groupExists')
+            ->with('group1')
+            ->willReturn(true);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $groups = $manager->getUserGroups($this->getTestUser('user1'));
+        $this->assertCount(1, $groups);
+        $group1 = reset($groups);
+        $this->assertEquals('group1', $group1->getGID());
+    }
+
+    public function testGetUserGroupIds(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $backend->method('getUserGroups')
+            ->with('myUID')
+            ->willReturn(['123', 'abc']);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        /** @var \OC\User\User|\PHPUnit\Framework\MockObject\MockObject $user */
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')
+            ->willReturn('myUID');
+
+        $groups = $manager->getUserGroupIds($user);
+        $this->assertCount(2, $groups);
+
+        foreach ($groups as $group) {
+            $this->assertIsString($group);
+        }
+    }
+
+    public function testGetUserGroupsWithDeletedGroup(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->createMock(Database::class);
+        $backend->expects($this->once())
+            ->method('getUserGroups')
+            ->with('user1')
+            ->willReturn(['group1']);
+        $backend->expects($this->any())
+            ->method('groupExists')
+            ->with('group1')
+            ->willReturn(false);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        /** @var \OC\User\User|\PHPUnit\Framework\MockObject\MockObject $user */
+        $user = $this->createMock(IUser::class);
+        $user->expects($this->atLeastOnce())
+            ->method('getUID')
+            ->willReturn('user1');
+
+        $groups = $manager->getUserGroups($user);
+        $this->assertEmpty($groups);
+    }
+
+    public function testInGroup(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $backend->expects($this->once())
+            ->method('getUserGroups')
+            ->with('user1')
+            ->willReturn(['group1', 'admin', 'group2']);
+        $backend->expects($this->any())
+            ->method('groupExists')
+            ->willReturn(true);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $this->assertTrue($manager->isInGroup('user1', 'group1'));
+    }
+
+    public function testIsAdmin(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $backend->expects($this->once())
+            ->method('getUserGroups')
+            ->with('user1')
+            ->willReturn(['group1', 'admin', 'group2']);
+        $backend->expects($this->any())
+            ->method('groupExists')
+            ->willReturn(true);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $this->assertTrue($manager->isAdmin('user1'));
+    }
+
+    public function testNotAdmin(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $backend->expects($this->once())
+            ->method('getUserGroups')
+            ->with('user1')
+            ->willReturn(['group1', 'group2']);
+        $backend->expects($this->any())
+            ->method('groupExists')
+            ->willReturn(true);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $this->assertFalse($manager->isAdmin('user1'));
+    }
+
+    public function testGetUserGroupsMultipleBackends(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend1
+         */
+        $backend1 = $this->getTestBackend();
+        $backend1->expects($this->once())
+            ->method('getUserGroups')
+            ->with('user1')
+            ->willReturn(['group1']);
+        $backend1->expects($this->any())
+            ->method('groupExists')
+            ->willReturn(true);
+
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend2
+         */
+        $backend2 = $this->getTestBackend();
+        $backend2->expects($this->once())
+            ->method('getUserGroups')
+            ->with('user1')
+            ->willReturn(['group1', 'group2']);
+        $backend1->expects($this->any())
+            ->method('groupExists')
+            ->willReturn(true);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend1);
+        $manager->addBackend($backend2);
+
+        $groups = $manager->getUserGroups($this->getTestUser('user1'));
+        $this->assertCount(2, $groups);
+        $group1 = reset($groups);
+        $group2 = next($groups);
+        $this->assertEquals('group1', $group1->getGID());
+        $this->assertEquals('group2', $group2->getGID());
+    }
+
+    public function testDisplayNamesInGroupWithOneUserBackend(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $backend->expects($this->exactly(1))
+            ->method('groupExists')
+            ->with('testgroup')
+            ->willReturn(true);
+
+        $backend->expects($this->any())
+            ->method('inGroup')
+            ->willReturnCallback(function ($uid, $gid) {
+                switch ($uid) {
+                    case 'user1': return false;
+                    case 'user2': return true;
+                    case 'user3': return false;
+                    case 'user33': return true;
+                    default:
+                        return null;
+                }
+            });
+
+        $this->userManager->expects($this->any())
+            ->method('searchDisplayName')
+            ->with('user3')
+            ->willReturnCallback(function ($search, $limit, $offset) {
+                switch ($offset) {
+                    case 0: return ['user3' => $this->getTestUser('user3'),
+                        'user33' => $this->getTestUser('user33')];
+                    case 2: return [];
+                }
+                return null;
+            });
+        $this->userManager->expects($this->any())
+            ->method('get')
+            ->willReturnCallback(function ($uid) {
+                switch ($uid) {
+                    case 'user1': return $this->getTestUser('user1');
+                    case 'user2': return $this->getTestUser('user2');
+                    case 'user3': return $this->getTestUser('user3');
+                    case 'user33': return $this->getTestUser('user33');
+                    default:
+                        return null;
+                }
+            });
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $users = $manager->displayNamesInGroup('testgroup', 'user3');
+        $this->assertCount(1, $users);
+        $this->assertFalse(isset($users['user1']));
+        $this->assertFalse(isset($users['user2']));
+        $this->assertFalse(isset($users['user3']));
+        $this->assertTrue(isset($users['user33']));
+    }
+
+    public function testDisplayNamesInGroupWithOneUserBackendWithLimitSpecified(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $backend->expects($this->exactly(1))
+            ->method('groupExists')
+            ->with('testgroup')
+            ->willReturn(true);
+
+        $backend->expects($this->any())
+            ->method('inGroup')
+            ->willReturnCallback(function ($uid, $gid) {
+                switch ($uid) {
+                    case 'user1': return false;
+                    case 'user2': return true;
+                    case 'user3': return false;
+                    case 'user33': return true;
+                    case 'user333': return true;
+                    default:
+                        return null;
+                }
+            });
+
+        $this->userManager->expects($this->any())
+            ->method('searchDisplayName')
+            ->with('user3')
+            ->willReturnCallback(function ($search, $limit, $offset) {
+                switch ($offset) {
+                    case 0: return ['user3' => $this->getTestUser('user3'),
+                        'user33' => $this->getTestUser('user33')];
+                    case 2: return ['user333' => $this->getTestUser('user333')];
+                }
+                return null;
+            });
+        $this->userManager->expects($this->any())
+            ->method('get')
+            ->willReturnCallback(function ($uid) {
+                switch ($uid) {
+                    case 'user1': return $this->getTestUser('user1');
+                    case 'user2': return $this->getTestUser('user2');
+                    case 'user3': return $this->getTestUser('user3');
+                    case 'user33': return $this->getTestUser('user33');
+                    case 'user333': return $this->getTestUser('user333');
+                    default:
+                        return null;
+                }
+            });
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $users = $manager->displayNamesInGroup('testgroup', 'user3', 1);
+        $this->assertCount(1, $users);
+        $this->assertFalse(isset($users['user1']));
+        $this->assertFalse(isset($users['user2']));
+        $this->assertFalse(isset($users['user3']));
+        $this->assertTrue(isset($users['user33']));
+        $this->assertFalse(isset($users['user333']));
+    }
+
+    public function testDisplayNamesInGroupWithOneUserBackendWithLimitAndOffsetSpecified(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $backend->expects($this->exactly(1))
+            ->method('groupExists')
+            ->with('testgroup')
+            ->willReturn(true);
+
+        $backend->expects($this->any())
+            ->method('inGroup')
+            ->willReturnCallback(function ($uid) {
+                switch ($uid) {
+                    case 'user1': return false;
+                    case 'user2': return true;
+                    case 'user3': return false;
+                    case 'user33': return true;
+                    case 'user333': return true;
+                    default:
+                        return null;
+                }
+            });
+
+        $this->userManager->expects($this->any())
+            ->method('searchDisplayName')
+            ->with('user3')
+            ->willReturnCallback(function ($search, $limit, $offset) {
+                switch ($offset) {
+                    case 0:
+                        return [
+                            'user3' => $this->getTestUser('user3'),
+                            'user33' => $this->getTestUser('user33'),
+                            'user333' => $this->getTestUser('user333')
+                        ];
+                }
+                return null;
+            });
+        $this->userManager->expects($this->any())
+            ->method('get')
+            ->willReturnCallback(function ($uid) {
+                switch ($uid) {
+                    case 'user1': return $this->getTestUser('user1');
+                    case 'user2': return $this->getTestUser('user2');
+                    case 'user3': return $this->getTestUser('user3');
+                    case 'user33': return $this->getTestUser('user33');
+                    case 'user333': return $this->getTestUser('user333');
+                    default:
+                        return null;
+                }
+            });
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $users = $manager->displayNamesInGroup('testgroup', 'user3', 1, 1);
+        $this->assertCount(1, $users);
+        $this->assertFalse(isset($users['user1']));
+        $this->assertFalse(isset($users['user2']));
+        $this->assertFalse(isset($users['user3']));
+        $this->assertFalse(isset($users['user33']));
+        $this->assertTrue(isset($users['user333']));
+    }
+
+    public function testDisplayNamesInGroupWithOneUserBackendAndSearchEmpty(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject|\OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $backend->expects($this->exactly(1))
+            ->method('groupExists')
+            ->with('testgroup')
+            ->willReturn(true);
+
+        $backend->expects($this->once())
+            ->method('searchInGroup')
+            ->with('testgroup', '', -1, 0)
+            ->willReturn(['user2' => $this->getTestUser('user2'), 'user33' => $this->getTestUser('user33')]);
+
+        $this->userManager->expects($this->never())->method('get');
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $users = $manager->displayNamesInGroup('testgroup', '');
+        $this->assertCount(2, $users);
+        $this->assertFalse(isset($users['user1']));
+        $this->assertTrue(isset($users['user2']));
+        $this->assertFalse(isset($users['user3']));
+        $this->assertTrue(isset($users['user33']));
+    }
+
+    public function testDisplayNamesInGroupWithOneUserBackendAndSearchEmptyAndLimitSpecified(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $backend->expects($this->exactly(1))
+            ->method('groupExists')
+            ->with('testgroup')
+            ->willReturn(true);
+
+        $backend->expects($this->once())
+            ->method('searchInGroup')
+            ->with('testgroup', '', 1, 0)
+            ->willReturn([new User('user2', null, $this->dispatcher)]);
+
+        $this->userManager->expects($this->never())->method('get');
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $users = $manager->displayNamesInGroup('testgroup', '', 1);
+        $this->assertCount(1, $users);
+        $this->assertFalse(isset($users['user1']));
+        $this->assertTrue(isset($users['user2']));
+        $this->assertFalse(isset($users['user3']));
+        $this->assertFalse(isset($users['user33']));
+    }
+
+    public function testDisplayNamesInGroupWithOneUserBackendAndSearchEmptyAndLimitAndOffsetSpecified(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $backend->expects($this->exactly(1))
+            ->method('groupExists')
+            ->with('testgroup')
+            ->willReturn(true);
+
+        $backend->expects($this->once())
+            ->method('searchInGroup')
+            ->with('testgroup', '', 1, 1)
+            ->willReturn(['user33' => $this->getTestUser('user33')]);
+
+        $this->userManager->expects($this->never())->method('get');
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $users = $manager->displayNamesInGroup('testgroup', '', 1, 1);
+        $this->assertCount(1, $users);
+        $this->assertFalse(isset($users['user1']));
+        $this->assertFalse(isset($users['user2']));
+        $this->assertFalse(isset($users['user3']));
+        $this->assertTrue(isset($users['user33']));
+    }
+
+    public function testGetUserGroupsWithAddUser(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $expectedGroups = [];
+        $backend->expects($this->any())
+            ->method('getUserGroups')
+            ->with('user1')
+            ->willReturnCallback(function () use (&$expectedGroups) {
+                return $expectedGroups;
+            });
+        $backend->expects($this->any())
+            ->method('groupExists')
+            ->with('group1')
+            ->willReturn(true);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        // prime cache
+        $user1 = $this->getTestUser('user1');
+        $groups = $manager->getUserGroups($user1);
+        $this->assertEquals([], $groups);
+
+        // add user
+        $group = $manager->get('group1');
+        $group->addUser($user1);
+        $expectedGroups[] = 'group1';
+
+        // check result
+        $groups = $manager->getUserGroups($user1);
+        $this->assertCount(1, $groups);
+        $group1 = reset($groups);
+        $this->assertEquals('group1', $group1->getGID());
+    }
+
+    public function testGetUserGroupsWithRemoveUser(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $expectedGroups = ['group1'];
+        $backend->expects($this->any())
+            ->method('getUserGroups')
+            ->with('user1')
+            ->willReturnCallback(function () use (&$expectedGroups) {
+                return $expectedGroups;
+            });
+        $backend->expects($this->any())
+            ->method('groupExists')
+            ->with('group1')
+            ->willReturn(true);
+        $backend->expects($this->once())
+            ->method('inGroup')
+            ->willReturn(true);
+        $backend->expects($this->once())
+            ->method('removeFromGroup')
+            ->willReturn(true);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        // prime cache
+        $user1 = $this->getTestUser('user1');
+        $groups = $manager->getUserGroups($user1);
+        $this->assertCount(1, $groups);
+        $group1 = reset($groups);
+        $this->assertEquals('group1', $group1->getGID());
+
+        // remove user
+        $group = $manager->get('group1');
+        $group->removeUser($user1);
+        $expectedGroups = [];
+
+        // check result
+        $groups = $manager->getUserGroups($user1);
+        $this->assertEquals($expectedGroups, $groups);
+    }
+
+    public function testGetUserIdGroups(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend();
+        $backend->expects($this->any())
+            ->method('getUserGroups')
+            ->with('user1')
+            ->willReturn(null);
+
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        $groups = $manager->getUserIdGroups('user1');
+        $this->assertEquals([], $groups);
+    }
+
+    public function testGroupDisplayName(): void {
+        /**
+         * @var \PHPUnit\Framework\MockObject\MockObject | \OC\Group\Backend $backend
+         */
+        $backend = $this->getTestBackend(
+            GroupInterface::ADD_TO_GROUP |
+            GroupInterface::REMOVE_FROM_GOUP |
+            GroupInterface::COUNT_USERS |
+            GroupInterface::CREATE_GROUP |
+            GroupInterface::DELETE_GROUP |
+            GroupInterface::GROUP_DETAILS
+        );
+        $backend->expects($this->any())
+            ->method('getGroupDetails')
+            ->willReturnMap([
+                ['group1', ['gid' => 'group1', 'displayName' => 'Group One']],
+                ['group2', ['gid' => 'group2']],
+            ]);
+        $manager = new \OC\Group\Manager($this->userManager, $this->dispatcher, $this->logger, $this->cache, $this->remoteIpAddress);
+        $manager->addBackend($backend);
+
+        // group with display name
+        $group = $manager->get('group1');
+        $this->assertNotNull($group);
+        $this->assertEquals('group1', $group->getGID());
+        $this->assertEquals('Group One', $group->getDisplayName());
+
+        // group without display name
+        $group = $manager->get('group2');
+        $this->assertNotNull($group);
+        $this->assertEquals('group2', $group->getGID());
+        $this->assertEquals('group2', $group->getDisplayName());
+    }
 }

tests/lib/LegacyHelperTest.php

Indentation +241 added lines, -241 removed lines

@@ -11,245 +11,245 @@
 use OC_Helper;
 
 class LegacyHelperTest extends \Test\TestCase {
-	/** @var string */
-	private $originalWebRoot;
-
-	protected function setUp(): void {
-		$this->originalWebRoot = \OC::$WEBROOT;
-	}
-
-	protected function tearDown(): void {
-		// Reset webRoot
-		\OC::$WEBROOT = $this->originalWebRoot;
-	}
-
-	/**
-	 * @dataProvider humanFileSizeProvider
-	 */
-	public function testHumanFileSize($expected, $input): void {
-		$result = OC_Helper::humanFileSize($input);
-		$this->assertEquals($expected, $result);
-	}
-
-	public static function humanFileSizeProvider(): array {
-		return [
-			['0 B', 0],
-			['1 KB', 1024],
-			['9.5 MB', 10000000],
-			['1.3 GB', 1395864371],
-			['465.7 GB', 500000000000],
-			['454.7 TB', 500000000000000],
-			['444.1 PB', 500000000000000000],
-		];
-	}
-
-	/**
-	 * @dataProvider providesComputerFileSize
-	 */
-	public function testComputerFileSize($expected, $input): void {
-		$result = OC_Helper::computerFileSize($input);
-		$this->assertEquals($expected, $result);
-	}
-
-	public static function providesComputerFileSize(): array {
-		return [
-			[0.0, '0 B'],
-			[1024.0, '1 KB'],
-			[1395864371.0, '1.3 GB'],
-			[9961472.0, '9.5 MB'],
-			[500041567437.0, '465.7 GB'],
-			[false, '12 GB etfrhzui']
-		];
-	}
-
-	public function testMb_array_change_key_case(): void {
-		$arrayStart = [
-			'Foo' => 'bar',
-			'Bar' => 'foo',
-		];
-		$arrayResult = [
-			'foo' => 'bar',
-			'bar' => 'foo',
-		];
-		$result = OC_Helper::mb_array_change_key_case($arrayStart);
-		$expected = $arrayResult;
-		$this->assertEquals($result, $expected);
-
-		$arrayStart = [
-			'foo' => 'bar',
-			'bar' => 'foo',
-		];
-		$arrayResult = [
-			'FOO' => 'bar',
-			'BAR' => 'foo',
-		];
-		$result = OC_Helper::mb_array_change_key_case($arrayStart, MB_CASE_UPPER);
-		$expected = $arrayResult;
-		$this->assertEquals($result, $expected);
-	}
-
-	public function testRecursiveArraySearch(): void {
-		$haystack = [
-			'Foo' => 'own',
-			'Bar' => 'Cloud',
-		];
-
-		$result = OC_Helper::recursiveArraySearch($haystack, 'own');
-		$expected = 'Foo';
-		$this->assertEquals($result, $expected);
-
-		$result = OC_Helper::recursiveArraySearch($haystack, 'NotFound');
-		$this->assertFalse($result);
-	}
-
-	public function testBuildNotExistingFileNameForView(): void {
-		$viewMock = $this->createMock(View::class);
-		$this->assertEquals('/filename', OC_Helper::buildNotExistingFileNameForView('/', 'filename', $viewMock));
-		$this->assertEquals('dir/filename.ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename.ext', $viewMock));
-
-		$viewMock = $this->createMock(View::class);
-		$viewMock->expects($this->exactly(2))
-			->method('file_exists')
-			->willReturnMap([
-				// Conflict on filename.ext
-				['dir/filename.ext', true],
-				['dir/filename (2).ext', false],
-			]);
-		$this->assertEquals('dir/filename (2).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename.ext', $viewMock));
-
-		$viewMock = $this->createMock(View::class);
-		$viewMock->expects($this->exactly(3))
-			->method('file_exists')
-			->willReturnMap([
-				// Conflict on filename.ext
-				['dir/filename.ext', true],
-				['dir/filename (2).ext', true],
-				['dir/filename (3).ext', false],
-			]);
-		$this->assertEquals('dir/filename (3).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename.ext', $viewMock));
-
-		$viewMock = $this->createMock(View::class);
-		$viewMock->expects($this->exactly(2))
-			->method('file_exists')
-			->willReturnMap([
-				['dir/filename (1).ext', true],
-				['dir/filename (2).ext', false],
-			]);
-		$this->assertEquals('dir/filename (2).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename (1).ext', $viewMock));
-
-		$viewMock = $this->createMock(View::class);
-		$viewMock->expects($this->exactly(2))
-			->method('file_exists')
-			->willReturnMap([
-				['dir/filename (2).ext', true],
-				['dir/filename (3).ext', false],
-			]);
-		$this->assertEquals('dir/filename (3).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename (2).ext', $viewMock));
-
-		$viewMock = $this->createMock(View::class);
-		$viewMock->expects($this->exactly(3))
-			->method('file_exists')
-			->willReturnMap([
-				['dir/filename (2).ext', true],
-				['dir/filename (3).ext', true],
-				['dir/filename (4).ext', false],
-			]);
-		$this->assertEquals('dir/filename (4).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename (2).ext', $viewMock));
-
-		$viewMock = $this->createMock(View::class);
-		$viewMock->expects($this->exactly(2))
-			->method('file_exists')
-			->willReturnMap([
-				['dir/filename(1).ext', true],
-				['dir/filename(2).ext', false],
-			]);
-		$this->assertEquals('dir/filename(2).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename(1).ext', $viewMock));
-
-		$viewMock = $this->createMock(View::class);
-		$viewMock->expects($this->exactly(2))
-			->method('file_exists')
-			->willReturnMap([
-				['dir/filename(1) (1).ext', true],
-				['dir/filename(1) (2).ext', false],
-			]);
-		$this->assertEquals('dir/filename(1) (2).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename(1) (1).ext', $viewMock));
-
-		$viewMock = $this->createMock(View::class);
-		$viewMock->expects($this->exactly(3))
-			->method('file_exists')
-			->willReturnMap([
-				['dir/filename(1) (1).ext', true],
-				['dir/filename(1) (2).ext', true],
-				['dir/filename(1) (3).ext', false],
-			]);
-		$this->assertEquals('dir/filename(1) (3).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename(1) (1).ext', $viewMock));
-
-		$viewMock = $this->createMock(View::class);
-		$viewMock->expects($this->exactly(2))
-			->method('file_exists')
-			->willReturnMap([
-				['dir/filename(1) (2) (3).ext', true],
-				['dir/filename(1) (2) (4).ext', false],
-			]);
-		$this->assertEquals('dir/filename(1) (2) (4).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename(1) (2) (3).ext', $viewMock));
-	}
-
-	/**
-	 * @dataProvider streamCopyDataProvider
-	 */
-	public function testStreamCopy($expectedCount, $expectedResult, $source, $target): void {
-		if (is_string($source)) {
-			$source = fopen($source, 'r');
-		}
-		if (is_string($target)) {
-			$target = fopen($target, 'w');
-		}
-
-		[$count, $result] = \OC_Helper::streamCopy($source, $target);
-
-		if (is_resource($source)) {
-			fclose($source);
-		}
-		if (is_resource($target)) {
-			fclose($target);
-		}
-
-		$this->assertSame($expectedCount, $count);
-		$this->assertSame($expectedResult, $result);
-	}
-
-
-	public static function streamCopyDataProvider(): array {
-		return [
-			[0, false, false, false],
-			[0, false, \OC::$SERVERROOT . '/tests/data/lorem.txt', false],
-			[filesize(\OC::$SERVERROOT . '/tests/data/lorem.txt'), true, \OC::$SERVERROOT . '/tests/data/lorem.txt', \OC::$SERVERROOT . '/tests/data/lorem-copy.txt'],
-			[3670, true, \OC::$SERVERROOT . '/tests/data/testimage.png', \OC::$SERVERROOT . '/tests/data/testimage-copy.png'],
-		];
-	}
-
-	/**
-	 * Tests recursive folder deletion with rmdirr()
-	 */
-	public function testRecursiveFolderDeletion(): void {
-		$baseDir = \OC::$server->getTempManager()->getTemporaryFolder() . '/';
-		mkdir($baseDir . 'a/b/c/d/e', 0777, true);
-		mkdir($baseDir . 'a/b/c1/d/e', 0777, true);
-		mkdir($baseDir . 'a/b/c2/d/e', 0777, true);
-		mkdir($baseDir . 'a/b1/c1/d/e', 0777, true);
-		mkdir($baseDir . 'a/b2/c1/d/e', 0777, true);
-		mkdir($baseDir . 'a/b3/c1/d/e', 0777, true);
-		mkdir($baseDir . 'a1/b', 0777, true);
-		mkdir($baseDir . 'a1/c', 0777, true);
-		file_put_contents($baseDir . 'a/test.txt', 'Hello file!');
-		file_put_contents($baseDir . 'a/b1/c1/test one.txt', 'Hello file one!');
-		file_put_contents($baseDir . 'a1/b/test two.txt', 'Hello file two!');
-		\OC_Helper::rmdirr($baseDir . 'a');
-
-		$this->assertFalse(file_exists($baseDir . 'a'));
-		$this->assertTrue(file_exists($baseDir . 'a1'));
-
-		\OC_Helper::rmdirr($baseDir);
-		$this->assertFalse(file_exists($baseDir));
-	}
+    /** @var string */
+    private $originalWebRoot;
+
+    protected function setUp(): void {
+        $this->originalWebRoot = \OC::$WEBROOT;
+    }
+
+    protected function tearDown(): void {
+        // Reset webRoot
+        \OC::$WEBROOT = $this->originalWebRoot;
+    }
+
+    /**
+     * @dataProvider humanFileSizeProvider
+     */
+    public function testHumanFileSize($expected, $input): void {
+        $result = OC_Helper::humanFileSize($input);
+        $this->assertEquals($expected, $result);
+    }
+
+    public static function humanFileSizeProvider(): array {
+        return [
+            ['0 B', 0],
+            ['1 KB', 1024],
+            ['9.5 MB', 10000000],
+            ['1.3 GB', 1395864371],
+            ['465.7 GB', 500000000000],
+            ['454.7 TB', 500000000000000],
+            ['444.1 PB', 500000000000000000],
+        ];
+    }
+
+    /**
+     * @dataProvider providesComputerFileSize
+     */
+    public function testComputerFileSize($expected, $input): void {
+        $result = OC_Helper::computerFileSize($input);
+        $this->assertEquals($expected, $result);
+    }
+
+    public static function providesComputerFileSize(): array {
+        return [
+            [0.0, '0 B'],
+            [1024.0, '1 KB'],
+            [1395864371.0, '1.3 GB'],
+            [9961472.0, '9.5 MB'],
+            [500041567437.0, '465.7 GB'],
+            [false, '12 GB etfrhzui']
+        ];
+    }
+
+    public function testMb_array_change_key_case(): void {
+        $arrayStart = [
+            'Foo' => 'bar',
+            'Bar' => 'foo',
+        ];
+        $arrayResult = [
+            'foo' => 'bar',
+            'bar' => 'foo',
+        ];
+        $result = OC_Helper::mb_array_change_key_case($arrayStart);
+        $expected = $arrayResult;
+        $this->assertEquals($result, $expected);
+
+        $arrayStart = [
+            'foo' => 'bar',
+            'bar' => 'foo',
+        ];
+        $arrayResult = [
+            'FOO' => 'bar',
+            'BAR' => 'foo',
+        ];
+        $result = OC_Helper::mb_array_change_key_case($arrayStart, MB_CASE_UPPER);
+        $expected = $arrayResult;
+        $this->assertEquals($result, $expected);
+    }
+
+    public function testRecursiveArraySearch(): void {
+        $haystack = [
+            'Foo' => 'own',
+            'Bar' => 'Cloud',
+        ];
+
+        $result = OC_Helper::recursiveArraySearch($haystack, 'own');
+        $expected = 'Foo';
+        $this->assertEquals($result, $expected);
+
+        $result = OC_Helper::recursiveArraySearch($haystack, 'NotFound');
+        $this->assertFalse($result);
+    }
+
+    public function testBuildNotExistingFileNameForView(): void {
+        $viewMock = $this->createMock(View::class);
+        $this->assertEquals('/filename', OC_Helper::buildNotExistingFileNameForView('/', 'filename', $viewMock));
+        $this->assertEquals('dir/filename.ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename.ext', $viewMock));
+
+        $viewMock = $this->createMock(View::class);
+        $viewMock->expects($this->exactly(2))
+            ->method('file_exists')
+            ->willReturnMap([
+                // Conflict on filename.ext
+                ['dir/filename.ext', true],
+                ['dir/filename (2).ext', false],
+            ]);
+        $this->assertEquals('dir/filename (2).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename.ext', $viewMock));
+
+        $viewMock = $this->createMock(View::class);
+        $viewMock->expects($this->exactly(3))
+            ->method('file_exists')
+            ->willReturnMap([
+                // Conflict on filename.ext
+                ['dir/filename.ext', true],
+                ['dir/filename (2).ext', true],
+                ['dir/filename (3).ext', false],
+            ]);
+        $this->assertEquals('dir/filename (3).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename.ext', $viewMock));
+
+        $viewMock = $this->createMock(View::class);
+        $viewMock->expects($this->exactly(2))
+            ->method('file_exists')
+            ->willReturnMap([
+                ['dir/filename (1).ext', true],
+                ['dir/filename (2).ext', false],
+            ]);
+        $this->assertEquals('dir/filename (2).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename (1).ext', $viewMock));
+
+        $viewMock = $this->createMock(View::class);
+        $viewMock->expects($this->exactly(2))
+            ->method('file_exists')
+            ->willReturnMap([
+                ['dir/filename (2).ext', true],
+                ['dir/filename (3).ext', false],
+            ]);
+        $this->assertEquals('dir/filename (3).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename (2).ext', $viewMock));
+
+        $viewMock = $this->createMock(View::class);
+        $viewMock->expects($this->exactly(3))
+            ->method('file_exists')
+            ->willReturnMap([
+                ['dir/filename (2).ext', true],
+                ['dir/filename (3).ext', true],
+                ['dir/filename (4).ext', false],
+            ]);
+        $this->assertEquals('dir/filename (4).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename (2).ext', $viewMock));
+
+        $viewMock = $this->createMock(View::class);
+        $viewMock->expects($this->exactly(2))
+            ->method('file_exists')
+            ->willReturnMap([
+                ['dir/filename(1).ext', true],
+                ['dir/filename(2).ext', false],
+            ]);
+        $this->assertEquals('dir/filename(2).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename(1).ext', $viewMock));
+
+        $viewMock = $this->createMock(View::class);
+        $viewMock->expects($this->exactly(2))
+            ->method('file_exists')
+            ->willReturnMap([
+                ['dir/filename(1) (1).ext', true],
+                ['dir/filename(1) (2).ext', false],
+            ]);
+        $this->assertEquals('dir/filename(1) (2).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename(1) (1).ext', $viewMock));
+
+        $viewMock = $this->createMock(View::class);
+        $viewMock->expects($this->exactly(3))
+            ->method('file_exists')
+            ->willReturnMap([
+                ['dir/filename(1) (1).ext', true],
+                ['dir/filename(1) (2).ext', true],
+                ['dir/filename(1) (3).ext', false],
+            ]);
+        $this->assertEquals('dir/filename(1) (3).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename(1) (1).ext', $viewMock));
+
+        $viewMock = $this->createMock(View::class);
+        $viewMock->expects($this->exactly(2))
+            ->method('file_exists')
+            ->willReturnMap([
+                ['dir/filename(1) (2) (3).ext', true],
+                ['dir/filename(1) (2) (4).ext', false],
+            ]);
+        $this->assertEquals('dir/filename(1) (2) (4).ext', OC_Helper::buildNotExistingFileNameForView('dir', 'filename(1) (2) (3).ext', $viewMock));
+    }
+
+    /**
+     * @dataProvider streamCopyDataProvider
+     */
+    public function testStreamCopy($expectedCount, $expectedResult, $source, $target): void {
+        if (is_string($source)) {
+            $source = fopen($source, 'r');
+        }
+        if (is_string($target)) {
+            $target = fopen($target, 'w');
+        }
+
+        [$count, $result] = \OC_Helper::streamCopy($source, $target);
+
+        if (is_resource($source)) {
+            fclose($source);
+        }
+        if (is_resource($target)) {
+            fclose($target);
+        }
+
+        $this->assertSame($expectedCount, $count);
+        $this->assertSame($expectedResult, $result);
+    }
+
+
+    public static function streamCopyDataProvider(): array {
+        return [
+            [0, false, false, false],
+            [0, false, \OC::$SERVERROOT . '/tests/data/lorem.txt', false],
+            [filesize(\OC::$SERVERROOT . '/tests/data/lorem.txt'), true, \OC::$SERVERROOT . '/tests/data/lorem.txt', \OC::$SERVERROOT . '/tests/data/lorem-copy.txt'],
+            [3670, true, \OC::$SERVERROOT . '/tests/data/testimage.png', \OC::$SERVERROOT . '/tests/data/testimage-copy.png'],
+        ];
+    }
+
+    /**
+     * Tests recursive folder deletion with rmdirr()
+     */
+    public function testRecursiveFolderDeletion(): void {
+        $baseDir = \OC::$server->getTempManager()->getTemporaryFolder() . '/';
+        mkdir($baseDir . 'a/b/c/d/e', 0777, true);
+        mkdir($baseDir . 'a/b/c1/d/e', 0777, true);
+        mkdir($baseDir . 'a/b/c2/d/e', 0777, true);
+        mkdir($baseDir . 'a/b1/c1/d/e', 0777, true);
+        mkdir($baseDir . 'a/b2/c1/d/e', 0777, true);
+        mkdir($baseDir . 'a/b3/c1/d/e', 0777, true);
+        mkdir($baseDir . 'a1/b', 0777, true);
+        mkdir($baseDir . 'a1/c', 0777, true);
+        file_put_contents($baseDir . 'a/test.txt', 'Hello file!');
+        file_put_contents($baseDir . 'a/b1/c1/test one.txt', 'Hello file one!');
+        file_put_contents($baseDir . 'a1/b/test two.txt', 'Hello file two!');
+        \OC_Helper::rmdirr($baseDir . 'a');
+
+        $this->assertFalse(file_exists($baseDir . 'a'));
+        $this->assertTrue(file_exists($baseDir . 'a1'));
+
+        \OC_Helper::rmdirr($baseDir);
+        $this->assertFalse(file_exists($baseDir));
+    }
 }

Spacing +19 added lines, -19 removed lines

@@ -222,9 +222,9 @@ 
 	public static function streamCopyDataProvider(): array {
 		return [
 			[0, false, false, false],
-			[0, false, \OC::$SERVERROOT . '/tests/data/lorem.txt', false],
-			[filesize(\OC::$SERVERROOT . '/tests/data/lorem.txt'), true, \OC::$SERVERROOT . '/tests/data/lorem.txt', \OC::$SERVERROOT . '/tests/data/lorem-copy.txt'],
-			[3670, true, \OC::$SERVERROOT . '/tests/data/testimage.png', \OC::$SERVERROOT . '/tests/data/testimage-copy.png'],
+			[0, false, \OC::$SERVERROOT.'/tests/data/lorem.txt', false],
+			[filesize(\OC::$SERVERROOT.'/tests/data/lorem.txt'), true, \OC::$SERVERROOT.'/tests/data/lorem.txt', \OC::$SERVERROOT.'/tests/data/lorem-copy.txt'],
+			[3670, true, \OC::$SERVERROOT.'/tests/data/testimage.png', \OC::$SERVERROOT.'/tests/data/testimage-copy.png'],
 		];
 	}
 
@@ -232,22 +232,22 @@ 
 	 * Tests recursive folder deletion with rmdirr()
 	 */
 	public function testRecursiveFolderDeletion(): void {
-		$baseDir = \OC::$server->getTempManager()->getTemporaryFolder() . '/';
-		mkdir($baseDir . 'a/b/c/d/e', 0777, true);
-		mkdir($baseDir . 'a/b/c1/d/e', 0777, true);
-		mkdir($baseDir . 'a/b/c2/d/e', 0777, true);
-		mkdir($baseDir . 'a/b1/c1/d/e', 0777, true);
-		mkdir($baseDir . 'a/b2/c1/d/e', 0777, true);
-		mkdir($baseDir . 'a/b3/c1/d/e', 0777, true);
-		mkdir($baseDir . 'a1/b', 0777, true);
-		mkdir($baseDir . 'a1/c', 0777, true);
-		file_put_contents($baseDir . 'a/test.txt', 'Hello file!');
-		file_put_contents($baseDir . 'a/b1/c1/test one.txt', 'Hello file one!');
-		file_put_contents($baseDir . 'a1/b/test two.txt', 'Hello file two!');
-		\OC_Helper::rmdirr($baseDir . 'a');
-
-		$this->assertFalse(file_exists($baseDir . 'a'));
-		$this->assertTrue(file_exists($baseDir . 'a1'));
+		$baseDir = \OC::$server->getTempManager()->getTemporaryFolder().'/';
+		mkdir($baseDir.'a/b/c/d/e', 0777, true);
+		mkdir($baseDir.'a/b/c1/d/e', 0777, true);
+		mkdir($baseDir.'a/b/c2/d/e', 0777, true);
+		mkdir($baseDir.'a/b1/c1/d/e', 0777, true);
+		mkdir($baseDir.'a/b2/c1/d/e', 0777, true);
+		mkdir($baseDir.'a/b3/c1/d/e', 0777, true);
+		mkdir($baseDir.'a1/b', 0777, true);
+		mkdir($baseDir.'a1/c', 0777, true);
+		file_put_contents($baseDir.'a/test.txt', 'Hello file!');
+		file_put_contents($baseDir.'a/b1/c1/test one.txt', 'Hello file one!');
+		file_put_contents($baseDir.'a1/b/test two.txt', 'Hello file two!');
+		\OC_Helper::rmdirr($baseDir.'a');
+
+		$this->assertFalse(file_exists($baseDir.'a'));
+		$this->assertTrue(file_exists($baseDir.'a1'));
 
 		\OC_Helper::rmdirr($baseDir);
 		$this->assertFalse(file_exists($baseDir));

tests/lib/Avatar/AvatarManagerTest.php

Indentation +248 added lines, -248 removed lines

@@ -28,252 +28,252 @@
  * Class AvatarManagerTest
  */
 class AvatarManagerTest extends \Test\TestCase {
-	/** @var IUserSession|\PHPUnit\Framework\MockObject\MockObject */
-	private $userSession;
-	/** @var Manager|\PHPUnit\Framework\MockObject\MockObject */
-	private $userManager;
-	/** @var IAppData|\PHPUnit\Framework\MockObject\MockObject */
-	private $appData;
-	/** @var IL10N|\PHPUnit\Framework\MockObject\MockObject */
-	private $l10n;
-	/** @var LoggerInterface|\PHPUnit\Framework\MockObject\MockObject */
-	private $logger;
-	/** @var IConfig|\PHPUnit\Framework\MockObject\MockObject */
-	private $config;
-	/** @var IAccountManager|\PHPUnit\Framework\MockObject\MockObject */
-	private $accountManager;
-	/** @var AvatarManager | \PHPUnit\Framework\MockObject\MockObject */
-	private $avatarManager;
-	/** @var KnownUserService | \PHPUnit\Framework\MockObject\MockObject */
-	private $knownUserService;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->userSession = $this->createMock(IUserSession::class);
-		$this->userManager = $this->createMock(Manager::class);
-		$this->appData = $this->createMock(IAppData::class);
-		$this->l10n = $this->createMock(IL10N::class);
-		$this->logger = $this->createMock(LoggerInterface::class);
-		$this->config = $this->createMock(IConfig::class);
-		$this->accountManager = $this->createMock(IAccountManager::class);
-		$this->knownUserService = $this->createMock(KnownUserService::class);
-
-		$this->avatarManager = new AvatarManager(
-			$this->userSession,
-			$this->userManager,
-			$this->appData,
-			$this->l10n,
-			$this->logger,
-			$this->config,
-			$this->accountManager,
-			$this->knownUserService
-		);
-	}
-
-	public function testGetAvatarInvalidUser(): void {
-		$this->expectException(\Exception::class);
-		$this->expectExceptionMessage('user does not exist');
-
-		$this->userManager
-			->expects($this->once())
-			->method('get')
-			->with('invalidUser')
-			->willReturn(null);
-
-		$this->avatarManager->getAvatar('invalidUser');
-	}
-
-	public function testGetAvatarForSelf(): void {
-		$user = $this->createMock(User::class);
-		$user
-			->expects($this->any())
-			->method('getUID')
-			->willReturn('valid-user');
-
-		$user
-			->expects($this->any())
-			->method('isEnabled')
-			->willReturn(true);
-
-		// requesting user
-		$this->userSession->expects($this->once())
-			->method('getUser')
-			->willReturn($user);
-
-		$this->userManager
-			->expects($this->once())
-			->method('get')
-			->with('valid-user')
-			->willReturn($user);
-
-		$account = $this->createMock(IAccount::class);
-		$this->accountManager->expects($this->once())
-			->method('getAccount')
-			->with($user)
-			->willReturn($account);
-
-		$property = $this->createMock(IAccountProperty::class);
-		$account->expects($this->once())
-			->method('getProperty')
-			->with(IAccountManager::PROPERTY_AVATAR)
-			->willReturn($property);
-
-		$property->expects($this->once())
-			->method('getScope')
-			->willReturn(IAccountManager::SCOPE_PRIVATE);
-
-		$this->knownUserService->expects($this->any())
-			->method('isKnownToUser')
-			->with('valid-user', 'valid-user')
-			->willReturn(true);
-
-		$folder = $this->createMock(ISimpleFolder::class);
-		$this->appData
-			->expects($this->once())
-			->method('getFolder')
-			->with('valid-user')
-			->willReturn($folder);
-
-		$expected = new UserAvatar($folder, $this->l10n, $user, $this->logger, $this->config);
-		$this->assertEquals($expected, $this->avatarManager->getAvatar('valid-user'));
-	}
-
-	public function testGetAvatarValidUserDifferentCasing(): void {
-		$user = $this->createMock(User::class);
-		$this->userManager->expects($this->once())
-			->method('get')
-			->with('vaLid-USER')
-			->willReturn($user);
-
-		$user->expects($this->once())
-			->method('getUID')
-			->willReturn('valid-user');
-
-		$user
-			->expects($this->any())
-			->method('isEnabled')
-			->willReturn(true);
-
-		$this->userSession->expects($this->once())
-			->method('getUser')
-			->willReturn($user);
-
-		$folder = $this->createMock(ISimpleFolder::class);
-		$this->appData
-			->expects($this->once())
-			->method('getFolder')
-			->with('valid-user')
-			->willReturn($folder);
-
-		$account = $this->createMock(IAccount::class);
-		$this->accountManager->expects($this->once())
-			->method('getAccount')
-			->with($user)
-			->willReturn($account);
-
-		$property = $this->createMock(IAccountProperty::class);
-		$account->expects($this->once())
-			->method('getProperty')
-			->with(IAccountManager::PROPERTY_AVATAR)
-			->willReturn($property);
-
-		$property->expects($this->once())
-			->method('getScope')
-			->willReturn(IAccountManager::SCOPE_FEDERATED);
-
-		$expected = new UserAvatar($folder, $this->l10n, $user, $this->logger, $this->config);
-		$this->assertEquals($expected, $this->avatarManager->getAvatar('vaLid-USER'));
-	}
-
-	public static function dataGetAvatarScopes(): array {
-		return [
-			// public access cannot see real avatar
-			[IAccountManager::SCOPE_PRIVATE, true, false, true],
-			// unknown users cannot see real avatar
-			[IAccountManager::SCOPE_PRIVATE, false, false, true],
-			// known users can see real avatar
-			[IAccountManager::SCOPE_PRIVATE, false, true, false],
-			[IAccountManager::SCOPE_LOCAL, false, false, false],
-			[IAccountManager::SCOPE_LOCAL, true, false, false],
-			[IAccountManager::SCOPE_FEDERATED, false, false, false],
-			[IAccountManager::SCOPE_FEDERATED, true, false, false],
-			[IAccountManager::SCOPE_PUBLISHED, false, false, false],
-			[IAccountManager::SCOPE_PUBLISHED, true, false, false],
-		];
-	}
-
-	/**
-	 * @dataProvider dataGetAvatarScopes
-	 */
-	public function testGetAvatarScopes($avatarScope, $isPublicCall, $isKnownUser, $expectedPlaceholder): void {
-		if ($isPublicCall) {
-			$requestingUser = null;
-		} else {
-			$requestingUser = $this->createMock(IUser::class);
-			$requestingUser->method('getUID')->willReturn('requesting-user');
-		}
-
-		// requesting user
-		$this->userSession->expects($this->once())
-			->method('getUser')
-			->willReturn($requestingUser);
-
-		$user = $this->createMock(User::class);
-		$user
-			->expects($this->once())
-			->method('getUID')
-			->willReturn('valid-user');
-
-		$user
-			->expects($this->any())
-			->method('isEnabled')
-			->willReturn(true);
-
-		$this->userManager
-			->expects($this->once())
-			->method('get')
-			->with('valid-user')
-			->willReturn($user);
-
-		$account = $this->createMock(IAccount::class);
-		$this->accountManager->expects($this->once())
-			->method('getAccount')
-			->with($user)
-			->willReturn($account);
-
-		$property = $this->createMock(IAccountProperty::class);
-		$account->expects($this->once())
-			->method('getProperty')
-			->with(IAccountManager::PROPERTY_AVATAR)
-			->willReturn($property);
-
-		$property->expects($this->once())
-			->method('getScope')
-			->willReturn($avatarScope);
-
-		$folder = $this->createMock(ISimpleFolder::class);
-		$this->appData
-			->expects($this->once())
-			->method('getFolder')
-			->with('valid-user')
-			->willReturn($folder);
-
-		if (!$isPublicCall) {
-			$this->knownUserService->expects($this->any())
-				->method('isKnownToUser')
-				->with('requesting-user', 'valid-user')
-				->willReturn($isKnownUser);
-		} else {
-			$this->knownUserService->expects($this->never())
-				->method('isKnownToUser');
-		}
-
-		if ($expectedPlaceholder) {
-			$expected = new PlaceholderAvatar($folder, $user, $this->createMock(LoggerInterface::class));
-		} else {
-			$expected = new UserAvatar($folder, $this->l10n, $user, $this->logger, $this->config);
-		}
-		$this->assertEquals($expected, $this->avatarManager->getAvatar('valid-user'));
-	}
+    /** @var IUserSession|\PHPUnit\Framework\MockObject\MockObject */
+    private $userSession;
+    /** @var Manager|\PHPUnit\Framework\MockObject\MockObject */
+    private $userManager;
+    /** @var IAppData|\PHPUnit\Framework\MockObject\MockObject */
+    private $appData;
+    /** @var IL10N|\PHPUnit\Framework\MockObject\MockObject */
+    private $l10n;
+    /** @var LoggerInterface|\PHPUnit\Framework\MockObject\MockObject */
+    private $logger;
+    /** @var IConfig|\PHPUnit\Framework\MockObject\MockObject */
+    private $config;
+    /** @var IAccountManager|\PHPUnit\Framework\MockObject\MockObject */
+    private $accountManager;
+    /** @var AvatarManager | \PHPUnit\Framework\MockObject\MockObject */
+    private $avatarManager;
+    /** @var KnownUserService | \PHPUnit\Framework\MockObject\MockObject */
+    private $knownUserService;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->userSession = $this->createMock(IUserSession::class);
+        $this->userManager = $this->createMock(Manager::class);
+        $this->appData = $this->createMock(IAppData::class);
+        $this->l10n = $this->createMock(IL10N::class);
+        $this->logger = $this->createMock(LoggerInterface::class);
+        $this->config = $this->createMock(IConfig::class);
+        $this->accountManager = $this->createMock(IAccountManager::class);
+        $this->knownUserService = $this->createMock(KnownUserService::class);
+
+        $this->avatarManager = new AvatarManager(
+            $this->userSession,
+            $this->userManager,
+            $this->appData,
+            $this->l10n,
+            $this->logger,
+            $this->config,
+            $this->accountManager,
+            $this->knownUserService
+        );
+    }
+
+    public function testGetAvatarInvalidUser(): void {
+        $this->expectException(\Exception::class);
+        $this->expectExceptionMessage('user does not exist');
+
+        $this->userManager
+            ->expects($this->once())
+            ->method('get')
+            ->with('invalidUser')
+            ->willReturn(null);
+
+        $this->avatarManager->getAvatar('invalidUser');
+    }
+
+    public function testGetAvatarForSelf(): void {
+        $user = $this->createMock(User::class);
+        $user
+            ->expects($this->any())
+            ->method('getUID')
+            ->willReturn('valid-user');
+
+        $user
+            ->expects($this->any())
+            ->method('isEnabled')
+            ->willReturn(true);
+
+        // requesting user
+        $this->userSession->expects($this->once())
+            ->method('getUser')
+            ->willReturn($user);
+
+        $this->userManager
+            ->expects($this->once())
+            ->method('get')
+            ->with('valid-user')
+            ->willReturn($user);
+
+        $account = $this->createMock(IAccount::class);
+        $this->accountManager->expects($this->once())
+            ->method('getAccount')
+            ->with($user)
+            ->willReturn($account);
+
+        $property = $this->createMock(IAccountProperty::class);
+        $account->expects($this->once())
+            ->method('getProperty')
+            ->with(IAccountManager::PROPERTY_AVATAR)
+            ->willReturn($property);
+
+        $property->expects($this->once())
+            ->method('getScope')
+            ->willReturn(IAccountManager::SCOPE_PRIVATE);
+
+        $this->knownUserService->expects($this->any())
+            ->method('isKnownToUser')
+            ->with('valid-user', 'valid-user')
+            ->willReturn(true);
+
+        $folder = $this->createMock(ISimpleFolder::class);
+        $this->appData
+            ->expects($this->once())
+            ->method('getFolder')
+            ->with('valid-user')
+            ->willReturn($folder);
+
+        $expected = new UserAvatar($folder, $this->l10n, $user, $this->logger, $this->config);
+        $this->assertEquals($expected, $this->avatarManager->getAvatar('valid-user'));
+    }
+
+    public function testGetAvatarValidUserDifferentCasing(): void {
+        $user = $this->createMock(User::class);
+        $this->userManager->expects($this->once())
+            ->method('get')
+            ->with('vaLid-USER')
+            ->willReturn($user);
+
+        $user->expects($this->once())
+            ->method('getUID')
+            ->willReturn('valid-user');
+
+        $user
+            ->expects($this->any())
+            ->method('isEnabled')
+            ->willReturn(true);
+
+        $this->userSession->expects($this->once())
+            ->method('getUser')
+            ->willReturn($user);
+
+        $folder = $this->createMock(ISimpleFolder::class);
+        $this->appData
+            ->expects($this->once())
+            ->method('getFolder')
+            ->with('valid-user')
+            ->willReturn($folder);
+
+        $account = $this->createMock(IAccount::class);
+        $this->accountManager->expects($this->once())
+            ->method('getAccount')
+            ->with($user)
+            ->willReturn($account);
+
+        $property = $this->createMock(IAccountProperty::class);
+        $account->expects($this->once())
+            ->method('getProperty')
+            ->with(IAccountManager::PROPERTY_AVATAR)
+            ->willReturn($property);
+
+        $property->expects($this->once())
+            ->method('getScope')
+            ->willReturn(IAccountManager::SCOPE_FEDERATED);
+
+        $expected = new UserAvatar($folder, $this->l10n, $user, $this->logger, $this->config);
+        $this->assertEquals($expected, $this->avatarManager->getAvatar('vaLid-USER'));
+    }
+
+    public static function dataGetAvatarScopes(): array {
+        return [
+            // public access cannot see real avatar
+            [IAccountManager::SCOPE_PRIVATE, true, false, true],
+            // unknown users cannot see real avatar
+            [IAccountManager::SCOPE_PRIVATE, false, false, true],
+            // known users can see real avatar
+            [IAccountManager::SCOPE_PRIVATE, false, true, false],
+            [IAccountManager::SCOPE_LOCAL, false, false, false],
+            [IAccountManager::SCOPE_LOCAL, true, false, false],
+            [IAccountManager::SCOPE_FEDERATED, false, false, false],
+            [IAccountManager::SCOPE_FEDERATED, true, false, false],
+            [IAccountManager::SCOPE_PUBLISHED, false, false, false],
+            [IAccountManager::SCOPE_PUBLISHED, true, false, false],
+        ];
+    }
+
+    /**
+     * @dataProvider dataGetAvatarScopes
+     */
+    public function testGetAvatarScopes($avatarScope, $isPublicCall, $isKnownUser, $expectedPlaceholder): void {
+        if ($isPublicCall) {
+            $requestingUser = null;
+        } else {
+            $requestingUser = $this->createMock(IUser::class);
+            $requestingUser->method('getUID')->willReturn('requesting-user');
+        }
+
+        // requesting user
+        $this->userSession->expects($this->once())
+            ->method('getUser')
+            ->willReturn($requestingUser);
+
+        $user = $this->createMock(User::class);
+        $user
+            ->expects($this->once())
+            ->method('getUID')
+            ->willReturn('valid-user');
+
+        $user
+            ->expects($this->any())
+            ->method('isEnabled')
+            ->willReturn(true);
+
+        $this->userManager
+            ->expects($this->once())
+            ->method('get')
+            ->with('valid-user')
+            ->willReturn($user);
+
+        $account = $this->createMock(IAccount::class);
+        $this->accountManager->expects($this->once())
+            ->method('getAccount')
+            ->with($user)
+            ->willReturn($account);
+
+        $property = $this->createMock(IAccountProperty::class);
+        $account->expects($this->once())
+            ->method('getProperty')
+            ->with(IAccountManager::PROPERTY_AVATAR)
+            ->willReturn($property);
+
+        $property->expects($this->once())
+            ->method('getScope')
+            ->willReturn($avatarScope);
+
+        $folder = $this->createMock(ISimpleFolder::class);
+        $this->appData
+            ->expects($this->once())
+            ->method('getFolder')
+            ->with('valid-user')
+            ->willReturn($folder);
+
+        if (!$isPublicCall) {
+            $this->knownUserService->expects($this->any())
+                ->method('isKnownToUser')
+                ->with('requesting-user', 'valid-user')
+                ->willReturn($isKnownUser);
+        } else {
+            $this->knownUserService->expects($this->never())
+                ->method('isKnownToUser');
+        }
+
+        if ($expectedPlaceholder) {
+            $expected = new PlaceholderAvatar($folder, $user, $this->createMock(LoggerInterface::class));
+        } else {
+            $expected = new UserAvatar($folder, $this->l10n, $user, $this->logger, $this->config);
+        }
+        $this->assertEquals($expected, $this->avatarManager->getAvatar('valid-user'));
+    }
 }

tests/lib/Avatar/UserAvatarTest.php

Indentation +274 added lines, -274 removed lines

@@ -18,282 +18,282 @@
 use Psr\Log\LoggerInterface;
 
 class UserAvatarTest extends \Test\TestCase {
-	/** @var SimpleFolder | \PHPUnit\Framework\MockObject\MockObject */
-	private $folder;
-
-	/** @var \OC\Avatar\UserAvatar */
-	private $avatar;
-
-	/** @var \OC\User\User | \PHPUnit\Framework\MockObject\MockObject $user */
-	private $user;
-
-	/** @var \OCP\IConfig|\PHPUnit\Framework\MockObject\MockObject */
-	private $config;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->folder = $this->createMock(SimpleFolder::class);
-		// abcdefghi is a convenient name that our algorithm convert to our nextcloud blue 0082c9
-		$this->user = $this->getUserWithDisplayName('abcdefghi');
-		$this->config = $this->createMock(IConfig::class);
-
-		$this->avatar = $this->getUserAvatar($this->user);
-	}
-
-	public static function avatarTextData(): array {
-		return [
-			['', '?'],
-			['matchish', 'M'],
-			['Firstname Lastname', 'FL'],
-			['Firstname Lastname Rest', 'FL'],
-		];
-	}
-
-	public function testGetNoAvatar(): void {
-		$file = $this->createMock(ISimpleFile::class);
-		$this->folder->method('newFile')
-			->willReturn($file);
-
-		$this->folder->method('getFile')
-			->willReturnCallback(function (string $path) {
-				if ($path === 'avatar.64.png') {
-					throw new NotFoundException();
-				}
-			});
-		$this->folder->method('fileExists')
-			->willReturnCallback(function ($path) {
-				if ($path === 'generated') {
-					return true;
-				}
-				return false;
-			});
-
-		$data = null;
-		$file->method('putContent')
-			->with($this->callback(function ($d) use (&$data) {
-				$data = $d;
-				return true;
-			}));
-
-		$file->method('getContent')
-			->willReturnCallback(function () use (&$data) {
-				return $data;
-			});
-
-		$result = $this->avatar->get();
-		$this->assertTrue($result->valid());
-	}
-
-	public function testGetAvatarSizeMatch(): void {
-		$this->folder->method('fileExists')
-			->willReturnMap([
-				['avatar.jpg', true],
-				['avatar.128.jpg', true],
-				['generated', false],
-			]);
-
-		$expected = new Image();
-		$expected->loadFromFile(\OC::$SERVERROOT . '/tests/data/testavatar.png');
-
-		$file = $this->createMock(ISimpleFile::class);
-		$file->method('getContent')->willReturn($expected->data());
-		$this->folder->method('getFile')->with('avatar.128.jpg')->willReturn($file);
-
-		$this->assertEquals($expected->data(), $this->avatar->get(128)->data());
-	}
-
-	public function testGetAvatarSizeMinusOne(): void {
-		$this->folder->method('fileExists')
-			->willReturnMap([
-				['avatar.jpg', true],
-				['generated', false],
-			]);
-
-		$expected = new Image();
-		$expected->loadFromFile(\OC::$SERVERROOT . '/tests/data/testavatar.png');
-
-		$file = $this->createMock(ISimpleFile::class);
-		$file->method('getContent')->willReturn($expected->data());
-		$this->folder->method('getFile')->with('avatar.jpg')->willReturn($file);
-
-		$this->assertEquals($expected->data(), $this->avatar->get(-1)->data());
-	}
-
-	public function testGetAvatarNoSizeMatch(): void {
-		$this->folder->method('fileExists')
-			->willReturnMap([
-				['avatar.jpg', false],
-				['avatar.png', true],
-				['avatar.32.png', false],
-				['generated', false],
-			]);
-
-		$expected = new Image();
-		$expected->loadFromFile(\OC::$SERVERROOT . '/tests/data/testavatar.png');
-		$expected2 = new Image();
-		$expected2->loadFromFile(\OC::$SERVERROOT . '/tests/data/testavatar.png');
-		$expected2->resize(32);
-
-		$file = $this->createMock(ISimpleFile::class);
-		$file->method('getContent')->willReturn($expected->data());
-
-		$this->folder->method('getFile')
-			->willReturnCallback(
-				function ($path) use ($file) {
-					if ($path === 'avatar.png') {
-						return $file;
-					} else {
-						throw new \OCP\Files\NotFoundException;
-					}
-				}
-			);
-
-		$newFile = $this->createMock(ISimpleFile::class);
-		$newFile->expects($this->once())
-			->method('putContent')
-			->with($expected2->data());
-		$newFile->expects($this->once())
-			->method('getContent')
-			->willReturn($expected2->data());
-		$this->folder->expects($this->once())
-			->method('newFile')
-			->with('avatar.32.png')
-			->willReturn($newFile);
-
-		$this->assertEquals($expected2->data(), $this->avatar->get(32)->data());
-	}
-
-	public function testExistsNo(): void {
-		$this->assertFalse($this->avatar->exists());
-	}
-
-	public function testExiststJPG(): void {
-		$this->folder->method('fileExists')
-			->willReturnMap([
-				['avatar.jpg', true],
-				['avatar.png', false],
-			]);
-		$this->assertTrue($this->avatar->exists());
-	}
-
-	public function testExistsPNG(): void {
-		$this->folder->method('fileExists')
-			->willReturnMap([
-				['avatar.jpg', false],
-				['avatar.png', true],
-			]);
-		$this->assertTrue($this->avatar->exists());
-	}
-
-	public function testSetAvatar(): void {
-		$avatarFileJPG = $this->createMock(File::class);
-		$avatarFileJPG->method('getName')
-			->willReturn('avatar.jpg');
-		$avatarFileJPG->expects($this->once())->method('delete');
-
-		$avatarFilePNG = $this->createMock(File::class);
-		$avatarFilePNG->method('getName')
-			->willReturn('avatar.png');
-		$avatarFilePNG->expects($this->once())->method('delete');
-
-		$resizedAvatarFile = $this->createMock(File::class);
-		$resizedAvatarFile->method('getName')
-			->willReturn('avatar.32.jpg');
-		$resizedAvatarFile->expects($this->once())->method('delete');
-
-		$this->folder->method('getDirectoryListing')
-			->willReturn([$avatarFileJPG, $avatarFilePNG, $resizedAvatarFile]);
-
-		$generated = $this->createMock(ISimpleFile::class);
-		$this->folder->method('getFile')
-			->with('generated')
-			->willReturn($generated);
-
-		$newFile = $this->createMock(ISimpleFile::class);
-		$this->folder->expects($this->once())
-			->method('newFile')
-			->with('avatar.png')
-			->willReturn($newFile);
-
-		$image = new Image();
-		$image->loadFromFile(\OC::$SERVERROOT . '/tests/data/testavatar.png');
-		$newFile->expects($this->once())
-			->method('putContent')
-			->with($image->data());
-
-		$this->config->expects($this->exactly(3))
-			->method('setUserValue');
-		$this->config->expects($this->once())
-			->method('getUserValue');
-
-		$this->user->expects($this->exactly(1))->method('triggerChange');
-
-		$this->avatar->set($image->data());
-	}
-
-	public function testGenerateSvgAvatar(): void {
-		$avatar = $this->invokePrivate($this->avatar, 'getAvatarVector', [64, false]);
-
-		$svg = '<?xml version="1.0" encoding="UTF-8" standalone="no"?>
+    /** @var SimpleFolder | \PHPUnit\Framework\MockObject\MockObject */
+    private $folder;
+
+    /** @var \OC\Avatar\UserAvatar */
+    private $avatar;
+
+    /** @var \OC\User\User | \PHPUnit\Framework\MockObject\MockObject $user */
+    private $user;
+
+    /** @var \OCP\IConfig|\PHPUnit\Framework\MockObject\MockObject */
+    private $config;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->folder = $this->createMock(SimpleFolder::class);
+        // abcdefghi is a convenient name that our algorithm convert to our nextcloud blue 0082c9
+        $this->user = $this->getUserWithDisplayName('abcdefghi');
+        $this->config = $this->createMock(IConfig::class);
+
+        $this->avatar = $this->getUserAvatar($this->user);
+    }
+
+    public static function avatarTextData(): array {
+        return [
+            ['', '?'],
+            ['matchish', 'M'],
+            ['Firstname Lastname', 'FL'],
+            ['Firstname Lastname Rest', 'FL'],
+        ];
+    }
+
+    public function testGetNoAvatar(): void {
+        $file = $this->createMock(ISimpleFile::class);
+        $this->folder->method('newFile')
+            ->willReturn($file);
+
+        $this->folder->method('getFile')
+            ->willReturnCallback(function (string $path) {
+                if ($path === 'avatar.64.png') {
+                    throw new NotFoundException();
+                }
+            });
+        $this->folder->method('fileExists')
+            ->willReturnCallback(function ($path) {
+                if ($path === 'generated') {
+                    return true;
+                }
+                return false;
+            });
+
+        $data = null;
+        $file->method('putContent')
+            ->with($this->callback(function ($d) use (&$data) {
+                $data = $d;
+                return true;
+            }));
+
+        $file->method('getContent')
+            ->willReturnCallback(function () use (&$data) {
+                return $data;
+            });
+
+        $result = $this->avatar->get();
+        $this->assertTrue($result->valid());
+    }
+
+    public function testGetAvatarSizeMatch(): void {
+        $this->folder->method('fileExists')
+            ->willReturnMap([
+                ['avatar.jpg', true],
+                ['avatar.128.jpg', true],
+                ['generated', false],
+            ]);
+
+        $expected = new Image();
+        $expected->loadFromFile(\OC::$SERVERROOT . '/tests/data/testavatar.png');
+
+        $file = $this->createMock(ISimpleFile::class);
+        $file->method('getContent')->willReturn($expected->data());
+        $this->folder->method('getFile')->with('avatar.128.jpg')->willReturn($file);
+
+        $this->assertEquals($expected->data(), $this->avatar->get(128)->data());
+    }
+
+    public function testGetAvatarSizeMinusOne(): void {
+        $this->folder->method('fileExists')
+            ->willReturnMap([
+                ['avatar.jpg', true],
+                ['generated', false],
+            ]);
+
+        $expected = new Image();
+        $expected->loadFromFile(\OC::$SERVERROOT . '/tests/data/testavatar.png');
+
+        $file = $this->createMock(ISimpleFile::class);
+        $file->method('getContent')->willReturn($expected->data());
+        $this->folder->method('getFile')->with('avatar.jpg')->willReturn($file);
+
+        $this->assertEquals($expected->data(), $this->avatar->get(-1)->data());
+    }
+
+    public function testGetAvatarNoSizeMatch(): void {
+        $this->folder->method('fileExists')
+            ->willReturnMap([
+                ['avatar.jpg', false],
+                ['avatar.png', true],
+                ['avatar.32.png', false],
+                ['generated', false],
+            ]);
+
+        $expected = new Image();
+        $expected->loadFromFile(\OC::$SERVERROOT . '/tests/data/testavatar.png');
+        $expected2 = new Image();
+        $expected2->loadFromFile(\OC::$SERVERROOT . '/tests/data/testavatar.png');
+        $expected2->resize(32);
+
+        $file = $this->createMock(ISimpleFile::class);
+        $file->method('getContent')->willReturn($expected->data());
+
+        $this->folder->method('getFile')
+            ->willReturnCallback(
+                function ($path) use ($file) {
+                    if ($path === 'avatar.png') {
+                        return $file;
+                    } else {
+                        throw new \OCP\Files\NotFoundException;
+                    }
+                }
+            );
+
+        $newFile = $this->createMock(ISimpleFile::class);
+        $newFile->expects($this->once())
+            ->method('putContent')
+            ->with($expected2->data());
+        $newFile->expects($this->once())
+            ->method('getContent')
+            ->willReturn($expected2->data());
+        $this->folder->expects($this->once())
+            ->method('newFile')
+            ->with('avatar.32.png')
+            ->willReturn($newFile);
+
+        $this->assertEquals($expected2->data(), $this->avatar->get(32)->data());
+    }
+
+    public function testExistsNo(): void {
+        $this->assertFalse($this->avatar->exists());
+    }
+
+    public function testExiststJPG(): void {
+        $this->folder->method('fileExists')
+            ->willReturnMap([
+                ['avatar.jpg', true],
+                ['avatar.png', false],
+            ]);
+        $this->assertTrue($this->avatar->exists());
+    }
+
+    public function testExistsPNG(): void {
+        $this->folder->method('fileExists')
+            ->willReturnMap([
+                ['avatar.jpg', false],
+                ['avatar.png', true],
+            ]);
+        $this->assertTrue($this->avatar->exists());
+    }
+
+    public function testSetAvatar(): void {
+        $avatarFileJPG = $this->createMock(File::class);
+        $avatarFileJPG->method('getName')
+            ->willReturn('avatar.jpg');
+        $avatarFileJPG->expects($this->once())->method('delete');
+
+        $avatarFilePNG = $this->createMock(File::class);
+        $avatarFilePNG->method('getName')
+            ->willReturn('avatar.png');
+        $avatarFilePNG->expects($this->once())->method('delete');
+
+        $resizedAvatarFile = $this->createMock(File::class);
+        $resizedAvatarFile->method('getName')
+            ->willReturn('avatar.32.jpg');
+        $resizedAvatarFile->expects($this->once())->method('delete');
+
+        $this->folder->method('getDirectoryListing')
+            ->willReturn([$avatarFileJPG, $avatarFilePNG, $resizedAvatarFile]);
+
+        $generated = $this->createMock(ISimpleFile::class);
+        $this->folder->method('getFile')
+            ->with('generated')
+            ->willReturn($generated);
+
+        $newFile = $this->createMock(ISimpleFile::class);
+        $this->folder->expects($this->once())
+            ->method('newFile')
+            ->with('avatar.png')
+            ->willReturn($newFile);
+
+        $image = new Image();
+        $image->loadFromFile(\OC::$SERVERROOT . '/tests/data/testavatar.png');
+        $newFile->expects($this->once())
+            ->method('putContent')
+            ->with($image->data());
+
+        $this->config->expects($this->exactly(3))
+            ->method('setUserValue');
+        $this->config->expects($this->once())
+            ->method('getUserValue');
+
+        $this->user->expects($this->exactly(1))->method('triggerChange');
+
+        $this->avatar->set($image->data());
+    }
+
+    public function testGenerateSvgAvatar(): void {
+        $avatar = $this->invokePrivate($this->avatar, 'getAvatarVector', [64, false]);
+
+        $svg = '<?xml version="1.0" encoding="UTF-8" standalone="no"?>
 		<svg width="64" height="64" version="1.1" viewBox="0 0 500 500" xmlns="http://www.w3.org/2000/svg">
 			<rect width="100%" height="100%" fill="#e5f2f9"></rect>
 			<text x="50%" y="350" style="font-weight:normal;font-size:280px;font-family:\'Noto Sans\';text-anchor:middle;fill:#0082c9">A</text>
 		</svg>';
-		$this->assertEquals($avatar, $svg);
-	}
-
-
-	/**
-	 * @dataProvider avatarTextData
-	 */
-	public function testGetAvatarText($displayName, $expectedAvatarText): void {
-		$user = $this->getUserWithDisplayName($displayName);
-		$avatar = $this->getUserAvatar($user);
-
-		$avatarText = $this->invokePrivate($avatar, 'getAvatarText');
-		$this->assertEquals($expectedAvatarText, $avatarText);
-	}
-
-	public function testHashToInt(): void {
-		$hashToInt = $this->invokePrivate($this->avatar, 'hashToInt', ['abcdef', 18]);
-		$this->assertTrue(gettype($hashToInt) === 'integer');
-	}
-
-	public function testMixPalette(): void {
-		$colorFrom = new \OCP\Color(0, 0, 0);
-		$colorTo = new \OCP\Color(6, 12, 18);
-		$steps = 6;
-		$palette = \OCP\Color::mixPalette($steps, $colorFrom, $colorTo);
-		foreach ($palette as $j => $color) {
-			// calc increment
-			$incR = $colorTo->red() / $steps * $j;
-			$incG = $colorTo->green() / $steps * $j;
-			$incB = $colorTo->blue() / $steps * $j;
-			// ensure everything is equal
-			$this->assertEquals($color, new \OCP\Color($incR, $incG, $incB));
-		}
-		$hashToInt = $this->invokePrivate($this->avatar, 'hashToInt', ['abcdef', 18]);
-		$this->assertTrue(gettype($hashToInt) === 'integer');
-	}
-
-	private function getUserWithDisplayName($name) {
-		$user = $this->createMock(User::class);
-		$user->method('getDisplayName')->willReturn($name);
-		return $user;
-	}
-
-	private function getUserAvatar($user) {
-		/** @var \OCP\IL10N | \PHPUnit\Framework\MockObject\MockObject $l */
-		$l = $this->createMock(IL10N::class);
-		$l->method('t')->willReturnArgument(0);
-
-		return new \OC\Avatar\UserAvatar(
-			$this->folder,
-			$l,
-			$user,
-			$this->createMock(LoggerInterface::class),
-			$this->config
-		);
-	}
+        $this->assertEquals($avatar, $svg);
+    }
+
+
+    /**
+     * @dataProvider avatarTextData
+     */
+    public function testGetAvatarText($displayName, $expectedAvatarText): void {
+        $user = $this->getUserWithDisplayName($displayName);
+        $avatar = $this->getUserAvatar($user);
+
+        $avatarText = $this->invokePrivate($avatar, 'getAvatarText');
+        $this->assertEquals($expectedAvatarText, $avatarText);
+    }
+
+    public function testHashToInt(): void {
+        $hashToInt = $this->invokePrivate($this->avatar, 'hashToInt', ['abcdef', 18]);
+        $this->assertTrue(gettype($hashToInt) === 'integer');
+    }
+
+    public function testMixPalette(): void {
+        $colorFrom = new \OCP\Color(0, 0, 0);
+        $colorTo = new \OCP\Color(6, 12, 18);
+        $steps = 6;
+        $palette = \OCP\Color::mixPalette($steps, $colorFrom, $colorTo);
+        foreach ($palette as $j => $color) {
+            // calc increment
+            $incR = $colorTo->red() / $steps * $j;
+            $incG = $colorTo->green() / $steps * $j;
+            $incB = $colorTo->blue() / $steps * $j;
+            // ensure everything is equal
+            $this->assertEquals($color, new \OCP\Color($incR, $incG, $incB));
+        }
+        $hashToInt = $this->invokePrivate($this->avatar, 'hashToInt', ['abcdef', 18]);
+        $this->assertTrue(gettype($hashToInt) === 'integer');
+    }
+
+    private function getUserWithDisplayName($name) {
+        $user = $this->createMock(User::class);
+        $user->method('getDisplayName')->willReturn($name);
+        return $user;
+    }
+
+    private function getUserAvatar($user) {
+        /** @var \OCP\IL10N | \PHPUnit\Framework\MockObject\MockObject $l */
+        $l = $this->createMock(IL10N::class);
+        $l->method('t')->willReturnArgument(0);
+
+        return new \OC\Avatar\UserAvatar(
+            $this->folder,
+            $l,
+            $user,
+            $this->createMock(LoggerInterface::class),
+            $this->config
+        );
+    }
 }

tests/lib/Log/PsrLoggerAdapterTest.php

Indentation +59 added lines, -59 removed lines

@@ -18,71 +18,71 @@
 use Test\TestCase;
 
 class PsrLoggerAdapterTest extends TestCase {
-	protected Log&MockObject $logger;
-	protected PsrLoggerAdapter $loggerAdapter;
+    protected Log&MockObject $logger;
+    protected PsrLoggerAdapter $loggerAdapter;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->logger = $this->createMock(Log::class);
-		$this->loggerAdapter = new PsrLoggerAdapter($this->logger);
-	}
+        $this->logger = $this->createMock(Log::class);
+        $this->loggerAdapter = new PsrLoggerAdapter($this->logger);
+    }
 
-	/**
-	 * @dataProvider dataPsrLoggingLevels
-	 */
-	public function testLoggingWithPsrLogLevels(string $level, int $expectedLevel): void {
-		$this->logger->expects(self::once())
-			->method('log')
-			->with($expectedLevel, 'test message', ['app' => 'test']);
-		$this->loggerAdapter->log($level, 'test message', ['app' => 'test']);
-	}
+    /**
+     * @dataProvider dataPsrLoggingLevels
+     */
+    public function testLoggingWithPsrLogLevels(string $level, int $expectedLevel): void {
+        $this->logger->expects(self::once())
+            ->method('log')
+            ->with($expectedLevel, 'test message', ['app' => 'test']);
+        $this->loggerAdapter->log($level, 'test message', ['app' => 'test']);
+    }
 
-	/**
-	 * @dataProvider dataPsrLoggingLevels
-	 */
-	public function testLogLevelToInt(string $level, int $expectedLevel): void {
-		$this->assertEquals($expectedLevel, PsrLoggerAdapter::logLevelToInt($level));
-	}
+    /**
+     * @dataProvider dataPsrLoggingLevels
+     */
+    public function testLogLevelToInt(string $level, int $expectedLevel): void {
+        $this->assertEquals($expectedLevel, PsrLoggerAdapter::logLevelToInt($level));
+    }
 
-	public static function dataPsrLoggingLevels(): array {
-		return [
-			[LogLevel::ALERT, ILogger::ERROR],
-			[LogLevel::CRITICAL, ILogger::ERROR],
-			[LogLevel::DEBUG, ILogger::DEBUG],
-			[LogLevel::EMERGENCY, ILogger::FATAL],
-			[LogLevel::ERROR, ILogger::ERROR],
-			[LogLevel::INFO, ILogger::INFO],
-			[LogLevel::NOTICE, ILogger::INFO],
-			[LogLevel::WARNING, ILogger::WARN],
-		];
-	}
+    public static function dataPsrLoggingLevels(): array {
+        return [
+            [LogLevel::ALERT, ILogger::ERROR],
+            [LogLevel::CRITICAL, ILogger::ERROR],
+            [LogLevel::DEBUG, ILogger::DEBUG],
+            [LogLevel::EMERGENCY, ILogger::FATAL],
+            [LogLevel::ERROR, ILogger::ERROR],
+            [LogLevel::INFO, ILogger::INFO],
+            [LogLevel::NOTICE, ILogger::INFO],
+            [LogLevel::WARNING, ILogger::WARN],
+        ];
+    }
 
-	/**
-	 * @dataProvider dataInvalidLoggingLevel
-	 */
-	public function testInvalidLoggingLevel($level): void {
-		$this->logger->expects(self::never())
-			->method('log');
-		$this->expectException(InvalidArgumentException::class);
+    /**
+     * @dataProvider dataInvalidLoggingLevel
+     */
+    public function testInvalidLoggingLevel($level): void {
+        $this->logger->expects(self::never())
+            ->method('log');
+        $this->expectException(InvalidArgumentException::class);
 
-		$this->loggerAdapter->log($level, 'valid message');
-	}
+        $this->loggerAdapter->log($level, 'valid message');
+    }
 
-	public static function dataInvalidLoggingLevel(): array {
-		return [
-			// invalid string
-			['this is not a level'],
-			// int out of range
-			[ILogger::DEBUG - 1],
-			[ILogger::FATAL + 1],
-			// float is not allowed
-			[1.2345],
-			// boolean is not a level
-			[true],
-			[false],
-			//
-			[null],
-		];
-	}
+    public static function dataInvalidLoggingLevel(): array {
+        return [
+            // invalid string
+            ['this is not a level'],
+            // int out of range
+            [ILogger::DEBUG - 1],
+            [ILogger::FATAL + 1],
+            // float is not allowed
+            [1.2345],
+            // boolean is not a level
+            [true],
+            [false],
+            //
+            [null],
+        ];
+    }
 }

tests/lib/HelperStorageTest.php

Indentation +227 added lines, -227 removed lines

@@ -18,231 +18,231 @@
  * @group DB
  */
 class HelperStorageTest extends \Test\TestCase {
-	use UserTrait;
-
-	/** @var string */
-	private $user;
-	/** @var \OC\Files\Storage\Storage */
-	private $storageMock;
-	/** @var \OC\Files\Storage\Storage */
-	private $storage;
-	private bool $savedQuotaIncludeExternalStorage;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->user = $this->getUniqueID('user_');
-		$this->createUser($this->user, $this->user);
-		$this->savedQuotaIncludeExternalStorage = $this->getIncludeExternalStorage();
-
-		\OC\Files\Filesystem::tearDown();
-		\OC_User::setUserId($this->user);
-		\OC\Files\Filesystem::init($this->user, '/' . $this->user . '/files');
-
-		/** @var IMountManager $manager */
-		$manager = \OC::$server->get(IMountManager::class);
-		$manager->removeMount('/' . $this->user);
-
-		$this->storageMock = null;
-	}
-
-	protected function tearDown(): void {
-		$this->setIncludeExternalStorage($this->savedQuotaIncludeExternalStorage);
-		$this->user = null;
-
-		if ($this->storageMock) {
-			$this->storageMock->getCache()->clear();
-			$this->storageMock = null;
-		}
-		\OC\Files\Filesystem::tearDown();
-
-		\OC_User::setUserId('');
-		\OC::$server->getConfig()->deleteAllUserValues($this->user);
-
-		parent::tearDown();
-	}
-
-	/**
-	 * Returns a storage mock that returns the given value as
-	 * free space
-	 *
-	 * @param int $freeSpace free space value
-	 * @return \OC\Files\Storage\Storage
-	 */
-	private function getStorageMock($freeSpace = 12) {
-		$this->storageMock = $this->getMockBuilder(Temporary::class)
-			->onlyMethods(['free_space'])
-			->setConstructorArgs([[]])
-			->getMock();
-
-		$this->storageMock->expects($this->once())
-			->method('free_space')
-			->willReturn($freeSpace);
-		return $this->storageMock;
-	}
-
-	/**
-	 * Test getting the storage info
-	 */
-	public function testGetStorageInfo(): void {
-		$homeStorage = $this->getStorageMock(12);
-		\OC\Files\Filesystem::mount($homeStorage, [], '/' . $this->user . '/files');
-		$homeStorage->file_put_contents('test.txt', '01234');
-
-		$storageInfo = \OC_Helper::getStorageInfo('');
-		$this->assertEquals(12, $storageInfo['free']);
-		$this->assertEquals(5, $storageInfo['used']);
-		$this->assertEquals(17, $storageInfo['total']);
-	}
-	private function getIncludeExternalStorage(): bool {
-		$class = new \ReflectionClass(\OC_Helper::class);
-		$prop = $class->getProperty('quotaIncludeExternalStorage');
-		$prop->setAccessible(true);
-		return $prop->getValue(null) ?? false;
-	}
-
-	private function setIncludeExternalStorage(bool $include) {
-		$class = new \ReflectionClass(\OC_Helper::class);
-		$prop = $class->getProperty('quotaIncludeExternalStorage');
-		$prop->setAccessible(true);
-		$prop->setValue(null, $include);
-	}
-
-	/**
-	 * Test getting the storage info, ignoring extra mount points
-	 */
-	public function testGetStorageInfoExcludingExtStorage(): void {
-		$homeStorage = $this->getStorageMock(12);
-		\OC\Files\Filesystem::mount($homeStorage, [], '/' . $this->user . '/files');
-		$homeStorage->file_put_contents('test.txt', '01234');
-
-		$extStorage = new \OC\Files\Storage\Temporary([]);
-		$extStorage->file_put_contents('extfile.txt', 'abcdefghijklmnopq');
-		$extStorage->getScanner()->scan(''); // update root size
-
-		$this->setIncludeExternalStorage(false);
-
-		\OC\Files\Filesystem::mount($extStorage, [], '/' . $this->user . '/files/ext');
-
-		$storageInfo = \OC_Helper::getStorageInfo('');
-		$this->assertEquals(12, $storageInfo['free']);
-		$this->assertEquals(5, $storageInfo['used']);
-		$this->assertEquals(17, $storageInfo['total']);
-	}
-
-	/**
-	 * Test getting the storage info, including extra mount points
-	 */
-	public function testGetStorageInfoIncludingExtStorage(): void {
-		$homeStorage = new \OC\Files\Storage\Temporary([]);
-		\OC\Files\Filesystem::mount($homeStorage, [], '/' . $this->user . '/files');
-		$homeStorage->file_put_contents('test.txt', '01234');
-
-		$extStorage = new \OC\Files\Storage\Temporary([]);
-		$extStorage->file_put_contents('extfile.txt', 'abcdefghijklmnopq');
-		$extStorage->getScanner()->scan(''); // update root size
-
-		\OC\Files\Filesystem::mount($extStorage, [], '/' . $this->user . '/files/ext');
-
-		$this->setIncludeExternalStorage(true);
-
-		$config = \OC::$server->get(IConfig::class);
-		$config->setUserValue($this->user, 'files', 'quota', '25');
-
-		$storageInfo = \OC_Helper::getStorageInfo('');
-		$this->assertEquals(3, $storageInfo['free']);
-		$this->assertEquals(22, $storageInfo['used']);
-		$this->assertEquals(25, $storageInfo['total']);
-
-		$config->setUserValue($this->user, 'files', 'quota', 'default');
-	}
-
-	/**
-	 * Test getting the storage info excluding extra mount points
-	 * when user has no quota set, even when quota ext storage option
-	 * was set
-	 */
-	public function testGetStorageInfoIncludingExtStorageWithNoUserQuota(): void {
-		$homeStorage = $this->getStorageMock(12);
-		\OC\Files\Filesystem::mount($homeStorage, [], '/' . $this->user . '/files');
-		$homeStorage->file_put_contents('test.txt', '01234');
-
-		$extStorage = new \OC\Files\Storage\Temporary([]);
-		$extStorage->file_put_contents('extfile.txt', 'abcdefghijklmnopq');
-		$extStorage->getScanner()->scan(''); // update root size
-
-		\OC\Files\Filesystem::mount($extStorage, [], '/' . $this->user . '/files/ext');
-
-		$config = \OC::$server->getConfig();
-		$this->setIncludeExternalStorage(true);
-
-		$storageInfo = \OC_Helper::getStorageInfo('');
-		$this->assertEquals(12, $storageInfo['free'], '12 bytes free in home storage');
-		$this->assertEquals(22, $storageInfo['used'], '5 bytes of home storage and 17 bytes of the temporary storage are used');
-		$this->assertEquals(34, $storageInfo['total'], '5 bytes used and 12 bytes free in home storage as well as 17 bytes used in temporary storage');
-	}
-
-
-	/**
-	 * Test getting the storage info with quota enabled
-	 */
-	public function testGetStorageInfoWithQuota(): void {
-		$homeStorage = $this->getStorageMock(12);
-		$homeStorage->file_put_contents('test.txt', '01234');
-		$homeStorage = new \OC\Files\Storage\Wrapper\Quota(
-			[
-				'storage' => $homeStorage,
-				'quota' => 7
-			]
-		);
-		\OC\Files\Filesystem::mount($homeStorage, [], '/' . $this->user . '/files');
-
-		$storageInfo = \OC_Helper::getStorageInfo('');
-		$this->assertEquals(2, $storageInfo['free']);
-		$this->assertEquals(5, $storageInfo['used']);
-		$this->assertEquals(7, $storageInfo['total']);
-	}
-
-	/**
-	 * Test getting the storage info when data exceeds quota
-	 */
-	public function testGetStorageInfoWhenSizeExceedsQuota(): void {
-		$homeStorage = $this->getStorageMock(12);
-		$homeStorage->file_put_contents('test.txt', '0123456789');
-		$homeStorage = new \OC\Files\Storage\Wrapper\Quota(
-			[
-				'storage' => $homeStorage,
-				'quota' => 7
-			]
-		);
-		\OC\Files\Filesystem::mount($homeStorage, [], '/' . $this->user . '/files');
-
-		$storageInfo = \OC_Helper::getStorageInfo('');
-		$this->assertEquals(0, $storageInfo['free']);
-		$this->assertEquals(10, $storageInfo['used']);
-		// total = quota
-		$this->assertEquals(7, $storageInfo['total']);
-	}
-
-	/**
-	 * Test getting the storage info when the remaining
-	 * free storage space is less than the quota
-	 */
-	public function testGetStorageInfoWhenFreeSpaceLessThanQuota(): void {
-		$homeStorage = $this->getStorageMock(12);
-		$homeStorage->file_put_contents('test.txt', '01234');
-		$homeStorage = new \OC\Files\Storage\Wrapper\Quota(
-			[
-				'storage' => $homeStorage,
-				'quota' => 18
-			]
-		);
-		\OC\Files\Filesystem::mount($homeStorage, [], '/' . $this->user . '/files');
-
-		$storageInfo = \OC_Helper::getStorageInfo('');
-		$this->assertEquals(12, $storageInfo['free']);
-		$this->assertEquals(5, $storageInfo['used']);
-		// total = free + used (because quota > total)
-		$this->assertEquals(17, $storageInfo['total']);
-	}
+    use UserTrait;
+
+    /** @var string */
+    private $user;
+    /** @var \OC\Files\Storage\Storage */
+    private $storageMock;
+    /** @var \OC\Files\Storage\Storage */
+    private $storage;
+    private bool $savedQuotaIncludeExternalStorage;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->user = $this->getUniqueID('user_');
+        $this->createUser($this->user, $this->user);
+        $this->savedQuotaIncludeExternalStorage = $this->getIncludeExternalStorage();
+
+        \OC\Files\Filesystem::tearDown();
+        \OC_User::setUserId($this->user);
+        \OC\Files\Filesystem::init($this->user, '/' . $this->user . '/files');
+
+        /** @var IMountManager $manager */
+        $manager = \OC::$server->get(IMountManager::class);
+        $manager->removeMount('/' . $this->user);
+
+        $this->storageMock = null;
+    }
+
+    protected function tearDown(): void {
+        $this->setIncludeExternalStorage($this->savedQuotaIncludeExternalStorage);
+        $this->user = null;
+
+        if ($this->storageMock) {
+            $this->storageMock->getCache()->clear();
+            $this->storageMock = null;
+        }
+        \OC\Files\Filesystem::tearDown();
+
+        \OC_User::setUserId('');
+        \OC::$server->getConfig()->deleteAllUserValues($this->user);
+
+        parent::tearDown();
+    }
+
+    /**
+     * Returns a storage mock that returns the given value as
+     * free space
+     *
+     * @param int $freeSpace free space value
+     * @return \OC\Files\Storage\Storage
+     */
+    private function getStorageMock($freeSpace = 12) {
+        $this->storageMock = $this->getMockBuilder(Temporary::class)
+            ->onlyMethods(['free_space'])
+            ->setConstructorArgs([[]])
+            ->getMock();
+
+        $this->storageMock->expects($this->once())
+            ->method('free_space')
+            ->willReturn($freeSpace);
+        return $this->storageMock;
+    }
+
+    /**
+     * Test getting the storage info
+     */
+    public function testGetStorageInfo(): void {
+        $homeStorage = $this->getStorageMock(12);
+        \OC\Files\Filesystem::mount($homeStorage, [], '/' . $this->user . '/files');
+        $homeStorage->file_put_contents('test.txt', '01234');
+
+        $storageInfo = \OC_Helper::getStorageInfo('');
+        $this->assertEquals(12, $storageInfo['free']);
+        $this->assertEquals(5, $storageInfo['used']);
+        $this->assertEquals(17, $storageInfo['total']);
+    }
+    private function getIncludeExternalStorage(): bool {
+        $class = new \ReflectionClass(\OC_Helper::class);
+        $prop = $class->getProperty('quotaIncludeExternalStorage');
+        $prop->setAccessible(true);
+        return $prop->getValue(null) ?? false;
+    }
+
+    private function setIncludeExternalStorage(bool $include) {
+        $class = new \ReflectionClass(\OC_Helper::class);
+        $prop = $class->getProperty('quotaIncludeExternalStorage');
+        $prop->setAccessible(true);
+        $prop->setValue(null, $include);
+    }
+
+    /**
+     * Test getting the storage info, ignoring extra mount points
+     */
+    public function testGetStorageInfoExcludingExtStorage(): void {
+        $homeStorage = $this->getStorageMock(12);
+        \OC\Files\Filesystem::mount($homeStorage, [], '/' . $this->user . '/files');
+        $homeStorage->file_put_contents('test.txt', '01234');
+
+        $extStorage = new \OC\Files\Storage\Temporary([]);
+        $extStorage->file_put_contents('extfile.txt', 'abcdefghijklmnopq');
+        $extStorage->getScanner()->scan(''); // update root size
+
+        $this->setIncludeExternalStorage(false);
+
+        \OC\Files\Filesystem::mount($extStorage, [], '/' . $this->user . '/files/ext');
+
+        $storageInfo = \OC_Helper::getStorageInfo('');
+        $this->assertEquals(12, $storageInfo['free']);
+        $this->assertEquals(5, $storageInfo['used']);
+        $this->assertEquals(17, $storageInfo['total']);
+    }
+
+    /**
+     * Test getting the storage info, including extra mount points
+     */
+    public function testGetStorageInfoIncludingExtStorage(): void {
+        $homeStorage = new \OC\Files\Storage\Temporary([]);
+        \OC\Files\Filesystem::mount($homeStorage, [], '/' . $this->user . '/files');
+        $homeStorage->file_put_contents('test.txt', '01234');
+
+        $extStorage = new \OC\Files\Storage\Temporary([]);
+        $extStorage->file_put_contents('extfile.txt', 'abcdefghijklmnopq');
+        $extStorage->getScanner()->scan(''); // update root size
+
+        \OC\Files\Filesystem::mount($extStorage, [], '/' . $this->user . '/files/ext');
+
+        $this->setIncludeExternalStorage(true);
+
+        $config = \OC::$server->get(IConfig::class);
+        $config->setUserValue($this->user, 'files', 'quota', '25');
+
+        $storageInfo = \OC_Helper::getStorageInfo('');
+        $this->assertEquals(3, $storageInfo['free']);
+        $this->assertEquals(22, $storageInfo['used']);
+        $this->assertEquals(25, $storageInfo['total']);
+
+        $config->setUserValue($this->user, 'files', 'quota', 'default');
+    }
+
+    /**
+     * Test getting the storage info excluding extra mount points
+     * when user has no quota set, even when quota ext storage option
+     * was set
+     */
+    public function testGetStorageInfoIncludingExtStorageWithNoUserQuota(): void {
+        $homeStorage = $this->getStorageMock(12);
+        \OC\Files\Filesystem::mount($homeStorage, [], '/' . $this->user . '/files');
+        $homeStorage->file_put_contents('test.txt', '01234');
+
+        $extStorage = new \OC\Files\Storage\Temporary([]);
+        $extStorage->file_put_contents('extfile.txt', 'abcdefghijklmnopq');
+        $extStorage->getScanner()->scan(''); // update root size
+
+        \OC\Files\Filesystem::mount($extStorage, [], '/' . $this->user . '/files/ext');
+
+        $config = \OC::$server->getConfig();
+        $this->setIncludeExternalStorage(true);
+
+        $storageInfo = \OC_Helper::getStorageInfo('');
+        $this->assertEquals(12, $storageInfo['free'], '12 bytes free in home storage');
+        $this->assertEquals(22, $storageInfo['used'], '5 bytes of home storage and 17 bytes of the temporary storage are used');
+        $this->assertEquals(34, $storageInfo['total'], '5 bytes used and 12 bytes free in home storage as well as 17 bytes used in temporary storage');
+    }
+
+
+    /**
+     * Test getting the storage info with quota enabled
+     */
+    public function testGetStorageInfoWithQuota(): void {
+        $homeStorage = $this->getStorageMock(12);
+        $homeStorage->file_put_contents('test.txt', '01234');
+        $homeStorage = new \OC\Files\Storage\Wrapper\Quota(
+            [
+                'storage' => $homeStorage,
+                'quota' => 7
+            ]
+        );
+        \OC\Files\Filesystem::mount($homeStorage, [], '/' . $this->user . '/files');
+
+        $storageInfo = \OC_Helper::getStorageInfo('');
+        $this->assertEquals(2, $storageInfo['free']);
+        $this->assertEquals(5, $storageInfo['used']);
+        $this->assertEquals(7, $storageInfo['total']);
+    }
+
+    /**
+     * Test getting the storage info when data exceeds quota
+     */
+    public function testGetStorageInfoWhenSizeExceedsQuota(): void {
+        $homeStorage = $this->getStorageMock(12);
+        $homeStorage->file_put_contents('test.txt', '0123456789');
+        $homeStorage = new \OC\Files\Storage\Wrapper\Quota(
+            [
+                'storage' => $homeStorage,
+                'quota' => 7
+            ]
+        );
+        \OC\Files\Filesystem::mount($homeStorage, [], '/' . $this->user . '/files');
+
+        $storageInfo = \OC_Helper::getStorageInfo('');
+        $this->assertEquals(0, $storageInfo['free']);
+        $this->assertEquals(10, $storageInfo['used']);
+        // total = quota
+        $this->assertEquals(7, $storageInfo['total']);
+    }
+
+    /**
+     * Test getting the storage info when the remaining
+     * free storage space is less than the quota
+     */
+    public function testGetStorageInfoWhenFreeSpaceLessThanQuota(): void {
+        $homeStorage = $this->getStorageMock(12);
+        $homeStorage->file_put_contents('test.txt', '01234');
+        $homeStorage = new \OC\Files\Storage\Wrapper\Quota(
+            [
+                'storage' => $homeStorage,
+                'quota' => 18
+            ]
+        );
+        \OC\Files\Filesystem::mount($homeStorage, [], '/' . $this->user . '/files');
+
+        $storageInfo = \OC_Helper::getStorageInfo('');
+        $this->assertEquals(12, $storageInfo['free']);
+        $this->assertEquals(5, $storageInfo['used']);
+        // total = free + used (because quota > total)
+        $this->assertEquals(17, $storageInfo['total']);
+    }
 }