nextcloud / server

apps/comments/tests/Unit/Notification/NotifierTest.php

Indentation +525 added lines, -525 removed lines

@@ -26,529 +26,529 @@
 use Test\TestCase;
 
 class NotifierTest extends TestCase {
-	protected IFactory&MockObject $l10nFactory;
-	protected IL10N&MockObject $l;
-	protected IRootFolder&MockObject $folder;
-	protected ICommentsManager&MockObject $commentsManager;
-	protected IURLGenerator&MockObject $url;
-	protected IUserManager&MockObject $userManager;
-	protected INotification&MockObject $notification;
-	protected IComment&MockObject $comment;
-	protected Notifier $notifier;
-	protected string $lc = 'tlh_KX';
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->l10nFactory = $this->createMock(IFactory::class);
-		$this->folder = $this->createMock(IRootFolder::class);
-		$this->commentsManager = $this->createMock(ICommentsManager::class);
-		$this->url = $this->createMock(IURLGenerator::class);
-		$this->userManager = $this->createMock(IUserManager::class);
-
-		$this->notifier = new Notifier(
-			$this->l10nFactory,
-			$this->folder,
-			$this->commentsManager,
-			$this->url,
-			$this->userManager
-		);
-
-		$this->l = $this->createMock(IL10N::class);
-		$this->l->expects($this->any())
-			->method('t')
-			->willReturnCallback(function ($text, $parameters = []) {
-				return vsprintf($text, $parameters);
-			});
-
-		$this->notification = $this->createMock(INotification::class);
-		$this->comment = $this->createMock(IComment::class);
-	}
-
-	public function testPrepareSuccess(): void {
-		$fileName = 'Gre\'thor.odp';
-		$displayName = 'Huraga';
-
-		/** @var Node&MockObject $node */
-		$node = $this->createMock(Node::class);
-		$node
-			->expects($this->atLeastOnce())
-			->method('getName')
-			->willReturn($fileName);
-		$node
-			->expects($this->atLeastOnce())
-			->method('getPath')
-			->willReturn('/you/files/' . $fileName);
-
-		$userFolder = $this->createMock(Folder::class);
-		$this->folder->expects($this->once())
-			->method('getUserFolder')
-			->with('you')
-			->willReturn($userFolder);
-		$userFolder->expects($this->once())
-			->method('getById')
-			->with('678')
-			->willReturn([$node]);
-
-		$this->notification->expects($this->exactly(2))
-			->method('getUser')
-			->willReturn('you');
-		$this->notification
-			->expects($this->once())
-			->method('getApp')
-			->willReturn('comments');
-		$this->notification
-			->expects($this->once())
-			->method('getSubject')
-			->willReturn('mention');
-		$this->notification
-			->expects($this->once())
-			->method('getSubjectParameters')
-			->willReturn(['files', '678']);
-		$this->notification
-			->expects($this->never())
-			->method('setParsedSubject');
-		$this->notification
-			->expects($this->once())
-			->method('setRichSubject')
-			->with('{user} mentioned you in a comment on "{file}"', $this->anything())
-			->willReturnSelf();
-		$this->notification
-			->expects($this->once())
-			->method('setRichMessage')
-			->with('Hi {mention-user1}!', ['mention-user1' => ['type' => 'user', 'id' => 'you', 'name' => 'Your name']])
-			->willReturnSelf();
-		$this->notification
-			->expects($this->never())
-			->method('setParsedMessage');
-		$this->notification
-			->expects($this->once())
-			->method('setIcon')
-			->with('absolute-image-path')
-			->willReturnSelf();
-
-		$this->url->expects($this->once())
-			->method('imagePath')
-			->with('core', 'actions/comment.svg')
-			->willReturn('image-path');
-		$this->url->expects($this->once())
-			->method('getAbsoluteURL')
-			->with('image-path')
-			->willReturn('absolute-image-path');
-
-		$this->l10nFactory
-			->expects($this->once())
-			->method('get')
-			->willReturn($this->l);
-
-		$this->comment
-			->expects($this->any())
-			->method('getActorId')
-			->willReturn('huraga');
-		$this->comment
-			->expects($this->any())
-			->method('getActorType')
-			->willReturn('users');
-		$this->comment
-			->expects($this->any())
-			->method('getMessage')
-			->willReturn('Hi @you!');
-		$this->comment
-			->expects($this->any())
-			->method('getMentions')
-			->willReturn([['type' => 'user', 'id' => 'you']]);
-		$this->comment->expects($this->atLeastOnce())
-			->method('getId')
-			->willReturn('1234');
-
-		$this->commentsManager
-			->expects($this->once())
-			->method('get')
-			->willReturn($this->comment);
-		$this->commentsManager
-			->expects($this->once())
-			->method('resolveDisplayName')
-			->with('user', 'you')
-			->willReturn('Your name');
-
-		$this->userManager
-			->expects($this->exactly(2))
-			->method('getDisplayName')
-			->willReturnMap([
-				['huraga', $displayName],
-				['you', 'You'],
-			]);
-
-		$this->notifier->prepare($this->notification, $this->lc);
-	}
-
-	public function testPrepareSuccessDeletedUser(): void {
-		$fileName = 'Gre\'thor.odp';
-
-		/** @var Node|MockObject $node */
-		$node = $this->createMock(Node::class);
-		$node
-			->expects($this->atLeastOnce())
-			->method('getName')
-			->willReturn($fileName);
-		$node
-			->expects($this->atLeastOnce())
-			->method('getPath')
-			->willReturn('/you/files/' . $fileName);
-
-		$userFolder = $this->createMock(Folder::class);
-		$this->folder->expects($this->once())
-			->method('getUserFolder')
-			->with('you')
-			->willReturn($userFolder);
-		$userFolder->expects($this->once())
-			->method('getById')
-			->with('678')
-			->willReturn([$node]);
-
-		$this->notification->expects($this->exactly(2))
-			->method('getUser')
-			->willReturn('you');
-		$this->notification
-			->expects($this->once())
-			->method('getApp')
-			->willReturn('comments');
-		$this->notification
-			->expects($this->once())
-			->method('getSubject')
-			->willReturn('mention');
-		$this->notification
-			->expects($this->once())
-			->method('getSubjectParameters')
-			->willReturn(['files', '678']);
-		$this->notification
-			->expects($this->never())
-			->method('setParsedSubject');
-		$this->notification
-			->expects($this->once())
-			->method('setRichSubject')
-			->with('You were mentioned on "{file}", in a comment by an account that has since been deleted', $this->anything())
-			->willReturnSelf();
-		$this->notification
-			->expects($this->once())
-			->method('setRichMessage')
-			->with('Hi {mention-user1}!', ['mention-user1' => ['type' => 'user', 'id' => 'you', 'name' => 'Your name']])
-			->willReturnSelf();
-		$this->notification
-			->expects($this->never())
-			->method('setParsedMessage');
-		$this->notification
-			->expects($this->once())
-			->method('setIcon')
-			->with('absolute-image-path')
-			->willReturnSelf();
-
-		$this->url->expects($this->once())
-			->method('imagePath')
-			->with('core', 'actions/comment.svg')
-			->willReturn('image-path');
-		$this->url->expects($this->once())
-			->method('getAbsoluteURL')
-			->with('image-path')
-			->willReturn('absolute-image-path');
-
-		$this->l10nFactory
-			->expects($this->once())
-			->method('get')
-			->willReturn($this->l);
-
-		$this->comment
-			->expects($this->any())
-			->method('getActorId')
-			->willReturn('huraga');
-		$this->comment
-			->expects($this->any())
-			->method('getActorType')
-			->willReturn(ICommentsManager::DELETED_USER);
-		$this->comment
-			->expects($this->any())
-			->method('getMessage')
-			->willReturn('Hi @you!');
-		$this->comment
-			->expects($this->any())
-			->method('getMentions')
-			->willReturn([['type' => 'user', 'id' => 'you']]);
-
-		$this->commentsManager
-			->expects($this->once())
-			->method('get')
-			->willReturn($this->comment);
-		$this->commentsManager
-			->expects($this->once())
-			->method('resolveDisplayName')
-			->with('user', 'you')
-			->willReturn('Your name');
-
-		$this->userManager
-			->expects($this->once())
-			->method('getDisplayName')
-			->willReturnMap([
-				['huraga', null],
-				['you', 'You'],
-			]);
-
-		$this->notifier->prepare($this->notification, $this->lc);
-	}
-
-
-	public function testPrepareDifferentApp(): void {
-		$this->expectException(UnknownNotificationException::class);
-
-		$this->folder
-			->expects($this->never())
-			->method('getById');
-
-		$this->notification
-			->expects($this->once())
-			->method('getApp')
-			->willReturn('constructions');
-		$this->notification
-			->expects($this->never())
-			->method('getSubject');
-		$this->notification
-			->expects($this->never())
-			->method('getSubjectParameters');
-		$this->notification
-			->expects($this->never())
-			->method('setParsedSubject');
-
-		$this->l10nFactory
-			->expects($this->never())
-			->method('get');
-
-		$this->commentsManager
-			->expects($this->never())
-			->method('get');
-
-		$this->userManager
-			->expects($this->never())
-			->method('getDisplayName');
-
-		$this->notifier->prepare($this->notification, $this->lc);
-	}
-
-
-	public function testPrepareNotFound(): void {
-		$this->expectException(UnknownNotificationException::class);
-
-		$this->folder
-			->expects($this->never())
-			->method('getById');
-
-		$this->notification
-			->expects($this->once())
-			->method('getApp')
-			->willReturn('comments');
-		$this->notification
-			->expects($this->never())
-			->method('getSubject');
-		$this->notification
-			->expects($this->never())
-			->method('getSubjectParameters');
-		$this->notification
-			->expects($this->never())
-			->method('setParsedSubject');
-
-		$this->l10nFactory
-			->expects($this->never())
-			->method('get');
-
-		$this->commentsManager
-			->expects($this->once())
-			->method('get')
-			->willThrowException(new NotFoundException());
-
-		$this->userManager
-			->expects($this->never())
-			->method('getDisplayName');
-
-		$this->notifier->prepare($this->notification, $this->lc);
-	}
-
-
-	public function testPrepareDifferentSubject(): void {
-		$this->expectException(UnknownNotificationException::class);
-
-		$displayName = 'Huraga';
-
-		$this->folder
-			->expects($this->never())
-			->method('getById');
-
-		$this->notification
-			->expects($this->once())
-			->method('getApp')
-			->willReturn('comments');
-		$this->notification
-			->expects($this->once())
-			->method('getSubject')
-			->willReturn('unlike');
-		$this->notification
-			->expects($this->never())
-			->method('getSubjectParameters');
-		$this->notification
-			->expects($this->never())
-			->method('setParsedSubject');
-
-		$this->l
-			->expects($this->never())
-			->method('t');
-
-		$this->l10nFactory
-			->expects($this->once())
-			->method('get')
-			->willReturn($this->l);
-
-		$this->comment
-			->expects($this->any())
-			->method('getActorId')
-			->willReturn('huraga');
-		$this->comment
-			->expects($this->any())
-			->method('getActorType')
-			->willReturn('users');
-
-		$this->commentsManager
-			->expects($this->once())
-			->method('get')
-			->willReturn($this->comment);
-
-		$this->userManager
-			->expects($this->once())
-			->method('getDisplayName')
-			->with('huraga')
-			->willReturn($displayName);
-
-		$this->notifier->prepare($this->notification, $this->lc);
-	}
-
-
-	public function testPrepareNotFiles(): void {
-		$this->expectException(UnknownNotificationException::class);
-
-		$displayName = 'Huraga';
-
-		$this->folder
-			->expects($this->never())
-			->method('getById');
-
-		$this->notification
-			->expects($this->once())
-			->method('getApp')
-			->willReturn('comments');
-		$this->notification
-			->expects($this->once())
-			->method('getSubject')
-			->willReturn('mention');
-		$this->notification
-			->expects($this->once())
-			->method('getSubjectParameters')
-			->willReturn(['ships', '678']);
-		$this->notification
-			->expects($this->never())
-			->method('setParsedSubject');
-
-		$this->l
-			->expects($this->never())
-			->method('t');
-
-		$this->l10nFactory
-			->expects($this->once())
-			->method('get')
-			->willReturn($this->l);
-
-		$this->comment
-			->expects($this->any())
-			->method('getActorId')
-			->willReturn('huraga');
-		$this->comment
-			->expects($this->any())
-			->method('getActorType')
-			->willReturn('users');
-
-		$this->commentsManager
-			->expects($this->once())
-			->method('get')
-			->willReturn($this->comment);
-
-		$this->userManager
-			->expects($this->once())
-			->method('getDisplayName')
-			->with('huraga')
-			->willReturn($displayName);
-
-		$this->notifier->prepare($this->notification, $this->lc);
-	}
-
-
-	public function testPrepareUnresolvableFileID(): void {
-		$this->expectException(AlreadyProcessedException::class);
-
-		$displayName = 'Huraga';
-
-		$userFolder = $this->createMock(Folder::class);
-		$this->folder->expects($this->once())
-			->method('getUserFolder')
-			->with('you')
-			->willReturn($userFolder);
-		$userFolder->expects($this->once())
-			->method('getById')
-			->with('678')
-			->willReturn([]);
-
-		$this->notification->expects($this->once())
-			->method('getUser')
-			->willReturn('you');
-		$this->notification
-			->expects($this->once())
-			->method('getApp')
-			->willReturn('comments');
-		$this->notification
-			->expects($this->once())
-			->method('getSubject')
-			->willReturn('mention');
-		$this->notification
-			->expects($this->once())
-			->method('getSubjectParameters')
-			->willReturn(['files', '678']);
-		$this->notification
-			->expects($this->never())
-			->method('setParsedSubject');
-
-		$this->l
-			->expects($this->never())
-			->method('t');
-
-		$this->l10nFactory
-			->expects($this->once())
-			->method('get')
-			->willReturn($this->l);
-
-		$this->comment
-			->expects($this->any())
-			->method('getActorId')
-			->willReturn('huraga');
-		$this->comment
-			->expects($this->any())
-			->method('getActorType')
-			->willReturn('users');
-
-		$this->commentsManager
-			->expects($this->once())
-			->method('get')
-			->willReturn($this->comment);
-
-		$this->userManager
-			->expects($this->once())
-			->method('getDisplayName')
-			->with('huraga')
-			->willReturn($displayName);
-
-		$this->notifier->prepare($this->notification, $this->lc);
-	}
+    protected IFactory&MockObject $l10nFactory;
+    protected IL10N&MockObject $l;
+    protected IRootFolder&MockObject $folder;
+    protected ICommentsManager&MockObject $commentsManager;
+    protected IURLGenerator&MockObject $url;
+    protected IUserManager&MockObject $userManager;
+    protected INotification&MockObject $notification;
+    protected IComment&MockObject $comment;
+    protected Notifier $notifier;
+    protected string $lc = 'tlh_KX';
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->l10nFactory = $this->createMock(IFactory::class);
+        $this->folder = $this->createMock(IRootFolder::class);
+        $this->commentsManager = $this->createMock(ICommentsManager::class);
+        $this->url = $this->createMock(IURLGenerator::class);
+        $this->userManager = $this->createMock(IUserManager::class);
+
+        $this->notifier = new Notifier(
+            $this->l10nFactory,
+            $this->folder,
+            $this->commentsManager,
+            $this->url,
+            $this->userManager
+        );
+
+        $this->l = $this->createMock(IL10N::class);
+        $this->l->expects($this->any())
+            ->method('t')
+            ->willReturnCallback(function ($text, $parameters = []) {
+                return vsprintf($text, $parameters);
+            });
+
+        $this->notification = $this->createMock(INotification::class);
+        $this->comment = $this->createMock(IComment::class);
+    }
+
+    public function testPrepareSuccess(): void {
+        $fileName = 'Gre\'thor.odp';
+        $displayName = 'Huraga';
+
+        /** @var Node&MockObject $node */
+        $node = $this->createMock(Node::class);
+        $node
+            ->expects($this->atLeastOnce())
+            ->method('getName')
+            ->willReturn($fileName);
+        $node
+            ->expects($this->atLeastOnce())
+            ->method('getPath')
+            ->willReturn('/you/files/' . $fileName);
+
+        $userFolder = $this->createMock(Folder::class);
+        $this->folder->expects($this->once())
+            ->method('getUserFolder')
+            ->with('you')
+            ->willReturn($userFolder);
+        $userFolder->expects($this->once())
+            ->method('getById')
+            ->with('678')
+            ->willReturn([$node]);
+
+        $this->notification->expects($this->exactly(2))
+            ->method('getUser')
+            ->willReturn('you');
+        $this->notification
+            ->expects($this->once())
+            ->method('getApp')
+            ->willReturn('comments');
+        $this->notification
+            ->expects($this->once())
+            ->method('getSubject')
+            ->willReturn('mention');
+        $this->notification
+            ->expects($this->once())
+            ->method('getSubjectParameters')
+            ->willReturn(['files', '678']);
+        $this->notification
+            ->expects($this->never())
+            ->method('setParsedSubject');
+        $this->notification
+            ->expects($this->once())
+            ->method('setRichSubject')
+            ->with('{user} mentioned you in a comment on "{file}"', $this->anything())
+            ->willReturnSelf();
+        $this->notification
+            ->expects($this->once())
+            ->method('setRichMessage')
+            ->with('Hi {mention-user1}!', ['mention-user1' => ['type' => 'user', 'id' => 'you', 'name' => 'Your name']])
+            ->willReturnSelf();
+        $this->notification
+            ->expects($this->never())
+            ->method('setParsedMessage');
+        $this->notification
+            ->expects($this->once())
+            ->method('setIcon')
+            ->with('absolute-image-path')
+            ->willReturnSelf();
+
+        $this->url->expects($this->once())
+            ->method('imagePath')
+            ->with('core', 'actions/comment.svg')
+            ->willReturn('image-path');
+        $this->url->expects($this->once())
+            ->method('getAbsoluteURL')
+            ->with('image-path')
+            ->willReturn('absolute-image-path');
+
+        $this->l10nFactory
+            ->expects($this->once())
+            ->method('get')
+            ->willReturn($this->l);
+
+        $this->comment
+            ->expects($this->any())
+            ->method('getActorId')
+            ->willReturn('huraga');
+        $this->comment
+            ->expects($this->any())
+            ->method('getActorType')
+            ->willReturn('users');
+        $this->comment
+            ->expects($this->any())
+            ->method('getMessage')
+            ->willReturn('Hi @you!');
+        $this->comment
+            ->expects($this->any())
+            ->method('getMentions')
+            ->willReturn([['type' => 'user', 'id' => 'you']]);
+        $this->comment->expects($this->atLeastOnce())
+            ->method('getId')
+            ->willReturn('1234');
+
+        $this->commentsManager
+            ->expects($this->once())
+            ->method('get')
+            ->willReturn($this->comment);
+        $this->commentsManager
+            ->expects($this->once())
+            ->method('resolveDisplayName')
+            ->with('user', 'you')
+            ->willReturn('Your name');
+
+        $this->userManager
+            ->expects($this->exactly(2))
+            ->method('getDisplayName')
+            ->willReturnMap([
+                ['huraga', $displayName],
+                ['you', 'You'],
+            ]);
+
+        $this->notifier->prepare($this->notification, $this->lc);
+    }
+
+    public function testPrepareSuccessDeletedUser(): void {
+        $fileName = 'Gre\'thor.odp';
+
+        /** @var Node|MockObject $node */
+        $node = $this->createMock(Node::class);
+        $node
+            ->expects($this->atLeastOnce())
+            ->method('getName')
+            ->willReturn($fileName);
+        $node
+            ->expects($this->atLeastOnce())
+            ->method('getPath')
+            ->willReturn('/you/files/' . $fileName);
+
+        $userFolder = $this->createMock(Folder::class);
+        $this->folder->expects($this->once())
+            ->method('getUserFolder')
+            ->with('you')
+            ->willReturn($userFolder);
+        $userFolder->expects($this->once())
+            ->method('getById')
+            ->with('678')
+            ->willReturn([$node]);
+
+        $this->notification->expects($this->exactly(2))
+            ->method('getUser')
+            ->willReturn('you');
+        $this->notification
+            ->expects($this->once())
+            ->method('getApp')
+            ->willReturn('comments');
+        $this->notification
+            ->expects($this->once())
+            ->method('getSubject')
+            ->willReturn('mention');
+        $this->notification
+            ->expects($this->once())
+            ->method('getSubjectParameters')
+            ->willReturn(['files', '678']);
+        $this->notification
+            ->expects($this->never())
+            ->method('setParsedSubject');
+        $this->notification
+            ->expects($this->once())
+            ->method('setRichSubject')
+            ->with('You were mentioned on "{file}", in a comment by an account that has since been deleted', $this->anything())
+            ->willReturnSelf();
+        $this->notification
+            ->expects($this->once())
+            ->method('setRichMessage')
+            ->with('Hi {mention-user1}!', ['mention-user1' => ['type' => 'user', 'id' => 'you', 'name' => 'Your name']])
+            ->willReturnSelf();
+        $this->notification
+            ->expects($this->never())
+            ->method('setParsedMessage');
+        $this->notification
+            ->expects($this->once())
+            ->method('setIcon')
+            ->with('absolute-image-path')
+            ->willReturnSelf();
+
+        $this->url->expects($this->once())
+            ->method('imagePath')
+            ->with('core', 'actions/comment.svg')
+            ->willReturn('image-path');
+        $this->url->expects($this->once())
+            ->method('getAbsoluteURL')
+            ->with('image-path')
+            ->willReturn('absolute-image-path');
+
+        $this->l10nFactory
+            ->expects($this->once())
+            ->method('get')
+            ->willReturn($this->l);
+
+        $this->comment
+            ->expects($this->any())
+            ->method('getActorId')
+            ->willReturn('huraga');
+        $this->comment
+            ->expects($this->any())
+            ->method('getActorType')
+            ->willReturn(ICommentsManager::DELETED_USER);
+        $this->comment
+            ->expects($this->any())
+            ->method('getMessage')
+            ->willReturn('Hi @you!');
+        $this->comment
+            ->expects($this->any())
+            ->method('getMentions')
+            ->willReturn([['type' => 'user', 'id' => 'you']]);
+
+        $this->commentsManager
+            ->expects($this->once())
+            ->method('get')
+            ->willReturn($this->comment);
+        $this->commentsManager
+            ->expects($this->once())
+            ->method('resolveDisplayName')
+            ->with('user', 'you')
+            ->willReturn('Your name');
+
+        $this->userManager
+            ->expects($this->once())
+            ->method('getDisplayName')
+            ->willReturnMap([
+                ['huraga', null],
+                ['you', 'You'],
+            ]);
+
+        $this->notifier->prepare($this->notification, $this->lc);
+    }
+
+
+    public function testPrepareDifferentApp(): void {
+        $this->expectException(UnknownNotificationException::class);
+
+        $this->folder
+            ->expects($this->never())
+            ->method('getById');
+
+        $this->notification
+            ->expects($this->once())
+            ->method('getApp')
+            ->willReturn('constructions');
+        $this->notification
+            ->expects($this->never())
+            ->method('getSubject');
+        $this->notification
+            ->expects($this->never())
+            ->method('getSubjectParameters');
+        $this->notification
+            ->expects($this->never())
+            ->method('setParsedSubject');
+
+        $this->l10nFactory
+            ->expects($this->never())
+            ->method('get');
+
+        $this->commentsManager
+            ->expects($this->never())
+            ->method('get');
+
+        $this->userManager
+            ->expects($this->never())
+            ->method('getDisplayName');
+
+        $this->notifier->prepare($this->notification, $this->lc);
+    }
+
+
+    public function testPrepareNotFound(): void {
+        $this->expectException(UnknownNotificationException::class);
+
+        $this->folder
+            ->expects($this->never())
+            ->method('getById');
+
+        $this->notification
+            ->expects($this->once())
+            ->method('getApp')
+            ->willReturn('comments');
+        $this->notification
+            ->expects($this->never())
+            ->method('getSubject');
+        $this->notification
+            ->expects($this->never())
+            ->method('getSubjectParameters');
+        $this->notification
+            ->expects($this->never())
+            ->method('setParsedSubject');
+
+        $this->l10nFactory
+            ->expects($this->never())
+            ->method('get');
+
+        $this->commentsManager
+            ->expects($this->once())
+            ->method('get')
+            ->willThrowException(new NotFoundException());
+
+        $this->userManager
+            ->expects($this->never())
+            ->method('getDisplayName');
+
+        $this->notifier->prepare($this->notification, $this->lc);
+    }
+
+
+    public function testPrepareDifferentSubject(): void {
+        $this->expectException(UnknownNotificationException::class);
+
+        $displayName = 'Huraga';
+
+        $this->folder
+            ->expects($this->never())
+            ->method('getById');
+
+        $this->notification
+            ->expects($this->once())
+            ->method('getApp')
+            ->willReturn('comments');
+        $this->notification
+            ->expects($this->once())
+            ->method('getSubject')
+            ->willReturn('unlike');
+        $this->notification
+            ->expects($this->never())
+            ->method('getSubjectParameters');
+        $this->notification
+            ->expects($this->never())
+            ->method('setParsedSubject');
+
+        $this->l
+            ->expects($this->never())
+            ->method('t');
+
+        $this->l10nFactory
+            ->expects($this->once())
+            ->method('get')
+            ->willReturn($this->l);
+
+        $this->comment
+            ->expects($this->any())
+            ->method('getActorId')
+            ->willReturn('huraga');
+        $this->comment
+            ->expects($this->any())
+            ->method('getActorType')
+            ->willReturn('users');
+
+        $this->commentsManager
+            ->expects($this->once())
+            ->method('get')
+            ->willReturn($this->comment);
+
+        $this->userManager
+            ->expects($this->once())
+            ->method('getDisplayName')
+            ->with('huraga')
+            ->willReturn($displayName);
+
+        $this->notifier->prepare($this->notification, $this->lc);
+    }
+
+
+    public function testPrepareNotFiles(): void {
+        $this->expectException(UnknownNotificationException::class);
+
+        $displayName = 'Huraga';
+
+        $this->folder
+            ->expects($this->never())
+            ->method('getById');
+
+        $this->notification
+            ->expects($this->once())
+            ->method('getApp')
+            ->willReturn('comments');
+        $this->notification
+            ->expects($this->once())
+            ->method('getSubject')
+            ->willReturn('mention');
+        $this->notification
+            ->expects($this->once())
+            ->method('getSubjectParameters')
+            ->willReturn(['ships', '678']);
+        $this->notification
+            ->expects($this->never())
+            ->method('setParsedSubject');
+
+        $this->l
+            ->expects($this->never())
+            ->method('t');
+
+        $this->l10nFactory
+            ->expects($this->once())
+            ->method('get')
+            ->willReturn($this->l);
+
+        $this->comment
+            ->expects($this->any())
+            ->method('getActorId')
+            ->willReturn('huraga');
+        $this->comment
+            ->expects($this->any())
+            ->method('getActorType')
+            ->willReturn('users');
+
+        $this->commentsManager
+            ->expects($this->once())
+            ->method('get')
+            ->willReturn($this->comment);
+
+        $this->userManager
+            ->expects($this->once())
+            ->method('getDisplayName')
+            ->with('huraga')
+            ->willReturn($displayName);
+
+        $this->notifier->prepare($this->notification, $this->lc);
+    }
+
+
+    public function testPrepareUnresolvableFileID(): void {
+        $this->expectException(AlreadyProcessedException::class);
+
+        $displayName = 'Huraga';
+
+        $userFolder = $this->createMock(Folder::class);
+        $this->folder->expects($this->once())
+            ->method('getUserFolder')
+            ->with('you')
+            ->willReturn($userFolder);
+        $userFolder->expects($this->once())
+            ->method('getById')
+            ->with('678')
+            ->willReturn([]);
+
+        $this->notification->expects($this->once())
+            ->method('getUser')
+            ->willReturn('you');
+        $this->notification
+            ->expects($this->once())
+            ->method('getApp')
+            ->willReturn('comments');
+        $this->notification
+            ->expects($this->once())
+            ->method('getSubject')
+            ->willReturn('mention');
+        $this->notification
+            ->expects($this->once())
+            ->method('getSubjectParameters')
+            ->willReturn(['files', '678']);
+        $this->notification
+            ->expects($this->never())
+            ->method('setParsedSubject');
+
+        $this->l
+            ->expects($this->never())
+            ->method('t');
+
+        $this->l10nFactory
+            ->expects($this->once())
+            ->method('get')
+            ->willReturn($this->l);
+
+        $this->comment
+            ->expects($this->any())
+            ->method('getActorId')
+            ->willReturn('huraga');
+        $this->comment
+            ->expects($this->any())
+            ->method('getActorType')
+            ->willReturn('users');
+
+        $this->commentsManager
+            ->expects($this->once())
+            ->method('get')
+            ->willReturn($this->comment);
+
+        $this->userManager
+            ->expects($this->once())
+            ->method('getDisplayName')
+            ->with('huraga')
+            ->willReturn($displayName);
+
+        $this->notifier->prepare($this->notification, $this->lc);
+    }
 }

