✅ Item: update tests - include sharer display name

Signed-off-by: Marco Nassabain <marco.nassabain@hotmail.com>

4 files changed, 17 insertions, 2 deletions

diff --git a/tests/Unit/Db/ItemMapperAfterTest.php b/tests/Unit/Db/ItemMapperAfterTest.php
index eccebbcf5..6e33465ba 100644
--- a/tests/Unit/Db/ItemMapperAfterTest.php
+++ b/tests/Unit/Db/ItemMapperAfterTest.php
@@ -28,6 +28,8 @@ class ItemMapperAfterTest extends MapperTestUtility
 
     /** @var ItemMapperV2 */
     private $class;
+    /** @var IUserManager */
+    private $userManager;
 
     /**
      * @covers \OCA\News\Db\ItemMapperV2::__construct
@@ -37,6 +39,8 @@ class ItemMapperAfterTest extends MapperTestUtility
         parent::setUp();
         $time = $this->getMockBuilder(Time::class)
                            ->getMock();
+        $this->userManager = $this->getMockBuilder(IUserManager::class)
+                                  ->getMock();
 
         $this->class = new ItemMapperV2($this->db, $time);
     }
diff --git a/tests/Unit/Db/ItemMapperPaginatedTest.php b/tests/Unit/Db/ItemMapperPaginatedTest.php
index 330eaff51..1a53fe616 100644
--- a/tests/Unit/Db/ItemMapperPaginatedTest.php
+++ b/tests/Unit/Db/ItemMapperPaginatedTest.php
@@ -30,6 +30,8 @@ class ItemMapperPaginatedTest extends MapperTestUtility
 
     /** @var ItemMapperV2 */
     private $class;
+    /** @var IUserManager */
+    private $userManager;
 
     /**
      * @covers \OCA\News\Db\ItemMapperV2::__construct
@@ -39,6 +41,8 @@ class ItemMapperPaginatedTest extends MapperTestUtility
         parent::setUp();
         $time = $this->getMockBuilder(Time::class)
                            ->getMock();
+        $this->userManager = $this->getMockBuilder(IUserManager::class)
+                                  ->getMock();
 
         $this->class = new ItemMapperV2($this->db, $time);
     }
diff --git a/tests/Unit/Db/ItemMapperTest.php b/tests/Unit/Db/ItemMapperTest.php
index 5cb42c3fa..a1b9a46ae 100644
--- a/tests/Unit/Db/ItemMapperTest.php
+++ b/tests/Unit/Db/ItemMapperTest.php
@@ -32,6 +32,7 @@ use OCP\DB\QueryBuilder\IFunctionBuilder;
 use OCP\DB\QueryBuilder\IQueryBuilder;
 use OCP\DB\QueryBuilder\IQueryFunction;
 use OCP\IDBConnection;
+use OCP\IUserManager;
 use Test\TestCase;
 
 /**
@@ -46,6 +47,8 @@ class ItemMapperTest extends MapperTestUtility
     private $time;
     /** @var ItemMapperV2 */
     private $class;
+    /** @var IUserManager */
+    private $userManager;
 
     /**
      * @covers \OCA\News\Db\ItemMapperV2::__construct
@@ -55,8 +58,10 @@ class ItemMapperTest extends MapperTestUtility
         parent::setUp();
         $this->time = $this->getMockBuilder(Time::class)
                            ->getMock();
+        $this->userManager = $this->getMockBuilder(IUserManager::class)
+                                  ->getMock();
 
-        $this->class = new ItemMapperV2($this->db, $this->time);
+        $this->class = new ItemMapperV2($this->db, $this->time, $this->userManager);
     }
 
     /**
diff --git a/tests/Unit/Db/ItemTest.php b/tests/Unit/Db/ItemTest.php
index 289126e32..8d556a445 100644
--- a/tests/Unit/Db/ItemTest.php
+++ b/tests/Unit/Db/ItemTest.php
@@ -201,6 +201,7 @@ class ItemTest extends TestCase
         $item->setLastModified(321);
         $item->setCategories(['food']);
         $item->setSharedBy('jack');
+        $item->setSharedByDisplayName('Jack');
 
         $this->assertEquals(
             [
@@ -225,7 +226,8 @@ class ItemTest extends TestCase
             'intro' => 'this is a test',
             'fingerprint' => 'fingerprint',
             'categories' => ['food'],
-            'sharedBy' => 'jack'
+            'sharedBy' => 'jack',
+            'sharedByDisplayName' => 'Jack'
             ], $item->jsonSerialize()
         );
     }