nextcloud / server

apps/dav/tests/unit/Migration/RefreshWebcalJobRegistrarTest.php

Indentation +98 added lines, -98 removed lines

@@ -18,102 +18,102 @@
 use Test\TestCase;
 
 class RefreshWebcalJobRegistrarTest extends TestCase {
-	private IDBConnection&MockObject $db;
-	private IJobList&MockObject $jobList;
-	private RefreshWebcalJobRegistrar $migration;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->db = $this->createMock(IDBConnection::class);
-		$this->jobList = $this->createMock(IJobList::class);
-
-		$this->migration = new RefreshWebcalJobRegistrar($this->db, $this->jobList);
-	}
-
-	public function testGetName(): void {
-		$this->assertEquals($this->migration->getName(), 'Registering background jobs to update cache for webcal calendars');
-	}
-
-	public function testRun(): void {
-		$output = $this->createMock(IOutput::class);
-
-		$queryBuilder = $this->createMock(IQueryBuilder::class);
-		$statement = $this->createMock(IResult::class);
-
-		$this->db->expects($this->once())
-			->method('getQueryBuilder')
-			->willReturn($queryBuilder);
-
-		$queryBuilder->expects($this->once())
-			->method('select')
-			->with(['principaluri', 'uri'])
-			->willReturn($queryBuilder);
-		$queryBuilder->expects($this->once())
-			->method('from')
-			->with('calendarsubscriptions')
-			->willReturn($queryBuilder);
-		$queryBuilder->expects($this->once())
-			->method('execute')
-			->willReturn($statement);
-
-		$statement->expects($this->exactly(4))
-			->method('fetch')
-			->with(\PDO::FETCH_ASSOC)
-			->willReturnOnConsecutiveCalls(
-				[
-					'principaluri' => 'foo1',
-					'uri' => 'bar1',
-				],
-				[
-					'principaluri' => 'foo2',
-					'uri' => 'bar2',
-				],
-				[
-					'principaluri' => 'foo3',
-					'uri' => 'bar3',
-				],
-				null
-			);
-
-		$this->jobList->expects($this->exactly(3))
-			->method('has')
-			->willReturnMap([
-				[RefreshWebcalJob::class, [
-					'principaluri' => 'foo1',
-					'uri' => 'bar1',
-				], false],
-				[RefreshWebcalJob::class, [
-					'principaluri' => 'foo2',
-					'uri' => 'bar2',
-				], true ],
-				[RefreshWebcalJob::class, [
-					'principaluri' => 'foo3',
-					'uri' => 'bar3',
-				], false],
-			]);
-
-		$calls = [
-			[RefreshWebcalJob::class, [
-				'principaluri' => 'foo1',
-				'uri' => 'bar1',
-			]],
-			[RefreshWebcalJob::class, [
-				'principaluri' => 'foo3',
-				'uri' => 'bar3',
-			]]
-		];
-		$this->jobList->expects($this->exactly(2))
-			->method('add')
-			->willReturnCallback(function () use (&$calls) {
-				$expected = array_shift($calls);
-				$this->assertEquals($expected, func_get_args());
-			});
-
-		$output->expects($this->once())
-			->method('info')
-			->with('Added 2 background jobs to update webcal calendars');
-
-		$this->migration->run($output);
-	}
+    private IDBConnection&MockObject $db;
+    private IJobList&MockObject $jobList;
+    private RefreshWebcalJobRegistrar $migration;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->db = $this->createMock(IDBConnection::class);
+        $this->jobList = $this->createMock(IJobList::class);
+
+        $this->migration = new RefreshWebcalJobRegistrar($this->db, $this->jobList);
+    }
+
+    public function testGetName(): void {
+        $this->assertEquals($this->migration->getName(), 'Registering background jobs to update cache for webcal calendars');
+    }
+
+    public function testRun(): void {
+        $output = $this->createMock(IOutput::class);
+
+        $queryBuilder = $this->createMock(IQueryBuilder::class);
+        $statement = $this->createMock(IResult::class);
+
+        $this->db->expects($this->once())
+            ->method('getQueryBuilder')
+            ->willReturn($queryBuilder);
+
+        $queryBuilder->expects($this->once())
+            ->method('select')
+            ->with(['principaluri', 'uri'])
+            ->willReturn($queryBuilder);
+        $queryBuilder->expects($this->once())
+            ->method('from')
+            ->with('calendarsubscriptions')
+            ->willReturn($queryBuilder);
+        $queryBuilder->expects($this->once())
+            ->method('execute')
+            ->willReturn($statement);
+
+        $statement->expects($this->exactly(4))
+            ->method('fetch')
+            ->with(\PDO::FETCH_ASSOC)
+            ->willReturnOnConsecutiveCalls(
+                [
+                    'principaluri' => 'foo1',
+                    'uri' => 'bar1',
+                ],
+                [
+                    'principaluri' => 'foo2',
+                    'uri' => 'bar2',
+                ],
+                [
+                    'principaluri' => 'foo3',
+                    'uri' => 'bar3',
+                ],
+                null
+            );
+
+        $this->jobList->expects($this->exactly(3))
+            ->method('has')
+            ->willReturnMap([
+                [RefreshWebcalJob::class, [
+                    'principaluri' => 'foo1',
+                    'uri' => 'bar1',
+                ], false],
+                [RefreshWebcalJob::class, [
+                    'principaluri' => 'foo2',
+                    'uri' => 'bar2',
+                ], true ],
+                [RefreshWebcalJob::class, [
+                    'principaluri' => 'foo3',
+                    'uri' => 'bar3',
+                ], false],
+            ]);
+
+        $calls = [
+            [RefreshWebcalJob::class, [
+                'principaluri' => 'foo1',
+                'uri' => 'bar1',
+            ]],
+            [RefreshWebcalJob::class, [
+                'principaluri' => 'foo3',
+                'uri' => 'bar3',
+            ]]
+        ];
+        $this->jobList->expects($this->exactly(2))
+            ->method('add')
+            ->willReturnCallback(function () use (&$calls) {
+                $expected = array_shift($calls);
+                $this->assertEquals($expected, func_get_args());
+            });
+
+        $output->expects($this->once())
+            ->method('info')
+            ->with('Added 2 background jobs to update webcal calendars');
+
+        $this->migration->run($output);
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -86,7 +86,7 @@ 
 				[RefreshWebcalJob::class, [
 					'principaluri' => 'foo2',
 					'uri' => 'bar2',
-				], true ],
+				], true],
 				[RefreshWebcalJob::class, [
 					'principaluri' => 'foo3',
 					'uri' => 'bar3',
@@ -105,7 +105,7 @@ 
 		];
 		$this->jobList->expects($this->exactly(2))
 			->method('add')
-			->willReturnCallback(function () use (&$calls) {
+			->willReturnCallback(function() use (&$calls) {
 				$expected = array_shift($calls);
 				$this->assertEquals($expected, func_get_args());
 			});

apps/dav/tests/unit/Migration/RegenerateBirthdayCalendarsTest.php

Indentation +59 added lines, -59 removed lines

@@ -16,63 +16,63 @@
 use Test\TestCase;
 
 class RegenerateBirthdayCalendarsTest extends TestCase {
-	private IJobList&MockObject $jobList;
-	private IConfig&MockObject $config;
-	private RegenerateBirthdayCalendars $migration;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->jobList = $this->createMock(IJobList::class);
-		$this->config = $this->createMock(IConfig::class);
-
-		$this->migration = new RegenerateBirthdayCalendars($this->jobList,
-			$this->config);
-	}
-
-	public function testGetName(): void {
-		$this->assertEquals(
-			'Regenerating birthday calendars to use new icons and fix old birthday events without year',
-			$this->migration->getName()
-		);
-	}
-
-	public function testRun(): void {
-		$this->config->expects($this->once())
-			->method('getAppValue')
-			->with('dav', 'regeneratedBirthdayCalendarsForYearFix')
-			->willReturn(null);
-
-		$output = $this->createMock(IOutput::class);
-		$output->expects($this->once())
-			->method('info')
-			->with('Adding background jobs to regenerate birthday calendar');
-
-		$this->jobList->expects($this->once())
-			->method('add')
-			->with(RegisterRegenerateBirthdayCalendars::class);
-
-		$this->config->expects($this->once())
-			->method('setAppValue')
-			->with('dav', 'regeneratedBirthdayCalendarsForYearFix', 'yes');
-
-		$this->migration->run($output);
-	}
-
-	public function testRunSecondTime(): void {
-		$this->config->expects($this->once())
-			->method('getAppValue')
-			->with('dav', 'regeneratedBirthdayCalendarsForYearFix')
-			->willReturn('yes');
-
-		$output = $this->createMock(IOutput::class);
-		$output->expects($this->once())
-			->method('info')
-			->with('Repair step already executed');
-
-		$this->jobList->expects($this->never())
-			->method('add');
-
-		$this->migration->run($output);
-	}
+    private IJobList&MockObject $jobList;
+    private IConfig&MockObject $config;
+    private RegenerateBirthdayCalendars $migration;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->jobList = $this->createMock(IJobList::class);
+        $this->config = $this->createMock(IConfig::class);
+
+        $this->migration = new RegenerateBirthdayCalendars($this->jobList,
+            $this->config);
+    }
+
+    public function testGetName(): void {
+        $this->assertEquals(
+            'Regenerating birthday calendars to use new icons and fix old birthday events without year',
+            $this->migration->getName()
+        );
+    }
+
+    public function testRun(): void {
+        $this->config->expects($this->once())
+            ->method('getAppValue')
+            ->with('dav', 'regeneratedBirthdayCalendarsForYearFix')
+            ->willReturn(null);
+
+        $output = $this->createMock(IOutput::class);
+        $output->expects($this->once())
+            ->method('info')
+            ->with('Adding background jobs to regenerate birthday calendar');
+
+        $this->jobList->expects($this->once())
+            ->method('add')
+            ->with(RegisterRegenerateBirthdayCalendars::class);
+
+        $this->config->expects($this->once())
+            ->method('setAppValue')
+            ->with('dav', 'regeneratedBirthdayCalendarsForYearFix', 'yes');
+
+        $this->migration->run($output);
+    }
+
+    public function testRunSecondTime(): void {
+        $this->config->expects($this->once())
+            ->method('getAppValue')
+            ->with('dav', 'regeneratedBirthdayCalendarsForYearFix')
+            ->willReturn('yes');
+
+        $output = $this->createMock(IOutput::class);
+        $output->expects($this->once())
+            ->method('info')
+            ->with('Repair step already executed');
+
+        $this->jobList->expects($this->never())
+            ->method('add');
+
+        $this->migration->run($output);
+    }
 }

apps/dav/tests/unit/Migration/CreateSystemAddressBookStepTest.php

Indentation +18 added lines, -18 removed lines

@@ -17,31 +17,31 @@
 
 class CreateSystemAddressBookStepTest extends TestCase {
 
-	private SyncService&MockObject $syncService;
-	private CreateSystemAddressBookStep $step;
+    private SyncService&MockObject $syncService;
+    private CreateSystemAddressBookStep $step;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->syncService = $this->createMock(SyncService::class);
+        $this->syncService = $this->createMock(SyncService::class);
 
-		$this->step = new CreateSystemAddressBookStep(
-			$this->syncService,
-		);
-	}
+        $this->step = new CreateSystemAddressBookStep(
+            $this->syncService,
+        );
+    }
 