apps/comments/tests/Unit/Activity/ListenerTest.php

Indentation +129 added lines, -129 removed lines

@@ -27,133 +27,133 @@
 use Test\TestCase;
 
 class ListenerTest extends TestCase {
-	protected IManager&MockObject $activityManager;
-	protected IUserSession&MockObject $session;
-	protected IAppManager&MockObject $appManager;
-	protected IMountProviderCollection&MockObject $mountProviderCollection;
-	protected IRootFolder&MockObject $rootFolder;
-	protected IShareHelper&MockObject $shareHelper;
-	protected Listener $listener;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->activityManager = $this->createMock(IManager::class);
-		$this->session = $this->createMock(IUserSession::class);
-		$this->appManager = $this->createMock(IAppManager::class);
-		$this->mountProviderCollection = $this->createMock(IMountProviderCollection::class);
-		$this->rootFolder = $this->createMock(IRootFolder::class);
-		$this->shareHelper = $this->createMock(IShareHelper::class);
-
-		$this->listener = new Listener(
-			$this->activityManager,
-			$this->session,
-			$this->appManager,
-			$this->mountProviderCollection,
-			$this->rootFolder,
-			$this->shareHelper
-		);
-	}
-
-	public function testCommentEvent(): void {
-		$this->appManager->expects($this->any())
-			->method('isEnabledForAnyone')
-			->with('activity')
-			->willReturn(true);
-
-		$comment = $this->createMock(IComment::class);
-		$comment->expects($this->any())
-			->method('getObjectType')
-			->willReturn('files');
-
-		/** @var CommentsEvent|MockObject $event */
-		$event = $this->createMock(CommentsEvent::class);
-		$event->expects($this->any())
-			->method('getComment')
-			->willReturn($comment);
-		$event->expects($this->any())
-			->method('getEvent')
-			->willReturn(CommentsEvent::EVENT_ADD);
-
-		/** @var IUser|MockObject $ownerUser */
-		$ownerUser = $this->createMock(IUser::class);
-		$ownerUser->expects($this->any())
-			->method('getUID')
-			->willReturn('937393');
-
-		/** @var MockObject $mount */
-		$mount = $this->createMock(ICachedMountFileInfo::class);
-		$mount->expects($this->any())
-			->method('getUser')
-			->willReturn($ownerUser); // perhaps not the right user, but does not matter in this scenario
-
-		$mounts = [ $mount, $mount ]; // to make sure duplicates are dealt with
-
-		$userMountCache = $this->createMock(IUserMountCache::class);
-		$userMountCache->expects($this->any())
-			->method('getMountsForFileId')
-			->willReturn($mounts);
-
-		$this->mountProviderCollection->expects($this->any())
-			->method('getMountCache')
-			->willReturn($userMountCache);
-
-		$node = $this->createMock(Node::class);
-		$nodes = [ $node ];
-
-		$ownerFolder = $this->createMock(Folder::class);
-		$ownerFolder->expects($this->any())
-			->method('getById')
-			->willReturn($nodes);
-
-		$this->rootFolder->expects($this->any())
-			->method('getUserFolder')
-			->willReturn($ownerFolder);
-
-		$al = [ 'users' => [
-			'873304' => 'i/got/it/here',
-			'254342' => 'there/i/have/it',
-			'sandra' => 'and/here/i/placed/it'
-		]];
-		$this->shareHelper->expects($this->any())
-			->method('getPathsForAccessList')
-			->willReturn($al);
-
-		$this->session->expects($this->any())
-			->method('getUser')
-			->willReturn($ownerUser);
-
-		/** @var MockObject $activity */
-		$activity = $this->createMock(IEvent::class);
-		$activity->expects($this->exactly(count($al['users'])))
-			->method('setAffectedUser');
-		$activity->expects($this->once())
-			->method('setApp')
-			->with('comments')
-			->willReturnSelf();
-		$activity->expects($this->once())
-			->method('setType')
-			->with('comments')
-			->willReturnSelf();
-		$activity->expects($this->once())
-			->method('setAuthor')
-			->with($ownerUser->getUID())
-			->willReturnSelf();
-		$activity->expects($this->once())
-			->method('setObject')
-			->with('files', $this->anything())
-			->willReturnSelf();
-		$activity->expects($this->once())
-			->method('setMessage')
-			->with('add_comment_message', $this->anything())
-			->willReturnSelf();
-
-		$this->activityManager->expects($this->once())
-			->method('generateEvent')
-			->willReturn($activity);
-		$this->activityManager->expects($this->exactly(count($al['users'])))
-			->method('publish');
-
-		$this->listener->commentEvent($event);
-	}
+    protected IManager&MockObject $activityManager;
+    protected IUserSession&MockObject $session;
+    protected IAppManager&MockObject $appManager;
+    protected IMountProviderCollection&MockObject $mountProviderCollection;
+    protected IRootFolder&MockObject $rootFolder;
+    protected IShareHelper&MockObject $shareHelper;
+    protected Listener $listener;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->activityManager = $this->createMock(IManager::class);
+        $this->session = $this->createMock(IUserSession::class);
+        $this->appManager = $this->createMock(IAppManager::class);
+        $this->mountProviderCollection = $this->createMock(IMountProviderCollection::class);
+        $this->rootFolder = $this->createMock(IRootFolder::class);
+        $this->shareHelper = $this->createMock(IShareHelper::class);
+
+        $this->listener = new Listener(
+            $this->activityManager,
+            $this->session,
+            $this->appManager,
+            $this->mountProviderCollection,
+            $this->rootFolder,
+            $this->shareHelper
+        );
+    }
+
+    public function testCommentEvent(): void {
+        $this->appManager->expects($this->any())
+            ->method('isEnabledForAnyone')
+            ->with('activity')
+            ->willReturn(true);
+
+        $comment = $this->createMock(IComment::class);
+        $comment->expects($this->any())
+            ->method('getObjectType')
+            ->willReturn('files');
+
+        /** @var CommentsEvent|MockObject $event */
+        $event = $this->createMock(CommentsEvent::class);
+        $event->expects($this->any())
+            ->method('getComment')
+            ->willReturn($comment);
+        $event->expects($this->any())
+            ->method('getEvent')
+            ->willReturn(CommentsEvent::EVENT_ADD);
+
+        /** @var IUser|MockObject $ownerUser */
+        $ownerUser = $this->createMock(IUser::class);
+        $ownerUser->expects($this->any())
+            ->method('getUID')
+            ->willReturn('937393');
+
+        /** @var MockObject $mount */
+        $mount = $this->createMock(ICachedMountFileInfo::class);
+        $mount->expects($this->any())
+            ->method('getUser')
+            ->willReturn($ownerUser); // perhaps not the right user, but does not matter in this scenario
+
+        $mounts = [ $mount, $mount ]; // to make sure duplicates are dealt with
+
+        $userMountCache = $this->createMock(IUserMountCache::class);
+        $userMountCache->expects($this->any())
+            ->method('getMountsForFileId')
+            ->willReturn($mounts);
+
+        $this->mountProviderCollection->expects($this->any())
+            ->method('getMountCache')
+            ->willReturn($userMountCache);
+
+        $node = $this->createMock(Node::class);
+        $nodes = [ $node ];
+
+        $ownerFolder = $this->createMock(Folder::class);
+        $ownerFolder->expects($this->any())
+            ->method('getById')
+            ->willReturn($nodes);
+
+        $this->rootFolder->expects($this->any())
+            ->method('getUserFolder')
+            ->willReturn($ownerFolder);
+
+        $al = [ 'users' => [
+            '873304' => 'i/got/it/here',
+            '254342' => 'there/i/have/it',
+            'sandra' => 'and/here/i/placed/it'
+        ]];
+        $this->shareHelper->expects($this->any())
+            ->method('getPathsForAccessList')
+            ->willReturn($al);
+
+        $this->session->expects($this->any())
+            ->method('getUser')
+            ->willReturn($ownerUser);
+
+        /** @var MockObject $activity */
+        $activity = $this->createMock(IEvent::class);
+        $activity->expects($this->exactly(count($al['users'])))
+            ->method('setAffectedUser');
+        $activity->expects($this->once())
+            ->method('setApp')
+            ->with('comments')
+            ->willReturnSelf();
+        $activity->expects($this->once())
+            ->method('setType')
+            ->with('comments')
+            ->willReturnSelf();
+        $activity->expects($this->once())
+            ->method('setAuthor')
+            ->with($ownerUser->getUID())
+            ->willReturnSelf();
+        $activity->expects($this->once())
+            ->method('setObject')
+            ->with('files', $this->anything())
+            ->willReturnSelf();
+        $activity->expects($this->once())
+            ->method('setMessage')
+            ->with('add_comment_message', $this->anything())
+            ->willReturnSelf();
+
+        $this->activityManager->expects($this->once())
+            ->method('generateEvent')
+            ->willReturn($activity);
+        $this->activityManager->expects($this->exactly(count($al['users'])))
+            ->method('publish');
+
+        $this->listener->commentEvent($event);
+    }
 }

