test(user_status): add tests for backup row protection in cleanup and queries

- testFindAllExcludesBackups: verify findAll() doesn't return backup rows
- testFindAllRecentExcludesBackups: verify findAllRecent() excludes backups
- testClearOlderThanClearAtPreservesBackups: verify backup rows survive
  clear_at cleanup
- testClearStatusesOlderThanPreservesBackups: verify backup statuses aren't
  overwritten to OFFLINE by age-based cleanup
- testHandleWithCorrectEvent: verify removeBackupUserStatus() is also called
  when a user is deleted

AI-Assisted-By: Claude Opus 4.6
Signed-off-by: Anna Larch <anna@nextcloud.com>

Author: miaulalala

apps/user_status/tests/Unit/Db/UserStatusMapperTest.php

@@ -47,6 +47,24 @@ public function testGetFindAll(): void {
 		$this->assertEquals('user2', $offsetResults[0]->getUserId());
 	}
 
+	public function testFindAllExcludesBackups(): void {
+		$this->insertSampleStatuses();
+
+		$backup = new UserStatus();
+		$backup->setUserId('_backupuser');
+		$backup->setStatus('dnd');
+		$backup->setStatusTimestamp(5000);
+		$backup->setIsUserDefined(true);
+		$backup->setIsBackup(true);
+		$this->mapper->insert($backup);
+
+		$allResults = $this->mapper->findAll();
+		$this->assertCount(3, $allResults);
+
+		$userIds = array_map(fn ($s) => $s->getUserId(), $allResults);
+		$this->assertNotContains('_backupuser', $userIds);
+	}
+
 	public function testFindAllRecent(): void {
 		$this->insertSampleStatuses();
 
@@ -56,6 +74,26 @@ public function testFindAllRecent(): void {
 		$this->assertEquals('user1', $allResults[1]->getUserId());
 	}
 
+	public function testFindAllRecentExcludesBackups(): void {
+		$this->insertSampleStatuses();
+
+		$backup = new UserStatus();
+		$backup->setUserId('_backupuser');
+		$backup->setStatus('dnd');
+		$backup->setStatusTimestamp(7000);
+		$backup->setStatusMessageTimestamp(7000);
+		$backup->setIsUserDefined(true);
+		$backup->setIsBackup(true);
+		$backup->setCustomMessage('Backed up status');
+		$this->mapper->insert($backup);
+
+		$allResults = $this->mapper->findAllRecent(10, 0);
+		$this->assertCount(2, $allResults);
+
+		$userIds = array_map(fn ($s) => $s->getUserId(), $allResults);
+		$this->assertNotContains('_backupuser', $userIds);
+	}
+
 	public function testGetFind(): void {
 		$this->insertSampleStatuses();
 
@@ -190,6 +228,43 @@ public function testClearOlderThanClearAt(): void {
 		$this->mapper->findByUserId('user1');
 	}
 
+	public function testClearOlderThanClearAtPreservesBackups(): void {
+		$backup = new UserStatus();
+		$backup->setUserId('_user1');
+		$backup->setStatus('dnd');
+		$backup->setStatusTimestamp(5000);
+		$backup->setIsUserDefined(true);
+		$backup->setIsBackup(true);
+		$backup->setClearAt(50000);
+		$this->mapper->insert($backup);
+
+		$this->mapper->clearOlderThanClearAt(55000);
+
+		// Backup should survive cleanup despite having expired clear_at
+		$backupStatus = $this->mapper->findByUserId('user1', true);
+		$this->assertEquals('_user1', $backupStatus->getUserId());
+		$this->assertEquals('dnd', $backupStatus->getStatus());
+		$this->assertTrue($backupStatus->getIsBackup());
+	}
+
+	public function testClearStatusesOlderThanPreservesBackups(): void {
+		$backup = new UserStatus();
+		$backup->setUserId('_user1');
+		$backup->setStatus('online');
+		$backup->setStatusTimestamp(1000);
+		$backup->setIsUserDefined(false);
+		$backup->setIsBackup(true);
+		$this->mapper->insert($backup);
+
+		$this->mapper->clearStatusesOlderThan(5000, 8000);
+
+		// Backup should survive cleanup despite old timestamp
+		$backupStatus = $this->mapper->findByUserId('user1', true);
+		$this->assertEquals('online', $backupStatus->getStatus());
+		$this->assertFalse($backupStatus->getIsUserDefined());
+		$this->assertEquals(1000, $backupStatus->getStatusTimestamp());
+	}
+
 	private function insertSampleStatuses(): void {
 		$userStatus1 = new UserStatus();
 		$userStatus1->setUserId('admin');

apps/user_status/tests/Unit/Listener/UserDeletedListenerTest.php

@@ -28,14 +28,17 @@ protected function setUp(): void {
 
 	public function testHandleWithCorrectEvent(): void {
 		$user = $this->createMock(IUser::class);
-		$user->expects($this->once())
-			->method('getUID')
+		$user->method('getUID')
 			->willReturn('john.doe');
 
 		$this->service->expects($this->once())
 			->method('removeUserStatus')
 			->with('john.doe');
 
+		$this->service->expects($this->once())
+			->method('removeBackupUserStatus')
+			->with('john.doe');
+
 		$event = new UserDeletedEvent($user);
 		$this->listener->handle($event);
 	}