-	public function testGetName(): void {
-		$name = $this->step->getName();
+    public function testGetName(): void {
+        $name = $this->step->getName();
 
-		self::assertEquals('Create system address book', $name);
-	}
+        self::assertEquals('Create system address book', $name);
+    }
 
-	public function testRun(): void {
-		$output = $this->createMock(IOutput::class);
+    public function testRun(): void {
+        $output = $this->createMock(IOutput::class);
 
-		$this->step->run($output);
+        $this->step->run($output);
 
-		$this->addToAssertionCount(1);
-	}
+        $this->addToAssertionCount(1);
+    }
 
 }

apps/dav/tests/unit/Migration/RemoveDeletedUsersCalendarSubscriptionsTest.php

Indentation +115 added lines, -115 removed lines

@@ -22,121 +22,121 @@
 use Test\TestCase;
 
 class RemoveDeletedUsersCalendarSubscriptionsTest extends TestCase {
-	private IDBConnection&MockObject $dbConnection;
-	private IUserManager&MockObject $userManager;
-	private IOutput&MockObject $output;
-	private RemoveDeletedUsersCalendarSubscriptions $migration;
+    private IDBConnection&MockObject $dbConnection;
+    private IUserManager&MockObject $userManager;
+    private IOutput&MockObject $output;
+    private RemoveDeletedUsersCalendarSubscriptions $migration;
 
 
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->dbConnection = $this->createMock(IDBConnection::class);
-		$this->userManager = $this->createMock(IUserManager::class);
-		$this->output = $this->createMock(IOutput::class);
-
-		$this->migration = new RemoveDeletedUsersCalendarSubscriptions($this->dbConnection, $this->userManager);
-	}
-
-	public function testGetName(): void {
-		$this->assertEquals(
-			'Clean up old calendar subscriptions from deleted users that were not cleaned-up',
-			$this->migration->getName()
-		);
-	}
-
-	/**
-	 * @dataProvider dataTestRun
-	 */
-	public function testRun(array $subscriptions, array $userExists, int $deletions): void {
-		$qb = $this->createMock(IQueryBuilder::class);
-
-		$qb->method('select')->willReturn($qb);
-
-		$functionBuilder = $this->createMock(IFunctionBuilder::class);
-
-		$qb->method('func')->willReturn($functionBuilder);
-		$functionBuilder->method('count')->willReturn($this->createMock(IQueryFunction::class));
-
-		$qb->method('selectDistinct')
-			->with(['id', 'principaluri'])
-			->willReturn($qb);
-
-		$qb->method('from')
-			->with('calendarsubscriptions')
-			->willReturn($qb);
-
-		$qb->method('setMaxResults')
-			->willReturn($qb);
-
-		$qb->method('setFirstResult')
-			->willReturn($qb);
-
-		$result = $this->createMock(IResult::class);
-
-		$qb->method('execute')
-			->willReturn($result);
-
-		$result->expects($this->once())
-			->method('fetchOne')
-			->willReturn(count($subscriptions));
-
-		$result
-			->method('fetch')
-			->willReturnOnConsecutiveCalls(...$subscriptions);
-
-		$qb->method('delete')
-			->with('calendarsubscriptions')
-			->willReturn($qb);
-
-		$expr = $this->createMock(IExpressionBuilder::class);
-
-		$qb->method('expr')->willReturn($expr);
-		$qb->method('createNamedParameter')->willReturn($this->createMock(IParameter::class));
-		$qb->method('where')->willReturn($qb);
-		// Only when user exists
-		$qb->expects($this->exactly($deletions))->method('executeStatement');
-
-		$this->dbConnection->method('getQueryBuilder')->willReturn($qb);
-
-
-		$this->output->expects($this->once())->method('startProgress');
-
-		$this->output->expects($subscriptions === [] ? $this->never(): $this->once())->method('advance');
-		if (count($subscriptions)) {
-			$this->userManager->method('userExists')
-				->willReturnCallback(function (string $username) use ($userExists) {
-					return $userExists[$username];
-				});
-		}
-		$this->output->expects($this->once())->method('finishProgress');
-		$this->output->expects($this->once())->method('info')->with(sprintf('%d calendar subscriptions without an user have been cleaned up', $deletions));
-
-		$this->migration->run($this->output);
-	}
-
-	public static function dataTestRun(): array {
-		return [
-			[[], [], 0],
-			[
-				[
-					[
-						'id' => 1,
-						'principaluri' => 'users/principals/foo1',
-					],
-					[
-						'id' => 2,
-						'principaluri' => 'users/principals/bar1',
-					],
-					[
-						'id' => 3,
-						'principaluri' => 'users/principals/bar1',
-					],
-					[],
-				],
-				['foo1' => true, 'bar1' => false],
-				2
-			],
-		];
-	}
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->dbConnection = $this->createMock(IDBConnection::class);
+        $this->userManager = $this->createMock(IUserManager::class);
+        $this->output = $this->createMock(IOutput::class);
+
+        $this->migration = new RemoveDeletedUsersCalendarSubscriptions($this->dbConnection, $this->userManager);
+    }
+
+    public function testGetName(): void {
+        $this->assertEquals(
+            'Clean up old calendar subscriptions from deleted users that were not cleaned-up',
+            $this->migration->getName()
+        );
+    }
+
+    /**
+     * @dataProvider dataTestRun
+     */
+    public function testRun(array $subscriptions, array $userExists, int $deletions): void {
+        $qb = $this->createMock(IQueryBuilder::class);
+
+        $qb->method('select')->willReturn($qb);
+
+        $functionBuilder = $this->createMock(IFunctionBuilder::class);
+
+        $qb->method('func')->willReturn($functionBuilder);
+        $functionBuilder->method('count')->willReturn($this->createMock(IQueryFunction::class));
+
+        $qb->method('selectDistinct')
+            ->with(['id', 'principaluri'])
+            ->willReturn($qb);
+
+        $qb->method('from')
+            ->with('calendarsubscriptions')
+            ->willReturn($qb);
+
+        $qb->method('setMaxResults')
+            ->willReturn($qb);
+
+        $qb->method('setFirstResult')
+            ->willReturn($qb);
+
+        $result = $this->createMock(IResult::class);
+
+        $qb->method('execute')
+            ->willReturn($result);
+
+        $result->expects($this->once())
+            ->method('fetchOne')
+            ->willReturn(count($subscriptions));
+
+        $result
+            ->method('fetch')
+            ->willReturnOnConsecutiveCalls(...$subscriptions);
+
+        $qb->method('delete')
+            ->with('calendarsubscriptions')
+            ->willReturn($qb);
+
+        $expr = $this->createMock(IExpressionBuilder::class);
+
+        $qb->method('expr')->willReturn($expr);
+        $qb->method('createNamedParameter')->willReturn($this->createMock(IParameter::class));
+        $qb->method('where')->willReturn($qb);
+        // Only when user exists
+        $qb->expects($this->exactly($deletions))->method('executeStatement');
+
+        $this->dbConnection->method('getQueryBuilder')->willReturn($qb);
+
+
+        $this->output->expects($this->once())->method('startProgress');
+
+        $this->output->expects($subscriptions === [] ? $this->never(): $this->once())->method('advance');
+        if (count($subscriptions)) {
+            $this->userManager->method('userExists')
+                ->willReturnCallback(function (string $username) use ($userExists) {
+                    return $userExists[$username];
+                });
+        }
+        $this->output->expects($this->once())->method('finishProgress');
+        $this->output->expects($this->once())->method('info')->with(sprintf('%d calendar subscriptions without an user have been cleaned up', $deletions));
+
+        $this->migration->run($this->output);
+    }
+
+    public static function dataTestRun(): array {
+        return [
+            [[], [], 0],
+            [
+                [
+                    [
+                        'id' => 1,
+                        'principaluri' => 'users/principals/foo1',
+                    ],
+                    [
+                        'id' => 2,
+                        'principaluri' => 'users/principals/bar1',
+                    ],
+                    [
+                        'id' => 3,
+                        'principaluri' => 'users/principals/bar1',
+                    ],
+                    [],
+                ],
+                ['foo1' => true, 'bar1' => false],
+                2
+            ],
+        ];
+    }
 }

apps/dav/tests/unit/Listener/ActivityUpdaterListenerTest.php

Indentation +54 added lines, -54 removed lines