apps/comments/tests/Unit/EventHandlerTest.php

Indentation +69 added lines, -69 removed lines

@@ -17,73 +17,73 @@
 use Test\TestCase;
 
 class EventHandlerTest extends TestCase {
-	protected ActivityListener&MockObject $activityListener;
-	protected NotificationListener&MockObject $notificationListener;
-	protected CommentsEventListener $eventHandler;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->activityListener = $this->createMock(ActivityListener::class);
-		$this->notificationListener = $this->createMock(NotificationListener::class);
-
-		$this->eventHandler = new CommentsEventListener($this->activityListener, $this->notificationListener);
-	}
-
-	public function testNotFiles(): void {
-		/** @var IComment|MockObject $comment */
-		$comment = $this->createMock(IComment::class);
-		$comment->expects($this->once())
-			->method('getObjectType')
-			->willReturn('smiles');
-
-		/** @var CommentsEvent|MockObject $event */
-		$event = $this->createMock(CommentsEvent::class);
-		$event->expects($this->once())
-			->method('getComment')
-			->willReturn($comment);
-		$event->expects($this->never())
-			->method('getEvent');
-
-		$this->eventHandler->handle($event);
-	}
-
-	public static function handledProvider(): array {
-		return [
-			[CommentsEvent::EVENT_DELETE],
-			[CommentsEvent::EVENT_UPDATE],
-			[CommentsEvent::EVENT_PRE_UPDATE],
-			[CommentsEvent::EVENT_ADD]
-		];
-	}
-
-	/**
-	 * @dataProvider handledProvider
-	 */
-	public function testHandled(string $eventType): void {
-		/** @var IComment|MockObject $comment */
-		$comment = $this->createMock(IComment::class);
-		$comment->expects($this->once())
-			->method('getObjectType')
-			->willReturn('files');
-
-		/** @var CommentsEvent|MockObject $event */
-		$event = $this->createMock(CommentsEvent::class);
-		$event->expects($this->atLeastOnce())
-			->method('getComment')
-			->willReturn($comment);
-		$event->expects($this->atLeastOnce())
-			->method('getEvent')
-			->willReturn($eventType);
-
-		$this->notificationListener->expects($this->once())
-			->method('evaluate')
-			->with($event);
-
-		$this->activityListener->expects($this->any())
-			->method('commentEvent')
-			->with($event);
-
-		$this->eventHandler->handle($event);
-	}
+    protected ActivityListener&MockObject $activityListener;
+    protected NotificationListener&MockObject $notificationListener;
+    protected CommentsEventListener $eventHandler;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->activityListener = $this->createMock(ActivityListener::class);
+        $this->notificationListener = $this->createMock(NotificationListener::class);
+
+        $this->eventHandler = new CommentsEventListener($this->activityListener, $this->notificationListener);
+    }
+
+    public function testNotFiles(): void {
+        /** @var IComment|MockObject $comment */
+        $comment = $this->createMock(IComment::class);
+        $comment->expects($this->once())
+            ->method('getObjectType')
+            ->willReturn('smiles');
+
+        /** @var CommentsEvent|MockObject $event */
+        $event = $this->createMock(CommentsEvent::class);
+        $event->expects($this->once())
+            ->method('getComment')
+            ->willReturn($comment);
+        $event->expects($this->never())
+            ->method('getEvent');
+
+        $this->eventHandler->handle($event);
+    }
+
+    public static function handledProvider(): array {
+        return [
+            [CommentsEvent::EVENT_DELETE],
+            [CommentsEvent::EVENT_UPDATE],
+            [CommentsEvent::EVENT_PRE_UPDATE],
+            [CommentsEvent::EVENT_ADD]
+        ];
+    }
+
+    /**
+     * @dataProvider handledProvider
+     */
+    public function testHandled(string $eventType): void {
+        /** @var IComment|MockObject $comment */
+        $comment = $this->createMock(IComment::class);
+        $comment->expects($this->once())
+            ->method('getObjectType')
+            ->willReturn('files');
+
+        /** @var CommentsEvent|MockObject $event */
+        $event = $this->createMock(CommentsEvent::class);
+        $event->expects($this->atLeastOnce())
+            ->method('getComment')
+            ->willReturn($comment);
+        $event->expects($this->atLeastOnce())
+            ->method('getEvent')
+            ->willReturn($eventType);
+
+        $this->notificationListener->expects($this->once())
+            ->method('evaluate')
+            ->with($event);
+
+        $this->activityListener->expects($this->any())
+            ->method('commentEvent')
+            ->with($event);
+
+        $this->eventHandler->handle($event);
+    }
 }

