nextcloud / server

apps/dav/tests/unit/CalDAV/Reminder/NotificationProvider/EmailProviderTest.php

Indentation +487 added lines, -487 removed lines

@@ -19,491 +19,491 @@
 use Sabre\VObject\Component\VCalendar;
 
 class EmailProviderTest extends AbstractNotificationProviderTestCase {
-	public const USER_EMAIL = 'frodo@hobb.it';
-	private IMailer&MockObject $mailer;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->mailer = $this->createMock(IMailer::class);
-
-		$this->provider = new EmailProvider(
-			$this->config,
-			$this->mailer,
-			$this->logger,
-			$this->l10nFactory,
-			$this->urlGenerator
-		);
-	}
-
-	public function testSendWithoutAttendees():void {
-		[$user1, $user2, $user3, , $user5] = $users = $this->getUsers();
-		$principalEmailAddresses = [$user1->getEmailAddress()];
-
-		$enL10N = $this->createMock(IL10N::class);
-		$enL10N->method('t')
-			->willReturnArgument(0);
-		$enL10N->method('l')
-			->willReturnArgument(0);
-
-		$deL10N = $this->createMock(IL10N::class);
-		$deL10N->method('t')
-			->willReturnArgument(0);
-		$deL10N->method('l')
-			->willReturnArgument(0);
-
-		$this->l10nFactory
-			->method('getUserLanguage')
-			->willReturnMap([
-				[$user1, 'en'],
-				[$user2, 'de'],
-				[$user3, 'de'],
-				[$user5, 'de'],
-			]);
-
-		$this->l10nFactory
-			->method('findGenericLanguage')
-			->willReturn('en');
-
-		$this->l10nFactory
-			->method('languageExists')
-			->willReturnMap([
-				['dav', 'en', true],
-				['dav', 'de', true],
-			]);
-
-		$this->l10nFactory
-			->method('get')
-			->willReturnMap([
-				['dav', 'en', null, $enL10N],
-				['dav', 'de', null, $deL10N],
-			]);
-
-		$template1 = $this->getTemplateMock();
-		$message11 = $this->getMessageMock('uid1@example.com', $template1);
-		$template2 = $this->getTemplateMock();
-		$message21 = $this->getMessageMock('uid2@example.com', $template2);
-		$message22 = $this->getMessageMock('uid3@example.com', $template2);
-
-		$this->mailer->expects($this->exactly(2))
-			->method('createEMailTemplate')
-			->with('dav.calendarReminder')
-			->willReturnOnConsecutiveCalls(
-				$template1,
-				$template2
-			);
-
-		$this->mailer->expects($this->exactly(4))
-			->method('validateMailAddress')
-			->willReturnMap([
-				['uid1@example.com', true],
-				['uid2@example.com', true],
-				['uid3@example.com', true],
-				['invalid', false],
-			]);
-
-		$this->mailer->expects($this->exactly(3))
-			->method('createMessage')
-			->with()
-			->willReturnOnConsecutiveCalls(
-				$message11,
-				$message21,
-				$message22
-			);
-
-		$calls = [
-			[$message11],
-			[$message21],
-			[$message22],
-		];
-		$this->mailer->expects($this->exactly(count($calls)))
-			->method('send')
-			->willReturnCallback(function () use (&$calls) {
-				$expected = array_shift($calls);
-				$this->assertEquals($expected, func_get_args());
-				return [];
-			});
-
-		$this->setupURLGeneratorMock(2);
-
-		$vcalendar = $this->getNoAttendeeVCalendar();
-		$this->provider->send($vcalendar->VEVENT, $this->calendarDisplayName, $principalEmailAddresses, $users);
-	}
-
-	public function testSendWithAttendeesWhenOwnerIsOrganizer(): void {
-		[$user1, $user2, $user3, , $user5] = $users = $this->getUsers();
-		$principalEmailAddresses = [$user1->getEmailAddress()];
-
-		$enL10N = $this->createMock(IL10N::class);
-		$enL10N->method('t')
-			->willReturnArgument(0);
-		$enL10N->method('l')
-			->willReturnArgument(0);
-
-		$deL10N = $this->createMock(IL10N::class);
-		$deL10N->method('t')
-			->willReturnArgument(0);
-		$deL10N->method('l')
-			->willReturnArgument(0);
-
-		$this->l10nFactory
-			->method('getUserLanguage')
-			->willReturnMap([
-				[$user1, 'en'],
-				[$user2, 'de'],
-				[$user3, 'de'],
-				[$user5, 'de'],
-			]);
-
-		$this->l10nFactory
-			->method('findGenericLanguage')
-			->willReturn('en');
-
-		$this->l10nFactory
-			->method('languageExists')
-			->willReturnMap([
-				['dav', 'en', true],
-				['dav', 'de', true],
-			]);
-
-		$this->l10nFactory
-			->method('get')
-			->willReturnMap([
-				['dav', 'en', null, $enL10N],
-				['dav', 'de', null, $deL10N],
-			]);
-
-		$template1 = $this->getTemplateMock();
-		$message11 = $this->getMessageMock('foo1@example.org', $template1);
-		$message12 = $this->getMessageMock('uid2@example.com', $template1);
-		$message13 = $this->getMessageMock('uid3@example.com', $template1);
-		$template2 = $this->getTemplateMock();
-		$message21 = $this->getMessageMock('foo3@example.org', $template2);
-		$message22 = $this->getMessageMock('foo4@example.org', $template2);
-		$message23 = $this->getMessageMock('uid1@example.com', $template2);
-
-		$this->mailer->expects(self::exactly(2))
-			->method('createEMailTemplate')
-			->with('dav.calendarReminder')
-			->willReturnOnConsecutiveCalls(
-				$template1,
-				$template2,
-			);
-		$this->mailer->expects($this->atLeastOnce())
-			->method('validateMailAddress')
-			->willReturnMap([
-				['foo1@example.org', true],
-				['foo3@example.org', true],
-				['foo4@example.org', true],
-				['uid1@example.com', true],
-				['uid2@example.com', true],
-				['uid3@example.com', true],
-				['invalid', false],
-			]);
-		$this->mailer->expects($this->exactly(6))
-			->method('createMessage')
-			->with()
-			->willReturnOnConsecutiveCalls(
-				$message11,
-				$message12,
-				$message13,
-				$message21,
-				$message22,
-				$message23,
-			);
-
-		$calls = [
-			[$message11],
-			[$message12],
-			[$message13],
-			[$message21],
-			[$message22],
-			[$message23],
-		];
-		$this->mailer->expects($this->exactly(count($calls)))
-			->method('send')
-			->willReturnCallback(function () use (&$calls) {
-				$expected = array_shift($calls);
-				$this->assertEquals($expected, func_get_args());
-				return [];
-			});
-		$this->setupURLGeneratorMock(2);
-
-		$vcalendar = $this->getAttendeeVCalendar();
-		$this->provider->send($vcalendar->VEVENT, $this->calendarDisplayName, $principalEmailAddresses, $users);
-	}
-
-	public function testSendWithAttendeesWhenOwnerIsAttendee(): void {
-		[$user1, $user2, $user3] = $this->getUsers();
-		$users = [$user2, $user3];
-		$principalEmailAddresses = [$user2->getEmailAddress()];
-
-		$deL10N = $this->createMock(IL10N::class);
-		$deL10N->method('t')
-			->willReturnArgument(0);
-		$deL10N->method('l')
-			->willReturnArgument(0);
-
-		$this->l10nFactory
-			->method('getUserLanguage')
-			->willReturnMap([
-				[$user2, 'de'],
-				[$user3, 'de'],
-			]);
-
-		$this->l10nFactory
-			->method('findGenericLanguage')
-			->willReturn('en');
-
-		$this->l10nFactory
-			->method('languageExists')
-			->willReturnMap([
-				['dav', 'de', true],
-			]);
-
-		$this->l10nFactory
-			->method('get')
-			->willReturnMap([
-				['dav', 'de', null, $deL10N],
-			]);
-
-		$template1 = $this->getTemplateMock();
-		$message12 = $this->getMessageMock('uid2@example.com', $template1);
-		$message13 = $this->getMessageMock('uid3@example.com', $template1);
-
-		$this->mailer->expects(self::once())
-			->method('createEMailTemplate')
-			->with('dav.calendarReminder')
-			->willReturnOnConsecutiveCalls(
-				$template1,
-			);
-		$this->mailer->expects($this->atLeastOnce())
-			->method('validateMailAddress')
-			->willReturnMap([
-				['foo1@example.org', true],
-				['foo3@example.org', true],
-				['foo4@example.org', true],
-				['uid1@example.com', true],
-				['uid2@example.com', true],
-				['uid3@example.com', true],
-				['invalid', false],
-			]);
-		$this->mailer->expects($this->exactly(2))
-			->method('createMessage')
-			->with()
-			->willReturnOnConsecutiveCalls(
-				$message12,
-				$message13,
-			);
-
-		$calls = [
-			[$message12],
-			[$message13],
-		];
-		$this->mailer->expects($this->exactly(count($calls)))
-			->method('send')
-			->willReturnCallback(function () use (&$calls) {
-				$expected = array_shift($calls);
-				$this->assertEquals($expected, func_get_args());
-				return [];
-			});
-		$this->setupURLGeneratorMock(1);
-
-		$vcalendar = $this->getAttendeeVCalendar();
-		$this->provider->send($vcalendar->VEVENT, $this->calendarDisplayName, $principalEmailAddresses, $users);
-	}
-
-	/**
-	 * @return IEMailTemplate
-	 */
-	private function getTemplateMock():IEMailTemplate {
-		$template = $this->createMock(IEMailTemplate::class);
-
-		$template->expects($this->once())
-			->method('addHeader')
-			->with()
-			->willReturn($template);
-
-		$template->expects($this->once())
-			->method('setSubject')
-			->with()
-			->willReturn($template);
-
-		$template->expects($this->once())
-			->method('addHeading')
-			->with()
-			->willReturn($template);
-
-		$template->expects($this->exactly(4))
-			->method('addBodyListItem')
-			->with()
-			->willReturn($template);
-
-		$template->expects($this->once())
-			->method('addFooter')
-			->with()
-			->willReturn($template);
-
-		return $template;
-	}
-
-	/**
-	 * @param string $toMail
-	 * @param IEMailTemplate $templateMock
-	 * @param array|null $replyTo
-	 * @return IMessage
-	 */
-	private function getMessageMock(string $toMail, IEMailTemplate $templateMock, ?array $replyTo = null):IMessage {
-		$message = $this->createMock(IMessage::class);
-		$i = 0;
-
-		$message->expects($this->once())
-			->method('setFrom')
-			->with([Util::getDefaultEmailAddress('reminders-noreply')])
-			->willReturn($message);
-
-		if ($replyTo) {
-			$message->expects($this->once())
-				->method('setReplyTo')
-				->with($replyTo)
-				->willReturn($message);
-		} else {
-			$message->expects($this->never())
-				->method('setReplyTo');
-		}
-
-		$message->expects($this->once())
-			->method('setTo')
-			->with([$toMail])
-			->willReturn($message);
-
-		$message->expects($this->once())
-			->method('useTemplate')
-			->with($templateMock)
-			->willReturn($message);
-
-		return $message;
-	}
-
-	private function getNoAttendeeVCalendar():VCalendar {
-		$vcalendar = new VCalendar();
-		$vcalendar->add('VEVENT', [
-			'SUMMARY' => 'Fellowship meeting',
-			'DTSTART' => new \DateTime('2017-01-01 00:00:00+00:00'), // 1483228800,
-			'UID' => 'uid1234',
-			'LOCATION' => 'Location 123',
-			'DESCRIPTION' => 'DESCRIPTION 456',
-		]);
-
-		return $vcalendar;
-	}
-
-	private function getAttendeeVCalendar():VCalendar {
-		$vcalendar = new VCalendar();
-		$vcalendar->add('VEVENT', [
-			'SUMMARY' => 'Fellowship meeting',
-			'DTSTART' => new \DateTime('2017-01-01 00:00:00+00:00'), // 1483228800,
-			'UID' => 'uid1234',
-			'LOCATION' => 'Location 123',
-			'DESCRIPTION' => 'DESCRIPTION 456',
-		]);
-
-		$vcalendar->VEVENT->add(
-			'ORGANIZER',
-			'mailto:uid1@example.com',
-			[
-				'LANG' => 'en'
-			]
-		);
-
-		$vcalendar->VEVENT->add(
-			'ATTENDEE',
-			'mailto:foo1@example.org',
-			[
-				'LANG' => 'de',
-				'PARTSTAT' => 'NEEDS-ACTION',
-			]
-		);
-
-		$vcalendar->VEVENT->add(
-			'ATTENDEE',
-			'mailto:foo2@example.org',
-			[
-				'LANG' => 'de',
-				'PARTSTAT' => 'DECLINED',
-			]
-		);
-
-		$vcalendar->VEVENT->add(
-			'ATTENDEE',
-			'mailto:foo3@example.org',
-			[
-				'LANG' => 'en',
-				'PARTSTAT' => 'CONFIRMED',
-			]
-		);
-
-		$vcalendar->VEVENT->add(
-			'ATTENDEE',
-			'mailto:foo4@example.org'
-		);
-
-		$vcalendar->VEVENT->add(
-			'ATTENDEE',
-			'tomail:foo5@example.org'
-		);
-
-		return $vcalendar;
-	}
-
-	private function setupURLGeneratorMock(int $times = 1): void {
-		$this->urlGenerator
-			->expects($this->exactly($times * 4))
-			->method('imagePath')
-			->willReturnMap([
-				['core', 'actions/info.png', 'imagePath1'],
-				['core', 'places/calendar.png', 'imagePath2'],
-				['core', 'actions/address.png', 'imagePath3'],
-				['core', 'actions/more.png', 'imagePath4'],
-			]);
-		$this->urlGenerator
-			->expects($this->exactly($times * 4))
-			->method('getAbsoluteURL')
-			->willReturnMap([
-				['imagePath1', 'AbsURL1'],
-				['imagePath2', 'AbsURL2'],
-				['imagePath3', 'AbsURL3'],
-				['imagePath4', 'AbsURL4'],
-			]);
-	}
-
-	private function getUsers(): array {
-		$user1 = $this->createMock(IUser::class);
-		$user1->method('getUID')
-			->willReturn('uid1');
-		$user1->method('getEMailAddress')
-			->willReturn('uid1@example.com');
-		$user2 = $this->createMock(IUser::class);
-		$user2->method('getUID')
-			->willReturn('uid2');
-		$user2->method('getEMailAddress')
-			->willReturn('uid2@example.com');
-		$user3 = $this->createMock(IUser::class);
-		$user3->method('getUID')
-			->willReturn('uid3');
-		$user3->method('getEMailAddress')
-			->willReturn('uid3@example.com');
-		$user4 = $this->createMock(IUser::class);
-		$user4->method('getUID')
-			->willReturn('uid4');
-		$user4->method('getEMailAddress')
-			->willReturn(null);
-		$user5 = $this->createMock(IUser::class);
-		$user5->method('getUID')
-			->willReturn('uid5');
-		$user5->method('getEMailAddress')
-			->willReturn('invalid');
-
-		return [$user1, $user2, $user3, $user4, $user5];
-	}
+    public const USER_EMAIL = 'frodo@hobb.it';
+    private IMailer&MockObject $mailer;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->mailer = $this->createMock(IMailer::class);
+
+        $this->provider = new EmailProvider(
+            $this->config,
+            $this->mailer,
+            $this->logger,
+            $this->l10nFactory,
+            $this->urlGenerator
+        );
+    }
+
+    public function testSendWithoutAttendees():void {
+        [$user1, $user2, $user3, , $user5] = $users = $this->getUsers();
+        $principalEmailAddresses = [$user1->getEmailAddress()];
+
+        $enL10N = $this->createMock(IL10N::class);
+        $enL10N->method('t')
+            ->willReturnArgument(0);
+        $enL10N->method('l')
+            ->willReturnArgument(0);
+
+        $deL10N = $this->createMock(IL10N::class);
+        $deL10N->method('t')
+            ->willReturnArgument(0);
+        $deL10N->method('l')
+            ->willReturnArgument(0);
+
+        $this->l10nFactory
+            ->method('getUserLanguage')
+            ->willReturnMap([
+                [$user1, 'en'],
+                [$user2, 'de'],
+                [$user3, 'de'],
+                [$user5, 'de'],
+            ]);
+
+        $this->l10nFactory
+            ->method('findGenericLanguage')
+            ->willReturn('en');
+
+        $this->l10nFactory
+            ->method('languageExists')
+            ->willReturnMap([
+                ['dav', 'en', true],
+                ['dav', 'de', true],
+            ]);
+
+        $this->l10nFactory
+            ->method('get')
+            ->willReturnMap([
+                ['dav', 'en', null, $enL10N],
+                ['dav', 'de', null, $deL10N],
+            ]);
+
+        $template1 = $this->getTemplateMock();
+        $message11 = $this->getMessageMock('uid1@example.com', $template1);
+        $template2 = $this->getTemplateMock();
+        $message21 = $this->getMessageMock('uid2@example.com', $template2);
+        $message22 = $this->getMessageMock('uid3@example.com', $template2);
+
+        $this->mailer->expects($this->exactly(2))
+            ->method('createEMailTemplate')
+            ->with('dav.calendarReminder')
+            ->willReturnOnConsecutiveCalls(
+                $template1,
+                $template2
+            );
+
+        $this->mailer->expects($this->exactly(4))
+            ->method('validateMailAddress')
+            ->willReturnMap([
+                ['uid1@example.com', true],
+                ['uid2@example.com', true],
+                ['uid3@example.com', true],
+                ['invalid', false],
+            ]);
+
+        $this->mailer->expects($this->exactly(3))
+            ->method('createMessage')
+            ->with()
+            ->willReturnOnConsecutiveCalls(
+                $message11,
+                $message21,
+                $message22
+            );
+
+        $calls = [
+            [$message11],
+            [$message21],
+            [$message22],
+        ];
+        $this->mailer->expects($this->exactly(count($calls)))
+            ->method('send')
+            ->willReturnCallback(function () use (&$calls) {
+                $expected = array_shift($calls);
+                $this->assertEquals($expected, func_get_args());
+                return [];
+            });
+
+        $this->setupURLGeneratorMock(2);
+
+        $vcalendar = $this->getNoAttendeeVCalendar();
+        $this->provider->send($vcalendar->VEVENT, $this->calendarDisplayName, $principalEmailAddresses, $users);
+    }
+
+    public function testSendWithAttendeesWhenOwnerIsOrganizer(): void {
+        [$user1, $user2, $user3, , $user5] = $users = $this->getUsers();
+        $principalEmailAddresses = [$user1->getEmailAddress()];
+
+        $enL10N = $this->createMock(IL10N::class);
+        $enL10N->method('t')
+            ->willReturnArgument(0);
+        $enL10N->method('l')
+            ->willReturnArgument(0);
+
+        $deL10N = $this->createMock(IL10N::class);
+        $deL10N->method('t')
+            ->willReturnArgument(0);
+        $deL10N->method('l')
+            ->willReturnArgument(0);
+
+        $this->l10nFactory
+            ->method('getUserLanguage')
+            ->willReturnMap([
+                [$user1, 'en'],
+                [$user2, 'de'],
+                [$user3, 'de'],
+                [$user5, 'de'],
+            ]);
+
+        $this->l10nFactory
+            ->method('findGenericLanguage')
+            ->willReturn('en');
+
+        $this->l10nFactory
+            ->method('languageExists')
+            ->willReturnMap([
+                ['dav', 'en', true],
+                ['dav', 'de', true],
+            ]);
+
+        $this->l10nFactory
+            ->method('get')
+            ->willReturnMap([
+                ['dav', 'en', null, $enL10N],
+                ['dav', 'de', null, $deL10N],
+            ]);
+
+        $template1 = $this->getTemplateMock();
+        $message11 = $this->getMessageMock('foo1@example.org', $template1);
+        $message12 = $this->getMessageMock('uid2@example.com', $template1);
+        $message13 = $this->getMessageMock('uid3@example.com', $template1);
+        $template2 = $this->getTemplateMock();
+        $message21 = $this->getMessageMock('foo3@example.org', $template2);
+        $message22 = $this->getMessageMock('foo4@example.org', $template2);
+        $message23 = $this->getMessageMock('uid1@example.com', $template2);
+
+        $this->mailer->expects(self::exactly(2))
+            ->method('createEMailTemplate')
+            ->with('dav.calendarReminder')
+            ->willReturnOnConsecutiveCalls(
+                $template1,
+                $template2,
+            );
+        $this->mailer->expects($this->atLeastOnce())
+            ->method('validateMailAddress')
+            ->willReturnMap([
+                ['foo1@example.org', true],
+                ['foo3@example.org', true],
+                ['foo4@example.org', true],
+                ['uid1@example.com', true],
+                ['uid2@example.com', true],
+                ['uid3@example.com', true],
+                ['invalid', false],
+            ]);
+        $this->mailer->expects($this->exactly(6))
+            ->method('createMessage')
+            ->with()
+            ->willReturnOnConsecutiveCalls(
+                $message11,
+                $message12,
+                $message13,
+                $message21,
+                $message22,
+                $message23,
+            );
+
+        $calls = [
+            [$message11],
+            [$message12],
+            [$message13],
+            [$message21],
+            [$message22],
+            [$message23],
+        ];
+        $this->mailer->expects($this->exactly(count($calls)))
+            ->method('send')
+            ->willReturnCallback(function () use (&$calls) {
+                $expected = array_shift($calls);
+                $this->assertEquals($expected, func_get_args());
+                return [];
+            });
+        $this->setupURLGeneratorMock(2);
+
+        $vcalendar = $this->getAttendeeVCalendar();
+        $this->provider->send($vcalendar->VEVENT, $this->calendarDisplayName, $principalEmailAddresses, $users);
+    }
+
+    public function testSendWithAttendeesWhenOwnerIsAttendee(): void {
+        [$user1, $user2, $user3] = $this->getUsers();
+        $users = [$user2, $user3];
+        $principalEmailAddresses = [$user2->getEmailAddress()];
+
+        $deL10N = $this->createMock(IL10N::class);
+        $deL10N->method('t')
+            ->willReturnArgument(0);
+        $deL10N->method('l')
+            ->willReturnArgument(0);
+
+        $this->l10nFactory
+            ->method('getUserLanguage')
+            ->willReturnMap([
+                [$user2, 'de'],
+                [$user3, 'de'],
+            ]);
+
+        $this->l10nFactory
+            ->method('findGenericLanguage')
+            ->willReturn('en');
+
+        $this->l10nFactory
+            ->method('languageExists')
+            ->willReturnMap([
+                ['dav', 'de', true],
+            ]);
+
+        $this->l10nFactory
+            ->method('get')
+            ->willReturnMap([
+                ['dav', 'de', null, $deL10N],
+            ]);
+
+        $template1 = $this->getTemplateMock();
+        $message12 = $this->getMessageMock('uid2@example.com', $template1);
+        $message13 = $this->getMessageMock('uid3@example.com', $template1);
+
+        $this->mailer->expects(self::once())
+            ->method('createEMailTemplate')
+            ->with('dav.calendarReminder')
+            ->willReturnOnConsecutiveCalls(
+                $template1,
+            );
+        $this->mailer->expects($this->atLeastOnce())
+            ->method('validateMailAddress')
+            ->willReturnMap([
+                ['foo1@example.org', true],
+                ['foo3@example.org', true],
+                ['foo4@example.org', true],
+                ['uid1@example.com', true],
+                ['uid2@example.com', true],
+                ['uid3@example.com', true],
+                ['invalid', false],
+            ]);
+        $this->mailer->expects($this->exactly(2))
+            ->method('createMessage')
+            ->with()
+            ->willReturnOnConsecutiveCalls(
+                $message12,
+                $message13,
+            );
+
+        $calls = [
+            [$message12],
+            [$message13],
+        ];
+        $this->mailer->expects($this->exactly(count($calls)))
+            ->method('send')
+            ->willReturnCallback(function () use (&$calls) {
+                $expected = array_shift($calls);
+                $this->assertEquals($expected, func_get_args());
+                return [];
+            });
+        $this->setupURLGeneratorMock(1);
+
+        $vcalendar = $this->getAttendeeVCalendar();
+        $this->provider->send($vcalendar->VEVENT, $this->calendarDisplayName, $principalEmailAddresses, $users);
+    }
+
+    /**
+     * @return IEMailTemplate
+     */
+    private function getTemplateMock():IEMailTemplate {
+        $template = $this->createMock(IEMailTemplate::class);
+
+        $template->expects($this->once())
+            ->method('addHeader')
+            ->with()
+            ->willReturn($template);
+
+        $template->expects($this->once())
+            ->method('setSubject')
+            ->with()
+            ->willReturn($template);
+
+        $template->expects($this->once())
+            ->method('addHeading')
+            ->with()
+            ->willReturn($template);
+
+        $template->expects($this->exactly(4))
+            ->method('addBodyListItem')
+            ->with()
+            ->willReturn($template);
+
+        $template->expects($this->once())
+            ->method('addFooter')
+            ->with()
+            ->willReturn($template);
+
+        return $template;
+    }
+
+    /**
+     * @param string $toMail
+     * @param IEMailTemplate $templateMock
+     * @param array|null $replyTo
+     * @return IMessage
+     */
+    private function getMessageMock(string $toMail, IEMailTemplate $templateMock, ?array $replyTo = null):IMessage {
+        $message = $this->createMock(IMessage::class);
+        $i = 0;
+
+        $message->expects($this->once())
+            ->method('setFrom')
+            ->with([Util::getDefaultEmailAddress('reminders-noreply')])
+            ->willReturn($message);
+
+        if ($replyTo) {
+            $message->expects($this->once())
+                ->method('setReplyTo')
+                ->with($replyTo)
+                ->willReturn($message);
+        } else {
+            $message->expects($this->never())
+                ->method('setReplyTo');
+        }
+
+        $message->expects($this->once())
+            ->method('setTo')
+            ->with([$toMail])
+            ->willReturn($message);
+
+        $message->expects($this->once())
+            ->method('useTemplate')
+            ->with($templateMock)
+            ->willReturn($message);
+
+        return $message;
+    }
+
+    private function getNoAttendeeVCalendar():VCalendar {
+        $vcalendar = new VCalendar();
+        $vcalendar->add('VEVENT', [
+            'SUMMARY' => 'Fellowship meeting',
+            'DTSTART' => new \DateTime('2017-01-01 00:00:00+00:00'), // 1483228800,
+            'UID' => 'uid1234',
+            'LOCATION' => 'Location 123',
+            'DESCRIPTION' => 'DESCRIPTION 456',
+        ]);
+
+        return $vcalendar;
+    }
+
+    private function getAttendeeVCalendar():VCalendar {
+        $vcalendar = new VCalendar();
+        $vcalendar->add('VEVENT', [
+            'SUMMARY' => 'Fellowship meeting',
+            'DTSTART' => new \DateTime('2017-01-01 00:00:00+00:00'), // 1483228800,
+            'UID' => 'uid1234',
+            'LOCATION' => 'Location 123',
+            'DESCRIPTION' => 'DESCRIPTION 456',
+        ]);
+
+        $vcalendar->VEVENT->add(
+            'ORGANIZER',
+            'mailto:uid1@example.com',
+            [
+                'LANG' => 'en'
+            ]
+        );
+
+        $vcalendar->VEVENT->add(
+            'ATTENDEE',
+            'mailto:foo1@example.org',
+            [
+                'LANG' => 'de',
+                'PARTSTAT' => 'NEEDS-ACTION',
+            ]
+        );
+
+        $vcalendar->VEVENT->add(
+            'ATTENDEE',
+            'mailto:foo2@example.org',
+            [
+                'LANG' => 'de',
+                'PARTSTAT' => 'DECLINED',
+            ]
+        );
+
+        $vcalendar->VEVENT->add(
+            'ATTENDEE',
+            'mailto:foo3@example.org',
+            [
+                'LANG' => 'en',
+                'PARTSTAT' => 'CONFIRMED',
+            ]
+        );
+
+        $vcalendar->VEVENT->add(
+            'ATTENDEE',
+            'mailto:foo4@example.org'
+        );
+
+        $vcalendar->VEVENT->add(
+            'ATTENDEE',
+            'tomail:foo5@example.org'
+        );
+
+        return $vcalendar;
+    }
+
+    private function setupURLGeneratorMock(int $times = 1): void {
+        $this->urlGenerator
+            ->expects($this->exactly($times * 4))
+            ->method('imagePath')
+            ->willReturnMap([
+                ['core', 'actions/info.png', 'imagePath1'],
+                ['core', 'places/calendar.png', 'imagePath2'],
+                ['core', 'actions/address.png', 'imagePath3'],
+                ['core', 'actions/more.png', 'imagePath4'],
+            ]);
+        $this->urlGenerator
+            ->expects($this->exactly($times * 4))
+            ->method('getAbsoluteURL')
+            ->willReturnMap([
+                ['imagePath1', 'AbsURL1'],
+                ['imagePath2', 'AbsURL2'],
+                ['imagePath3', 'AbsURL3'],
+                ['imagePath4', 'AbsURL4'],
+            ]);
+    }
+
+    private function getUsers(): array {
+        $user1 = $this->createMock(IUser::class);
+        $user1->method('getUID')
+            ->willReturn('uid1');
+        $user1->method('getEMailAddress')
+            ->willReturn('uid1@example.com');
+        $user2 = $this->createMock(IUser::class);
+        $user2->method('getUID')
+            ->willReturn('uid2');
+        $user2->method('getEMailAddress')
+            ->willReturn('uid2@example.com');
+        $user3 = $this->createMock(IUser::class);
+        $user3->method('getUID')
+            ->willReturn('uid3');
+        $user3->method('getEMailAddress')
+            ->willReturn('uid3@example.com');
+        $user4 = $this->createMock(IUser::class);
+        $user4->method('getUID')
+            ->willReturn('uid4');
+        $user4->method('getEMailAddress')
+            ->willReturn(null);
+        $user5 = $this->createMock(IUser::class);
+        $user5->method('getUID')
+            ->willReturn('uid5');
+        $user5->method('getEMailAddress')
+            ->willReturn('invalid');
+
+        return [$user1, $user2, $user3, $user4, $user5];
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -37,7 +37,7 @@ 
 	}
 
 	public function testSendWithoutAttendees():void {
-		[$user1, $user2, $user3, , $user5] = $users = $this->getUsers();
+		[$user1, $user2, $user3,, $user5] = $users = $this->getUsers();
 		$principalEmailAddresses = [$user1->getEmailAddress()];
 
 		$enL10N = $this->createMock(IL10N::class);
@@ -118,7 +118,7 @@ 
 		];
 		$this->mailer->expects($this->exactly(count($calls)))
 			->method('send')