@@ -20,65 +20,65 @@
 
 class ActivityUpdaterListenerTest extends TestCase {
 
-	private ActivityBackend&MockObject $activityBackend;
-	private LoggerInterface&MockObject $logger;
-	private ActivityUpdaterListener $listener;
+    private ActivityBackend&MockObject $activityBackend;
+    private LoggerInterface&MockObject $logger;
+    private ActivityUpdaterListener $listener;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->activityBackend = $this->createMock(ActivityBackend::class);
-		$this->logger = $this->createMock(LoggerInterface::class);
+        $this->activityBackend = $this->createMock(ActivityBackend::class);
+        $this->logger = $this->createMock(LoggerInterface::class);
 
-		$this->listener = new ActivityUpdaterListener(
-			$this->activityBackend,
-			$this->logger
-		);
-	}
+        $this->listener = new ActivityUpdaterListener(
+            $this->activityBackend,
+            $this->logger
+        );
+    }
 
-	/**
-	 * @dataProvider dataForTestHandleCalendarObjectDeletedEvent
-	 */
-	public function testHandleCalendarObjectDeletedEvent(int $calendarId, array $calendarData, array $shares, array $objectData, bool $createsActivity): void {
-		$event = new CalendarObjectDeletedEvent($calendarId, $calendarData, $shares, $objectData);
-		$this->logger->expects($this->once())->method('debug')->with(
-			$createsActivity ? "Activity generated for deleted calendar object in calendar $calendarId" : "Calendar object in calendar $calendarId was already in trashbin, skipping deletion activity"
-		);
-		$this->activityBackend->expects($createsActivity ? $this->once() : $this->never())->method('onTouchCalendarObject')->with(
-			Event::SUBJECT_OBJECT_DELETE,
-			$calendarData,
-			$shares,
-			$objectData
-		);
-		$this->listener->handle($event);
-	}
+    /**
+     * @dataProvider dataForTestHandleCalendarObjectDeletedEvent
+     */
+    public function testHandleCalendarObjectDeletedEvent(int $calendarId, array $calendarData, array $shares, array $objectData, bool $createsActivity): void {
+        $event = new CalendarObjectDeletedEvent($calendarId, $calendarData, $shares, $objectData);
+        $this->logger->expects($this->once())->method('debug')->with(
+            $createsActivity ? "Activity generated for deleted calendar object in calendar $calendarId" : "Calendar object in calendar $calendarId was already in trashbin, skipping deletion activity"
+        );
+        $this->activityBackend->expects($createsActivity ? $this->once() : $this->never())->method('onTouchCalendarObject')->with(
+            Event::SUBJECT_OBJECT_DELETE,
+            $calendarData,
+            $shares,
+            $objectData
+        );
+        $this->listener->handle($event);
+    }
 
-	public static function dataForTestHandleCalendarObjectDeletedEvent(): array {
-		return [
-			[1, [], [], [], true],
-			[1, [], [], ['{' . SharingPlugin::NS_NEXTCLOUD . '}deleted-at' => 120], false],
-		];
-	}
+    public static function dataForTestHandleCalendarObjectDeletedEvent(): array {
+        return [
+            [1, [], [], [], true],
+            [1, [], [], ['{' . SharingPlugin::NS_NEXTCLOUD . '}deleted-at' => 120], false],
+        ];
+    }
 
-	/**
-	 * @dataProvider dataForTestHandleCalendarDeletedEvent
-	 */
-	public function testHandleCalendarDeletedEvent(int $calendarId, array $calendarData, array $shares, bool $createsActivity): void {
-		$event = new CalendarDeletedEvent($calendarId, $calendarData, $shares);
-		$this->logger->expects($this->once())->method('debug')->with(
-			$createsActivity ? "Activity generated for deleted calendar $calendarId" : "Calendar $calendarId was already in trashbin, skipping deletion activity"
-		);
-		$this->activityBackend->expects($createsActivity ? $this->once() : $this->never())->method('onCalendarDelete')->with(
-			$calendarData,
-			$shares
-		);
-		$this->listener->handle($event);
-	}
+    /**
+     * @dataProvider dataForTestHandleCalendarDeletedEvent
+     */
+    public function testHandleCalendarDeletedEvent(int $calendarId, array $calendarData, array $shares, bool $createsActivity): void {
+        $event = new CalendarDeletedEvent($calendarId, $calendarData, $shares);
+        $this->logger->expects($this->once())->method('debug')->with(
+            $createsActivity ? "Activity generated for deleted calendar $calendarId" : "Calendar $calendarId was already in trashbin, skipping deletion activity"
+        );
+        $this->activityBackend->expects($createsActivity ? $this->once() : $this->never())->method('onCalendarDelete')->with(
+            $calendarData,
+            $shares
+        );
+        $this->listener->handle($event);
+    }
 
-	public static function dataForTestHandleCalendarDeletedEvent(): array {
-		return [
-			[1, [], [], true],
-			[1, ['{' . SharingPlugin::NS_NEXTCLOUD . '}deleted-at' => 120], [], false],
-		];
-	}
+    public static function dataForTestHandleCalendarDeletedEvent(): array {
+        return [
+            [1, [], [], true],
+            [1, ['{' . SharingPlugin::NS_NEXTCLOUD . '}deleted-at' => 120], [], false],
+        ];
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -56,7 +56,7 @@ 
 	public static function dataForTestHandleCalendarObjectDeletedEvent(): array {
 		return [
 			[1, [], [], [], true],
-			[1, [], [], ['{' . SharingPlugin::NS_NEXTCLOUD . '}deleted-at' => 120], false],
+			[1, [], [], ['{'.SharingPlugin::NS_NEXTCLOUD.'}deleted-at' => 120], false],
 		];
 	}
 
@@ -78,7 +78,7 @@ 
 	public static function dataForTestHandleCalendarDeletedEvent(): array {
 		return [
 			[1, [], [], true],
-			[1, ['{' . SharingPlugin::NS_NEXTCLOUD . '}deleted-at' => 120], [], false],
+			[1, ['{'.SharingPlugin::NS_NEXTCLOUD.'}deleted-at' => 120], [], false],
 		];
 	}
 }

apps/dav/tests/unit/Listener/CalendarContactInteractionListenerTest.php

Indentation +71 added lines, -71 removed lines