apps/contactsinteraction/tests/Db/RecentContactMapperTest.php

Indentation +68 added lines, -68 removed lines

@@ -21,95 +21,95 @@
  * @group DB
  */
 class RecentContactMapperTest extends TestCase {
-	private RecentContactMapper $recentContactMapper;
-	private ITimeFactory $time;
+    private RecentContactMapper $recentContactMapper;
+    private ITimeFactory $time;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->recentContactMapper = Server::get(RecentContactMapper::class);
-		$this->time = Server::get(ITimeFactory::class);
-	}
+        $this->recentContactMapper = Server::get(RecentContactMapper::class);
+        $this->time = Server::get(ITimeFactory::class);
+    }
 
-	protected function tearDown(): void {
-		parent::tearDown();
+    protected function tearDown(): void {
+        parent::tearDown();
 
-		$this->recentContactMapper->cleanUp(time());
-	}
+        $this->recentContactMapper->cleanUp(time());
+    }
 
-	public function testCreateRecentContact(): void {
-		$contact = $this->createRecentContact();
-		$this->assertNotNull($contact->getId());
-	}
+    public function testCreateRecentContact(): void {
+        $contact = $this->createRecentContact();
+        $this->assertNotNull($contact->getId());
+    }
 
-	public function testFindAll(): void {
-		$this->createRecentContact();
-		$this->createRecentContact();
+    public function testFindAll(): void {
+        $this->createRecentContact();
+        $this->createRecentContact();
 
-		$contacts = $this->recentContactMapper->findAll('admin');
-		$this->assertCount(2, $contacts);
-	}
+        $contacts = $this->recentContactMapper->findAll('admin');
+        $this->assertCount(2, $contacts);
+    }
 
