Merge pull request #283 from danopz/entity-methods

Entity setters + getters

2 files changed, 10 insertions, 2 deletions

diff --git a/tests/Unit/Db/FeedTest.php b/tests/Unit/Db/FeedTest.php
index 961e1d9b7..53bfe63f4 100644
--- a/tests/Unit/Db/FeedTest.php
+++ b/tests/Unit/Db/FeedTest.php
@@ -104,9 +104,11 @@ class FeedTest extends TestCase
 
     public function testSetXSSUrl() 
     {
+        $this->setExpectedException(\TypeError::class);
+
         $feed = new Feed();
         $feed->setUrl('javascript:alert()');
-        $this->assertEquals('', $feed->getUrl());
+        $feed->getUrl();
     }
 
 
diff --git a/tests/Unit/Db/FolderMapperTest.php b/tests/Unit/Db/FolderMapperTest.php
index c8230ff0f..f3ee15456 100644
--- a/tests/Unit/Db/FolderMapperTest.php
+++ b/tests/Unit/Db/FolderMapperTest.php
@@ -19,9 +19,11 @@ use OCA\News\Utility\Time;
 
 class FolderMapperTest extends MapperTestUtility
 {
-
+    /** @var FolderMapper */
     private $folderMapper;
+    /** @var Folder[] */
     private $folders;
+    /** @var string */
     private $user;
 
     protected function setUp()
@@ -32,7 +34,11 @@ class FolderMapperTest extends MapperTestUtility
 
         // create mock folders
         $folder1 = new Folder();
+        $folder1->setId(4);
+        $folder1->resetUpdatedFields();
         $folder2 = new Folder();
+        $folder2->setId(5);
+        $folder2->resetUpdatedFields();
 
         $this->folders = [$folder1, $folder2];
         $this->user = 'hh';