@@ -23,59 +23,59 @@ 
 use Test\TestCase;
 
 class CalendarContactInteractionListenerTest extends TestCase {
-	private IEventDispatcher&MockObject $eventDispatcher;
-	private IUserSession&MockObject $userSession;
-	private Principal&MockObject $principalConnector;
-	private LoggerInterface&MockObject $logger;
-	private IMailer&MockObject $mailer;
-	private CalendarContactInteractionListener $listener;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->eventDispatcher = $this->createMock(IEventDispatcher::class);
-		$this->userSession = $this->createMock(IUserSession::class);
-		$this->principalConnector = $this->createMock(Principal::class);
-		$this->mailer = $this->createMock(IMailer::class);
-		$this->logger = $this->createMock(LoggerInterface::class);
-
-		$this->listener = new CalendarContactInteractionListener(
-			$this->eventDispatcher,
-			$this->userSession,
-			$this->principalConnector,
-			$this->mailer,
-			$this->logger
-		);
-	}
-
-	public function testParseUnrelated(): void {
-		$event = new Event();
-		$this->eventDispatcher->expects(self::never())->method('dispatchTyped');
-
-		$this->listener->handle($event);
-	}
-
-	public function testHandleWithoutAnythingInteresting(): void {
-		$event = new CalendarShareUpdatedEvent(123, [], [], [], []);
-		$user = $this->createMock(IUser::class);
-		$this->userSession->expects(self::once())->method('getUser')->willReturn($user);
-		$this->eventDispatcher->expects(self::never())->method('dispatchTyped');
-
-		$this->listener->handle($event);
-	}
-
-	public function testParseInvalidData(): void {
-		$event = new CalendarObjectCreatedEvent(123, [], [], ['calendardata' => 'BEGIN:FOO']);
-		$user = $this->createMock(IUser::class);
-		$this->userSession->expects(self::once())->method('getUser')->willReturn($user);
-		$this->eventDispatcher->expects(self::never())->method('dispatchTyped');
-		$this->logger->expects(self::once())->method('warning');
-
-		$this->listener->handle($event);
-	}
-
-	public function testParseCalendarEventWithInvalidEmail(): void {
-		$event = new CalendarObjectCreatedEvent(123, [], [], ['calendardata' => <<<EVENT
+    private IEventDispatcher&MockObject $eventDispatcher;
+    private IUserSession&MockObject $userSession;
+    private Principal&MockObject $principalConnector;
+    private LoggerInterface&MockObject $logger;
+    private IMailer&MockObject $mailer;
+    private CalendarContactInteractionListener $listener;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->eventDispatcher = $this->createMock(IEventDispatcher::class);
+        $this->userSession = $this->createMock(IUserSession::class);
+        $this->principalConnector = $this->createMock(Principal::class);
+        $this->mailer = $this->createMock(IMailer::class);
+        $this->logger = $this->createMock(LoggerInterface::class);
+
+        $this->listener = new CalendarContactInteractionListener(
+            $this->eventDispatcher,
+            $this->userSession,
+            $this->principalConnector,
+            $this->mailer,
+            $this->logger
+        );
+    }
+
+    public function testParseUnrelated(): void {
+        $event = new Event();
+        $this->eventDispatcher->expects(self::never())->method('dispatchTyped');
+
+        $this->listener->handle($event);
+    }
+
+    public function testHandleWithoutAnythingInteresting(): void {
+        $event = new CalendarShareUpdatedEvent(123, [], [], [], []);
+        $user = $this->createMock(IUser::class);
+        $this->userSession->expects(self::once())->method('getUser')->willReturn($user);
+        $this->eventDispatcher->expects(self::never())->method('dispatchTyped');
+
+        $this->listener->handle($event);
+    }
+
+    public function testParseInvalidData(): void {
+        $event = new CalendarObjectCreatedEvent(123, [], [], ['calendardata' => 'BEGIN:FOO']);
+        $user = $this->createMock(IUser::class);
+        $this->userSession->expects(self::once())->method('getUser')->willReturn($user);
+        $this->eventDispatcher->expects(self::never())->method('dispatchTyped');
+        $this->logger->expects(self::once())->method('warning');
+
+        $this->listener->handle($event);
+    }
+
+    public function testParseCalendarEventWithInvalidEmail(): void {
+        $event = new CalendarObjectCreatedEvent(123, [], [], ['calendardata' => <<<EVENT
 BEGIN:VCALENDAR
 VERSION:2.0
 CALSCALE:GREGORIAN
@@ -113,16 +113,16 @@ 
 END:VEVENT
 END:VCALENDAR
 EVENT]);
-		$user = $this->createMock(IUser::class);
-		$this->userSession->expects(self::once())->method('getUser')->willReturn($user);
-		$this->eventDispatcher->expects(self::never())->method('dispatchTyped');
-		$this->logger->expects(self::never())->method('warning');
+        $user = $this->createMock(IUser::class);
+        $this->userSession->expects(self::once())->method('getUser')->willReturn($user);
+        $this->eventDispatcher->expects(self::never())->method('dispatchTyped');
+        $this->logger->expects(self::never())->method('warning');
 
-		$this->listener->handle($event);
-	}
+        $this->listener->handle($event);
+    }
 
-	public function testParseCalendarEvent(): void {
-		$event = new CalendarObjectCreatedEvent(123, [], [], ['calendardata' => <<<EVENT
+    public function testParseCalendarEvent(): void {
+        $event = new CalendarObjectCreatedEvent(123, [], [], ['calendardata' => <<<EVENT
 BEGIN:VCALENDAR
 VERSION:2.0
 CALSCALE:GREGORIAN
@@ -160,14 +160,14 @@ 
 END:VEVENT
 END:VCALENDAR
 EVENT]);
-		$user = $this->createMock(IUser::class);
-		$this->userSession->expects(self::once())->method('getUser')->willReturn($user);
-		$this->mailer->expects(self::once())->method('validateMailAddress')->willReturn(true);
-		$this->eventDispatcher->expects(self::once())
-			->method('dispatchTyped')
-			->with(self::equalTo((new ContactInteractedWithEvent($user))->setEmail('user@domain.tld')));
-		$this->logger->expects(self::never())->method('warning');
-
-		$this->listener->handle($event);
-	}
+        $user = $this->createMock(IUser::class);
+        $this->userSession->expects(self::once())->method('getUser')->willReturn($user);
+        $this->mailer->expects(self::once())->method('validateMailAddress')->willReturn(true);
+        $this->eventDispatcher->expects(self::once())
+            ->method('dispatchTyped')
+            ->with(self::equalTo((new ContactInteractedWithEvent($user))->setEmail('user@domain.tld')));
+        $this->logger->expects(self::never())->method('warning');
+
+        $this->listener->handle($event);
+    }
 }

apps/dav/tests/unit/Listener/OutOfOfficeListenerTest.php

Indentation +562 added lines, -562 removed lines

@@ -41,566 +41,566 @@
  */
 class OutOfOfficeListenerTest extends TestCase {
 
-	private ServerFactory&MockObject $serverFactory;
-	private IConfig&MockObject $appConfig;
-	private LoggerInterface&MockObject $loggerInterface;
-	private TimezoneService&MockObject $timezoneService;
-	private OutOfOfficeListener $listener;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->serverFactory = $this->createMock(ServerFactory::class);
-		$this->appConfig = $this->createMock(IConfig::class);
-		$this->timezoneService = $this->createMock(TimezoneService::class);
-		$this->loggerInterface = $this->createMock(LoggerInterface::class);
-
-		$this->listener = new OutOfOfficeListener(
-			$this->serverFactory,
-			$this->appConfig,
-			$this->timezoneService,
-			$this->loggerInterface,
-		);
-	}
-
-	public function testHandleUnrelated(): void {
-		$event = new Event();
-
-		$this->listener->handle($event);
-
-		$this->addToAssertionCount(1);
-	}
-
-	public function testHandleSchedulingNoCalendarHome(): void {
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')->willReturn('user123');
-		$data = $this->createMock(IOutOfOfficeData::class);
-		$data->method('getUser')->willReturn($user);
-		$davServer = $this->createMock(Server::class);
-		$invitationServer = $this->createMock(InvitationResponseServer::class);
-		$invitationServer->method('getServer')->willReturn($davServer);
-		$this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
-		$caldavPlugin = $this->createMock(Plugin::class);
-		$davServer->expects(self::once())
-			->method('getPlugin')
-			->with('caldav')
-			->willReturn($caldavPlugin);
-		$event = new OutOfOfficeScheduledEvent($data);
-
-		$this->listener->handle($event);
-	}
-
-	public function testHandleSchedulingNoCalendarHomeNode(): void {
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')->willReturn('user123');
-		$data = $this->createMock(IOutOfOfficeData::class);
-		$data->method('getUser')->willReturn($user);
-		$davServer = $this->createMock(Server::class);
-		$invitationServer = $this->createMock(InvitationResponseServer::class);
-		$invitationServer->method('getServer')->willReturn($davServer);
-		$this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
-		$caldavPlugin = $this->createMock(Plugin::class);
-		$davServer->expects(self::once())
-			->method('getPlugin')
-			->with('caldav')
-			->willReturn($caldavPlugin);
-		$caldavPlugin->expects(self::once())
-			->method('getCalendarHomeForPrincipal')
-			->willReturn('/home/calendar');
-		$tree = $this->createMock(Tree::class);
-		$davServer->tree = $tree;
-		$tree->expects(self::once())
-			->method('getNodeForPath')
-			->with('/home/calendar')
-			->willThrowException(new NotFound('nope'));
-		$event = new OutOfOfficeScheduledEvent($data);
-
-		$this->listener->handle($event);
-	}
-
-	public function testHandleSchedulingPersonalCalendarNotFound(): void {
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')->willReturn('user123');
-		$data = $this->createMock(IOutOfOfficeData::class);
-		$data->method('getUser')->willReturn($user);
-		$davServer = $this->createMock(Server::class);
-		$invitationServer = $this->createMock(InvitationResponseServer::class);
-		$invitationServer->method('getServer')->willReturn($davServer);
-		$this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
-		$caldavPlugin = $this->createMock(Plugin::class);
-		$davServer->expects(self::once())
-			->method('getPlugin')
-			->with('caldav')
-			->willReturn($caldavPlugin);
-		$caldavPlugin->expects(self::once())
-			->method('getCalendarHomeForPrincipal')
-			->willReturn('/home/calendar');
-		$tree = $this->createMock(Tree::class);
-		$davServer->tree = $tree;
-		$calendarHome = $this->createMock(CalendarHome::class);
-		$tree->expects(self::once())
-			->method('getNodeForPath')
-			->with('/home/calendar')
-			->willReturn($calendarHome);
-		$this->appConfig->expects(self::once())
-			->method('getUserValue')
-			->with('user123', 'dav', 'defaultCalendar', 'personal')
-			->willReturn('personal-1');
-		$calendarHome->expects(self::once())
-			->method('getChild')
-			->with('personal-1')
-			->willThrowException(new NotFound('nope'));
-		$event = new OutOfOfficeScheduledEvent($data);
-
-		$this->listener->handle($event);
-	}
-
-	public function testHandleSchedulingWithDefaultTimezone(): void {
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')->willReturn('user123');
-		$data = $this->createMock(IOutOfOfficeData::class);
-		$data->method('getUser')->willReturn($user);
-		$data->method('getStartDate')
-			->willReturn((new DateTimeImmutable('2023-12-12T00:00:00Z'))->getTimestamp());
-		$data->method('getEndDate')
-			->willReturn((new DateTimeImmutable('2023-12-13T00:00:00Z'))->getTimestamp());
-		$davServer = $this->createMock(Server::class);
-		$invitationServer = $this->createMock(InvitationResponseServer::class);
-		$invitationServer->method('getServer')->willReturn($davServer);
-		$this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
-		$caldavPlugin = $this->createMock(Plugin::class);
-		$davServer->expects(self::once())
-			->method('getPlugin')
-			->with('caldav')
-			->willReturn($caldavPlugin);
-		$caldavPlugin->expects(self::once())
-			->method('getCalendarHomeForPrincipal')
-			->willReturn('/home/calendar');
-		$tree = $this->createMock(Tree::class);
-		$davServer->tree = $tree;
-		$calendarHome = $this->createMock(CalendarHome::class);
-		$tree->expects(self::once())
-			->method('getNodeForPath')
-			->with('/home/calendar')
-			->willReturn($calendarHome);
-		$this->appConfig->expects(self::once())
-			->method('getUserValue')
-			->with('user123', 'dav', 'defaultCalendar', 'personal')
-			->willReturn('personal-1');
-		$calendar = $this->createMock(Calendar::class);
-		$this->timezoneService->expects(self::once())
-			->method('getUserTimezone')
-			->with('user123')
-			->willReturn('Europe/Prague');
-		$calendarHome->expects(self::once())
-			->method('getChild')
-			->with('personal-1')
-			->willReturn($calendar);
-		$calendar->expects(self::once())
-			->method('createFile')
-			->willReturnCallback(function ($name, $data): void {
-				$vcalendar = Reader::read($data);
-				if (!($vcalendar instanceof VCalendar)) {
-					throw new InvalidArgumentException('Calendar data should be a VCALENDAR');
-				}
-				$vevent = $vcalendar->VEVENT;
-				if ($vevent === null || !($vevent instanceof VEvent)) {
-					throw new InvalidArgumentException('Calendar data should contain a VEVENT');
-				}
-				self::assertSame('Europe/Prague', $vevent->DTSTART['TZID']?->getValue());
-				self::assertSame('Europe/Prague', $vevent->DTEND['TZID']?->getValue());
-			});
-		$event = new OutOfOfficeScheduledEvent($data);
-
-		$this->listener->handle($event);
-	}
-
-	public function testHandleChangeNoCalendarHome(): void {
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')->willReturn('user123');
-		$data = $this->createMock(IOutOfOfficeData::class);
-		$data->method('getUser')->willReturn($user);
-		$davServer = $this->createMock(Server::class);
-		$invitationServer = $this->createMock(InvitationResponseServer::class);
-		$invitationServer->method('getServer')->willReturn($davServer);
-		$this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
-		$caldavPlugin = $this->createMock(Plugin::class);
-		$davServer->expects(self::once())
-			->method('getPlugin')
-			->with('caldav')
-			->willReturn($caldavPlugin);
-		$event = new OutOfOfficeChangedEvent($data);
-
-		$this->listener->handle($event);
-	}
-
-	public function testHandleChangeNoCalendarHomeNode(): void {
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')->willReturn('user123');
-		$data = $this->createMock(IOutOfOfficeData::class);
-		$data->method('getUser')->willReturn($user);
-		$davServer = $this->createMock(Server::class);
-		$invitationServer = $this->createMock(InvitationResponseServer::class);
-		$invitationServer->method('getServer')->willReturn($davServer);
-		$this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
-		$caldavPlugin = $this->createMock(Plugin::class);
-		$davServer->expects(self::once())
-			->method('getPlugin')
-			->with('caldav')
-			->willReturn($caldavPlugin);
-		$caldavPlugin->expects(self::once())
-			->method('getCalendarHomeForPrincipal')
-			->willReturn('/home/calendar');
-		$tree = $this->createMock(Tree::class);
-		$davServer->tree = $tree;
-		$tree->expects(self::once())
-			->method('getNodeForPath')
-			->with('/home/calendar')
-			->willThrowException(new NotFound('nope'));
-		$event = new OutOfOfficeChangedEvent($data);
-
-		$this->listener->handle($event);
-	}
-
-	public function testHandleChangePersonalCalendarNotFound(): void {
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')->willReturn('user123');
-		$data = $this->createMock(IOutOfOfficeData::class);
-		$data->method('getUser')->willReturn($user);
-		$davServer = $this->createMock(Server::class);
-		$invitationServer = $this->createMock(InvitationResponseServer::class);
-		$invitationServer->method('getServer')->willReturn($davServer);
-		$this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
-		$caldavPlugin = $this->createMock(Plugin::class);
-		$davServer->expects(self::once())
-			->method('getPlugin')
-			->with('caldav')
-			->willReturn($caldavPlugin);
-		$caldavPlugin->expects(self::once())
-			->method('getCalendarHomeForPrincipal')
-			->willReturn('/home/calendar');
-		$tree = $this->createMock(Tree::class);
-		$davServer->tree = $tree;
-		$calendarHome = $this->createMock(CalendarHome::class);
-		$tree->expects(self::once())
-			->method('getNodeForPath')
-			->with('/home/calendar')
-			->willReturn($calendarHome);
-		$this->appConfig->expects(self::once())
-			->method('getUserValue')
-			->with('user123', 'dav', 'defaultCalendar', 'personal')
-			->willReturn('personal-1');
-		$calendarHome->expects(self::once())
-			->method('getChild')
-			->with('personal-1')
-			->willThrowException(new NotFound('nope'));
-		$event = new OutOfOfficeChangedEvent($data);
-
-		$this->listener->handle($event);
-	}
-
-	public function testHandleChangeRecreate(): void {
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')->willReturn('user123');
-		$data = $this->createMock(IOutOfOfficeData::class);
-		$data->method('getUser')->willReturn($user);
-		$data->method('getStartDate')
-			->willReturn((new DateTimeImmutable('2023-12-12T00:00:00Z'))->getTimestamp());
-		$data->method('getEndDate')
-			->willReturn((new DateTimeImmutable('2023-12-14T00:00:00Z'))->getTimestamp());
-		$davServer = $this->createMock(Server::class);
-		$invitationServer = $this->createMock(InvitationResponseServer::class);
-		$invitationServer->method('getServer')->willReturn($davServer);
-		$this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
-		$caldavPlugin = $this->createMock(Plugin::class);
-		$davServer->expects(self::once())
-			->method('getPlugin')
-			->with('caldav')
-			->willReturn($caldavPlugin);
-		$caldavPlugin->expects(self::once())
-			->method('getCalendarHomeForPrincipal')
-			->willReturn('/home/calendar');
-		$tree = $this->createMock(Tree::class);
-		$davServer->tree = $tree;
-		$calendarHome = $this->createMock(CalendarHome::class);
-		$tree->expects(self::once())
-			->method('getNodeForPath')
-			->with('/home/calendar')
-			->willReturn($calendarHome);
-		$this->appConfig->expects(self::once())
-			->method('getUserValue')
-			->with('user123', 'dav', 'defaultCalendar', 'personal')
-			->willReturn('personal-1');
-		$calendar = $this->createMock(Calendar::class);
-		$this->timezoneService->expects(self::once())
-			->method('getUserTimezone')
-			->with('user123')
-			->willReturn(null);
-		$this->timezoneService->expects(self::once())
-			->method('getDefaultTimezone')
-			->willReturn('Europe/Berlin');
-		$calendarHome->expects(self::once())
-			->method('getChild')
-			->with('personal-1')
-			->willReturn($calendar);
-		$calendar->expects(self::once())
-			->method('getChild')
-			->willThrowException(new NotFound());
-		$calendar->expects(self::once())
-			->method('createFile')
-			->willReturnCallback(function ($name, $data): void {
-				$vcalendar = Reader::read($data);
-				if (!($vcalendar instanceof VCalendar)) {
-					throw new InvalidArgumentException('Calendar data should be a VCALENDAR');
-				}
-				$vevent = $vcalendar->VEVENT;
-				if ($vevent === null || !($vevent instanceof VEvent)) {
-					throw new InvalidArgumentException('Calendar data should contain a VEVENT');
-				}
-				self::assertSame('Europe/Berlin', $vevent->DTSTART['TZID']?->getValue());
-				self::assertSame('Europe/Berlin', $vevent->DTEND['TZID']?->getValue());
-			});
-		$event = new OutOfOfficeChangedEvent($data);
-
-		$this->listener->handle($event);
-	}
-
-	public function testHandleChangeWithoutTimezone(): void {
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')->willReturn('user123');
-		$data = $this->createMock(IOutOfOfficeData::class);
-		$data->method('getUser')->willReturn($user);
-		$data->method('getStartDate')
-			->willReturn((new DateTimeImmutable('2023-01-12T00:00:00Z'))->getTimestamp());
-		$data->method('getEndDate')
-			->willReturn((new DateTimeImmutable('2023-12-14T00:00:00Z'))->getTimestamp());
-		$davServer = $this->createMock(Server::class);
-		$invitationServer = $this->createMock(InvitationResponseServer::class);
-		$invitationServer->method('getServer')->willReturn($davServer);
-		$this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
-		$caldavPlugin = $this->createMock(Plugin::class);
-		$davServer->expects(self::once())
-			->method('getPlugin')
-			->with('caldav')
-			->willReturn($caldavPlugin);
-		$caldavPlugin->expects(self::once())
-			->method('getCalendarHomeForPrincipal')
-			->willReturn('/home/calendar');
-		$tree = $this->createMock(Tree::class);
-		$davServer->tree = $tree;
-		$calendarHome = $this->createMock(CalendarHome::class);
-		$tree->expects(self::once())
-			->method('getNodeForPath')
-			->with('/home/calendar')
-			->willReturn($calendarHome);
-		$this->appConfig->expects(self::once())
-			->method('getUserValue')
-			->with('user123', 'dav', 'defaultCalendar', 'personal')
-			->willReturn('personal-1');
-		$calendar = $this->createMock(Calendar::class);
-		$calendarHome->expects(self::once())
-			->method('getChild')
-			->with('personal-1')
-			->willReturn($calendar);
-		$eventNode = $this->createMock(CalendarObject::class);
-		$this->timezoneService->expects(self::once())
-			->method('getUserTimezone')
-			->with('user123')
-			->willReturn(null);
-		$this->timezoneService->expects(self::once())
-			->method('getDefaultTimezone')
-			->willReturn('UTC');
-		$calendar->expects(self::once())
-			->method('getChild')
-			->willReturn($eventNode);
-		$eventNode->expects(self::once())
-			->method('put')
-			->willReturnCallback(function ($data): void {
-				$vcalendar = Reader::read($data);
-				if (!($vcalendar instanceof VCalendar)) {
-					throw new InvalidArgumentException('Calendar data should be a VCALENDAR');
-				}
-				$vevent = $vcalendar->VEVENT;
-				if ($vevent === null || !($vevent instanceof VEvent)) {
-					throw new InvalidArgumentException('Calendar data should contain a VEVENT');
-				}
-				// UTC datetimes are stored without a TZID
-				self::assertSame(null, $vevent->DTSTART['TZID']?->getValue());
-				self::assertSame(null, $vevent->DTEND['TZID']?->getValue());
-			});
-		$calendar->expects(self::never())
-			->method('createFile');
-		$event = new OutOfOfficeChangedEvent($data);
-
-		$this->listener->handle($event);
-	}
-
-	public function testHandleClearNoCalendarHome(): void {
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')->willReturn('user123');
-		$data = $this->createMock(IOutOfOfficeData::class);
-		$data->method('getUser')->willReturn($user);
-		$davServer = $this->createMock(Server::class);
-		$invitationServer = $this->createMock(InvitationResponseServer::class);
-		$invitationServer->method('getServer')->willReturn($davServer);
-		$this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
-		$caldavPlugin = $this->createMock(Plugin::class);
-		$davServer->expects(self::once())
-			->method('getPlugin')
-			->with('caldav')
-			->willReturn($caldavPlugin);
-		$event = new OutOfOfficeClearedEvent($data);
-
-		$this->listener->handle($event);
-	}
-
-	public function testHandleClearNoCalendarHomeNode(): void {
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')->willReturn('user123');
-		$data = $this->createMock(IOutOfOfficeData::class);
-		$data->method('getUser')->willReturn($user);
-		$davServer = $this->createMock(Server::class);
-		$invitationServer = $this->createMock(InvitationResponseServer::class);
-		$invitationServer->method('getServer')->willReturn($davServer);
-		$this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
-		$caldavPlugin = $this->createMock(Plugin::class);
-		$davServer->expects(self::once())
-			->method('getPlugin')
-			->with('caldav')
-			->willReturn($caldavPlugin);
-		$caldavPlugin->expects(self::once())
-			->method('getCalendarHomeForPrincipal')
-			->willReturn('/home/calendar');
-		$tree = $this->createMock(Tree::class);
-		$davServer->tree = $tree;
-		$tree->expects(self::once())
-			->method('getNodeForPath')
-			->with('/home/calendar')
-			->willThrowException(new NotFound('nope'));
-		$event = new OutOfOfficeClearedEvent($data);
-
-		$this->listener->handle($event);
-	}
-
-	public function testHandleClearPersonalCalendarNotFound(): void {
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')->willReturn('user123');
-		$data = $this->createMock(IOutOfOfficeData::class);
-		$data->method('getUser')->willReturn($user);
-		$davServer = $this->createMock(Server::class);
-		$invitationServer = $this->createMock(InvitationResponseServer::class);
-		$invitationServer->method('getServer')->willReturn($davServer);
-		$this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
-		$caldavPlugin = $this->createMock(Plugin::class);
-		$davServer->expects(self::once())
-			->method('getPlugin')
-			->with('caldav')
-			->willReturn($caldavPlugin);
-		$caldavPlugin->expects(self::once())
-			->method('getCalendarHomeForPrincipal')
-			->willReturn('/home/calendar');
-		$tree = $this->createMock(Tree::class);
-		$davServer->tree = $tree;
-		$calendarHome = $this->createMock(CalendarHome::class);
-		$tree->expects(self::once())
-			->method('getNodeForPath')
-			->with('/home/calendar')
-			->willReturn($calendarHome);
-		$this->appConfig->expects(self::once())
-			->method('getUserValue')
-			->with('user123', 'dav', 'defaultCalendar', 'personal')
-			->willReturn('personal-1');
-		$calendarHome->expects(self::once())
-			->method('getChild')
-			->with('personal-1')
-			->willThrowException(new NotFound('nope'));
-		$event = new OutOfOfficeClearedEvent($data);
-
-		$this->listener->handle($event);
-	}
-
-	public function testHandleClearRecreate(): void {
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')->willReturn('user123');
-		$data = $this->createMock(IOutOfOfficeData::class);
-		$data->method('getUser')->willReturn($user);
-		$davServer = $this->createMock(Server::class);
-		$invitationServer = $this->createMock(InvitationResponseServer::class);
-		$invitationServer->method('getServer')->willReturn($davServer);
-		$this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
-		$caldavPlugin = $this->createMock(Plugin::class);
-		$davServer->expects(self::once())
-			->method('getPlugin')
-			->with('caldav')
-			->willReturn($caldavPlugin);
-		$caldavPlugin->expects(self::once())
-			->method('getCalendarHomeForPrincipal')
-			->willReturn('/home/calendar');
-		$tree = $this->createMock(Tree::class);
-		$davServer->tree = $tree;
-		$calendarHome = $this->createMock(CalendarHome::class);
-		$tree->expects(self::once())
-			->method('getNodeForPath')
-			->with('/home/calendar')
-			->willReturn($calendarHome);
-		$this->appConfig->expects(self::once())
-			->method('getUserValue')
-			->with('user123', 'dav', 'defaultCalendar', 'personal')
-			->willReturn('personal-1');
-		$calendar = $this->createMock(Calendar::class);
-		$calendarHome->expects(self::once())
-			->method('getChild')
-			->with('personal-1')
-			->willReturn($calendar);
-		$calendar->expects(self::once())
-			->method('getChild')
-			->willThrowException(new NotFound());
-		$event = new OutOfOfficeClearedEvent($data);
-
-		$this->listener->handle($event);
-	}
-
-	public function testHandleClear(): void {
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')->willReturn('user123');
-		$data = $this->createMock(IOutOfOfficeData::class);
-		$data->method('getUser')->willReturn($user);
-		$davServer = $this->createMock(Server::class);
-		$invitationServer = $this->createMock(InvitationResponseServer::class);
-		$invitationServer->method('getServer')->willReturn($davServer);
-		$this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
-		$caldavPlugin = $this->createMock(Plugin::class);
-		$davServer->expects(self::once())
-			->method('getPlugin')
-			->with('caldav')
-			->willReturn($caldavPlugin);
-		$caldavPlugin->expects(self::once())
-			->method('getCalendarHomeForPrincipal')
-			->willReturn('/home/calendar');
-		$tree = $this->createMock(Tree::class);
-		$davServer->tree = $tree;
-		$calendarHome = $this->createMock(CalendarHome::class);
-		$tree->expects(self::once())
-			->method('getNodeForPath')
-			->with('/home/calendar')
-			->willReturn($calendarHome);
-		$this->appConfig->expects(self::once())
-			->method('getUserValue')
-			->with('user123', 'dav', 'defaultCalendar', 'personal')
-			->willReturn('personal-1');
-		$calendar = $this->createMock(Calendar::class);
-		$calendarHome->expects(self::once())
-			->method('getChild')
-			->with('personal-1')
-			->willReturn($calendar);
-		$eventNode = $this->createMock(CalendarObject::class);
-		$calendar->expects(self::once())
-			->method('getChild')
-			->willReturn($eventNode);
-		$eventNode->expects(self::once())
-			->method('delete');
-		$event = new OutOfOfficeClearedEvent($data);
-
-		$this->listener->handle($event);
-	}
+    private ServerFactory&MockObject $serverFactory;
+    private IConfig&MockObject $appConfig;
+    private LoggerInterface&MockObject $loggerInterface;
+    private TimezoneService&MockObject $timezoneService;
+    private OutOfOfficeListener $listener;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->serverFactory = $this->createMock(ServerFactory::class);
+        $this->appConfig = $this->createMock(IConfig::class);
+        $this->timezoneService = $this->createMock(TimezoneService::class);
+        $this->loggerInterface = $this->createMock(LoggerInterface::class);
+
+        $this->listener = new OutOfOfficeListener(
+            $this->serverFactory,
+            $this->appConfig,
+            $this->timezoneService,
+            $this->loggerInterface,
+        );
+    }
+
+    public function testHandleUnrelated(): void {
+        $event = new Event();
+
+        $this->listener->handle($event);
+
+        $this->addToAssertionCount(1);
+    }
+
+    public function testHandleSchedulingNoCalendarHome(): void {
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')->willReturn('user123');
+        $data = $this->createMock(IOutOfOfficeData::class);
+        $data->method('getUser')->willReturn($user);
+        $davServer = $this->createMock(Server::class);
+        $invitationServer = $this->createMock(InvitationResponseServer::class);
+        $invitationServer->method('getServer')->willReturn($davServer);
+        $this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
+        $caldavPlugin = $this->createMock(Plugin::class);
+        $davServer->expects(self::once())
+            ->method('getPlugin')
+            ->with('caldav')
+            ->willReturn($caldavPlugin);
+        $event = new OutOfOfficeScheduledEvent($data);
+
+        $this->listener->handle($event);
+    }
+
+    public function testHandleSchedulingNoCalendarHomeNode(): void {
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')->willReturn('user123');
+        $data = $this->createMock(IOutOfOfficeData::class);
+        $data->method('getUser')->willReturn($user);
+        $davServer = $this->createMock(Server::class);
+        $invitationServer = $this->createMock(InvitationResponseServer::class);
+        $invitationServer->method('getServer')->willReturn($davServer);
+        $this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
+        $caldavPlugin = $this->createMock(Plugin::class);
+        $davServer->expects(self::once())
+            ->method('getPlugin')
+            ->with('caldav')
+            ->willReturn($caldavPlugin);
+        $caldavPlugin->expects(self::once())
+            ->method('getCalendarHomeForPrincipal')
+            ->willReturn('/home/calendar');
+        $tree = $this->createMock(Tree::class);
+        $davServer->tree = $tree;
+        $tree->expects(self::once())
+            ->method('getNodeForPath')
+            ->with('/home/calendar')
+            ->willThrowException(new NotFound('nope'));
+        $event = new OutOfOfficeScheduledEvent($data);
+
+        $this->listener->handle($event);
+    }
+
+    public function testHandleSchedulingPersonalCalendarNotFound(): void {
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')->willReturn('user123');
+        $data = $this->createMock(IOutOfOfficeData::class);
+        $data->method('getUser')->willReturn($user);
+        $davServer = $this->createMock(Server::class);
+        $invitationServer = $this->createMock(InvitationResponseServer::class);
+        $invitationServer->method('getServer')->willReturn($davServer);
+        $this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
+        $caldavPlugin = $this->createMock(Plugin::class);
+        $davServer->expects(self::once())
+            ->method('getPlugin')
+            ->with('caldav')
+            ->willReturn($caldavPlugin);
+        $caldavPlugin->expects(self::once())
+            ->method('getCalendarHomeForPrincipal')
+            ->willReturn('/home/calendar');
+        $tree = $this->createMock(Tree::class);
+        $davServer->tree = $tree;
+        $calendarHome = $this->createMock(CalendarHome::class);
+        $tree->expects(self::once())
+            ->method('getNodeForPath')
+            ->with('/home/calendar')
+            ->willReturn($calendarHome);
+        $this->appConfig->expects(self::once())
+            ->method('getUserValue')
+            ->with('user123', 'dav', 'defaultCalendar', 'personal')
+            ->willReturn('personal-1');
+        $calendarHome->expects(self::once())
+            ->method('getChild')
+            ->with('personal-1')
+            ->willThrowException(new NotFound('nope'));
+        $event = new OutOfOfficeScheduledEvent($data);
+
+        $this->listener->handle($event);
+    }
+
+    public function testHandleSchedulingWithDefaultTimezone(): void {
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')->willReturn('user123');
+        $data = $this->createMock(IOutOfOfficeData::class);
+        $data->method('getUser')->willReturn($user);
+        $data->method('getStartDate')
+            ->willReturn((new DateTimeImmutable('2023-12-12T00:00:00Z'))->getTimestamp());
+        $data->method('getEndDate')
+            ->willReturn((new DateTimeImmutable('2023-12-13T00:00:00Z'))->getTimestamp());
+        $davServer = $this->createMock(Server::class);
+        $invitationServer = $this->createMock(InvitationResponseServer::class);
+        $invitationServer->method('getServer')->willReturn($davServer);
+        $this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
+        $caldavPlugin = $this->createMock(Plugin::class);
+        $davServer->expects(self::once())
+            ->method('getPlugin')
+            ->with('caldav')
+            ->willReturn($caldavPlugin);
+        $caldavPlugin->expects(self::once())
+            ->method('getCalendarHomeForPrincipal')
+            ->willReturn('/home/calendar');
+        $tree = $this->createMock(Tree::class);
+        $davServer->tree = $tree;
+        $calendarHome = $this->createMock(CalendarHome::class);
+        $tree->expects(self::once())
+            ->method('getNodeForPath')
+            ->with('/home/calendar')
+            ->willReturn($calendarHome);
+        $this->appConfig->expects(self::once())
+            ->method('getUserValue')
+            ->with('user123', 'dav', 'defaultCalendar', 'personal')
+            ->willReturn('personal-1');
+        $calendar = $this->createMock(Calendar::class);
+        $this->timezoneService->expects(self::once())
+            ->method('getUserTimezone')
+            ->with('user123')
+            ->willReturn('Europe/Prague');
+        $calendarHome->expects(self::once())
+            ->method('getChild')
+            ->with('personal-1')
+            ->willReturn($calendar);
+        $calendar->expects(self::once())
+            ->method('createFile')
+            ->willReturnCallback(function ($name, $data): void {
+                $vcalendar = Reader::read($data);
+                if (!($vcalendar instanceof VCalendar)) {
+                    throw new InvalidArgumentException('Calendar data should be a VCALENDAR');
+                }
+                $vevent = $vcalendar->VEVENT;
+                if ($vevent === null || !($vevent instanceof VEvent)) {
+                    throw new InvalidArgumentException('Calendar data should contain a VEVENT');
+                }
+                self::assertSame('Europe/Prague', $vevent->DTSTART['TZID']?->getValue());
+                self::assertSame('Europe/Prague', $vevent->DTEND['TZID']?->getValue());
+            });
+        $event = new OutOfOfficeScheduledEvent($data);
+
+        $this->listener->handle($event);
+    }
+
+    public function testHandleChangeNoCalendarHome(): void {
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')->willReturn('user123');
+        $data = $this->createMock(IOutOfOfficeData::class);
+        $data->method('getUser')->willReturn($user);
+        $davServer = $this->createMock(Server::class);
+        $invitationServer = $this->createMock(InvitationResponseServer::class);
+        $invitationServer->method('getServer')->willReturn($davServer);
+        $this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
+        $caldavPlugin = $this->createMock(Plugin::class);
+        $davServer->expects(self::once())
+            ->method('getPlugin')
+            ->with('caldav')
+            ->willReturn($caldavPlugin);
+        $event = new OutOfOfficeChangedEvent($data);
+
+        $this->listener->handle($event);
+    }
+
+    public function testHandleChangeNoCalendarHomeNode(): void {
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')->willReturn('user123');
+        $data = $this->createMock(IOutOfOfficeData::class);
+        $data->method('getUser')->willReturn($user);
+        $davServer = $this->createMock(Server::class);
+        $invitationServer = $this->createMock(InvitationResponseServer::class);
+        $invitationServer->method('getServer')->willReturn($davServer);
+        $this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
+        $caldavPlugin = $this->createMock(Plugin::class);
+        $davServer->expects(self::once())
+            ->method('getPlugin')
+            ->with('caldav')
+            ->willReturn($caldavPlugin);
+        $caldavPlugin->expects(self::once())
+            ->method('getCalendarHomeForPrincipal')
+            ->willReturn('/home/calendar');
+        $tree = $this->createMock(Tree::class);
+        $davServer->tree = $tree;
+        $tree->expects(self::once())
+            ->method('getNodeForPath')
+            ->with('/home/calendar')
+            ->willThrowException(new NotFound('nope'));
+        $event = new OutOfOfficeChangedEvent($data);
+
+        $this->listener->handle($event);
+    }
+
+    public function testHandleChangePersonalCalendarNotFound(): void {
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')->willReturn('user123');
+        $data = $this->createMock(IOutOfOfficeData::class);
+        $data->method('getUser')->willReturn($user);
+        $davServer = $this->createMock(Server::class);
+        $invitationServer = $this->createMock(InvitationResponseServer::class);
+        $invitationServer->method('getServer')->willReturn($davServer);
+        $this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
+        $caldavPlugin = $this->createMock(Plugin::class);
+        $davServer->expects(self::once())
+            ->method('getPlugin')
+            ->with('caldav')
+            ->willReturn($caldavPlugin);
+        $caldavPlugin->expects(self::once())
+            ->method('getCalendarHomeForPrincipal')
+            ->willReturn('/home/calendar');
+        $tree = $this->createMock(Tree::class);
+        $davServer->tree = $tree;
+        $calendarHome = $this->createMock(CalendarHome::class);
+        $tree->expects(self::once())
+            ->method('getNodeForPath')
+            ->with('/home/calendar')
+            ->willReturn($calendarHome);
+        $this->appConfig->expects(self::once())
+            ->method('getUserValue')
+            ->with('user123', 'dav', 'defaultCalendar', 'personal')
+            ->willReturn('personal-1');
+        $calendarHome->expects(self::once())
+            ->method('getChild')
+            ->with('personal-1')
+            ->willThrowException(new NotFound('nope'));
+        $event = new OutOfOfficeChangedEvent($data);
+
+        $this->listener->handle($event);
+    }
+
+    public function testHandleChangeRecreate(): void {
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')->willReturn('user123');
+        $data = $this->createMock(IOutOfOfficeData::class);
+        $data->method('getUser')->willReturn($user);
+        $data->method('getStartDate')
+            ->willReturn((new DateTimeImmutable('2023-12-12T00:00:00Z'))->getTimestamp());
+        $data->method('getEndDate')
+            ->willReturn((new DateTimeImmutable('2023-12-14T00:00:00Z'))->getTimestamp());
+        $davServer = $this->createMock(Server::class);
+        $invitationServer = $this->createMock(InvitationResponseServer::class);
+        $invitationServer->method('getServer')->willReturn($davServer);
+        $this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
+        $caldavPlugin = $this->createMock(Plugin::class);
+        $davServer->expects(self::once())
+            ->method('getPlugin')
+            ->with('caldav')
+            ->willReturn($caldavPlugin);
+        $caldavPlugin->expects(self::once())
+            ->method('getCalendarHomeForPrincipal')
+            ->willReturn('/home/calendar');
+        $tree = $this->createMock(Tree::class);
+        $davServer->tree = $tree;
+        $calendarHome = $this->createMock(CalendarHome::class);
+        $tree->expects(self::once())
+            ->method('getNodeForPath')
+            ->with('/home/calendar')
+            ->willReturn($calendarHome);
+        $this->appConfig->expects(self::once())
+            ->method('getUserValue')
+            ->with('user123', 'dav', 'defaultCalendar', 'personal')
+            ->willReturn('personal-1');
+        $calendar = $this->createMock(Calendar::class);
+        $this->timezoneService->expects(self::once())
+            ->method('getUserTimezone')
+            ->with('user123')
+            ->willReturn(null);
+        $this->timezoneService->expects(self::once())
+            ->method('getDefaultTimezone')
+            ->willReturn('Europe/Berlin');
+        $calendarHome->expects(self::once())
+            ->method('getChild')
+            ->with('personal-1')
+            ->willReturn($calendar);
+        $calendar->expects(self::once())
+            ->method('getChild')
+            ->willThrowException(new NotFound());
+        $calendar->expects(self::once())
+            ->method('createFile')
+            ->willReturnCallback(function ($name, $data): void {
+                $vcalendar = Reader::read($data);
+                if (!($vcalendar instanceof VCalendar)) {
+                    throw new InvalidArgumentException('Calendar data should be a VCALENDAR');
+                }
+                $vevent = $vcalendar->VEVENT;
+                if ($vevent === null || !($vevent instanceof VEvent)) {
+                    throw new InvalidArgumentException('Calendar data should contain a VEVENT');
+                }
+                self::assertSame('Europe/Berlin', $vevent->DTSTART['TZID']?->getValue());
+                self::assertSame('Europe/Berlin', $vevent->DTEND['TZID']?->getValue());
+            });
+        $event = new OutOfOfficeChangedEvent($data);
+
+        $this->listener->handle($event);
+    }
+
+    public function testHandleChangeWithoutTimezone(): void {
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')->willReturn('user123');
+        $data = $this->createMock(IOutOfOfficeData::class);
+        $data->method('getUser')->willReturn($user);
+        $data->method('getStartDate')
+            ->willReturn((new DateTimeImmutable('2023-01-12T00:00:00Z'))->getTimestamp());
+        $data->method('getEndDate')
+            ->willReturn((new DateTimeImmutable('2023-12-14T00:00:00Z'))->getTimestamp());
+        $davServer = $this->createMock(Server::class);
+        $invitationServer = $this->createMock(InvitationResponseServer::class);
+        $invitationServer->method('getServer')->willReturn($davServer);
+        $this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
+        $caldavPlugin = $this->createMock(Plugin::class);
+        $davServer->expects(self::once())
+            ->method('getPlugin')
+            ->with('caldav')
+            ->willReturn($caldavPlugin);
+        $caldavPlugin->expects(self::once())
+            ->method('getCalendarHomeForPrincipal')
+            ->willReturn('/home/calendar');
+        $tree = $this->createMock(Tree::class);
+        $davServer->tree = $tree;
+        $calendarHome = $this->createMock(CalendarHome::class);
+        $tree->expects(self::once())
+            ->method('getNodeForPath')
+            ->with('/home/calendar')
+            ->willReturn($calendarHome);
+        $this->appConfig->expects(self::once())
+            ->method('getUserValue')
+            ->with('user123', 'dav', 'defaultCalendar', 'personal')
+            ->willReturn('personal-1');
+        $calendar = $this->createMock(Calendar::class);
+        $calendarHome->expects(self::once())
+            ->method('getChild')
+            ->with('personal-1')
+            ->willReturn($calendar);
+        $eventNode = $this->createMock(CalendarObject::class);
+        $this->timezoneService->expects(self::once())
+            ->method('getUserTimezone')
+            ->with('user123')
+            ->willReturn(null);
+        $this->timezoneService->expects(self::once())
+            ->method('getDefaultTimezone')
+            ->willReturn('UTC');
+        $calendar->expects(self::once())
+            ->method('getChild')
+            ->willReturn($eventNode);
+        $eventNode->expects(self::once())
+            ->method('put')
+            ->willReturnCallback(function ($data): void {
+                $vcalendar = Reader::read($data);
+                if (!($vcalendar instanceof VCalendar)) {
+                    throw new InvalidArgumentException('Calendar data should be a VCALENDAR');
+                }
+                $vevent = $vcalendar->VEVENT;
+                if ($vevent === null || !($vevent instanceof VEvent)) {
+                    throw new InvalidArgumentException('Calendar data should contain a VEVENT');
+                }
+                // UTC datetimes are stored without a TZID
+                self::assertSame(null, $vevent->DTSTART['TZID']?->getValue());
+                self::assertSame(null, $vevent->DTEND['TZID']?->getValue());
+            });
+        $calendar->expects(self::never())
+            ->method('createFile');
+        $event = new OutOfOfficeChangedEvent($data);
+
+        $this->listener->handle($event);
+    }
+
+    public function testHandleClearNoCalendarHome(): void {
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')->willReturn('user123');
+        $data = $this->createMock(IOutOfOfficeData::class);
+        $data->method('getUser')->willReturn($user);
+        $davServer = $this->createMock(Server::class);
+        $invitationServer = $this->createMock(InvitationResponseServer::class);
+        $invitationServer->method('getServer')->willReturn($davServer);
+        $this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
+        $caldavPlugin = $this->createMock(Plugin::class);
+        $davServer->expects(self::once())
+            ->method('getPlugin')
+            ->with('caldav')
+            ->willReturn($caldavPlugin);
+        $event = new OutOfOfficeClearedEvent($data);
+
+        $this->listener->handle($event);
+    }
+
+    public function testHandleClearNoCalendarHomeNode(): void {
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')->willReturn('user123');
+        $data = $this->createMock(IOutOfOfficeData::class);
+        $data->method('getUser')->willReturn($user);
+        $davServer = $this->createMock(Server::class);
+        $invitationServer = $this->createMock(InvitationResponseServer::class);
+        $invitationServer->method('getServer')->willReturn($davServer);
+        $this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
+        $caldavPlugin = $this->createMock(Plugin::class);
+        $davServer->expects(self::once())
+            ->method('getPlugin')
+            ->with('caldav')
+            ->willReturn($caldavPlugin);
+        $caldavPlugin->expects(self::once())
+            ->method('getCalendarHomeForPrincipal')
+            ->willReturn('/home/calendar');
+        $tree = $this->createMock(Tree::class);
+        $davServer->tree = $tree;
+        $tree->expects(self::once())
+            ->method('getNodeForPath')
+            ->with('/home/calendar')
+            ->willThrowException(new NotFound('nope'));
+        $event = new OutOfOfficeClearedEvent($data);
+
+        $this->listener->handle($event);
+    }
+
+    public function testHandleClearPersonalCalendarNotFound(): void {
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')->willReturn('user123');
+        $data = $this->createMock(IOutOfOfficeData::class);
+        $data->method('getUser')->willReturn($user);
+        $davServer = $this->createMock(Server::class);
+        $invitationServer = $this->createMock(InvitationResponseServer::class);
+        $invitationServer->method('getServer')->willReturn($davServer);
+        $this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
+        $caldavPlugin = $this->createMock(Plugin::class);
+        $davServer->expects(self::once())
+            ->method('getPlugin')
+            ->with('caldav')
+            ->willReturn($caldavPlugin);
+        $caldavPlugin->expects(self::once())
+            ->method('getCalendarHomeForPrincipal')
+            ->willReturn('/home/calendar');
+        $tree = $this->createMock(Tree::class);
+        $davServer->tree = $tree;
+        $calendarHome = $this->createMock(CalendarHome::class);
+        $tree->expects(self::once())
+            ->method('getNodeForPath')
+            ->with('/home/calendar')
+            ->willReturn($calendarHome);
+        $this->appConfig->expects(self::once())
+            ->method('getUserValue')
+            ->with('user123', 'dav', 'defaultCalendar', 'personal')
+            ->willReturn('personal-1');
+        $calendarHome->expects(self::once())
+            ->method('getChild')
+            ->with('personal-1')
+            ->willThrowException(new NotFound('nope'));
+        $event = new OutOfOfficeClearedEvent($data);
+
+        $this->listener->handle($event);
+    }
+
+    public function testHandleClearRecreate(): void {
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')->willReturn('user123');
+        $data = $this->createMock(IOutOfOfficeData::class);
+        $data->method('getUser')->willReturn($user);
+        $davServer = $this->createMock(Server::class);
+        $invitationServer = $this->createMock(InvitationResponseServer::class);
+        $invitationServer->method('getServer')->willReturn($davServer);
+        $this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
+        $caldavPlugin = $this->createMock(Plugin::class);
+        $davServer->expects(self::once())
+            ->method('getPlugin')
+            ->with('caldav')
+            ->willReturn($caldavPlugin);
+        $caldavPlugin->expects(self::once())
+            ->method('getCalendarHomeForPrincipal')
+            ->willReturn('/home/calendar');
+        $tree = $this->createMock(Tree::class);
+        $davServer->tree = $tree;
+        $calendarHome = $this->createMock(CalendarHome::class);
+        $tree->expects(self::once())
+            ->method('getNodeForPath')
+            ->with('/home/calendar')
+            ->willReturn($calendarHome);
+        $this->appConfig->expects(self::once())
+            ->method('getUserValue')
+            ->with('user123', 'dav', 'defaultCalendar', 'personal')
+            ->willReturn('personal-1');
+        $calendar = $this->createMock(Calendar::class);
+        $calendarHome->expects(self::once())
+            ->method('getChild')
+            ->with('personal-1')
+            ->willReturn($calendar);
+        $calendar->expects(self::once())
+            ->method('getChild')
+            ->willThrowException(new NotFound());
+        $event = new OutOfOfficeClearedEvent($data);
+
+        $this->listener->handle($event);
+    }
+
+    public function testHandleClear(): void {
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')->willReturn('user123');
+        $data = $this->createMock(IOutOfOfficeData::class);
+        $data->method('getUser')->willReturn($user);
+        $davServer = $this->createMock(Server::class);
+        $invitationServer = $this->createMock(InvitationResponseServer::class);
+        $invitationServer->method('getServer')->willReturn($davServer);
+        $this->serverFactory->method('createInviationResponseServer')->willReturn($invitationServer);
+        $caldavPlugin = $this->createMock(Plugin::class);
+        $davServer->expects(self::once())
+            ->method('getPlugin')
+            ->with('caldav')
+            ->willReturn($caldavPlugin);
+        $caldavPlugin->expects(self::once())
+            ->method('getCalendarHomeForPrincipal')
+            ->willReturn('/home/calendar');
+        $tree = $this->createMock(Tree::class);
+        $davServer->tree = $tree;
+        $calendarHome = $this->createMock(CalendarHome::class);
+        $tree->expects(self::once())
+            ->method('getNodeForPath')
+            ->with('/home/calendar')
+            ->willReturn($calendarHome);
+        $this->appConfig->expects(self::once())
+            ->method('getUserValue')
+            ->with('user123', 'dav', 'defaultCalendar', 'personal')
+            ->willReturn('personal-1');
+        $calendar = $this->createMock(Calendar::class);
+        $calendarHome->expects(self::once())
+            ->method('getChild')
+            ->with('personal-1')
+            ->willReturn($calendar);
+        $eventNode = $this->createMock(CalendarObject::class);
+        $calendar->expects(self::once())
+            ->method('getChild')
+            ->willReturn($eventNode);
+        $eventNode->expects(self::once())
+            ->method('delete');
+        $event = new OutOfOfficeClearedEvent($data);
+
+        $this->listener->handle($event);
+    }
 }