-	public function testFindMatchByEmail(): void {
-		$this->createRecentContact();
-		$contact = $this->createRecentContact('foo@bar');
+    public function testFindMatchByEmail(): void {
+        $this->createRecentContact();
+        $contact = $this->createRecentContact('foo@bar');
 
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')
-			->willReturn('admin');
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')
+            ->willReturn('admin');
 
-		$result = $this->recentContactMapper->findMatch($user, null, 'foo@bar', null);
+        $result = $this->recentContactMapper->findMatch($user, null, 'foo@bar', null);
 
-		$this->assertCount(1, $result);
-		$this->assertEquals($contact->getId(), $result[0]->getId());
-	}
+        $this->assertCount(1, $result);
+        $this->assertEquals($contact->getId(), $result[0]->getId());
+    }
 
-	public function testFindMatchByFederatedCloudId(): void {
-		$this->createRecentContact();
-		$contact = $this->createRecentContact(null, 'foo.bar');
+    public function testFindMatchByFederatedCloudId(): void {
+        $this->createRecentContact();
+        $contact = $this->createRecentContact(null, 'foo.bar');
 
-		$user = $this->createMock(IUser::class);
-		$user->method('getUID')
-			->willReturn('admin');
+        $user = $this->createMock(IUser::class);
+        $user->method('getUID')
+            ->willReturn('admin');
 
-		$result = $this->recentContactMapper->findMatch($user, null, null, 'foo.bar');
+        $result = $this->recentContactMapper->findMatch($user, null, null, 'foo.bar');
 
-		$this->assertCount(1, $result);
-		$this->assertEquals($contact->getId(), $result[0]->getId());
-	}
+        $this->assertCount(1, $result);
+        $this->assertEquals($contact->getId(), $result[0]->getId());
+    }
 