-			->willReturnCallback(function () use (&$calls) {
+			->willReturnCallback(function() use (&$calls) {
 				$expected = array_shift($calls);
 				$this->assertEquals($expected, func_get_args());
 				return [];
@@ -131,7 +131,7 @@ 
 	}
 
 	public function testSendWithAttendeesWhenOwnerIsOrganizer(): void {
-		[$user1, $user2, $user3, , $user5] = $users = $this->getUsers();
+		[$user1, $user2, $user3,, $user5] = $users = $this->getUsers();
 		$principalEmailAddresses = [$user1->getEmailAddress()];
 
 		$enL10N = $this->createMock(IL10N::class);
@@ -222,7 +222,7 @@ 
 		];
 		$this->mailer->expects($this->exactly(count($calls)))
 			->method('send')
-			->willReturnCallback(function () use (&$calls) {
+			->willReturnCallback(function() use (&$calls) {
 				$expected = array_shift($calls);
 				$this->assertEquals($expected, func_get_args());
 				return [];
@@ -302,7 +302,7 @@ 
 		];
 		$this->mailer->expects($this->exactly(count($calls)))
 			->method('send')
-			->willReturnCallback(function () use (&$calls) {
+			->willReturnCallback(function() use (&$calls) {
 				$expected = array_shift($calls);
 				$this->assertEquals($expected, func_get_args());
 				return [];

apps/dav/tests/unit/CalDAV/Reminder/NotificationProviderManagerTest.php

Indentation +48 added lines, -48 removed lines

@@ -21,63 +21,63 @@
  * @group DB
  */
 class NotificationProviderManagerTest extends TestCase {
-	private NotificationProviderManager $providerManager;
+    private NotificationProviderManager $providerManager;
 
-	/**
-	 * @throws QueryException
-	 */
-	protected function setUp(): void {
-		parent::setUp();
+    /**
+     * @throws QueryException
+     */
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->providerManager = new NotificationProviderManager();
-		$this->providerManager->registerProvider(EmailProvider::class);
-	}
+        $this->providerManager = new NotificationProviderManager();
+        $this->providerManager->registerProvider(EmailProvider::class);
+    }
 
-	/**
-	 * @throws ProviderNotAvailableException
-	 * @throws NotificationTypeDoesNotExistException
-	 */
-	public function testGetProviderForUnknownType(): void {
-		$this->expectException(NotificationTypeDoesNotExistException::class);
-		$this->expectExceptionMessage('Type NOT EXISTENT is not an accepted type of notification');
+    /**
+     * @throws ProviderNotAvailableException
+     * @throws NotificationTypeDoesNotExistException
+     */
+    public function testGetProviderForUnknownType(): void {
+        $this->expectException(NotificationTypeDoesNotExistException::class);
+        $this->expectExceptionMessage('Type NOT EXISTENT is not an accepted type of notification');
 
-		$this->providerManager->getProvider('NOT EXISTENT');
-	}
+        $this->providerManager->getProvider('NOT EXISTENT');
+    }
 
-	/**
-	 * @throws NotificationTypeDoesNotExistException
-	 * @throws ProviderNotAvailableException
-	 */
-	public function testGetProviderForUnRegisteredType(): void {
-		$this->expectException(ProviderNotAvailableException::class);
-		$this->expectExceptionMessage('No notification provider for type AUDIO available');
+    /**
+     * @throws NotificationTypeDoesNotExistException
+     * @throws ProviderNotAvailableException
+     */
+    public function testGetProviderForUnRegisteredType(): void {
+        $this->expectException(ProviderNotAvailableException::class);
+        $this->expectExceptionMessage('No notification provider for type AUDIO available');
 
-		$this->providerManager->getProvider('AUDIO');
-	}
+        $this->providerManager->getProvider('AUDIO');
+    }
 
-	public function testGetProvider(): void {
-		$provider = $this->providerManager->getProvider('EMAIL');
-		$this->assertInstanceOf(EmailProvider::class, $provider);
-	}
+    public function testGetProvider(): void {
+        $provider = $this->providerManager->getProvider('EMAIL');
+        $this->assertInstanceOf(EmailProvider::class, $provider);
+    }
 
-	public function testRegisterProvider(): void {
-		$this->providerManager->registerProvider(PushProvider::class);
-		$provider = $this->providerManager->getProvider('DISPLAY');
-		$this->assertInstanceOf(PushProvider::class, $provider);
-	}
+    public function testRegisterProvider(): void {
+        $this->providerManager->registerProvider(PushProvider::class);
+        $provider = $this->providerManager->getProvider('DISPLAY');
+        $this->assertInstanceOf(PushProvider::class, $provider);
+    }
 
-	/**
-	 * @throws QueryException
-	 */
-	public function testRegisterBadProvider(): void {
-		$this->expectException(\InvalidArgumentException::class);
-		$this->expectExceptionMessage('Invalid notification provider registered');
+    /**
+     * @throws QueryException
+     */
+    public function testRegisterBadProvider(): void {
+        $this->expectException(\InvalidArgumentException::class);
+        $this->expectExceptionMessage('Invalid notification provider registered');
 
-		$this->providerManager->registerProvider(Capabilities::class);
-	}
+        $this->providerManager->registerProvider(Capabilities::class);
+    }
 
-	public function testHasProvider(): void {
-		$this->assertTrue($this->providerManager->hasProvider('EMAIL'));
-		$this->assertFalse($this->providerManager->hasProvider('EMAIL123'));
-	}
+    public function testHasProvider(): void {
+        $this->assertTrue($this->providerManager->hasProvider('EMAIL'));
+        $this->assertFalse($this->providerManager->hasProvider('EMAIL123'));
+    }
 }

apps/dav/tests/unit/BackgroundJob/UserStatusAutomationTest.php

Indentation +156 added lines, -156 removed lines

@@ -29,95 +29,95 @@ 
  * @group DB
  */
 class UserStatusAutomationTest extends TestCase {
-	protected ITimeFactory&MockObject $time;
-	protected IJobList&MockObject $jobList;
-	protected LoggerInterface&MockObject $logger;
-	protected IManager&MockObject $statusManager;
-	protected IConfig&MockObject $config;
-	private IAvailabilityCoordinator&MockObject $coordinator;
-	private IUserManager&MockObject $userManager;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->time = $this->createMock(ITimeFactory::class);
-		$this->jobList = $this->createMock(IJobList::class);
-		$this->logger = $this->createMock(LoggerInterface::class);
-		$this->statusManager = $this->createMock(IManager::class);
-		$this->config = $this->createMock(IConfig::class);
-		$this->coordinator = $this->createMock(IAvailabilityCoordinator::class);
-		$this->userManager = $this->createMock(IUserManager::class);
-
-	}
-
-	protected function getAutomationMock(array $methods): MockObject|UserStatusAutomation {
-		if (empty($methods)) {
-			return new UserStatusAutomation(
-				$this->time,
-				Server::get(IDBConnection::class),
-				$this->jobList,
-				$this->logger,
-				$this->statusManager,
-				$this->config,
-				$this->coordinator,
-				$this->userManager,
-			);
-		}
-
-		return $this->getMockBuilder(UserStatusAutomation::class)
-			->setConstructorArgs([
-				$this->time,
-				Server::get(IDBConnection::class),
-				$this->jobList,
-				$this->logger,
-				$this->statusManager,
-				$this->config,
-				$this->coordinator,
-				$this->userManager,
-			])
-			->onlyMethods($methods)
-			->getMock();
-	}
-
-	public static function dataRun(): array {
-		return [
-			['20230217', '2023-02-24 10:49:36.613834', true],
-			['20230224', '2023-02-24 10:49:36.613834', true],
-			['20230217', '2023-02-24 13:58:24.479357', false],
-			['20230224', '2023-02-24 13:58:24.479357', false],
-		];
-	}
-
-	/**
-	 * @dataProvider dataRun
-	 */
-	public function testRunNoOOO(string $ruleDay, string $currentTime, bool $isAvailable): void {
-		$user = $this->createConfiguredMock(IUser::class, [
-			'getUID' => 'user'
-		]);
-
-		$this->userManager->expects(self::once())
-			->method('get')
-			->willReturn($user);
-		$this->coordinator->expects(self::once())
-			->method('getCurrentOutOfOfficeData')
-			->willReturn(null);
-		$this->config->method('getUserValue')
-			->with('user', 'dav', 'user_status_automation', 'no')
-			->willReturn('yes');
-		$this->time->method('getDateTime')
-			->willReturn(new \DateTime($currentTime, new \DateTimeZone('UTC')));
-		$this->logger->expects(self::exactly(4))
-			->method('debug');
-		if (!$isAvailable) {
-			$this->statusManager->expects(self::once())
-				->method('setUserStatus')
-				->with('user', IUserStatus::MESSAGE_AVAILABILITY, IUserStatus::DND, true);
-		}
-		$automation = $this->getAutomationMock(['getAvailabilityFromPropertiesTable']);
-		$automation->method('getAvailabilityFromPropertiesTable')
-			->with('user')
-			->willReturn('BEGIN:VCALENDAR
+    protected ITimeFactory&MockObject $time;
+    protected IJobList&MockObject $jobList;
+    protected LoggerInterface&MockObject $logger;
+    protected IManager&MockObject $statusManager;
+    protected IConfig&MockObject $config;
+    private IAvailabilityCoordinator&MockObject $coordinator;
+    private IUserManager&MockObject $userManager;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->time = $this->createMock(ITimeFactory::class);
+        $this->jobList = $this->createMock(IJobList::class);
+        $this->logger = $this->createMock(LoggerInterface::class);
+        $this->statusManager = $this->createMock(IManager::class);
+        $this->config = $this->createMock(IConfig::class);
+        $this->coordinator = $this->createMock(IAvailabilityCoordinator::class);
+        $this->userManager = $this->createMock(IUserManager::class);
+
+    }
+
+    protected function getAutomationMock(array $methods): MockObject|UserStatusAutomation {
+        if (empty($methods)) {
+            return new UserStatusAutomation(
+                $this->time,
+                Server::get(IDBConnection::class),
+                $this->jobList,
+                $this->logger,
+                $this->statusManager,
+                $this->config,
+                $this->coordinator,
+                $this->userManager,
+            );
+        }
+
+        return $this->getMockBuilder(UserStatusAutomation::class)
+            ->setConstructorArgs([
+                $this->time,
+                Server::get(IDBConnection::class),
+                $this->jobList,
+                $this->logger,
+                $this->statusManager,
+                $this->config,
+                $this->coordinator,
+                $this->userManager,
+            ])
+            ->onlyMethods($methods)
+            ->getMock();
+    }
+
+    public static function dataRun(): array {
+        return [
+            ['20230217', '2023-02-24 10:49:36.613834', true],
+            ['20230224', '2023-02-24 10:49:36.613834', true],
+            ['20230217', '2023-02-24 13:58:24.479357', false],
+            ['20230224', '2023-02-24 13:58:24.479357', false],
+        ];
+    }
+
+    /**
+     * @dataProvider dataRun
+     */
+    public function testRunNoOOO(string $ruleDay, string $currentTime, bool $isAvailable): void {
+        $user = $this->createConfiguredMock(IUser::class, [
+            'getUID' => 'user'
+        ]);
+
+        $this->userManager->expects(self::once())
+            ->method('get')
+            ->willReturn($user);
+        $this->coordinator->expects(self::once())
+            ->method('getCurrentOutOfOfficeData')
+            ->willReturn(null);
+        $this->config->method('getUserValue')
+            ->with('user', 'dav', 'user_status_automation', 'no')
+            ->willReturn('yes');
+        $this->time->method('getDateTime')
+            ->willReturn(new \DateTime($currentTime, new \DateTimeZone('UTC')));
+        $this->logger->expects(self::exactly(4))
+            ->method('debug');
+        if (!$isAvailable) {
+            $this->statusManager->expects(self::once())
+                ->method('setUserStatus')
+                ->with('user', IUserStatus::MESSAGE_AVAILABILITY, IUserStatus::DND, true);
+        }
+        $automation = $this->getAutomationMock(['getAvailabilityFromPropertiesTable']);
+        $automation->method('getAvailabilityFromPropertiesTable')
+            ->with('user')
+            ->willReturn('BEGIN:VCALENDAR
 PRODID:Nextcloud DAV app
 BEGIN:VTIMEZONE
 TZID:Europe/Berlin
@@ -152,71 +152,71 @@ 
 END:VAVAILABILITY
 END:VCALENDAR');
 
-		self::invokePrivate($automation, 'run', [['userId' => 'user']]);
-	}
-
-	public function testRunNoAvailabilityNoOOO(): void {
-		$user = $this->createConfiguredMock(IUser::class, [
-			'getUID' => 'user'
-		]);
-
-		$this->userManager->expects(self::once())
-			->method('get')
-			->willReturn($user);
-		$this->coordinator->expects(self::once())
-			->method('getCurrentOutOfOfficeData')
-			->willReturn(null);
-		$this->config->method('getUserValue')
-			->with('user', 'dav', 'user_status_automation', 'no')
-			->willReturn('yes');
-		$this->time->method('getDateTime')
-			->willReturn(new \DateTime('2023-02-24 13:58:24.479357', new \DateTimeZone('UTC')));
-		$this->jobList->expects($this->once())
-			->method('remove')
-			->with(UserStatusAutomation::class, ['userId' => 'user']);
-		$this->logger->expects(self::once())
-			->method('debug');
-		$this->logger->expects(self::once())
-			->method('info');
-		$automation = $this->getAutomationMock(['getAvailabilityFromPropertiesTable']);
-		$automation->method('getAvailabilityFromPropertiesTable')
-			->with('user')
-			->willReturn(false);
-
-		self::invokePrivate($automation, 'run', [['userId' => 'user']]);
-	}
-
-	public function testRunNoAvailabilityWithOOO(): void {
-		$user = $this->createConfiguredMock(IUser::class, [
-			'getUID' => 'user'
-		]);
-		$ooo = $this->createConfiguredMock(OutOfOfficeData::class, [
-			'getShortMessage' => 'On Vacation',
-			'getEndDate' => 123456,
-		]);
-
-		$this->userManager->expects(self::once())
-			->method('get')
-			->willReturn($user);
-		$this->coordinator->expects(self::once())
-			->method('getCurrentOutOfOfficeData')
-			->willReturn($ooo);
-		$this->coordinator->expects(self::once())
-			->method('isInEffect')
-			->willReturn(true);
-		$this->statusManager->expects(self::once())
-			->method('setUserStatus')
-			->with('user', IUserStatus::MESSAGE_OUT_OF_OFFICE, IUserStatus::DND, true, $ooo->getShortMessage());
-		$this->config->expects(self::never())
-			->method('getUserValue');
-		$this->time->method('getDateTime')
-			->willReturn(new \DateTime('2023-02-24 13:58:24.479357', new \DateTimeZone('UTC')));
-		$this->jobList->expects($this->never())
-			->method('remove');
-		$this->logger->expects(self::exactly(2))
-			->method('debug');
-		$automation = $this->getAutomationMock([]);
-
-		self::invokePrivate($automation, 'run', [['userId' => 'user']]);
-	}
+        self::invokePrivate($automation, 'run', [['userId' => 'user']]);
+    }
+
+    public function testRunNoAvailabilityNoOOO(): void {
+        $user = $this->createConfiguredMock(IUser::class, [
+            'getUID' => 'user'
+        ]);
+
+        $this->userManager->expects(self::once())
+            ->method('get')
+            ->willReturn($user);
+        $this->coordinator->expects(self::once())
+            ->method('getCurrentOutOfOfficeData')
+            ->willReturn(null);
+        $this->config->method('getUserValue')
+            ->with('user', 'dav', 'user_status_automation', 'no')
+            ->willReturn('yes');
+        $this->time->method('getDateTime')
+            ->willReturn(new \DateTime('2023-02-24 13:58:24.479357', new \DateTimeZone('UTC')));
+        $this->jobList->expects($this->once())
+            ->method('remove')
+            ->with(UserStatusAutomation::class, ['userId' => 'user']);
+        $this->logger->expects(self::once())
+            ->method('debug');
+        $this->logger->expects(self::once())
+            ->method('info');
+        $automation = $this->getAutomationMock(['getAvailabilityFromPropertiesTable']);
+        $automation->method('getAvailabilityFromPropertiesTable')
+            ->with('user')
+            ->willReturn(false);
+
+        self::invokePrivate($automation, 'run', [['userId' => 'user']]);
+    }
+
+    public function testRunNoAvailabilityWithOOO(): void {
+        $user = $this->createConfiguredMock(IUser::class, [
+            'getUID' => 'user'
+        ]);
+        $ooo = $this->createConfiguredMock(OutOfOfficeData::class, [
+            'getShortMessage' => 'On Vacation',
+            'getEndDate' => 123456,
+        ]);
+
+        $this->userManager->expects(self::once())
+            ->method('get')
+            ->willReturn($user);
+        $this->coordinator->expects(self::once())
+            ->method('getCurrentOutOfOfficeData')
+            ->willReturn($ooo);
+        $this->coordinator->expects(self::once())
+            ->method('isInEffect')
+            ->willReturn(true);
+        $this->statusManager->expects(self::once())
+            ->method('setUserStatus')
+            ->with('user', IUserStatus::MESSAGE_OUT_OF_OFFICE, IUserStatus::DND, true, $ooo->getShortMessage());
+        $this->config->expects(self::never())
+            ->method('getUserValue');
+        $this->time->method('getDateTime')
+            ->willReturn(new \DateTime('2023-02-24 13:58:24.479357', new \DateTimeZone('UTC')));
+        $this->jobList->expects($this->never())
+            ->method('remove');
+        $this->logger->expects(self::exactly(2))
+            ->method('debug');
+        $automation = $this->getAutomationMock([]);
+
+        self::invokePrivate($automation, 'run', [['userId' => 'user']]);
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -50,7 +50,7 @@ 
 
 	}
 
-	protected function getAutomationMock(array $methods): MockObject|UserStatusAutomation {
+	protected function getAutomationMock(array $methods): MockObject | UserStatusAutomation {
 		if (empty($methods)) {
 			return new UserStatusAutomation(
 				$this->time,
@@ -138,14 +138,14 @@ 
 END:VTIMEZONE
 BEGIN:VAVAILABILITY
 BEGIN:AVAILABLE
-DTSTART;TZID=Europe/Berlin:' . $ruleDay . 'T090000
-DTEND;TZID=Europe/Berlin:' . $ruleDay . 'T170000
+DTSTART;TZID=Europe/Berlin:' . $ruleDay.'T090000
+DTEND;TZID=Europe/Berlin:' . $ruleDay.'T170000
 UID:3e6feeec-8e00-4265-b822-b73174e8b39f
 RRULE:FREQ=WEEKLY;BYDAY=TH
 END:AVAILABLE
 BEGIN:AVAILABLE
-DTSTART;TZID=Europe/Berlin:' . $ruleDay . 'T090000
-DTEND;TZID=Europe/Berlin:' . $ruleDay . 'T120000
+DTSTART;TZID=Europe/Berlin:' . $ruleDay.'T090000
+DTEND;TZID=Europe/Berlin:' . $ruleDay.'T120000
 UID:8a634e99-07cf-443b-b480-005a0e1db323
 RRULE:FREQ=WEEKLY;BYDAY=FR
 END:AVAILABLE

apps/dav/tests/unit/BackgroundJob/PruneOutdatedSyncTokensJobTest.php

Indentation +55 added lines, -55 removed lines

@@ -20,65 +20,65 @@
 use Test\TestCase;
 
 class PruneOutdatedSyncTokensJobTest extends TestCase {
-	private ITimeFactory&MockObject $timeFactory;
-	private CalDavBackend&MockObject $calDavBackend;
-	private CardDavBackend&MockObject $cardDavBackend;
-	private IConfig&MockObject $config;
-	private LoggerInterface&MockObject $logger;
-	private PruneOutdatedSyncTokensJob $backgroundJob;
+    private ITimeFactory&MockObject $timeFactory;
+    private CalDavBackend&MockObject $calDavBackend;
+    private CardDavBackend&MockObject $cardDavBackend;
+    private IConfig&MockObject $config;
+    private LoggerInterface&MockObject $logger;
+    private PruneOutdatedSyncTokensJob $backgroundJob;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->timeFactory = $this->createMock(ITimeFactory::class);
-		$this->calDavBackend = $this->createMock(CalDavBackend::class);
-		$this->cardDavBackend = $this->createMock(CardDavBackend::class);
-		$this->config = $this->createMock(IConfig::class);
-		$this->logger = $this->createMock(LoggerInterface::class);
+        $this->timeFactory = $this->createMock(ITimeFactory::class);
+        $this->calDavBackend = $this->createMock(CalDavBackend::class);
+        $this->cardDavBackend = $this->createMock(CardDavBackend::class);
+        $this->config = $this->createMock(IConfig::class);
+        $this->logger = $this->createMock(LoggerInterface::class);
 
-		$this->backgroundJob = new PruneOutdatedSyncTokensJob($this->timeFactory, $this->calDavBackend, $this->cardDavBackend, $this->config, $this->logger);
-	}
+        $this->backgroundJob = new PruneOutdatedSyncTokensJob($this->timeFactory, $this->calDavBackend, $this->cardDavBackend, $this->config, $this->logger);
+    }
 
-	/**
-	 * @dataProvider dataForTestRun
-	 */
-	public function testRun(string $configToKeep, string $configRetentionDays, int $actualLimit, int $retentionDays, int $deletedCalendarSyncTokens, int $deletedAddressBookSyncTokens): void {
-		$this->config->expects($this->exactly(2))
-			->method('getAppValue')
-			->with(Application::APP_ID, self::anything(), self::anything())
-			->willReturnCallback(function ($app, $key) use ($configToKeep, $configRetentionDays) {
-				switch ($key) {
-					case 'totalNumberOfSyncTokensToKeep':
-						return $configToKeep;
-					case 'syncTokensRetentionDays':
-						return $configRetentionDays;
-					default:
-						throw new InvalidArgumentException();
-				}
-			});
-		$this->calDavBackend->expects($this->once())
-			->method('pruneOutdatedSyncTokens')
-			->with($actualLimit)
-			->willReturn($deletedCalendarSyncTokens);
-		$this->cardDavBackend->expects($this->once())
-			->method('pruneOutdatedSyncTokens')
-			->with($actualLimit, $retentionDays)
-			->willReturn($deletedAddressBookSyncTokens);
-		$this->logger->expects($this->once())
-			->method('info')
-			->with('Pruned {calendarSyncTokensNumber} calendar sync tokens and {addressBooksSyncTokensNumber} address book sync tokens', [
-				'calendarSyncTokensNumber' => $deletedCalendarSyncTokens,
-				'addressBooksSyncTokensNumber' => $deletedAddressBookSyncTokens
-			]);
+    /**
+     * @dataProvider dataForTestRun
+     */
+    public function testRun(string $configToKeep, string $configRetentionDays, int $actualLimit, int $retentionDays, int $deletedCalendarSyncTokens, int $deletedAddressBookSyncTokens): void {
+        $this->config->expects($this->exactly(2))
+            ->method('getAppValue')
+            ->with(Application::APP_ID, self::anything(), self::anything())
+            ->willReturnCallback(function ($app, $key) use ($configToKeep, $configRetentionDays) {
+                switch ($key) {
+                    case 'totalNumberOfSyncTokensToKeep':
+                        return $configToKeep;
+                    case 'syncTokensRetentionDays':
+                        return $configRetentionDays;
+                    default:
+                        throw new InvalidArgumentException();
+                }
+            });
+        $this->calDavBackend->expects($this->once())
+            ->method('pruneOutdatedSyncTokens')
+            ->with($actualLimit)
+            ->willReturn($deletedCalendarSyncTokens);
+        $this->cardDavBackend->expects($this->once())
+            ->method('pruneOutdatedSyncTokens')
+            ->with($actualLimit, $retentionDays)
+            ->willReturn($deletedAddressBookSyncTokens);
+        $this->logger->expects($this->once())
+            ->method('info')
+            ->with('Pruned {calendarSyncTokensNumber} calendar sync tokens and {addressBooksSyncTokensNumber} address book sync tokens', [
+                'calendarSyncTokensNumber' => $deletedCalendarSyncTokens,
+                'addressBooksSyncTokensNumber' => $deletedAddressBookSyncTokens
+            ]);
 
-		$this->backgroundJob->run(null);
-	}
+        $this->backgroundJob->run(null);
+    }
 
-	public static function dataForTestRun(): array {
-		return [
-			['100', '2', 100, 7 * 24 * 3600, 2, 3],
-			['100', '14', 100, 14 * 24 * 3600, 2, 3],
-			['0', '60', 1, 60 * 24 * 3600, 0, 0]
-		];
-	}
+    public static function dataForTestRun(): array {
+        return [
+            ['100', '2', 100, 7 * 24 * 3600, 2, 3],
+            ['100', '14', 100, 14 * 24 * 3600, 2, 3],
+            ['0', '60', 1, 60 * 24 * 3600, 0, 0]
+        ];
+    }
 }

apps/dav/tests/unit/BackgroundJob/GenerateBirthdayCalendarBackgroundJobTest.php

Indentation +94 added lines, -94 removed lines

@@ -16,98 +16,98 @@
 use Test\TestCase;
 
 class GenerateBirthdayCalendarBackgroundJobTest extends TestCase {
-	private ITimeFactory&MockObject $time;
-	private BirthdayService&MockObject $birthdayService;
-	private IConfig&MockObject $config;
-	private GenerateBirthdayCalendarBackgroundJob $backgroundJob;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->time = $this->createMock(ITimeFactory::class);
-		$this->birthdayService = $this->createMock(BirthdayService::class);
-		$this->config = $this->createMock(IConfig::class);
-
-		$this->backgroundJob = new GenerateBirthdayCalendarBackgroundJob(
-			$this->time,
-			$this->birthdayService,
-			$this->config,
-		);
-	}
-
-	public function testRun(): void {
-		$this->config->expects($this->once())
-			->method('getAppValue')
-			->with('dav', 'generateBirthdayCalendar', 'yes')
-			->willReturn('yes');
-
-		$this->config->expects($this->once())
-			->method('getUserValue')
-			->with('user123', 'dav', 'generateBirthdayCalendar', 'yes')
-			->willReturn('yes');
-
-		$this->birthdayService->expects($this->never())
-			->method('resetForUser')
-			->with('user123');
-
-		$this->birthdayService->expects($this->once())
-			->method('syncUser')
-			->with('user123');
-
-		$this->backgroundJob->run(['userId' => 'user123']);
-	}
-
-	public function testRunAndReset(): void {
-		$this->config->expects($this->once())
-			->method('getAppValue')
-			->with('dav', 'generateBirthdayCalendar', 'yes')
-			->willReturn('yes');
-
-		$this->config->expects($this->once())
-			->method('getUserValue')
-			->with('user123', 'dav', 'generateBirthdayCalendar', 'yes')
-			->willReturn('yes');
-
-		$this->birthdayService->expects($this->once())
-			->method('resetForUser')
-			->with('user123');
-
-		$this->birthdayService->expects($this->once())
-			->method('syncUser')
-			->with('user123');
-
-		$this->backgroundJob->run(['userId' => 'user123', 'purgeBeforeGenerating' => true]);
-	}
-
-	public function testRunGloballyDisabled(): void {
-		$this->config->expects($this->once())
-			->method('getAppValue')
-			->with('dav', 'generateBirthdayCalendar', 'yes')
-			->willReturn('no');
-
-		$this->config->expects($this->never())
-			->method('getUserValue');
-
-		$this->birthdayService->expects($this->never())
-			->method('syncUser');
-
-		$this->backgroundJob->run(['userId' => 'user123']);
-	}
-
-	public function testRunUserDisabled(): void {
-		$this->config->expects($this->once())
-			->method('getAppValue')
-			->with('dav', 'generateBirthdayCalendar', 'yes')
-			->willReturn('yes');
-
-		$this->config->expects($this->once())
-			->method('getUserValue')
-			->with('user123', 'dav', 'generateBirthdayCalendar', 'yes')
-			->willReturn('no');
-
-		$this->birthdayService->expects($this->never())
-			->method('syncUser');
-
-		$this->backgroundJob->run(['userId' => 'user123']);
-	}
+    private ITimeFactory&MockObject $time;
+    private BirthdayService&MockObject $birthdayService;
+    private IConfig&MockObject $config;
+    private GenerateBirthdayCalendarBackgroundJob $backgroundJob;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->time = $this->createMock(ITimeFactory::class);
+        $this->birthdayService = $this->createMock(BirthdayService::class);
+        $this->config = $this->createMock(IConfig::class);
+
+        $this->backgroundJob = new GenerateBirthdayCalendarBackgroundJob(
+            $this->time,
+            $this->birthdayService,
+            $this->config,
+        );
+    }
+
+    public function testRun(): void {
+        $this->config->expects($this->once())
+            ->method('getAppValue')
+            ->with('dav', 'generateBirthdayCalendar', 'yes')
+            ->willReturn('yes');
+
+        $this->config->expects($this->once())
+            ->method('getUserValue')
+            ->with('user123', 'dav', 'generateBirthdayCalendar', 'yes')
+            ->willReturn('yes');
+
+        $this->birthdayService->expects($this->never())
+            ->method('resetForUser')
+            ->with('user123');
+
+        $this->birthdayService->expects($this->once())
+            ->method('syncUser')
+            ->with('user123');
+
+        $this->backgroundJob->run(['userId' => 'user123']);
+    }
+
+    public function testRunAndReset(): void {
+        $this->config->expects($this->once())
+            ->method('getAppValue')
+            ->with('dav', 'generateBirthdayCalendar', 'yes')
+            ->willReturn('yes');
+
+        $this->config->expects($this->once())
+            ->method('getUserValue')
+            ->with('user123', 'dav', 'generateBirthdayCalendar', 'yes')
+            ->willReturn('yes');
+
+        $this->birthdayService->expects($this->once())
+            ->method('resetForUser')
+            ->with('user123');
+
+        $this->birthdayService->expects($this->once())
+            ->method('syncUser')
+            ->with('user123');
+
+        $this->backgroundJob->run(['userId' => 'user123', 'purgeBeforeGenerating' => true]);
+    }
+
+    public function testRunGloballyDisabled(): void {
+        $this->config->expects($this->once())
+            ->method('getAppValue')
+            ->with('dav', 'generateBirthdayCalendar', 'yes')
+            ->willReturn('no');
+
+        $this->config->expects($this->never())
+            ->method('getUserValue');
+
+        $this->birthdayService->expects($this->never())
+            ->method('syncUser');
+
+        $this->backgroundJob->run(['userId' => 'user123']);
+    }
+
+    public function testRunUserDisabled(): void {
+        $this->config->expects($this->once())
+            ->method('getAppValue')
+            ->with('dav', 'generateBirthdayCalendar', 'yes')
+            ->willReturn('yes');
+
+        $this->config->expects($this->once())
+            ->method('getUserValue')
+            ->with('user123', 'dav', 'generateBirthdayCalendar', 'yes')
+            ->willReturn('no');
+
+        $this->birthdayService->expects($this->never())
+            ->method('syncUser');
+
+        $this->backgroundJob->run(['userId' => 'user123']);
+    }
 }

apps/dav/tests/unit/BackgroundJob/EventReminderJobTest.php

Indentation +46 added lines, -46 removed lines

@@ -16,57 +16,57 @@
 use Test\TestCase;
 
 class EventReminderJobTest extends TestCase {
-	private ITimeFactory&MockObject $time;
-	private ReminderService&MockObject $reminderService;
-	private IConfig&MockObject $config;
-	private EventReminderJob $backgroundJob;
+    private ITimeFactory&MockObject $time;
+    private ReminderService&MockObject $reminderService;
+    private IConfig&MockObject $config;
+    private EventReminderJob $backgroundJob;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->time = $this->createMock(ITimeFactory::class);
-		$this->reminderService = $this->createMock(ReminderService::class);
-		$this->config = $this->createMock(IConfig::class);
+        $this->time = $this->createMock(ITimeFactory::class);
+        $this->reminderService = $this->createMock(ReminderService::class);
+        $this->config = $this->createMock(IConfig::class);
 
-		$this->backgroundJob = new EventReminderJob(
-			$this->time,
-			$this->reminderService,
-			$this->config,
-		);
-	}
+        $this->backgroundJob = new EventReminderJob(
+            $this->time,
+            $this->reminderService,
+            $this->config,
+        );
+    }
 
-	public static function data(): array {
-		return [
-			[true, true, true],
-			[true, false, false],
-			[false, true, false],
-			[false, false, false],
-		];
-	}
+    public static function data(): array {
+        return [
+            [true, true, true],
+            [true, false, false],
+            [false, true, false],
+            [false, false, false],
+        ];
+    }
 
-	/**
-	 * @dataProvider data
-	 *
-	 * @param bool $sendEventReminders
-	 * @param bool $sendEventRemindersMode
-	 * @param bool $expectCall
-	 */
-	public function testRun(bool $sendEventReminders, bool $sendEventRemindersMode, bool $expectCall): void {
-		$this->config->expects($this->exactly($sendEventReminders ? 2 : 1))
-			->method('getAppValue')
-			->willReturnMap([
-				['dav', 'sendEventReminders', 'yes', ($sendEventReminders ? 'yes' : 'no')],
-				['dav', 'sendEventRemindersMode', 'backgroundjob', ($sendEventRemindersMode ? 'backgroundjob' : 'cron')],
-			]);
+    /**
+     * @dataProvider data
+     *
+     * @param bool $sendEventReminders
+     * @param bool $sendEventRemindersMode
+     * @param bool $expectCall
+     */
+    public function testRun(bool $sendEventReminders, bool $sendEventRemindersMode, bool $expectCall): void {
+        $this->config->expects($this->exactly($sendEventReminders ? 2 : 1))
+            ->method('getAppValue')
+            ->willReturnMap([
+                ['dav', 'sendEventReminders', 'yes', ($sendEventReminders ? 'yes' : 'no')],
+                ['dav', 'sendEventRemindersMode', 'backgroundjob', ($sendEventRemindersMode ? 'backgroundjob' : 'cron')],
+            ]);
 
-		if ($expectCall) {
-			$this->reminderService->expects($this->once())
-				->method('processReminders');
-		} else {
-			$this->reminderService->expects($this->never())
-				->method('processReminders');
-		}
+        if ($expectCall) {
+            $this->reminderService->expects($this->once())
+                ->method('processReminders');
+        } else {
+            $this->reminderService->expects($this->never())
+                ->method('processReminders');
+        }
 
-		$this->backgroundJob->run([]);
-	}
+        $this->backgroundJob->run([]);
+    }
 }

apps/dav/tests/unit/BackgroundJob/RegisterRegenerateBirthdayCalendarsTest.php

Indentation +51 added lines, -51 removed lines

@@ -18,62 +18,62 @@
 use Test\TestCase;
 
 class RegisterRegenerateBirthdayCalendarsTest extends TestCase {
-	private ITimeFactory&MockObject $time;
-	private IUserManager&MockObject $userManager;
-	private IJobList&MockObject $jobList;
-	private RegisterRegenerateBirthdayCalendars $backgroundJob;
+    private ITimeFactory&MockObject $time;
+    private IUserManager&MockObject $userManager;
+    private IJobList&MockObject $jobList;
+    private RegisterRegenerateBirthdayCalendars $backgroundJob;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->time = $this->createMock(ITimeFactory::class);
-		$this->userManager = $this->createMock(IUserManager::class);
-		$this->jobList = $this->createMock(IJobList::class);
+        $this->time = $this->createMock(ITimeFactory::class);
+        $this->userManager = $this->createMock(IUserManager::class);
+        $this->jobList = $this->createMock(IJobList::class);
 
-		$this->backgroundJob = new RegisterRegenerateBirthdayCalendars(
-			$this->time,
-			$this->userManager,
-			$this->jobList
-		);
-	}
+        $this->backgroundJob = new RegisterRegenerateBirthdayCalendars(
+            $this->time,
+            $this->userManager,
+            $this->jobList
+        );
+    }
 
-	public function testRun(): void {
-		$this->userManager->expects($this->once())
-			->method('callForSeenUsers')
-			->willReturnCallback(function ($closure): void {
-				$user1 = $this->createMock(IUser::class);
-				$user1->method('getUID')->willReturn('uid1');
-				$user2 = $this->createMock(IUser::class);
-				$user2->method('getUID')->willReturn('uid2');
-				$user3 = $this->createMock(IUser::class);
-				$user3->method('getUID')->willReturn('uid3');
+    public function testRun(): void {
+        $this->userManager->expects($this->once())
+            ->method('callForSeenUsers')
+            ->willReturnCallback(function ($closure): void {
+                $user1 = $this->createMock(IUser::class);
+                $user1->method('getUID')->willReturn('uid1');
+                $user2 = $this->createMock(IUser::class);
+                $user2->method('getUID')->willReturn('uid2');
+                $user3 = $this->createMock(IUser::class);
+                $user3->method('getUID')->willReturn('uid3');
 
-				$closure($user1);
-				$closure($user2);
-				$closure($user3);
-			});
+                $closure($user1);
+                $closure($user2);
+                $closure($user3);
+            });
 
-		$calls = [
-			'uid1',
-			'uid2',
-			'uid3',
-		];
-		$this->jobList->expects($this->exactly(3))
-			->method('add')
-			->willReturnCallback(function () use (&$calls): void {
-				$expected = array_shift($calls);
-				$this->assertEquals(
-					[
-						GenerateBirthdayCalendarBackgroundJob::class,
-						[
-							'userId' => $expected,
-							'purgeBeforeGenerating' => true
-						]
-					],
-					func_get_args()
-				);
-			});
+        $calls = [
+            'uid1',
+            'uid2',
+            'uid3',
+        ];
+        $this->jobList->expects($this->exactly(3))
+            ->method('add')
+            ->willReturnCallback(function () use (&$calls): void {
+                $expected = array_shift($calls);
+                $this->assertEquals(
+                    [
+                        GenerateBirthdayCalendarBackgroundJob::class,
+                        [
+                            'userId' => $expected,
+                            'purgeBeforeGenerating' => true
+                        ]
+                    ],
+                    func_get_args()
+                );
+            });
 
-		$this->backgroundJob->run([]);
-	}
+        $this->backgroundJob->run([]);
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
 	public function testRun(): void {
 		$this->userManager->expects($this->once())
 			->method('callForSeenUsers')
-			->willReturnCallback(function ($closure): void {
+			->willReturnCallback(function($closure): void {
 				$user1 = $this->createMock(IUser::class);
 				$user1->method('getUID')->willReturn('uid1');
 				$user2 = $this->createMock(IUser::class);
@@ -60,7 +60,7 @@ 
 		];
 		$this->jobList->expects($this->exactly(3))
 			->method('add')
-			->willReturnCallback(function () use (&$calls): void {
+			->willReturnCallback(function() use (&$calls): void {
 				$expected = array_shift($calls);
 				$this->assertEquals(
 					[

apps/dav/tests/unit/BackgroundJob/UpdateCalendarResourcesRoomsBackgroundJobTest.php

Indentation +27 added lines, -27 removed lines

@@ -17,31 +17,31 @@
 use Test\TestCase;
 
 class UpdateCalendarResourcesRoomsBackgroundJobTest extends TestCase {
-	private UpdateCalendarResourcesRoomsBackgroundJob $backgroundJob;
-	private ITimeFactory&MockObject $time;
-	private IResourceManager&MockObject $resourceManager;
-	private IRoomManager&MockObject $roomManager;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->time = $this->createMock(ITimeFactory::class);
-		$this->resourceManager = $this->createMock(IResourceManager::class);
-		$this->roomManager = $this->createMock(IRoomManager::class);
-
-		$this->backgroundJob = new UpdateCalendarResourcesRoomsBackgroundJob(
-			$this->time,
-			$this->resourceManager,
-			$this->roomManager,
-		);
-	}
-
-	public function testRun(): void {
-		$this->resourceManager->expects(self::once())
-			->method('update');
-		$this->roomManager->expects(self::once())
-			->method('update');
-
-		$this->backgroundJob->run([]);
-	}
+    private UpdateCalendarResourcesRoomsBackgroundJob $backgroundJob;
+    private ITimeFactory&MockObject $time;
+    private IResourceManager&MockObject $resourceManager;
+    private IRoomManager&MockObject $roomManager;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->time = $this->createMock(ITimeFactory::class);
+        $this->resourceManager = $this->createMock(IResourceManager::class);
+        $this->roomManager = $this->createMock(IRoomManager::class);
+
+        $this->backgroundJob = new UpdateCalendarResourcesRoomsBackgroundJob(
+            $this->time,
+            $this->resourceManager,
+            $this->roomManager,
+        );
+    }
+
+    public function testRun(): void {
+        $this->resourceManager->expects(self::once())
+            ->method('update');
+        $this->roomManager->expects(self::once())
+            ->method('update');
+
+        $this->backgroundJob->run([]);
+    }
 }

apps/dav/tests/unit/BackgroundJob/OutOfOfficeEventDispatcherJobTest.php

Indentation +121 added lines, -121 removed lines

@@ -24,125 +24,125 @@
 use Test\TestCase;
 
 class OutOfOfficeEventDispatcherJobTest extends TestCase {
-	private OutOfOfficeEventDispatcherJob $job;
-	private ITimeFactory&MockObject $timeFactory;
-	private AbsenceMapper&MockObject $absenceMapper;
-	private LoggerInterface&MockObject $logger;
-	private IEventDispatcher&MockObject $eventDispatcher;
-	private IUserManager&MockObject $userManager;
-	private MockObject|TimezoneService $timezoneService;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->timeFactory = $this->createMock(ITimeFactory::class);
-		$this->absenceMapper = $this->createMock(AbsenceMapper::class);
-		$this->logger = $this->createMock(LoggerInterface::class);
-		$this->eventDispatcher = $this->createMock(IEventDispatcher::class);
-		$this->userManager = $this->createMock(IUserManager::class);
-		$this->timezoneService = $this->createMock(TimezoneService::class);
-
-		$this->job = new OutOfOfficeEventDispatcherJob(
-			$this->timeFactory,
-			$this->absenceMapper,
-			$this->logger,
-			$this->eventDispatcher,
-			$this->userManager,
-			$this->timezoneService,
-		);
-	}
-
-	public function testDispatchStartEvent(): void {
-		$this->timezoneService->method('getUserTimezone')->with('user')->willReturn('Europe/Berlin');
-
-		$absence = new Absence();
-		$absence->setId(200);
-		$absence->setUserId('user');
-
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')
-			->willReturn('user');
-
-		$this->absenceMapper->expects(self::once())
-			->method('findById')
-			->with(1)
-			->willReturn($absence);
-		$this->userManager->expects(self::once())
-			->method('get')
-			->with('user')
-			->willReturn($user);
-		$this->eventDispatcher->expects(self::once())
-			->method('dispatchTyped')
-			->with(self::callback(static function ($event): bool {
-				self::assertInstanceOf(OutOfOfficeStartedEvent::class, $event);
-				return true;
-			}));
-
-		$this->job->run([
-			'id' => 1,
-			'event' => OutOfOfficeEventDispatcherJob::EVENT_START,
-		]);
-	}
-
-	public function testDispatchStopEvent(): void {
-		$this->timezoneService->method('getUserTimezone')->with('user')->willReturn('Europe/Berlin');
-
-		$absence = new Absence();
-		$absence->setId(200);
-		$absence->setUserId('user');
-
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')
-			->willReturn('user');
-
-		$this->absenceMapper->expects(self::once())
-			->method('findById')
-			->with(1)
-			->willReturn($absence);
-		$this->userManager->expects(self::once())
-			->method('get')
-			->with('user')
-			->willReturn($user);
-		$this->eventDispatcher->expects(self::once())
-			->method('dispatchTyped')
-			->with(self::callback(static function ($event): bool {
-				self::assertInstanceOf(OutOfOfficeEndedEvent::class, $event);
-				return true;
-			}));
-
-		$this->job->run([
-			'id' => 1,
-			'event' => OutOfOfficeEventDispatcherJob::EVENT_END,
-		]);
-	}
-
-	public function testDoesntDispatchUnknownEvent(): void {
-		$this->timezoneService->method('getUserTimezone')->with('user')->willReturn('Europe/Berlin');
-
-		$absence = new Absence();
-		$absence->setId(100);
-		$absence->setUserId('user');
-
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')
-			->willReturn('user');
-
-		$this->absenceMapper->expects(self::once())
-			->method('findById')
-			->with(1)
-			->willReturn($absence);
-		$this->userManager->expects(self::once())
-			->method('get')
-			->with('user')
-			->willReturn($user);
-		$this->eventDispatcher->expects(self::never())
-			->method('dispatchTyped');
-		$this->logger->expects(self::once())
-			->method('error');
-
-		$this->job->run([
-			'id' => 1,
-			'event' => 'foobar',
-		]);
-	}
+    private OutOfOfficeEventDispatcherJob $job;
+    private ITimeFactory&MockObject $timeFactory;
+    private AbsenceMapper&MockObject $absenceMapper;
+    private LoggerInterface&MockObject $logger;
+    private IEventDispatcher&MockObject $eventDispatcher;
+    private IUserManager&MockObject $userManager;
+    private MockObject|TimezoneService $timezoneService;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->timeFactory = $this->createMock(ITimeFactory::class);
+        $this->absenceMapper = $this->createMock(AbsenceMapper::class);
+        $this->logger = $this->createMock(LoggerInterface::class);
+        $this->eventDispatcher = $this->createMock(IEventDispatcher::class);
+        $this->userManager = $this->createMock(IUserManager::class);
+        $this->timezoneService = $this->createMock(TimezoneService::class);
+
+        $this->job = new OutOfOfficeEventDispatcherJob(
+            $this->timeFactory,
+            $this->absenceMapper,
+            $this->logger,
+            $this->eventDispatcher,
+            $this->userManager,
+            $this->timezoneService,
+        );
+    }
+
+    public function testDispatchStartEvent(): void {
+        $this->timezoneService->method('getUserTimezone')->with('user')->willReturn('Europe/Berlin');
+
+        $absence = new Absence();
+        $absence->setId(200);
+        $absence->setUserId('user');
+
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')
+            ->willReturn('user');
+
+        $this->absenceMapper->expects(self::once())
+            ->method('findById')
+            ->with(1)
+            ->willReturn($absence);
+        $this->userManager->expects(self::once())
+            ->method('get')
+            ->with('user')
+            ->willReturn($user);
+        $this->eventDispatcher->expects(self::once())
+            ->method('dispatchTyped')
+            ->with(self::callback(static function ($event): bool {
+                self::assertInstanceOf(OutOfOfficeStartedEvent::class, $event);
+                return true;
+            }));
+
+        $this->job->run([
+            'id' => 1,
+            'event' => OutOfOfficeEventDispatcherJob::EVENT_START,
+        ]);
+    }
+
+    public function testDispatchStopEvent(): void {
+        $this->timezoneService->method('getUserTimezone')->with('user')->willReturn('Europe/Berlin');
+
+        $absence = new Absence();
+        $absence->setId(200);
+        $absence->setUserId('user');
+
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')
+            ->willReturn('user');
+
+        $this->absenceMapper->expects(self::once())
+            ->method('findById')
+            ->with(1)
+            ->willReturn($absence);
+        $this->userManager->expects(self::once())
+            ->method('get')
+            ->with('user')
+            ->willReturn($user);
+        $this->eventDispatcher->expects(self::once())
+            ->method('dispatchTyped')
+            ->with(self::callback(static function ($event): bool {
+                self::assertInstanceOf(OutOfOfficeEndedEvent::class, $event);
+                return true;
+            }));
+
+        $this->job->run([
+            'id' => 1,
+            'event' => OutOfOfficeEventDispatcherJob::EVENT_END,
+        ]);
+    }
+
+    public function testDoesntDispatchUnknownEvent(): void {
+        $this->timezoneService->method('getUserTimezone')->with('user')->willReturn('Europe/Berlin');
+
+        $absence = new Absence();
+        $absence->setId(100);
+        $absence->setUserId('user');
+
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')
+            ->willReturn('user');
+
+        $this->absenceMapper->expects(self::once())
+            ->method('findById')
+            ->with(1)
+            ->willReturn($absence);
+        $this->userManager->expects(self::once())
+            ->method('get')
+            ->with('user')
+            ->willReturn($user);
+        $this->eventDispatcher->expects(self::never())
+            ->method('dispatchTyped');
+        $this->logger->expects(self::once())
+            ->method('error');
+
+        $this->job->run([
+            'id' => 1,
+            'event' => 'foobar',
+        ]);
+    }
 }

Spacing +3 added lines, -3 removed lines

@@ -30,7 +30,7 @@ 
 	private LoggerInterface&MockObject $logger;
 	private IEventDispatcher&MockObject $eventDispatcher;
 	private IUserManager&MockObject $userManager;
-	private MockObject|TimezoneService $timezoneService;
+	private MockObject | TimezoneService $timezoneService;
 
 	protected function setUp(): void {
 		parent::setUp();
@@ -73,7 +73,7 @@ 
 			->willReturn($user);
 		$this->eventDispatcher->expects(self::once())
 			->method('dispatchTyped')
-			->with(self::callback(static function ($event): bool {
+			->with(self::callback(static function($event): bool {
 				self::assertInstanceOf(OutOfOfficeStartedEvent::class, $event);
 				return true;
 			}));
@@ -105,7 +105,7 @@ 
 			->willReturn($user);
 		$this->eventDispatcher->expects(self::once())
 			->method('dispatchTyped')
-			->with(self::callback(static function ($event): bool {
+			->with(self::callback(static function($event): bool {
 				self::assertInstanceOf(OutOfOfficeEndedEvent::class, $event);
 				return true;
 			}));