Spacing +3 added lines, -3 removed lines

@@ -198,7 +198,7 @@ 
 			->willReturn($calendar);
 		$calendar->expects(self::once())
 			->method('createFile')
-			->willReturnCallback(function ($name, $data): void {
+			->willReturnCallback(function($name, $data): void {
 				$vcalendar = Reader::read($data);
 				if (!($vcalendar instanceof VCalendar)) {
 					throw new InvalidArgumentException('Calendar data should be a VCALENDAR');
@@ -348,7 +348,7 @@ 
 			->willThrowException(new NotFound());
 		$calendar->expects(self::once())
 			->method('createFile')
-			->willReturnCallback(function ($name, $data): void {
+			->willReturnCallback(function($name, $data): void {
 				$vcalendar = Reader::read($data);
 				if (!($vcalendar instanceof VCalendar)) {
 					throw new InvalidArgumentException('Calendar data should be a VCALENDAR');
@@ -415,7 +415,7 @@ 
 			->willReturn($eventNode);
 		$eventNode->expects(self::once())
 			->method('put')
-			->willReturnCallback(function ($data): void {
+			->willReturnCallback(function($data): void {
 				$vcalendar = Reader::read($data);
 				if (!($vcalendar instanceof VCalendar)) {
 					throw new InvalidArgumentException('Calendar data should be a VCALENDAR');