-	public function testCleanUp(): void {
-		$this->createRecentContact();
-		$this->createRecentContact();
-		$this->assertCount(2, $this->recentContactMapper->findAll('admin'));
+    public function testCleanUp(): void {
+        $this->createRecentContact();
+        $this->createRecentContact();
+        $this->assertCount(2, $this->recentContactMapper->findAll('admin'));
 
-		$this->recentContactMapper->cleanUp(time());
-		$this->assertCount(0, $this->recentContactMapper->findAll('admin'));
-	}
+        $this->recentContactMapper->cleanUp(time());
+        $this->assertCount(0, $this->recentContactMapper->findAll('admin'));
+    }
 
-	protected function createRecentContact(?string $email = null, ?string $federatedCloudId = null): RecentContact {
-		$props = [
-			'URI' => UUIDUtil::getUUID(),
-			'FN' => 'Foo Bar',
-			'CATEGORIES' => 'Recently contacted',
-		];
+    protected function createRecentContact(?string $email = null, ?string $federatedCloudId = null): RecentContact {
+        $props = [
+            'URI' => UUIDUtil::getUUID(),
+            'FN' => 'Foo Bar',
+            'CATEGORIES' => 'Recently contacted',
+        ];
 
-		$time = $this->time->getDateTime();
-		$time->modify('-14days');
+        $time = $this->time->getDateTime();
+        $time->modify('-14days');
 
-		$contact = new RecentContact();
-		$contact->setActorUid('admin');
-		$contact->setCard((new VCard($props))->serialize());
-		$contact->setLastContact($time->getTimestamp());
+        $contact = new RecentContact();
+        $contact->setActorUid('admin');
+        $contact->setCard((new VCard($props))->serialize());
+        $contact->setLastContact($time->getTimestamp());
 
-		if ($email !== null) {
-			$contact->setEmail($email);
-		}
+        if ($email !== null) {
+            $contact->setEmail($email);
+        }
 
-		if ($federatedCloudId !== null) {
-			$contact->setFederatedCloudId($federatedCloudId);
-		}
+        if ($federatedCloudId !== null) {
+            $contact->setFederatedCloudId($federatedCloudId);
+        }
 
-		return $this->recentContactMapper->insert($contact);
-	}
+        return $this->recentContactMapper->insert($contact);
+    }
 }

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

