Merge pull request #4241 from nextcloud/tests/testUsersIndex

Author: juliusknorr

lib/Controller/UserApiController.php

@@ -41,7 +41,7 @@ public function index(int $documentId, int $sessionId, string $sessionToken, str
 			$sessionUserId = $session['userId'];
 			if ($sessionUserId !== null && !isset($users[$sessionUserId])) {
 				$displayName = $this->userManager->getDisplayName($sessionUserId);
-				if (stripos($displayName, $filter) !== false || stripos($sessionUserId, $filter) !== false) {
+				if ($displayName && stripos($displayName, $filter) !== false || stripos($sessionUserId, $filter) !== false) {
 					$users[$sessionUserId] = $displayName;
 				}
 			}

tests/unit/Controller/UserApiControllerTest.php

@@ -69,6 +69,9 @@ public function testUsersIndex(int $documentId, int $sessionId, string $sessionT
 				'userId' => 'admin',
 				'displayName' => 'admin',
 			]]);
+		$this->userManager->expects($this->once())
+			->method('getDisplayName')
+			->willReturn('Administrator');
 		$this->sessionService
 			->expects($this->once())
 			->method('getSession')->willReturn($session);