diff options
-rw-r--r-- | lib/Service/ItemService.php | 4 | ||||
-rw-r--r-- | tests/Unit/Service/ItemServiceTest.php | 6 |
2 files changed, 4 insertions, 6 deletions
diff --git a/lib/Service/ItemService.php b/lib/Service/ItemService.php index 882e875c5..58533cb7d 100644 --- a/lib/Service/ItemService.php +++ b/lib/Service/ItemService.php @@ -329,9 +329,9 @@ class ItemService extends Service /** * @param string $userId from which user the items should be taken - * @return int of items which are starred or unread + * @return array of items which are starred or unread */ - public function getUnreadOrStarred($userId) + public function getUnreadOrStarred($userId): array { return $this->oldItemMapper->findAllUnreadOrStarred($userId); } diff --git a/tests/Unit/Service/ItemServiceTest.php b/tests/Unit/Service/ItemServiceTest.php index 89328b6ec..1ab9a7b25 100644 --- a/tests/Unit/Service/ItemServiceTest.php +++ b/tests/Unit/Service/ItemServiceTest.php @@ -506,16 +506,14 @@ class ItemServiceTest extends TestCase public function testGetUnreadOrStarred() { - $star = 18; - $this->oldItemMapper->expects($this->once()) ->method('findAllUnreadOrStarred') ->with($this->equalTo('jack')) - ->will($this->returnValue($star)); + ->will($this->returnValue([])); $result = $this->itemService->getUnreadOrStarred('jack'); - $this->assertEquals($star, $result); + $this->assertEquals([], $result); } |