Indentation +318 added lines, -318 removed lines

@@ -15,322 +15,322 @@
 use Test\TestCase;
 
 class UserStatusMapperTest extends TestCase {
-	private UserStatusMapper $mapper;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		// make sure that DB is empty
-		$qb = self::$realDatabase->getQueryBuilder();
-		$qb->delete('user_status')->execute();
-
-		$this->mapper = new UserStatusMapper(self::$realDatabase);
-	}
-
-	public function testGetTableName(): void {
-		$this->assertEquals('user_status', $this->mapper->getTableName());
-	}
-
-	public function testGetFindAll(): void {
-		$this->insertSampleStatuses();
-
-		$allResults = $this->mapper->findAll();
-		$this->assertCount(3, $allResults);
-
-		$limitedResults = $this->mapper->findAll(2);
-		$this->assertCount(2, $limitedResults);
-		$this->assertEquals('admin', $limitedResults[0]->getUserId());
-		$this->assertEquals('user1', $limitedResults[1]->getUserId());
-
-		$offsetResults = $this->mapper->findAll(null, 2);
-		$this->assertCount(1, $offsetResults);
-		$this->assertEquals('user2', $offsetResults[0]->getUserId());
-	}
-
-	public function testFindAllRecent(): void {
-		$this->insertSampleStatuses();
-
-		$allResults = $this->mapper->findAllRecent(2, 0);
-		$this->assertCount(2, $allResults);
-		$this->assertEquals('user2', $allResults[0]->getUserId());
-		$this->assertEquals('user1', $allResults[1]->getUserId());
-	}
-
-	public function testGetFind(): void {
-		$this->insertSampleStatuses();
-
-		$adminStatus = $this->mapper->findByUserId('admin');
-		$this->assertEquals('admin', $adminStatus->getUserId());
-		$this->assertEquals('offline', $adminStatus->getStatus());
-		$this->assertEquals(0, $adminStatus->getStatusTimestamp());
-		$this->assertEquals(false, $adminStatus->getIsUserDefined());
-		$this->assertEquals(null, $adminStatus->getCustomIcon());
-		$this->assertEquals(null, $adminStatus->getCustomMessage());
-		$this->assertEquals(null, $adminStatus->getClearAt());
-
-		$user1Status = $this->mapper->findByUserId('user1');
-		$this->assertEquals('user1', $user1Status->getUserId());
-		$this->assertEquals('dnd', $user1Status->getStatus());
-		$this->assertEquals(5000, $user1Status->getStatusTimestamp());
-		$this->assertEquals(true, $user1Status->getIsUserDefined());
-		$this->assertEquals('

apps/user_status/tests/Unit/Connector/UserStatusProviderTest.php

Indentation +47 added lines, -47 removed lines

@@ -15,59 +15,59 @@
 use Test\TestCase;
 
 class UserStatusProviderTest extends TestCase {
-	private StatusService&MockObject $service;
-	private UserStatusProvider $provider;
+    private StatusService&MockObject $service;
+    private UserStatusProvider $provider;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->service = $this->createMock(StatusService::class);
-		$this->provider = new UserStatusProvider($this->service);
-	}
+        $this->service = $this->createMock(StatusService::class);
+        $this->provider = new UserStatusProvider($this->service);
+    }
 
-	public function testGetUserStatuses(): void {
-		$userStatus2 = new UserStatus();
-		$userStatus2->setUserId('userId2');
-		$userStatus2->setStatus('dnd');
-		$userStatus2->setStatusTimestamp(5000);
-		$userStatus2->setIsUserDefined(true);
-		$userStatus2->setCustomIcon('

apps/user_status/tests/Unit/Controller/StatusesControllerTest.php

Indentation +73 added lines, -73 removed lines

@@ -18,77 +18,77 @@
 use Test\TestCase;
 
 class StatusesControllerTest extends TestCase {
-	private StatusService&MockObject $service;
-	private StatusesController $controller;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$request = $this->createMock(IRequest::class);
-		$this->service = $this->createMock(StatusService::class);
-
-		$this->controller = new StatusesController('user_status', $request, $this->service);
-	}
-
-	public function testFindAll(): void {
-		$userStatus = $this->getUserStatus();
-
-		$this->service->expects($this->once())
-			->method('findAll')
-			->with(20, 40)
-			->willReturn([$userStatus]);
-
-		$response = $this->controller->findAll(20, 40);
-		$this->assertEquals([[
-			'userId' => 'john.doe',
-			'status' => 'offline',
-			'icon' => '

apps/user_status/tests/Unit/Controller/UserStatusControllerTest.php

Indentation +289 added lines, -289 removed lines

@@ -27,293 +27,293 @@
 use Throwable;
 
 class UserStatusControllerTest extends TestCase {
-	private LoggerInterface&MockObject $logger;
-	private StatusService&MockObject $statusService;
-	private CalendarStatusService&MockObject $calendarStatusService;
-	private UserStatusController $controller;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$request = $this->createMock(IRequest::class);
-		$userId = 'john.doe';
-		$this->logger = $this->createMock(LoggerInterface::class);
-		$this->statusService = $this->createMock(StatusService::class);
-		$this->calendarStatusService = $this->createMock(CalendarStatusService::class);
-
-		$this->controller = new UserStatusController(
-			'user_status',
-			$request,
-			$userId,
-			$this->logger,
-			$this->statusService,
-			$this->calendarStatusService,
-		);
-	}
-
-	public function testGetStatus(): void {
-		$userStatus = $this->getUserStatus();
-
-		$this->statusService->expects($this->once())
-			->method('findByUserId')
-			->with('john.doe')
-			->willReturn($userStatus);
-
-		$response = $this->controller->getStatus();
-		$this->assertEquals([
-			'userId' => 'john.doe',
-			'status' => 'invisible',
-			'icon' => '

apps/user_status/tests/Unit/Controller/PredefinedStatusControllerTest.php

Indentation +35 added lines, -35 removed lines

@@ -15,39 +15,39 @@
 use Test\TestCase;
 
 class PredefinedStatusControllerTest extends TestCase {
-	private PredefinedStatusService&MockObject $service;
-	private PredefinedStatusController $controller;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$request = $this->createMock(IRequest::class);
-		$this->service = $this->createMock(PredefinedStatusService::class);
-
-		$this->controller = new PredefinedStatusController('user_status', $request, $this->service);
-	}
-
-	public function testFindAll(): void {
-		$this->service->expects($this->once())
-			->method('getDefaultStatuses')
-			->with()
-			->willReturn([
-				[
-					'id' => 'predefined-status-one',
-				],
-				[
-					'id' => 'predefined-status-two',
-				],
-			]);
-
-		$actual = $this->controller->findAll();
-		$this->assertEquals([
-			[
-				'id' => 'predefined-status-one',
-			],
-			[
-				'id' => 'predefined-status-two',
-			],
-		], $actual->getData());
-	}
+    private PredefinedStatusService&MockObject $service;
+    private PredefinedStatusController $controller;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $request = $this->createMock(IRequest::class);
+        $this->service = $this->createMock(PredefinedStatusService::class);
+
+        $this->controller = new PredefinedStatusController('user_status', $request, $this->service);
+    }
+
+    public function testFindAll(): void {
+        $this->service->expects($this->once())
+            ->method('getDefaultStatuses')
+            ->with()
+            ->willReturn([
+                [
+                    'id' => 'predefined-status-one',
+                ],
+                [
+                    'id' => 'predefined-status-two',
+                ],
+            ]);
+
+        $actual = $this->controller->findAll();
+        $this->assertEquals([
+            [
+                'id' => 'predefined-status-one',
+            ],
+            [
+                'id' => 'predefined-status-two',
+            ],
+        ], $actual->getData());
+    }
 }